Closed herbertkb closed 1 month ago
Good catch! environment here is a dictionary, not a list (and while you are here, can you please also set the PATH, like in here: https://github.com/ansible-middleware/keycloak/pull/211/files#diff-9c8ae367389a73fbea8f6569f448840c4ef96621d629804053473acf5ccf7547R6 )
Ah, that would explain why VSCode was highlighting it in red after I copied it over from my client laptop. Strange that it didn't complain there.
added PATH
PATH: "{{ activemq_java_home | default(activemq_rpm_java_home, true) }}/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin"
added PATH
PATH: "{{ activemq_java_home | default(activemq_rpm_java_home, true) }}/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin"
Thanks; sorry but activemq_rpm_java_home
is not defined for this task; you'll need to use rpm_java_home
(for which a set_fact exists at the top of systemd.yml)
fixed, and tested on a local VM. (Should have put together a local testing flow before opening the PR.)
Merging; I'll have to reword the title and first comment a little (they are used/linked from the changelog)
see https://issues.redhat.com/browse/AMWSUP-21
Given a VM where Java 8 is still the installed version of Java
When the
amq_broker
role is executed, the deployment fails. The task to run theartemis create
command to create the broker instance fails becauseartemis
was compiled with a more recent version of Java than Java8 can execute.Solution: set the
JAVA_HOME
envvar to matchamq_broker_jvm_package
.