Project

General

Profile

Feature #43937

cephadm: make default image configurable

Added by Sebastian Wagner 10 months ago. Updated 9 months ago.

Status:
Rejected
Priority:
Normal
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

Related to Orchestrator - Bug #44440: cephadm should be able to infer running container Resolved

History

#1 Updated by Sebastian Wagner 10 months ago

  • Priority changed from Normal to Low

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

#2 Updated by Nathan Cutler 9 months ago

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

#3 Updated by Nathan Cutler 9 months 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?

#4 Updated by Ricardo Marques 9 months 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.

#5 Updated by Ricardo Marques 9 months ago

  • Assignee set to Ricardo Marques

#6 Updated by Ricardo Marques 9 months ago

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

#7 Updated by Ricardo Marques 9 months ago

  • Duplicated by Bug #44440: cephadm should be able to infer running container added

#8 Updated by Ricardo Marques 9 months ago

  • Status changed from New to In Progress

#9 Updated by Ricardo Marques 9 months 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

#10 Updated by Ricardo Marques 9 months ago

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

#11 Updated by Ricardo Marques 9 months ago

  • Duplicated by deleted (Bug #44440: cephadm should be able to infer running container)

#12 Updated by Ricardo Marques 9 months ago

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

#13 Updated by Ricardo Marques 9 months ago

  • Status changed from Fix Under Review to Rejected

Also available in: Atom PDF