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

Update base image #8228

Merged
merged 2 commits into from
Mar 15, 2023
Merged

Update base image #8228

merged 2 commits into from
Mar 15, 2023

Conversation

lbergelson
Copy link
Member

I've updated the image and it seems to be working locally. Now for the true test...

@gatk-bot
Copy link

gatk-bot commented Mar 1, 2023

Github actions tests reported job failures from actions build 4297780099
Failures in the following jobs:

Test Type JDK Job ID Logs
conda 17.0.6+10 4297780099.3 logs

@gatk-bot
Copy link

gatk-bot commented Mar 14, 2023

Github actions tests reported job failures from actions build 4419433150
Failures in the following jobs:

Test Type JDK Job ID Logs
conda 17.0.6+10 4419433150.3 logs
integration 17.0.6+10 4419433150.0 logs

* Updating ubuntu to the newest 18.04 release
@lbergelson lbergelson changed the title Testing new ubuntu image Update base image Mar 14, 2023
Copy link
Collaborator

@cmnbroad cmnbroad left a comment

Choose a reason for hiding this comment

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

A couple of minor questions. Otherwise looks good. Thanks for doing this!

@@ -1,3 +1,4 @@
# These are files that for whatever reason we don't want to include in our distribution docker images
src/test/resources
src/test/resources/*
build/
Copy link
Collaborator

Choose a reason for hiding this comment

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

Oh, good idea....

software-properties-common && \

Copy link
Collaborator

Choose a reason for hiding this comment

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

Is this empty line not deliberate ? Does it not matter ?

Copy link
Member Author

Choose a reason for hiding this comment

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

huh, that's definitely not deliberate and I would have expected it to make it break...

Copy link
Member Author

Choose a reason for hiding this comment

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

It seems to make no difference.

apt-get update -y && \
apt-get install -y --no-install-recommends google-cloud-cli && \
apt-get -y clean && \
apt-get -y autoclean && \
Copy link
Collaborator

Choose a reason for hiding this comment

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

I'm so glad someone knows how todo this...

Copy link
Member Author

Choose a reason for hiding this comment

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

I suspect I'm repeating myself because I don't really know how to do it. Throw all the cleaning at the wall and hope something sticks.

@@ -6,7 +6,7 @@ FROM ubuntu:18.04
RUN apt-get update && \
Copy link
Collaborator

Choose a reason for hiding this comment

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

OpenJDK 8 reference on line 1 but GitHub won't let me comment there

Copy link
Member Author

Choose a reason for hiding this comment

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

@mcovarr Thank you, good catch!

@lbergelson lbergelson merged commit bab0adc into master Mar 15, 2023
@lbergelson lbergelson deleted the lb_update_ubuntu branch March 15, 2023 17:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants