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

test: Creating dummy spanner instance for ITInstanceAdminTest#1977

Merged
gauravpurohit06 merged 7 commits intogoogleapis:mainfrom
gauravpurohit06:issue_1626_fix
Aug 22, 2022
Merged

test: Creating dummy spanner instance for ITInstanceAdminTest#1977
gauravpurohit06 merged 7 commits intogoogleapis:mainfrom
gauravpurohit06:issue_1626_fix

Conversation

@gauravpurohit06
Copy link
Copy Markdown
Contributor

Fixes #1626

@gauravpurohit06 gauravpurohit06 requested a review from a team August 17, 2022 12:15
@product-auto-label product-auto-label Bot added size: s Pull request size is small. api: spanner Issues related to the googleapis/java-spanner API. labels Aug 17, 2022
@gauravpurohit06 gauravpurohit06 added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Aug 17, 2022
@gauravpurohit06 gauravpurohit06 removed the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Aug 17, 2022
@gauravpurohit06 gauravpurohit06 changed the title test: Creating dummy spanner instance for ITInstanceAdminTest [Do not merge] test: Creating dummy spanner instance for ITInstanceAdminTest Aug 17, 2022
Copy link
Copy Markdown
Contributor

@rajatbhatta rajatbhatta left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall it looks good to me, except a comment. PTAL.

@rajatbhatta rajatbhatta requested a review from olavloite August 18, 2022 12:07
Copy link
Copy Markdown
Collaborator

@olavloite olavloite left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, with a suggestion regarding the naming of the createNewInstance / alwaysCreateNewInstance property.

Copy link
Copy Markdown
Contributor

@rajatbhatta rajatbhatta left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@rajatbhatta rajatbhatta added the owlbot:run Add this label to trigger the Owlbot post processor. label Aug 22, 2022
@gcf-owl-bot gcf-owl-bot Bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Aug 22, 2022
@gauravpurohit06 gauravpurohit06 added the owlbot:run Add this label to trigger the Owlbot post processor. label Aug 22, 2022
@gcf-owl-bot gcf-owl-bot Bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Aug 22, 2022
@gauravpurohit06 gauravpurohit06 added the owlbot:run Add this label to trigger the Owlbot post processor. label Aug 22, 2022
@gcf-owl-bot gcf-owl-bot Bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Aug 22, 2022
@gauravpurohit06 gauravpurohit06 merged commit 7f5bac6 into googleapis:main Aug 22, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

api: spanner Issues related to the googleapis/java-spanner API. size: s Pull request size is small.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ITInstanceAdminTest #update / #updateViaEntity fail when two tests are running simultaneously

3 participants