[botanist][health_checker] Check body of response from amt reboot to see if it
actually worked or not.
Currently in the health_checker, we just think it works if the http POST
doesn't return an error, but we need to check the body of the response to see
if we get a return value of 0. The other return value we've seen is 2, which is
[gndoc] Fix gndoc unstable output
This change adds an additional sort step when gndoc iterates over
a map. It should fix the gndoc output instability issue.
[serial][reboot] Add serial rebooting.
This adds support for rebooting over serial and gives preference to it
[ssh][reboot] Make SSH timeout correctly in the reboot paths.
This sets the TCP timeout in the SSH config, and uses a more complex
command to execute reboot that *should* cause SSH to return.
[ssh] Wait for boot before attempting to SSH.
This rearranges the construction of the goroutines in execute() such
that we are no longer attempting to SSH before the system is paved and
has invoked mexec/reboot.
[dev_finder] protection from malformed responses
Bug: DX-1498 #comment
[fastboot] Remove fastboot support.
vim2s which previously needed this are transitioning to a non-Fastboot
flow, we will at a later date add Fastboot support to a future iteration
of health checker to provide more complex device support.
[buildidtool] Update to handle groups of DSOs
This changes alters the way the tool works in order to be more efficent.
It handles groups of DSOs that all have the same build ID. Later we might
add support for additional files like .dwp or .gsym files which won't have
build IDs at all but still should be associated. We will see.
[testrunner] Print test durations
[runner] Fix environment specification
"A=B;cmd" doesn't work on fuchsia. Use the session's method of setting
[botanist] Don't rely on host memory detection
Detecting the amount of available memory using sysinfo is unreliable,
especially inside the container and it seems like there's no reliable
way to do so, see for more context:
So instead simply rely on a value provided on the command line.
Switch to more forcefull reboot with PowerStatePowerCycleSoft
We have seen Sata M2 devices that were invisible after reset.
PowerStateMasterBusReset seems to do the equivalent of hitting
the now defunct reset button motherboards used to have, which is
not the same than a power cycle.
PowerStatePowerCycleSoft is not a real power cycle either given
that the CPU (AMT), memory, and the NIC are still running, but
we are hopeful that the rest of the system is powered off as indicated
by the power button LED going off when we send that command.
As a result, we think this may help with Sata devices being gone
after being potentially being wedged by a bad driver and not fixed
by a simple reset.
[dev_finder] sends requests to multiple addresses
Due to a netstack bug, multiple sockets can't join the same multicast
address. As a temporary workaround, sends mDNS requests to multiple
Adds 22.214.171.124 in addition to the current default value 126.96.36.199.
Bug: DX-1453 #comment
[mdns] add support for custom addresses to the mdns library
[mdns] make tests actually run
Tests never ran before because the test method names started with the
lowercase "test". They should start with "Test".
Also fix a failing test.
[botanist] Allow specifying number of CPU and memory for QEMU
Provide a default which is selected based on the underlying machine.
[upload_debug_symbols] Fix incorrect exit code
[dev_finder] add tests for the `resolve` command
Also there's some code shuffling for better code reuse.
Bug: DX-1402 #comment
[dev_finder] add tests
We have no tests for dev_finder. We should.
This CL adds tests for the "list" command. I'm going to add tests for
the "resolve" command in a follow-up CL.
For this, add some facilities in the implementation that help testing.
[upload_debug_symbols] Don't enumerate bucket state.
GCS uploads already support no clobber options and will make individual
requests against the bucket to see if a file already exists, there is no
need to map existing files locally.
[testsharder] Adding "pool" as a dimension
Bug: IN-1154 #comment
[dev_finder] Issue mDNS queries on multiple ports (5353,5356 by default)
This teaches the dev_finder tool to issue mDNS queries on multiple
ports, responding to whichever port answers first. The -port flag now
accepts a comma separated list of port numbers. The default is to query
on port 5353 and 5356. This way the dev_finder tool will keep working
while we (temporarily) shift the responder port around.
[testsharder] Adding "testbed" as a dimension
Revert "[qemu] Revert to user networking"
This reverts commit 87b65fadc2a6ab114dc111a805bf692b1d89a679.
Reason for revert: This change was only temporary
Original change's description:
> [qemu] Revert to user networking
> The infrastructure is not yet ready to support a tun/tap netdev backend,
> so revert back to a user backend in the meantime.
> Change-Id: Id5be695d59f190ee2118de2f1c14534e4228c12f
# Not skipping CQ checks because original CL landed > 1 day ago.
[bootserver] Add support for --bootloader.