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

Invoke the OLM pipeline from release pipeline #767

Merged
merged 1 commit into from
Nov 22, 2021

Conversation

jkremser
Copy link
Member

manually tested on my fork: https://github.com/jkremser/k8gb/actions/workflows/test.yaml

this action will send HTTP post to trigger the workflow dispatch type of pipeline. This way we will still have a way to run the pipeline manually to hot-fix issues in OLM's manifests (no need to make a release for this).

Signed-off-by: Jirka Kremser [email protected]

with:
workflow: OLM bundle and PR
token: ${{ secrets.CR_TOKEN }}
inputs: '{ "bundleVersion": "master" }'
Copy link
Collaborator

Choose a reason for hiding this comment

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

shouldn't we have a tag here?

Copy link
Member Author

@jkremser jkremser Nov 22, 2021

Choose a reason for hiding this comment

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

both would work, if master is set, it will take the code from master (which is the same as the release tag) and calculate the correct tag within the OLM action. The current tag is not available anywhere and I didn't want to do something like this or this again. But I can add that if you think it can help with readability.

Copy link
Collaborator

@k0da k0da left a comment

Choose a reason for hiding this comment

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

LGTM

@jkremser jkremser merged commit 58e0834 into k8gb-io:master Nov 22, 2021
@jkremser jkremser deleted the auto-run-olm-pipeline branch November 22, 2021 16:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants