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

random generation of keyring / cryptokey names #786

Merged
merged 2 commits into from
Feb 8, 2017

Conversation

ryanmats
Copy link
Contributor

@ryanmats ryanmats commented Feb 7, 2017

No description provided.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Feb 7, 2017
@ryanmats ryanmats requested a review from theacodes February 7, 2017 23:04
@@ -22,10 +25,14 @@
LOCATION = 'global'

# Your Google Cloud Platform KeyRing name
KEYRING = 'sample-keyring-43'
KEYRING = ''
Copy link
Contributor

Choose a reason for hiding this comment

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

Here's a one-liner for you

KEYRING = ''.join(random.choice(string.ascii_lowercase + string.digits) for _ in range(12))

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

@theacodes
Copy link
Contributor

@dpebot please merge when travis passes

@dpebot
Copy link
Collaborator

dpebot commented Feb 8, 2017

Okay! I'll merge when all statuses are green and all reviewers approve.

@dpebot dpebot added the automerge Merge the pull request once unit tests and other checks pass. label Feb 8, 2017
@dpebot dpebot self-assigned this Feb 8, 2017
@dpebot dpebot merged commit 1d76ef0 into master Feb 8, 2017
@dpebot dpebot deleted the kms-keyring-randomly-generated branch February 8, 2017 22:41
busunkim96 pushed a commit to googleapis/python-kms that referenced this pull request Jun 4, 2020
…atform/python-docs-samples#786)

* random generation of keyring / cryptokey names

* Fixed formatting of keyring name and cryptokey name
rsamborski pushed a commit that referenced this pull request Nov 8, 2022
* random generation of keyring / cryptokey names

* Fixed formatting of keyring name and cryptokey name
rsamborski pushed a commit that referenced this pull request Nov 8, 2022
* random generation of keyring / cryptokey names

* Fixed formatting of keyring name and cryptokey name
rsamborski pushed a commit that referenced this pull request Nov 11, 2022
* random generation of keyring / cryptokey names

* Fixed formatting of keyring name and cryptokey name
rsamborski pushed a commit that referenced this pull request Nov 14, 2022
* random generation of keyring / cryptokey names

* Fixed formatting of keyring name and cryptokey name
dandhlee pushed a commit that referenced this pull request Nov 14, 2022
* random generation of keyring / cryptokey names

* Fixed formatting of keyring name and cryptokey name
parthea pushed a commit to googleapis/google-cloud-python that referenced this pull request Oct 21, 2023
…atform/python-docs-samples#786)

* random generation of keyring / cryptokey names

* Fixed formatting of keyring name and cryptokey name
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.

4 participants