[L] Change in fuchsia/fuchsia[main]: [fshost] Remove BlockWatcher

2 views
Skip to first unread message

'James Sullivan (Gerrit)' via owners-override

unread,
Oct 9, 2024, 6:59:04 PM10/9/24
to Owners Override
Attention needed from Owners Override

James Sullivan has uploaded the change for review

James Sullivan would like Owners Override to review this change.

Commit message

[fshost] Remove BlockWatcher
Fixed: 371048873
Change-Id: Id9d3f66ff21d860240983eebc726f2f892b3e140

Change diff


Change information

Files:
  • M sdk/fidl/fuchsia.fshost/fshost.fidl
  • M src/bringup/bin/console/meta/console.shard.cml
  • M src/developer/sshd-host/meta/sshd-host.cml
  • M src/developer/sshd-host/meta/sshd-host.core_shard.cml
  • M src/developer/sshd-host/meta/sshd.cml
  • M src/devices/testing/driver-integration-test-shim/fixture.cc
  • M src/firmware/fastboot-tcp/meta/fastboot_tcp.cml
  • M src/firmware/fastboot-tcp/meta/fastboot_tcp.core_shard.cml
  • M src/firmware/fastboot-usb/meta/fastboot_usb.cml
  • M src/firmware/fastboot-usb/meta/fastboot_usb.core_shard.cml
  • M src/storage/fshost/BUILD.gn
  • M src/storage/fshost/integration/src/fshost_builder.rs
  • M src/storage/fshost/integration/tests/fshost_integration_test.rs
  • M src/storage/fshost/meta/base_fshost.cml
  • M src/storage/fshost/meta/fshost.bootstrap_shard.cml
  • M src/storage/fshost/src/main.rs
  • M src/storage/fshost/src/service.rs
  • M src/storage/fshost/src/watcher.rs
  • M src/sys/core/meta/core.root_shard.cml
Change size: L
Delta: 19 files changed, 32 insertions(+), 311 deletions(-)
Open in Gerrit

Related details

Attention is currently required from:
  • Owners Override
Submit Requirements:
  • requirement satisfiedAPI-Review
  • requirement satisfiedCode-Review
  • requirement satisfiedReview-Enforcement
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: newchange
Gerrit-Project: fuchsia
Gerrit-Branch: main
Gerrit-Change-Id: Id9d3f66ff21d860240983eebc726f2f892b3e140
Gerrit-Change-Number: 1129983
Gerrit-PatchSet: 14
Gerrit-Owner: James Sullivan <jfsu...@google.com>
Gerrit-Reviewer: Chris Suter <csu...@google.com>
Gerrit-Reviewer: James Sullivan <jfsu...@google.com>
Gerrit-Reviewer: Owners Override <owners-...@fuchsia.dev>
Gerrit-Reviewer: Stephen Demos <sde...@google.com>
Gerrit-CC: Fuchsia Try Builder <fuchsia-t...@fuchsia-infra.iam.gserviceaccount.com>
Gerrit-Attention: Owners Override <owners-...@fuchsia.dev>

--
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 on the web visit https://groups.google.com/a/fuchsia.dev/d/msgid/owners-override/eb3e588a99bb4d8374b1f16f601a34b11e542b57-HTML%40fuchsia-review.googlesource.com.
satisfied_requirement
open
diffy

'James Sullivan (Gerrit)' via owners-override

unread,
Oct 9, 2024, 6:59:07 PM10/9/24
to Owners Override, Chris Suter, Stephen Demos, CQ Bot, Fuchsia Try Builder, fuchsia-c...@fuchsia-infra.iam.gserviceaccount.com
Attention needed from Owners Override

James Sullivan voted Fuchsia-Auto-Submit+1

Fuchsia-Auto-Submit+1
Open in Gerrit

Related details

Attention is currently required from:
  • Owners Override
