-
Notifications
You must be signed in to change notification settings - Fork 67
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
Remove networkplugin syncer deployment #2759
Conversation
The controller now deletes the Deployment and associated RBAC resources. Signed-off-by: Tom Pantelis <[email protected]>
🤖 Created branch: z_pr2759/tpantelis/rem_np_syncer |
LGTM , @vthapar I think this does some part of submariner-io/enhancements#193 |
The OVN test should pass once this is merged submariner-io/submariner#2621 |
Added a Depends on in the commit message. |
@tpantelis the ovn failure now seems to be related to uninstall. |
Looks an issue with the route agent uninstall on the submariner side. Maybe it’s crashing. |
|
🤖 Closed branches: [z_pr2759/tpantelis/rem_np_syncer] |
The controller now deletes the Deployment and associated RBAC resources.
Fixes submariner-io/enhancements#191
Depends on: submariner-io/submariner#2621