[DO_NOT_MERGE] fix recording upload failure in nightly vcr run
Release Note Template for Downstream PRs (will be copied)
@shuyama1 I created this branch to test the recording
- it disabled fetch recording at replay, to trigger replay failure
- it only runs the service/compute
- it uploads to a temp folder
- include the fix for copy from src + "/*" to destination to see whether that helps.
Could you help setup the trigger?
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.
Done!
@iyabchen you'll also need to modify .ci/gcb-vcr-nightly.yml to use the golang version in this PR, similar to #11148. I'll create a testing trigger after.
Trigger the testing trigger
/gcbrun
@iyabchen Hey, I've set up the trigger. The current run is failing, you'll probably need to resolve the merge conflicts first. Any commits to the branch nightly-vcr-test-go-record-fix should trigger the cloud build job automatically. Please let me know if you have any questions.
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.
@shuyama1
- Tests showed that it can do the recording.
- I have reverted the debug code.
- Main change is to just print warning when upload failed instead of returning an error.
- Some template change to make it print the error if there is any.
@shuyama1 This PR has been waiting for review for 3 weekdays. Please take a look! Use the label disable-review-reminders to disable these notifications.
@GoogleCloudPlatform/terraform-team @shuyama1 This PR has been waiting for review for 1 week. Please take a look! Use the label disable-review-reminders to disable these notifications.
Just rebased to latest main
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.
Sorry for delay, just rebased and force pushed.
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.