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

CI: Make mandrel.yml trigger on workflow_dispatch #240

Merged
merged 1 commit into from
Apr 21, 2021

Conversation

zakkak
Copy link
Collaborator

@zakkak zakkak commented Apr 21, 2021

This allows us to run the workflow on demand and on schedule with different parameters.

For the currently scheduled builds see nightly.yml

With gh (aka github CLI) we can also trigger tests from the command line like this:

# Use latest builder image of Mandrel 20.3 from quay,io and test with Quarkus main
gh workflow -R zakkak/mandrel run base.yml --ref ci-refactoring \
    -f quarkus-version=main -f builder-image=quay.io/quarkus/ubi-quarkus-mandrel:20.3-java11
# Download GraalVM CE 21.1.0 and test with Quarkus main
gh workflow -R zakkak/mandrel run base.yml --ref ci-refactoring \
    -f quarkus-version=main -f version=vm-21.1.0 -f distribution=graalvm -f build-from-source=false
# Download Mandrel 21.0.0.0-Final and test with Quarkus main
gh workflow -R zakkak/mandrel run base.yml --ref ci-refactoring \
    -f quarkus-version=main -f version=mandrel-21.0.0.0-Final -f build-from-source=false
# Default: build graal/master with mandrel-packaging and test with Quarkus main
gh workflow -R zakkak/mandrel run base.yml --ref ci-refactoring
# etc.

Relates to: quarkusio/quarkus#16631

This allows us to run the workflow on demand and on schedule with
different parameters.
@zakkak zakkak added the testing CI and testing related issues and PRs label Apr 21, 2021
@zakkak zakkak requested a review from Karm April 21, 2021 21:05
@zakkak zakkak self-assigned this Apr 21, 2021
@zakkak
Copy link
Collaborator Author

zakkak commented Apr 21, 2021

The failures are because base.yml does not exist in the repository yet. After merging it should be OK.

Copy link
Collaborator

@Karm Karm left a comment

Choose a reason for hiding this comment

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

👍

@zakkak zakkak merged commit 53d1c28 into graalvm:master Apr 21, 2021
@zakkak zakkak deleted the ci-refactoring branch April 21, 2021 21:23
@zakkak
Copy link
Collaborator Author

zakkak commented Apr 21, 2021

The failures are because base.yml does not exist in the repository yet. After merging it should be OK.

Hmm it looks like I was wrong https://github.com/graalvm/mandrel/actions/runs/772097210
At least now it will be easier for me to test the triggering API since https://github.com/graalvm/mandrel/actions/workflows/base.yml is in place :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
testing CI and testing related issues and PRs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants