-
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
[Telink] change Docker version 26 -> 29 #30979
Conversation
PR #30979: Size comparison from a9ad2c4 to fdf3008 Decreases (1 build for efr32)
Full report (72 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
|
Failing build on Telink platform after "29 : [ESP] Update esp-idf to v5.1.2"
fdf3008
to
d79845f
Compare
PR #30979: Size comparison from a9ad2c4 to d79845f Decreases (1 build for efr32)
Full report (72 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
|
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.
I am unclear about why we bump telink only: the latest telink update seems to be version 26 only.
Looking at the history, https://github.com/project-chip/connectedhomeip/commits/master/integrations/docker/images/stage-2/chip-build-telink/Dockerfile updates at #30462 which is version 26 |
Fix is applied here: #30987 |
Failing bootstrap after Docker version updated in other places ( possibly "29 : [ESP] Update esp-idf to v5.1.2" #30920 ).
In fact, Github CI wouldn't show any build failures as it used 26 version.
Tests manually on Telink platform showed issues during bootstrap with python module
prompt_toolkit
.Changing its version in
scripts/setup/constraints.txt
to3.0.43
fixed bootstrap issues