This reverts commit 1423535d.

Reason for revert: OTA broke, this CL is a dependent on the change that breaks OTA

Original change's description:
> [svchost][devmgr][miscsvc] Add miscsvc
> 
> Startup a miscsvc binary, and provide a channel to it's '/svc/
> directory to svchost, which will then forward it services.
> 
> To start miscsvc only implements the paver service.
> 
> ZX-3861
> 
> Change-Id: Id411227e4c3d2b8add49978cf6ce1a76de509a48

TBR=teisenbe@google.com,abarth@google.com,shayba@google.com,surajmalhotra@google.com

Change-Id: I63906a5e8061fd2f5eb69bb5894533d6239127a2
No-Presubmit: true
No-Tree-Checks: true
No-Try: true

This reverts commit ffcb46fb.

Reason for revert: This change broke OTA.

Original change's description:
> [paver] Switch install-disk-image to talk to paver service
> 
> The last change will be to have the netsvc talk to the paver service in
> order to pave, rather than have it call into the install-disk-image
> binary.
> 
> ZX-3861
> 
> Tested: Paved NUC, pixelbook, vim2, astro
> 
> Change-Id: If1e0651fbae9c13ea75cf8d1aea1c7fac3b130ee

TBR=smklein@google.com,abarth@google.com,surajmalhotra@google.com

Change-Id: Idd58a94448947373c29ae051d0f13ab8cbd87a05
No-Presubmit: true
No-Tree-Checks: true
No-Try: true

Change-Id: I3f0d6f65221481d955d7b936ba0d8423090bc268

Test: CQ

Change-Id: I9e2b41561db1075f2f7a5e5fbc032ae9c7b52802

The last change will be to have the netsvc talk to the paver service in
order to pave, rather than have it call into the install-disk-image
binary.

ZX-3861

Tested: Paved NUC, pixelbook, vim2, astro

Change-Id: If1e0651fbae9c13ea75cf8d1aea1c7fac3b130ee

Moving from pyformat to yapf to better integrate with ClusterFuzz.

Change-Id: Id8f41ccbea1e819b6b72d8ce13c00f965f6d7ed7

Refactor two nearly-identical cases into a template.  It also
supports new cases that draw from the primary image manifest rather
than the tests manifest.

Test: no effect on the build
Change-Id: I3512e08e74b8afb54bbae7f4e1ada16c95069c3c

This should never be an explicit dependency for normal code.
It's always there implicitly.

Test: no immediate effect on the build
Change-Id: Ib0a334c1fcbcf95a11d10bfd0d24906697d8f5fc

These elements are isolated from the Core SDK as they are only our
take at a testing framework for Fuchsia.

Test: built SDK archive with "ninja sdk:e2e_testing"
Bug: DX-1315
Change-Id: I097a11d32c5c0572347581bbe67ed6f55a7085ee

Test=Included

Change-Id: Ied09ad9245ae11ba6139f23c015641ad22a6b2a6

Move examples which demonstrate testing fixtures to correct folder so that
they are easy to find.

TEST=CQ

Change-Id: Ibf3e440adbec6b2acb9722ab46c70ace0f8f66de

Startup a miscsvc binary, and provide a channel to it's '/svc/
directory to svchost, which will then forward it services.

To start miscsvc only implements the paver service.

ZX-3861

Change-Id: Id411227e4c3d2b8add49978cf6ce1a76de509a48

MA-602 #done

Change-Id: Ib77132f8e1d90a45d80eb2fcd34d5b7d19ac9ac8

The paver service implements the fuchsia.paver/Paver FIDL
service. A new process managed by devcoordinator will need to be
created to host it until such a time we can launch components in
the zedboot environment.

ZX-3861

Change-Id: I23e032447e4ebfdf45e63719c02782c0c753d017

This CL is a step toward gathering more process data. It adds reporting of
process names.

Change-Id: I88b6dd1810d69e27fd6fa87ab184ed6e08809b9e

The protection type will soon add WEP support.
The new enum replaces the previously used Option<Rsna> with
the following mapping:
None => Protection::Open
Some(rsna) => Protection::Rsna(rsna)

Test=Existing tests cover this CL already.

Change-Id: I271013f8df246afc66c56e5baf3471c684a3291d

This removes the need to add them via the
--extra_authorized_ssh_keys GN flag.

DX-1308 #comment

Change-Id: Ib9af7b8b2e44a9e4356afbbb45c04dd1d8b7a982

Adds documentation under //docs to describe best practices for test
components and how to write tests for rust.

Adds example hello world components with accompanying test components to
//examples/hello_world.

Change-Id: I8be0821baf869c9002ccafd7e94e2f7096f68f49

Generating various bitwise operators on bits values.

Test: added one which shows how to use operators, and verifies their
result

Change-Id: I0a80d75af392d9bd30f8fd253c1a60f99f470990

All metric/property/children keys are strings in inspect, but sometimes
lexicographic sorting is not desired. For example, values with numeric
keys make the most sense to sort numerically.

This change changes the logic to the following:
- If all children of a node have keys consisting of digits, sort
numerically.
- Otherwise, sort lexicographically.
- Perform the above for metrics and properties as well.

This behavior makes a sensible default based on current use cases.
Future use cases may implement their own sorting if needed.

CF-684: #done
Change-Id: I378dfa7ce60e51252139e14bb3c9ccc14ef7eb72

Also adds Envelope support and a way to stop a value that is being calculated from being
included in the resulting JSON.

DX-1013 #comment Add support for tables.

Change-Id: I72046cf2c401fdd1c1556a058502048105d96dfe

As part of the plan outlined in PT-135, catapult converter will run
after the host side metrics / Fuchsia benchmark results computation.
Create a catapult converter host side tool in order to allow for this.

