Bug #5200
closed
Nodes which are not part of distributed architecture displays an error log on agent execution about missing '/opt/rudder/etc/server-roles.d/' directory
Added by Nicolas PERRON over 10 years ago.
Updated over 9 years ago.
Description
The use of cf-promises on nodes display some errors about non existent folder:
2014-07-02T07:19:28-0700 error: Directory '/opt/rudder/etc/server-roles.d/' could not be accessed in lsdir(), (opendir: No such file or directory)
2014-07-02T07:19:29-0700 error: Directory '/opt/rudder/etc/server-roles.d/' could not be accessed in lsdir(), (opendir: No such file or directory)
2014-07-02T07:19:29-0700 error: Directory '/opt/rudder/etc/server-roles.d/' could not be accessed in lsdir(), (opendir: No such file or directory)
2014-07-02T07:19:29-0700 error: Directory '/opt/rudder/etc/server-roles.d/' could not be accessed in lsdir(), (opendir: No such file or directory)
This happens with cf-promises on nodes as the foldr /opt/rudder/etc/server-roles.d/ does not exist. I suppose the only way to not have these messages is to create this folder everywhere.
- Status changed from New to Discussion
- Assignee set to Jonathan CLARKE
I suppose that adding by default an empty folder '/opt/rudder/etc/server-roles.d/' is harmless, isn't it ? What's you opinion Jon ?
No, the code should be flexible. Add a test to see if the directory exists. Condition all tests in it by that class.
- Status changed from Discussion to In progress
- Assignee changed from Jonathan CLARKE to Nicolas PERRON
- Status changed from In progress to Discussion
- Assignee changed from Nicolas PERRON to Jonathan CLARKE
Nicolas PERRON wrote:
I suppose that adding by default an empty folder '/opt/rudder/etc/server-roles.d/' is harmless, isn't it ? What's you opinion Jon ?
This "error" is made by cf-promises. No class will prevent from read the lines leading to these messages.
- Target version set to 2.11.0~rc2
I was completly wrong about that issue. The problem is from the promise creating inventories and it is possible to stop these errors without make the folder.
- Project changed from Rudder to 24
- Status changed from Discussion to Pending technical review
- % Done changed from 0 to 100
- Pull Request set to https://github.com/Normation/rudder-techniques/pull/443
- Status changed from Pending technical review to Pending release
Applied in changeset commit:0c4735f79b6d194a34ba65ddb392f31f1d4aeb66.
Applied in changeset commit:2dfbac7e6342e56a534bb3358049cb7977ba1415.
- Subject changed from The use of cf-promises display errors about non-existent '/opt/rudder/etc/server-roles.d/' on nodes to Nodes which are not part of distributed architecture displays an error log on agent execution about missing '/opt/rudder/etc/server-roles.d/' directory
- Status changed from Pending release to Released
- Project changed from 24 to Rudder
- Category set to Techniques
Also available in: Atom
PDF
Updated by Nicolas PERRON 
Updated by Jonathan CLARKE 
Updated by 
Updated by