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

feat: upgrade docker image #124

Merged
merged 3 commits into from
Mar 25, 2024
Merged

feat: upgrade docker image #124

merged 3 commits into from
Mar 25, 2024

Conversation

IlonaShishov
Copy link
Collaborator

@IlonaShishov IlonaShishov commented Mar 25, 2024

Description

Upgraded the Dockerfile to include latest versions for Maven, Npm and Go ecosystems.
Python ecosystem is suported by providing pip feeze --all data and pip show data for all packages (base64 encrypted) to EXHORT_PIP_FREEZE and EXHORT_PIP_SHOW environment variables

Checklist

  • I have followed this repository's contributing guidelines.
  • I will adhere to the project's code of conduct.

Additional information

Anything else?

@IlonaShishov IlonaShishov force-pushed the feat/upgrade-docker-image branch 3 times, most recently from 2128f0d to 59a78fa Compare March 25, 2024 10:28
…n in docker image

Signed-off-by: Ilona Shishov <Ilona.Shishov@gmail.com>
Signed-off-by: Ilona Shishov <Ilona.Shishov@gmail.com>
@IlonaShishov IlonaShishov force-pushed the feat/upgrade-docker-image branch 3 times, most recently from 53ae5f5 to c583590 Compare March 25, 2024 11:45
Copy link
Collaborator

@zvigrinberg zvigrinberg left a comment

Choose a reason for hiding this comment

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

@IlonaShishov Everything is good.
Just 2 slight changes required in comments of new PIP env vars.
Please change and i'll approve and merge.

docker-image/Dockerfiles/Dockerfile Outdated Show resolved Hide resolved
docker-image/Dockerfiles/Dockerfile Outdated Show resolved Hide resolved
Signed-off-by: Ilona Shishov <Ilona.Shishov@gmail.com>
@IlonaShishov IlonaShishov force-pushed the feat/upgrade-docker-image branch from c583590 to 184b315 Compare March 25, 2024 12:08
Copy link
Collaborator

@zvigrinberg zvigrinberg left a comment

Choose a reason for hiding this comment

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

LGTM

@zvigrinberg zvigrinberg merged commit 79f6033 into main Mar 25, 2024
5 checks passed
@zvigrinberg zvigrinberg deleted the feat/upgrade-docker-image branch March 25, 2024 12:14
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.

2 participants