-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
create integration test for machine controller. #11
Comments
From @k-turowski on January 17, 2018 15:55 I can take this. |
This is currently unassigned. |
/assign @spew |
is this to be tested with all real actuator implementations? or would it be using say a fake actuator stub? |
@enxebre - we should have a conformance-style test that each provider can run to verify that they work "correctly". It may be worthwhile to create an actuator stub in this repository so that we can put the test code here (and run it) once all of the provider code has been migrated out. |
/unassign |
Minor fixes to make clusterctl work with CRDs.
looks like no one is working on this, plan to take on this issue and assign to myself, let me know if any objection. |
@figo Go for it! You can mark the issue with |
@figo I was about to pick it! 🏃♂️ 😄 |
@pablochacin i have a WIP PR to create a simple mock up provider and integration with Prow. i definitely welcome cooperations. |
/lifecycle active |
@figo I kind reminder to share with me the repository fork on which you are working you PR. |
@pablochacin will share with you soon, thank you |
@pablochacin let me know if you want to have some discussion around this, here is the branch: https://github.com/figo/cluster-api/tree/integration |
@figo I just saw your PR. I'll comment there. Thanks |
/assign |
README: modify the build step for binaries
…ore-linters Activate more linters
✨ Add support for configuring NTP servers on bottlerocket through CAPI
From @medinatiger on January 8, 2018 19:46
We need to have integration test to test machine controller. The following scenario is interested.
Copied from original issue: kubernetes-retired/kube-deploy#476
The text was updated successfully, but these errors were encountered: