-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
add rpm tooling to bazelbuild image #5461
Conversation
TODO: push new images and bump, though there's not much rush until we add actually building RPMs to any of the jobs so this can realistically be a follow up along with support for that. |
lgtm do you want to bump in this PR or a separate one? |
I'll go ahead and bump. |
ae73dc5
to
8fd33aa
Compare
Grrr, xref #5411. |
8fd33aa
to
bcf8928
Compare
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
(actually started self review and ixdy LGTM above)
@@ -545,8 +545,9 @@ func TestBazelbuildArgs(t *testing.T) { | |||
} | |||
pinnedJobs := map[string]string{ | |||
//job: reason for pinning | |||
"pull-test-infra-bazel": "test-infra adopts bazel upgrades first", | |||
"ci-test-infra-bazel": "test-infra adopts bazel upgrades first", | |||
// these frequently need to be pinned... |
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.
we might want to consider updating the presubmit to allow jobs to remain in the pinned whitelist even if they are not current considered pinned, these jobs are repeatedly pinned / unpinned since we dogfood bazel upgrades on test-infra first.
@BenTheElder: you cannot LGTM your own PR. 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. |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: BenTheElder, ixdy The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these OWNERS Files:
You can indicate your approval by writing |
@BenTheElder: I updated Prow config for you! 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. |
see: #5429
follow-ish up to: #5456
/area bazel
/area images