Skip to content

make prow works mostly - #209

Merged
0xmichalis merged 1 commit into
openshift:masterfrom
0xmichalis:make-prow
Oct 2, 2017
Merged

0xmichalis merged 1 commit into
openshift:masterfrom
0xmichalis:make-prow

Conversation

@0xmichalis

Copy link
Copy Markdown
Contributor

No description provided.

Signed-off-by: Michalis Kargakis <mkargaki@redhat.com>
@0xmichalis
0xmichalis merged commit 4089f31 into openshift:master Oct 2, 2017
@0xmichalis
0xmichalis deleted the make-prow branch October 2, 2017 16:00
@openshift-ci-robot openshift-ci-robot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Oct 2, 2017
Comment thread Makefile
oc create secret generic jenkins-tokens --from-literal=basic=${BASIC_AUTH_PASS} --from-literal=origin-bearer=${BEARER_TOKEN} -o yaml --dry-run | oc apply -n ci -f -
# HMAC_TOKEN is used for encrypting Github webhook payloads.
oc create secret generic hmac-token --from-literal=hmac=${HMAC_TOKEN} -o yaml --dry-run | oc apply -f -
oc create secret generic hmac-token --from-literal=hmac=${HMAC_TOKEN} -o yaml --dry-run | oc apply -n ci -f -

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.

not sure -n ci is appropriate here, would make local testing of this hard

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yeah, not ideal but better than not defining the namespace and letting the config be created successfully in something that is not ci.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Will push a fix that parameterizes this in the Makefile today or tomorrow.

derekhiggins pushed a commit to derekhiggins/release that referenced this pull request Oct 24, 2023
Use current rdo ironic python agent tar
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants