-
Notifications
You must be signed in to change notification settings - Fork 16
OCPBUGS-57800: Rebase release-4.18 onto release-1.31 #322
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
OCPBUGS-57800: Rebase release-4.18 onto release-1.31 #322
Conversation
Co-authored-by: pýrus <[email protected]>
…netes#2882) Co-authored-by: pýrus <[email protected]>
|
Hi @shiftstack-merge-bot[bot]. Thanks for your PR. I'm waiting for a openshift member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. DetailsInstructions 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. |
|
/ok-to-test |
|
Let's merge this to get it off our queue. /approve (I don't think I can do the last one. @mandre to the rescue if not) |
|
@stephenfin: Can not set label backport-risk-assessed: Must be member in one of these teams: [openshift-staff-engineers] DetailsIn response to this:
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. |
|
/label backport-risk-assessed |
|
/retitle OCPBUGS-57800: Rebase release-4.18 onto release-1.31 |
|
@shiftstack-merge-bot[bot]: This pull request references Jira Issue OCPBUGS-57800, which is invalid:
Comment The bug has been updated to refer to the pull request using the external bug tracker. DetailsIn response to this: 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 openshift-eng/jira-lifecycle-plugin repository. |
|
/label cherry-pick-approved |
Co-authored-by: pýrus <[email protected]>
Unblock the gate while we investigate the perforamnce regressions seen in Dalmatian (and possibly Caracal). python-barbicanclient does not have a 2023.2-eol tag so we can't use install from Git. Since we're not hacking on though there's no good reason to be installing from Git, so we remove both this and python-octaviaclient from the list. Signed-off-by: Stephen Finucane <[email protected]> Co-authored-by: Stephen Finucane <[email protected]>
Signed-off-by: Justin Lamp <[email protected]> Co-authored-by: Justin Lamp <[email protected]>
…hen --with-topology=False (kubernetes#2872) * cinder-csi-plugin, manila-csi-plugin: Correct formatting character %T prints the type. %t prints the word true/false, which is what we want. [1] [1] https://pkg.go.dev/fmt Conflicts: pkg/csi/manila/driver.go NOTE(stephenfin): Merge conflicts are due to the absence of PR kubernetes#2734, which we don't want to backport. Signed-off-by: Stephen Finucane <[email protected]> * cinder-csi-plugin: Don't report topology capability when --with-topology=False The 'PluginCapability_Service_VOLUME_ACCESSIBILITY_CONSTRAINTS' capability flag determines whether the provisioner attempts to look up topology information from the node or not [1][2]. If we report it but don't return topology information from the node, we end up with failures to provision [3]. Fix the issue by optionally reporting the capability, like Manila already does. [1] https://github.com/kubernetes-csi/external-provisioner/blob/17e2429e9f/pkg/controller/controller.go#L685-L700 [2] https://github.com/kubernetes-csi/external-provisioner/blob/17e2429e9f/pkg/controller/controller.go#L994-L996 [3] https://github.com/kubernetes-csi/external-provisioner/blob/17e2429e9f/pkg/controller/topology.go#L177 Signed-off-by: Stephen Finucane <[email protected]> * manila-csi-plugin: Unify some logging between cinder, manila Signed-off-by: Stephen Finucane <[email protected]> --------- Signed-off-by: Stephen Finucane <[email protected]>
* update alpine to latest patch release * update go
d404832 to
3d669e2
Compare
Co-authored-by: pýrus <[email protected]>
Co-authored-by: Jesse Haka <[email protected]>
3d669e2 to
5aeb3bf
Compare
Co-authored-by: Ansou FALL <[email protected]>
5aeb3bf to
28d287d
Compare
* tests: Bump DevStack to Epoxy (2025.1) (kubernetes#2742) * tests: Prefer ginkgo timeout So that we actually get test results. Signed-off-by: Stephen Finucane <[email protected]> * tests: Align opts for Cinder, Manila tests Add a timeout to the Manila job and otherwise move some lines around. Signed-off-by: Stephen Finucane <[email protected]> * tests: Suffix image version While boskos will reap most resources for us, it doesn't reap images [1]. This has resulted in us using the same image for who knows how long at this point. Encode the Ubuntu version to prevent us picking up other version by mistake. [1] https://github.com/kubernetes-sigs/boskos/blob/5993cef5a1c719c33c0936d416b7d935058e1204/cmd/janitor/gcp_janitor.py#L46-L88 Signed-off-by: Stephen Finucane <[email protected]> * tests: Bump DevStack to Epoxy (2025.1) Signed-off-by: Stephen Finucane <[email protected]> * tests: Bump amphora image Use the Ubuntu 24.04 version, rather than the 22.04 version. This aligns with what we're using for DevStack itself. Signed-off-by: Stephen Finucane <[email protected]> * devstack: Remove USE_PYTHON3 It's all Python 3 now, baby. Signed-off-by: Stephen Finucane <[email protected]> * tests: Install Ansible from Debian Testing Signed-off-by: Stephen Finucane <[email protected]> * tests: Correct broken conditions Per the Ansible 2.19 porting guide [1]. [1] https://ansible.readthedocs.io/projects/ansible-core/devel/porting_guides/porting_guide_core_2.19.html Signed-off-by: Stephen Finucane <[email protected]> * tests: Decrease device detach timeout Wait less time before retrying. Signed-off-by: Stephen Finucane <[email protected]> --------- Signed-off-by: Stephen Finucane <[email protected]> * Bump CSI test timeouts (kubernetes#2966) * Bump CSI test timeouts We have worked around Nova bug #2119114 by lowering the device detach threshold to 1 second. Unfortunately this still leaves us with a N seconds of additional runtime, where N is the number of device detaches incurred by our test suite (since we run tests serially). This has put us right on the cusp of timeouts, meaning our jobs occasionally pass and occasionally fail, depending on the node we end up on. Add a bit more breathing room for the jobs while we wait for the Nova fix. Note that we do this for both Cinder and Manila to try keep those jobs consistent where possible. [1] https://bugs.launchpad.net/nova/+bug/2119114 Signed-off-by: Stephen Finucane <[email protected]> * tests: Temporarily remove share v1 endpoint Signed-off-by: Stephen Finucane <[email protected]> --------- Signed-off-by: Stephen Finucane <[email protected]> --------- Signed-off-by: Stephen Finucane <[email protected]> Co-authored-by: Stephen Finucane <[email protected]>
…1.31 into release-4.18
28d287d to
144fccf
Compare
|
/retest |
|
@stephenfin: This pull request references Jira Issue OCPBUGS-57800, which is valid. The bug has been moved to the POST state. 7 validation(s) were run on this bug
No GitHub users were found matching the public email listed for the QA contact in Jira ([email protected]), skipping review request. DetailsIn response to this:
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 openshift-eng/jira-lifecycle-plugin repository. |
go.mod
Outdated
| module k8s.io/cloud-provider-openstack | ||
|
|
||
| go 1.22.0 | ||
| go 1.23.11 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As noted by @stephenfin, commit d9937d12c492f80f89a5a0775b12aed4d8be001b bumps the go version and we can't really do that. We'll have to carry a downstream patch in branch release-4.18 to restore the go version to 1.22.
|
/test e2e-openstack-csi-manila |
|
/test e2e-openstack-parallel |
|
@shiftstack-merge-bot[bot]: The following tests failed, say
Full PR test history. Your PR dashboard. DetailsInstructions 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. |
|
Those failures in e2e-openstack-parallel are almost certainly due to ongoing resource contention. I'm happy with this as-is. /lgtm |
|
@stephenfin: This PR has been marked as verified by DetailsIn response to this:
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 openshift-eng/jira-lifecycle-plugin repository. |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: shiftstack-merge-bot[bot], stephenfin The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
8857214
into
openshift:release-4.18
|
@shiftstack-merge-bot[bot]: Jira Issue Verification Checks: Jira Issue OCPBUGS-57800 Jira Issue OCPBUGS-57800 has been moved to the MODIFIED state and will move to the VERIFIED state when the change is available in an accepted nightly payload. 🕓 DetailsIn response to this: 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 openshift-eng/jira-lifecycle-plugin repository. |
|
Fix included in accepted release 4.18.0-0.nightly-2025-10-08-051110 |
No description provided.