| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
| |
This updates all dependencies on the project, which is mainly needed
by python3.8 but can be done independentely.
This also disables multiple false positive lint errors and disable a
new check that we don't need.
|
|
|
|
|
| |
Newer pylint versions detect and complain about unnecessary elif/else
after a continue/break/return clause. Let's remove them
|
|
|
|
|
| |
Newer version of pylint detect when a comprehension would not be needed.
Let's remove all the ones that are indeed extraneous
|
| |
|
| |
|
|
|
|
|
| |
The working directory must be specified relative to the input root,
i.e., the leading slash must be dropped.
|
|
|
|
| |
Sandbox implementations rely on this.
|
| |
|
| |
|
|
|
|
|
|
| |
Set the workspace plugin `__last_successful` attribute.
loader: Use empty string as default when last_successful is missing
|
| |
|
| |
|
|
|
|
|
| |
We don't have many of the `--track` options anymore, so we can remove
handling for them in parts of the code that don't need it.
|
| |
|
|
|
|
| |
This allows access by buildbox-casd running as different user.
|
| |
|
| |
|
|
|
|
|
|
|
| |
This allows access by buildbox-casd running as different user with a
suitable umask. As this is not generally safe in global temp directories
such as /tmp, make the `dir` parameter mandatory and add a note to the
documentation.
|
|
|
|
| |
This allows access by buildbox-casd running as different user.
|
| |
|
|
|
|
|
| |
It doesn't seem like we need to call GetCapabilities here, as we're not
querying anything. Remove the call to save some time.
|
| |
|
|
|
|
|
|
| |
Change _CASCacheUsageMonitor to hold a CASDChannel instead of
it's parent CASCache. This means that when in spawn mode, we don't need
to pickle the CASCache, only the CASDChannel.
|
|
|
|
|
|
|
|
|
|
|
|
| |
Encapsulate the management of a connection to CASD, so we can hide the
details of how it happens. This will make it easier to port to Windows,
as we will have to take a different approach there.
Remove the _LimitedCASDProcessManagerProxy, as CASDChannel does
everything the child jobs will need.
Also make get_local_cas() public, since it is already used outside of
the CASCache class. Make get_cas() public to match.
|
|
|
|
|
|
|
| |
Elsewhere in cascache, cleaning up is called 'release_resources', so
follow that convention for consistency.
Also fix a case where self.process was not set to None in terminate().
|
|
|
|
|
|
|
|
|
|
| |
Make it easier to specialize handling of the buildbox-casd process on
Windows, by splitting it into it's own class. This allows us to
encapsulate some decisions, and decreases the complexity of the CASCache
class.
Take some of the complexity out of this file by splitting the
responsibility of managing the process out to another file.
|
|
|
|
|
|
| |
There was some duplicated functionality between `Stream._load()` and
`Stream._load_tracking()`, this is factored out and hopefully makes
the cognitive complexity a bit lower.
|
| |
|
|
|
|
|
|
|
|
|
|
| |
These were caused by unhandled errors from plugins when calling
`Source.get_consistency()`. This doesn't really solve the problem,
since that interface is still used un-wrapped elsewhere, but it
enables removing `Element.__schedule_tracking()` and fixes a bug.
Ultimately we'd like to remove `Source.get_consistency()`, so this
isn't too long-term of a problem.
|
| |
|
| |
|
| |
|
| |
|
| |
|
|
|
|
|
|
| |
Just as with the previous commit, inspired by our code quality
linter. Poor practice from the previous function probably snuck in
here too.
|
|
|
|
|
| |
This is on suggestion of our code quality linter - it should make
understanding the pre-requisites of the function simpler.
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
|
|
|
|
| |
This was previously used to handle unstable cache keys, but has been
obsoleted by @traveltissues' recent improvements.
|
|
|
|
|
|
| |
Move the logic that decides whether we can schedule an element for
assembly *into* this function. This makes it a fair bit easier to
grok, and reduces the complexity of `_update_state()`.
|
|
|
|
|
|
|
|
|
| |
In the olden days on linting BuildStream, we did not had plugin
dependencies installed in the environment. Now that we run these things
via tox, we have all the dependencies (including plugin dependencies) in
the virtual environment that tox uses to run the commands.
So remove the disabling of import errors for `arpy` module.
|
| |
|
|
|
|
|
|
|
| |
As discussed over the mailing list, reformat code using Black. This is a
one-off change to reformat all our codebase. Moving forward, we
shouldn't expect such blanket reformats. Rather, we expect each change
to already comply with the Black formatting style.
|
|
|
|
|
| |
This is implemented with a new ProfileError, ensuring a user
doesn't wait for a profile to complete that was never valid.
|