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

[Bugfix] Change docker image of proxy-service to pinned version #36

Merged
merged 1 commit into from
Mar 15, 2023

Conversation

MfCrizz
Copy link
Contributor

@MfCrizz MfCrizz commented Mar 9, 2023

Change docker image to a pinned version and use gadlew for building

The gradle version used in "gradle:jdk8" was updated to version 8.x, causing the build to fail.
We are using now a pinned openjdk image and use the gradle wrapper to download and build with the desired gradle version 6.8.3.

@MfCrizz MfCrizz requested a review from a team as a code owner March 9, 2023 15:31
@MfCrizz MfCrizz changed the title bug [proxy-service] Change docker image to a pinned version and use g… [Bugfix] Change docker image of proxy-service to pinned version Mar 10, 2023
@AlejandroCamba
Copy link
Contributor

If you have the time would you update your commits to use the propper tag? in this case it would be 🐛, we internally use Gitmoji sorry that this is not documented, still early days of unguard opensource 😄

@MfCrizz MfCrizz force-pushed the bugfix/pin-docker-image-version-and-use-gradlew branch from c55a40b to 1ea3f5a Compare March 13, 2023 09:20
The gradle version used in "gradle:jdk8" was updated to version 8.x, causing the build to fail.
We are using now a pinned openjdk image and use the gradle wrapper to download and build with the desired gradle version 6.8.3.
@MfCrizz MfCrizz force-pushed the bugfix/pin-docker-image-version-and-use-gradlew branch from 1ea3f5a to ad56840 Compare March 13, 2023 14:49
@MfCrizz MfCrizz merged commit 1aeb8cb into main Mar 15, 2023
@MfCrizz MfCrizz deleted the bugfix/pin-docker-image-version-and-use-gradlew branch March 15, 2023 08:30
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