| Commit message (Collapse) | Author | Age | Files | Lines |
| ... | |
| |\ \ \
| |_|/
|/| |
| | |
| | | |
CI: Run coverage test on bigger runners
See merge request tpo/core/arti!1944
|
| | |/
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
This is starting to get SIGKILL during linking. Lack of memory,
presumably.
@anarcat suggests we should run this on bigger runners, and that this
could be achieved by applying the `tpa` tag.
CC @trinity-1686a
See also !1943.
|
| |\ \
| | |
| | |
| | |
| | |
| | |
| | | |
Clean up, tune, and correct various parameters related to introduction points.
Closes #1210
See merge request tpo/core/arti!1924
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
(Also, correct the comments that describe them.)
We may as well match the spec names when they aren't completely
bogus.
We are already renaming these parameters for this release, so it
isn't an additional breaking change.
|
| | | |
| | |
| | |
| | | |
See #1210 "question 4".
|
| | | |
| | |
| | |
| | | |
See discussion at #1210, "question 5".
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | | |
Part of #1210.
These values are not necessarily the cleverest possible, but they
match the C implementation. We can tune them if we need to.
|
| | | |
| | |
| | |
| | |
| | | |
This makes us more spec-conformant. See discussion at #1210
"question 3".
|
| | | |
| | |
| | |
| | |
| | |
| | | |
The old names were somewhat inaccurate. (I would have considered
keeping and deprecating the old names, but we already have breaking
changes in tor-netdir.)
|
| | | | |
|
| | | |
| | |
| | |
| | | |
See #1210 "question 2" for more discussion here.
|
| | | |
| | |
| | |
| | | |
See discussion and rationale on #1210 "question 1".
|
| |\ \ \
| | | |
| | | |
| | | |
| | | | |
CI: Run some tests a bit earlier
See merge request tpo/core/arti!1940
|
| | | | |
| | | |
| | | |
| | | |
| | | | |
This is a very fast check. It's last in the list so that nothing gets
blocked *by* it, but we want it to run early.
|
| | | | |
| | | |
| | | |
| | | |
| | | | |
In particular, let's not wait for all the other builds, to make
iterating minimal-versions through CI less painful.
|
| |\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | | |
CI: Run coverage test only in scheduled pipelines
See merge request tpo/core/arti!1943
|
| | | |_|/
| |/| |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
This is starting to get SIGKILL during linking. Lack of memory,
presumably.
@nickm suggests running it only daily since apparently we don't look
at the output that often anyway.
|
| |\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
tor-keymgr: Lowercase the role when building the ArtiPath.
Closes #1195
See merge request tpo/core/arti!1934
|
| | | | | | |
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
The `slug` docs say slugs are not allowed to contain uppercase
characters, so let's return an error if they do.
Part of #1195
|
| | | |/ /
| |/| |
| | | |
| | | |
| | | |
| | | |
| | | | |
The role is a slug, and slugs are not allowed to contain uppercase
characters.
Closes #1195
|
| |\ \ \ \
| |_|/ /
|/| | |
| | | |
| | | | |
rtcompat: Fix a rustdoc link.
See merge request tpo/core/arti!1938
|
| | | | | |
|
| |\ \ \ \
| |_|/ /
|/| | |
| | | |
| | | | |
Implement most of state_dir
See merge request tpo/core/arti!1935
|
| | | | | |
|
| | | | |
| | | |
| | | |
| | | | |
This is now InstanceStateHandle::purge.
|
| | | | |
| | | |
| | | |
| | | |
| | | | |
We should add more tests for these error representations but we want
to rework these errors so let's not do all that now.
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Apparently we need
https://github.com/dbrgn/tracing-test/pull/17
too.
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
We need this bugfix
https://github.com/dbrgn/tracing-test/pull/15
for `#[traced_test]` when `Result` is redefined.
Fixes the tests with minimal-versions.
|
| | | | | |
|
| | | | |
| | | |
| | | |
| | | | |
We use `/` to separate, not `+`.
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | |
| | | |
| | | |
| | | |
| | | | |
This would fix the tests on Windows I think ? (Tests which don't seem
to run in routine CI AFAICT.)
|
| | | | |
| | | |
| | | |
| | | | |
As previously decided and documented.
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
We're about to reuse this.
This changes the Resource in the error to be more accurate, too. This
will be more important as we add another call site.
|
| | | | |
| | | |
| | | |
| | | | |
We're going to want this some more.
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | |
| | | |
| | | |
| | | | |
Remaining formatting churn.
|
| | | | | |
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
And also instance_peek_storage, which needs to share some code, so
determined the shape of this implementation.
Make StateDirectory have the (one) field we need.
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
These are the straightforward methods which, given a CheckedDir,
perform the relevant operation.
Make InstanceStateHandle have the necessary fields.
Constructing an InstanceStateHandle is a bit complex and will come in
a moment.
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
In practice, allowing Bug here was quite inconvenient for everyone.
I think very few call sites will want to throw anything interesting.
If they *do* want to cause an error, without panicking, they could
write a syntactically invalid slug.
|
| | | | |
| | | |
| | | |
| | | |
| | | | |
state_dir doesn't support a "shared read-only" mode, so it needs an
error.
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Introduce a Bug variant.
Then, use it for conversions from BadSlug. See the comment for why I
think this is appropriate.
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
You don't need to consume a Mistrust to get a CheckedDir - you use a
Verifier and then something clones it for you. So StateDirectory::new
should take &Mistrust.
And instance_peek_storage needs T to be deserialisable!
|
| | | | |
| | | |
| | | |
| | | |
| | | | |
This is rather more serious than many deletions - it wipes out the
whole directory. And this name goes better with purge_instances.
|