Skip to content
This repository has been archived by the owner on Dec 7, 2023. It is now read-only.

Can't ssh to machine with prefixed name #121

Closed
alexellis opened this issue Jul 10, 2019 · 5 comments · Fixed by #138
Closed

Can't ssh to machine with prefixed name #121

alexellis opened this issue Jul 10, 2019 · 5 comments · Fixed by #138
Assignees
Milestone

Comments

@alexellis
Copy link
Contributor

This isn't an ambiguous name, it is an exact match, but I can't use "my-vm" because I have "my-vm-2"

Hope this is enough info

alex@nuc7:~/go/src/github.com/openfaas/faas-netes$ sudo ignite ssh my-vm
ambiguous VM query: "my-vm" matched the following IDs/names: my-vm-2, my-vm
@alexellis
Copy link
Contributor Author

Even when I did correct the name, it still failed to log in with centos - connection refused.

Ubuntu worked though:

ignite run weaveworks/ignite-ubuntu \
    --cpus 2 \
    --memory 1GB \
    --ssh \
    --name my-vm

Might be worth updating the ignite run example to use the above in that case?

@luxas luxas added this to the v0.4.1 milestone Jul 10, 2019
@luxas
Copy link
Contributor

luxas commented Jul 10, 2019

Right, I think the code that also respected exact matches was unintentionally refactored away 😅. Let's fix it!

centos - connection refused.

I've experienced that centos takes some more time than Ubuntu to initialize SSH. I'll open a new issue for better SSH UX (retries and custom error messages)

@twelho
Copy link
Contributor

twelho commented Jul 10, 2019

The code doing exact matches wasn't removed, but since it's called for each object individually and not tracked in the filters, it has no effect. Will move it to the IDNameFilter so we get it working again.

@twelho
Copy link
Contributor

twelho commented Jul 10, 2019

In the meantime, you can use the VM's ID (or a prefix of it) to select the correct VM.

@twelho
Copy link
Contributor

twelho commented Jul 11, 2019

Fixed in PR #138, thanks for noticing this @alexellis!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants