Skip to content
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

Merged
merged 2 commits into from
Dec 13, 2024
Merged

Conversation

miampf
Copy link
Contributor

@miampf miampf commented Dec 3, 2024

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.

@miampf miampf requested a review from katexochen as a code owner December 3, 2024 11:30
@miampf miampf added the no changelog PRs not listed in the release notes label Dec 3, 2024
.github/workflows/e2e.yml Outdated Show resolved Hide resolved
@miampf miampf force-pushed the miampf/log-collection-download branch from 63b4c8e to 48e6d93 Compare December 5, 2024 10:48
@miampf miampf requested a review from burgerdev as a code owner December 5, 2024 10:54
Copy link
Contributor

@burgerdev burgerdev left a 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?

.github/workflows/e2e_aks_runtime.yml Outdated Show resolved Hide resolved
@miampf
Copy link
Contributor Author

miampf commented Dec 5, 2024

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 :)

@miampf miampf force-pushed the miampf/log-collection-download branch 3 times, most recently from 5edcb5d to 905e771 Compare December 6, 2024 12:05
@miampf miampf force-pushed the miampf/log-collection-download branch 2 times, most recently from 0628e73 to da11a37 Compare December 12, 2024 11:49
@miampf miampf requested review from 3u13r and burgerdev December 12, 2024 14:42
Copy link
Contributor

@burgerdev burgerdev left a 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).

.github/workflows/e2e_aks_runtime.yml Outdated Show resolved Hide resolved
e2e/aks-runtime/aks_runtime_test.go Outdated Show resolved Hide resolved
@miampf miampf force-pushed the miampf/log-collection-download branch 3 times, most recently from 175a333 to c1d25d6 Compare December 12, 2024 15:44
@miampf miampf requested a review from burgerdev December 12, 2024 15:45
@miampf miampf force-pushed the miampf/log-collection-download branch from c1d25d6 to 33ea1b0 Compare December 12, 2024 15:45
@miampf miampf merged commit 213a7e5 into main Dec 13, 2024
12 checks passed
@miampf miampf deleted the miampf/log-collection-download branch December 13, 2024 09:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no changelog PRs not listed in the release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants