feat(networkservices): add DNS peering config to AgentGateway:#17545
Conversation
|
Googlers: For automatic test runs see go/terraform-auto-test-runs. @melinath, a repository maintainer, has been assigned to review your changes. If you have not received review feedback within 2 business days, please leave a comment on this PR asking them to take a look. You can help make sure that review is quick by doing a self-review and by running impacted tests locally. |
This comment was marked as outdated.
This comment was marked as outdated.
|
/gcbrun |
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
melinath
left a comment
There was a problem hiding this comment.
The change looks straightforward.
However, it looks like the tests (and docs) for this resource are disabled (and have been since the resource was introduced in #16391) due to things needing to "stabilize".
Could you try enabling the tests (and docs?) at this point? I'd like to have automated test runs verifying that the resource & the new fields work as expected.
This comment was marked as outdated.
This comment was marked as outdated.
melinath
left a comment
There was a problem hiding this comment.
Thanks for enabling a few tests! It looks like there are now a large number of failures.
Three of the AgentGateway tests are now failing with "Inconsistent dependency lock file" - see https://googlecloudplatform.github.io/magic-modules/test/run-tests/#error-inconsistent-dependency-lock-file
The final test is failing with: Error code 3, message: Config validation failed.
| versions: | ||
| - name: 'beta' | ||
| base_url: 'https://networkservices.googleapis.com/v1/' | ||
| base_url: 'https://networkservices.googleapis.com/v1beta1/' |
There was a problem hiding this comment.
Any chance this is causing all the other test failures? Terraform expects that beta APIs will be a strict superset of GA APIs; if that's not true of this API for all resources, that could explain it.
(If this is the problem, we'll need to discuss potential solutions.)
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
7d33697 to
14a45d4
Compare
This comment was marked as outdated.
This comment was marked as outdated.
14a45d4 to
357c429
Compare
|
Hi there, I'm the Modular magician. I've detected the following information about your changes for commit 357c429: Diff reportYour PR generated the following diffs in downstream repositories:
Test reportAnalytics
Affected Service Packages
Step 1: Replaying Mode Action takenFound 1 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected tests
View the replaying VCR build log Step 2: Recording Mode
🟢 All tests passed! View the recording VCR build log or the debug logs folder for detailed results. @JamesDuncanNz, @melinath VCR tests complete for 357c429! |
c7d465a
Adds the dnsPeeringConfig nested object under privateConnectivity.egress
on google_network_services_agent_gateway, allowing users to specify
domains, targetProject, and targetNetwork so the gateway resolves
queries via Cloud DNS in a peered VPC.
Beta Provider Test Log
GA Provider Test Log
Release Note Template for Downstream PRs (will be copied)
See Write release notes for guidance.