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 webhook validations secret object #18

Merged
merged 2 commits into from
Sep 9, 2022
Merged

Add webhook validations secret object #18

merged 2 commits into from
Sep 9, 2022

Conversation

Anandkumar26
Copy link
Contributor

@Anandkumar26 Anandkumar26 commented Aug 25, 2022

  • Update of secrets are allowed only when the secret key
    (which is reffered by CPA) is not changed.
  • Delete of secrets are allowed only when it is not being
    referred by CPA.
  • Add unit tests for validating webhook for Secret
  • Coverage at 95.1%

Signed-off-by: Anand Kumar [email protected]

@Anandkumar26
Copy link
Contributor Author

Yet to add the kubebuilder verbs to auto-generate the RBAC for webhook.

config/nephe.yml Outdated Show resolved Hide resolved
@Anandkumar26 Anandkumar26 force-pushed the webhook branch 3 times, most recently from a00a8ca to d42e3ea Compare August 30, 2022 04:24
@Anandkumar26 Anandkumar26 changed the title [WIP] Add webhook validations secret object Add webhook validations secret object Aug 30, 2022
config/nephe.yml Outdated Show resolved Hide resolved
Makefile Outdated Show resolved Hide resolved
@Anandkumar26 Anandkumar26 force-pushed the webhook branch 3 times, most recently from 47cb929 to d2e4b44 Compare September 8, 2022 05:04
@reachjainrahul
Copy link
Contributor

/nephe-test-e2e-kind

@Anandkumar26
Copy link
Contributor Author

The nephe-test-e2e-aws and nephe-test-e2e-azure has passed.
Rebased on top of main branch.

Changed function descriptions for few functions, and updated comment in the test file.

- Update of secrets are allowed only when the secret key
  (which is reffered by CPA) is not changed.

- Delete of secrets are allowed only when it is not being
  referred by CPA.

Signed-off-by: Anand Kumar <[email protected]>
- Coverage at 95.1%

Signed-off-by: Anand Kumar <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants