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 ability to use IPv6 monitoring jobs. #141

Merged

Conversation

JesseLukeWalker
Copy link
Contributor

No description provided.

@Zach-Johnson
Copy link
Collaborator

@JesseLukeWalker this looks good to me. Before we merge can you confirm:

  • The full acceptance test suite has been ran and is passing (i.e. make testacc)
  • Some manual testing of this change specifically has been done to verify it works as expected, i.e. some manual test like this:
    • add the field in a terraform config
    • terraform plan and see the expected change
    • terraform apply
    • terraform plan and see that there are no changes
    • remove the field
    • terraform plan and see that the expected change to remove it
    • terraform apply
    • terraform plan and see that there are no changes

@JesseLukeWalker
Copy link
Contributor Author

make testacc fails on origin and my branch, no matter what, with:

=== RUN   TestAccDataSourceRecord
--- FAIL: TestAccDataSourceRecord (0.20s)
    testing.go:640: Step 0 error: errors during apply:

        Error: zone does not exist

          on /tmp/tf-test657900993/main.tf line 1:
          (source code not available)

    testing.go:701: Error destroying resource! WARNING: Dangling resources
        may exist. The full state and error is shown below.

        Error: Check failed: record still exists: (*dns.Record)(nil) &rest.Error{Resp:(*http.Response)(0xc000874630), Message:"Not Found"}

        State: <no state>

so I can't get the tests to pass at all.

However, I can confirm that add ipv6 = true, terraform [plan/apply/plan], remove ipv6, terraform [plan/apply/plan], add ipv6 = true, terraform [plan/apply/plan], all works fine for me.

Copy link
Collaborator

@Zach-Johnson Zach-Johnson left a comment

Choose a reason for hiding this comment

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

lgtm, failing test is not related to this

@mburtless mburtless merged commit b67476e into ns1-terraform:master Oct 19, 2020
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.

3 participants