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(): add CRD for GCPAuthEngineConfig #242

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

Conversation

erlisb
Copy link
Contributor

@erlisb erlisb commented Jun 3, 2024

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

@raffaelespazzoli
Copy link
Collaborator

@erlisb could you resolve the conflicts?

@erlisb
Copy link
Contributor Author

erlisb commented Jun 25, 2024

Done

@raffaelespazzoli
Copy link
Collaborator

there is something wrong here, the mgcpauthengineconfig webhook configuration appears twice. this file shold be generated via make manifests it should not be manually modified....

Copy link
Collaborator

@raffaelespazzoli raffaelespazzoli left a 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?

@@ -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
Copy link
Collaborator

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

@@ -287,9 +287,6 @@ rules:
- get
- patch
- update
- apiGroups:
Copy link
Collaborator

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.

failurePolicy: Fail
name: mdatabasesecretengineconfig.kb.io
Copy link
Collaborator

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

@erlisb
Copy link
Contributor Author

erlisb commented Jul 23, 2024

@raffaelespazzoli I regenerated the affected manifests. Please, check again. Thanks.

@erlisb
Copy link
Contributor Author

erlisb commented Oct 6, 2024

Hi @raffaelespazzoli kind reminder!

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.

2 participants