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

helm: add option to scale down rook operator #12048

Merged
merged 1 commit into from
Apr 10, 2023
Merged

Conversation

TomHellier
Copy link
Contributor

Description of your changes:
Some administrative actions require the user to scale down the rook operator to zero replicas to stop any potential conflicts where the operator is attempting to do something contrary to the current goals of the administrator.

Which issue is resolved by this Pull Request:
Addresses #11927

Checklist:

  • Commit Message Formatting: Commit titles and messages follow guidelines in the developer guide).
  • Skip Tests for Docs: If this is only a documentation change, add the label skip-ci on the PR.
  • Reviewed the developer guide on Submitting a Pull Request
  • Pending release notes updated with breaking and/or notable changes for the next minor release.
  • Documentation has been updated, if necessary.
  • Unit tests have been added, if necessary.
  • Integration tests have been added, if necessary.

@TomHellier
Copy link
Contributor Author

TomHellier commented Apr 7, 2023

Please backport this to 1.11

@@ -567,6 +567,11 @@ cephCommandsTimeoutSeconds: "15"
# -- if true, run rook operator on the host network
useOperatorHostNetwork:

# -- if true, scale down the rook operator
Copy link
Member

Choose a reason for hiding this comment

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

nit: The generated doc doesn't read smoothly without a new sentence.

Suggested change
# -- if true, scale down the rook operator
# -- If true, scale down the rook operator.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks @travisn - I've applied this fix

Some administrative actions require the user to scale down the rook operator to zero replicas to stop any potential conflicts where the operator is attempting to do something contrary to the current goals of the administrator.

closes: rook#11927
Signed-off-by: TomHellier <me@tomhellier.com>
@travisn travisn merged commit a02b828 into rook:master Apr 10, 2023
45 of 50 checks passed
travisn added a commit that referenced this pull request Apr 11, 2023
helm: add option to scale down rook operator (backport #12048)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants