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 upstream master for rook #778

Merged
merged 8 commits into from
Nov 19, 2024
Merged

Conversation

df-build-team
Copy link

PR containing the latest commits from upstream master branch

amrut-asm and others added 8 commits November 18, 2024 12:45
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 <[email protected]>
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] <[email protected]>
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] <[email protected]>
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 <[email protected]>
rook: add warning when duplicate node topology values are detected
@df-build-team df-build-team requested a review from a team November 19, 2024 08:06
@travisn
Copy link

travisn commented Nov 19, 2024

/approve
/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Nov 19, 2024
@travisn travisn merged commit 5b176fc into master Nov 19, 2024
52 of 53 checks passed
Copy link

openshift-ci bot commented Nov 19, 2024

[APPROVALNOTIFIER] This PR is NOT APPROVED

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

The full list of commands accepted by this bot can be found 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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm Indicates that a PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants