-
Notifications
You must be signed in to change notification settings - Fork 16
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
[CPDEV-101357] Remove PSP and master role support #665
Conversation
88c19a8
to
ac5bc5d
Compare
ac5bc5d
to
ea5aa5c
Compare
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.
Probably it's worth to remove Service Account Issuer description from IN? This FeatureGate was removed in v1.23.
Only remove the |
yes, remove the |
It is necessary to repeat the steps and make sure it is still working. Also there are few other guides that require actualization. Created issue #666 |
ea5aa5c
to
8ec0c19
Compare
Description
master
role.do
procedure.Fixes #55
Breaking changes
master
is not supported neither innodes.roles
nor in other sections that refer to roles, e.gthirdparties.group
.master
role existence in jinja templates will no longer work, e. g.{{ nodes | select("has_role", "master") | list }}
Checklist
Unit tests
test_do.py - cover support of
master
role fordo
procedure.