Skip to content

Add migration center import job resource and tests#17577

Open
slevenick wants to merge 1 commit into
GoogleCloudPlatform:mainfrom
slevenick:migration-center-import-job
Open

Add migration center import job resource and tests#17577
slevenick wants to merge 1 commit into
GoogleCloudPlatform:mainfrom
slevenick:migration-center-import-job

Conversation

@slevenick
Copy link
Copy Markdown
Contributor

Release Note Template for Downstream PRs (will be copied)

See Write release notes for guidance.

`google_migration_center_import_job`

@modular-magician
Copy link
Copy Markdown
Collaborator

modular-magician commented May 15, 2026

Hi there, I'm the Modular magician. I've detected the following information about your changes for commit 8b42205:

Diff report

Your PR generated the following diffs in downstream repositories:

Repository Diff Link Changes
google provider View Diff 5 files changed, 3668 insertions(+)
google-beta provider View Diff 5 files changed, 3668 insertions(+)
terraform-google-conversion View Diff 1 file changed, 1038 insertions(+)
Open in Cloud Shell View Diff 4 files changed, 116 insertions(+)

Missing test report

Your PR includes resource fields which are not covered by any test.

Resource: google_migration_center_import_job (1 total tests)
Please add an acceptance test which includes these fields. The test should include the following:

resource "google_migration_center_import_job" "primary" {
  execution_report {
    execution_errors {
      file_validations {
        file_errors {
          error_details = # value needed
          severity      = # value needed
        }
        file_name      = # value needed
        partial_report = # value needed
        row_errors {
          archive_error {
            csv_error {
              row_number = # value needed
            }
          }
          csv_error {
            row_number = # value needed
          }
          errors {
            error_details = # value needed
            severity      = # value needed
          }
          row_number = # value needed
          vm_name    = # value needed
          vm_uuid    = # value needed
          xlsx_error {
            row_number = # value needed
            sheet      = # value needed
          }
        }
      }
      job_errors {
        error_details = # value needed
        severity      = # value needed
      }
    }
    frames_reported = # value needed
  }
  validation_report {
    file_validations {
      file_errors {
        error_details = # value needed
        severity      = # value needed
      }
      file_name      = # value needed
      partial_report = # value needed
      row_errors {
        archive_error {
          csv_error {
            row_number = # value needed
          }
        }
        csv_error {
          row_number = # value needed
        }
        errors {
          error_details = # value needed
          severity      = # value needed
        }
        row_number = # value needed
        vm_name    = # value needed
        vm_uuid    = # value needed
        xlsx_error {
          row_number = # value needed
          sheet      = # value needed
        }
      }
    }
    job_errors {
      error_details = # value needed
      severity      = # value needed
    }
  }
}

Test report

Analytics

Total Tests Passed Skipped Affected
7 6 0 1
Affected Service Packages
  • migrationcenter

Learn how VCR tests work


Step 1: Replaying Mode

Action taken

Found 1 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected tests
  • TestAccMigrationCenterImportJob_migrationCenterImportJobBasicExample

View the build log


Step 2: Recording Mode

Recording Mode Replaying Rerun Test Name
❌ Error · Log - TestAccMigrationCenterImportJob_migrationCenterImportJobBasicExample

Caution

Issues requiring attention before PR completion

🔴 Initial Recording Failed: Some tests failed during the recording step. See the table above for details.

Please address these issues to complete your PR. If you believe these detections are incorrect or unrelated to your change, please raise the concern with your reviewer.

View the build log or the debug logs folder for detailed results.

@slevenick VCR tests complete for 8b42205!

@slevenick
Copy link
Copy Markdown
Contributor Author

Depends on #17578

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants