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

Interop ci test #5

Open
wants to merge 14 commits into
base: interop_testing_te9
Choose a base branch
from
Open

Interop ci test #5

wants to merge 14 commits into from

Conversation

cpagravel
Copy link
Owner

Test PR for checking CI.

rgoliver and others added 9 commits June 25, 2022 01:36
@cpagravel
Pigweed: Update to 67bc89cdda8c (project-chip#19560)
157cc9a
@cpagravel @rochaferraz
@mauimauer @MtTsai @vinitg1 @restyled-commits
Fix esp32 wifi and add RPC support on linux (project-chip#19260)
ee777df 
* 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 <rochaferraz@google.com>

* 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 <rgoliver@google.com>
Reviewed-by: Kevin Cheung <kkmcheun@google.com>
Reviewed-by: Jerry Lee <jerrymhlee@google.com>

* 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 <spang@google.com>
Reviewed-by: Jerry Lee <jerrymhlee@google.com>

* 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 <rochaferraz@google.com>

* 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 <rochaferraz@google.com>

* 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 <rgoliver@google.com>
Reviewed-by: Kevin Cheung <kkmcheun@google.com>
Reviewed-by: Jerry Lee <jerrymhlee@google.com>

* 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 <spang@google.com>
Reviewed-by: Jerry Lee <jerrymhlee@google.com>

* 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 <rochaferraz@google.com>

* 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 <mauimauer@google.com>
Co-authored-by: MT Tsai <mttsai@google.com>
Co-authored-by: vinitg1 <62729931+vinitg1@users.noreply.github.com>
Co-authored-by: Restyled.io <commits@restyled.io>
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 <rochaferraz@google.com>
Co-authored-by: Sebastian Mauer <mauimauer@google.com>
Co-authored-by: MT Tsai <mttsai@google.com>
Co-authored-by: vinitg1 <62729931+vinitg1@users.noreply.github.com>
Co-authored-by: Restyled.io <commits@restyled.io>
@MtTsai @cpagravel
Support to return software_version_string with <branch>:<commit_id> o…
39f6090 
…n linux platform (project-chip#19710)
@cpagravel
Chef - Add 13 sample apps (project-chip#19663)
f5fd0dd 
* 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
@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.
@MtTsai @cpagravel
Truncate the branch info of the software version string while the tot…
33cde87 
…al 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'.
@aBozowski @cpagravel
Make Chef CI use existing zzz_generated and complete Chef CD for exis…
8c5435d 
…ting platforms (project-chip#19478)

* Squashed commit of the following:

commit b5d5920
Author: Austin Bozowski <bozowski@google.com>
Date:   Fri Jun 10 18:17:34 2022 +0000

    [no ci] comment

commit 7c5fb08
Author: Austin Bozowski <bozowski@google.com>
Date:   Fri Jun 10 18:15:37 2022 +0000

    [no ci] Make border nice

commit d67a7bc
Author: Austin Bozowski <bozowski@google.com>
Date:   Fri Jun 10 18:09:29 2022 +0000

    [no ci] restyle

commit 5783829
Author: Austin Bozowski <bozowski@google.com>
Date:   Fri Jun 10 17:48:39 2022 +0000

    Add error handling to build all

commit 083b072
Author: Austin Bozowski <bozowski@google.com>
Date:   Fri Jun 10 17:15:33 2022 +0000

    [no ci] update build image version

commit 48805a8
Author: Austin Bozowski <bozowski@google.com>
Date:   Fri Jun 10 08:20:49 2022 +0000

    [no ci] minor

commit 3cae95c
Author: Austin Bozowski <bozowski@google.com>
Date:   Fri Jun 10 08:15:22 2022 +0000

    [no ci] Minor

commit df929b2
Author: Austin Bozowski <bozowski@google.com>
Date:   Fri Jun 10 07:37:49 2022 +0000

    Minor

commit 759ddb0
Author: Austin Bozowski <bozowski@google.com>
Date:   Fri Jun 10 07:28:36 2022 +0000

    Minor

commit d9ca2a6
Author: Austin Bozowski <bozowski@google.com>
Date:   Fri Jun 10 07:22:03 2022 +0000

    Style

commit dbb9b30
Author: Austin Bozowski <bozowski@google.com>
Date:   Fri Jun 10 06:50:11 2022 +0000

    Style

commit 3daff46
Author: Austin Bozowski <bozowski@google.com>
Date:   Fri Jun 10 02:58:43 2022 +0000

    nrf bundle

commit 7d38eb4
Author: Austin Bozowski <bozowski@google.com>
Date:   Wed Jun 8 16:34:57 2022 +0000

    Linux bundle

commit fd5689e
Author: Austin Bozowski <bozowski@google.com>
Date:   Wed Jun 8 09:08:42 2022 +0000

    Make esp32 generic

commit 1af3a2e
Author: Austin Bozowski <bozowski@google.com>
Date:   Tue Jun 7 08:35:52 2022 +0000

    Missed comment

commit 5f7c41c
Author: Austin Bozowski <bozowski@google.com>
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
@cpagravel
Chef - Fix broken esp32 RPC build caused by project-chip#18621 (proje…
2811140 
…ct-chip#19876)

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

* Chef - Fix esp32 RPC build issue caused by pigweed update
@jepenven-silabs @cpagravel
Make Scenes name support dependent on FeatureMap attribute (project-c…
68095a5 
…hip#18667)

* Make Scenes name support dependent on FeatureMap attribute

* Fix CI
@cpagravel cpagravel force-pushed the interop_ci_test branch 4 times, most recently from 4020b3c to 0021a5e Compare June 26, 2022 07:42
@cpagravel cpagravel mentioned this pull request Jun 26, 2022
Merged
cpagravel pushed a commit that referenced this pull request Jul 11, 2022
@mspang
Remove racy access to readline (project-chip#20466)
5745a46 
It's not safe to access line editing state from the IO thread while
inside readline() on the main thread.

Remove the code that attempts to redraw readline after printing logs.
This avoids segfaults during logging at the cost of those logs
overwriting the prompt (this is not trivial to fix as readline
is a blocking API).

==================
WARNING: ThreadSanitizer: data race (pid=63005)
  Write of size 1 at 0x55f81c7745ff by main thread:
    #0 InteractiveStartCommand::ParseCommand(char*) ../../examples/chip-tool/commands/interactive/InteractiveCommands.cpp:127 (chip-tool+0x874911)
    #1 InteractiveStartCommand::RunCommand() ../../examples/chip-tool/commands/interactive/InteractiveCommands.cpp:85 (chip-tool+0x874594)
    #2 CHIPCommand::StartWaiting(std::chrono::duration<unsigned int, std::ratio<1l, 1000l> >) ../../examples/chip-tool/commands/common/CHIPCommand.cpp:408 (chip-tool+0x83e478)
    #3 CHIPCommand::Run() ../../examples/chip-tool/commands/common/CHIPCommand.cpp:187 (chip-tool+0x83c839)
    #4 Commands::RunCommand(int, char**, bool) ../../examples/chip-tool/commands/common/Commands.cpp:147 (chip-tool+0x85d4f7)
    #5 Commands::Run(int, char**) ../../examples/chip-tool/commands/common/Commands.cpp:51 (chip-tool+0x85c288)
    #6 main <null> (chip-tool+0x569c0a)

  Previous read of size 1 at 0x55f81c7745ff by thread T5 (mutexes: write M185):
    #0 LoggingCallback ../../examples/chip-tool/commands/interactive/InteractiveCommands.cpp:46 (chip-tool+0x874479)
    #1 chip::Logging::LogV(unsigned char, unsigned char, char const*, __va_list_tag*) ../../src/lib/support/logging/CHIPLogging.cpp:221 (chip-tool+0x8ee4dc)
    #2 chip::Logging::Log(unsigned char, unsigned char, char const*, ...) ../../src/lib/support/logging/CHIPLogging.cpp:172 (chip-tool+0x8ee30a)
    #3 chip::app::ReadClient::RefreshLivenessCheckTimer() <null> (chip-tool+0x8b1746)
    #4 chip::app::ReadClient::ProcessSubscribeResponse(chip::System::PacketBufferHandle&&) ../../src/app/ReadClient.cpp:845 (chip-tool+0x8b20ec)
    #5 chip::app::ReadClient::OnMessageReceived(chip::Messaging::ExchangeContext*, chip::PayloadHeader const&, chip::System::PacketBufferHandle&&) ../../src/app/ReadClient.cpp:409 (chip-tool+0x8ae2a4)
    #6 chip::Messaging::ExchangeContext::HandleMessage(unsigned int, chip::PayloadHeader const&, chip::BitFlags<chip::Messaging::MessageFlagValues, unsigned int>, chip::System::PacketBufferHandle&&) <null> (chip-tool+0xa0517a)
    #7 operator()<chip::Messaging::ExchangeContext> ../../src/messaging/ExchangeMgr.cpp:219 (chip-tool+0xa08c73)
    #8 Call ../../src/lib/support/Pool.h:126 (chip-tool+0xa0912d)
    #9 chip::internal::HeapObjectList::ForEachNode(void*, chip::Loop (*)(void*, void*)) ../../src/lib/support/Pool.cpp:127 (chip-tool+0x8ee05a)
    #10 ForEachActiveObject<chip::Messaging::ExchangeManager::OnMessageReceived(const chip::PacketHeader&, const chip::PayloadHeader&, const chip::SessionHandle&, chip::SessionMessageDelegate::DuplicateMessage, chip::System::PacketBufferHandle&&)::<lambda(auto:2*)> > ../../src/lib/support/Pool.h:396 (chip-tool+0xa08d10)
    #11 chip::Messaging::ExchangeManager::OnMessageReceived(chip::PacketHeader const&, chip::PayloadHeader const&, chip::SessionHandle const&, chip::SessionMessageDelegate::DuplicateMessage, chip::System::PacketBufferHandle&&) ../../src/messaging/ExchangeMgr.cpp:212 (chip-tool+0xa07e91)
    #12 chip::SessionManager::SecureUnicastMessageDispatch(chip::PacketHeader const&, chip::Transport::PeerAddress const&, chip::System::PacketBufferHandle&&) ../../src/transport/SessionManager.cpp:616 (chip-tool+0xa1548b)
    #13 chip::SessionManager::OnMessageReceived(chip::Transport::PeerAddress const&, chip::System::PacketBufferHandle&&) ../../src/transport/SessionManager.cpp:443 (chip-tool+0xa14426)
    #14 chip::TransportMgrBase::HandleMessageReceived(chip::Transport::PeerAddress const&, chip::System::PacketBufferHandle&&) ../../src/transport/TransportMgrBase.cpp:76 (chip-tool+0xa17dfa)
    #15 chip::Transport::Base::HandleMessageReceived(chip::Transport::PeerAddress const&, chip::System::PacketBufferHandle&&) ../../src/transport/raw/Base.h:102 (chip-tool+0xb19728)
    #16 chip::Transport::UDP::OnUdpReceive(chip::Inet::UDPEndPoint*, chip::System::PacketBufferHandle&&, chip::Inet::IPPacketInfo const*) ../../src/transport/raw/UDP.cpp:122 (chip-tool+0xb1a48b)
    #17 chip::Inet::UDPEndPointImplSockets::HandlePendingIO(chip::BitFlags<chip::System::SocketEventFlags, unsigned char>) ../../src/inet/UDPEndPointImplSockets.cpp:688 (chip-tool+0xb00aa0)
    #18 chip::Inet::UDPEndPointImplSockets::HandlePendingIO(chip::BitFlags<chip::System::SocketEventFlags, unsigned char>, long) ../../src/inet/UDPEndPointImplSockets.cpp:569 (chip-tool+0xafff89)
    #19 chip::System::LayerImplSelect::HandleEvents() ../../src/system/SystemLayerImplSelect.cpp:406 (chip-tool+0xb07563)
    #20 chip::DeviceLayer::Internal::GenericPlatformManagerImpl_POSIX<chip::DeviceLayer::PlatformManagerImpl>::_RunEventLoop() ../../src/include/platform/internal/GenericPlatformManagerImpl_POSIX.ipp:181 (chip-tool+0x98a227)
    #21 chip::DeviceLayer::PlatformManager::RunEventLoop() ../../src/include/platform/PlatformManager.h:362 (chip-tool+0x988f75)
    #22 chip::DeviceLayer::Internal::GenericPlatformManagerImpl_POSIX<chip::DeviceLayer::PlatformManagerImpl>::EventLoopTaskMain(void*) ../../src/include/platform/internal/GenericPlatformManagerImpl_POSIX.ipp:205 (chip-tool+0x98a87c)

  Location is global '(anonymous namespace)::gIsCommandRunning' of size 1 at 0x55f81c7745ff (chip-tool+0x000000c485ff)

  Mutex M185 (0x55f81c776180) created at:
    #0 pthread_mutex_lock ../../../../src/libsanitizer/sanitizer_common/sanitizer_common_interceptors.inc:4240 (libtsan.so.0+0x4f30a)
    #1 chip::DeviceLayer::Internal::GenericPlatformManagerImpl_POSIX<chip::DeviceLayer::PlatformManagerImpl>::_LockChipStack() ../../src/include/platform/internal/GenericPlatformManagerImpl_POSIX.ipp:78 (chip-tool+0x989e90)
    #2 chip::DeviceLayer::PlatformManager::LockChipStack() ../../src/include/platform/PlatformManager.h:410 (chip-tool+0x988fa5)
    #3 chip::DeviceLayer::Internal::GenericPlatformManagerImpl_POSIX<chip::DeviceLayer::PlatformManagerImpl>::_RunEventLoop() ../../src/include/platform/internal/GenericPlatformManagerImpl_POSIX.ipp:170 (chip-tool+0x98a147)
    #4 chip::DeviceLayer::PlatformManager::RunEventLoop() ../../src/include/platform/PlatformManager.h:362 (chip-tool+0x988f75)
    #5 chip::DeviceLayer::Internal::GenericPlatformManagerImpl_POSIX<chip::DeviceLayer::PlatformManagerImpl>::EventLoopTaskMain(void*) ../../src/include/platform/internal/GenericPlatformManagerImpl_POSIX.ipp:205 (chip-tool+0x98a87c)

  Thread T5 (tid=63013, running) created by main thread at:
    #0 pthread_create ../../../../src/libsanitizer/tsan/tsan_interceptors_posix.cpp:969 (libtsan.so.0+0x5ad75)
    #1 chip::DeviceLayer::Internal::GenericPlatformManagerImpl_POSIX<chip::DeviceLayer::PlatformManagerImpl>::_StartEventLoopTask() ../../src/include/platform/internal/GenericPlatformManagerImpl_POSIX.ipp:231 (chip-tool+0x98a40a)
    #2 chip::DeviceLayer::PlatformManager::StartEventLoopTask() ../../src/include/platform/PlatformManager.h:375 (chip-tool+0xaacca2)
    #3 chip::Controller::DeviceControllerFactory::ServiceEvents() ../../src/controller/CHIPDeviceControllerFactory.cpp:331 (chip-tool+0xab0417)
    #4 CHIPCommand::StartWaiting(std::chrono::duration<unsigned int, std::ratio<1l, 1000l> >) ../../examples/chip-tool/commands/common/CHIPCommand.cpp:403 (chip-tool+0x83e353)
    #5 CHIPCommand::Run() ../../examples/chip-tool/commands/common/CHIPCommand.cpp:187 (chip-tool+0x83c839)
    #6 Commands::RunCommand(int, char**, bool) ../../examples/chip-tool/commands/common/Commands.cpp:147 (chip-tool+0x85d4f7)
    #7 Commands::Run(int, char**) ../../examples/chip-tool/commands/common/Commands.cpp:51 (chip-tool+0x85c288)
    #8 main <null> (chip-tool+0x569c0a)

SUMMARY: ThreadSanitizer: data race ../../examples/chip-tool/commands/interactive/InteractiveCommands.cpp:127 in InteractiveStartCommand::ParseCommand(char*)
==================
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@cpagravel @MtTsai @mspang @aBozowski @jepenven-silabs