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

Add Linux podman build to CI #33464

Merged
merged 3 commits into from
May 31, 2023
Merged

Add Linux podman build to CI #33464

merged 3 commits into from
May 31, 2023

Conversation

holly-cummins
Copy link
Contributor

@holly-cummins holly-cummins commented May 18, 2023

I've added a workflow to give us some coverage of podman on Linux. I've set it up to run on a schedule, rather than every PR or push.

Notes: In my first revision, I have the build triggering on PRs, so we can see it running. I'll remove those stanzas before merging. I do have a build which ran clean on my fork apart from permissions failures on the report upload.

I've been running this build on my own fork, and observed a number of failures. Most (all?) are not reproducible on local linux, and seem to be related to GitHub Actions virtualisation. I've disabled the tests in this PR, and raised issues. https://github.com/quarkusio/quarkus/issues?q=is%3Aissue+is%3Aopen+label%3Aenv%2Fpodman is the most convenient way to see a list of the issues (if you ignore ones raised by other people).

We're seeing issues building podman native, so I will also add some runs for that in a follow-on PR, but I wanted to get the JVM tests running first.

@quarkus-bot quarkus-bot bot added area/cli Related to quarkus cli (not maven/gradle/etc.) area/devtools Issues/PR related to maven, gradle, platform and cli tooling/plugins area/hibernate-orm Hibernate ORM area/infra-automation anything related to CI, bots, etc. that are used to automated our infrastructure area/logging area/metrics area/panache area/persistence OBSOLETE, DO NOT USE labels May 18, 2023
@quarkus-bot
Copy link

quarkus-bot bot commented May 18, 2023

/cc @Sanne (hibernate-orm), @brunobat (micrometer), @ebullient (micrometer), @gsmet (hibernate-orm), @yrodiere (hibernate-orm)

@holly-cummins holly-cummins changed the title Add podman build Add Linux podman build to CI May 18, 2023
@quarkus-bot

This comment has been minimized.

@holly-cummins
Copy link
Contributor Author

Oh rats, the report for my new workflow is still failing when run in the main build. I assumed it was just failing because I'd been testing in a fork.

 {"message":"Bad credentials","documentation_url":"https://docs.github.com/rest"}

I'll remove that step and we can work on figuring out why it's not working.

@quarkus-bot

This comment has been minimized.

@holly-cummins
Copy link
Contributor Author

I've removed the PR-trigger for the podman build, so the build of this PR should be very boring, but I believe this change is now ready to merge. We need native testing, but I will do that as a follow-on work item.

I've spoken with @gsmet, and we've agreed that the workflow file is rather large, which creates a maintenance burden. Longer term, it will be better to include podman in the regular matrix build. We don't want to exercise podman every PR, but we can do some tricks with the schedule and the matrix calculation. For example, if the schedule time matches a magic number, the matrix calculation will include podman. This scales better as we increase our podman coverage to mac, windows, etc. It means podman build results will be mixed in with the main CI results, which has advantages (one place to look, less clutter of workflows) and disadvantages (confusing if something fails).

The matrix calculation should be straightforward for JVM, and slightly harder for native, but we can get there.

Here's what a trigger-test might look like

on: 
  schedule: 
     - cron: '30 5 * * 1,3' 
     - cron: '30 5 * * 2,4' 
   jobs: 
      test_schedule: 
        runs-on: ubuntu-latest 
        steps: 
           - name: Not on Monday or Wednesday 
              if: github.event.schedule != '30 5 * * 1,3' 
              run: echo "This step will be skipped on Monday and Wednesday"

@gsmet
Copy link
Member

gsmet commented May 31, 2023

@holly-cummins FWIW, you might be able to use github.event.schedule != '30 5 * * 1,3' in the global env: section of the job so you might be able to introduce a IS_PODMAN env var instead of using this test again and again. At least it's worth a try.

@gsmet gsmet merged commit 1cf8513 into quarkusio:main May 31, 2023
@quarkus-bot quarkus-bot bot added this to the 3.2 - main milestone May 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/cli Related to quarkus cli (not maven/gradle/etc.) area/devtools Issues/PR related to maven, gradle, platform and cli tooling/plugins area/hibernate-orm Hibernate ORM area/infra-automation anything related to CI, bots, etc. that are used to automated our infrastructure area/logging area/metrics area/panache area/persistence OBSOLETE, DO NOT USE env/podman
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants