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(swabbie): Dockerize Swabbie #668

Merged
merged 6 commits into from
Jun 13, 2024
Merged

feat(swabbie): Dockerize Swabbie #668

merged 6 commits into from
Jun 13, 2024

Conversation

jervi
Copy link
Contributor

@jervi jervi commented May 29, 2024

No description provided.

@jervi jervi requested a review from jeyrschabu as a code owner May 29, 2024 09:39
@@ -0,0 +1,8 @@
FROM ubuntu:jammy
RUN apt-get update && apt-get install -y \
openjdk-11-jdk \
Copy link
Member

Choose a reason for hiding this comment

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

Wondering if shouldn't run on 17 vs. 11 kinda thing for builds now. The target on the gradle plugin should still compile to 11... but can use the 17 JDK for this.

Copy link
Member

Choose a reason for hiding this comment

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

NOTE: This is how other projects work, so separate PR for that.

Copy link
Member

@jasonmcintosh jasonmcintosh left a comment

Choose a reason for hiding this comment

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

NICE and thank you!!

@jasonmcintosh jasonmcintosh added the ready to merge Reviewed and ready for merge label Jun 13, 2024
@mergify mergify bot added the auto merged label Jun 13, 2024
@mergify mergify bot merged commit c2ac7d3 into spinnaker:master Jun 13, 2024
4 checks passed
@jervi jervi deleted the dockerize branch June 13, 2024 14:45
@jervi
Copy link
Contributor Author

jervi commented Jun 13, 2024

Thanks @jasonmcintosh! Not sure if any credentials or other env variables needs to be added to the repo, but hopefully everything is inherited from the org.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto merged ready to merge Reviewed and ready for merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants