Skip to content
This repository has been archived by the owner on May 12, 2021. It is now read-only.

virtcontainers: Reduce path lengths #294

Merged
merged 1 commit into from
May 10, 2018

Conversation

jodh-intel
Copy link
Contributor

Reduce the virtcontainers prefix path to avoid hitting the 107 byte
Unix domain socket path limit.

Related #268.

Fixes #290.

Signed-off-by: James O. D. Hunt [email protected]

Reduce the virtcontainers prefix path to avoid hitting the 107 byte
Unix domain socket path limit.

Related kata-containers#268.

Fixes kata-containers#290.

Signed-off-by: James O. D. Hunt <[email protected]>
@jodh-intel
Copy link
Contributor Author

With this change:

$ sudo find /var/run/vc/sbs | grep console.sock | wc -c
94

Saving 112 - 94 = 18 bytes and giving us 107 - 94 = 13 bytes before we hit the limit.

@codecov
Copy link

codecov bot commented May 9, 2018

Codecov Report

Merging #294 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #294   +/-   ##
=======================================
  Coverage   64.53%   64.53%           
=======================================
  Files          86       86           
  Lines        8397     8397           
=======================================
  Hits         5419     5419           
  Misses       2404     2404           
  Partials      574      574
Impacted Files Coverage Δ
virtcontainers/filesystem.go 68.39% <ø> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f6544a3...6a47808. Read the comment docs.

Copy link
Member

@amshinde amshinde left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@jodh-intel
Copy link
Contributor Author

Ping @egernst, @jcvenegas, @devimc, @bergwolf.

@egernst egernst merged commit 91e9ed0 into kata-containers:master May 10, 2018
jodh-intel added a commit to jodh-intel/ksm-throttler that referenced this pull request May 29, 2018
Update for the new Virtcontainers root directory.

See:

- kata-containers/runtime#200
- kata-containers/runtime#294

Fixes kata-containers#37.

Signed-off-by: James O. D. Hunt <[email protected]>
jodh-intel added a commit to jodh-intel/ksm-throttler that referenced this pull request Jun 5, 2018
Update for the new Virtcontainers root directory.

See:

- kata-containers/runtime#200
- kata-containers/runtime#294

Fixes kata-containers#37.

Signed-off-by: James O. D. Hunt <[email protected]>
jodh-intel added a commit to jodh-intel/ksm-throttler that referenced this pull request Sep 10, 2018
Update for the new Virtcontainers root directory.

See:

- kata-containers/runtime#200
- kata-containers/runtime#294

Fixes kata-containers#37.

Signed-off-by: James O. D. Hunt <[email protected]>
zklei pushed a commit to zklei/runtime that referenced this pull request Jun 13, 2019
We don't know how much time a sandbox can be paused,
hence connection write timeout should be disabled to
don't close the connection while the sandbox is paused.

The same issue has been fixed in kata-proxy, for katabuiltin proxy,
it also needs this fix.

fixes kata-containers#294

Signed-off-by: fupan <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Reduce virtcontainers path lengths
3 participants