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

test: Address flaky tests in linode_client and lke #412

Merged

Conversation

ykim-1
Copy link
Contributor

@ykim-1 ykim-1 commented Jun 4, 2024

📝 Description

Fixing failures in http://198.19.5.79:1212/builds/665e32593caad400014ecfa0?team=DX&buildName=linode_api4&bld_id=665e32593caad400014ecfa0&testcaseStatus=failure

Skipping test_list_child_accounts for now until test account is confiured

✔️ How to Test

make test
make TEST_CASE="test_image_create" testint
make TEST_SUITE="lke" testint

How do I run the relevant unit/integration tests?

📷 Preview

If applicable, include a screenshot or code snippet of this change. Otherwise, please remove this section.

@ykim-1 ykim-1 requested a review from a team as a code owner June 4, 2024 16:29
@ykim-1 ykim-1 requested review from jriddle-linode and zliang-akamai and removed request for a team June 4, 2024 16:29
Copy link
Contributor

@lgarber-akamai lgarber-akamai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One small change, otherwise LGTM!

@ykim-1 ykim-1 merged commit 90b43ab into linode:dev Jun 4, 2024
7 checks passed
@zliang-akamai zliang-akamai added the testing for updates to the testing suite in the changelog. label Jun 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
testing for updates to the testing suite in the changelog.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants