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

Support to return software_version_string with <branch>:<commit_id> on linux platform #19710

Merged
merged 1 commit into from
Jun 17, 2022
Merged

Support to return software_version_string with <branch>:<commit_id> on linux platform #19710

merged 1 commit into from
Jun 17, 2022

Conversation

MtTsai
Copy link
Contributor

@MtTsai MtTsai commented Jun 17, 2022

Problem

The value of software version string is need for some test automation.

Change overview

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

Testing

Testing on linux sample app of chef build, the build command is as below.

$ cd examples/chef.py
$ ./chef.py --bootstrap_zap
$ ./chef.py -zbr -t linux -d lighting-app

And use rpc console to getDeviceInfo to check the value of software version string.

@restyled-io restyled-io bot mentioned this pull request Jun 17, 2022
Closed
@github-actions
Copy link

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

PR #19710: Size comparison from 26d6c54 to 937ee69

Increases (5 builds for cc13x2_26x2, esp32, telink)
platform target config section 26d6c54 937ee69 change % change
cc13x2_26x2 lock-ftd LP_CC2652R7 (read/write) 150524 150532 8 0.0
shell LP_CC2652R7 (read/write) 157412 157420 8 0.0
esp32 all-clusters-app c3devkit (read only) 1012632 1012634 2 0.0
.flash.text 1012632 1012634 2 0.0
telink light-switch-app tlsr9518adk80d text 559602 559604 2 0.0
lighting-app tlsr9518adk80d text 576066 576068 2 0.0
Decreases (2 builds for cc13x2_26x2)
platform target config section 26d6c54 937ee69 change % change
cc13x2_26x2 lock-ftd LP_CC2652R7 (read only) 690451 690443 -8 -0.0
.text 590804 590796 -8 -0.0
shell LP_CC2652R7 (read only) 688946 688938 -8 -0.0
.text 578976 578968 -8 -0.0
Full report (30 builds for cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
platform target config section 26d6c54 937ee69 change % change
cc13x2_26x2 all-clusters-app LP_CC2652R7 (read only) 659531 659531 0 0.0
(read/write) 191332 191332 0 0.0
.bss 73756 73756 0 0.0
.data 3356 3356 0 0.0
.rodata 87083 87083 0 0.0
.text 572136 572136 0 0.0
all-clusters-minimal-app LP_CC2652R7 (read only) 647999 647999 0 0.0
(read/write) 157316 157316 0 0.0
.bss 73044 73044 0 0.0
.data 3356 3356 0 0.0
.rodata 90407 90407 0 0.0
.text 557272 557272 0 0.0
lock-ftd LP_CC2652R7 (read only) 690451 690443 -8 -0.0
(read/write) 150524 150532 8 0.0
.bss 70756 70756 0 0.0
.data 3280 3280 0 0.0
.rodata 99163 99163 0 0.0
.text 590804 590796 -8 -0.0
lock-mtd LP_CC2652R7 (read only) 639859 639859 0 0.0
(read/write) 143888 143888 0 0.0
.bss 66492 66492 0 0.0
.data 3280 3280 0 0.0
.rodata 99043 99043 0 0.0
.text 540324 540324 0 0.0
pump-app LP_CC2652R7 (read only) 671403 671403 0 0.0
(read/write) 170460 170460 0 0.0
.bss 70876 70876 0 0.0
.data 3280 3280 0 0.0
.rodata 87259 87259 0 0.0
.text 583660 583660 0 0.0
pump-controller-app LP_CC2652R7 (read only) 662399 662399 0 0.0
(read/write) 179592 179592 0 0.0
.bss 71004 71004 0 0.0
.data 3276 3276 0 0.0
.rodata 85183 85183 0 0.0
.text 576732 576732 0 0.0
shell LP_CC2652R7 (read only) 688946 688938 -8 -0.0
(read/write) 157412 157420 8 0.0
.bss 76052 76052 0 0.0
.data 3360 3360 0 0.0
.rodata 109658 109658 0 0.0
.text 578976 578968 -8 -0.0
cyw30739 light cyw930739m2evb_01 (read/write) 612738 612738 0 0.0
.app_xip_area 469404 469404 0 0.0
.bss 86288 86288 0 0.0
.data 728 728 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
lock cyw930739m2evb_01 (read/write) 610046 610046 0 0.0
.app_xip_area 466536 466536 0 0.0
.bss 86464 86464 0 0.0
.data 732 732 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
ota-requestor-no-progress-logging cyw930739m2evb_01 (read/write) 616054 616054 0 0.0
.app_xip_area 473608 473608 0 0.0
.bss 85456 85456 0 0.0
.data 672 672 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
efr32 lighting-app BRD4161A (read only) 924008 924008 0 0.0
(read/write) 132416 132416 0 0.0
.bss 130336 130336 0 0.0
.data 2080 2080 0 0.0
.text 924000 924000 0 0.0
BRD4161A+rpc (read only) 959712 959712 0 0.0
(read/write) 149296 149296 0 0.0
.bss 147008 147008 0 0.0
.data 2284 2284 0 0.0
.text 959704 959704 0 0.0
BRD4161A+rs911x (read only) 799092 799092 0 0.0
(read/write) 128692 128692 0 0.0
.bss 126604 126604 0 0.0
.data 2088 2088 0 0.0
.text 799084 799084 0 0.0
lock-app BRD4161A+wf200 (read only) 965524 965524 0 0.0
(read/write) 129068 129068 0 0.0
.bss 126980 126980 0 0.0
.data 2088 2088 0 0.0
.text 965516 965516 0 0.0
window-app BRD4161A (read only) 909104 909104 0 0.0
(read/write) 132520 132520 0 0.0
.bss 130408 130408 0 0.0
.data 2108 2108 0 0.0
.text 909096 909096 0 0.0
esp32 all-clusters-app c3devkit (read only) 1012632 1012634 2 0.0
(read/write) 1482898 1482898 0 0.0
.dram0.bss 69392 69392 0 0.0
.dram0.data 14632 14632 0 0.0
.flash.rodata 213440 213440 0 0.0
.flash.text 1012632 1012634 2 0.0
.iram0.text 62902 62902 0 0.0
m5stack (read only) 1067171 1067171 0 0.0
(read/write) 485016 485016 0 0.0
.dram0.bss 74912 74912 0 0.0
.dram0.data 34144 34144 0 0.0
.flash.rodata 243964 243964 0 0.0
.flash.text 1061787 1061787 0 0.0
.iram0.text 123267 123267 0 0.0
k32w light k32w061+release (read/write) 658876 658876 0 0.0
.bss 68756 68756 0 0.0
.data 1992 1992 0 0.0
.text 582328 582328 0 0.0
lock k32w061+release (read/write) 720848 720848 0 0.0
.bss 69196 69196 0 0.0
.data 2000 2000 0 0.0
.text 643852 643852 0 0.0
linux chip-tool-no-interactive-ipv6only arm64 (read only) 9881788 9881788 0 0.0
(read/write) 674225 674225 0 0.0
.bss 42641 42641 0 0.0
.data 1152 1152 0 0.0
.data.rel.ro 613208 613208 0 0.0
.dynamic 528 528 0 0.0
.got 13416 13416 0 0.0
.init 24 24 0 0.0
.init_array 192 192 0 0.0
.rodata 472300 472300 0 0.0
.text 7873060 7873060 0 0.0
thermostat-no-ble arm64 (read only) 2570668 2570668 0 0.0
(read/write) 192193 192193 0 0.0
.bss 99489 99489 0 0.0
.data 1688 1688 0 0.0
.data.rel.ro 82928 82928 0 0.0
.dynamic 528 528 0 0.0
.got 5072 5072 0 0.0
.init 24 24 0 0.0
.init_array 400 400 0 0.0
.rodata 163324 163324 0 0.0
.text 2168224 2168224 0 0.0
mbed lock-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2437720 2437720 0 0.0
.bss 208204 208204 0 0.0
.data 5864 5864 0 0.0
.text 1400364 1400364 0 0.0
nrfconnect all-clusters-app nrf52840dk_nrf52840 (read/write) 1198699 1198699 0 0.0
bss 141598 141598 0 0.0
rodata 155676 155676 0 0.0
text 822504 822504 0 0.0
all-clusters-minimal-app nrf52840dk_nrf52840 (read/write) 1146655 1146655 0 0.0
bss 140850 140850 0 0.0
rodata 132200 132200 0 0.0
text 794724 794724 0 0.0
p6 all-clusters-app default (read/write) 2553568 2553568 0 0.0
.bss 143384 143384 0 0.0
.data 2776 2776 0 0.0
.text 1511832 1511832 0 0.0
all-clusters-minimal-app default (read/write) 2499416 2499416 0 0.0
.bss 142664 142664 0 0.0
.data 2776 2776 0 0.0
.text 1457680 1457680 0 0.0
light-app default (read/write) 2430160 2430160 0 0.0
.bss 134744 134744 0 0.0
.data 2592 2592 0 0.0
.text 1388424 1388424 0 0.0
lock-app default (read/write) 2450784 2450784 0 0.0
.bss 134568 134568 0 0.0
.data 2600 2600 0 0.0
.text 1409048 1409048 0 0.0
telink light-switch-app tlsr9518adk80d (read/write) 789016 789016 0 0.0
bss 69892 69892 0 0.0
noinit 40416 40416 0 0.0
text 559602 559604 2 0.0
lighting-app tlsr9518adk80d (read/write) 808728 808728 0 0.0
bss 70140 70140 0 0.0
noinit 40416 40416 0 0.0
text 576066 576068 2 0.0

@andy31415
Copy link
Contributor

fast track build changes for test tools. Does not affect SDK functionality.

@andy31415 andy31415 merged commit 8da6fc8 into project-chip:master Jun 17, 2022
msandstedt
msandstedt reviewed Jun 17, 2022
View reviewed changes
examples/chef/chef.py
Comment on lines +492 to +494
branch = shell.run_cmd(
"git branch | awk -v FS=' ' '/\*/{print $NF}' | sed 's|[()]||g'", return_cmd_output=True).replace("\n", "")
commit_id = shell.run_cmd("git rev-parse HEAD", return_cmd_output=True).replace("\n", "")
Copy link
Contributor

Choose a reason for hiding this comment

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

Tag + #commits + hash from git describe --tags is easier and probably more useful.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks for suggestion, I'll consider that in the future!

Copy link
Contributor

Choose a reason for hiding this comment

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

This is too confusing, now changing branches will affect the resulting binary.

This type of stamping should be opt-in, not opt-out, since it's also seriously detrimental to performance and artifact caching since even a documentation change will result in a different binary.

mspang added a commit that referenced this pull request Jun 21, 2022
@mspang
Revert "Support to return software_version_string with <branch>:<comm…
7eac438 
…it_id> on linux platform (#19710)"

This reverts commit 8da6fc8.
@mspang mspang mentioned this pull request Jun 21, 2022
Merged
andy31415 pushed a commit that referenced this pull request Jun 21, 2022
@mspang
Revert "Support to return software_version_string with <branch>:<comm…
89c5d50 
…it_id> on linux platform (#19710)" (#19786)

This reverts commit 8da6fc8.
cpagravel pushed a commit to cpagravel/connectedhomeip that referenced this pull request Jun 25, 2022
@MtTsai @cpagravel
Support to return software_version_string with <branch>:<commit_id> o…
343c871 
…n linux platform (project-chip#19710)
cpagravel pushed a commit to cpagravel/connectedhomeip that referenced this pull request Jun 25, 2022
@mspang @cpagravel
Revert "Support to return software_version_string with <branch>:<comm…
11a8d75 
…it_id> on linux platform (project-chip#19710)" (project-chip#19786)

This reverts commit 8da6fc8.
cpagravel pushed a commit to cpagravel/connectedhomeip that referenced this pull request Jun 25, 2022
@MtTsai @cpagravel
Support to return software_version_string with <branch>:<commit_id> o…
e6f0db8 
…n linux platform (project-chip#19710)
cpagravel pushed a commit to cpagravel/connectedhomeip that referenced this pull request Jun 25, 2022
@mspang @cpagravel
Revert "Support to return software_version_string with <branch>:<comm…
bc553a5 
…it_id> on linux platform (project-chip#19710)" (project-chip#19786)

This reverts commit 8da6fc8.
@cpagravel cpagravel mentioned this pull request Jun 25, 2022
Closed
cpagravel pushed a commit to cpagravel/connectedhomeip that referenced this pull request Jun 25, 2022
@MtTsai @cpagravel
Support to return software_version_string with <branch>:<commit_id> o…
39f6090 
…n linux platform (project-chip#19710)
cpagravel pushed a commit to cpagravel/connectedhomeip that referenced this pull request Jun 25, 2022
@mspang @cpagravel
Revert "Support to return software_version_string with <branch>:<comm…
0269bb3 
…it_id> on linux platform (project-chip#19710)" (project-chip#19786)

This reverts commit 8da6fc8.
@cpagravel cpagravel mentioned this pull request Jun 25, 2022
Merged
andy31415 pushed a commit that referenced this pull request Jun 27, 2022
@cpagravel @rochaferraz
@mauimauer @MtTsai @vinitg1 @restyled-commits @mspang @aBozowski @jepenven-silabs
Cherry pick - Chef CD, sample apps, esp32 rpc fix, to interop branch (#…
5092282 
…19979)

* Pigweed: Update to 67bc89cdda8c (#19560)

* Fix esp32 wifi and add RPC support on linux (#19260)

* Chef - Extended Pigweed RPC support (#17781)

* Build flash_script package for ESP32

Change-Id: I4e0375c9b9837b3b9f8a3d2570635536e7e34e42
Reviewed-on: https://team-review.git.corp.google.com/c/npe-tse-chip-sdk/third_party/connectedhomeip/+/1419520
Reviewed-by: Doug Ferraz <[email protected]>

* new: Wifi and Descriptor clusters on ESP32

Change-Id: I37b61425a55d8210c2286becce47917fddf54a3f
Reviewed-on: https://team-review.git.corp.google.com/c/npe-tse-chip-sdk/third_party/connectedhomeip/+/1418359
Reviewed-by: Rob Oliver <[email protected]>
Reviewed-by: Kevin Cheung <[email protected]>
Reviewed-by: Jerry Lee <[email protected]>

* new: added stub file. Included callbacks for successful operation of lock/unlock commands

Change-Id: I9f4d601feb3b043d3c6689775bcaf40a8c5564d2
Reviewed-on: https://team-review.git.corp.google.com/c/npe-tse-chip-sdk/third_party/connectedhomeip/+/1420959
Reviewed-by: Michael Spang <[email protected]>
Reviewed-by: Jerry Lee <[email protected]>

* Enable PW RPC server on chef-built virtual device app

Change-Id: I878955e9df26e893c136650654019a8fa14e2a8b
Reviewed-on: https://team-review.git.corp.google.com/c/npe-tse-chip-sdk/third_party/connectedhomeip/+/1417979
Reviewed-by: Doug Ferraz <[email protected]>

* cosmetic change

Change-Id: Ia2063a395a8d3f16965356d4a76b0d9df0834e57

* changes to linux pigweed dependencies. Still has issues building Pigweed

Change-Id: I2be4e81f6c2e14112e31a0803813ef56d012b206

* Fix nrfconnect compilation errors (#17713)

* Fix nrfconnect compilation errors

* Chef: fix arguments for clean build

* Build flash_script package for ESP32

Change-Id: I4e0375c9b9837b3b9f8a3d2570635536e7e34e42
Reviewed-on: https://team-review.git.corp.google.com/c/npe-tse-chip-sdk/third_party/connectedhomeip/+/1419520
Reviewed-by: Doug Ferraz <[email protected]>

* new: Wifi and Descriptor clusters on ESP32

Change-Id: I37b61425a55d8210c2286becce47917fddf54a3f
Reviewed-on: https://team-review.git.corp.google.com/c/npe-tse-chip-sdk/third_party/connectedhomeip/+/1418359
Reviewed-by: Rob Oliver <[email protected]>
Reviewed-by: Kevin Cheung <[email protected]>
Reviewed-by: Jerry Lee <[email protected]>

* new: added stub file. Included callbacks for successful operation of lock/unlock commands

Change-Id: I9f4d601feb3b043d3c6689775bcaf40a8c5564d2
Reviewed-on: https://team-review.git.corp.google.com/c/npe-tse-chip-sdk/third_party/connectedhomeip/+/1420959
Reviewed-by: Michael Spang <[email protected]>
Reviewed-by: Jerry Lee <[email protected]>

* Enable PW RPC server on chef-built virtual device app

Change-Id: I878955e9df26e893c136650654019a8fa14e2a8b
Reviewed-on: https://team-review.git.corp.google.com/c/npe-tse-chip-sdk/third_party/connectedhomeip/+/1417979
Reviewed-by: Doug Ferraz <[email protected]>

* cosmetic change

Change-Id: Ia2063a395a8d3f16965356d4a76b0d9df0834e57

* changes to linux pigweed dependencies. Still has issues building Pigweed

Change-Id: I2be4e81f6c2e14112e31a0803813ef56d012b206

* Restyled by gn

* Restyled by autopep8

* Restyled by gn

Co-authored-by: Sebastian Mauer <[email protected]>
Co-authored-by: MT Tsai <[email protected]>
Co-authored-by: vinitg1 <[email protected]>
Co-authored-by: Restyled.io <[email protected]>
Change-Id: I827aa39ea41b8084d6fdca202e5a87e0f29cb4f3

* fix: added initialization to platform wifi network commissioning instance (#17983)

Change-Id: I03b6b523d4c4557ab6cc432216e48b6d6b2b6964

* Chef - Revert changes on lighting-app that separate wifi and thread

Co-authored-by: Douglas Rocha Ferraz <[email protected]>
Co-authored-by: Sebastian Mauer <[email protected]>
Co-authored-by: MT Tsai <[email protected]>
Co-authored-by: vinitg1 <[email protected]>
Co-authored-by: Restyled.io <[email protected]>

* Support to return software_version_string with <branch>:<commit_id> on linux platform (#19710)

* Chef - Add 13 sample apps (#19663)

* Chef - Remove redundant hash_string in sample_app_util rename func

* Chef - Add debugg output on exception to stateful_shell

* Chef - Add choices for device arg

* Chef - Add 13 sample apps

* Revert "Support to return software_version_string with <branch>:<commit_id> on linux platform (#19710)" (#19786)

This reverts commit 8da6fc8.

* Truncate the branch info of the software version string while the total length greater equal to 64 bytes. (#19788)

Make the filling of software version string with <branch>:<commit_id> as
an option.
The new option for it is '-a' or '--automated_test_stamp'.

* Make Chef CI use existing zzz_generated and complete Chef CD for existing platforms (#19478)

* Squashed commit of the following:

commit b5d5920
Author: Austin Bozowski <[email protected]>
Date:   Fri Jun 10 18:17:34 2022 +0000

    [no ci] comment

commit 7c5fb08
Author: Austin Bozowski <[email protected]>
Date:   Fri Jun 10 18:15:37 2022 +0000

    [no ci] Make border nice

commit d67a7bc
Author: Austin Bozowski <[email protected]>
Date:   Fri Jun 10 18:09:29 2022 +0000

    [no ci] restyle

commit 5783829
Author: Austin Bozowski <[email protected]>
Date:   Fri Jun 10 17:48:39 2022 +0000

    Add error handling to build all

commit 083b072
Author: Austin Bozowski <[email protected]>
Date:   Fri Jun 10 17:15:33 2022 +0000

    [no ci] update build image version

commit 48805a8
Author: Austin Bozowski <[email protected]>
Date:   Fri Jun 10 08:20:49 2022 +0000

    [no ci] minor

commit 3cae95c
Author: Austin Bozowski <[email protected]>
Date:   Fri Jun 10 08:15:22 2022 +0000

    [no ci] Minor

commit df929b2
Author: Austin Bozowski <[email protected]>
Date:   Fri Jun 10 07:37:49 2022 +0000

    Minor

commit 759ddb0
Author: Austin Bozowski <[email protected]>
Date:   Fri Jun 10 07:28:36 2022 +0000

    Minor

commit d9ca2a6
Author: Austin Bozowski <[email protected]>
Date:   Fri Jun 10 07:22:03 2022 +0000

    Style

commit dbb9b30
Author: Austin Bozowski <[email protected]>
Date:   Fri Jun 10 06:50:11 2022 +0000

    Style

commit 3daff46
Author: Austin Bozowski <[email protected]>
Date:   Fri Jun 10 02:58:43 2022 +0000

    nrf bundle

commit 7d38eb4
Author: Austin Bozowski <[email protected]>
Date:   Wed Jun 8 16:34:57 2022 +0000

    Linux bundle

commit fd5689e
Author: Austin Bozowski <[email protected]>
Date:   Wed Jun 8 09:08:42 2022 +0000

    Make esp32 generic

commit 1af3a2e
Author: Austin Bozowski <[email protected]>
Date:   Tue Jun 7 08:35:52 2022 +0000

    Missed comment

commit 5f7c41c
Author: Austin Bozowski <[email protected]>
Date:   Tue Jun 7 08:32:07 2022 +0000

    Example bundle function

* Bundle .matter, metadata & ensure .matter commit

* Add partial check for .matter regen

* Restyle and spelling

* Address comments

* Remove reflection

* Generate zzz, .MATTERMD5 should go away in future

* Temp del workflows

* Remove chef exclusion from regen

* Make regen all put chef exs in individual dirs

* Remove validate zzz

* Remove unused code from chef, change use_zzz

* Skip util test files in gen all

* Commit chef in root zzz

* Delete chef zzz

* Clean chef .matter files

* Add post build for chef

* Change .matter bundle rc to oot zzz

* Generated files

* Restore workflows

* Change open to with

* Restyle

* Chef - Fix broken esp32 RPC build caused by #18621 (#19876)

* Chef - Fix broken esp32 RPC build caused by #18621

* Chef - Fix esp32 RPC build issue caused by pigweed update

* Make Scenes name support dependent on FeatureMap attribute (#18667)

* Make Scenes name support dependent on FeatureMap attribute

* Fix CI

* Rebuild Zap generated files

* Fix CI for ipv6only/esp32 example build

* Fix errors regarding ownership of repo. (Taken from #18302)

* Bump chef.yaml platform containers to 0.5.75

Co-authored-by: rgoliver <[email protected]>
Co-authored-by: Douglas Rocha Ferraz <[email protected]>
Co-authored-by: Sebastian Mauer <[email protected]>
Co-authored-by: MT Tsai <[email protected]>
Co-authored-by: vinitg1 <[email protected]>
Co-authored-by: Restyled.io <[email protected]>
Co-authored-by: MtTsai <[email protected]>
Co-authored-by: Michael Spang <[email protected]>
Co-authored-by: Austin Bozowski <[email protected]>
Co-authored-by: Jean-Francois Penven <[email protected]>
rerasool pushed a commit to SiliconLabs/matter that referenced this pull request Sep 9, 2022
@cpagravel @rochaferraz
@mauimauer @MtTsai @vinitg1 @restyled-commits @mspang @aBozowski @jepenven-silabs @danielsilabs
Cherry pick - Chef CD, sample apps, esp32 rpc fix, to interop branch (p…
8a4d2a8 
…roject-chip#19979)

* Pigweed: Update to 67bc89cdda8c (project-chip#19560)

* Fix esp32 wifi and add RPC support on linux (project-chip#19260)

* Chef - Extended Pigweed RPC support (project-chip#17781)

* Build flash_script package for ESP32

Change-Id: I4e0375c9b9837b3b9f8a3d2570635536e7e34e42
Reviewed-on: https://team-review.git.corp.google.com/c/npe-tse-chip-sdk/third_party/connectedhomeip/+/1419520
Reviewed-by: Doug Ferraz <[email protected]>

* new: Wifi and Descriptor clusters on ESP32

Change-Id: I37b61425a55d8210c2286becce47917fddf54a3f
Reviewed-on: https://team-review.git.corp.google.com/c/npe-tse-chip-sdk/third_party/connectedhomeip/+/1418359
Reviewed-by: Rob Oliver <[email protected]>
Reviewed-by: Kevin Cheung <[email protected]>
Reviewed-by: Jerry Lee <[email protected]>

* new: added stub file. Included callbacks for successful operation of lock/unlock commands

Change-Id: I9f4d601feb3b043d3c6689775bcaf40a8c5564d2
Reviewed-on: https://team-review.git.corp.google.com/c/npe-tse-chip-sdk/third_party/connectedhomeip/+/1420959
Reviewed-by: Michael Spang <[email protected]>
Reviewed-by: Jerry Lee <[email protected]>

* Enable PW RPC server on chef-built virtual device app

Change-Id: I878955e9df26e893c136650654019a8fa14e2a8b
Reviewed-on: https://team-review.git.corp.google.com/c/npe-tse-chip-sdk/third_party/connectedhomeip/+/1417979
Reviewed-by: Doug Ferraz <[email protected]>

* cosmetic change

Change-Id: Ia2063a395a8d3f16965356d4a76b0d9df0834e57

* changes to linux pigweed dependencies. Still has issues building Pigweed

Change-Id: I2be4e81f6c2e14112e31a0803813ef56d012b206

* Fix nrfconnect compilation errors (project-chip#17713)

* Fix nrfconnect compilation errors

* Chef: fix arguments for clean build

* Build flash_script package for ESP32

Change-Id: I4e0375c9b9837b3b9f8a3d2570635536e7e34e42
Reviewed-on: https://team-review.git.corp.google.com/c/npe-tse-chip-sdk/third_party/connectedhomeip/+/1419520
Reviewed-by: Doug Ferraz <[email protected]>

* new: Wifi and Descriptor clusters on ESP32

Change-Id: I37b61425a55d8210c2286becce47917fddf54a3f
Reviewed-on: https://team-review.git.corp.google.com/c/npe-tse-chip-sdk/third_party/connectedhomeip/+/1418359
Reviewed-by: Rob Oliver <[email protected]>
Reviewed-by: Kevin Cheung <[email protected]>
Reviewed-by: Jerry Lee <[email protected]>

* new: added stub file. Included callbacks for successful operation of lock/unlock commands

Change-Id: I9f4d601feb3b043d3c6689775bcaf40a8c5564d2
Reviewed-on: https://team-review.git.corp.google.com/c/npe-tse-chip-sdk/third_party/connectedhomeip/+/1420959
Reviewed-by: Michael Spang <[email protected]>
Reviewed-by: Jerry Lee <[email protected]>

* Enable PW RPC server on chef-built virtual device app

Change-Id: I878955e9df26e893c136650654019a8fa14e2a8b
Reviewed-on: https://team-review.git.corp.google.com/c/npe-tse-chip-sdk/third_party/connectedhomeip/+/1417979
Reviewed-by: Doug Ferraz <[email protected]>

* cosmetic change

Change-Id: Ia2063a395a8d3f16965356d4a76b0d9df0834e57

* changes to linux pigweed dependencies. Still has issues building Pigweed

Change-Id: I2be4e81f6c2e14112e31a0803813ef56d012b206

* Restyled by gn

* Restyled by autopep8

* Restyled by gn

Co-authored-by: Sebastian Mauer <[email protected]>
Co-authored-by: MT Tsai <[email protected]>
Co-authored-by: vinitg1 <[email protected]>
Co-authored-by: Restyled.io <[email protected]>
Change-Id: I827aa39ea41b8084d6fdca202e5a87e0f29cb4f3

* fix: added initialization to platform wifi network commissioning instance (project-chip#17983)

Change-Id: I03b6b523d4c4557ab6cc432216e48b6d6b2b6964

* Chef - Revert changes on lighting-app that separate wifi and thread

Co-authored-by: Douglas Rocha Ferraz <[email protected]>
Co-authored-by: Sebastian Mauer <[email protected]>
Co-authored-by: MT Tsai <[email protected]>
Co-authored-by: vinitg1 <[email protected]>
Co-authored-by: Restyled.io <[email protected]>

* Support to return software_version_string with <branch>:<commit_id> on linux platform (project-chip#19710)

* Chef - Add 13 sample apps (project-chip#19663)

* Chef - Remove redundant hash_string in sample_app_util rename func

* Chef - Add debugg output on exception to stateful_shell

* Chef - Add choices for device arg

* Chef - Add 13 sample apps

* Revert "Support to return software_version_string with <branch>:<commit_id> on linux platform (project-chip#19710)" (project-chip#19786)

This reverts commit 8da6fc8.

* Truncate the branch info of the software version string while the total length greater equal to 64 bytes. (project-chip#19788)

Make the filling of software version string with <branch>:<commit_id> as
an option.
The new option for it is '-a' or '--automated_test_stamp'.

* Make Chef CI use existing zzz_generated and complete Chef CD for existing platforms (project-chip#19478)

* Squashed commit of the following:

commit b5d5920
Author: Austin Bozowski <[email protected]>
Date:   Fri Jun 10 18:17:34 2022 +0000

    [no ci] comment

commit 7c5fb08
Author: Austin Bozowski <[email protected]>
Date:   Fri Jun 10 18:15:37 2022 +0000

    [no ci] Make border nice

commit d67a7bc
Author: Austin Bozowski <[email protected]>
Date:   Fri Jun 10 18:09:29 2022 +0000

    [no ci] restyle

commit 5783829
Author: Austin Bozowski <[email protected]>
Date:   Fri Jun 10 17:48:39 2022 +0000

    Add error handling to build all

commit 083b072
Author: Austin Bozowski <[email protected]>
Date:   Fri Jun 10 17:15:33 2022 +0000

    [no ci] update build image version

commit 48805a8
Author: Austin Bozowski <[email protected]>
Date:   Fri Jun 10 08:20:49 2022 +0000

    [no ci] minor

commit 3cae95c
Author: Austin Bozowski <[email protected]>
Date:   Fri Jun 10 08:15:22 2022 +0000

    [no ci] Minor

commit df929b2
Author: Austin Bozowski <[email protected]>
Date:   Fri Jun 10 07:37:49 2022 +0000

    Minor

commit 759ddb0
Author: Austin Bozowski <[email protected]>
Date:   Fri Jun 10 07:28:36 2022 +0000

    Minor

commit d9ca2a6
Author: Austin Bozowski <[email protected]>
Date:   Fri Jun 10 07:22:03 2022 +0000

    Style

commit dbb9b30
Author: Austin Bozowski <[email protected]>
Date:   Fri Jun 10 06:50:11 2022 +0000

    Style

commit 3daff46
Author: Austin Bozowski <[email protected]>
Date:   Fri Jun 10 02:58:43 2022 +0000

    nrf bundle

commit 7d38eb4
Author: Austin Bozowski <[email protected]>
Date:   Wed Jun 8 16:34:57 2022 +0000

    Linux bundle

commit fd5689e
Author: Austin Bozowski <[email protected]>
Date:   Wed Jun 8 09:08:42 2022 +0000

    Make esp32 generic

commit 1af3a2e
Author: Austin Bozowski <[email protected]>
Date:   Tue Jun 7 08:35:52 2022 +0000

    Missed comment

commit 5f7c41c
Author: Austin Bozowski <[email protected]>
Date:   Tue Jun 7 08:32:07 2022 +0000

    Example bundle function

* Bundle .matter, metadata & ensure .matter commit

* Add partial check for .matter regen

* Restyle and spelling

* Address comments

* Remove reflection

* Generate zzz, .MATTERMD5 should go away in future

* Temp del workflows

* Remove chef exclusion from regen

* Make regen all put chef exs in individual dirs

* Remove validate zzz

* Remove unused code from chef, change use_zzz

* Skip util test files in gen all

* Commit chef in root zzz

* Delete chef zzz

* Clean chef .matter files

* Add post build for chef

* Change .matter bundle rc to oot zzz

* Generated files

* Restore workflows

* Change open to with

* Restyle

* Chef - Fix broken esp32 RPC build caused by project-chip#18621 (project-chip#19876)

* Chef - Fix broken esp32 RPC build caused by project-chip#18621

* Chef - Fix esp32 RPC build issue caused by pigweed update

* Make Scenes name support dependent on FeatureMap attribute (project-chip#18667)

* Make Scenes name support dependent on FeatureMap attribute

* Fix CI

* Rebuild Zap generated files

* Fix CI for ipv6only/esp32 example build

* Fix errors regarding ownership of repo. (Taken from project-chip#18302)

* Bump chef.yaml platform containers to 0.5.75

Co-authored-by: rgoliver <[email protected]>
Co-authored-by: Douglas Rocha Ferraz <[email protected]>
Co-authored-by: Sebastian Mauer <[email protected]>
Co-authored-by: MT Tsai <[email protected]>
Co-authored-by: vinitg1 <[email protected]>
Co-authored-by: Restyled.io <[email protected]>
Co-authored-by: MtTsai <[email protected]>
Co-authored-by: Michael Spang <[email protected]>
Co-authored-by: Austin Bozowski <[email protected]>
Co-authored-by: Jean-Francois Penven <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mspang mspang mspang left review comments

@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

@hawk248 hawk248 Awaiting requested review from hawk248

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

@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

@andy31415 andy31415 Awaiting requested review from andy31415

@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

@rgoliver rgoliver Awaiting requested review from rgoliver

@robszewczyk robszewczyk Awaiting requested review from robszewczyk

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

@saurabhst saurabhst Awaiting requested review from saurabhst

@selissia selissia Awaiting requested review from selissia

@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

@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

@msandstedt msandstedt Awaiting requested review from msandstedt

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

Successfully merging this pull request may close these issues.
4 participants
@MtTsai @andy31415 @mspang @msandstedt