-
Notifications
You must be signed in to change notification settings - Fork 31
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 skip preflight check flag #212
base: main
Are you sure you want to change the base?
Conversation
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: aniruddha2000 The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
58c4edb
to
49ab69d
Compare
|
Yes, we'll need to cleanup any namespaced resources. |
@rjsadow i meant as we are reusing the namespace, could we reuse the named resources as well? if we do a cleanup here then there is no point of reusing the namespace because |
03d52ff
to
728b38c
Compare
@rjsadow i will sqush it later |
/lgtm |
Signed-off-by: Aniruddha Basak <[email protected]>
5a9e1f2
to
d67f42c
Compare
/lgtm |
fixes #211