Skip to content
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

Native functions cleanup #106

Open
frisso opened this issue Aug 5, 2016 · 3 comments
Open

Native functions cleanup #106

frisso opened this issue Aug 5, 2016 · 3 comments
Assignees

Comments

@frisso
Copy link
Member

frisso commented Aug 5, 2016

(sorry for the Italian here)
Leaving this open as a reminder, as Roberto is on holiday and we need to fix those issues.

https://github.com/netgroup-polito/un-orchestrator/blob/master/orchestrator/compute_controller/plugins/native/scripts/pullAndRunNativeNF_ns.sh#L132 come mai non lanci lo script in back ground? E se non ritorna? Lo universal node orchestrator rimarrebbe piantato li.
Secondo me lo script dovrebbe essere lanciato in background.. Ti torna?

Inoltre ci sono due script nella cartella, che hanno lo stesso nome ma uno termina co "_ns". Vedendo il codice, questo mette le native nel namespace, l'altro no, e credo sia una cosa vecchia. Se cosi' e', posso cancellare questi script vecchi?

@frisso frisso assigned frisso and rbrtbnfgl and unassigned frisso Aug 5, 2016
@ivanocerrato
Copy link
Member

@rbrtbnfgl are you sure that this is correct?

https://github.com/netgroup-polito/un-orchestrator/blob/stable/orchestrator/compute_controller/plugins/native/scripts/pullAndRunNativeNF_ns.sh#L132

You only change the network namespace with this command. Then, you use it to execute for example ls, you list the files in the directory containing the un-orchestrator binary.
Is this what you want?

@ivanocerrato
Copy link
Member

@rbrtbnfgl is this issue solved?

@ivanocerrato
Copy link
Member

@rbrtbnfgl what is the status of this issue?

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

No branches or pull requests

3 participants