Submit Requirements:
  • requirement satisfiedAPI-Review
  • requirement satisfiedCode-Review
  • requirement satisfiedReview-Enforcement
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: comment
Gerrit-Project: fuchsia
Gerrit-Branch: main
Gerrit-Change-Id: Id9d3f66ff21d860240983eebc726f2f892b3e140
Gerrit-Change-Number: 1129983
Gerrit-PatchSet: 14
Gerrit-Owner: James Sullivan <jfsu...@google.com>
Gerrit-Reviewer: Chris Suter <csu...@google.com>
Gerrit-Reviewer: James Sullivan <jfsu...@google.com>
Gerrit-Reviewer: Owners Override <owners-...@fuchsia.dev>
Gerrit-Reviewer: Stephen Demos <sde...@google.com>
Gerrit-CC: Fuchsia Try Builder <fuchsia-t...@fuchsia-infra.iam.gserviceaccount.com>
Gerrit-Attention: Owners Override <owners-...@fuchsia.dev>
Gerrit-Comment-Date: Wed, 09 Oct 2024 22:59:01 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes

--
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 on the web visit https://groups.google.com/a/fuchsia.dev/d/msgid/owners-override/eb3e588a99bb4d8374b1f16f601a34b11e542b57-EmailReviewComments-HTML%40fuchsia-review.googlesource.com.
satisfied_requirement
open
diffy

'Adam Barth (Gerrit)' via owners-override

unread,
Oct 9, 2024, 11:22:46 PM10/9/24
to James Sullivan, Owners Override, Chris Suter, Stephen Demos, CQ Bot, Fuchsia Try Builder, fuchsia-c...@fuchsia-infra.iam.gserviceaccount.com
Attention needed from James Sullivan and Owners Override

Adam Barth voted Owners-Override+1

Owners-Override+1
Open in Gerrit

Related details

Attention is currently required from:
  • James Sullivan
  • Owners Override
Submit Requirements:
  • requirement satisfiedAPI-Review
  • requirement satisfiedCode-Review
  • requirement satisfiedReview-Enforcement
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: comment
Gerrit-Project: fuchsia
Gerrit-Branch: main
Gerrit-Change-Id: Id9d3f66ff21d860240983eebc726f2f892b3e140
Gerrit-Change-Number: 1129983
Gerrit-PatchSet: 14
Gerrit-Owner: James Sullivan <jfsu...@google.com>
Gerrit-Reviewer: Adam Barth <aba...@google.com>
Gerrit-Reviewer: Chris Suter <csu...@google.com>
Gerrit-Reviewer: James Sullivan <jfsu...@google.com>
Gerrit-Reviewer: Owners Override <owners-...@fuchsia.dev>
Gerrit-Reviewer: Stephen Demos <sde...@google.com>
Gerrit-CC: Fuchsia Try Builder <fuchsia-t...@fuchsia-infra.iam.gserviceaccount.com>
Gerrit-Attention: Owners Override <owners-...@fuchsia.dev>
Gerrit-Attention: James Sullivan <jfsu...@google.com>
Gerrit-Comment-Date: Thu, 10 Oct 2024 03:22:37 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes

--
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 on the web visit https://groups.google.com/a/fuchsia.dev/d/msgid/owners-override/50191b431cd30e6e16d777254d555974d4b20e95-EmailReviewComments-HTML%40fuchsia-review.googlesource.com.
satisfied_requirement
open
diffy

'CQ Bot (Gerrit)' via owners-override

unread,
Oct 10, 2024, 1:38:07 AM10/10/24
to James Sullivan, Auto-Submit, Adam Barth, Owners Override, Chris Suter, Stephen Demos, Fuchsia Try Builder, fuchsia-c...@fuchsia-infra.iam.gserviceaccount.com

CQ Bot submitted the change with unreviewed changes

Unreviewed changes

10 is the latest approved patch-set.
The change was submitted with unreviewed changes in the following files:

```
The name of the file: src/storage/fshost/meta/fshost.bootstrap_shard.cml
Insertions: 5, Deletions: 8.

The diff is too large to show. Please review the diff.
```
```
The name of the file: src/storage/fshost/src/watcher.rs
Insertions: 1, Deletions: 2.

The diff is too large to show. Please review the diff.
```

Change information

