Skip to content
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

Refactor muxing 2 : Remove unused code, resulting from #11903 #11578

Draft
wants to merge 15 commits into
base: main
Choose a base branch
from

Conversation

SarahFrench
Copy link
Collaborator

@SarahFrench SarahFrench commented Aug 29, 2024

Follow up to #11903. Most commits in this PR are in that PR, and will 'disappear' once this PR is unblocked and ready for review.

This PR deletes all the parallel implementations of provider configuration logic. The PR #11903 stops the provider using this duplicated logic, and this PR removes that redundant code.

In future the team may want to revisit this deleted code to inform work to pulls out the SDKv2 from the codebase completely and the provider becomes 100% plugin-framework. As that is a long way in the future I think it's best to delete this code now for clarity. It will still be in version control for reference in the future.

Release Note Template for Downstream PRs (will be copied)


@modular-magician

This comment was marked as outdated.

@modular-magician

This comment was marked as outdated.

@modular-magician

This comment was marked as outdated.

@SarahFrench SarahFrench force-pushed the mux-refactor-5-delete-unused-code branch from 43bd0a6 to 47bb9bc Compare August 29, 2024 17:31
@modular-magician

This comment was marked as outdated.

@SarahFrench SarahFrench force-pushed the mux-refactor-5-delete-unused-code branch from 5e8070f to 47bb9bc Compare August 29, 2024 17:54
@modular-magician

This comment was marked as outdated.

1 similar comment
@modular-magician

This comment was marked as outdated.

@modular-magician

This comment was marked as outdated.

@modular-magician

This comment was marked as outdated.

@modular-magician

This comment was marked as outdated.

1 similar comment
@modular-magician

This comment was marked as outdated.

@modular-magician

This comment was marked as outdated.

@modular-magician

This comment was marked as outdated.

@SarahFrench SarahFrench force-pushed the mux-refactor-5-delete-unused-code branch from 47bb9bc to 98bba88 Compare September 23, 2024 15:52
@modular-magician

This comment was marked as outdated.

@modular-magician

This comment was marked as outdated.

@SarahFrench SarahFrench force-pushed the mux-refactor-5-delete-unused-code branch from 98bba88 to 265c2f0 Compare September 26, 2024 11:06
@SarahFrench
Copy link
Collaborator Author

Rebased to pull in Go rewrite

…ruct

This allows the PF provider to access any data on the SDK provider, including the meta/Config struct that will be created when the SDK provider is configured.
…struct to all data sources and resources, instead of `FrameworkProviderConfig`
… Config struct to be usable in those places.

- Replace use of FrameworkProviderConfig with Config
- Add Beta-only `NewFirebaseClient` method to Config struct for use with Firebase data sources
- Update LocationDescriber interface
- Misc places where the SDK and PF type systems meet and string needs to be converted to types.StringType
- This carries over the idea of configuring the PF provider using the SDK provider. The changes in the MuxedProviders func mimic changes already made in main.go.
- Remove unnecessary duplication of cached configs per test name; now one used regardless of PF/SDK
- Updates to some DestroyProducer functions so they access the cached SDK Config struct to get a client
- Remove GetFwTestProvider and the file containing it - this isn't needed.
…ovider-google#14158

- Use of older TPG versions through ExternalProviders breaks VCR, so that is also removed from TestAccDataSourceGoogleFirebaseAppleAppConfig
…e now ignored and ENVs can be used instead
…el_addition_strategy` acctest to reflect how the SDK configuration logic now affects the PF provider
…se of ExternalProviders to get other versions of TPG/TPGB
@SarahFrench SarahFrench force-pushed the mux-refactor-5-delete-unused-code branch from 265c2f0 to dee2be2 Compare October 4, 2024 11:49
@SarahFrench SarahFrench changed the title Refactor muxing 2 : Remove unused code, resulting from #11577 Refactor muxing 2 : Remove unused code, resulting from #11903 Oct 4, 2024
@modular-magician
Copy link
Collaborator

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.

google provider: Diff ( 25 files changed, 171 insertions(+), 3891 deletions(-))
google-beta provider: Diff ( 31 files changed, 215 insertions(+), 4129 deletions(-))

@modular-magician
Copy link
Collaborator

Tests analytics

Total tests: 4139
Passed tests: 3721
Skipped tests: 411
Affected tests: 7

Click here to see the affected service packages

All service packages are affected

Action taken

Found 7 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected tests
  • TestAccDataformRepositoryReleaseConfig_dataformRepositoryReleaseConfigExample
  • TestAccDataformRepositoryWorkflowConfig_dataformRepositoryWorkflowConfigExample
  • TestAccDataformRepository_dataformRepositoryWithCloudsourceRepoAndSshExample
  • TestAccDataformRepository_updated
  • TestAccDataprocCluster_withAutoscalingPolicy
  • TestAccFrameworkProviderBasePath_setBasePath
  • TestAccFrameworkProviderMeta_setModuleName

Get to know how VCR tests work

@modular-magician
Copy link
Collaborator

🟢 Tests passed during RECORDING mode:
TestAccFrameworkProviderBasePath_setBasePath[Debug log]
TestAccFrameworkProviderMeta_setModuleName[Debug log]

🟢 No issues found for passed tests after REPLAYING rerun.


🔴 Tests failed during RECORDING mode:
TestAccDataformRepositoryReleaseConfig_dataformRepositoryReleaseConfigExample[Error message] [Debug log]
TestAccDataformRepositoryWorkflowConfig_dataformRepositoryWorkflowConfigExample[Error message] [Debug log]
TestAccDataformRepository_dataformRepositoryWithCloudsourceRepoAndSshExample[Error message] [Debug log]
TestAccDataformRepository_updated[Error message] [Debug log]
TestAccDataprocCluster_withAutoscalingPolicy[Error message] [Debug log]

🔴 Errors occurred during RECORDING mode. Please fix them to complete your PR.

View the build log or the debug log for each test

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