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

workflows: Run trigger-anaconda in our tasks container #20477

Conversation

mvollmer
Copy link
Member

It has everything and Rawhide is moving stuff around a lot recently.

It has everything and Rawhide is moving stuff around a lot recently.
@mvollmer
Copy link
Member Author

A almost-successful run here: https://github.com/mvollmer/cockpit/actions/runs/9115006365/job/25060286605?pr=50

This was in a PR against my own fork, and it failed because packit doesn't run for my fork.

@mvollmer mvollmer added the github_actions Pull requests that update GitHub Actions code label May 17, 2024
@@ -19,16 +19,18 @@ jobs:
permissions:
contents: read
statuses: write
container: registry.fedoraproject.org/fedora:rawhide
container:
image: ghcr.io/cockpit-project/tasks
Copy link
Member

Choose a reason for hiding this comment

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

Why not a more usual stable fedora image like 40?

Copy link
Member Author

Choose a reason for hiding this comment

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

Good question! My impression was that we are moving towards running as much as possible inside our task container, so that we need to worry about as few containers as possible. And this workflow calls bots/tests-trigger, and our task container is meant to execute bots code like this, no?

So, my impression is task-container == some stable thing plus dependencies required by our bots. Is that not so?

Copy link
Member

Choose a reason for hiding this comment

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

If we want to run this in the tasks container, then I feel like we should make sure we already have the dependencies that this thing needs present in the tasks container.

So like:

  • use tasks container; or
  • standard image + dnf install some stuff

pick one.

Downloading the much larger image and having to do dnf anyway seems like a real waste of time...

Copy link
Member Author

Choose a reason for hiding this comment

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

Alright: #20478

@mvollmer mvollmer closed this May 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
github_actions Pull requests that update GitHub Actions code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants