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

Syncing latest changes from master for rook #779

Merged
merged 16 commits into from
Nov 20, 2024

Conversation

df-build-team
Copy link

PR containing the latest commits from master branch

prazumovsky and others added 16 commits November 12, 2024 21:20
If host path keyring file exists and created during osd-prepare but not
imported to ceph auth then rook-ceph-osd activate init container will
create new keyring in ceph auth by ceph auth get-or-create command. To
avoid keyrings difference in host path and in ceph auth, consider
importing key from host path keyring file if no keyring found in ceph
auth.

Resolves: rook#14825
Signed-off-by: Peter Razumovsky <prazumovsky@mirantis.com>
Remove unnecessary if-else statement inside "if keyring file exists"
condition of osd activate script.

Resolves: rook#14825
Signed-off-by: Peter Razumovsky <prazumovsky@mirantis.com>
Add test to canary-integration-test scenarios which removes
osd auth and then recovers it on osd activation from keyring
file.

Resolves: rook#14825
Signed-off-by: Peter Razumovsky <prazumovsky@mirantis.com>
update the incstruction for where to run python script

Signed-off-by: parth-gr <partharora1010@gmail.com>
osd: import keyring file on activate to ceph auth if not imported yet
run the tools pod with the rook-ceph-default service account

setting the service account for the pod prevents SCC violations on openshift

Signed-off-by: amrut-asm <amrutrn1@gmail.com>
Syncing latest changes from upstream master for rook
Bumps [dcarbone/install-jq-action](https://github.com/dcarbone/install-jq-action) from 2.1.0 to 3.0.1.
- [Release notes](https://github.com/dcarbone/install-jq-action/releases)
- [Commits](dcarbone/install-jq-action@8867ddb...e397bd8)

---
updated-dependencies:
- dependency-name: dcarbone/install-jq-action
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Bumps [DavidAnson/markdownlint-cli2-action](https://github.com/davidanson/markdownlint-cli2-action) from 17.0.0 to 18.0.0.
- [Release notes](https://github.com/davidanson/markdownlint-cli2-action/releases)
- [Commits](DavidAnson/markdownlint-cli2-action@db43aef...eb5ca3a)

---
updated-dependencies:
- dependency-name: DavidAnson/markdownlint-cli2-action
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
helm: set service account for toolbox pod
…rbone/install-jq-action-3.0.1

build(deps): bump dcarbone/install-jq-action from 2.1.0 to 3.0.1
…idAnson/markdownlint-cli2-action-18.0.0

build(deps): bump DavidAnson/markdownlint-cli2-action from 17.0.0 to 18.0.0
When node topology labels with duplicate values are detected, rook silently discards all but one label.
This adds a warning informing administrators of this, allowing for easier identification of labeling issues.

Signed-off-by: Fred Heinecke <fred.heinecke@yahoo.com>
rook: add warning when duplicate node topology values are detected
Syncing latest changes from upstream master for rook
@df-build-team df-build-team requested a review from a team November 20, 2024 09:49
@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Nov 20, 2024
@subhamkrai subhamkrai added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Nov 20, 2024
Copy link

openshift-ci bot commented Nov 20, 2024

[APPROVALNOTIFIER] This PR is APPROVED

Approval requirements bypassed by manually added approval.

This pull-request has been approved by: df-build-team, subhamkrai

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-merge-bot openshift-merge-bot bot merged commit 20473b3 into release-4.18 Nov 20, 2024
141 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants