Skip to content

add controller modify secret to rbd sc#4018

Open
Madhu-1 wants to merge 1 commit into
red-hat-storage:mainfrom
Madhu-1:add-controller-modify-secret
Open

add controller modify secret to rbd sc#4018
Madhu-1 wants to merge 1 commit into
red-hat-storage:mainfrom
Madhu-1:add-controller-modify-secret

Conversation

@Madhu-1

@Madhu-1 Madhu-1 commented Jul 8, 2026

Copy link
Copy Markdown
Member

Adding controller modify secret to rbd sc which is required to modify the QoS or add new QoS for existing pvc.

Adding controller modify secret to
rbd sc which is required to modify the
QoS or add new QoS for existing pvc.

Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
@Madhu-1

Madhu-1 commented Jul 8, 2026

Copy link
Copy Markdown
Member Author

/assign @leelavg

@openshift-ci openshift-ci Bot added the lgtm Indicates that a PR is ready to be merged. label Jul 8, 2026
@openshift-ci

openshift-ci Bot commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: leelavg, Madhu-1

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

The pull request process is described here

Details 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-ci openshift-ci Bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jul 8, 2026
@Madhu-1

Madhu-1 commented Jul 8, 2026

Copy link
Copy Markdown
Member Author

/hold

@openshift-ci openshift-ci Bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Jul 8, 2026
@Madhu-1

Madhu-1 commented Jul 8, 2026

Copy link
Copy Markdown
Member Author

@leelavg am keeping this one hold, this can cause problem with provider/consumer where consumer is on OCP N-4, the sidecar will be very old and might cause problem due to these new parameters. we need to comeup with a plan to create SC with only know parameters specific to the sidecar, This is not really related to this feature but it can happen when new secrets are added to the SC.

@openshift-ci

openshift-ci Bot commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

@Madhu-1: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/ocs-operator-bundle-e2e-aws da95daf link true /test ocs-operator-bundle-e2e-aws

Full PR test history. Your PR dashboard.

Details

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. I understand the commands that are listed here.

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. do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants