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

no proper docker file and image on docker hub does not follow instructions #89

Closed
HomeACcessoryKid opened this issue Nov 14, 2022 · 9 comments · Fixed by #138
Closed

Comments

@HomeACcessoryKid
Copy link

The docker file given in tools is not a propor file because it is not complete and does not start with FROM ubuntu or similar.
The file should follow the instructions in https://docs.espressif.com/projects/esp-matter/en/main/esp32/developing.html#development-setup

And then the docker hub file should reflect this all.

Please update to idf 4.4.3 as well.

@shubhamdp
Copy link
Contributor

@HomeACcessoryKid docker file is in correct format and it does contain FROM statement.
As it is using some another base image that's why it does not contain FROM ubuntu or similar statement at the start.

Just to clear one more thing, it is not the development setup for esp-matter, the purpose of this docker image is to run the ci using this one.

Regarding v4.4.3, esp-matter is using branch: v1.0-branch of connectedhomeip repository and it is still on v4.4.2.
Will update to v4.4.3 after some testing is done and things looks stable.

@HomeACcessoryKid
Copy link
Author

Yes, the actual docker file starts with a FROM. But the thing it is starting from itself does not have such FROM.
And it is OK it has a different purpose.

I suggest you make a second docker image that provides the environment that you suggest people use in the developing page. And it must be something that I can track back to something official. Else I must build it myself.
I cannot afford a backdoor or so because the source is not traceable.
So please provide a clear docker file and as a convenience, the image that comes from it in docker hub.

@HomeACcessoryKid
Copy link
Author

sorry, Only now did I see you created a reference to the docker file that is the source and it does contain a FROM ubuntu.
I doubt however that I would be able to link the two from the information I have available.
But fundamentally, I still hope to get a image that focuses on the 'developing' page.

@shubhamdp
Copy link
Contributor

@HomeACcessoryKid Thanks for the suggestion, will generate and push the esp-matter development image to docker hub soon.

@HomeACcessoryKid
Copy link
Author

HomeACcessoryKid commented Dec 2, 2022 via email

@dhrishi
Copy link
Collaborator

dhrishi commented Dec 16, 2022

@HomeACcessoryKid We haven't officially moved Matter for ESP32, ESP32-C3, ESP32-S3 to ESP-IDF v5.0 yet. You can refer this if you want to give it a try

@dhrishi
Copy link
Collaborator

dhrishi commented Dec 16, 2022

To give support for the c2 and h2 and to embrace the future today...

Yes, C2 support will be added soon and will be based on IDF v5.0. H2 already supports v5.0.

@HomeACcessoryKid
Copy link
Author

Maybe I am missing something, but https://hub.docker.com/r/espressif/esp-matter/tags only shows the two month old image.
Did something go wrong after you would "generate and push the esp-matter development image to docker hub soon"?

@shubhamdp
Copy link
Contributor

@HomeACcessoryKid We have added the esp-matter development docker image. You can pull it as follows

docker pull espressif/esp-matter:latest

We have not yet consider the IDF v5.0 in this, but will soon do that.
We are closing this issue and feel free to open one for IDF v5.0

@dhrishi dhrishi closed this as completed Jan 12, 2023
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

Successfully merging a pull request may close this issue.

3 participants