Dynatrace / Dynatrace-AppMon-Chef

Installs the Dynatrace Application Monitoring solution using Chef.
MIT License
12 stars 15 forks source link

Dynatrace java attributes #27

Closed tobymaro closed 7 years ago

tobymaro commented 7 years ago

Hello,

There is attribute: override['java']['jdk_version'] = '7'

I thinks it's bad idea to set this as "override". I have another services on the same node that installs java 8. But once I added java_agent recipe I get java 1.7 on machine.

Could you please correct it or explain why you do it?

Thank You.

pozieblo commented 7 years ago

Dynatrace 6.5 packages require Java version 1.7 to unpack the .jar files thus the need for this line - by default the java cookbook installs version 6. What we can do is to replace the override keyword by default which should work just fine in this case and will allow to override this attribute in other cookbooks.

tobymaro commented 7 years ago

Hi, It's ok to use default. thanx

pozieblo commented 7 years ago

Fix available in commit aa0775c1ae