Bug 44062 - univention-app shell returns 126
univention-app shell returns 126
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: App Center
UCS 4.2
Other Linux
: P5 normal (vote)
: UCS 4.2
Assigned To: Felix Botner
Dirk Wiesenthal
: interim-4
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2017-03-24 11:07 CET by Felix Botner
Modified: 2017-04-04 18:28 CEST (History)
1 user (show)

See Also:
What kind of report is it?: Development Internal
What type of bug is this?: ---
Who will be affected by this bug?: ---
How will those affected feel about the bug?: ---
User Pain:
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional):
Max CVSS v3 score:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Felix Botner univentionstaff 2017-03-24 11:07:11 CET
4.2 + tine20 App

-> univention-app shell tine20 php -r '$config = require "/etc/tine20/config.inc.php"; echo $config["setupuser"]["password"];'

-> echo $!
126

syslog:
Mar 23 22:42:19 master dockerd[1364]: time="2017-03-23T22:42:19.840660438+01:00" level=error msg="failed to close stdin: rpc error: code = 2 desc = containerd: process not found for container"


works with docker exec
Comment 1 Felix Botner univentionstaff 2017-03-24 11:09:14 CET
docker exec -it also fails, so the -it options may be the problem
Comment 2 Felix Botner univentionstaff 2017-03-24 15:14:22 CET
univention-app shell app with a command now runs without -it, without a command the default command (bash) is used and then -it is set, also added parameters for -i and -t
Comment 3 Dirk Wiesenthal univentionstaff 2017-04-03 10:32:00 CEST
OK, works. Can you please add a changelog?

Because this is in a way an API change:

univention-app shell dudle vim /etc/inputrc
  => Unusable, used to work
univention-app shell -it dudle vim /etc/inputrc
  => Works

I guess noone used that, but still...
Apart from that I think the change is sane.
Comment 4 Felix Botner univentionstaff 2017-04-03 14:41:22 CEST
added changelog entry
Comment 5 Dirk Wiesenthal univentionstaff 2017-04-04 13:33:55 CEST
OK
Comment 6 Stefan Gohmann univentionstaff 2017-04-04 18:28:46 CEST
UCS 4.2 has been released:
 https://docs.software-univention.de/release-notes-4.2-0-en.html
 https://docs.software-univention.de/release-notes-4.2-0-de.html

If this error occurs again, please use "Clone This Bug".