You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
[BUG]: The github_repository resource churns on vulnerability_alerts if not explicitly set #2489
When creating a new repository with the github_repository resource, this should not churn if vulnerability_alerts is not set.
Actual Behavior
If vulnerability_alerts is not set it will constantly churn.
Terraform Version
n/a
Affected Resource(s)
github_repository
Terraform Configuration Files
resource"github_repository""test" {
name="test"
}
Steps to Reproduce
Apply the above resource and then run a plan.
This issue is being caused by a missing Computed: true value on the schema. I'll open a PR to fix this, but long term this attribute should be deprecated and handled by it's own resource.
changed the title [-][BUG]: Repository data source churns on `vulnerability_alerts` if not explicitly set[/-][+][BUG]: The `github_repository` resource churns on `vulnerability_alerts` if not explicitly set[/+]on Dec 4, 2024
data_source_github_repository_autolink_references_test.go:44: Step 1/2 error: After applying this test step, the plan was not empty.
stdout:
Terraform used the selected providers to generate the following execution
plan. Resource actions are indicated with the following symbols:
~ update in-place
Terraform will perform the following actions:
# github_repository.test will be updated in-place
~ resource "github_repository" "test" {
id = "tf-acc-test-uzqg7"
name = "tf-acc-test-uzqg7"
- vulnerability_alerts = true -> null
# (35 unchanged attributes hidden)
# (1 unchanged block hidden)
}
Plan: 0 to add, 1 to change, 0 to destroy.
@acouvreur I'm confused as to what you're saying? This PR is intended to fix the flaky tests, so if you've got a branch from main it will have the issues this PR fixes present.
@acouvreur I'm confused as to what you're saying? This PR is intended to fix the flaky tests, so if you've got a branch from main it will have the issues this PR fixes present.
I'm sorry about the confusion, I wanted to point out that this issue is still happening and is preventing me from contributing properly.
So it was more something like: "this issue is still relevant and happens to me, can confirm."
Activity
stevehipwell commentedon Dec 3, 2024
It also looks like if a repository is created with an explicit value for
vulnerability_alertsthat doesn't match the default it will also churn.[-][BUG]: Repository data source churns on `vulnerability_alerts` if not explicitly set[/-][+][BUG]: The `github_repository` resource churns on `vulnerability_alerts` if not explicitly set[/+]stevehipwell commentedon Dec 4, 2024
The code to fix this is currently in #2476 but I'm planning on opening a seperate PR for this change before that gets merged.
acouvreur commentedon Feb 11, 2025
I'm having this issue running acceptance tests:
stevehipwell commentedon Feb 11, 2025
@acouvreur acceptance tests on this branch?
acouvreur commentedon Feb 11, 2025
My feature branch from main
stevehipwell commentedon Feb 11, 2025
@acouvreur I'm confused as to what you're saying? This PR is intended to fix the flaky tests, so if you've got a branch from main it will have the issues this PR fixes present.
acouvreur commentedon Feb 11, 2025
I'm sorry about the confusion, I wanted to point out that this issue is still happening and is preventing me from contributing properly.
So it was more something like: "this issue is still relevant and happens to me, can confirm."
I'm just showing support for this PR 😀
stevehipwell commentedon Feb 11, 2025
Sorry @acouvreur I thought you'd commented on the PR to fix this (#2501). FYI PR #2476 makes all of the automation tests work correctly.
Lekensteyn commentedon Mar 7, 2025
Hey @stevehipwell, could you clarify what kind of churn you are seeing? What does the plan output look like?
stevehipwell commentedon Mar 7, 2025
@Lekensteyn I covered how to reproduce in the description as well as what I'm seeing. You can also look at my PR to fix it.