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

Add support for missing "Check failed" Health Event notifications #927

Merged
merged 2 commits into from
Jun 13, 2023

Conversation

lucymhdavies
Copy link
Contributor

@lucymhdavies lucymhdavies commented Jun 13, 2023

Description

Closes #926

Remember to:

Testing plan

#926

Output from acceptance tests

N/A: No tests needed changing by the looks of it

@lucymhdavies lucymhdavies requested a review from a team as a code owner June 13, 2023 15:45
@lucymhdavies lucymhdavies force-pushed the strawb/tfe_notification_configuration-check_failure branch from a180228 to bbffc4f Compare June 13, 2023 15:46
Copy link
Collaborator

@brandonc brandonc left a comment

Choose a reason for hiding this comment

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

Tiny copy paste issue looks like

website/docs/r/notification_configuration.html.markdown Outdated Show resolved Hide resolved
Co-authored-by: Brandon Croft <brandon.croft@gmail.com>
@lucymhdavies
Copy link
Contributor Author

Well spotted!

@brandonc brandonc merged commit 9057f21 into main Jun 13, 2023
6 of 7 checks passed
@brandonc brandonc deleted the strawb/tfe_notification_configuration-check_failure branch June 13, 2023 16:52
github-merge-queue bot pushed a commit to panda-den/colorful-pandas that referenced this pull request Jul 4, 2023
This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
| [tfe](https://registry.terraform.io/providers/hashicorp/tfe)
([source](https://togithub.com/hashicorp/terraform-provider-tfe)) |
required_provider | minor | `0.45.0` -> `0.46.0` |

---

### ⚠ Dependency Lookup Warnings ⚠

Warnings were logged while processing this repo. Please check the
Dependency Dashboard for more information.

---

### Release Notes

<details>
<summary>hashicorp/terraform-provider-tfe (tfe)</summary>

###
[`v0.46.0`](https://togithub.com/hashicorp/terraform-provider-tfe/blob/HEAD/CHANGELOG.md#v0460-July-3-2023)

[Compare
Source](https://togithub.com/hashicorp/terraform-provider-tfe/compare/v0.45.0...v0.46.0)

FEATURES:

- **New Resource**: `r/tfe_agent_pool_allowed_workspaces` restricts the
use of an agent pool to particular workspaces, by
[@&#8203;hs26gill](https://togithub.com/hs26gill)
[870](https://togithub.com/hashicorp/terraform-provider-tfe/pull/870)
- `r/tfe_organization_token`: Add optional `expired_at` field to
organization tokens, by
[@&#8203;juliannatetreault](https://togithub.com/juliannatetreault)
(#&#8203[hashicorp/terraform-provider-tfe#844))
- `r/tfe_team_token`: Add optional `expired_at` field to team tokens, by
[@&#8203;juliannatetreault](https://togithub.com/juliannatetreault)
(#&#8203[hashicorp/terraform-provider-tfe#844))
- `r/tfe_agent_pool`: Add attribute `organization_scoped` to set the
scope of an agent pool, by
[@&#8203;hs26gill](https://togithub.com/hs26gill)
[870](https://togithub.com/hashicorp/terraform-provider-tfe/pull/870)
- `d/tfe_agent_pool`: Add attribute `organization_scoped` and
`allowed_workspace_ids` to retrieve agent pool scope and associated
allowed workspace ids, by
[@&#8203;hs26gill](https://togithub.com/hs26gill)
[870](https://togithub.com/hashicorp/terraform-provider-tfe/pull/870)

BUG FIXES:

- `r/tfe_workspace_run`: Ensure `wait_for_run` correctly results in a
fire-and-forget run when set to `false`, by
[@&#8203;lucymhdavies](https://togithub.com/lucymhdavies)
(#&#8203[hashicorp/terraform-provider-tfe#910))
- `r/tfe_workspace_run`: Fix rare random run failures; adjust lists of
expected run statuses to ensure that a plan is completely processed
before attempting to apply it, by
[@&#8203;uk1288](https://togithub.com/uk1288)
(#&#8203[hashicorp/terraform-provider-tfe#921))
- `r/tfe_notification_configuration`: Add support for missing "Check
failed" Health Event notifications, by
[@&#8203;lucymhdavies](https://togithub.com/lucymhdavies)
(#&#8203[hashicorp/terraform-provider-tfe#927))
- `r/tfe_registry_module`: Fix a bug that prevented users from being
able to create a registry module using a github app, by
[@&#8203;dsa0x](https://togithub.com/dsa0x)
(#&#8203[hashicorp/terraform-provider-tfe#935))

</details>

---

### Configuration

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

🚦 **Automerge**: Enabled.

♻ **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:eyJjcmVhdGVkSW5WZXIiOiIzNS4xNTkuNCIsInVwZGF0ZWRJblZlciI6IjM1LjE1OS40IiwidGFyZ2V0QnJhbmNoIjoibWFpbiJ9-->

Co-authored-by: astronaut-panda[bot] <137164246+astronaut-panda[bot]@users.noreply.github.com>
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.

tfe_notification_configuration missing assessment:check_failure
2 participants