[GitHub] [cloudstack-documentation] DaanHoogland commented on pull request #120: remove-environment.properties-note

[GitBox <gi...@apache.org>]

DaanHoogland commented on pull request #120:
URL: https://github.com/apache/cloudstack-documentation/pull/120#issuecomment-625688121


   > The starting premise "if it doesn't exist" is impossible in the first place... (tested on 4.11.0 - agent will never connect if that line is missing in the file) - then no need to the warning/message - so removing it from 4.11+ Upgrade pages.
   
   I don't see how this is a reason to remove this line. Users need to be attended to adding it, right? How is it impossible that a line is missing from a conf file? (unless we are adding it in automated fashion.
   
   Should
   ```
   paths.script=/usr/share/cloudstack-common
   ```
   be in `/etc/cloudstack/agent/environment.properties`, than  i see no reason for this removal.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org