-
Notifications
You must be signed in to change notification settings - Fork 66
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
Move pkg/utils/crds to pkg/crd #1743
Conversation
🤖 Created branch: z_pr1743/tpantelis/move_crdutils |
07ea441
to
5ff0374
Compare
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.
LGTM, re-running the UTs...
5ff0374
to
be78886
Compare
I fixed them. |
be78886
to
d18ba1a
Compare
Also move the separate CRD CreateOrUpdate functions from utils to the Updater interface. Signed-off-by: Tom Pantelis <[email protected]>
d18ba1a
to
60b591b
Compare
🤖 Closed branches: [z_pr1743/tpantelis/move_crdutils] |
Also move the separate CRD
CreateOrUpdate
functions from utils to theUpdater
interface.