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.

Sign up for GitHub

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

Jump to bottom

Put software version string as <branch>:<commit_id> on esp32 while doing automation testing #19943

Merged
merged 1 commit into from
Jun 26, 2022
Merged

Put software version string as <branch>:<commit_id> on esp32 while doing automation testing #19943

merged 1 commit into from
Jun 26, 2022

Conversation

MtTsai
Copy link
Contributor

@MtTsai MtTsai commented Jun 24, 2022

Problem

The value of software version string is needed for test automation on esp32 platform.

Change overview

Put the software version string with format :<commit_id> in the CHIP_DEVICE_CONFIG_DEVICE_SOFTWARE_VERSION_STRING by chef.py for esp32 sample app of chef build

Testing

For esp32 platform, build the binary with command ./chef.py -zbra -d rootnode_dimmablelight_gY80DaqEUL -t esp32
For linux platform, build the binary with command ./chef.py -zbra -d rootnode_dimmablelight_gY80DaqEUL -t linux
And use RPC console to check the output is correct.

@github-actions
Copy link

github-actions bot commented Jun 24, 2022 •
edited
Loading

PR #19943: Size comparison from 47c9ea5 to c5ae469

Increases (1 build for telink)
platform target config section 47c9ea5 c5ae469 change % change
telink light-switch-app tlsr9518adk80d text 561222 561224 2 0.0
Decreases (1 build for nrfconnect)
platform target config section 47c9ea5 c5ae469 change % change
nrfconnect all-clusters-minimal-app nrf52840dk_nrf52840 text 796064 796060 -4 -0.0
Full report (41 builds for cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
platform target config section 47c9ea5 c5ae469 change % change
cc13x2_26x2 all-clusters-app LP_CC2652R7 (read only) 660539 660539 0 0.0
(read/write) 191532 191532 0 0.0
.bss 74964 74964 0 0.0
.data 3352 3352 0 0.0
.rodata 87059 87059 0 0.0
.text 573168 573168 0 0.0
all-clusters-minimal-app LP_CC2652R7 (read only) 649431 649431 0 0.0
(read/write) 158520 158520 0 0.0
.bss 74252 74252 0 0.0
.data 3352 3352 0 0.0
.rodata 90671 90671 0 0.0
.text 558440 558440 0 0.0
lock-ftd LP_CC2652R7 (read only) 663523 663523 0 0.0
(read/write) 178692 178692 0 0.0
.bss 71996 71996 0 0.0
.data 3276 3276 0 0.0
.rodata 75179 75179 0 0.0
.text 587864 587864 0 0.0
lock-mtd LP_CC2652R7 (read only) 612939 612939 0 0.0
(read/write) 145124 145124 0 0.0
.bss 67732 67732 0 0.0
.data 3276 3276 0 0.0
.rodata 75059 75059 0 0.0
.text 537392 537392 0 0.0
pump-app LP_CC2652R7 (read only) 673439 673439 0 0.0
(read/write) 169640 169640 0 0.0
.bss 72092 72092 0 0.0
.data 3276 3276 0 0.0
.rodata 87831 87831 0 0.0
.text 585124 585124 0 0.0
pump-controller-app LP_CC2652R7 (read only) 659319 659319 0 0.0
(read/write) 183872 183872 0 0.0
.bss 72204 72204 0 0.0
.data 3272 3272 0 0.0
.rodata 83671 83671 0 0.0
.text 575168 575168 0 0.0
shell LP_CC2652R7 (read only) 690378 690378 0 0.0
(read/write) 157196 157196 0 0.0
.bss 77268 77268 0 0.0
.data 3356 3356 0 0.0
.rodata 109922 109922 0 0.0
.text 580144 580144 0 0.0
cyw30739 light cyw930739m2evb_01 (read/write) 592610 592610 0 0.0
.app_xip_area 462732 462732 0 0.0
.bss 72848 72848 0 0.0
.data 712 712 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
lock cyw930739m2evb_01 (read/write) 594218 594218 0 0.0
.app_xip_area 464132 464132 0 0.0
.bss 73048 73048 0 0.0
.data 716 716 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
ota-requestor-no-progress-logging cyw930739m2evb_01 (read/write) 595238 595238 0 0.0
.app_xip_area 466232 466232 0 0.0
.bss 72032 72032 0 0.0
.data 656 656 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
efr32 lighting-app BRD4161A (read/write) 1072668 1072668 0 0.0
.bss 133716 133716 0 0.0
.data 2044 2044 0 0.0
.text 936888 936888 0 0.0
BRD4161A+rpc (read/write) 1126756 1126756 0 0.0
.bss 150404 150404 0 0.0
.data 2256 2256 0 0.0
.text 974076 974076 0 0.0
BRD4161A+rs911x (read/write) 937376 937376 0 0.0
.bss 140088 140088 0 0.0
.data 2044 2044 0 0.0
.text 795220 795220 0 0.0
lock-app BRD4161A+wf200 (read/write) 1117580 1117580 0 0.0
.bss 140536 140536 0 0.0
.data 2056 2056 0 0.0
.text 974968 974968 0 0.0
window-app BRD4161A (read/write) 1057920 1057920 0 0.0
.bss 133788 133788 0 0.0
.data 2072 2072 0 0.0
.text 922040 922040 0 0.0
esp32 all-clusters-app c3devkit (read only) 1014322 1014322 0 0.0
(read/write) 1485698 1485698 0 0.0
.dram0.bss 71424 71424 0 0.0
.dram0.data 14624 14624 0 0.0
.flash.rodata 214224 214224 0 0.0
.flash.text 1014322 1014322 0 0.0
.iram0.text 62902 62902 0 0.0
m5stack (read only) 1068583 1068583 0 0.0
(read/write) 487800 487800 0 0.0
.dram0.bss 76936 76936 0 0.0
.dram0.data 34144 34144 0 0.0
.flash.rodata 244724 244724 0 0.0
.flash.text 1063199 1063199 0 0.0
.iram0.text 123267 123267 0 0.0
k32w light k32w061+release (read/write) 662232 662232 0 0.0
.bss 70396 70396 0 0.0
.data 1988 1988 0 0.0
.text 584048 584048 0 0.0
lock k32w061+release (read/write) 694528 694528 0 0.0
.bss 70868 70868 0 0.0
.data 2000 2000 0 0.0
.text 615860 615860 0 0.0
linux all-clusters-app debug (read only) 2938833 2938833 0 0.0
(read/write) 177040 177040 0 0.0
.bss 84192 84192 0 0.0
.data 2016 2016 0 0.0
.data.rel.ro 84632 84632 0 0.0
.dynamic 608 608 0 0.0
.got 4536 4536 0 0.0
.init 27 27 0 0.0
.init_array 1032 1032 0 0.0
.rodata 261597 261597 0 0.0
.text 2500914 2500914 0 0.0
all-clusters-minimal-app debug (read only) 2792913 2792913 0 0.0
(read/write) 168944 168944 0 0.0
.bss 83520 83520 0 0.0
.data 2016 2016 0 0.0
.data.rel.ro 77272 77272 0 0.0
.dynamic 608 608 0 0.0
.got 4488 4488 0 0.0
.init 27 27 0 0.0
.init_array 1032 1032 0 0.0
.rodata 263229 263229 0 0.0
.text 2355538 2355538 0 0.0
bridge-app debug+rpc (read only) 2296897 2296897 0 0.0
(read/write) 147872 147872 0 0.0
.bss 71616 71616 0 0.0
.data 3792 3792 0 0.0
.data.rel.ro 66696 66696 0 0.0
.dynamic 608 608 0 0.0
.got 4400 4400 0 0.0
.init 27 27 0 0.0
.init_array 728 728 0 0.0
.rodata 196416 196416 0 0.0
.text 1939394 1939394 0 0.0
chip-tool debug (read only) 10166973 10166973 0 0.0
(read/write) 610600 610600 0 0.0
.bss 24416 24416 0 0.0
.data 1088 1088 0 0.0
.data.rel.ro 578800 578800 0 0.0
.dynamic 624 624 0 0.0
.got 5008 5008 0 0.0
.init 27 27 0 0.0
.init_array 640 640 0 0.0
.rodata 508549 508549 0 0.0
.text 8250389 8250389 0 0.0
chip-tool-no-interactive-ipv6only arm64 (read only) 9903788 9903788 0 0.0
(read/write) 675249 675249 0 0.0
.bss 42641 42641 0 0.0
.data 1152 1152 0 0.0
.data.rel.ro 614176 614176 0 0.0
.dynamic 528 528 0 0.0
.got 13472 13472 0 0.0
.init 24 24 0 0.0
.init_array 192 192 0 0.0
.rodata 471484 471484 0 0.0
.text 7893428 7893428 0 0.0
lighting-app debug+rpc (read only) 2533665 2533665 0 0.0
(read/write) 151896 151896 0 0.0
.bss 72096 72096 0 0.0
.data 2064 2064 0 0.0
.data.rel.ro 71848 71848 0 0.0
.dynamic 608 608 0 0.0
.got 4432 4432 0 0.0
.init 27 27 0 0.0
.init_array 816 816 0 0.0
.rodata 212168 212168 0 0.0
.text 2151826 2151826 0 0.0
lock-app debug (read only) 2495849 2495849 0 0.0
(read/write) 146904 146904 0 0.0
.bss 70528 70528 0 0.0
.data 1680 1680 0 0.0
.data.rel.ro 68824 68824 0 0.0
.dynamic 608 608 0 0.0
.got 4432 4432 0 0.0
.init 27 27 0 0.0
.init_array 784 784 0 0.0
.rodata 227016 227016 0 0.0
.text 2104322 2104322 0 0.0
ota-provider-app debug (read only) 2304809 2304809 0 0.0
(read/write) 140712 140712 0 0.0
.bss 70176 70176 0 0.0
.data 1912 1912 0 0.0
.data.rel.ro 62808 62808 0 0.0
.dynamic 608 608 0 0.0
.got 4496 4496 0 0.0
.init 27 27 0 0.0
.init_array 672 672 0 0.0
.rodata 202072 202072 0 0.0
.text 1940610 1940610 0 0.0
ota-requestor-app debug (read only) 2421881 2421881 0 0.0
(read/write) 147584 147584 0 0.0
.bss 72544 72544 0 0.0
.data 2200 2200 0 0.0
.data.rel.ro 66984 66984 0 0.0
.dynamic 608 608 0 0.0
.got 4488 4488 0 0.0
.init 27 27 0 0.0
.init_array 728 728 0 0.0
.rodata 205920 205920 0 0.0
.text 2045458 2045458 0 0.0
shell debug (read only) 2612817 2612817 0 0.0
(read/write) 196056 196056 0 0.0
.bss 111304 111304 0 0.0
.data 1232 1232 0 0.0
.data.rel.ro 77768 77768 0 0.0
.dynamic 608 608 0 0.0
.got 4168 4168 0 0.0
.init 27 27 0 0.0
.init_array 936 936 0 0.0
.rodata 231666 231666 0 0.0
.text 2221538 2221538 0 0.0
thermostat-no-ble arm64 (read only) 2579164 2579164 0 0.0
(read/write) 180593 180593 0 0.0
.bss 87905 87905 0 0.0
.data 1696 1696 0 0.0
.data.rel.ro 82888 82888 0 0.0
.dynamic 528 528 0 0.0
.got 5088 5088 0 0.0
.init 24 24 0 0.0
.init_array 400 400 0 0.0
.rodata 164812 164812 0 0.0
.text 2175296 2175296 0 0.0
tv-app debug (read only) 3082313 3082313 0 0.0
(read/write) 277800 277800 0 0.0
.bss 187720 187720 0 0.0
.data 4656 4656 0 0.0
.data.rel.ro 78984 78984 0 0.0
.dynamic 608 608 0 0.0
.got 4840 4840 0 0.0
.init 27 27 0 0.0
.init_array 952 952 0 0.0
.rodata 247840 247840 0 0.0
.text 2647682 2647682 0 0.0
tv-casting-app debug (read only) 5572105 5572105 0 0.0
(read/write) 184048 184048 0 0.0
.bss 72872 72872 0 0.0
.data 2416 2416 0 0.0
.data.rel.ro 102544 102544 0 0.0
.dynamic 608 608 0 0.0
.got 4712 4712 0 0.0
.init 27 27 0 0.0
.init_array 864 864 0 0.0
.rodata 341865 341865 0 0.0
.text 4954082 4954082 0 0.0
mbed lock-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2444192 2444192 0 0.0
.bss 214724 214724 0 0.0
.data 5872 5872 0 0.0
.text 1406836 1406836 0 0.0
nrfconnect all-clusters-app nrf52840dk_nrf52840 (read/write) 1203259 1203259 0 0.0
bss 143646 143646 0 0.0
rodata 156436 156436 0 0.0
text 824272 824272 0 0.0
all-clusters-minimal-app nrf52840dk_nrf52840 (read/write) 1150607 1150607 0 0.0
bss 142882 142882 0 0.0
rodata 132768 132768 0 0.0
text 796064 796060 -4 -0.0
p6 all-clusters-app default (read/write) 2556024 2556024 0 0.0
.bss 149872 149872 0 0.0
.data 2768 2768 0 0.0
.text 1514288 1514288 0 0.0
all-clusters-minimal-app default (read/write) 2501872 2501872 0 0.0
.bss 149152 149152 0 0.0
.data 2768 2768 0 0.0
.text 1460136 1460136 0 0.0
light-app default (read/write) 2432632 2432632 0 0.0
.bss 141232 141232 0 0.0
.data 2584 2584 0 0.0
.text 1390896 1390896 0 0.0
lock-app default (read/write) 2458896 2458896 0 0.0
.bss 141080 141080 0 0.0
.data 2600 2600 0 0.0
.text 1417160 1417160 0 0.0
telink light-switch-app tlsr9518adk80d (read/write) 793188 793188 0 0.0
bss 71924 71924 0 0.0
noinit 40416 40416 0 0.0
text 561222 561224 2 0.0
lighting-app tlsr9518adk80d (read/write) 812900 812900 0 0.0
bss 72172 72172 0 0.0
noinit 40416 40416 0 0.0
text 577684 577684 0 0.0

@tcarmelveilleux tcarmelveilleux merged commit c06a7af into project-chip:master Jun 26, 2022
cpagravel pushed a commit to cpagravel/connectedhomeip that referenced this pull request Jul 2, 2022
@MtTsai @cpagravel
Put software version string as <branch>:<commit_id> on esp32 while do…
5c6baa9 
…ing automation testing (project-chip#19943)
@cpagravel cpagravel mentioned this pull request Jul 2, 2022
Merged
andy31415 pushed a commit that referenced this pull request Jul 3, 2022
@cpagravel @MtTsai @aBozowski
Cherry pick - Chef changes (06-27:07-01). RPC logging, Linux cross pl…
dc14d56 
…atform building. (#20248)

* Put software version string as <branch>:<commit_id> on esp32 while doing automation testing (#19943)

* RPC: Support pw logging for linux rpc examples (#20057)

Add a config option CHIP_USE_PW_LOGGING, which uses pw log output
instead of printf on linux. This allows the logs to correctly get
HDLC encoded and be piped over the socket for linux RPC builds.

* Chef - Continue instead of breaking on build error (#19988)

* Move chef .matter files back to /chef/devices (#20142)

* Remove matter files from root zzz

* Move .matter regen for chef

* Update bundle for changed .matter location

* Chef - Support cross platform Linux builds for arm64 (#20232)

* Chef - Use array for passing linux args to args.gni

* Chef - Add support for cross compiling linux on arm64

Co-authored-by: MtTsai <7930801+MtTsai@users.noreply.github.com>
Co-authored-by: rgoliver <rgoliver@google.com>
Co-authored-by: Austin Bozowski <bozowski@google.com>
rerasool pushed a commit to SiliconLabs/matter that referenced this pull request Sep 9, 2022
@cpagravel @MtTsai
@aBozowski @danielsilabs
Cherry pick - Chef changes (06-27:07-01). RPC logging, Linux cross pl…
f059d5d 
…atform building. (project-chip#20248)

* Put software version string as <branch>:<commit_id> on esp32 while doing automation testing (project-chip#19943)

* RPC: Support pw logging for linux rpc examples (project-chip#20057)

Add a config option CHIP_USE_PW_LOGGING, which uses pw log output
instead of printf on linux. This allows the logs to correctly get
HDLC encoded and be piped over the socket for linux RPC builds.

* Chef - Continue instead of breaking on build error (project-chip#19988)

* Move chef .matter files back to /chef/devices (project-chip#20142)

* Remove matter files from root zzz

* Move .matter regen for chef

* Update bundle for changed .matter location

* Chef - Support cross platform Linux builds for arm64 (project-chip#20232)

* Chef - Use array for passing linux args to args.gni

* Chef - Add support for cross compiling linux on arm64

Co-authored-by: MtTsai <7930801+MtTsai@users.noreply.github.com>
Co-authored-by: rgoliver <rgoliver@google.com>
Co-authored-by: Austin Bozowski <bozowski@google.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@anush-apple anush-apple Awaiting requested review from anush-apple

@arkq arkq Awaiting requested review from arkq

@Byungjoo-Lee Byungjoo-Lee Awaiting requested review from Byungjoo-Lee

@carol-apple carol-apple Awaiting requested review from carol-apple

@chrisdecenzo chrisdecenzo Awaiting requested review from chrisdecenzo

@chshu chshu Awaiting requested review from chshu

@chulspro chulspro Awaiting requested review from chulspro

@Damian-Nordic Damian-Nordic Awaiting requested review from Damian-Nordic

@andy31415 andy31415 Awaiting requested review from andy31415

@dhrishi dhrishi Awaiting requested review from dhrishi

@electrocucaracha electrocucaracha Awaiting requested review from electrocucaracha

@emargolis emargolis Awaiting requested review from emargolis

@bzbarsky-apple bzbarsky-apple Awaiting requested review from bzbarsky-apple

@franck-apple franck-apple Awaiting requested review from franck-apple

@gjc13 gjc13 Awaiting requested review from gjc13

@harsha-rajendran harsha-rajendran Awaiting requested review from harsha-rajendran

@hawk248 hawk248 Awaiting requested review from hawk248

@isiu-apple isiu-apple Awaiting requested review from isiu-apple

@jelderton jelderton Awaiting requested review from jelderton

@jepenven-silabs jepenven-silabs Awaiting requested review from jepenven-silabs

@jmartinez-silabs jmartinez-silabs Awaiting requested review from jmartinez-silabs

@jtung-apple jtung-apple Awaiting requested review from jtung-apple

@lazarkov lazarkov Awaiting requested review from lazarkov

@LuDuda LuDuda Awaiting requested review from LuDuda

@mlepage-google mlepage-google Awaiting requested review from mlepage-google

@mrjerryjohns mrjerryjohns Awaiting requested review from mrjerryjohns

@msandstedt msandstedt Awaiting requested review from msandstedt

@mspang mspang Awaiting requested review from mspang

@sagar-apple sagar-apple Awaiting requested review from sagar-apple

@saurabhst saurabhst Awaiting requested review from saurabhst

@selissia selissia Awaiting requested review from selissia

@tcarmelveilleux tcarmelveilleux Awaiting requested review from tcarmelveilleux

@tecimovic tecimovic Awaiting requested review from tecimovic

@tehampson tehampson Awaiting requested review from tehampson

@vijs vijs Awaiting requested review from vijs

@vivien-apple vivien-apple Awaiting requested review from vivien-apple

@wbschiller wbschiller Awaiting requested review from wbschiller

@woody-apple woody-apple Awaiting requested review from woody-apple

@xylophone21 xylophone21 Awaiting requested review from xylophone21

@yufengwangca yufengwangca Awaiting requested review from yufengwangca

@yunhanw-google yunhanw-google Awaiting requested review from yunhanw-google

@rgoliver rgoliver Awaiting requested review from rgoliver

Assignees
No one assigned
Labels
examples review - approved
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@MtTsai @andy31415 @rgoliver @bzbarsky-apple @tcarmelveilleux