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

Updating check-java.sh and server.xml templates to upstream version #348

Conversation

timdeluxe
Copy link
Contributor

Pull Request (PR) description

Updates check-java.sh and server.xml templates to upstream version. Especially the check-java update is required, when using a java version > 8 (or 1.8, depending which versioning scheme you look at.

This Pull Request (PR) fixes the following issues

Fixes #339
Kind of fixes #300

This replaces PR #317.

templates/check-java.sh.erb Show resolved Hide resolved
templates/server.xml.erb Show resolved Hide resolved
@ekohl
Copy link
Member

ekohl commented Mar 5, 2021

The tests are failing. Could you have a look?

@timdeluxe
Copy link
Contributor Author

Will do (probably on weekend), didn't find the time yet, sorry.
Will revert the removal of < 8 support - as you said, it should be another PR, that should make the tests be green again.

@timdeluxe
Copy link
Contributor Author

@ekohl PR is now updated

@kenyon
Copy link
Member

kenyon commented Mar 7, 2021

Can you clean up the commit history (get rid of merge commit, combine fixup commits)? Or set the PR to allow edits by maintainers so we can do it.

@kenyon
Copy link
Member

kenyon commented Mar 7, 2021

Also note that your kps email address is not registered with your GitHub account.

@timdeluxe timdeluxe force-pushed the feature/update-check-java-and-server-xml-templates branch from d0b5777 to 574ce40 Compare March 8, 2021 10:46
@timdeluxe
Copy link
Contributor Author

@kenyon Thanks for the hints. Hope i won my fight against git rebase successfully now... (i am always fighting with it)

@timdeluxe
Copy link
Contributor Author

@kenyon @ekohl Can i get a merge please?

@ekohl ekohl merged commit 915a612 into voxpupuli:master Mar 17, 2021
cegeka-jenkins pushed a commit to cegeka/puppet-jira that referenced this pull request Oct 13, 2021
…ava-and-server-xml-templates

Updating check-java.sh and server.xml templates to upstream version
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.

server.xml - StuckThreadDetectionValve introduced in 7.6.12 Java 11 Compatibility
3 participants