-
Notifications
You must be signed in to change notification settings - Fork 38.7k
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
adding --ssh-key
and --ssh-user
for kubetest2
#105637
Conversation
Signed-off-by: Namanl2001 <namanlakhwani@gmail.com>
@Namanl2001: This issue is currently awaiting triage. If a SIG or subproject determines this is a relevant issue, they will accept it by applying the The Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Welcome @Namanl2001! |
Hi @Namanl2001. Thanks for your PR. I'm waiting for a kubernetes member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/ok-to-test |
/sig node |
@@ -47,6 +47,8 @@ timeout_arg="" | |||
system_spec_name=${SYSTEM_SPEC_NAME:-} | |||
extra_envs=${EXTRA_ENVS:-} | |||
runtime_config=${RUNTIME_CONFIG:-} | |||
ssh_user=${SSH_USER:-} | |||
ssh_key=${SSH_KEY:-} |
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.
these would most likely be empty when not run with kubetest2 (i.e. directly make)
maybe add something like:
ssh_key=${SSH_KEY:-${GCE_SSH_PRIVATE_KEY_FILE:-${USER}/.ssh/google_compute_engine}}
/test pull-kubernetes-node-e2e-kubetest2 |
non kubetest2 jobs passes after adding quotes for |
Signed-off-by: Namanl2001 <namanlakhwani@gmail.com>
/test pull-kubernetes-node-e2e-kubetest2 |
Oh I meant there might be an issue with how we are passing the args, not sure The bash command is trying to run seems to be off https://prow.k8s.io/view/gs/kubernetes-jenkins/pr-logs/pull/105637/pull-kubernetes-node-e2e-kubetest2/1450522436058484736#1:build-log.txt%3A293
seems like we are somehow unquoting the skip flags and it's dropping everything before the pipe |
Signed-off-by: Namanl2001 <namanlakhwani@gmail.com>
reverted previous commit change: c6e0564 |
/test pull-kubernetes-node-e2e-kubetest2 |
looks like more quoting errors 😅
the makefile seems to be adding it's own quotes kubernetes/hack/make-rules/test-e2e-node.sh Lines 57 to 63 in 6cbe185
so we probably need to just pass it raw without any quotes |
let's try this too 🤞 |
/test pull-kubernetes-node-e2e-kubetest2 |
for the failure of |
pull-kubernetes-node-e2e-containerd-kubetest2 successful run is awesome to see!
yeah, either that or quote the entire thing including |
/test pull-kubernetes-node-e2e-kubetest2 |
🤩 Awesome to see green kubetest2 jobs. just testing the remaining ones /test pull-kubernetes-node-e2e-containerd-features-kubetest2 |
@amwat all tests should pass after the changes in this PR, right? /retest |
looking at the failures, seems like they are not related to ssh. Stabilizing the other jobs might be involved and could need follow ups. /lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: amwat, dims, Namanl2001 The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/hold cancel |
@Namanl2001: The following tests failed, say
Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
Signed-off-by: Namanl2001 namanlakhwani@gmail.com
This PR is part of the CI migration from kubetest to kubetest2
https://github.com/kubernetes/enhancements/tree/master/keps/sig-testing/2464-kubetest2-ci-migration
--ssh-key
and--ssh-user
are not present for kubetest2, which is leading to this error:failed to run SSH command: out: Warning: Identity file /root/.ssh/google_compute_engine not accessible: No such file or directory :
ref: https://prow.k8s.io/view/gs/kubernetes-jenkins/pr-logs/pull/92316/pull-kubernetes-node-e2e-kubetest2/1446852497472753664#1:build-log.txt%3A114
Using these args in kubetest2 via kubernetes-sigs/kubetest2#169