-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Use the latest docker image for for CI and cloud builds #22157
Use the latest docker image for for CI and cloud builds #22157
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@shubhamdp should we change integrations/cloudbuild images as well in case new changes depend on new IDF SDK support?
PR #22157: Size comparison from bf75296 to 6b110fa Increases above 0.2%:
Increases (5 builds for esp32, psoc6, telink)
Decreases (5 builds for cyw30739, esp32, psoc6, telink)
Full report (32 builds for bl602, cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, telink)
|
@andy31415 I update those images as well. Shall I do that in this PR or another PR? |
Either works - cloudbuild changes will not be validated by CI anyway. |
Head branch was pushed to by a user without write access
acea927
to
10c7627
Compare
PR #22157: Size comparison from 1952eb7 to 10c7627 Increases above 0.2%:
Increases (3 builds for cc13x2_26x2, esp32)
Decreases (8 builds for cc13x2_26x2, cyw30739, esp32, psoc6, telink)
Full report (43 builds for bl602, cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, telink)
|
10c7627
to
2862dca
Compare
PR #22157: Size comparison from 35c4d69 to 2862dca Increases above 0.2%:
Increases (4 builds for bl602, esp32, psoc6)
Decreases (5 builds for cc13x2_26x2, esp32, nrfconnect, telink)
Full report (43 builds for bl602, cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, telink)
|
ToT now supports ESP-IDF v4.4.2. Also, the docker image is uploaded to the dockerhub. Start using the latest docker image with ESP-IDF v4.4.2
2862dca
to
dea73c2
Compare
PR #22157: Size comparison from 934adfd to dea73c2 Increases above 0.2%:
Increases (5 builds for cyw30739, esp32, nrfconnect)
Decreases (7 builds for bl602, esp32, psoc6, telink)
Full report (43 builds for bl602, cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, telink)
|
…#22157) * [ESP32] Use the latest docker image for for ci builds ToT now supports ESP-IDF v4.4.2. Also, the docker image is uploaded to the dockerhub. Start using the latest docker image with ESP-IDF v4.4.2 * Update image to chip-build-vscode-0.5.96 for integrations/cloudbuild
After the merge of #21815, ToT supports ESP-IDF v4.4.2.
Also, the chip-build-esp32 docker image is uploaded to the dockerhub.
Problem
Change overview
Start using the latest docker container with ESP-IDF v4.4.2
Testing
CI should be green