Commit message:
[fshost] Remove BlockWatcher
Fixed: 371048873
Change-Id: Id9d3f66ff21d860240983eebc726f2f892b3e140
Reviewed-by: Stephen Demos <sde...@google.com>
Fuchsia-Auto-Submit: James Sullivan <jfsu...@google.com>
API-Review: Chris Suter <csu...@google.com>
Owners-Override: Adam Barth <aba...@google.com>
Files:
  • M sdk/fidl/fuchsia.fshost/fshost.fidl
  • M src/bringup/bin/console/meta/console.shard.cml
  • M src/developer/sshd-host/meta/sshd-host.cml
  • M src/developer/sshd-host/meta/sshd-host.core_shard.cml
  • M src/developer/sshd-host/meta/sshd.cml
  • M src/devices/testing/driver-integration-test-shim/fixture.cc
  • M src/firmware/fastboot-tcp/meta/fastboot_tcp.cml
  • M src/firmware/fastboot-tcp/meta/fastboot_tcp.core_shard.cml
  • M src/firmware/fastboot-usb/meta/fastboot_usb.cml
  • M src/firmware/fastboot-usb/meta/fastboot_usb.core_shard.cml
  • M src/storage/fshost/BUILD.gn
  • M src/storage/fshost/integration/src/fshost_builder.rs
  • M src/storage/fshost/integration/tests/fshost_integration_test.rs
  • M src/storage/fshost/meta/base_fshost.cml
  • M src/storage/fshost/meta/fshost.bootstrap_shard.cml
  • M src/storage/fshost/src/main.rs
  • M src/storage/fshost/src/service.rs
  • M src/storage/fshost/src/watcher.rs
  • M src/sys/core/meta/core.root_shard.cml
Change size: L
Delta: 19 files changed, 32 insertions(+), 311 deletions(-)
Branch: refs/heads/main
Submit Requirements:
  • requirement satisfiedAPI-Review: +1 by Chris Suter
  • requirement satisfiedCode-Review: +2 by Stephen Demos
Open in Gerrit
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: merged
Gerrit-Project: fuchsia
Gerrit-Branch: main
Gerrit-Change-Id: Id9d3f66ff21d860240983eebc726f2f892b3e140
Gerrit-Change-Number: 1129983
Gerrit-PatchSet: 16
Gerrit-Owner: James Sullivan <jfsu...@google.com>
Gerrit-Reviewer: Adam Barth <aba...@google.com>
Gerrit-Reviewer: Chris Suter <csu...@google.com>
Gerrit-Reviewer: James Sullivan <jfsu...@google.com>
Gerrit-Reviewer: Owners Override <owners-...@fuchsia.dev>
Gerrit-Reviewer: Stephen Demos <sde...@google.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.
To view this discussion on the web visit https://groups.google.com/a/fuchsia.dev/d/msgid/owners-override/6a3bb760c2dc6f4ebcf2a6d91fffa5ecce5d159e-HTML%40fuchsia-review.googlesource.com.
open
diffy
satisfied_requirement

'GI Roller (Gerrit)' via owners-override

unread,
Oct 10, 2024, 1:45:00 AM10/10/24
to James Sullivan, CQ Bot, Auto-Submit, Adam Barth, Owners Override, Chris Suter, Stephen Demos, Fuchsia Try Builder, fuchsia-c...@fuchsia-infra.iam.gserviceaccount.com

Message from GI Roller

Open in Gerrit

Related details

Attention set is empty
Submit Requirements:
  • requirement satisfiedAPI-Review
  • requirement satisfiedCode-Review
  • requirement satisfiedReview-Enforcement
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: comment
Gerrit-Project: fuchsia
Gerrit-Branch: main
Gerrit-Change-Id: Id9d3f66ff21d860240983eebc726f2f892b3e140
Gerrit-Change-Number: 1129983
Gerrit-PatchSet: 16
Gerrit-Owner: James Sullivan <jfsu...@google.com>
Gerrit-Reviewer: Adam Barth <aba...@google.com>
Gerrit-Reviewer: Auto-Submit <auto-...@fuchsia-infra.iam.gserviceaccount.com>
Gerrit-Reviewer: Chris Suter <csu...@google.com>
Gerrit-Reviewer: James Sullivan <jfsu...@google.com>
Gerrit-Reviewer: Owners Override <owners-...@fuchsia.dev>
Gerrit-Reviewer: Stephen Demos <sde...@google.com>
Gerrit-CC: Fuchsia Try Builder <fuchsia-t...@fuchsia-infra.iam.gserviceaccount.com>
Gerrit-Comment-Date: Thu, 10 Oct 2024 05:44:55 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No

--
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 on the web visit https://groups.google.com/a/fuchsia.dev/d/msgid/owners-override/a41e66c112ea05e3cb8162b6e68b7e473e257268-EmailReviewComments-HTML%40fuchsia-review.googlesource.com.
satisfied_requirement
open
diffy
Reply all
Reply to author
Forward
0 new messages