-
Notifications
You must be signed in to change notification settings - Fork 5
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
Build and test images #14
Conversation
Signed-off-by: Din Music <[email protected]>
Signed-off-by: Din Music <[email protected]>
Signed-off-by: Din Music <[email protected]>
d71e2f8
to
b075f0f
Compare
using: composite | ||
steps: | ||
- name: Install dependencies | ||
shell: bash |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please replace all those by a single top section:
defaults:
run:
# Make sure bash is always invoked with `-eo pipefail`
# https://docs.github.com/en/actions/using-workflows/workflow-syntax-for-github-actions#jobsjob_idstepsshell
shell: bash
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've tried this one, but composite actions do not support default.run.shell
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh bummer. Have you tried putting it in the top composite job?
b075f0f
to
ae546c0
Compare
pull_request: | ||
# schedule: | ||
# # Run at 00:00 UTC daily. | ||
# - cron: '0 0 * * *' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
workflow_dispatch:
would be handing to trigger those workflows on demand.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will add. Currently, this is added only for each separate image action.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also, do we want to run those on "push" and "pull_request"? I would expect only workflow_dispatch and cron? Because once we add all images this will most likely trigger few hundred builds.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a call for Tom to make but I think we'll at least want cron and manual dispatch. I think you are right in that we should avoid the thundering herd on pull/push :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also, do we want to run those on "push" and "pull_request"? I would expect only workflow_dispatch and cron? Because once we add all images this will most likely trigger few hundred builds.
Yes indeed, daily and manually only.
We could potentially do something cleverer later where we detect if a pull request has modified recipe(s) and then runs the rebuild of only those modified in the PR.
fedora: | ||
uses: ./.github/workflows/image-fedora.yml | ||
# ubuntu: | ||
# uses: ./.github/workflows/image-ubuntu.yml |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like those composite jobs, pretty nifty. You might want to also include some steps to replicate https://github.com/canonical/lxd-ci/blob/main/.github/workflows/tests.yml#L83-L122
Signed-off-by: Din Music <[email protected]>
Signed-off-by: Din Music <[email protected]>
ae546c0
to
a83f946
Compare
Signed-off-by: Din Music <[email protected]>
Signed-off-by: Din Music <[email protected]>
Signed-off-by: Din Music <[email protected]>
Signed-off-by: Din Music <[email protected]>
Signed-off-by: Din Music <[email protected]>
Signed-off-by: Din Music <[email protected]>
Signed-off-by: Din Music <[email protected]>
a83f946
to
417577c
Compare
@MusicDin fedora 36 is EOL, 39 is where it's at now ;) |
Signed-off-by: Din Music <[email protected]>
Signed-off-by: Din Music <[email protected]>
Signed-off-by: Din Music <[email protected]>
Signed-off-by: Din Music <[email protected]>
417577c
to
1aaaecf
Compare
Moving to LXD-CI. |
Adds workflows for building and testing lxd-imagebuilder images.
TODO:
drivers: "lxc"
)