Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Fixes #20567: It is impossible to read group properties with a read-only account #4107

Merged
merged 1 commit into from
Jan 13, 2022
Merged

Fixes #20567: It is impossible to read group properties with a read-only account #4107

merged 1 commit into from
Jan 13, 2022

Conversation

ElaadF
Copy link
Member

@ElaadF ElaadF commented Jan 13, 2022

@ElaadF ElaadF requested a review from fanf January 13, 2022 11:00
@ElaadF ElaadF added Trigger test WIP Use that label for a Work In Progress PR that must not be merged yet labels Jan 13, 2022
@ElaadF ElaadF removed the WIP Use that label for a Work In Progress PR that must not be merged yet label Jan 13, 2022
@Normation-Quality-Assistant
Copy link
Contributor

This PR is not mergeable to upper versions.
Since it is "Ready for merge" you must merge it by yourself using the following command:
rudder-dev merge https://github.com/Normation/rudder/pull/4107
-- Your faithful QA
Kant merge: "Science is organized knowledge. Wisdom is organized life."
(https://ci.normation.com/jenkins/job/merge-accepted-pr/49506/console)

@ElaadF
Copy link
Member Author

ElaadF commented Jan 13, 2022

OK, merging this PR

@ElaadF ElaadF merged commit aa2caa6 into Normation:branches/rudder/6.1 Jan 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@VinceMacBuche VinceMacBuche VinceMacBuche approved these changes

@fanf fanf Awaiting requested review from fanf

@ncharles ncharles Awaiting requested review from ncharles

Assignees

@fanf fanf

@ElaadF ElaadF

Labels
qa: Can't merge Trigger test
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@ElaadF @Normation-Quality-Assistant @VinceMacBuche @fanf