Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

enh: 35% faster is_valid_ip() when fast=1 #516

Draft
wants to merge 11 commits into
base: master
Choose a base branch
from
Draft

Conversation

speed47
Copy link
Collaborator

@speed47 speed47 commented Dec 19, 2024

No description provided.

When restricted commands need to be used during tests,
we now use "account0" which has all these commands granted,
instead of granting/revoking commands every time with no added
value with respect to the tests.

This was previously required for OSes that have a limit to the
number of groups an account can be a member of, but these OSes
have now long been unsupported.
This is slow and almost never catched a bug, so consistency check is
still supported but will not run by default, as it is quite slow,
checking the system between each and every test. The option
--skip-consistency-check is now ignored, and a new option to enable
it has been added: --consistency-check
Remove 'partial_members' and 'full_members' from JSON output,
which were replaced by 'members' and 'guests' since pre-v3.00.00
@speed47 speed47 added the tests:long Launch tests (multi-distros, w/o cc) label Dec 19, 2024
To avoid confusion, we now use 'netblock' to talk about 10.0.0.0/8,
with 10.0.0.0 being the 'prefix' and '8' being the 'prefix size'
or 'netblock size'. Use these words everywhere in the code and
documentation for clarity
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tests:long Launch tests (multi-distros, w/o cc)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant