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 1935174: RHCOS bump for LUKS, prjquota, etc #4791
Bug 1935174: RHCOS bump for LUKS, prjquota, etc #4791
Conversation
This bumps the boot image to to include fixes for the following: BZ#1922417 - Issue configuring nodes with VLAN and teaming BZ#1940966 - [4.7.z] prjquota is dropped from rootflags if rootfs is reprovisioned BZ#1941760 - [4.7.z] rootfs too small when enabling NBDE (redo) BZ#1942706 - [4.7.z] support new AWS region ap-northeast-3
@miabbott: This pull request references Bugzilla bug 1935174, which is invalid:
Comment In 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/test-infra repository. |
/assign jlebon |
/lgtm |
Will examine other failures later. /retest |
/bugzilla refresh Recalculating validity in case the underlying Bugzilla bug has changed. |
@openshift-bot: This pull request references Bugzilla bug 1935174, which is invalid:
Comment In 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/test-infra repository. |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: staebler The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
@staebler: Overrode contexts on behalf of staebler: ci/prow/e2e-kubevirt, ci/prow/e2e-metal-assisted, ci/prow/e2e-openstack-ipi, ci/prow/e2e-vsphere-upi In 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/test-infra repository. |
/retest Please review the full test history for this PR and help us cut down flakes. |
4 similar comments
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
Regarding the BZ status: We can probably get 1922417, 1940966, 1941760 verified today or early next week. 1942706 is the support for ap-northeast-3 which is covered by this PR. All of the referenced BZs are targeting 4.7.z as they are expected to be fixed in that release, so I don't understand why the bot complains about them. |
/bugzilla refresh Recalculating validity in case the underlying Bugzilla bug has changed. |
@openshift-bot: This pull request references Bugzilla bug 1935174, which is invalid:
Comment In 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/test-infra repository. |
/bugzilla refresh Recalculating validity in case the underlying Bugzilla bug has changed. |
@openshift-bot: This pull request references Bugzilla bug 1935174, which is invalid:
Comment In 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/test-infra repository. |
/bugzilla refresh Recalculating validity in case the underlying Bugzilla bug has changed. |
@openshift-bot: This pull request references Bugzilla bug 1935174, which is invalid:
Comment In 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/test-infra repository. |
/hold It looks like the AWS SDK needs to be bumped to support the |
Right. But adding support in the RHCOS image is separate from the installer offering the region as a valid region. |
@staebler could you override the invalid bug flag? The associated BZs are fixed in RHCOS 4.7 (except for the ap-northeast-3 one) and are targeting 4.7.z (They actually were incorrectly attached to the last 4.7.z errata that went out, so the sooner we can get this change shipped the better) |
I can try. I've never done that before. Although, this needs cherry-pick approval anyway. Would it be better for the patch manager to override the invalid-bug label? |
Looks like it worked! Next time I'll be more patient and wait for the patch manager. |
/bugzilla refresh |
@miabbott: This pull request references Bugzilla bug 1935174, which is invalid:
Comment In 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/test-infra repository. |
/bugzilla refresh |
@miabbott: This pull request references Bugzilla bug 1935174, which is invalid:
Comment In 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/test-infra repository. |
/bugzilla refresh |
@miabbott: This pull request references Bugzilla bug 1935174, which is valid. The bug has been updated to refer to the pull request using the external bug tracker. 6 validation(s) were run on this bug
Requesting review from QA contact: In 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/test-infra repository. |
/retest Please review the full test history for this PR and help us cut down flakes. |
5 similar comments
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
@miabbott: The following test failed, say
Full PR test history. Your PR dashboard. 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. I understand the commands that are listed here. |
@miabbott: All pull requests linked via external trackers have merged: Bugzilla bug 1935174 has been moved to the MODIFIED state. In 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/test-infra repository. |
This bumps the boot image to to include fixes for the following:
BZ#1922417 - Issue configuring nodes with VLAN and teaming
BZ#1940966 - [4.7.z] prjquota is dropped from rootflags if rootfs is reprovisioned
BZ#1941760 - [4.7.z] rootfs too small when enabling NBDE (redo)
BZ#1942706 - [4.7.z] support new AWS region ap-northeast-3