Syncing latest changes from master for rook #812
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Currently, the sc name is hardcoded, in future if we support multiple rados namespace per tenant we need a variable name updating sc names to contain rados-namespace name ocs pr: red-hat-storage/ocs-operator#2955 Signed-off-by: parth-gr <[email protected]>
external: fix rados namespace sc name
This change continues an effort started earlier to make some make targets and ci workflows more consistent and systematic. see rook#14922 it adds a make target gen.helm-docs as an alias to helm-docs. Additionally, gen.docs is added as alias to docs. targets check.docs and check.helm-docs are removed because it was agreed that targets using git are of little value to developers. Their functionality is moved back into the corresponding docs workflow. xiFinally, the redundant "Check helm-docs" check is removed from the docs-check workflow Signed-off-by: Michael Adam <[email protected]>
ci : rework docs-related workflows and Makefile targets a bit to be more systematic and consistent
Bumps the k8s-dependencies group with 1 update: [sigs.k8s.io/controller-runtime](https://github.com/kubernetes-sigs/controller-runtime). Updates `sigs.k8s.io/controller-runtime` from 0.19.3 to 0.19.4 - [Release notes](https://github.com/kubernetes-sigs/controller-runtime/releases) - [Changelog](https://github.com/kubernetes-sigs/controller-runtime/blob/main/RELEASE.md) - [Commits](kubernetes-sigs/controller-runtime@v0.19.3...v0.19.4) --- updated-dependencies: - dependency-name: sigs.k8s.io/controller-runtime dependency-type: direct:production update-type: version-update:semver-patch dependency-group: k8s-dependencies ... Signed-off-by: dependabot[bot] <[email protected]>
Signed-off-by: Joshua Hoblitt <[email protected]>
The operator namespace is where the csi driver resources are being created. The prefix is expected to match the namespace where the driver is being created. It was an incorrect change from 15245 where it had been assumed that the cluster prefix name was required instead. Signed-off-by: Travis Nielsen <[email protected]>
Signed-off-by: licheng <[email protected]>
doc: add adopter
csi: Set driver name to same as the operator namespace
…-docs doc: add basic obc bucketPolicy description
For the upcoming 4.17 release, we reset the pending release notes so we can add new features to the list. Signed-off-by: Travis Nielsen <[email protected]>
Signed-off-by: Artem Torubarov <[email protected]>
docs: Reset pending release notes
For the upcoming 1.17 release, update the high level features that are targeted. Signed-off-by: Travis Nielsen <[email protected]>
The Kubernetes CSI sidecars have had several releases that were not included in deployments by Rook yet, update them to the versions that are available today: - csi-node-driver-registrar:v2.13.0 - csi-provisioner:v5.1.0 - csi-attacher:v4.8.0 - csi-resizer:v1.13.1 Signed-off-by: Niels de Vos <[email protected]>
…0250115 Update Kubernetes CSI sidecar images to current versions
in the image building Makefile, a comment explaining the construction of the operator-sdk download URL was in a wrong place. Now it is moved to the lines that it actually explains. Signed-off-by: Michael Adam <[email protected]>
In image building, when building on arm based systems, the s5cmd tool was added with the wrong architecture (amd64 instead of arm64). This change fixes the architecture of the s5cmd to match the architecture of the image built. Fixes: rook#15262 Signed-off-by: Michael Adam <[email protected]>
docs: Update roadmap for release 1.17
Fix s5cmd arch
No need for adopters to send mail to the steering committee, maintainers can manage the adopters just fine. Signed-off-by: Travis Nielsen <[email protected]>
docs: Simplify adopters instructions
…endencies-1fd157afd7 build(deps): bump sigs.k8s.io/controller-runtime from 0.19.3 to 0.19.4 in the k8s-dependencies group
This PR addresses an issue where the CrashCollector Pruner CronJob pods were stuck in a Pending state due to missing tolerations in their PodTemplateSpec. These tolerations were not being propagated from the cephClusterSpec.placement.all field. Added tolerations to the PodTemplateSpec of the CrashCollector Pruner CronJob. Signed-off-by: Oded Viner <[email protected]>
Co-authored-by: Travis Nielsen <[email protected]> Signed-off-by: Artem Torubarov <[email protected]>
core: add tolerations to crashcollector pruner cronJob pod
operator: omit values from mon db config logs
The rgw multisite testing job is consistently failing and blocking the backport of PRs that are passing all other CI. Until issue 15177 is fixed let's disable this check. Signed-off-by: Travis Nielsen <[email protected]>
build: Auto merge backports without blocking on multisite test
log applied mon store config values with trace lvl for debug purpose Signed-off-by: Artem Torubarov <[email protected]>
operator: log mon db config values with trace lvl
Signed-off-by: Artem Torubarov <[email protected]>
operator: improve operator error logging for ok-to-stop failures
The lint make target is not used anywhere in the codebase and is particular not run in the CI. It was recently discovered that `make lint` does not even work at all. It was subsequently suggested to remove the broken and unused target altogether. This change settles that issue by removing the lint target and the preparatory golint installation target. Fixes: rook#15317 Signed-off-by: Michael Adam <[email protected]>
build: remove broken make target lint
This is to allow developers to conveniently run the same check locally as run in the CI. Signed-off-by: Michael Adam <[email protected]>
Ci: refactor a shellcheck make target out of the github ci workflow
Add Thalassa Cloud to the list of adopter of Rook Ceph Signed-off-by: Thomas Kooi <[email protected]>
Bumps the k8s-dependencies group with 1 update: [sigs.k8s.io/controller-runtime](https://github.com/kubernetes-sigs/controller-runtime). Updates `sigs.k8s.io/controller-runtime` from 0.20.0 to 0.20.1 - [Release notes](https://github.com/kubernetes-sigs/controller-runtime/releases) - [Changelog](https://github.com/kubernetes-sigs/controller-runtime/blob/main/RELEASE.md) - [Commits](kubernetes-sigs/controller-runtime@v0.20.0...v0.20.1) --- updated-dependencies: - dependency-name: sigs.k8s.io/controller-runtime dependency-type: direct:production update-type: version-update:semver-patch dependency-group: k8s-dependencies ... Signed-off-by: dependabot[bot] <[email protected]>
Bumps [docker/setup-qemu-action](https://github.com/docker/setup-qemu-action) from 3.2.0 to 3.3.0. - [Release notes](https://github.com/docker/setup-qemu-action/releases) - [Commits](docker/setup-qemu-action@49b3bc8...53851d1) --- updated-dependencies: - dependency-name: docker/setup-qemu-action dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]>
Bumps [actions/stale](https://github.com/actions/stale) from 9.0.0 to 9.1.0. - [Release notes](https://github.com/actions/stale/releases) - [Changelog](https://github.com/actions/stale/blob/main/CHANGELOG.md) - [Commits](actions/stale@28ca103...5bef64f) --- updated-dependencies: - dependency-name: actions/stale dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]>
Bumps [actions/setup-go](https://github.com/actions/setup-go) from 5.2.0 to 5.3.0. - [Release notes](https://github.com/actions/setup-go/releases) - [Commits](actions/setup-go@3041bf5...f111f33) --- updated-dependencies: - dependency-name: actions/setup-go dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]>
Bumps [github/codeql-action](https://github.com/github/codeql-action) from 3.27.9 to 3.28.5. - [Release notes](https://github.com/github/codeql-action/releases) - [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md) - [Commits](github/codeql-action@df409f7...f6091c0) --- updated-dependencies: - dependency-name: github/codeql-action dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]>
Bumps [DavidAnson/markdownlint-cli2-action](https://github.com/davidanson/markdownlint-cli2-action) from 18.0.0 to 19.1.0. - [Release notes](https://github.com/davidanson/markdownlint-cli2-action/releases) - [Commits](DavidAnson/markdownlint-cli2-action@eb5ca3a...05f3221) --- 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]>
…endencies-9d3459377c build(deps): bump sigs.k8s.io/controller-runtime from 0.20.0 to 0.20.1 in the k8s-dependencies group
…ker/setup-qemu-action-3.3.0 build(deps): bump docker/setup-qemu-action from 3.2.0 to 3.3.0
…ions/stale-9.1.0 build(deps): bump actions/stale from 9.0.0 to 9.1.0
…ions/setup-go-5.3.0 build(deps): bump actions/setup-go from 5.2.0 to 5.3.0
…hub/codeql-action-3.28.5 build(deps): bump github/codeql-action from 3.27.9 to 3.28.5
…idAnson/markdownlint-cli2-action-19.1.0 build(deps): bump DavidAnson/markdownlint-cli2-action from 18.0.0 to 19.1.0
docs: add Thalassa Cloud to Adopters
The newly added make target 'lint' runs various linters on the code base. Signed-off-by: Michael Adam <[email protected]>
…abel Hostname topology label
Build: add make lint
openshift-merge-robot
added
the
needs-rebase
|
PR needs rebase. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: df-build-team 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 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
PR containing the latest commits from master branch