-
Notifications
You must be signed in to change notification settings - Fork 0
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
Fix container building by moving to ubuntu-latest #140
Conversation
Moving to ubuntu-latest at least fixed the container building for the jupyter image I was after. However, the CI won't try building the github actions container (which hasn't been built in a while) because that only happens on merge to master. Should we give this a try @alfonsomunozpomer @pmb59 ? That github actions runner only affects the galaxy tools I think. But I suspect that other containers being merged have not been pushed recently. |
- "*/post_install_dockerfile" | ||
- "miniconda_template/*" | ||
- ".github/workflows/miniconda_template.yaml" | ||
|
||
jobs: | ||
getchanges: | ||
name: Get changed repos | ||
runs-on: ubuntu-20.04 | ||
runs-on: ubuntu-latest |
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.
using latest
here will always use stable or LTS version right?
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.
Yes, it is usually the latest "stable" image. Since this is the OS, this should be relatively safe, currently points to 22.04 I think.
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.
As long as using latest version of ubuntu wont break in future
LGTM
Container building seems broken. I will start by trying changing the OS.