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

Fix issues with vcr status and logs #9987

Merged
merged 2 commits into from
Feb 15, 2024

Conversation

trodge
Copy link
Contributor

@trodge trodge commented Feb 14, 2024

Fixes an issue with replaying after recording logs getting improperly uploaded and test state being improperly reported.

Release Note Template for Downstream PRs (will be copied)


@trodge trodge requested a review from a team as a code owner February 14, 2024 21:34
@trodge trodge requested review from c2thorn and removed request for a team February 14, 2024 21:34
@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 hasn't generated any diffs, but I'll let you know if a future commit does.

@shuyama1
Copy link
Member

shuyama1 commented Feb 14, 2024

I could be wrong, but it seems we'll mistakenly mark the VCR status as success if there're some tests failed in recording but only because we have passed ones run with no issues in re-replaying

@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 hasn't generated any diffs, but I'll let you know if a future commit does.

@trodge trodge merged commit 1253916 into GoogleCloudPlatform:main Feb 15, 2024
12 checks passed
@trodge trodge deleted the magician-test-vcr branch February 15, 2024 21:13
tdbhacks pushed a commit to tdbhacks/magic-modules that referenced this pull request Feb 23, 2024
* Fix issues with vcr status and logs

* Report success if recording succeeds, not replaying after recording
BBBmau pushed a commit to BBBmau/magic-modules that referenced this pull request Mar 6, 2024
* Fix issues with vcr status and logs

* Report success if recording succeeds, not replaying after recording
balanaguharsha pushed a commit to balanaguharsha/magic-modules that referenced this pull request Apr 19, 2024
* Fix issues with vcr status and logs

* Report success if recording succeeds, not replaying after recording
balanaguharsha pushed a commit to balanaguharsha/magic-modules that referenced this pull request May 2, 2024
* Fix issues with vcr status and logs

* Report success if recording succeeds, not replaying after recording
pengq-google pushed a commit to pengq-google/magic-modules that referenced this pull request May 21, 2024
* Fix issues with vcr status and logs

* Report success if recording succeeds, not replaying after recording
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants