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.

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

NET-6704 Modify-server-acl-init-role #3957

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

sarahalsmiller
Copy link
Member

Changes proposed in this PR

  • Update server-acl-init-role to use 'core' instead of '""'

How I've tested this PR

How I expect reviewers to test this PR

Checklist

@sarahalsmiller sarahalsmiller added pr/no-changelog PR does not need a corresponding .changelog entry backport/1.4.x labels Apr 29, 2024
@sarahalsmiller sarahalsmiller requested review from a team, nathancoleman and missylbytes and removed request for a team April 29, 2024 19:50
@sarahalsmiller sarahalsmiller force-pushed the NET-6704-Modify-server-acl-init-role-to-specify-apiGroups-as-core- branch from 9488d15 to 1ef5aef Compare April 30, 2024 21:02
@jm96441n
Copy link
Member

jm96441n commented May 2, 2024

overall LGTM but I think we should probably add a changelog here

@sarahalsmiller sarahalsmiller force-pushed the NET-6704-Modify-server-acl-init-role-to-specify-apiGroups-as-core- branch from 3a0be38 to 1ef5aef Compare May 6, 2024 17:09
@sarahalsmiller sarahalsmiller force-pushed the NET-6704-Modify-server-acl-init-role-to-specify-apiGroups-as-core- branch from 1ef5aef to c94c45a Compare May 6, 2024 17:09
@sarahalsmiller sarahalsmiller force-pushed the NET-6704-Modify-server-acl-init-role-to-specify-apiGroups-as-core- branch from 87739e5 to 96214cf Compare May 6, 2024 18:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport/1.4.x pr/no-changelog PR does not need a corresponding .changelog entry
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants