Skip to content

random generation of keyring / cryptokey names - #786

Merged
dpebot merged 2 commits into
masterfrom
kms-keyring-randomly-generated
Feb 8, 2017
Merged

random generation of keyring / cryptokey names#786
dpebot merged 2 commits into
masterfrom
kms-keyring-randomly-generated

Conversation

@ryanmats

@ryanmatsryanmats commented Feb 7, 2017

Copy link
Copy Markdown
Contributor

No description provided.

@googlebotgooglebot added the cla: yes This human has signed the Contributor License Agreement. label Feb 7, 2017
Comment threadkms/api-client/snippets_test.py Outdated

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

Copy link
Copy Markdown
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_inrange(12))

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

done

@theacodes

Copy link
Copy Markdown
Contributor

@dpebot please merge when travis passes

@dpebot

Copy link
Copy Markdown
Collaborator

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

@dpebotdpebot added the automerge Merge the pull request once unit tests and other checks pass. label Feb 8, 2017
@dpebotdpebot self-assigned this Feb 8, 2017
@dpebot
dpebot merged commit 1d76ef0 into masterFeb 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 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 freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automergeMerge the pull request once unit tests and other checks pass.cla: yesThis human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@ryanmats@theacodes@dpebot@googlebot