-
Notifications
You must be signed in to change notification settings - Fork 22
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fixes #18286 #1626
Fixes #18286 #1626
Conversation
PR updated with a new commit |
PR updated with a new commit |
PR updated with a new commit |
PR updated with a new commit |
trustkeysfrom => { | ||
@{def.acl} , | ||
}; #trustkey allows the exchange of keys | ||
}; | ||
# our policy server should already be trusted at bootstraping |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
at bootstraping -> after bootstraping is done
PR updated with a new commit |
PR updated with a new commit |
PR updated with a new commit |
@@ -428,3 +451,10 @@ body perms u_mog(mode,user,group) | |||
owners => { "${user}" }; | |||
groups => { "${group}" }; | |||
} | |||
|
|||
|
|||
bundle edit_line rudder_update_lines_present(lines) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
all other bodys/bundle have a more specific name. I fear collision with a name that common
Could you name it u_rudder_update_lines_present ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Where is this used
PR updated with a new commit |
This PR is not mergeable to upper versions. |
OK, squash merging this PR |
No description provided.