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 #604

Merged
merged 128 commits into from
Mar 27, 2024
Merged

Conversation

df-build-team
Copy link

PR containing the latest commits from master branch

sp98 and others added 30 commits February 21, 2024 11:04
Users on Microsoft Azure can make use of the Azure key
vault service rather than replying on any third party
service for KMS.

Signed-off-by: sp98 <[email protected]>
Syncing latest changes from upstream master for rook
When a private docker registry is used and an
image pull secret is specified in the chart,
the pods with default Service Account fail to pull
the image due to authentication issues.
Added rook-ceph-default service account and modify the pods
specifications by adding the serviceAccountName

closes: rook#12786

Closes: rook#6673
Co-authored-by: Tareq Sharafy <[email protected]>
Signed-off-by: parth-gr <[email protected]>
(cherry picked from commit 737fb09)
Signed-off-by: parth-gr <[email protected]>
…aseNotes

doc: PendingReleaseNotes for update netNamespaceFilePath PR
core: Set default service account on all Ceph daemons
Syncing latest changes from upstream master for rook
Fixing the broken links in the docs.

Signed-off-by: karthik-us <[email protected]>
This reverts commit ea700bc.

Signed-off-by: karthik-us <[email protected]>
This commit gives builders the necessary tooling to save off a docker build context for use
with other tools that dont follow the same command format as $DOCKERCMD.

Signed-off-by: Scott Miller <[email protected]>
Added required rbac's for required rbac
for volumegroupsnapshot feature.

Signed-off-by: Madhu Rajanna <[email protected]>
update the snapshot controller to 7.0.1
and install new Volumegroup CRD's

Signed-off-by: Madhu Rajanna <[email protected]>
This patch fixes the broken links in the non rendered documents
on github and keeps the structre of the rendered documents on
the website intact as well.

Signed-off-by: karthik-us <[email protected]>
volumegroupsnapshot feature will be enabled
by default if the required CRD's are present
if not its disabled and user will have an option
to disable it if they dont require this feature.

Signed-off-by: Madhu Rajanna <[email protected]>
r is the variable name for the CSI reconciler
and same is used for a local variable as well
chaning it to avoid confusion and variable
shadowing.

Signed-off-by: Madhu Rajanna <[email protected]>
updating all the csi sidecars to the
latest release.

Signed-off-by: Madhu Rajanna <[email protected]>
build: add ability to stash docker build context
During cluster creation, csi config map was first filled with
mon ips and without CSIDriverOptions.
This commit makes sure CSIDriverOptions are added at the begining
when the entry is first created.

Signed-off-by: Rakshith R <[email protected]>
currently  the script requires to have both v2 and v1 port
to enable v2 port, but that is not the necessary condition,
so removing the chek, and enabling it only v2 is present to
successfully configure with v2 only

part-of: rook#13827

Signed-off-by: parth-gr <[email protected]>
Bumps [azure/setup-helm](https://github.com/azure/setup-helm) from 3 to 4.
- [Release notes](https://github.com/azure/setup-helm/releases)
- [Changelog](https://github.com/Azure/setup-helm/blob/main/CHANGELOG.md)
- [Commits](Azure/setup-helm@v3...v4)

---
updated-dependencies:
- dependency-name: azure/setup-helm
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
csi: update CSIDriverOption params during saving cluster config
…re/setup-helm-4

build(deps): bump azure/setup-helm from 3 to 4
The rook/ceph Dockerfile uses dnf to ensure iproute (containing the 'ip'
CLI tool) is installed in the Rook image for Multus usage. This comes
from the 'baseos' repo, but if any other repos are unavailable
temporarily, it can cause the container build to fail.

Use the '--repo baseos' flag to help prevent these kinds of failures.
Additionally, this will speed up the build slightly since it does not
attempt to load any non-necessary repos.

This change may make the container build slightly fragile in the future
if CentOS changes the name of its baseos repo, or if the Ceph image
switches to a non-CentOS base image.

Signed-off-by: Blaine Gardner <[email protected]>
external: enable the use of only v2 mon port
build: use 'baseos' as repo for iproute install
doc: add support for using azure kms
sp98 and others added 19 commits March 20, 2024 18:38
Added new rook bundle creation script
Update csiadons/k8s-sidecar image to 0.8.0 in csv

Signed-off-by: Nikhil-Ladha <[email protected]>
Added the new stanza to `deploy/examples/operator-openshift.yaml` and
ran `make gen-csv`

We are going to update the configmap from ocs-operator to disable the
CSI during runtime and these values are required for rook to realize the
update.

Signed-off-by: Leela Venkaiah G <[email protected]>
Update script to export ROOK_IMAGE env variable so that
it can be updated with custom values during the build

Signed-off-by: Nikhil-Ladha <[email protected]>
add generated changes for csv

Signed-off-by: Nikhil-Ladha <[email protected]>
build: add new env value ref for operator
add all the csi related images version in csv which
ocs-operator csv merger tool was changing.

Signed-off-by: subhamkrai <[email protected]>
these are auto generated csv changes when run
`make gen-csv`.

Signed-off-by: subhamkrai <[email protected]>
…-master

This commit undo's all the csv removal changes that were done in upstream

Signed-off-by: Nikhil-Ladha <[email protected]>
added a new flag ROOK_CSI_DISABLE_DRIVER
to disable csi controller.

Signed-off-by: parth-gr <[email protected]>
(cherry picked from commit a72e029)
Signed-off-by: Leela Venkaiah G <[email protected]>
(cherry picked from commit 1bb9cce)
Signed-off-by: parth-gr <[email protected]>
csi: add a new flag to disable csi driver
Syncing latest changes from upstream master for rook
Moved provider field to metadata-common.yaml file,
such that it overrides the default in the rook csv

Signed-off-by: Nikhil-Ladha <[email protected]>
add generated changes for csv

Signed-off-by: Nikhil-Ladha <[email protected]>
@df-build-team df-build-team requested a review from a team March 27, 2024 08:26
Copy link

openshift-ci bot commented Mar 27, 2024

@df-build-team: No Bugzilla bug is referenced in the title of this pull request.
To reference a bug, add 'Bug XXX:' to the title of this pull request and request another bug refresh with /bugzilla refresh.

In response to this:

Syncing latest changes from master for rook

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/test-infra repository.

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Mar 27, 2024
Copy link

openshift-ci bot commented Mar 27, 2024

[APPROVALNOTIFIER] This PR is NOT APPROVED

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

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

@df-build-team df-build-team requested a review from a team March 27, 2024 12:35
@openshift-ci openshift-ci bot removed the lgtm Indicates that a PR is ready to be merged. label Mar 27, 2024
Copy link

openshift-ci bot commented Mar 27, 2024

New changes are detected. LGTM label has been removed.

@travisn travisn merged commit 153bc8b into release-4.16 Mar 27, 2024
111 of 112 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.