| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
|
| |
Test the arti-extra bin, which includes most of `full` and
`experimental` features, too.
|
| | |
|
| |
|
|
|
| |
The default has been increased to 300, so there's no need to override it
here.
|
| |
|
|
|
|
|
|
|
|
|
|
|
| |
This will give us a bit more flexibility, e.g. to pass additional
arguments.
Now that the python linters understand it, we fix some linting issues.
We preserve the ability to run the test on other networks by making the
Config.network optional. The new default of `None` means to use the
built-in module to create the network.
The functionality of the test itself is the same.
|
| |
|
|
|
| |
This helps avoid subtle mismatches of the sort fixed in the previous
commit.
|
| |
|
|
|
| |
We were accidentally using the arti-bin command-line arg to also
override arti-bench-bin.
|
| | |
|
| |
|
|
| |
Everything now uses the json version, instead.
|
| | |
|
| | |
|
| |
|
|
|
| |
No particular need to separate them, and the merged version is easier to
follow.
|
| |
|
|
|
|
| |
This organizes the code a bit better and gives nicer output. It also
separates individual test cases and subcases, continuing to try to
complete other tests when one fails instead of exiting.
|
| | |
|
| |
|
|
|
|
|
| |
chutney replaced Node.socksport (a single optional port) with
Node.socksport_endpoints (a sequence of bound addresses and ports).
Update for compatibility, and test all of the exposed endpoints where
relevant.
|
| | |
|
| | |
|
| | |
|
| |
|
|
|
|
|
|
|
|
|
| |
* Gets rid of shelling out to jq. We could also accomplish that with the
json module, but this way also gives us:
* Better error detection and handling. In particular this lets mypy
statically detect trying to access fields of the wrong name or type.
It will also make it easier to bump to the latest chutney, which has
replaced `socksport` with `socksport_endpoints`, which would be more
difficult to work with as json, especially via `jq`.
|
| | |
|
| | |
|
| |
|
|
|
| |
This is primarily a proof-of-concept for converting these scripts before
tackling the more complext 'test' script.
|
| | |
|
| |
|
|
|
| |
In particular, use `str` instead of `Path`, so that it's directly
encodable and decodable by the json module.
|
| | |
|
| |
|
|
|
|
| |
Previously this was naively wrapping strings in double-quotes. This
works ok for escaping internal whitespace, but doesn't escape other
special characters.
|
| |
|
|
| |
Bump chutney, and migrate to CLI changes introduced in chutney!101
|
| | |
|
| |
|
|
| |
Also search several candidate paths.
|
| |
|
|
|
|
| |
This is partly just to get more code out of "main", but also paves the
way a bit for reusing this config code in the `integration-e2e-shadow`
wrapper script, to support the same command-line arguments there.
|
| |
|
|
|
|
| |
Since we're validating the presence of jq, we might as well handle it
consistently with the other dependencies and provide a way to specify
where it is.
|
| |
|
|
|
|
| |
Encapsulate the logic of finding and validating each option into their
own objects (e.g. so that specification of corresponding command-line
option goes together with other related logic).
|
| |
|
|
|
|
| |
This wasn't a pre-existing environment variable, and now it can be set
via command-line if it needs to be overridden. I don't see much reason
to use an environment variable here.
|
| | |
|
| | |
|
| |
|
|
|
| |
We avoid much refactoring at this step to minimize functional changes
from the shell script.
|
| | |
|
| |
|
|
|
|
| |
This makes the `setup` script a bit more clearly single-purpose of
resolving and storing the configuration, without actually acting on it
yet.
|
| |
|
|
|
|
|
|
|
| |
It made sense to have this as part of the setup when the chutney network
itself didn't include arti; i.e. this used to be just validating the
non-chutney tor network before adding chutney nodes.
Now that the chutney network natively includes arti nodes, `chutney
verify` is testing those nodes; i.e., testing arti.
|
| | |
|
| |
|
|
|
|
|
|
| |
In particular, this is to avoid floods of:
> compute_drain_rate(): Bug: Computing stream drain rate with zero time delta
in tor logs.
|
| |
|
|
|
|
| |
arti isn't trying to use conflux yet, but the tor nodes are. Things
were stumbling along without this, but adding it reduces some log noise,
and might help with consistency.
|
| |
|
|
|
|
| |
This reduces some noise and warnings, and may help with consistency.
e.g. this avoids everything trying to repeatedly contact the directory
authorities before they're up.
|
| |
|
|
|
|
|
|
|
| |
In this commit there should be little effective difference, other than
combining the "setup", "configure", and "wait_for_bootstrap" steps.
"chutney bootstrap" also support multi-phase networks, though, in which
chutney waits for some nodes to fully bootstrap before bringing up other
nodes.
|
| |
|
|
|
|
|
|
|
|
|
| |
This is the time we wait for the network to bootstrap before giving up.
With the larger network we seem to need this extra time.
We've already increased the default in chutney to 300; no reason to use
a smaller value here.
Keep the explicit-setting for now as a convenience in case we want to
modify it again.
|
| | |
|
| |
|
|
| |
Authorities also function as exit relays; we don't need to add so many.
|
| | |
|
| | |
|
| | |
|
| |
|
|
| |
An arti proxy is already started as part of the chutney network.
|