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

Resync from upstream release-1.12 to downstream release-4.14 #509

Merged
merged 17 commits into from
Aug 16, 2023

Conversation

travisn
Copy link

@travisn travisn commented Aug 16, 2023

Description of your changes:
Resync from upstream release-1.12 to downstream release-4.14

This will be the final sync for 4.14.

Checklist:

  • Commit Message Formatting: Commit titles and messages follow guidelines in the developer guide).
  • Skip Tests for Docs: If this is only a documentation change, add the label skip-ci on the PR.
  • Reviewed the developer guide on Submitting a Pull Request
  • Pending release notes updated with breaking and/or notable changes for the next minor release.
  • Documentation has been updated, if necessary.
  • Unit tests have been added, if necessary.
  • Integration tests have been added, if necessary.

Madhu-1 and others added 17 commits August 15, 2023 17:23
Ignore the exit code 22 when adding the
datapool to a filesystem.

updates: #12673

Signed-off-by: Madhu Rajanna <[email protected]>
(cherry picked from commit bd9b42d)
file: ignore exit code 22 when addding datapool (backport #12725)
For the patch release the examples and the docs are updated
to v1.12.2.

Signed-off-by: travisn <[email protected]>
build: Update patch version to v1.12.2
The build and release actions will rarely ever need to be
debugged. To make them more efficient and reduce their
runtime, we remove the tmate action so it won't add 10
unnecessary minutes to the end of the action.

Signed-off-by: travisn <[email protected]>
(cherry picked from commit 90a95ce)
ci: Disable tmate for build actions (backport #12643)
moving the toolbox-operator creation early in
ci, so that by the time we wait for others pods
to come up, toolbox-operator will be ready or
will wait for 180s.

Signed-off-by: subhamkrai <[email protected]>
(cherry picked from commit bffa766)
The sccs were missing in the helm installation,
added it

Closes: #12280

Signed-off-by: parth-gr <[email protected]>
(cherry picked from commit 62e3ffb)
earlier, we're always logging when `GetValue` method is
called to get operator config  values which somettime
spamming the logs for example `ROOK_WATCH_FOR_NODE_FAILURE`
keep logging every time 5 min.

Now, we'll store the settings in the map and log only when
map key value is changed.

Signed-off-by: subhamkrai <[email protected]>
(cherry picked from commit 733be02)
update vault and other indirect imports. Also,
updating k8s package to `v0.28.0-rc.0` since with
v0.27.4 it was through build error.

Signed-off-by: subhamkrai <[email protected]>
(cherry picked from commit ab37637)
golangci linter was throughing error `k8s.io/utils/pointer`
package is deprecated. So, I have removed that.

Signed-off-by: subhamkrai <[email protected]>
(cherry picked from commit a25071a)
helm: install rook in openshift cluster (backport #12675)
ci: create toolbox-operator early in ci (backport #12736)
build: update vault and other indirect imports (backport #12672)
operator: log op. config cm setting when changed (backport #12679)
As part of a normal restart, an OSD can go down just as prometheus tries
to scrape it, causing this alert to fire. Alerts shouldn't fire as part
of normal operations. This change requires the OSD to be down for 5
consecutive minutes before the alert will fire.

Signed-off-by: Chris Jones <[email protected]>
(cherry picked from commit 9f1e5ec)
monitoring: add "for" to CephOSDDownHigh alert (backport #12731)
@openshift-ci
Copy link

openshift-ci bot commented Aug 16, 2023

@travisn: 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:

Resync from upstream release-1.12 to downstream release-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.

@openshift-ci
Copy link

openshift-ci bot commented Aug 16, 2023

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: 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

@travisn travisn merged commit d91e8e4 into red-hat-storage:release-4.14 Aug 16, 2023
43 of 45 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.

5 participants