transport: add Resource in ListCallOptions for refreshing resourceData in listPages#17543
transport: add Resource in ListCallOptions for refreshing resourceData in listPages#17543BBBmau wants to merge 1 commit into
transport: add Resource in ListCallOptions for refreshing resourceData in listPages#17543Conversation
|
Hi there, I'm the Modular magician. I've detected the following information about your changes for commit 987d800: Diff reportYour PR generated the following diffs in downstream repositories:
Test reportAnalytics
Affected Service Packages
Step 1: Replaying Mode Action takenFound 13 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 build log Step 2: Recording Mode
Caution Issues requiring attention before PR completion 🔴 Initial Recording Failed: Some tests failed during the recording step. See the table above for details. 🔴 Replaying Rerun Failed: Some tests failed due to non-determinism when VCR replayed the response. 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. |
|
Would simplify Storage Bucket PR to this: return transport_tpg.ListPages(transport_tpg.ListPagesOptions{
Config: config,
TempData: d,
Resource: ResourceStorageBucket(),
ListURL: url,
BillingProject: billingProject,
UserAgent: userAgent,
Flattener: flattenStorageBucketListItem,
Callback: callback,
})
} |
|
/gcbrun |
|
Hi there, I'm the Modular magician. I've detected the following information about your changes for commit 987d800: Diff reportYour PR generated the following diffs in downstream repositories:
Test reportAnalytics
Affected Service Packages
Step 1: Replaying Mode Action takenFound 53 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 build log Step 2: Recording Mode
Caution Issues requiring attention before PR completion 🔴 Initial Recording Failed: Some tests failed during the recording step. See the table above for details. 🔴 Replaying Rerun Failed: Some tests failed due to non-determinism when VCR replayed the response. 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. |
|
/gcbrun |
|
Hi there, I'm the Modular magician. I've detected the following information about your changes for commit 987d800: Diff reportYour PR generated the following diffs in downstream repositories:
Test reportAnalytics
Affected Service Packages
Step 1: Replaying Mode Action takenFound 22 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 build log Step 2: Recording Mode
Caution Issues requiring attention before PR completion 🔴 Initial Recording Failed: Some tests failed during the recording step. See the table above for details. 🔴 Replaying Rerun Failed: Some tests failed due to non-determinism when VCR replayed the response. 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. |
Release Note Template for Downstream PRs (will be copied)
When reviewing the following PR:
we run into an issue where the flatteners produced on the target handwritten resource do not overwrite all fields leading to failures on tests and the need to manually integrate the list call on the list resource itself instead of relying on the existing
ListPages.Resolving this would be introducing a refresh of the resourceData for every item that's in the list method response.
each item contains the seedResourceData which contains the list configure values that are set prior to the listPages call.
a new field in
ListCallOptionsis necessary in order to call the resource schema:See Write release notes for guidance.