-
Notifications
You must be signed in to change notification settings - Fork 48
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
Fix user apps test suite #944
Comments
@milan-zededa could it be because of eden SDN? |
@uncleDecart Networking is OK but the nodered application halts shortly after booting:
|
For comparison, this is how nodered logs look like when everything is OK (run on 9.4.10 EVE):
|
So, the error (
Probably we try to get it from somewhere else... |
@shjala Could do you please check if this issue is related to your changes in PR lf-edge/eve#3521 ? You can run it with:
|
@milan-zededa let me check it. |
@milan-zededa @giggsoff you need to have that entry in the VM's |
But in case of USER defined in Dockerfile we will have it only inside of container's rootfs. Assume that VM's |
@giggsoff but we want to run the container as the host's user 1000, not the container's user 1000. |
we can create the user in host if doesn't exist with some default permissions. |
@shjala by host here you mean VM where container is running in EVE and by user 1000 you mean first created user, right? In that case, how should we handle potential case where we run multiple containers in same VM to reduce overhead (lf-edge/eve#3658)? |
@uncleDecart
It is OK to run multiple containers under the same user, but that has security concerns which I need to address.
We should change the init script, if user is not root and does not exist on the vm, create it with default permissions. I'm busy right now, but I have plan submit PR next week to fix that, feel free to beat me to it. |
After a re-run of lf-edge/eve#3742 I see that the User apps test suite passed. |
That's great, I guess we can close this issue once we release EVE with a fix and bump version in Eden repo, so that there will be red test on eden as well |
Bumped and fixed |
What: Test Bug
Currently in master user app test suite fails. Probably it has to do something with port forwarding, because test fails on retrieving curl from a website which is actually available, so link is not broken.
Happens on master: yes
Reproducable locally: yes
The text was updated successfully, but these errors were encountered: