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-18545: CORS-2445: GCP: Add osImage to the install config #7468
OCPBUGS-18545: CORS-2445: GCP: Add osImage to the install config #7468
Conversation
barbacbd
commented
Sep 5, 2023
- The initial stage towards enabling users to specify a boot images from Google Cloud Marketplace. This involves the introduction of a new field called oSImage. Within this field, users can indicate the project and name of the desired image.
- Fixup tests and added generated data.
- Updated google compute version to the minimum version where Architecture is a valid field in the Image struct. The field is required for GCP marketplace images.
@barbacbd: This pull request references Jira Issue OCPBUGS-18545, which is invalid:
Comment The bug has been updated to refer to the pull request using the external bug tracker. 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 |
/jira refresh |
@barbacbd: This pull request references Jira Issue OCPBUGS-18545, 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. |
/jira refresh |
@barbacbd: This pull request references Jira Issue OCPBUGS-18545, which is valid. The bug has been moved to the POST state. 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. |
/label qe-approved |
/uncc @jstuever |
/approve |
/label backport-risk-assessed |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: patrickdillon 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 |
/label cherry-pick-approved |
Brent mentioned in slack that some further changes may be needed here, although it looks like this is testing well. If further changes are needed, let's re-evaluate this label. |
8fe4390
to
4033b18
Compare
/hold cancel |
ae69f15
to
519bcda
Compare
/cc @r4f4 |
/cc @cristianoveiga |
- name | ||
- project | ||
type: object | ||
tags: |
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.
@barbacbd Looks like you are introducing user specified tags here and on lines 1047 and 2238. Is that intentional? We released that enhancement in 4.12. https://issues.redhat.com/browse/CORS-2209
/hold Please refer to comment, thanks! |
The initial stage towards enabling users to specify a boot images from Google Cloud Marketplace. This involves the introduction of a new field called oSImage. Within this field, users can indicate the project and name of the desired image. Fixup tests and added generated data.
…etplace image to be used. This is the minimal version of google compute where the Image contains the necessary fields including Architecture.
f0cc52d
to
fcef8a5
Compare
@jianli-wei I pulled that data out of the PR (cherry-pick issues). Looks like it should mirror previous backports except for the dependency updates. |
/hold cancel |
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.
/lgtm
/retest-required |
/label acknowledge-critical-fixes-only |
/lgtm |
@barbacbd: The following tests 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. |
f87b87d
into
openshift:release-4.11
@barbacbd: Jira Issue OCPBUGS-18545: All pull requests linked via external trackers have merged: Jira Issue OCPBUGS-18545 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. |
Fix included in accepted release 4.11.0-0.nightly-2023-09-14-210203 |