Project

General

Profile

Actions

Feature #43937

closed

cephadm: make default image configurable

Added by Sebastian Wagner about 4 years ago. Updated about 4 years ago.

Status:
Rejected
Priority:
Normal
Assignee:
Ricardo Marques
Category:
cephadm (binary)
Target version:
-
% Done:

0%

Source:
Tags:
Backport:
Reviewed:
Affected Versions:
Pull request ID:

Description

This is mostly a thing for pre-configured distributions.

In case the default upstream image is not desired, because there is a supported or inhouse alternative, it should be possible to change the default cephadm container image to a different image.

E.g. a user runs

cephadm shell

and suddenly the default image from upstream is used to spawn the container.


Related issues 1 (0 open1 closed)

Related to Orchestrator - Bug #44440: cephadm should be able to infer running containerResolvedRicardo Marques

Actions
Actions #1

Updated by Sebastian Wagner about 4 years ago

  • Priority changed from Normal to Low

prio low, as this can be done by setting the env variable system-wide

Actions #2

Updated by Nathan Cutler about 4 years ago

  • Related to Bug #44440: cephadm should be able to infer running container added
Actions #3

Updated by Nathan Cutler about 4 years ago

  • Priority changed from Low to Normal

Sebastian Wagner wrote:

prio low, as this can be done by setting the env variable system-wide

How does one set an env variable system-wide?

Actions #4

Updated by Ricardo Marques about 4 years ago

I would prefer to be able to set a MON config-key with the default value.

If env variable is defined, it should override the MON config-key.

Actions #5

Updated by Ricardo Marques about 4 years ago

  • Assignee set to Ricardo Marques
Actions #6

Updated by Ricardo Marques about 4 years ago

  • Related to deleted (Bug #44440: cephadm should be able to infer running container)
Actions #7

Updated by Ricardo Marques about 4 years ago

  • Has duplicate Bug #44440: cephadm should be able to infer running container added
Actions #8

Updated by Ricardo Marques about 4 years ago

  • Status changed from New to In Progress
Actions #9

Updated by Ricardo Marques about 4 years ago

I've learned that `cephadm bootstrap` is already storing the pulled image path in a config-key:

# ceph config-key exists 'config/global/container_image'
key 'config/global/container_image' exists
Actions #10

Updated by Ricardo Marques about 4 years ago

  • Status changed from In Progress to Fix Under Review
  • Pull request ID set to 33781
Actions #11

Updated by Ricardo Marques about 4 years ago

  • Has duplicate deleted (Bug #44440: cephadm should be able to infer running container)
Actions #12

Updated by Ricardo Marques about 4 years ago

  • Related to Bug #44440: cephadm should be able to infer running container added
Actions #13

Updated by Ricardo Marques about 4 years ago

  • Status changed from Fix Under Review to Rejected
Actions

Also available in: Atom PDF