-
Notifications
You must be signed in to change notification settings - Fork 1.3k
cks: fix k8s cluster network deletion #4134
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
cks: fix k8s cluster network deletion #4134
Conversation
Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>
|
@blueorangutan package |
|
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. |
|
Packaging result: ✔centos7 ✔debian. JID-1340 |
|
@blueorangutan test |
|
@shwstppr a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests |
|
Trillian test result (tid-1713)
|
|
@shwstppr is this ready for review/testing? |
|
@rhtyd will test and update. Tests failed. I've to check the reason. And maybe refactor tests alongwith this change to make them more reliable |
Fixes apache#4056 Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>
|
Packaging result: ✖centos7 ✖debian. JID-1394 |
|
Packaging result: ✖centos7 ✖debian. JID-1395 |
Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>
|
Packaging result: ✔centos7 ✔debian. JID-1396 |
|
@shwstppr can you club your cks PRs, and at least change base branch to 4.14? |
|
Moved #4116 |
Description
Fixes #4133
Fixes #4056
Types of changes
Screenshots (if appropriate):
How Has This Been Tested?