networkservices: add list resources - #18630
Conversation
|
Hi there, I'm the Modular magician. I've detected the following information about your changes for commit 05ecff1: Diff reportYour PR generated the following diffs in downstream repositories:
Test reportAnalytics
Affected Service Packages
Step 1: Replaying Mode 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
View the replaying VCR build log Step 2: Recording Mode Caution Panic occurred during VCR tests 🔴 RECORDING mode: The provider crashed with a panic. Please check the build log for details. Please fix the issue to complete your PR. View the build log |
…tructureExascaleConfig (P-18 - no list endpoint)
…ope mismatch (P-17) Removed generate_list_resource from 12 resources that have hardcoded /locations/global/ in base_url or location parameter with default global. These resources cannot support list resources without adding a location parameter, which would be a breaking change per ABSOLUTE PROHIBITIONS. Failing tests all showed 'no query results found after filtering' due to P-17 scope mismatch - the list query searches a different scope than where the resource was created. Resources removed: - HttpRoute, EndpointPolicy, TcpRoute, EdgeCacheKeyset, EdgeCacheOrigin, EdgeCacheService (hardcoded /locations/global/ in base_url) - Gateway, Mesh, GrpcRoute, TlsRoute, ServiceLbPolicies, AgentGateway (location parameter with default global or scope mismatch)
|
Hi there, I'm the Modular magician. I've detected the following information about your changes for commit 6cd4112: Diff reportYour PR generated the following diffs in downstream repositories:
Step 1: Replaying Mode Caution Build Failure during VCR tests 🔴 REPLAYING mode: The following packages failed to build:
Please fix the compilation errors to complete your PR. View the build log |
|
Hi there, I'm the Modular magician. I've detected the following information about your changes for commit 1fdc68a: Diff reportYour PR generated the following diffs in downstream repositories:
Step 1: Replaying Mode Caution Build Failure during VCR tests 🔴 REPLAYING mode: The following packages failed to build:
Please fix the compilation errors to complete your PR. View the build log |
|
Hi there, I'm the Modular magician. I've detected the following information about your changes for commit 47bb21b: Diff reportYour PR generated the following diffs in downstream repositories:
Step 1: Replaying Mode Caution Build Failure during VCR tests 🔴 REPLAYING mode: The following packages failed to build:
Please fix the compilation errors to complete your PR. View the build log |
|
Hi there, I'm the Modular magician. I've detected the following information about your changes for commit bcf09b5: Diff reportYour PR generated the following diffs in downstream repositories:
Step 1: Replaying Mode Caution Build Failure during VCR tests 🔴 REPLAYING mode: The following packages failed to build:
Please fix the compilation errors to complete your PR. View the build log |
The query_test_file.go.tmpl template was missing the fmt import, causing compile errors in generated list query tests across multiple packages including oracledatabase. Added fmt to the import block to fix the undefined: fmt errors.
|
Hi there, I'm the Modular magician. I've detected the following information about your changes for commit 1c3d9f3: Diff reportYour PR generated the following diffs in downstream repositories:
Step 1: Replaying Mode Caution Build Failure during VCR tests 🔴 REPLAYING mode: The following packages failed to build:
Please fix the compilation errors to complete your PR. View the build log |
|
Hi there, I'm the Modular magician. I've detected the following information about your changes for commit 07baf06: Diff reportYour PR generated the following diffs in downstream repositories:
Step 1: Replaying Mode Caution Build Failure during VCR tests 🔴 REPLAYING mode: The following packages failed to build:
Please fix the compilation errors to complete your PR. View the build log |
|
Hi there, I'm the Modular magician. I've detected the following information about your changes for commit da9dd0a: Diff reportYour PR generated the following diffs in downstream repositories:
Step 1: Replaying Mode Caution Build Failure during VCR tests 🔴 REPLAYING mode: The following packages failed to build:
Please fix the compilation errors to complete your PR. View the build log |
|
Hi there, I'm the Modular magician. I've detected the following information about your changes for commit 6010bf2: Diff reportYour PR generated the following diffs in downstream repositories:
Step 1: Replaying Mode Caution Build Failure during VCR tests 🔴 REPLAYING mode: The following packages failed to build:
Please fix the compilation errors to complete your PR. View the build log |
|
Hi there, I'm the Modular magician. I've detected the following information about your changes for commit 0744b93: Diff reportYour PR generated the following diffs in downstream repositories:
Test reportAnalytics
Affected Service Packages
Step 1: Replaying Mode 🟢 All tests passed in Replaying mode! No Recording was needed. View the replaying VCR build log |
c2146ae to
2d92336
Compare
|
Hi there, I'm the Modular magician. I've detected the following information about your changes for commit c2146ae: Diff reportYour PR generated the following diffs in downstream repositories:
Test reportAnalytics
Affected Service Packages
Step 1: Replaying Mode 🟢 All tests passed in Replaying mode! No Recording was needed. View the replaying VCR build log |
|
Hi there, I'm the Modular magician. I've detected the following information about your changes for commit 6b73bc3: Diff reportYour PR generated the following diffs in downstream repositories:
Test reportAnalytics
Affected Service Packages
Step 1: Replaying Mode Action takenFound 19 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected tests
View the replaying VCR build log Step 2: Recording Mode Caution Panic occurred during VCR tests 🔴 RECORDING mode: The provider crashed with a panic. Please check the build log for details. Please fix the issue to complete your PR. View the build log @BBBmau, @malhotrasagar2212 VCR tests complete for 6b73bc3! |
6b73bc3 to
c2146ae
Compare
|
Hi there, I'm the Modular magician. I've detected the following information about your changes for commit c2146ae: Diff reportYour PR generated the following diffs in downstream repositories:
Test reportAnalytics
Affected Service Packages
Step 1: Replaying Mode 🟢 All tests passed in Replaying mode! No Recording was needed. View the replaying VCR build log @BBBmau, @malhotrasagar2212 VCR tests complete for c2146ae! |
Adds list-resource generation for the following networkservices resources:
google_networkservices_authz_extensiongoogle_networkservices_multicast_consumer_associationgoogle_networkservices_multicast_domaingoogle_networkservices_multicast_domain_activationgoogle_networkservices_multicast_domain_groupgoogle_networkservices_multicast_group_consumer_activationgoogle_networkservices_multicast_group_producer_activationgoogle_networkservices_multicast_group_rangegoogle_networkservices_multicast_group_range_activationgoogle_networkservices_multicast_producer_associationTests: Acceptance tests PASSED (confirmed by Validator agent).