Skip to content

Remove flaky integ test that has been tested in setUp method#6417

Merged
zoewangg merged 1 commit intomasterfrom
zoewang/flakyStsIntegTest
Sep 12, 2025
Merged

Remove flaky integ test that has been tested in setUp method#6417
zoewangg merged 1 commit intomasterfrom
zoewang/flakyStsIntegTest

Conversation

@zoewangg
Copy link
Copy Markdown
Contributor

Remove flaky integ test that has been tested in setUp

// Try to assume the role to make sure we won't hit issues during testing.
StsClient userCredentialSts = StsClient.builder()
.credentialsProvider(() -> userCredentials)
.build();
// Ensure the new credentials have propagated and are valid.
Waiter.run(userCredentialSts::getCallerIdentity)
.ignoringException(StsException.class)
.orFailAfter(Duration.ofMinutes(2));
Waiter.run(() -> userCredentialSts.assumeRole(r -> r.durationSeconds(SESSION_DURATION)
.roleArn(ROLE_ARN)
.roleSessionName("Test")))
.ignoringException(StsException.class)
.orFailAfter(Duration.ofMinutes(8));

@zoewangg zoewangg requested a review from a team as a code owner September 12, 2025 21:10
@zoewangg zoewangg enabled auto-merge September 12, 2025 21:27
@sonarqubecloud
Copy link
Copy Markdown

@zoewangg zoewangg added this pull request to the merge queue Sep 12, 2025
Merged via the queue into master with commit 47e2f4a Sep 12, 2025
38 checks passed
@github-actions
Copy link
Copy Markdown

This pull request has been closed and the conversation has been locked. Comments on closed PRs are hard for our team to see. If you need more assistance, please open a new issue that references this one.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 12, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants