Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@tcarmelveilleux
Copy link
Contributor

Description

  • TestInetEndPoint was not hermetic, it required at least one network interface connected to a link and up.
  • There was a comment before to skip tests if the link was not up, but it was not actually skipping the test. This made it fail for me when working offline and trying to run unit tests.

Testing

  • Test passes with and without link up.

Description:

- TestInetEndPoint was not hermetic, it required at least one network
  interface connected to a link and up.
- There was a comment before to skip tests if the link was not up,
  but it was not actually skipping the test. This made it fail for
  me when working offline and trying to run unit tests.

Testing done:
- Test passes with and without link up.
@github-actions
Copy link

github-actions bot commented Mar 16, 2025

PR #38015: Size comparison from d815fde to e1c84cd

Full report (75 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
platform target config section d815fde e1c84cd change % change
bl602 lighting-app bl602+mfd+littlefs+rpc FLASH 1098418 1098418 0 0.0
RAM 94866 94866 0 0.0
bl702 lighting-app bl702+eth FLASH 653332 653332 0 0.0
RAM 33533 33533 0 0.0
bl702+wifi FLASH 830604 830604 0 0.0
RAM 22257 22257 0 0.0
bl706+mfd+rpc+littlefs FLASH 1063062 1063062 0 0.0
RAM 32181 32181 0 0.0
bl702l contact-sensor-app bl702l+mfd+littlefs FLASH 893844 893844 0 0.0
RAM 26920 26920 0 0.0
lighting-app bl702l+mfd+littlefs FLASH 976740 976740 0 0.0
RAM 24668 24668 0 0.0
cc13x4_26x4 lighting-app LP_EM_CC1354P10_6 FLASH 818472 818472 0 0.0
RAM 120296 120296 0 0.0
lock-ftd LP_EM_CC1354P10_6 FLASH 827384 827384 0 0.0
RAM 125392 125392 0 0.0
pump-app LP_EM_CC1354P10_6 FLASH 774244 774244 0 0.0
RAM 113764 113764 0 0.0
pump-controller-app LP_EM_CC1354P10_6 FLASH 758528 758528 0 0.0
RAM 113972 113972 0 0.0
cc32xx air-purifier CC3235SF_LAUNCHXL FLASH 541934 541934 0 0.0
RAM 205152 205152 0 0.0
lock CC3235SF_LAUNCHXL FLASH 576034 576034 0 0.0
RAM 205400 205400 0 0.0
cyw30739 light CYW30739B2-P5-EVK-01 unknown 2040 2040 0 0.0
FLASH 660069 660069 0 0.0
RAM 75436 75436 0 0.0
CYW30739B2-P5-EVK-02 unknown 2040 2040 0 0.0
FLASH 679929 679929 0 0.0
RAM 78076 78076 0 0.0
CYW30739B2-P5-EVK-03 unknown 2040 2040 0 0.0
FLASH 679929 679929 0 0.0
RAM 78076 78076 0 0.0
CYW930739M2EVB-02 unknown 2040 2040 0 0.0
FLASH 636853 636853 0 0.0
RAM 70504 70504 0 0.0
light-switch CYW30739B2-P5-EVK-01 unknown 2040 2040 0 0.0
FLASH 620309 620309 0 0.0
RAM 71676 71676 0 0.0
CYW30739B2-P5-EVK-02 unknown 2040 2040 0 0.0
FLASH 639953 639953 0 0.0
RAM 74220 74220 0 0.0
CYW30739B2-P5-EVK-03 unknown 2040 2040 0 0.0
FLASH 639953 639953 0 0.0
RAM 74220 74220 0 0.0
lock CYW30739B2-P5-EVK-01 unknown 2040 2040 0 0.0
FLASH 639805 639805 0 0.0
RAM 74684 74684 0 0.0
CYW30739B2-P5-EVK-02 unknown 2040 2040 0 0.0
FLASH 659521 659521 0 0.0
RAM 77228 77228 0 0.0
CYW30739B2-P5-EVK-03 unknown 2040 2040 0 0.0
FLASH 659521 659521 0 0.0
RAM 77228 77228 0 0.0
thermostat CYW30739B2-P5-EVK-01 unknown 2040 2040 0 0.0
FLASH 616145 616145 0 0.0
RAM 68772 68772 0 0.0
CYW30739B2-P5-EVK-02 unknown 2040 2040 0 0.0
FLASH 636005 636005 0 0.0
RAM 71412 71412 0 0.0
CYW30739B2-P5-EVK-03 unknown 2040 2040 0 0.0
FLASH 636005 636005 0 0.0
RAM 71412 71412 0 0.0
efr32 lock-app BRD4187C FLASH 941040 941040 0 0.0
RAM 159944 159944 0 0.0
BRD4338a FLASH 734520 734520 0 0.0
RAM 234844 234844 0 0.0
window-app BRD4187C FLASH 1033408 1033408 0 0.0
RAM 128048 128048 0 0.0
esp32 all-clusters-app c3devkit DRAM 98728 98728 0 0.0
FLASH 1594582 1594582 0 0.0
IRAM 83820 83820 0 0.0
m5stack DRAM 117524 117524 0 0.0
FLASH 1561190 1561190 0 0.0
IRAM 117039 117039 0 0.0
linux air-purifier-app debug unknown 4752 4752 0 0.0
FLASH 2659329 2659329 0 0.0
RAM 112304 112304 0 0.0
all-clusters-app debug unknown 5560 5560 0 0.0
FLASH 5981024 5981024 0 0.0
RAM 516696 516696 0 0.0
all-clusters-minimal-app debug unknown 5456 5456 0 0.0
FLASH 5317372 5317372 0 0.0
RAM 222680 222680 0 0.0
bridge-app debug unknown 5472 5472 0 0.0
FLASH 4631156 4631156 0 0.0
RAM 201000 201000 0 0.0
camera-app debug unknown 5456 5456 0 0.0
FLASH 4682450 4682450 0 0.0
RAM 195968 195968 0 0.0
camera-controller debug unknown 5776 5776 0 0.0
FLASH 11345651 11345651 0 0.0
RAM 597312 597312 0 0.0
chip-tool debug unknown 6112 6112 0 0.0
FLASH 13365709 13365709 0 0.0
RAM 605952 605952 0 0.0
chip-tool-ipv6only arm64 unknown 22120 22120 0 0.0
FLASH 11551528 11551528 0 0.0
RAM 658632 658632 0 0.0
fabric-admin debug unknown 5800 5800 0 0.0
FLASH 11636901 11636901 0 0.0
RAM 605736 605736 0 0.0
fabric-bridge-app debug unknown 4720 4720 0 0.0
FLASH 4462112 4462112 0 0.0
RAM 188200 188200 0 0.0
fabric-sync debug unknown 4976 4976 0 0.0
FLASH 5579237 5579237 0 0.0
RAM 471968 471968 0 0.0
lighting-app debug+rpc+ui unknown 6192 6192 0 0.0
FLASH 5525761 5525761 0 0.0
RAM 205200 205200 0 0.0
lock-app debug unknown 5424 5424 0 0.0
FLASH 4697900 4697900 0 0.0
RAM 192360 192360 0 0.0
ota-provider-app debug unknown 4760 4760 0 0.0
FLASH 4320110 4320110 0 0.0
RAM 181016 181016 0 0.0
ota-requestor-app debug unknown 4712 4712 0 0.0
FLASH 4450466 4450466 0 0.0
RAM 185504 185504 0 0.0
shell debug unknown 4240 4240 0 0.0
FLASH 2957724 2957724 0 0.0
RAM 145456 145456 0 0.0
thermostat-no-ble arm64 unknown 9456 9456 0 0.0
FLASH 4146936 4146936 0 0.0
RAM 229848 229848 0 0.0
tv-app debug unknown 5752 5752 0 0.0
FLASH 5918085 5918085 0 0.0
RAM 595400 595400 0 0.0
tv-casting-app debug unknown 5320 5320 0 0.0
FLASH 11538413 11538413 0 0.0
RAM 721744 721744 0 0.0
nrfconnect all-clusters-app nrf52840dk_nrf52840 FLASH 916048 916048 0 0.0
RAM 167457 167457 0 0.0
nrf7002dk_nrf5340_cpuapp FLASH 909028 909028 0 0.0
RAM 145701 145701 0 0.0
all-clusters-minimal-app nrf52840dk_nrf52840 FLASH 852812 852812 0 0.0
RAM 141243 141243 0 0.0
nxp contact k32w0+release FLASH 588480 588480 0 0.0
RAM 71004 71004 0 0.0
mcxw71+release FLASH 603704 603704 0 0.0
RAM 63144 63144 0 0.0
light k32w0+release FLASH 614292 614292 0 0.0
RAM 70292 70292 0 0.0
k32w1+release FLASH 687888 687888 0 0.0
RAM 72056 72056 0 0.0
lock mcxw71+release FLASH 752664 752664 0 0.0
RAM 67556 67556 0 0.0
psoc6 all-clusters cy8ckit_062s2_43012 FLASH 1661556 1661556 0 0.0
RAM 212344 212344 0 0.0
all-clusters-minimal cy8ckit_062s2_43012 FLASH 1565796 1565796 0 0.0
RAM 208560 208560 0 0.0
light cy8ckit_062s2_43012 FLASH 1442548 1442548 0 0.0
RAM 197320 197320 0 0.0
lock cy8ckit_062s2_43012 FLASH 1471444 1471444 0 0.0
RAM 224984 224984 0 0.0
qpg lighting-app qpg6105+debug FLASH 665084 665084 0 0.0
RAM 105180 105180 0 0.0
lock-app qpg6105+debug FLASH 623536 623536 0 0.0
RAM 99792 99792 0 0.0
stm32 light STM32WB5MM-DK FLASH 461056 461056 0 0.0
RAM 141496 141496 0 0.0
telink bridge-app tl7218x FLASH 665848 665848 0 0.0
RAM 90712 90712 0 0.0
contact-sensor-app tlsr9528a_retention FLASH 623428 623428 0 0.0
RAM 31488 31488 0 0.0
light-app-ota-compress-lzma-shell-factory-data tl3218x FLASH 762206 762206 0 0.0
RAM 40420 40420 0 0.0
light-app-ota-shell-factory-data tl7218x FLASH 755314 755314 0 0.0
RAM 97540 97540 0 0.0
light-switch-app-ota-compress-lzma-factory-data tl7218x_retention FLASH 682364 682364 0 0.0
RAM 52192 52192 0 0.0
light-switch-app-ota-compress-lzma-shell-factory-data tlsr9528a FLASH 710922 710922 0 0.0
RAM 73400 73400 0 0.0
light-switch-app-ota-shell-factory-data tl3218x_retention FLASH 703472 703472 0 0.0
RAM 37664 37664 0 0.0
lighting-app-ota-factory-data tlsr9118bdk40d FLASH 603042 603042 0 0.0
RAM 138640 138640 0 0.0
lighting-app-ota-rpc-factory-data-4mb tlsr9518adk80d FLASH 790380 790380 0 0.0
RAM 96388 96388 0 0.0
tizen all-clusters-app arm unknown 5152 5152 0 0.0
FLASH 1783392 1783392 0 0.0
RAM 94168 94168 0 0.0
chip-tool-ubsan arm unknown 11560 11560 0 0.0
FLASH 19092918 19092918 0 0.0
RAM 8355044 8355044 0 0.0

@mergify mergify bot merged commit b2a86bd into project-chip:master Mar 17, 2025
71 checks passed
juandediosg pushed a commit to juandediosg/connectedhomeip that referenced this pull request Mar 28, 2025
…hip#38015)

Description:

- TestInetEndPoint was not hermetic, it required at least one network
  interface connected to a link and up.
- There was a comment before to skip tests if the link was not up,
  but it was not actually skipping the test. This made it fail for
  me when working offline and trying to run unit tests.

Testing done:
- Test passes with and without link up.
juandediosg pushed a commit to juandediosg/connectedhomeip that referenced this pull request Mar 28, 2025
…hip#38015)

Description:

- TestInetEndPoint was not hermetic, it required at least one network
  interface connected to a link and up.
- There was a comment before to skip tests if the link was not up,
  but it was not actually skipping the test. This made it fail for
  me when working offline and trying to run unit tests.

Testing done:
- Test passes with and without link up.
sayondeep pushed a commit to sayondeep/connectedhomeip that referenced this pull request Apr 10, 2025
…hip#38015)

Description:

- TestInetEndPoint was not hermetic, it required at least one network
  interface connected to a link and up.
- There was a comment before to skip tests if the link was not up,
  but it was not actually skipping the test. This made it fail for
  me when working offline and trying to run unit tests.

Testing done:
- Test passes with and without link up.
lpbeliveau-silabs pushed a commit to lpbeliveau-silabs/connectedhomeip that referenced this pull request Aug 15, 2025
Hayk10002 pushed a commit to Hayk10002/connectedhomeip that referenced this pull request Aug 20, 2025
…hip#38015)

Description:

- TestInetEndPoint was not hermetic, it required at least one network
  interface connected to a link and up.
- There was a comment before to skip tests if the link was not up,
  but it was not actually skipping the test. This made it fail for
  me when working offline and trying to run unit tests.

Testing done:
- Test passes with and without link up.
Hayk10002 pushed a commit to Hayk10002/connectedhomeip that referenced this pull request Aug 29, 2025
…hip#38015)

Description:

- TestInetEndPoint was not hermetic, it required at least one network
  interface connected to a link and up.
- There was a comment before to skip tests if the link was not up,
  but it was not actually skipping the test. This made it fail for
  me when working offline and trying to run unit tests.

Testing done:
- Test passes with and without link up.
andy31415 added a commit that referenced this pull request Sep 3, 2025
* Fix TestInetEndPoint to pass even when computer is offline (#38015)

Description:

- TestInetEndPoint was not hermetic, it required at least one network
  interface connected to a link and up.
- There was a comment before to skip tests if the link was not up,
  but it was not actually skipping the test. This made it fail for
  me when working offline and trying to run unit tests.

Testing done:
- Test passes with and without link up.

* Add debug logs for test inet limits (#38790)

* Add debug log for endpoint full error

* Also add assertions that no in use items

* Clean up includes

* Fix typo in message

* Setup ipv6 environment in `REPL Tests - Linux` CI job

This change is picked from the PR [#38845](#38845)

* Setup ipv6 environment in other places where it's needed

* Add sudo to IPV6 setup commands for proper permissions

* Revert "Add sudo to IPV6 setup commands for proper permissions"

This reverts commit 025cc0c.

* Remove `add-apv6` action from `Build on Linux (python_lib)`

This doesn't work because of permission issues.

* Stop using old ubuntu version (#37870)

Co-authored-by: Andrei Litvin <[email protected]>

---------

Co-authored-by: Tennessee Carmel-Veilleux <[email protected]>
Co-authored-by: Andrei Litvin <[email protected]>
Co-authored-by: Andrei Litvin <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants