Skip to content
This repository has been archived by the owner on Mar 28, 2018. It is now read-only.

Implement "exec" #18

Closed
jodh-intel opened this issue Jul 18, 2016 · 7 comments
Closed

Implement "exec" #18

jodh-intel opened this issue Jul 18, 2016 · 7 comments

Comments

@jodh-intel
Copy link
Contributor

The exec command is blocked on network support. Once networking is available, the function clr_oci_vm_connect() needs to be updated to remove the g_critical ("not implemented yet") call.

@jodh-intel
Copy link
Contributor Author

Blocked on #38.

@devimc
Copy link
Contributor

devimc commented Jul 25, 2016

I took a look to current implementation of exec and I noticed that it uses ssh to connect to VM, this is a good approach but what will happen with minimal container images (busybox) that has not sshd ?

@gorozco1
Copy link
Contributor

good catch @devimc, I think the ssh daemon will be provided by the mini-os when the clear-container is launched.

@gorozco1
Copy link
Contributor

@devimc has some updates about this issue

@jcvenegas jcvenegas self-assigned this Sep 29, 2016
@grahamwhaley
Copy link

now depends on #302

@dlespiau
Copy link
Contributor

As part of this, we need to document that while the exec will work, the things people may use the exec function for (like manual networking setup) may not necessary work because it will be running inside of the VM.

@dlespiau
Copy link
Contributor

Alright, time to close that one. Bugs found for exec should be opened as new issues (we've started already).

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

No branches or pull requests

7 participants