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

Add role to view all resources for propagate resource #21

Merged
merged 1 commit into from
Oct 6, 2021

Conversation

kmdkuk
Copy link
Contributor

@kmdkuk kmdkuk commented Sep 30, 2021

#20

Signed-off-by: kouki [email protected]

@kmdkuk kmdkuk self-assigned this Sep 30, 2021
@@ -1,3 +1,4 @@
//go:build !ignore_autogenerated
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is an auto-generated file, so do not edit it by hand.

@@ -29,6 +29,7 @@ type SubNamespaceReconciler struct {
//+kubebuilder:rbac:groups=accurate.cybozu.com,resources=subnamespaces/status,verbs=get;update;patch
//+kubebuilder:rbac:groups=accurate.cybozu.com,resources=subnamespaces/finalizers,verbs=update
//+kubebuilder:rbac:groups="",resources=namespaces,verbs=get;list;watch;create;update;patch;delete
//+kubebuilder:rbac:groups=*,resources=*,verbs=get;list;watch
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This comment should be added to controllers/propagate.go.

@kmdkuk kmdkuk changed the title Add role to view all resources to subnamespace controller Add role to view all resources for propagate resource Sep 30, 2021
@kmdkuk kmdkuk force-pushed the add-role-to-view-all-resources branch from cb2510d to d9e29be Compare September 30, 2021 09:02
@kmdkuk kmdkuk force-pushed the add-role-to-view-all-resources branch from d9e29be to 3dfde45 Compare October 4, 2021 02:50
@kmdkuk kmdkuk marked this pull request as ready for review October 4, 2021 04:30
@kmdkuk kmdkuk requested review from ymmt2005 and zoetrope October 4, 2021 04:30
Copy link
Member

@ymmt2005 ymmt2005 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Member

@zoetrope zoetrope left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ymmt2005 ymmt2005 merged commit 15ef59a into main Oct 6, 2021
@ymmt2005 ymmt2005 deleted the add-role-to-view-all-resources branch October 6, 2021 05:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants