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
build: Update bazel to 3.4.1 and k/repo-infra to v0.2.0 #96011
Conversation
@mikedanese: This issue is currently awaiting triage. If a SIG or subproject determines this is a relevant issue, they will accept it by applying the The 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 |
/test pull-kubernetes-bazel-test-canary |
1 similar comment
/test pull-kubernetes-bazel-test-canary |
/test pull-kubernetes-bazel-test-canary |
/retest |
I don't think we need this until after code freeze, but I'll let @justaugustus say when. |
/retitle WIP: bump bazel to 3.4.1 |
There was a bug with init() but it was resolved in bazelbuild/rules_go#2696 Changed to match other fixture methods. Change-Id: I882b8535e5c5c117fb10c41d34c8eed1ccdb74bb
// | ||
// The approach taken here works for both go test and bazel on the assumption | ||
// that if and only if trimpath is passed, we are running under bazel. | ||
func pkgPath() (string, error) { |
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.
This is copied from #90872. There was a bug fix to data loading in init() that was resolved in bazelbuild/rules_go#1918. This ~removes the workaround.
/retest |
For staging/src/k8s.io/legacy-cloud-providers/vsphere/vclib/fixtures/fixtures.go change: /assign @andrewsykim |
/lgtm |
/approve (For code organization / dependency updates only) :) |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: dims, justaugustus, mikedanese 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 |
/hold ( for @andrewsykim ) |
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 for staging/src/k8s.io/legacy-cloud-providers/vsphere/vclib/fixtures/fixtures.go
/hold cancel |
/retest |
Awesome work, @mikedanese!!! |
build: Update bazel to 3.4.1 and k/repo-infra to v0.2.0
Needed for Golang 1.16 updates: kubernetes/release#1834
/kind cleanup
/sig release