Skip to content

Conversation

@tariq1890
Copy link
Contributor

No description provided.

@tariq1890 tariq1890 requested a review from cdesiniotis August 26, 2025 02:32
@tariq1890 tariq1890 force-pushed the ubuntu-vanilla-base branch from 799440e to 29e9ab1 Compare August 27, 2025 01:30
@tariq1890
Copy link
Contributor Author

tariq1890 commented Aug 27, 2025

@cdesiniotis requesting another review. I have added a build arg to specify the base image, should we ever need to use a different one

@@ -1,4 +1,6 @@
FROM nvcr.io/nvidia/cuda:13.0.0-base-ubuntu24.04 as build
ARG BASE_IMAGE=ubuntu:noble-20250716
Copy link
Contributor

Choose a reason for hiding this comment

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

Question: will dependabot still be able to update this line now that it is not a FROM statement?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I am not sure. It depends on dependabot's Dockerfile provider I suppose

Copy link
Contributor Author

@tariq1890 tariq1890 Aug 27, 2025

Choose a reason for hiding this comment

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

On doing a cursory lookup, it seems unlikely that dependabot would update any docker image reference other than one in a FROM statement. We'd have to use renovate to automate updates

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'll add the renovate job in a follow-up.

@tariq1890 tariq1890 merged commit 22b9702 into main Aug 27, 2025
24 of 34 checks passed
@tariq1890 tariq1890 deleted the ubuntu-vanilla-base branch August 27, 2025 19:46
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.

3 participants