Github user rhtyd commented on the issue:

    https://github.com/apache/cloudstack/pull/1647
  
    @wido @vincentbernat  I could not get rid of the postinst step as shell is 
not executed when I move the code in the EnvironmentFile itself. I've reduced 
its impact to only run when the default /etc/default/cloudstack-{usage, agent} 
are installed.
    
    Since we're re-using the common systemd files, the JAVA_HOME declared in 
the files are set to the default we see on CentOS and only in that case the 
postinst step will replace it with the system default. On future upgrades, the 
postinst code won't run. Sorry, but this hack was needed.
    
    Please see one last time, if we're in agreement as soon as my packaging 
testing succeed I'll merge this. /cc @jburwell @karuturi 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to