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

Updates to jetty.sh and testing to reduce failures #10790

Merged
merged 1 commit into from
Oct 26, 2023

Conversation

joakime
Copy link
Contributor

@joakime joakime commented Oct 26, 2023

  • Updating AmazonCorretto to JDK 17
  • Making sure /etc/default/jetty is populated correctly in the user_change mode
  • Removing warnings from jetty startup about --start-log-file=... being unrecognized
  • Adding unique jetty12 identifier to docker image names (helps to keep different jetty versions apart when manually testing)

+ Updating AmazonCorretto to JDK 17
+ Making sure /etc/default/jetty is populated
  correctly in the user_change mode
+ Removing warnings from jetty startup
  about --start-log-file=... being
  unrecognized
+ Adding unique jetty12 identifier to docker
  image names (helps to keep different jetty
  versions apart when manually testing)
@joakime joakime requested review from gregw and sbordet October 26, 2023 01:18
joakime added a commit that referenced this pull request Oct 26, 2023
Backport of #10790 to Jetty 10.

+ Making sure /etc/default/jetty is populated correctly in the user_change mode
+ Removing warnings from jetty startup about --start-log-file=... being unrecognized
+ Adding unique jetty10 identifier to docker image names (helps to keep different jetty versions apart when manually testing)

Signed-off-by: Joakim Erdfelt <joakim.erdfelt@gmail.com>
@joakime joakime self-assigned this Oct 26, 2023
@joakime joakime added this to the 12.0.x milestone Oct 26, 2023
@gregw gregw merged commit 3eddc92 into jetty-12.0.x Oct 26, 2023
3 of 4 checks passed
@joakime joakime deleted the fix/12.0.x/jetty-sh-updates branch October 26, 2023 03:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

2 participants