Skip to content
This repository was archived by the owner on Mar 31, 2026. It is now read-only.

feat: make tests run against environments other than prod - #883

Merged
gcf-merge-on-green[bot] merged 70 commits into
googleapis:mainfrom
rsaksham-dev:main_pr
Oct 21, 2022
Merged

feat: make tests run against environments other than prod#883
gcf-merge-on-green[bot] merged 70 commits into
googleapis:mainfrom
rsaksham-dev:main_pr

Conversation

@rsaksham

@rsakshamrsaksham commented Oct 7, 2022

Copy link
Copy Markdown
Contributor

Made changes that override api endpoint information while triggering builds from kokoro .
All tests against prod are still successful.
Some system tests are currently incompatible with nonprod environments and hence are marked to be skipped if endpoint is being overriden.

@rsaksham
rsaksham requested review from a teamOctober 7, 2022 16:54
@product-auto-labelproduct-auto-labelBot added size: m Pull request size is medium. api: storage Issues related to the googleapis/python-storage API. labels Oct 7, 2022
@rsakshamrsaksham changed the title Make tests run against environments other than prodfeat: make tests run against environments other than prodOct 10, 2022
andrewsg
andrewsg suggested changes Oct 13, 2022
View reviewed changes
Comment threadgoogle/cloud/storage/blob.py
Comment thread.kokoro/build.sh
Comment threadgoogle/cloud/storage/client.py Outdated
Comment threadtests/system/test__signing.py Outdated
Comment threadtests/system/test_bucket.py Outdated
Comment threadtests/system/test_client.py Outdated
andrewsg
andrewsg approved these changes Oct 17, 2022
View reviewed changes
Comment threadnoxfile.py Outdated
@andrewsgandrewsg added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Oct 17, 2022
@yoshi-kokoroyoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Oct 17, 2022
@rsaksham
Merge pull request#11from googleapis/main
f6d3bff
chore(deps): update dependency pandas to v1.5.1 (googleapis#889)
@andrewsgandrewsg added the automerge Merge the pull request once unit tests and other checks pass. label Oct 20, 2022
@gcf-merge-on-green

gcf-merge-on-greenBot commented Oct 20, 2022

Copy link
Copy Markdown
Contributor

Merge-on-green attempted to merge your PR for 6 hours, but it was not mergeable because either one of your required status checks failed, one of your required reviews was not approved, or there is a do not merge label. Learn more about your required status checks here: https://help.github.com/en/github/administering-a-repository/enabling-required-status-checks. You can remove and reapply the label to re-run the bot.

@gcf-merge-on-greengcf-merge-on-greenBot removed the automerge Merge the pull request once unit tests and other checks pass. label Oct 20, 2022
@rsaksham
Merge pull request#12from googleapis/main
15623c4
docs: streamline docs for c.g.c migration (googleapis#876)
@frankynfrankyn added kokoro:force-run Add this label to force Kokoro to re-run the tests. automerge Merge the pull request once unit tests and other checks pass. labels Oct 20, 2022
@yoshi-kokoroyoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Oct 20, 2022
@gcf-merge-on-green
gcf-merge-on-greenBot merged commit 7dfeb62 into googleapis:mainOct 21, 2022
@gcf-merge-on-greengcf-merge-on-greenBot removed the automerge Merge the pull request once unit tests and other checks pass. label Oct 21, 2022
@release-pleaserelease-pleaseBot mentioned this pull request Oct 21, 2022
Merged
@partheaparthea mentioned this pull request Oct 22, 2022
Merged
@rsaksham
rsaksham deleted the main_pr branch December 8, 2022 10:07
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

1 more reviewer
@andrewsgandrewsgandrewsg approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

api: storageIssues related to the googleapis/python-storage API.size: mPull request size is medium.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@rsaksham@andrewsg@frankyn@yoshi-kokoro