add: vertex ia pipeline job resource#15513
add: vertex ia pipeline job resource#15513victorsantos-cit wants to merge 25 commits intoGoogleCloudPlatform:mainfrom
Conversation
Removed location and datasetId variables from PipelineJob.
|
Hello! I am a robot. Tests will require approval from a repository maintainer to run. Googlers: For automatic test runs see go/terraform-auto-test-runs. @hao-nan-li, 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_vertex_ai_pipeline_job" "primary" {
network = # value needed
original_pipeline_job_id = # value needed
pipeline_spec {
additional_properties = # value needed
}
preflight_validations = # value needed
psc_interface_config {
raw = # value needed
}
region = # value needed
reserved_ip_ranges = # value needed
satisfies_pzi = # value needed
service_account = # value needed
}
Missing service labelsThe following new resources do not have corresponding service labels:
If you believe this detection to be incorrect please raise the concern with your reviewer. Googlers: This error is safe to ignore once you've completed go/fix-missing-service-labels. |
Tests analyticsTotal tests: 79 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. |
Added Google project resource and updated pipeline job configuration.
|
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 service labelsThe following new resources do not have corresponding service labels:
If you believe this detection to be incorrect please raise the concern with your reviewer. Googlers: This error is safe to ignore once you've completed go/fix-missing-service-labels. |
Tests analyticsTotal tests: 79 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. |
|
@hao-nan-li cound you please send the error |
--- FAIL: TestAccVertexAIPipelineJob_vertexAiPipelineJobMinimalExample (0.43s) |
|
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 service labelsThe following new resources do not have corresponding service labels:
If you believe this detection to be incorrect please raise the concern with your reviewer. Googlers: This error is safe to ignore once you've completed go/fix-missing-service-labels. |
Tests analyticsTotal tests: 79 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. |
|
@hao-nan-li cound you please send the error |
Same as the previous one |
|
But, what kind of resource need version, because , i put in the create project, there is the only one the dont haved the provide versin |
|
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_vertex_ai_pipeline_job" "primary" {
display_name = # value needed
labels = # value needed
network = # value needed
original_pipeline_job_id = # value needed
pipeline_spec {
additional_properties = # value needed
}
preflight_validations = # value needed
psc_interface_config {
raw = # value needed
}
region = # value needed
reserved_ip_ranges = # value needed
runtime_config {
gcs_output_directory = # value needed
}
satisfies_pzi = # value needed
service_account = # value needed
template_uri = # value needed
}
Missing service labelsThe following new resources do not have corresponding service labels:
If you believe this detection to be incorrect please raise the concern with your reviewer. Googlers: This error is safe to ignore once you've completed go/fix-missing-service-labels. |
Tests analyticsTotal tests: 83 Click here to see the affected service packages
🟢 All tests passed! View the build log |
|
Hey @hao-nan-li could you please tell me whats is wrong if the terraform-provider-missing-service-labels |
|
@GoogleCloudPlatform/terraform-team @hao-nan-li This PR has been waiting for review for 16 weeks. Please take a look! Use the label |
|
Hey @hao-nan-li could you please tell me whats is wrong if the terraform-provider-missing-service-labels |
Updated pipelineSpec and pscInterfaceConfig properties with new descriptions and types.
Removed optional PSC-I configuration from PipelineJob.
|
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_vertex_ai_pipeline_job" "primary" {
display_name = # value needed
labels = # value needed
network = # value needed
original_pipeline_job_id = # value needed
pipeline_spec = # value needed
preflight_validations = # value needed
psc_interface_config = # value needed
region = # value needed
reserved_ip_ranges = # value needed
runtime_config {
gcs_output_directory = # value needed
}
satisfies_pzi = # value needed
service_account = # value needed
template_uri = # value needed
}
Missing service labelsThe following new resources do not have corresponding service labels:
If you believe this detection to be incorrect please raise the concern with your reviewer. Googlers: This error is safe to ignore once you've completed go/fix-missing-service-labels. |
Tests analyticsTotal tests: 84 Click here to see the affected service packages
🟢 All tests passed! View the build log |
|
@GoogleCloudPlatform/terraform-team @hao-nan-li This PR has been waiting for review for 17 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.
Missing test reportYour PR includes resource fields which are not covered by any test. Resource: resource "google_vertex_ai_pipeline_job" "primary" {
display_name = # value needed
labels = # value needed
network = # value needed
original_pipeline_job_id = # value needed
pipeline_spec = # value needed
preflight_validations = # value needed
psc_interface_config = # value needed
region = # value needed
reserved_ip_ranges = # value needed
runtime_config {
gcs_output_directory = # value needed
}
satisfies_pzi = # value needed
service_account = # value needed
template_uri = # value needed
}
Missing service labelsThe following new resources do not have corresponding service labels:
If you believe this detection to be incorrect please raise the concern with your reviewer. Googlers: This error is safe to ignore once you've completed go/fix-missing-service-labels. |
Tests analyticsTotal tests: 84 Click here to see the affected service packages
🟢 All tests passed! View the build log |
|
@GoogleCloudPlatform/terraform-team @hao-nan-li This PR has been waiting for review for 18 weeks. Please take a look! Use the label |
|
@GoogleCloudPlatform/terraform-team @hao-nan-li This PR has been waiting for review for 19 weeks. Please take a look! Use the label |
|
@GoogleCloudPlatform/terraform-team @hao-nan-li This PR has been waiting for review for 20 weeks. Please take a look! Use the label |
|
@hao-nan-li hey could you please tell me whats is the current error |
|
@GoogleCloudPlatform/terraform-team @hao-nan-li This PR has been waiting for review for 21 weeks. Please take a look! Use the label |
Hello Folks.
This PR is a new implementation and to fix hashicorp/terraform-provider-google#23290
Release Note Template for Downstream PRs (will be copied)
See Write release notes for guidance.