Moved some clients from config.go to individual service packages#17276
Moved some clients from config.go to individual service packages#17276melinath merged 4 commits intoGoogleCloudPlatform:mainfrom
Conversation
… respective packages Also removed unused Cloud Build client
|
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: 6344 Click here to see the affected service packages
Action takenFound 24 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. |
|
VCR failures look unrelated. |
|
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.
|
|
TGC failures are unrelated; moving forward with merge since that's all that changed in the last commit. |
bd4ed8b
Tests analyticsTotal tests: 6344 Click here to see the affected service packages
Action takenFound 12 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. |
This moves a few clients from config.go to their individual service packages, reducing the amount of global code & simplifying a future move to using service packages as the location for BaseUrl computation (instead of global computation in config.go)
Release Note Template for Downstream PRs (will be copied)
See Write release notes for guidance.