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

Software version string on nrf while doing automation testing #20939

Merged
merged 1 commit into from
Jul 19, 2022
Merged

Software version string on nrf while doing automation testing #20939

merged 1 commit into from
Jul 19, 2022

Conversation

MtTsai
Copy link
Contributor

@MtTsai MtTsai commented Jul 19, 2022

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

Change overview
Put the software version string in the CHIP_DEVICE_CONFIG_DEVICE_SOFTWARE_VERSION_STRING by chef.py for nrf sample app of chef build

Testing
Build the binary with command ./chef.py -zbra -d rootnode_dimmablelight_bCwGYSDpoe -t nrfconnect
And use RPC console to check the output is correct.

@MtTsai MtTsai changed the title oftware version string on nrf while doing automation testing Software version string on nrf while doing automation testing Jul 19, 2022
@github-actions
Copy link

github-actions bot commented Jul 19, 2022 •
edited
Loading

PR #20939: Size comparison from 3099482 to e511d6c

Increases (4 builds for cc13x2_26x2, nrfconnect, telink)
platform target config section 3099482 e511d6c change % change
cc13x2_26x2 lock-ftd LP_CC2652R7 (read only) 671059 671067 8 0.0
.text 594136 594144 8 0.0
nrfconnect lighting-app nrf52840dk_nrf52840 (read/write) 1167091 1167107 16 0.0
text 819664 819668 4 0.0
telink light-switch-app tlsr9518adk80d text 566346 566348 2 0.0
lighting-app tlsr9518adk80d text 582890 582894 4 0.0
Decreases (4 builds for cc13x2_26x2, nrfconnect)
platform target config section 3099482 e511d6c change % change
cc13x2_26x2 lock-ftd LP_CC2652R7 (read/write) 170364 170356 -8 -0.0
nrfconnect lock-app nrf52840dk_nrf52840 (read/write) 1127939 1127923 -16 -0.0
text 771668 771664 -4 -0.0
nrf5340dk_nrf5340_cpuapp text 684280 684276 -4 -0.0
pump-app nrf52840dk_nrf52840 text 749064 749060 -4 -0.0
Full report (40 builds for bl602, cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
platform target config section 3099482 e511d6c change % change
bl602 lighting-app bl602 (read/write) 1379314 1379314 0 0.0
.bss 116970 116970 0 0.0
.data 4480 4480 0 0.0
.text 1049952 1049952 0 0.0
bl602+rpc (read/write) 1424722 1424722 0 0.0
.bss 124418 124418 0 0.0
.data 4600 4600 0 0.0
.text 1081608 1081608 0 0.0
cc13x2_26x2 all-clusters-app LP_CC2652R7 (read only) 667959 667959 0 0.0
(read/write) 183288 183288 0 0.0
.bss 74140 74140 0 0.0
.data 3356 3356 0 0.0
.rodata 88319 88319 0 0.0
.text 579324 579324 0 0.0
all-clusters-minimal-app LP_CC2652R7 (read only) 633559 633559 0 0.0
(read/write) 157708 157708 0 0.0
.bss 73436 73436 0 0.0
.data 3356 3356 0 0.0
.rodata 77551 77551 0 0.0
.text 555684 555684 0 0.0
lock-ftd LP_CC2652R7 (read only) 671059 671067 8 0.0
(read/write) 170364 170356 -8 -0.0
.bss 71204 71204 0 0.0
.data 3280 3280 0 0.0
.rodata 76443 76443 0 0.0
.text 594136 594144 8 0.0
lock-mtd LP_CC2652R7 (read only) 653343 653343 0 0.0
(read/write) 183768 183768 0 0.0
.bss 66892 66892 0 0.0
.data 3280 3280 0 0.0
.rodata 101191 101191 0 0.0
.text 551672 551672 0 0.0
pump-app LP_CC2652R7 (read only) 680059 680059 0 0.0
(read/write) 162212 162212 0 0.0
.bss 71284 71284 0 0.0
.data 3280 3280 0 0.0
.rodata 88811 88811 0 0.0
.text 590764 590764 0 0.0
pump-controller-app LP_CC2652R7 (read only) 665843 665843 0 0.0
(read/write) 176548 176548 0 0.0
.bss 71404 71404 0 0.0
.data 3276 3276 0 0.0
.rodata 84675 84675 0 0.0
.text 580688 580688 0 0.0
shell LP_CC2652R7 (read only) 660434 660434 0 0.0
(read/write) 186316 186316 0 0.0
.bss 76444 76444 0 0.0
.data 3360 3360 0 0.0
.rodata 85090 85090 0 0.0
.text 575028 575028 0 0.0
cyw30739 light cyw930739m2evb_01 (read/write) 581382 581382 0 0.0
.app_xip_area 459936 459936 0 0.0
.bss 64240 64240 0 0.0
.data 716 716 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
lock cyw930739m2evb_01 (read/write) 587290 587290 0 0.0
.app_xip_area 461116 461116 0 0.0
.bss 68968 68968 0 0.0
.data 720 720 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
ota-requestor-no-progress-logging cyw930739m2evb_01 (read/write) 584738 584738 0 0.0
.app_xip_area 464140 464140 0 0.0
.bss 63448 63448 0 0.0
.data 660 660 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
efr32 lighting-app BRD4161A (read/write) 1085616 1085616 0 0.0
.bss 133052 133052 0 0.0
.data 2048 2048 0 0.0
.text 950496 950496 0 0.0
BRD4161A+rpc (read/write) 1139916 1139916 0 0.0
.bss 149732 149732 0 0.0
.data 2260 2260 0 0.0
.text 987900 987900 0 0.0
BRD4161A+rs911x (read/write) 950840 950840 0 0.0
.bss 140776 140776 0 0.0
.data 2048 2048 0 0.0
.text 807996 807996 0 0.0
lock-app BRD4161A+wf200 (read/write) 1127228 1127228 0 0.0
.bss 144144 144144 0 0.0
.data 2056 2056 0 0.0
.text 981008 981008 0 0.0
window-app BRD4161A (read/write) 1079108 1079108 0 0.0
.bss 134524 134524 0 0.0
.data 2076 2076 0 0.0
.text 942488 942488 0 0.0
esp32 all-clusters-app c3devkit (read only) 1021372 1021372 0 0.0
(read/write) 1486042 1486042 0 0.0
.dram0.bss 70088 70088 0 0.0
.dram0.data 14600 14600 0 0.0
.flash.rodata 215912 215912 0 0.0
.flash.text 1021372 1021372 0 0.0
.iram0.text 62902 62902 0 0.0
m5stack (read only) 1075171 1075171 0 0.0
(read/write) 488072 488072 0 0.0
.dram0.bss 75608 75608 0 0.0
.dram0.data 34144 34144 0 0.0
.flash.rodata 246324 246324 0 0.0
.flash.text 1069787 1069787 0 0.0
.iram0.text 123267 123267 0 0.0
k32w light k32w061+release (read/write) 660176 660176 0 0.0
.bss 69548 69548 0 0.0
.data 1992 1992 0 0.0
.text 582836 582836 0 0.0
lock k32w061+release (read/write) 687148 687148 0 0.0
.bss 70012 70012 0 0.0
.data 2004 2004 0 0.0
.text 609332 609332 0 0.0
linux chip-tool-ipv6only arm64 (read only) 10054716 10054716 0 0.0
(read/write) 694801 694801 0 0.0
.bss 42993 42993 0 0.0
.data 3304 3304 0 0.0
.data.rel.ro 630920 630920 0 0.0
.dynamic 528 528 0 0.0
.got 13656 13656 0 0.0
.init 24 24 0 0.0
.init_array 192 192 0 0.0
.rodata 479836 479836 0 0.0
.text 7994516 7994516 0 0.0
thermostat-no-ble arm64 (read only) 2603180 2603180 0 0.0
(read/write) 158481 158481 0 0.0
.bss 65281 65281 0 0.0
.data 1704 1704 0 0.0
.data.rel.ro 83368 83368 0 0.0
.dynamic 528 528 0 0.0
.got 5104 5104 0 0.0
.init 24 24 0 0.0
.init_array 400 400 0 0.0
.rodata 165748 165748 0 0.0
.text 2197136 2197136 0 0.0
mbed lock-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2448416 2448416 0 0.0
.bss 213948 213948 0 0.0
.data 5872 5872 0 0.0
.text 1411060 1411060 0 0.0
nrfconnect all-clusters-app nrf52840dk_nrf52840 (read/write) 1175875 1175875 0 0.0
bss 142908 142908 0 0.0
rodata 142204 142204 0 0.0
text 811864 811864 0 0.0
all-clusters-minimal-app nrf52840dk_nrf52840 (read/write) 1155939 1155939 0 0.0
bss 142144 142144 0 0.0
rodata 133748 133748 0 0.0
text 801176 801176 0 0.0
lighting-app nrf52840dk_nrf52840 (read/write) 1167091 1167107 16 0.0
bss 144471 144471 0 0.0
rodata 124116 124116 0 0.0
text 819664 819668 4 0.0
nrf52840dk_nrf52840+rpc (read/write) 1137351 1137351 0 0.0
bss 144900 144900 0 0.0
rodata 115656 115656 0 0.0
text 797428 797428 0 0.0
nrf52840dongle_nrf52840 (read/write) 1073599 1073599 0 0.0
bss 151702 151702 0 0.0
rodata 93136 93136 0 0.0
text 747640 747640 0 0.0
nrf5340dk_nrf5340_cpuapp (read/write) 1070646 1070646 0 0.0
bss 140842 140842 0 0.0
rodata 116532 116532 0 0.0
text 732208 732208 0 0.0
lock-app nrf52840dk_nrf52840 (read/write) 1127939 1127923 -16 -0.0
bss 135439 135439 0 0.0
rodata 142020 142020 0 0.0
text 771668 771664 -4 -0.0
nrf5340dk_nrf5340_cpuapp (read/write) 1031666 1031666 0 0.0
bss 131842 131842 0 0.0
rodata 134496 134496 0 0.0
text 684280 684276 -4 -0.0
pump-app nrf52840dk_nrf52840 (read/write) 1079719 1079719 0 0.0
bss 134366 134366 0 0.0
rodata 117464 117464 0 0.0
text 749064 749060 -4 -0.0
pump-controller-app nrf52840dk_nrf52840 (read/write) 1067671 1067671 0 0.0
bss 134161 134161 0 0.0
rodata 114248 114248 0 0.0
text 740356 740356 0 0.0
p6 all-clusters-app default (read only) 882128 882128 0 0.0
(read/write) 1685196 1685196 0 0.0
.bss 148568 148568 0 0.0
.data 2648 2648 0 0.0
.text 1525592 1525592 0 0.0
all-clusters-minimal-app default (read only) 882848 882848 0 0.0
(read/write) 1629324 1629324 0 0.0
.bss 147848 147848 0 0.0
.data 2648 2648 0 0.0
.text 1470440 1470440 0 0.0
light-app default (read only) 891152 891152 0 0.0
(read/write) 1549332 1549332 0 0.0
.bss 139752 139752 0 0.0
.data 2440 2440 0 0.0
.text 1398752 1398752 0 0.0
lock-app default (read only) 891280 891280 0 0.0
(read/write) 1576332 1576332 0 0.0
.bss 139608 139608 0 0.0
.data 2456 2456 0 0.0
.text 1425880 1425880 0 0.0
telink light-switch-app tlsr9518adk80d (read/write) 798212 798212 0 0.0
bss 70584 70584 0 0.0
noinit 40416 40416 0 0.0
text 566346 566348 2 0.0
lighting-app tlsr9518adk80d (read/write) 818260 818260 0 0.0
bss 71428 71428 0 0.0
noinit 40416 40416 0 0.0
text 582890 582894 4 0.0

woody-apple
woody-apple approved these changes Jul 19, 2022
View reviewed changes
Copy link
Contributor

@woody-apple woody-apple left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fast tracking platform changes

@woody-apple woody-apple enabled auto-merge (squash) July 19, 2022 20:56
@woody-apple woody-apple merged commit 612714f into project-chip:master Jul 19, 2022
github-actions bot pushed a commit that referenced this pull request Jul 19, 2022
@MtTsai @web-flow
Software version string on nrf while doing automation testing (#20939)
63eb397
woody-apple added a commit that referenced this pull request Jul 20, 2022
@woody-apple @MtTsai
Software version string on nrf while doing automation testing (#20939) (
eed5027 
#20963)

Co-authored-by: MtTsai <7930801+MtTsai@users.noreply.github.com>
MtTsai added a commit to MtTsai/connectedhomeip that referenced this pull request Jul 21, 2022
@MtTsai
Cherry Pick - Software version string on nrf while doing automation t…
c3685f5 
…esting (project-chip#20939)

* oftware version string on efr32 chef build while doing automation testing (project-chip#20982)

Add new config['silabs-thread']['EFR32_BOARD'] in config.yaml
cpagravel pushed a commit to cpagravel/connectedhomeip that referenced this pull request Jul 21, 2022
@MtTsai @cpagravel
Software version string on nrf while doing automation testing (projec…
e6187a6 
…t-chip#20939)
@cpagravel cpagravel mentioned this pull request Jul 21, 2022
Merged
cpagravel pushed a commit to cpagravel/connectedhomeip that referenced this pull request Jul 21, 2022
@MtTsai @cpagravel
Software version string on nrf while doing automation testing (projec…
7e86c0d 
…t-chip#20939)
cpagravel added a commit to nestlabs/connectedhomeip that referenced this pull request Jul 22, 2022
@cpagravel @tzm41 @MtTsai
Cherry pick - Chef noip fix, Chef automated_test_stamp to Chef CD bui…
28e3d28 
…lds (#4)

* RPC: Add RPCs to ota-requester app on ESP32 (project-chip#18760)

- Add RPC target to the esp32 requester example.
- Implement TriggerOta rpc which triggers the OTA request from
the provider.

* Add m5stack build configs for ota-requestor app (project-chip#19702)

* Add m5stack build configs for ota-requestor app

* Update build target testdata

* Chef - Separate Chef build noip into separate step (project-chip#21011)

* Software version string on nrf while doing automation testing (project-chip#20939)

* Software version string on efr32 chef build while doing automation testing (project-chip#20982)

Add new config['silabs-thread']['EFR32_BOARD'] in config.yaml

* Chef - Add automated_test_stamp to Chef CD builds (project-chip#21059)

Co-authored-by: rgoliver <rgoliver@google.com>
Co-authored-by: Colin Tan <guangzhoutzm@gmail.com>
Co-authored-by: MtTsai <7930801+MtTsai@users.noreply.github.com>
isiu-apple pushed a commit to isiu-apple/connectedhomeip that referenced this pull request Sep 16, 2022
@MtTsai @isiu-apple
Software version string on nrf while doing automation testing (projec…
5bec21a 
…t-chip#20939)
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

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

@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

@dhrishi dhrishi Awaiting requested review from dhrishi

@electrocucaracha electrocucaracha Awaiting requested review from electrocucaracha

@erjiaqing erjiaqing Awaiting requested review from erjiaqing

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

@gjc13 gjc13 Awaiting requested review from gjc13

@harimau-qirex harimau-qirex Awaiting requested review from harimau-qirex

@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

@kghost kghost Awaiting requested review from kghost

@kpschoedel kpschoedel Awaiting requested review from kpschoedel

@lazarkov lazarkov Awaiting requested review from lazarkov

@LuDuda LuDuda Awaiting requested review from LuDuda

@mrjerryjohns mrjerryjohns Awaiting requested review from mrjerryjohns

@msandstedt msandstedt Awaiting requested review from msandstedt

@mspang mspang Awaiting requested review from mspang

@rgoliver rgoliver Awaiting requested review from rgoliver

@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

@turon turon Awaiting requested review from turon

@vijs vijs Awaiting requested review from vijs

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

@wbschiller wbschiller Awaiting requested review from wbschiller

@xylophone21 xylophone21 Awaiting requested review from xylophone21

@yufengwangca yufengwangca Awaiting requested review from yufengwangca

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

Assignees
No one assigned
Labels
examples fast track platform review - pending
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@MtTsai @woody-apple