-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Promote Server Tls Policy to V1 #11538
Promote Server Tls Policy to V1 #11538
Conversation
Hello! I am a robot. Tests will require approval from a repository maintainer to run. @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. |
Hi there, I'm the Modular magician. I've detected the following information about your changes: Diff reportYour PR generated some diffs in downstreams - here they are.
|
Tests analyticsTotal tests: 39 Click here to see the affected service packages
Tests were added that are GA-only additions and require manual runs:
Action takenFound 3 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected tests
|
|
3e5be14
to
f458dea
Compare
Hi there, I'm the Modular magician. I've detected the following information about your changes: Diff reportYour PR generated some diffs in downstreams - here they are.
|
Tests analyticsTotal tests: 39 Click here to see the affected service packages
Tests were added that are GA-only additions and require manual runs:
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
|
|
f458dea
to
564144e
Compare
Hi there, I'm the Modular magician. I've detected the following information about your changes: Diff reportYour PR generated some diffs in downstreams - here they are.
|
Tests analyticsTotal tests: 39 Click here to see the affected service packages
Tests were added that are GA-only additions and require manual runs:
View the build log |
@melinath This PR has been waiting for review for 3 weekdays. Please take a look! Use the label |
mmv1/templates/terraform/examples/network_security_server_tls_policy_basic.tf.erb
Show resolved
Hide resolved
mmv1/templates/terraform/examples/network_security_server_tls_policy_basic.tf.erb
Show resolved
Hide resolved
It looks like the tests all passed in GA, so I'll approve & merge when the tests finish running again. |
Hi there, I'm the Modular magician. I've detected the following information about your changes: Diff reportYour PR generated some diffs in downstreams - here they are.
|
Tests analyticsTotal tests: 39 Click here to see the affected service packages
Tests were added that are GA-only additions and require manual runs:
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
|
|
Co-authored-by: Stephen Lewis (Burrows) <[email protected]>
Promote Server TLS Policy to V1
Also fixed hashicorp/terraform-provider-google#18103
Release Note Template for Downstream PRs (will be copied)