Skip to content

download server CA cert for tests#9086

Merged
dandhlee merged 11 commits intomainfrom
use-ca-cert
Feb 8, 2023
Merged

download server CA cert for tests#9086
dandhlee merged 11 commits intomainfrom
use-ca-cert

Conversation

@shubha-rajan
Copy link
Contributor

@shubha-rajan shubha-rajan commented Feb 3, 2023

Description

Download and use the server CA cert in SQL server tests

Note: Before submitting a pull request, please open an issue for discussion if you are not associated with Google.

Checklist

@product-auto-label product-auto-label bot added the samples Issues that are directly related to samples. label Feb 3, 2023
@shubha-rajan shubha-rajan added kokoro:force-run Add this label to force Kokoro to re-run the tests. and removed kokoro:force-run Add this label to force Kokoro to re-run the tests. labels Feb 3, 2023
@shubha-rajan shubha-rajan marked this pull request as ready for review February 7, 2023 01:06
@shubha-rajan shubha-rajan requested a review from a team February 7, 2023 01:06
@shubha-rajan shubha-rajan requested a review from a team as a code owner February 7, 2023 01:06
f"/projects/{project}/instances/{instance}/connectSettings")

resp = requests.get(url, headers=headers)
print(resp.json())
Copy link
Contributor

Choose a reason for hiding this comment

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

We probably want to remove this print statement.

@shubha-rajan shubha-rajan added automerge Merge the pull request once unit tests and other checks pass. kokoro:force-run Add this label to force Kokoro to re-run the tests. labels Feb 7, 2023
@kokoro-team kokoro-team removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Feb 8, 2023
@gcf-merge-on-green
Copy link
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.

@dandhlee dandhlee merged commit 4a288ce into main Feb 8, 2023
@dandhlee dandhlee deleted the use-ca-cert branch February 8, 2023 14:54
telpirion pushed a commit that referenced this pull request Mar 13, 2023
* download server CA cert for tests

* add note to connect_tcp

* remove raise_for_status

* debug print response

* refresh invalid credential

* linting

* Update connection_test.py

---------

Co-authored-by: Jack Wotherspoon <jackwoth@google.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: cloudsql samples Issues that are directly related to samples.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants