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
Update GPU e2e tests. #47467
Update GPU e2e tests. #47467
Conversation
That installer decouples itself from COS image version (as long as the image version is newer than cos-stable-59-9460-60-0). A separate commit in the test-infra repo will update the cos version used for this test to cos-stable-59-9460-60-0.
Hi @mindprince. Thanks for your PR. I'm waiting for a kubernetes member to verify that this patch is reasonable to test. If it is, they should reply with I understand the commands that are listed here. 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. |
cc - @vishh |
This is required by the updated installer added in kubernetes/kubernetes#47467
test/e2e/nvidia-gpus.go
Outdated
data := generated.ReadOrDie(fileName) | ||
framework.Logf("Parsing ds from %v", url) | ||
|
||
response, err := http.Get(url) |
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.
Should we retry a few times just to avoid spurious network issues or github serving issues?
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.
Added retries.
Just one comment. Otherwise LGTM |
@k8s-bot ok to test |
/approve |
71d7949
to
21d2601
Compare
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: dchen1107, mindprince, vishh Associated issue: 47388 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 |
/retest |
Automatic merge from submit-queue |
Waiting for kubernetes/test-infra#3050 to get merged. |
- It contains a fix for ipaliasing. - It contains a fix which decouples GPU driver installation from kernel version. Remove dead code that has now moved to another repo as part of kubernetes#47467
Automatic merge from submit-queue (batch tested with PRs 47530, 47679) Use cos-stable-59-9460-64-0 instead of cos-beta-59-9460-20-0. Remove dead code that has now moved to another repo as part of #47467 **Release note**: ```release-note NONE ``` /sig node
Use nvidia driver installer from external repo.
That installer decouples itself from COS image version (as long as the
image version is newer than cos-stable-59-9460-60-0).
A separate commit in the test-infra repo will update the cos version
used for this test to cos-stable-59-9460-60-0.
Use cos-stable-59-9460-60-0 and newer installer for GPU node e2e tests.
This is to enable #47388.
This supercedes #47091.
Release note:
/sig node