[L] Change in fuchsia/fuchsia[main]: [lacewing] Run async on the same event loop everywhere
0 views
Skip to first unread message
'Charles Celerier (Gerrit)' via owners-override
Feb 26, 2026, 2:55:22 PMFeb 26
to Owners Override
Attention needed from Owners Override
Charles Celerier has uploaded the change for review![]( "Open in Gerrit")
Charles Celerier would like Owners Override to review this change.
Commit message
[lacewing] Run async on the same event loop everywhere
Instead of creating a new loop every time a test needs to run a
coroutine, this change runs all the coroutines on the same global event
loop. This is a preparatory change for spreading the use of async to all
the tests.
Depends-on: turquoise-internal:I526098579b1e6444874fb880e0c098efa6ba2288
Change-Id: If23a4f15ce3c2931ce226f4a25ea9a152cc9c5f9
Change diff
Change information
Files:
- M docs/development/tools/fuchsia-controller/async-python.md
- M scripts/lib/async_utils/BUILD.gn
- M scripts/lib/async_utils/tests/async_utils_test.py
- M scripts/lib/ffx_cmd/BUILD.gn
- M scripts/lib/ffx_cmd/e2e_tests/test_ffx_inspect.py
- M scripts/lib/ffx_cmd/lib.py
- M scripts/lib/ffx_cmd/tests/__init__.py
- M scripts/lib/ffx_cmd/tests/test_ffx_cmd.py
- M scripts/lib/fx_cmd/lib.py
- M scripts/lib/fx_cmd/tests/test_fx_cmd.py
- M src/connectivity/network/tests/benchmarks/iperf/BUILD.gn
- M src/connectivity/network/tests/benchmarks/iperf/netstack_iperf_test.py
- M src/connectivity/wlan/tests/wlancfg/BUILD.gn
- M src/connectivity/wlan/tests/wlancfg/RegulatoryRecoveryTest.py
- M src/connectivity/wlan/tests/wlanix/ApfTest.py
- M src/connectivity/wlan/tests/wlanix/BUILD.gn
- M src/connectivity/wlan/tests/wlanix/IfaceCorrectnessAndConsistencyTest.py
- M src/connectivity/wlan/tests/wlanix/IfaceLifecycleTest.py
- M src/connectivity/wlan/tests/wlanix/WifiChipCorrectnessAndConsistencyTest.py
- M src/connectivity/wlan/tests/wlanix/WifiChipExistenceTest.py
- M src/connectivity/wlan/tests/wlanix/wlanix_testing/BUILD.gn
- M src/connectivity/wlan/tests/wlanix/wlanix_testing/base_test.py
- M src/developer/ffx/lib/fuchsia-controller/BUILD.gn
- M src/developer/ffx/lib/fuchsia-controller/python/fidl/_client.py
- M src/developer/ffx/lib/fuchsia-controller/python/fidl/_ipc.py
- D src/developer/ffx/lib/fuchsia-controller/python/fidl/tests/BUILD.gn
- D src/developer/ffx/lib/fuchsia-controller/python/fidl/tests/queue_wrapper_test.py
- M src/developer/ffx/lib/fuchsia-controller/python/fuchsia_controller_py/wrappers.py
- M src/storage/blackout/host/tests/BUILD.gn
- M src/storage/blackout/host/tests/blackout.py
- M src/testing/end_to_end/honeydew/honeydew/affordances/connectivity/bluetooth/bluetooth_common/bluetooth_common_using_fc.py
- M src/testing/end_to_end/honeydew/honeydew/affordances/connectivity/bluetooth/bluetooth_common/tests/unit_tests/bluetooth_common_using_fc_test.py
- M src/testing/end_to_end/honeydew/honeydew/affordances/rtc/rtc_using_fc.py
- M src/testing/end_to_end/honeydew/honeydew/affordances/rtc/tests/unit_tests/rtc_using_fc_test.py
- M src/testing/end_to_end/honeydew/honeydew/affordances/ui/user_input/user_input_using_fc.py
- M src/testing/end_to_end/honeydew/honeydew/affordances/virtual_audio/audio_using_fuchsia_controller.py
- M src/testing/end_to_end/honeydew/honeydew/affordances/virtual_audio/tests/functional_tests/BUILD.gn
- M src/testing/end_to_end/honeydew/honeydew/affordances/virtual_audio/tests/functional_tests/test_audio_inject_record.py
- M src/testing/end_to_end/honeydew/honeydew/fuchsia_device/fuchsia_device_impl.py
- M tools/devshell/python/system-status/main.py
- M tools/devshell/python/system-status/tests/test_system_status.py
- M tools/fidl/fidlgen_python/tests/test_client.py
Change size: L
Delta: 42 files changed, 359 insertions(+), 539 deletions(-)
Related details
Attention is currently required from:
- Owners Override
Submit Requirements:
Code-Owners
Code-Review
Dependencies-Satisfied
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
You received this message because you are subscribed to the Google Groups "owners-override" group.
To unsubscribe from this group and stop receiving emails from it, send an email to owners-overri...@fuchsia.dev.
To view this discussion visit https://groups.google.com/a/fuchsia.dev/d/msgid/owners-override/a45c012c0b43411333abd91016c77dac5b8ff2a2-HTML%40fuchsia-review.googlesource.com.
'Charles Celerier (Gerrit)' via owners-override
Feb 26, 2026, 2:57:57 PMFeb 26
to Owners Override, Andrew Davies, GI Try Builder, CQ Bot
Attention needed from Owners Override
Charles Celerier added 1 comment![]( "Open in Gerrit")
Patchset-level comments
File-level comment, Patchset 14 (Latest):
Charles Celerier . unresolved
Context for Owners Override: This change will be accompanied by an announcement to encourage e2e test writers to reach for `fuchsia_async_extension.get_loop().run_until_complete` instead of `asyncio.run`. Later changes to spread usage of async in e2e tests will eliminate needing to interface with `fuchsia_async_extension.get_loop().run_until_complete` at all. This change is necessary to ensure a smooth migration to that point because otherwise moving some affordances to a new event loop can cause unexpected problems.
Related details
Attention is currently required from:
- Owners Override
Submit Requirements:
Code-Owners
Code-Review
Dependencies-Satisfied
No-Unresolved-Comments
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
--
You received this message because you are subscribed to the Google Groups "owners-override" group.
To unsubscribe from this group and stop receiving emails from it, send an email to owners-overri...@fuchsia.dev.
To view this discussion visit https://groups.google.com/a/fuchsia.dev/d/msgid/owners-override/6f2f17fee5705ac8fed15ccb99e43a278e2910db-EmailReviewComments-HTML%40fuchsia-review.googlesource.com.You received this message because you are subscribed to the Google Groups "owners-override" group.
To unsubscribe from this group and stop receiving emails from it, send an email to owners-overri...@fuchsia.dev.
'Charles Celerier (Gerrit)' via owners-override
Feb 26, 2026, 3:12:22 PMFeb 26
to Owners Override, Andrew Davies, GI Try Builder, CQ Bot
Attention needed from Owners Override
Charles Celerier voted Fuchsia-Auto-Submit+1![]( "Open in Gerrit")
| Fuchsia-Auto-Submit | +1 |
Related details
Attention is currently required from:
- Owners Override
Submit Requirements:
Code-Owners
Code-Review
Dependencies-Satisfied
No-Unresolved-Comments
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
--
You received this message because you are subscribed to the Google Groups "owners-override" group.
To unsubscribe from this group and stop receiving emails from it, send an email to owners-overri...@fuchsia.dev.
To view this discussion visit https://groups.google.com/a/fuchsia.dev/d/msgid/owners-override/33099c6945126b6218392b2cc2272735260b50ea-EmailReviewComments-HTML%40fuchsia-review.googlesource.com.You received this message because you are subscribed to the Google Groups "owners-override" group.
To unsubscribe from this group and stop receiving emails from it, send an email to owners-overri...@fuchsia.dev.
'Charles Celerier (Gerrit)' via owners-override
Feb 26, 2026, 4:40:22 PMFeb 26
to Owners Override
Charles Celerier has uploaded the change for review![]( "Open in Gerrit")
Charles Celerier removed Owners Override from reviewers of this change.
Change information
Files:
- M src/testing/end_to_end/mobly_base_tests/fuchsia_async_extension/BUILD.gn
- M tools/devshell/python/system-status/main.py
- M tools/devshell/python/system-status/tests/test_system_status.py
- M tools/fidl/fidlgen_python/tests/test_client.py
Change size: L
Delta: 43 files changed, 360 insertions(+), 542 deletions(-)
Related details
Attention set is empty
Submit Requirements:
Code-Owners
Code-Review
Dependencies-Satisfied
No-Unresolved-Comments
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
You received this message because you are subscribed to the Google Groups "owners-override" group.
To unsubscribe from this group and stop receiving emails from it, send an email to owners-overri...@fuchsia.dev.
To view this discussion visit https://groups.google.com/a/fuchsia.dev/d/msgid/owners-override/44191a1f326ae851d148a38f3cab7dd7d944a1f2-HTML%40fuchsia-review.googlesource.com.To unsubscribe from this group and stop receiving emails from it, send an email to owners-overri...@fuchsia.dev.
'Charles Celerier (Gerrit)' via owners-override
Feb 26, 2026, 4:40:29 PMFeb 26
to Owners Override
Attention needed from Owners Override
Charles Celerier has uploaded the change for review![]( "Open in Gerrit")
Charles Celerier would like Owners Override to review this change.
Related details
Attention is currently required from:
- Owners Override
Submit Requirements:
Code-Owners
Code-Review
Dependencies-Satisfied
No-Unresolved-Comments
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
--
You received this message because you are subscribed to the Google Groups "owners-override" group.
To unsubscribe from this group and stop receiving emails from it, send an email to owners-overri...@fuchsia.dev.
To view this discussion visit https://groups.google.com/a/fuchsia.dev/d/msgid/owners-override/c3f20ab54b9a88f9afb1a87eb7635d5e2e1d0005-HTML%40fuchsia-review.googlesource.com.You received this message because you are subscribed to the Google Groups "owners-override" group.
To unsubscribe from this group and stop receiving emails from it, send an email to owners-overri...@fuchsia.dev.
'Hunter Freyer (Gerrit)' via owners-override
Feb 27, 2026, 10:02:26 AMFeb 27
to Charles Celerier, Owners Override, Andrew Davies, GI Try Builder, CQ Bot
Attention needed from Charles Celerier and Owners Override
Hunter Freyer added 1 comment![]( "Open in Gerrit")
Patchset-level comments
Context for Owners Override: This change will be accompanied by an announcement to encourage e2e test writers to reach for `fuchsia_async_extension.get_loop().run_until_complete` instead of `asyncio.run`. Later changes to spread usage of async in e2e tests will eliminate needing to interface with `fuchsia_async_extension.get_loop().run_until_complete` at all. This change is necessary to ensure a smooth migration to that point because otherwise moving some affordances to a new event loop can cause unexpected problems.
Attention is currently required from:
- Charles Celerier
- Owners Override
Submit Requirements:
Code-Owners
Code-Review
Dependencies-Satisfied
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
--
You received this message because you are subscribed to the Google Groups "owners-override" group.
To unsubscribe from this group and stop receiving emails from it, send an email to owners-overri...@fuchsia.dev.
To view this discussion visit https://groups.google.com/a/fuchsia.dev/d/msgid/owners-override/154e3eb1bdf73b4e4b67fe3200a48ed4374ba869-EmailReviewComments-HTML%40fuchsia-review.googlesource.com.You received this message because you are subscribed to the Google Groups "owners-override" group.
To unsubscribe from this group and stop receiving emails from it, send an email to owners-overri...@fuchsia.dev.
'Hunter Freyer (Gerrit)' via owners-override
Feb 27, 2026, 10:02:49 AMFeb 27
to Charles Celerier, Owners Override, Andrew Davies, GI Try Builder, CQ Bot
Attention needed from Charles Celerier and Owners Override
Hunter Freyer voted Owners-Override+1![]( "Open in Gerrit")
| Owners-Override | +1 |
Related details
Attention is currently required from:
- Charles Celerier
- Owners Override
Submit Requirements:
Code-Owners
Code-Review
Dependencies-Satisfied
No-Unresolved-Comments
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
You received this message because you are subscribed to the Google Groups "owners-override" group.
To unsubscribe from this group and stop receiving emails from it, send an email to owners-overri...@fuchsia.dev.
To view this discussion visit https://groups.google.com/a/fuchsia.dev/d/msgid/owners-override/5175aa78028d99148ac245c9bf8a34b906eef6cf-EmailReviewComments-HTML%40fuchsia-review.googlesource.com.To unsubscribe from this group and stop receiving emails from it, send an email to owners-overri...@fuchsia.dev.
'Charles Celerier (Gerrit)' via owners-override
Feb 27, 2026, 10:07:20 AMFeb 27
to Hunter Freyer, Owners Override, Andrew Davies, GI Try Builder, CQ Bot
Attention needed from Owners Override
Charles Celerier voted Fuchsia-Auto-Submit+0![]( "Open in Gerrit")
Attention is currently required from:
- Owners Override
Submit Requirements:
Code-Owners
Code-Review
Dependencies-Satisfied
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
--
You received this message because you are subscribed to the Google Groups "owners-override" group.
To unsubscribe from this group and stop receiving emails from it, send an email to owners-overri...@fuchsia.dev.
To view this discussion visit https://groups.google.com/a/fuchsia.dev/d/msgid/owners-override/70bef351f3fdea72f0e2f731fc4a79c8748437b8-EmailReviewComments-HTML%40fuchsia-review.googlesource.com.You received this message because you are subscribed to the Google Groups "owners-override" group.
To unsubscribe from this group and stop receiving emails from it, send an email to owners-overri...@fuchsia.dev.
'Charles Celerier (Gerrit)' via owners-override
Feb 27, 2026, 10:07:36 AMFeb 27
to Hunter Freyer, Owners Override, Andrew Davies, GI Try Builder, CQ Bot
Attention needed from Owners Override
Charles Celerier voted Fuchsia-Auto-Submit+1![]( "Open in Gerrit")
| Fuchsia-Auto-Submit | +1 |
--
You received this message because you are subscribed to the Google Groups "owners-override" group.
To unsubscribe from this group and stop receiving emails from it, send an email to owners-overri...@fuchsia.dev.
To view this discussion visit https://groups.google.com/a/fuchsia.dev/d/msgid/owners-override/97db6caf072e97d5091bd44c9818bf7720a0cdcb-EmailReviewComments-HTML%40fuchsia-review.googlesource.com.You received this message because you are subscribed to the Google Groups "owners-override" group.
To unsubscribe from this group and stop receiving emails from it, send an email to owners-overri...@fuchsia.dev.
'CQ Bot (Gerrit)' via owners-override
Feb 27, 2026, 10:09:46 AMFeb 27
to Charles Celerier, CL Deps Checker, Hunter Freyer, Owners Override, Andrew Davies, GI Try Builder
CQ Bot submitted the change with unreviewed changes![]( "Open in Gerrit")
Unreviewed changes
14 is the latest approved patch-set.
The change was submitted with unreviewed changes in the following files:
```
The name of the file: scripts/lib/async_utils/BUILD.gn
Insertions: 1, Deletions: 4.
The diff is too large to show. Please review the diff.
```
```
The name of the file: docs/development/tools/fuchsia-controller/async-python.md
Insertions: 1, Deletions: 1.
The diff is too large to show. Please review the diff.
```
```
The name of the file: scripts/lib/async_utils/__init__.py
Insertions: 0, Deletions: 3.
The diff is too large to show. Please review the diff.
```
Change information
Commit message:
[lacewing] Run async on the same event loop everywhere
Instead of creating a new loop every time a test needs to run a
coroutine, this change runs all the coroutines on the same global event
loop. This is a preparatory change for spreading the use of async to all
the tests.
Depends-on: turquoise-internal:I526098579b1e6444874fb880e0c098efa6ba2288
Change-Id: If23a4f15ce3c2931ce226f4a25ea9a152cc9c5f9
Dependencies-Satisfied: CL Deps Checker <cl-deps...@fuchsia-internal-service-accts.iam.gserviceaccount.com>
Commit-Queue: Charles Celerier <ch...@google.com>
Owners-Override: Hunter Freyer <hjfr...@google.com>
Reviewed-by: Andrew Davies <awda...@google.com>
Fuchsia-Auto-Submit: Charles Celerier <ch...@google.com>
Files:
- M docs/development/tools/fuchsia-controller/async-python.md
- M scripts/lib/async_utils/__init__.py
Change size: L
Delta: 43 files changed, 356 insertions(+), 544 deletions(-)
Branch: refs/heads/main
Submit Requirements:
Code-Review: +2 by Andrew Davies
Dependencies-Satisfied: +1 by CL Deps Checker
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
--
You received this message because you are subscribed to the Google Groups "owners-override" group.
To unsubscribe from this group and stop receiving emails from it, send an email to owners-overri...@fuchsia.dev.
To view this discussion visit https://groups.google.com/a/fuchsia.dev/d/msgid/owners-override/5bd2d74e3585655e6e190b3bdfde0038596615dc-HTML%40fuchsia-review.googlesource.com.You received this message because you are subscribed to the Google Groups "owners-override" group.
To unsubscribe from this group and stop receiving emails from it, send an email to owners-overri...@fuchsia.dev.
'GI Roller (Gerrit)' via owners-override
Feb 27, 2026, 10:12:06 AMFeb 27
to Charles Celerier, CQ Bot, CL Deps Checker, Hunter Freyer, Owners Override, Andrew Davies, GI Try Builder
Message from GI Roller![]( "Open in Gerrit")
Change has been successfully rolled: https://turquoise-internal-review.googlesource.com/1181434
Related details
Attention set is empty
Submit Requirements:
Code-Owners
Code-Review
Dependencies-Satisfied
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
--
You received this message because you are subscribed to the Google Groups "owners-override" group.
To unsubscribe from this group and stop receiving emails from it, send an email to owners-overri...@fuchsia.dev.
To view this discussion visit https://groups.google.com/a/fuchsia.dev/d/msgid/owners-override/3d5738f15daa8c65319307735cb4388d576f9191-EmailReviewComments-HTML%40fuchsia-review.googlesource.com.You received this message because you are subscribed to the Google Groups "owners-override" group.
To unsubscribe from this group and stop receiving emails from it, send an email to owners-overri...@fuchsia.dev.
'Yilong Li (Gerrit)' via owners-override
Feb 27, 2026, 1:13:42 PMFeb 27
to CQ Bot, Charles Celerier, Yilong Li, GI Roller, CL Deps Checker, Hunter Freyer, Owners Override, Andrew Davies, GI Try Builder
Yilong Li added 1 comment![]( "Open in Gerrit")
Patchset-level comments
File-level comment, Patchset 20 (Latest):
Yilong Li . resolved
This broke the tree. go/bbid/8688701886522434225
--
You received this message because you are subscribed to the Google Groups "owners-override" group.
To unsubscribe from this group and stop receiving emails from it, send an email to owners-overri...@fuchsia.dev.
To view this discussion visit https://groups.google.com/a/fuchsia.dev/d/msgid/owners-override/07535ee2ce00467aeccf0de4771717ddcf82cc02-EmailReviewComments-HTML%40fuchsia-review.googlesource.com.You received this message because you are subscribed to the Google Groups "owners-override" group.
To unsubscribe from this group and stop receiving emails from it, send an email to owners-overri...@fuchsia.dev.
'Yilong Li (Gerrit)' via owners-override
Feb 27, 2026, 1:16:37 PMFeb 27
to CQ Bot, Charles Celerier, Yilong Li, GI Roller, CL Deps Checker, Hunter Freyer, Owners Override, Andrew Davies, GI Try Builder
Yilong Li has created a revert of this change![]( "Open in Gerrit")
Related details
Attention set is empty
Submit Requirements:
Code-Owners
Code-Review
Dependencies-Satisfied
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
You received this message because you are subscribed to the Google Groups "owners-override" group.
To unsubscribe from this group and stop receiving emails from it, send an email to owners-overri...@fuchsia.dev.
To view this discussion visit https://groups.google.com/a/fuchsia.dev/d/msgid/owners-override/096e90a693a9711027295a044adc6e8f76f5ef44-HTML%40fuchsia-review.googlesource.com.To unsubscribe from this group and stop receiving emails from it, send an email to owners-overri...@fuchsia.dev.
Reply all
Reply to author
Forward
0 new messages