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

Misc fixes to the Wolfi-based Dockerfile #114

Merged
merged 2 commits into from
Aug 29, 2024
Merged

Conversation

acrewdson
Copy link
Contributor

@acrewdson acrewdson commented Aug 28, 2024

In Dockerfile.wolfi:

  • switch back to the java user for running the application itself
  • use newer ENV= syntax
  • remove /app/.git from the image to decrease size

* switch back to the 'java' user for running the application itself

* use newer ENV= syntax

* remove /app/.git from the image
@acrewdson acrewdson requested review from a team and oli-g August 28, 2024 17:44
Copy link
Member

@seanstory seanstory left a comment

Choose a reason for hiding this comment

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

:rubberstamp:

@acrewdson acrewdson merged commit 10ea50b into main Aug 29, 2024
2 checks passed
@acrewdson acrewdson deleted the acrewdson/wolfi-fixes branch August 29, 2024 15:34
github-actions bot pushed a commit that referenced this pull request Aug 29, 2024
In `Dockerfile.wolfi`:

* switch back to the `java` user for running the application itself
* use newer `ENV=` syntax
* remove `/app/.git` from the image to decrease size
Copy link

💚 Backport PR(s) successfully created

Status Branch Result
0.1 #119

This backport PR will be merged automatically after passing CI.

acrewdson added a commit that referenced this pull request Aug 29, 2024
Backports the following commits to 0.1:
 - Misc fixes to the Wolfi-based Dockerfile (#114)

Co-authored-by: acrewdson <acrewdson@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants