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

feat: Update .kokoro/build.sh to run proxy when testing Cloud SQL samples #1783

Merged
merged 2 commits into from
May 7, 2020

Conversation

shubha-rajan
Copy link
Contributor

For integration testing of connectivity samples. This PR is blocking #1745 from being merged.

@shubha-rajan shubha-rajan requested a review from ace-n May 5, 2020 18:07
@shubha-rajan shubha-rajan requested a review from fhinkel as a code owner May 5, 2020 18:07
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label May 5, 2020
@ace-n
Copy link
Contributor

ace-n commented May 5, 2020

Basic idea LGTM. @shubha-rajan can you get the build passing, and ask @fhinkel to take another look once that's done?

(I'll probably be OOO later this week due to an internal COVID-19 project.)

@shubha-rajan shubha-rajan force-pushed the build-proxy branch 7 times, most recently from d3a95d4 to 7666b84 Compare May 6, 2020 02:11
@shubha-rajan shubha-rajan added the kokoro:run Add this label to force Kokoro to re-run the tests. label May 6, 2020
@kokoro-team kokoro-team removed the kokoro:run Add this label to force Kokoro to re-run the tests. label May 6, 2020
@shubha-rajan shubha-rajan added kokoro:run Add this label to force Kokoro to re-run the tests. and removed kokoro:run Add this label to force Kokoro to re-run the tests. labels May 6, 2020
@shubha-rajan shubha-rajan force-pushed the build-proxy branch 2 times, most recently from 27b4e66 to bcb9278 Compare May 6, 2020 04:18
@shubha-rajan
Copy link
Contributor Author

@fhinkel this is ready for review!

@fhinkel fhinkel added kokoro: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 May 7, 2020
@kokoro-team kokoro-team removed the kokoro:run Add this label to force Kokoro to re-run the tests. label May 7, 2020
@fhinkel fhinkel merged commit a4ac634 into GoogleCloudPlatform:master May 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge Merge the pull request once unit tests and other checks pass. cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants