-
Notifications
You must be signed in to change notification settings - Fork 49
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(): add CRD for GCPAuthEngineConfig #242
base: main
Are you sure you want to change the base?
Conversation
@erlisb could you resolve the conflicts? |
Done |
there is something wrong here, the |
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 the actual code for this engine config?
config/webhook/manifests.yaml
Outdated
@@ -89,9 +89,9 @@ webhooks: | |||
service: | |||
name: webhook-service | |||
namespace: system | |||
path: /mutate-redhatcop-redhat-io-v1alpha1-databasesecretengineconfig | |||
path: /mutate-redhatcop-redhat-io-v1alpha1-gcpauthengineconfig |
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.
is this correct, it seems you are replacing the databasesecretengineconfig, this file is automatically generated, just run make manifests
config/rbac/role.yaml
Outdated
@@ -287,9 +287,6 @@ rules: | |||
- get | |||
- patch | |||
- update | |||
- apiGroups: |
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.
there is a problem here, again please generate this file with make config.
config/webhook/manifests.yaml
Outdated
failurePolicy: Fail | ||
name: mdatabasesecretengineconfig.kb.io |
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.
also these substitutions look suspicious, please regenerate this file with make manifests
@raffaelespazzoli I regenerated the affected manifests. Please, check again. Thanks. |
Hi @raffaelespazzoli kind reminder! |
@erlisb can you re-base and fix the conflicts |
This PR consists in :
API Resource / CRD for GCPAuthEngineConfig
Defaulter / Validator Webhook for GCPAuthEngineConfig CRD
Controller Reconciliator for for GCPAuthEngineConfig CRD
Documentation for GCPAuthEngineConfig CRD
@raffaelespazzoli , please review it.
Thnx