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

chore: fix flaky test #4228

Merged
merged 1 commit into from
Jul 15, 2024
Merged

chore: fix flaky test #4228

merged 1 commit into from
Jul 15, 2024

Conversation

corymhall
Copy link
Contributor

This test has been failing recently due to a resource with the same name
already existing. I think we have been running into a race condition
between different jobs.

Remove the hardcoded name.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
This test has been failing recently due to a resource with the same name
already existing. I think we have been running into a race condition
between different jobs.

Remove the hardcoded name.
@corymhall corymhall requested review from t0yv0 and flostadler July 15, 2024 16:57
Copy link
Contributor

@flostadler flostadler left a comment

Choose a reason for hiding this comment

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

nice!

Copy link

Does the PR have any schema changes?

Looking good! No breaking changes found.
No new resources/functions.

Maintainer note: consult the runbook for dealing with any breaking changes.

@corymhall corymhall merged commit 2684053 into master Jul 15, 2024
31 checks passed
@corymhall corymhall deleted the corymhall/2555-flaky-test branch July 15, 2024 19:57
@pulumi-bot
Copy link
Contributor

This PR has been shipped in release v6.45.0.

lumiere-bot bot referenced this pull request in coolguy1771/home-ops Jul 17, 2024
This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
| [@pulumi/aws](https://pulumi.io)
([source](https://togithub.com/pulumi/pulumi-aws)) | dependencies |
minor | [`6.44.0` ->
`6.45.0`](https://renovatebot.com/diffs/npm/@pulumi%2faws/6.44.0/6.45.0)
|

---

### Release Notes

<details>
<summary>pulumi/pulumi-aws (@&#8203;pulumi/aws)</summary>

###
[`v6.45.0`](https://togithub.com/pulumi/pulumi-aws/releases/tag/v6.45.0)

[Compare
Source](https://togithub.com/pulumi/pulumi-aws/compare/v6.44.0...v6.45.0)

##### Does the PR have any schema changes?

Found 2 breaking changes:

##### Types

- `🟢`
"aws:ec2/getLaunchTemplateNetworkInterface:getLaunchTemplateNetworkInterface":
required: "primaryIpv6" property has changed to Required
- `🟢` "aws:eks/getClusterAccessConfig:getClusterAccessConfig": required:
"bootstrapClusterCreatorAdminPermissions" property has changed to
Required

##### New resources:

-   `cloudwatch/logAccountPolicy.LogAccountPolicy`
-   `verifiedpermissions/identitySource.IdentitySource`

##### What's Changed

- Update GitHub Actions workflows. by
[@&#8203;pulumi-bot](https://togithub.com/pulumi-bot) in
[https://github.com/pulumi/pulumi-aws/pull/4202](https://togithub.com/pulumi/pulumi-aws/pull/4202)
- Update GitHub Actions workflows. by
[@&#8203;pulumi-bot](https://togithub.com/pulumi-bot) in
[https://github.com/pulumi/pulumi-aws/pull/4209](https://togithub.com/pulumi/pulumi-aws/pull/4209)
- Upstream v5.58.0 by
[@&#8203;flostadler](https://togithub.com/flostadler) in
[https://github.com/pulumi/pulumi-aws/pull/4217](https://togithub.com/pulumi/pulumi-aws/pull/4217)
- chore: fix flaky test by
[@&#8203;corymhall](https://togithub.com/corymhall) in
[https://github.com/pulumi/pulumi-aws/pull/4228](https://togithub.com/pulumi/pulumi-aws/pull/4228)
- chore: use pulumi v3.124.1 in examples by
[@&#8203;corymhall](https://togithub.com/corymhall) in
[https://github.com/pulumi/pulumi-aws/pull/4233](https://togithub.com/pulumi/pulumi-aws/pull/4233)

**Full Changelog**:
pulumi/pulumi-aws@v6.44.0...v6.45.0

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Renovate
Bot](https://togithub.com/renovatebot/renovate).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy40MzIuMCIsInVwZGF0ZWRJblZlciI6IjM3LjQzMi4wIiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6WyJ0eXBlL21pbm9yIl19-->

Co-authored-by: lumiere-bot[bot] <98047013+lumiere-bot[bot]@users.noreply.github.com>
lumiere-bot bot referenced this pull request in coolguy1771/home-ops Jul 17, 2024

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
| [@pulumi/aws](https://pulumi.io)
([source](https://togithub.com/pulumi/pulumi-aws)) | dependencies |
minor | [`6.44.0` ->
`6.45.0`](https://renovatebot.com/diffs/npm/@pulumi%2faws/6.44.0/6.45.0)
|

---

### Release Notes

<details>
<summary>pulumi/pulumi-aws (@&#8203;pulumi/aws)</summary>

###
[`v6.45.0`](https://togithub.com/pulumi/pulumi-aws/releases/tag/v6.45.0)

[Compare
Source](https://togithub.com/pulumi/pulumi-aws/compare/v6.44.0...v6.45.0)

##### Does the PR have any schema changes?

Found 2 breaking changes:

##### Types

- `🟢`
"aws:ec2/getLaunchTemplateNetworkInterface:getLaunchTemplateNetworkInterface":
required: "primaryIpv6" property has changed to Required
- `🟢` "aws:eks/getClusterAccessConfig:getClusterAccessConfig": required:
"bootstrapClusterCreatorAdminPermissions" property has changed to
Required

##### New resources:

-   `cloudwatch/logAccountPolicy.LogAccountPolicy`
-   `verifiedpermissions/identitySource.IdentitySource`

##### What's Changed

- Update GitHub Actions workflows. by
[@&#8203;pulumi-bot](https://togithub.com/pulumi-bot) in
[https://github.com/pulumi/pulumi-aws/pull/4202](https://togithub.com/pulumi/pulumi-aws/pull/4202)
- Update GitHub Actions workflows. by
[@&#8203;pulumi-bot](https://togithub.com/pulumi-bot) in
[https://github.com/pulumi/pulumi-aws/pull/4209](https://togithub.com/pulumi/pulumi-aws/pull/4209)
- Upstream v5.58.0 by
[@&#8203;flostadler](https://togithub.com/flostadler) in
[https://github.com/pulumi/pulumi-aws/pull/4217](https://togithub.com/pulumi/pulumi-aws/pull/4217)
- chore: fix flaky test by
[@&#8203;corymhall](https://togithub.com/corymhall) in
[https://github.com/pulumi/pulumi-aws/pull/4228](https://togithub.com/pulumi/pulumi-aws/pull/4228)
- chore: use pulumi v3.124.1 in examples by
[@&#8203;corymhall](https://togithub.com/corymhall) in
[https://github.com/pulumi/pulumi-aws/pull/4233](https://togithub.com/pulumi/pulumi-aws/pull/4233)

**Full Changelog**:
pulumi/pulumi-aws@v6.44.0...v6.45.0

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about these
updates again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Renovate
Bot](https://togithub.com/renovatebot/renovate).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy40MzIuMCIsInVwZGF0ZWRJblZlciI6IjM3LjQzMi4wIiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6WyJ0eXBlL21pbm9yIl19-->

Co-authored-by: lumiere-bot[bot] <98047013+lumiere-bot[bot]@users.noreply.github.com>
@mjeffryes mjeffryes added this to the 0.107 milestone Jul 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants