Skip to content

Add more supported GCP KMS key ref formats in config doc#506

Merged
tekton-robot merged 1 commit into
tektoncd:mainfrom
chuangw6:update-doc-kms
Aug 4, 2022
Merged

Add more supported GCP KMS key ref formats in config doc#506
tekton-robot merged 1 commit into
tektoncd:mainfrom
chuangw6:update-doc-kms

Conversation

@chuangw6

@chuangw6 chuangw6 commented Aug 3, 2022

Copy link
Copy Markdown
Member

Prior to this commit, the config doc only mentions that the key ref
without version specified is supported. However, the key refs with
version specified are also specified. And there are two ways to specify
the version ../cryptoKeyVersions/.. (new) or ../versions/.. (old).

In this commit, we mention all support formats explicitly in the doc.

Prior to this commit, the config doc only mentions that the key ref
without version specified is supported. However, the key refs with
version specified are also specified. And there are two ways to specify
the version `../cryptoKeyVersions/..` (new) or `../versions/..` (old).

In this commit, we mention all support formats explicitly in the doc.
@tekton-robot

Copy link
Copy Markdown

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@tekton-robot tekton-robot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Aug 3, 2022
@tekton-robot tekton-robot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Aug 3, 2022
@chuangw6 chuangw6 marked this pull request as ready for review August 4, 2022 16:08
@tekton-robot tekton-robot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Aug 4, 2022
@tekton-robot tekton-robot requested review from font and mattmoor August 4, 2022 16:08
@tekton-robot

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: priyawadhwa

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@tekton-robot tekton-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Aug 4, 2022
@priyawadhwa

Copy link
Copy Markdown
Contributor

/lgtm

@tekton-robot tekton-robot added the lgtm Indicates that a PR is ready to be merged. label Aug 4, 2022
@tekton-robot tekton-robot merged commit d4b02fe into tektoncd:main Aug 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants