Skip to content
This repository was archived by the owner on Nov 24, 2025. It is now read-only.

Changing openjdk to java#3017

Closed
smalenfant wants to merge 1 commit intoapache:masterfrom
smalenfant:sm-jdk-3010
Closed

Changing openjdk to java#3017
smalenfant wants to merge 1 commit intoapache:masterfrom
smalenfant:sm-jdk-3010

Conversation

@smalenfant
Copy link
Copy Markdown
Contributor

  • This allows either JDK to be used

What does this PR do?

Fixes #3010 - Variance to be used for backport

When using the requirement java. If Oracle JDK is available, then it will use it, if not openjdk will be used. I removed the requirement for Tomcat for java-1.8.0-openjdk-devel as it seemed unnecessary. I would need a review on this.

Which TC components are affected by this PR?

  • Documentation
  • Grove
  • Traffic Analytics
  • Traffic Monitor
  • Traffic Ops
  • Traffic Ops ORT
  • Traffic Portal
  • Traffic Router
  • Traffic Stats
  • Traffic Vault
  • Other _________

What is the best way to verify this PR?

Verify traffic router builds and install against Oracle JDK and OpenJDK.

Check all that apply

  • This PR includes tests
  • This PR includes documentation updates
  • This PR includes an update to CHANGELOG.md
  • This PR includes all required license headers
  • This PR includes a database migration (ensure that migration sequence is correct)
  • This PR fixes a serious security flaw. Read more: www.apache.org/security

- This allows either JDK to be used
@asfgit
Copy link
Copy Markdown
Contributor

asfgit commented Nov 12, 2018

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/2765/
Test PASSed.

@smalenfant
Copy link
Copy Markdown
Contributor Author

Closing. Be part of bigger PR.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Traffic Router RPM requirement for jdk defaults to 1.7

3 participants