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

document secretrefs for build webhook triggers #6552

Merged
merged 1 commit into from Jan 3, 2018
Merged

document secretrefs for build webhook triggers #6552

merged 1 commit into from Jan 3, 2018

Conversation

bparees
Copy link
Contributor

@bparees bparees commented Nov 29, 2017

@openshift-ci-robot openshift-ci-robot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Nov 29, 2017
name: mysecret
creationTimestamp:
data:
WebHookSecretKey: c2VjcmV0dmFsdWUx
Copy link

Choose a reason for hiding this comment

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

Where does this data come from?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

it's the base64 encoded value of the webhook secret. just like any k8s secret key/value.

$ base64 -d
c2VjcmV0dmFsdWUx
^D
secretvalue1

Copy link

Choose a reason for hiding this comment

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

OK, thanks for reply : )

@bparees
Copy link
Contributor Author

bparees commented Jan 2, 2018

@openshift/team-documentation ptal, ready for merge.

@bmcelvee bmcelvee self-assigned this Jan 3, 2018

[source,yaml]
----
type: "GitHub"
github:
secret: "secret101"
secretRefence: "mysecret"
Copy link
Contributor

Choose a reason for hiding this comment

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

@bparees, should this be: s/secretRefence/secretReference? (all of the following are the same, so I just wanted to check)

Otherwise LGTM!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yes, should be. fixed, thanks!

Copy link
Contributor

Choose a reason for hiding this comment

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

Thank you! Merging now :)

@bmcelvee bmcelvee added this to the Future Release milestone Jan 3, 2018
@bmcelvee bmcelvee merged commit 5bed74c into openshift:master Jan 3, 2018
bmcelvee pushed a commit to bmcelvee/openshift-docs that referenced this pull request Jan 3, 2018
bmcelvee pushed a commit to bmcelvee/openshift-docs that referenced this pull request Jan 3, 2018
bmcelvee pushed a commit to bmcelvee/openshift-docs that referenced this pull request Jan 3, 2018
@bmcelvee
Copy link
Contributor

bmcelvee commented Jan 4, 2018

[rev_history]
|xref:../dev_guide/builds/triggering_builds.adoc#dev-guide-triggering-builds[Triggering Builds]
|Added content describing secretReference for build webhook triggers in the xref:../dev_guide/builds/triggering_builds.adoc#webhook-triggers[Webhook Triggers] section.
%

@bparees bparees deleted the webhooksecrets branch February 6, 2018 21:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
branch/dedicated branch/enterprise-3.9 branch/online size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants