Bug #2557
closed
(ex PT/ Technique) User Management: Checking a User can only return success or warning but no error
Added by Nicolas PERRON almost 12 years ago.
Updated over 11 years ago.
Description
Two cases:
Firstly, a user checked to be present on a system:
If it exists, it will return "success"
If it exists not, it will return "warning"
Secondly, a user checked to be not present on a system:
If it exists, it will return "warning"
If it exists not, it will return "success"
With "warning", on Rules screen, the Rule will be on error but if we check technical logs, we will not be able to find any error.
- Target version changed from 2.3.8 to 2.3.9
- Status changed from New to In progress
- % Done changed from 0 to 90
Nicolas PERRON wrote:
It seems that a bug in CFEngine (https://cfengine.com/bugtracker/view.php?id=1230) keep us from only checking (without edition) the fullname of an user.
The fullname should be checked now but since the bug below, it is modified too.
log_warning has been replaced by result_error, then the reports is now OK.
- Assignee deleted (
Nicolas PERRON)
Nicolas PERRON wrote:
Nicolas PERRON wrote:
It seems that a bug in CFEngine (https://cfengine.com/bugtracker/view.php?id=1230) keep us from only checking (without edition) the fullname of an user.
The fullname should be checked now but since the bug below, it is modified too.
log_warning has been replaced by result_error, then the reports is now OK.
Since the bug on CFEngine is still here, this issue can't be completly done. It will not be close for the beta5 or beta6.
Nicolas PERRON wrote:
Nicolas PERRON wrote:
Nicolas PERRON wrote:
It seems that a bug in CFEngine (https://cfengine.com/bugtracker/view.php?id=1230) keep us from only checking (without edition) the fullname of an user.
The fullname should be checked now but since the bug below, it is modified too.
log_warning has been replaced by result_error, then the reports is now OK.
Since the bug on CFEngine is still here, this issue can't be completly done. It will not be close for the beta5 or beta6.
I just checked this, and it turns out this is not a bug on the "warn_only" feature, but an error in the CFEngine code : the "action =>" attribute should be followed by a body name without quotes. So in your example, change this:
- action => "warn_only"
+ action => warn_only
- Status changed from In progress to 2
- Assignee set to Nicolas PERRON
Jonathan CLARKE wrote:
Nicolas PERRON wrote:
Nicolas PERRON wrote:
Nicolas PERRON wrote:
It seems that a bug in CFEngine (https://cfengine.com/bugtracker/view.php?id=1230) keep us from only checking (without edition) the fullname of an user.
The fullname should be checked now but since the bug below, it is modified too.
log_warning has been replaced by result_error, then the reports is now OK.
Since the bug on CFEngine is still here, this issue can't be completly done. It will not be close for the beta5 or beta6.
I just checked this, and it turns out this is not a bug on the "warn_only" feature, but an error in the CFEngine code : the "action =>" attribute should be followed by a body name without quotes. So in your example, change this:
[...]
Ok, then I can finish this issue at the light of your comment :)
- Status changed from 2 to Pending technical review
- % Done changed from 90 to 100
Applied in changeset commit:f84a7fc2dbb29d7417ecb70ba7eb3b3af9fe929a.
- Status changed from Pending technical review to Released
This looks valid. However, as in ticket #2558 I wonder if it might not have been a bit more cfenginesque to use the bundle edit_line set_colon_field of the stdlib, and define classes in the files: promises part rather than in the bundle edit_line
Also available in: Atom
PDF
Updated by 
Updated by Nicolas PERRON 
Updated by