Also, because src/lib/uuid has a dependency on Fuchsia, inline the UUID
creation logic into catapult converter (since it's actually pretty
simple).

Add additional casts to appease the mac toolchain.

PT-135 #comment-patch

Test:
$ out/default/host_x64/host_catapult_converter_test
catapult_converter test passes on CQ
host_catapult_converter test runs and passes on CQ

Change-Id: I72e2fb62939b99701494f6f1cd7832b2882ec4f2

given that we don't see the warnings regarding the timeout in the flaky runs,
these new warnings would help us figuring out why the logs are empty: is LogMany()
called with no messages? is Done() called whith logs_ empty?

FLK-179 #comment

TESTED=`fx run-test feedback_agent_tests -t log_listener_unittest`

Change-Id: I11ffb1d9efad8c12a136140405e3b5aca51c9c54

this allows crashpad_agent to not block when waiting for feedback data and serve other requests meanwhile

DX-1469 #done #comment

TESTED=`fx run-test crashpad_agent_tests`
TESTED=`fx shell crasher` (report id ef90b0e8f74f35b9)
TESTED=`fx k crash` (report id 991c16cca67d56ea)

Change-Id: I59adfb71f63be1cc87f0d4a7646957a55e34f3b7

We consume some SDK clients as prebuilts, so they must implement the new
API before we call into it.

Tested: No behavior changed; modular_tests, started story in sysUI
SCN-1018 #comment
SCN-1291 #comment

Change-Id: Ifb8797b81c12c7ae293e295c1960e02ee3be1168

CF-719 #done

Change-Id: I3c747006b48d3fb0d2551b42062dfd440aea4970

Key idea is to reduce indentation to make following
changes more readable.

Test=Existing tests, non functional change, just moving code

Change-Id: If5ac42978fe42b8850396fb6a9840fce3826f8de

We consume some SDK clients as prebuilts, so they must implement the new
API before we call into it.

Tested: No behavior changed; modular_tests, started story in sysUI
SCN-1018 #comment
SCN-1291 #comment

Change-Id: Id6a2354b942e5390fce855f81fd913f770423340

fidl_publisher.h was in garnet/bin/media/util, but was only used in
the netconnector implementation. This CL moves it to
garnet/bin/netconnector, which will be deleted once netconnector
is no longer used.

TEST: no behavior change
Change-Id: Iabada3ec6da2084861e7a9c2d41c9ffb7d1309d8

Using -i as repeat value will cause to run indefinitely. If combined
with break on failure, will run until the first fails occures.

Added test.

ZX-4009 #done

TEST=zxtest-test

Change-Id: I10db1f74cd811842e8a82cd5913eb00b8b8dd017

Test: CQ

Change-Id: I964370ac85b97ea7e9bf21b1ed306c78f39d53bf

We no longer want a separate bin/ directory for inspect_unittests.

Change-Id: I05de32a36ba7a3718981a14e77882aa94806b956

Because struct member defaults do not get properly resolved (FIDL-486),
we need to workaround that when generating JSON. Recent change broke
functionality for string constants.

FIDL-486 #comment
FIDL-619 #done

Change-Id: If33aa05d5b6267b3e461bb76a9f87755b3388870

When enabling power domain, if the rail is already enabled, we fail.
This changeset rectifies this wrong behavior by returning success.
All this code is changing with https://fuchsia-review.googlesource.com/c/fuchsia/+/273632/
and that change will not be submitted without the unit tests, that I am
currently writing. The current change is to unblock folks before the
other change is reviewed.

ZX-3990 #Done

Test: fx full-build. Without this change, I see the bind failure for
power driver. With this change, I do not see the error. device-enumeration-test is fixed here
https://fuchsia-review.googlesource.com/c/fuchsia/+/277336. It fails without this change.
Succeeds with this change

Change-Id: Iec6fe6d8ec041a301287e4f0b787c3b5519d47d6

In interrupt_suspend_test, wait for the thread under test to terminate
to reduce interference with subsequent tests.

Change-Id: Icb20edf46bb8b3b645c74bbd298b3dc1c00a9bfd

Create class-based AHCI controller.
Some improvements in error handling in initialization, more to come.

Tests: QEMU boot with disk and blktest
Change-Id: I102c44abc46e2ae1b57987cf97cada442794a480

Removing its usage from our BUILD files. it would be replaced with a new
thing in future.

This is part 2

CF-749 #progress

TEST=CQ

Change-Id: I25144d011b42ba1f574e62d17951714c1645df8d

This reverts commit c30d780d.

Reason for revert: <INSERT REASONING HERE>

Original change's description:
> [system-ota-tests] Switch to consuming packages.tar.gz
> 
> Previously, infra tried to pass in the amber-files produced by
> the build into every test, but this was consuming too much
> space. Instead, this patch switches the system ota tests to
> consume the packages.tar.gz that's produced by the build.
> 
> Change-Id: I42e683d4b39f43544d755c8754649309c72de61b

TBR=ctiller@google.com,etryzelaar@google.com,computerdruid@google.com

Change-Id: I86e7c4675705fd99c7d1fe4f0936ebe3e81506e7
No-Presubmit: true
No-Tree-Checks: true
No-Try: true

Brings behavior up to parity with the available internal
documentation on the limitations of the GI infra.

Change-Id: I6abc8b76918ab3ded93c28647d487de3f98e57c8

Add ktracing for kernel mutex events; defaulted to off at compile time.

Note: this CL only adds trace definitions and tracing behavior to the
kernel.  To make any use of these events, a follow up change which
implementes the trace importer behavior is needed.

Change-Id: Ia759ff3a62cb0593fa269963a7959b17aabedd24