Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

cephadm-boostrap: add new boostrap parameters #276

Merged
merged 1 commit into from
Feb 15, 2024

Conversation

asm0deuz
Copy link
Collaborator

Added a list of boostrap parameters related to IBM call home and HW monitoring feature only available downstream

Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=2246266

library/cephadm_bootstrap.py Outdated Show resolved Hide resolved
Comment on lines +305 to +315
# Build cephadm with parameters
cmd = extend_append('cephadm', cephadm_params)
# Extends with boostrap parameters
cmd.extend(extend_append('bootstrap', cephadm_bootstrap_params))
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice

library/cephadm_bootstrap.py Outdated Show resolved Hide resolved
Copy link
Collaborator

@guits guits left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I usually prefer actually addressing flake8 errors when possible rather than ignoring them.

Otherwise, that looks good to me

Added a list of boostrap parameters related to IBM call home
and HW monitoring feature only available downstream

Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=2246266

Signed-off-by: Teoman ONAY <tonay@ibm.com>
@asm0deuz asm0deuz force-pushed the bootstrap_downstream_params branch from fdd5f89 to faa7b2f Compare February 15, 2024 13:47
@asm0deuz asm0deuz added backport-quincy backport quincy backport-reef backport reef labels Feb 15, 2024
@guits guits merged commit 018cb0f into ceph:devel Feb 15, 2024
9 checks passed
asm0deuz added a commit that referenced this pull request Feb 15, 2024
cephadm-boostrap: add new boostrap parameters (backport #276)
asm0deuz added a commit that referenced this pull request Feb 15, 2024
cephadm-boostrap: add new boostrap parameters (backport #276)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-quincy backport quincy backport-reef backport reef
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants