-
Notifications
You must be signed in to change notification settings - Fork 44
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
implement minikube class for cluster create/delete/modify/display #268
Conversation
WoW! This is an exciting feature! |
@312hzeng Let us know when you think you can merge it. |
Thank you Professor Xu! I think I'll ask Tyler about his suggestions for the code revision regarding class method implementation and related testcases on tomorrow's meeting before this pr can get merged. |
great btw how did you test your feature/PR? @312hzeng |
Thank you for the question, I use pytest to test individual feature. I think there's more to add to the testcases, I just want to discuss the update with Tyler first tomorrow before I add more testing |
BTW the End-to-End Test runs on riverrun, which has been flaky recently. The End-to-End Test may fail due to timeout (24 hrs), I will run it on a cloudlab machine. |
ok, thank you for the information |
7cfe95a
to
1102aa8
Compare
633d80c
to
39b317e
Compare
Coverage reportClick to see where and how coverage changed
This report was generated by python-coverage-comment-action |
* Prepare system state interface and collection Signed-off-by: Tyler Gu <[email protected]> * Fix system state test Signed-off-by: Tyler Gu <[email protected]> * Fix temporary file creation in test Signed-off-by: Tyler Gu <[email protected]> --------- Signed-off-by: Tyler Gu <[email protected]>
Signed-off-by: Tyler Gu <[email protected]>
Signed-off-by: Tyler Gu <[email protected]>
Signed-off-by: Tyler Gu <[email protected]>
…lass(not pass pylint)
No description provided.