-
Notifications
You must be signed in to change notification settings - Fork 8
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
e2e: fix log collection download #1046
Conversation
63b4c8e
to
48e6d93
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.
Do you want to add the default_platform
for the AKS runtime test in this PR?
Do you mean to allow selection between AKS-CLH-SNP and AKS-PEER-SNP? If yes, I can do that if you want :) |
5edcb5d
to
905e771
Compare
0628e73
to
da11a37
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.
After addressing the comments, please squash the commits (maybe it makes sense to have one for the flag removal and one for the test fix, but if it does not separate cleanly then 1 commit is fine).
175a333
to
c1d25d6
Compare
c1d25d6
to
33ea1b0
Compare
Currently, downloading logs from an e2e cluster may fail since the cluster may be undeployed before the download is initiated. This PR fixes that problem by undeploying the cluster after the logs were downloaded.