| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
|
| |
Avoid unnecessary copies and use CopyFrom() instead of copying fields
one by one where copies are necessary.
|
| | |
|
| | |
|
| |
|
|
|
|
| |
Let the sandbox know whether the buildtree will be required. This allows
the remote execution sandbox to skip buildtree download when it's not
needed.
|
| |
|
|
| |
This enables conditional output file fetching with remote execution.
|
| |
|
|
|
| |
This will enable SandboxRemote to download the file blobs of the build
directory only when needed.
|
| |
|
|
|
| |
Output files need to be downloaded only if they are required in the
local cache or if artifact push remotes need them.
|
| | |
|
| |
|
|
| |
Extracted from remote_missing_blobs_for_directory().
|
| |
|
|
| |
Download them separately as they are not always required.
|
| |
|
|
|
| |
This allows skipping file download in pull_tree()/_fetch_tree() and is
preparation for conditional output file fetching.
|
| | |
|
| | |
|
| | |
|
| |
|
|
|
| |
Ensure required artifact files are available for elements that need them
even if the context does not require artifact files for all elements.
|
| |
|
|
|
| |
This will be used to allow build sessions with a partial local CAS when
using remote execution.
|
| |
|
|
|
|
| |
Element plugins will be required to support virtual directories before
the next major version. Drop support for fallback from remote to local
execution as this blocks partial CAS.
|
| |\
| |
| |
| |
| | |
test: Fix remaining references to bst push/pull
See merge request BuildStream/buildstream!1285
|
| | |
| |
| |
| |
| | |
As the `bst push` is now called `bst artifact push`, update the
remaining comments in testsuite that still refer to it as `bst push`.
|
| |/
|
|
|
| |
As the `bst pull` is now called `bst artifact pull`, update the
remaining comments in testsuite that still refer to it as `bst pull`.
|
| |\
| |
| |
| |
| |
| |
| | |
_project.py: Do not find default targets in .bst directory
Closes #992
See merge request BuildStream/buildstream!1284
|
| |/
|
|
|
|
|
|
|
|
| |
In cases where element path is set to `.` and the project has some
junctions, we have to be careful when trying to find the default
targets. We can't simply find all files ending with `.bst`. But, we
first need to ensure that we do not recurse down the `.bst` directory
which is used to store BuildStream internals.
Fixes #992.
|
| |\
| |
| |
| |
| | |
cleanupjob: remove unused '_artifacts' member
See merge request BuildStream/buildstream!1283
|
| |/ |
|
| |\
| |
| |
| |
| | |
{artifact,source,cas}cache: fix one-shot callbacks
See merge request BuildStream/buildstream!1273
|
| |/
|
|
|
|
|
|
|
|
|
|
| |
Remove the unintended side-effect of only being able to iterate the
ref_callbacks results once. This would mean that subsequent calls to
clean() were not honoring the required_refs correctly.
Update CasQuota.clean() to call the ref callbacks and iterate over
their results.
When registering ref callbacks; pass the methods, and not the generators
that they return.
|
| |\
| |
| |
| |
| |
| |
| | |
Artifact as a Proto: protos and service
Closes #965
See merge request BuildStream/buildstream!1259
|
| | |
| |
| |
| | |
Part of #965
|
| | |
| |
| |
| | |
Part of #965
|
| |/
|
|
| |
Part of #965
|
| |\
| |
| |
| |
| | |
Re-enable `key in somenode` construct after YAML new world order
See merge request BuildStream/buildstream!1280
|
| | |
| |
| |
| |
| |
| |
| | |
Now that we permit `key in somenode` remove the no longer needed function
to check if a node contains a key.
Signed-off-by: Daniel Silverstone <daniel.silverstone@codethink.co.uk>
|
| | |
| |
| |
| |
| |
| |
| | |
Now that we permit `key in somenode` again, remove the unnecessary extra
API of Plugin.node_has_member()
Signed-off-by: Daniel Silverstone <daniel.silverstone@codethink.co.uk>
|
| |/
|
|
|
|
| |
Now that we've had the fallout settle, support `key in somenode` again.
Signed-off-by: Daniel Silverstone <daniel.silverstone@codethink.co.uk>
|
| |\
| |
| |
| |
| | |
_yaml.py: Add node_keys()
See merge request BuildStream/buildstream!1274
|
| |/
|
|
|
|
|
|
| |
There were a number of places where `node_items()` was called and the values
discarded. Since values can be expensive to compute, this replaces any such
places with a `node_keys()` function call instead
Signed-off-by: Daniel Silverstone <daniel.silverstone@codethink.co.uk>
|
| |\
| |
| |
| |
| | |
Some small cleanups
See merge request BuildStream/buildstream!1272
|
| | |
| |
| |
| |
| |
| |
| | |
We don't need to spend the CPU constructing path except when there is
an error which needs it, so don't.
Signed-off-by: Daniel Silverstone <daniel.silverstone@codethink.co.uk>
|
| |/
|
|
|
|
|
|
|
| |
In the case that `load_data()` raises a `LoadError` we want to catch
and re-raise it, augmenting the message with where the load came from
which is best done with the `displayname` to include junction name if
available.
Signed-off-by: Daniel Silverstone <daniel.silverstone@codethink.co.uk>
|
| |\
| |
| |
| |
| | |
lint: Fix or silence 'cyclic-import' errors and enable pylint for it
See merge request BuildStream/buildstream!1250
|
| |/
|
|
|
|
|
|
| |
Cyclic imports can be confusing because the order in which we import
dependencies can make the import fail or not. We should not rely on
ordering of imports for our code. This fixes everywhere possible the
imports and silence explicitely some which are not convenient or would
require big refactors
|
| |\
| |
| |
| |
| | |
typo: writtent -> written
See merge request BuildStream/buildstream!1270
|
| |/ |
|
| |\
| |
| |
| |
| | |
FOLLOW UP: Add NEWS entry for YAML New World Order
See merge request BuildStream/buildstream!1264
|
| |/
|
|
|
|
| |
This is a follow-up to !1257 - YAML New World Order.
Now that !1257 has landed, plugin's *must* use the Plugin.node_*
family of functions to access or modify node members.
|
| |\
| |
| |
| |
| | |
app.py: fix spelling 'depency' -> 'dependency'
See merge request BuildStream/buildstream!1266
|
| |/ |
|
| |\
| |
| |
| |
| | |
_project.py: cache full project path for quicker lookup
See merge request BuildStream/buildstream!1269
|
| |/
|
|
|
|
|
|
| |
The resolved path to the project directory is queried a lot when
using local elements, patch elements, and such.
Having the path cached and stored on the project gives nice speedups
when the underlying storage system is slow.
|
| |\
| |
| |
| |
| |
| |
| | |
Cleanup profiler and make is as a context manager
Closes #877 and #876
See merge request BuildStream/buildstream!1255
|