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

NASA-PDS/devops#10 #48

Merged
merged 4 commits into from
Sep 21, 2021
Merged

NASA-PDS/devops#10 #48

merged 4 commits into from
Sep 21, 2021

Conversation

nutjob4life
Copy link
Member

🗒️ Summary

Merge this and it just might make this repository look a bit more like the pds-template-java-repo .

⚙️ Test Data and/or Report

$ env JAVA_HOME=/Library/Java/JavaVirtualMachines/jdk1.8.0_161.jdk/Contents/Home mvn test
[INFO] Scanning for projects...

[INFO] Tests run: 672, Failures: 0, Errors: 0, Skipped: 0
[INFO] 
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------

♻️ Related Issues

nutjob4life and others added 4 commits September 10, 2021 15:48
To support backwards compatibility with some users. Not sure if it is entirely necessary anymore, but will keep it in until we are sure.
* Update per template rollout
* Add changelog and notice
@jordanpadams
Copy link
Member

@nutjob4life looks good. for future java rollouts, can we make updates similar to d46b049 ? I just noticed we were not including the NOTICE and CHANGELOG (where it exists). As we are going through and scrubbing all these anyways, I think they would be useful for users?

@nutjob4life
Copy link
Member Author

Okay, thanks @jordanpadams. I'll try to remember to update the assembly files with the administrivia.

Also, the Roundup uses openjdk8 by default:

$ docker container run --rm --entrypoint /usr/bin/java nasapds/pds-github-actions-base -version
openjdk version "1.8.0_275"
OpenJDK Runtime Environment (IcedTea 3.17.1) (Alpine 8.275.01-r0)
OpenJDK 64-Bit Server VM (build 25.275-b01, mixed mode)

so explicitly mentioning it won't do anything unless we revert to an older version of Java in the GitHub Actions base image. (A newer version also has no effect, because to use an older version than Roundup also requires both installing the package and setting an environment variable.)

@jordanpadams jordanpadams merged commit 62a4a97 into main Sep 21, 2021
@jordanpadams jordanpadams deleted the template-rollout-summer-2021 branch September 21, 2021 16:10
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