-
Notifications
You must be signed in to change notification settings - Fork 220
testing how-to for k8s changes #186
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
testing how-to for k8s changes #186
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
awesome, esp. the behoove
this is great, thanks for writing these up Shane. |
not a problem! after spending 2-3 days helping someone test their PR before finding out they were using the |
lgtm, note I didn't actually try the steps but probably will be in the next couple week.s |
FWIW, these commands are pulled directly from the k8s integration test build, so things should be relatively simple to get set up on a dev's end. regarding merging -- do i have perms to run the merge script in the website dir, or does someone else do that for me? ;) |
I'm not sure on permissions |
alright, i'll try and push from my end and see what happens. |
ok let us know if it doesn't work |
woot! |
i think that this will be quite useful. :)