Made breaking change detector own its own setup and build process

[melinath]

This is the first step towards making this a single diff-processor library that will implement all diff-time tooling.

b/296599247

[modular-magician]

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

Diff report

Your PR hasn't generated any diffs, but I'll let you know if a future commit does.

[ScottSuarez]

Since we no longer base the breaking-change-detector unit tests off the customer's change we will likely want to communicate failure of the tool to execute/run.

We've had instances in the past where the breaking-change-detector would fail silently due to the package structure changes.

[modular-magician]

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

Breaking Change Detection Failed

The breaking change detector crashed during execution. This is usually due to the downstream provider(s) failing to compile. Please investigate or follow up with your reviewer.

Diff report

Your PR generated some diffs in downstreams - here they are.

Terraform GA: Diff ( 2 files changed, 86 deletions(-))
Terraform Beta: Diff ( 2 files changed, 86 deletions(-))
TF Conversion: Diff ( 1 file changed, 29 deletions(-))

[modular-magician]

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

Breaking Change Detection Failed

The breaking change detector crashed during execution. This is usually due to the downstream provider(s) failing to compile. Please investigate or follow up with your reviewer.

Diff report

Your PR generated some diffs in downstreams - here they are.

Terraform GA: Diff ( 2 files changed, 86 deletions(-))
Terraform Beta: Diff ( 2 files changed, 86 deletions(-))
TF Conversion: Diff ( 1 file changed, 29 deletions(-))

[modular-magician]

Tests analytics

Total tests: 2987
Passed tests 2688
Skipped tests: 296
Affected tests: 3

Action taken

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

TestAccAlloydbBackup_usingCMEK|TestAccBigtableAppProfile_bigtableAppProfileMulticlusterExample|TestAccDataprocJobIamPolicy

Get to know how VCR tests work

[modular-magician]

$\textcolor{green}{\textsf{Tests passed during RECORDING mode:}}$
TestAccBigtableAppProfile_bigtableAppProfileMulticlusterExample[Debug log]
TestAccDataprocJobIamPolicy[Debug log]

Rerun these tests in REPLAYING mode to catch issues

$\textcolor{green}{\textsf{No issues found for passed tests after REPLAYING rerun.}}$

---

$\textcolor{red}{\textsf{Tests failed during RECORDING mode:}}$
TestAccAlloydbBackup_usingCMEK[Error message] [Debug log]

$\textcolor{red}{\textsf{Please fix these to complete your PR.}}$
View the build log or the debug log for each test

[modular-magician]

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

Diff report

Your PR generated some diffs in downstreams - here they are.

Terraform GA: Diff ( 2 files changed, 86 deletions(-))
Terraform Beta: Diff ( 2 files changed, 86 deletions(-))
TF Conversion: Diff ( 1 file changed, 29 deletions(-))

[modular-magician]

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

Breaking Change(s) Detected

The following breaking change(s) were detected within your pull request.

• Field encryption_config.kms_key_name within resource google_alloydb_backup was either removed or renamed - reference
• Field encryption_config within resource google_alloydb_backup was either removed or renamed - reference

If you believe this detection to be incorrect please raise the concern with your reviewer. If you intend to make this change you will need to wait for a major release window. An override-breaking-change label can be added to allow merging.

Diff report

Your PR generated some diffs in downstreams - here they are.

Terraform GA: Diff ( 2 files changed, 86 deletions(-))
Terraform Beta: Diff ( 2 files changed, 86 deletions(-))
TF Conversion: Diff ( 1 file changed, 29 deletions(-))

[melinath]

@ScottSuarez that ended up taking a few more rounds than I was expecting to get right - could you take another look and verify the manual tests seem okay?

[ScottSuarez]

What happens if these folders don't exist? will the command still succeed?

[melinath]

no - you need to run make clone first (or set them up some other way.) See README changes for more details

[melinath]

happy to address in a follow-on PR if you have concerns with this approach

[ScottSuarez]

What happens if these folders don't exist? will the command still succeed?

[melinath]

same as above

[ScottSuarez]

Things look good from my perspective, I do have some outstanding questions about the MAKE failure scenarios but you can always address that in sequential PR

[modular-magician]

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

Diff report

Your PR hasn't generated any diffs, but I'll let you know if a future commit does.

[modular-magician]

Tests analytics

Total tests: 2987
Passed tests 2691
Skipped tests: 296
Affected tests: 0

$\textcolor{green}{\textsf{All tests passed in REPLAYING mode.}}$
View the build log