-
Notifications
You must be signed in to change notification settings - Fork 70
"message too long" crashes runtime #891
Comments
@dhrp nice catch ! |
Perhaps you can give me some pointers to how to include / replace this, and rebuild as it is not obvious to me where the source comes from / how it gets into the pkg directory. I'm on #clearcontainers irc as thatcher |
Hi @dhrp - please don't feel you have to raise a PR, but if you'd like to contribute, that would be awesome. What you'll need to do is:
If that all works, you can then raise a PR on virtcontainers (not the runtime) to remove the Once that fix lands in virtcontainers, we can then raise a PR on the runtime to "re-vendor" (update) the version of virtcontainers used by the runtime. At that point the bug will have been fully fixed. I've raised an issue on our I'm also aware that the dev doc referenced above is missing the docker config instructions wrt the runtime I've outlined above, so I'll raise a PR to get that added... |
Dev guide update PR raised: #893. |
Thanks for the help @jodh-intel; it was confusing to see a directory pkg that is not a vendor dir; that's why.. And I have good news and bad news: The good news is that with the latest (master) the error occurs but no longer completely crashes the runtime. It just says "message too long" (not a huge issue)_ The bad news is that simply uncommenting the length check doesn't solve the problem, in fact. if I uncomment that line (and recompile), the system will crash and halt everything (again). |
Hi @dhrp - yes, there is more to this than we initially thought. In fact, the length check needs to be removed from the runtime, the agent and the proxy. But an env var
Could you take a look @amshinde? |
re-ping @amshinde :) |
Description of problem
When I try to start a container with a (very) long docker run command the runtime errors with something like "message too long"
The real issue is that the runtime then also immediately stops working and I can no longer 'docker ps' or start or stop containers on this host until I reboot.
Steps to reproduce:
$ docker run -e ENV11000=$ENV11000 ubuntu echo $ENV11000
where $ENV11000 is a string more then 10240 characters long.
I have traced the source of the "message too long" to here: https://github.com/containers/virtcontainers/blob/54b8c0cc68933561bc947c8b76aa491bb925f3ae/pkg/hyperstart/hyperstart.go#L345
journalctl thows some errors like:
Expected result
the container runs
Actual result
see above
root@c1:~# cc-collect-data.sh
Meta details
Running
cc-collect-data.sh
version3.0.10 (commit 3d402d1)
at2018-01-04.08:59:52.838419177
.Runtime is
/usr/bin/cc-runtime
.cc-env
Output of "
/usr/bin/cc-runtime cc-env
":Runtime config files
Runtime default config files
Runtime config file contents
Config file
/etc/clear-containers/configuration.toml
not foundOutput of "
cat "/usr/share/defaults/clear-containers/configuration.toml"
":Logfiles
Runtime logs
Recent runtime problems found in system journal:
Proxy logs
No recent proxy problems found in system journal.
Shim logs
No recent shim problems found in system journal.
Container manager details
Have
docker
Docker
Output of "
docker info
":No
kubectl
Packages
Have
dpkg
Output of "
dpkg -l|egrep "(cc-proxy|cc-runtime|cc-shim|clear-containers-image|linux-container|qemu-lite|qemu-system-x86|cc-oci-runtime)"
":No
rpm
The text was updated successfully, but these errors were encountered: