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

Bug 2248450: Fusion HCI integration changes for OCS client operator [4.14 Backport] #41

Merged
merged 15 commits into from
Nov 21, 2023

Conversation

nb-ohad
Copy link
Contributor

@nb-ohad nb-ohad commented Nov 20, 2023

This PR contains backports of multiple fixes and enhancements to allow integration of the OCS Client Operator with Fusion HCI deployment and management model

leelavg and others added 15 commits October 6, 2023 08:10
update golang and ocs-operator dependency version
preserve the resourceVersion of the consolePlugin
which is required for consolePlugin update.

Signed-off-by: Madhu Rajanna <[email protected]>
added required linter comment
to skip go-lint ci checks.

Signed-off-by: Madhu Rajanna <[email protected]>
preserve the resourceVersion of the consolePlugin
- send storage client operator version and version of platform to which
cluster last updated to successfully
- the status will be sent on top of existing heartbeat mechanism

Signed-off-by: Leela Venkaiah G <[email protected]>
controllers: send platform and operator version while reporting status
- ClusterVersion and ClusterServiceVersion APIs addition to scheme is
missing in status reporter job

Signed-off-by: Leela Venkaiah G <[email protected]>
- Status reporter role and role binding is missing in generated CSV bundle

Signed-off-by: Leela Venkaiah G <[email protected]>
Bug 2247737: add APIs to scheme, role and role binding to bundle
added the capability to keep resources on failed job execution with a timeout

Signed-off-by: Amit Berner <[email protected]>
Bug 2247748: A storage-client CronJob create too many jobs and pods causing maxPod limit to be reached
when PVC is scheduled on the master node CSI pod fails to run

Signed-off-by: rchikatw <[email protected]>
Bug 2247735: Adding toleration to schedule CSI pods on master nodes
Copy link

openshift-ci bot commented Nov 20, 2023

@nb-ohad: 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:

Backport of Fusion HCI Phase 1 integration work to 4.14

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.

Copy link

openshift-ci bot commented Nov 20, 2023

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: nb-ohad

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

@nb-ohad nb-ohad changed the title Backport of Fusion HCI Phase 1 integration work to 4.14 Bug 2248450: Fusion HCI integration changes for OCS client operator [4.14 Backport] Nov 20, 2023
Copy link

openshift-ci bot commented Nov 20, 2023

@nb-ohad: This pull request references Bugzilla bug 2248450, which is valid. The bug has been moved to the POST state. The bug has been updated to refer to the pull request using the external bug tracker.

3 validation(s) were run on this bug
  • bug is open, matching expected state (open)
  • bug target release (ODF 4.14.1) matches configured target release for branch (ODF 4.14.1)
  • bug is in the state NEW, which is one of the valid states (NEW, ASSIGNED, ON_DEV, POST, POST)

Requesting review from QA contact:
/cc @nehaberry

In response to this:

Bug 2248450: Fusion HCI integration changes for OCS client operator [4.14 Backport]

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.

Copy link

openshift-ci bot commented Nov 20, 2023

@openshift-ci[bot]: GitHub didn't allow me to request PR reviews from the following users: nehaberry.

Note that only red-hat-storage members and repo collaborators can review this PR, and authors cannot review their own PRs.

In response to this:

@nb-ohad: This pull request references Bugzilla bug 2248450, which is valid. The bug has been moved to the POST state. The bug has been updated to refer to the pull request using the external bug tracker.

3 validation(s) were run on this bug
  • bug is open, matching expected state (open)
  • bug target release (ODF 4.14.1) matches configured target release for branch (ODF 4.14.1)
  • bug is in the state NEW, which is one of the valid states (NEW, ASSIGNED, ON_DEV, POST, POST)

Requesting review from QA contact:
/cc @nehaberry

In response to this:

Bug 2248450: Fusion HCI integration changes for OCS client operator [4.14 Backport]

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.

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.

@agarwal-mudit agarwal-mudit merged commit c0935a9 into release-4.14 Nov 21, 2023
26 checks passed
Copy link

openshift-ci bot commented Nov 21, 2023

@nb-ohad: All pull requests linked via external trackers have merged:

Bugzilla bug 2248450 has been moved to the MODIFIED state.

In response to this:

Bug 2248450: Fusion HCI integration changes for OCS client operator [4.14 Backport]

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants