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

docker broken on latest dunfell branch commit "http: invalid Host header" #277

Open
dwalkes opened this issue Oct 29, 2023 · 2 comments
Open

Comments

@dwalkes
Copy link
Member

dwalkes commented Oct 29, 2023

I just realized docker is completely broken on the latest dunfell branch of tegra-demo-distro with demo-image-full, I believe due to this issue. Any attempts to start a container fail with

root@jetson-nano-2gb-devkit:~# docker run hello-world
Unable to find image 'hello-world:latest' locally
latest: Pulling from library/hello-world
70f5ac315c5a: Pull complete
Digest: sha256:88ec0acaa3ec199d3b7eaf73588f4518c25f9d34f58ce9a0df68429c5af48e8d
Status: Downloaded newer image for hello-world:latest
http: invalid Host header

I don't see any fixes for this in meta-virtualization dunfell branch, nor do I see any in the corresponding docker-ce 19.03 or moby 19.03 branches.

I have verified this was introduced with this poky patch and have verified that reverting this commit resolves the issue.

I would rather just switch to kirkstone-l4t-r32.7.x, however I think there's a blocking issue there at OE4T/meta-tegra#1194 which would need to be backported before my deepstream docker example will work on kirkstone, and it looks like it won't be trivial to backport that one (although I haven't fully wrapped my head around it yet).

I’m curious if anyone has suggestions about how to resolve and/or if anyone else cares about docker support on dunfell.

@dwalkes dwalkes changed the title docker broken on latest dunfell branch commit docker broken on latest dunfell branch commit "http: invalid Host header" Oct 29, 2023
@moto-timo
Copy link

CVE patches can be imperfect. So it is possible something specific about that poky commit is at issue. If you can figure out some hints as to how it broke things, you can either file a bug or send email to the mailing lists.

dwalkes added a commit to cu-ecen-aeld/tegra-demo-distro that referenced this issue Oct 29, 2023
Use a temporary poky repo until a better solution for [1]

1: OE4T#277

Signed-off-by: Dan Walkes <danwalkes@trellis-logic.com>
@dwalkes
Copy link
Member Author

dwalkes commented Oct 30, 2023

Thanks @moto-timo

If you can figure out some hints as to how it broke things, you can either file a bug or send email to the mailing lists.

I think it's spelled out here ... as I understand it a breaking change in a CVE for go which required a change on docker to accomodate. The problem is that docker 19.03 from dunfell isn't maintained upstream based on discussion here in the meta-virtualization mailing list.

I've posted a message to meta-virtualization asking for suggestions.

dwalkes added a commit to cu-ecen-aeld/tegra-demo-distro that referenced this issue Nov 11, 2023
Use a temporary poky repo until a better solution for [1]

1: OE4T#277

Signed-off-by: Dan Walkes <danwalkes@trellis-logic.com>
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

2 participants