-
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
Add google_gkeonprem_vmware_admin_cluster resource #12011
Add google_gkeonprem_vmware_admin_cluster resource #12011
Conversation
…google/magic-modules into vmware-admin-cluster
Hello! I am a robot. Tests will require approval from a repository maintainer to run. @trodge, 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.
Missing test reportYour PR includes resource fields which are not covered by any test. Resource: resource "google_gkeonprem_vmware_admin_cluster" "primary" {
addon_node {
auto_resize_config {
enabled = # value needed
}
}
annotations = # value needed
anti_affinity_groups {
aag_config_disabled = # value needed
}
authorization {
viewer_users {
username = # value needed
}
}
auto_repair_config {
enabled = # value needed
}
control_plane_node {
replicas = # value needed
}
load_balancer {
f5_config {
snat_pool = # value needed
}
manual_lb_config {
addons_node_port = # value needed
control_plane_node_port = # value needed
ingress_http_node_port = # value needed
ingress_https_node_port = # value needed
konnectivity_server_node_port = # value needed
}
metal_lb_config = # value needed
}
network_config {
ha_control_plane_config {
control_plane_ip_block {
gateway = # value needed
ips {
hostname = # value needed
ip = # value needed
}
netmask = # value needed
}
}
host_config {
dns_search_domains = # value needed
dns_servers = # value needed
ntp_servers = # value needed
}
static_ip_config {
ip_blocks {
gateway = # value needed
ips {
hostname = # value needed
ip = # value needed
}
netmask = # value needed
}
}
vcenter_network = # value needed
}
platform_config {
required_platform_version = # value needed
}
prepared_secrets {
enabled = # value needed
}
vcenter {
storage_policy_name = # value needed
}
}
|
Tests analyticsTotal tests: 21 Click here to see the affected service packages
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
|
🔴 Tests failed during RECORDING mode: 🔴 Errors occurred during RECORDING mode. Please fix them to complete your PR. |
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.
Missing test reportYour PR includes resource fields which are not covered by any test. Resource: resource "google_gkeonprem_vmware_admin_cluster" "primary" {
addon_node {
auto_resize_config {
enabled = # value needed
}
}
annotations = # value needed
anti_affinity_groups {
aag_config_disabled = # value needed
}
authorization {
viewer_users {
username = # value needed
}
}
auto_repair_config {
enabled = # value needed
}
control_plane_node {
replicas = # value needed
}
load_balancer {
f5_config {
snat_pool = # value needed
}
manual_lb_config {
addons_node_port = # value needed
control_plane_node_port = # value needed
ingress_http_node_port = # value needed
ingress_https_node_port = # value needed
konnectivity_server_node_port = # value needed
}
metal_lb_config = # value needed
}
network_config {
ha_control_plane_config {
control_plane_ip_block {
gateway = # value needed
ips {
hostname = # value needed
ip = # value needed
}
netmask = # value needed
}
}
host_config {
dns_search_domains = # value needed
dns_servers = # value needed
ntp_servers = # value needed
}
static_ip_config {
ip_blocks {
gateway = # value needed
ips {
hostname = # value needed
ip = # value needed
}
netmask = # value needed
}
}
vcenter_network = # value needed
}
platform_config {
required_platform_version = # value needed
}
prepared_secrets {
enabled = # value needed
}
vcenter {
storage_policy_name = # value needed
}
}
|
Tests analyticsTotal tests: 21 Click here to see the affected service packages
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
|
🔴 Tests failed during RECORDING mode: 🔴 Errors occurred during RECORDING mode. Please fix them to complete your PR. |
@trodge This PR has been waiting for review for 3 weekdays. Please take a look! Use the label |
The API does not seem to be returning the resource fields in the GET response after the create operation finishes. The POST request looks like this:
But the GET response looks like this:
Any idea why this would be the case? |
@naitianliu-google, this PR is waiting for action from you. If no action is taken, this PR will be closed in 28 days. Please address any comments or change requests, or re-request review from a core reviewer if no action is required. This notification can be disabled with the |
Thanks for checking! This is due to an issue of our API backend. The issue has been fixed. The tests should work |
@trodge Could you please help guide how to support empty message in terraform? We have a field |
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.
Missing test reportYour PR includes resource fields which are not covered by any test. Resource: resource "google_gkeonprem_vmware_admin_cluster" "primary" {
load_balancer {
metal_lb_config {
enabled = # value needed
}
}
}
|
1 similar comment
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.
Missing test reportYour PR includes resource fields which are not covered by any test. Resource: resource "google_gkeonprem_vmware_admin_cluster" "primary" {
load_balancer {
metal_lb_config {
enabled = # value needed
}
}
}
|
Tests analyticsTotal tests: 23 Click here to see the affected service packages
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
|
🟢 Tests passed during RECORDING mode: 🟢 No issues found for passed tests after REPLAYING rerun. 🔴 Tests failed during RECORDING mode: 🔴 Errors occurred during RECORDING mode. Please fix them to complete your PR. |
Tests analyticsTotal tests: 23 Click here to see the affected service packages
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
|
🟢 Tests passed during RECORDING mode: 🟢 No issues found for passed tests after REPLAYING rerun. 🔴 Tests failed during RECORDING mode: 🔴 Errors occurred during RECORDING mode. Please fix them to complete your PR. |
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.
Missing test reportYour PR includes resource fields which are not covered by any test. Resource: resource "google_gkeonprem_vmware_admin_cluster" "primary" {
load_balancer {
metal_lb_config {
enabled = # value needed
}
}
}
|
Tests analyticsTotal tests: 23 Click here to see the affected service packages
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
|
🔴 Tests failed during RECORDING mode: 🔴 Errors occurred during RECORDING mode. Please fix them to complete your PR. |
You could try using |
@trodge This PR has been waiting for review for 3 weekdays. Please take a look! Use the label |
@GoogleCloudPlatform/terraform-team @trodge This PR has been waiting for review for 1 week. Please take a look! Use the label |
@GoogleCloudPlatform/terraform-team @trodge This PR has been waiting for review for 2 weeks. Please take a look! Use the label |
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: 23 Click here to see the affected service packages
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
|
🟢 Tests passed during RECORDING mode: 🟢 No issues found for passed tests after REPLAYING rerun. 🟢 All tests passed! |
@trodge We have fixed the all issues mentioned in the PR. Could you please help take a look? Thanks! |
@GoogleCloudPlatform/terraform-team @trodge This PR has been waiting for review for 3 weeks. Please take a look! Use the label |
@GoogleCloudPlatform/terraform-team @trodge This PR has been waiting for review for 4 weeks. Please take a look! Use the label |
@GoogleCloudPlatform/terraform-team @trodge This PR has been waiting for review for 5 weeks. Please take a look! Use the label |
@GoogleCloudPlatform/terraform-team @trodge This PR has been waiting for review for 6 weeks. Please take a look! Use the label |
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: 23 Click here to see the affected service packages
🟢 All tests passed! View the build log |
Release Note Template for Downstream PRs (will be copied)