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

operator: Move the CRD registration logic to a separate package #1350

Merged
merged 1 commit into from
Aug 11, 2023

Conversation

michi-covalent
Copy link
Contributor

We'll introduce a proper operator deployment in #794. Once the operator deployment is available, we can move the CRD registration logic there instead of calling it from an init container in the Tetragon daemonset. This commit moves the CRD registration logic to a separate package so that it can be called from outside the main package.

We'll introduce a proper operator deployment in #794. Once the operator
deployment is available, we can move the CRD registration logic there
instead of calling it from an init container in the Tetragon daemonset.
This commit moves the CRD registration logic to a separate package so
that it can be called from outside the main package.

Signed-off-by: Michi Mutsuzaki <michi@isovalent.com>
@michi-covalent michi-covalent force-pushed the pr/michi/more-refactor branch from ceb6372 to 4d4dedf Compare August 11, 2023 00:47
@michi-covalent michi-covalent marked this pull request as ready for review August 11, 2023 02:08
@michi-covalent michi-covalent requested a review from a team as a code owner August 11, 2023 02:08
@michi-covalent michi-covalent requested a review from tixxdz August 11, 2023 02:08
Copy link
Contributor

@jrfastab jrfastab left a comment

Choose a reason for hiding this comment

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

sure

@michi-covalent michi-covalent merged commit 88b37ce into main Aug 11, 2023
@michi-covalent michi-covalent deleted the pr/michi/more-refactor branch August 11, 2023 14:21
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