-
Notifications
You must be signed in to change notification settings - Fork 12
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
Add git workflow for a test that submits a k8s job #35
Conversation
/test-k8s |
13a7f64
to
69cd14e
Compare
.github/workflows/go.yml
Outdated
@@ -1,7 +1,7 @@ | |||
# This workflow will build a golang project | |||
# For more information see: https://docs.github.com/en/actions/automating-builds-and-tests/building-and-testing-go | |||
|
|||
name: Go | |||
name: PR Checks |
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.
Please rename it back to Go. I think it is expected.
eda9c0b
to
0e6622e
Compare
.github/workflows/go.yml
Outdated
kubectl create ns k8s-test | ||
kubectl create priorityclass normal-priority --value=100000 | ||
./bin/knavigator -tasks resources/tests/k8s/test-job.yml | ||
kubectl get job -n k8s-test |
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.
Suggestion: replace lines 111-114 with:
./bin/knavigator -tasks resources/tests/k8s/test-job.yml
kubectl get job
kubectl get pod
kubectl delete job job1
02b82fa
to
b7499e3
Compare
Signed-off-by: Shinae Woo <[email protected]>
229ee0e
to
05b0032
Compare
Signed-off-by: Shinae Woo <[email protected]>
No description provided.