| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
| |
|
|
|
|
|
|
|
|
| |
All communication with CAS servers should be proxied through
buildbox-casd to allow connection sharing among job subprocesses.
It is no longer needed to query the server capabilities as buildbox-casd
already does that, so we can simply remove the code from BuildStream.
|
|
|
|
| |
Remote CAS servers should not be contacted directly.
|
| |
|
| |
|
|
|
|
|
| |
The push status check is implemented by BuildStream artifact and source
services. It's not part of the CAS protocol.
|
|
|
|
|
|
|
| |
It's sufficient to check the capabilities once per bst session. This
avoids the extra round trip in remote.init().
This also adds a check for allow_updates for push remotes.
|
|
|
|
|
|
|
| |
It's sufficient to check the capabilities once per bst session. This
avoids the extra round trip in remote.init().
This also adds a check for allow_updates for push remotes.
|
|
|
|
|
| |
There is no need to create a new stub for every request. This also
matches what SourceRemote does.
|
|\
| |
| |
| |
| | |
tests/testutils/artifactshare.py: Add SIGTERM handler to subprocess
See merge request BuildStream/buildstream!1606
|
|/
|
|
|
|
|
|
| |
pytest-cov 2.7 calls os._exit() in its SIGTERM handler, skipping
cleanup, unless another SIGTERM handler was already registered.
Add our own SIGTERM handler to the subprocess to ensure proper cleanup
such as terminating buildbox-casd.
|
|\
| |
| |
| |
| | |
Add UI section in CONTRIBUTING
See merge request BuildStream/buildstream!1584
|
| |
| |
| |
| |
| |
| | |
Now that the frontend has been mostly reworked/standardized,
this patch attempts to put our some guidelines/information in
around UI contributions.
|
| |
| |
| |
| |
| | |
This patch adds internal cross references for the artifact and
source commands.
|
|/
|
|
|
| |
This patch ensures that we document the recently introduced
artifact subcommands "show" and "list-contents".
|
|\
| |
| |
| |
| |
| |
| | |
plugins/elements/filter.py: Allow passing integration commands through, fail if dep is a stack element
Closes #1107 and #1104
See merge request BuildStream/buildstream!1593
|
| |
| |
| |
| |
| | |
Adds an integration test to check the `pass-integration` option for
filter elements works.
|
| |
| |
| |
| |
| |
| |
| |
| | |
Currently the stack element does not produce an artifact, and sometimes
has to be treated differently because of this. It is conceivable that
someone will write a plugin that must be treated similarly and doesn't
produce an artifact. As a result this commit abstracts this "not
producing an artifact" feature of elements.
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
As stack elements do not provide any artifacts, it is misleading to
allow them as dependencies for filter elements. This commit makes a
dependency on a stack element a failure with a descriptive error
message.
Additionally adds a test to make sure this works.
Addresses #1104
|
|/
|
|
|
|
|
|
|
|
|
|
|
| |
It is tedious to manually copy the integration commands of the parent
element into a filter element, so this allows it to be done
automatically. Here we modify FilterElement.integrate() to allow us to
pass through the parent's integration commands, iff an option is set.
Also adds a test for the new feature, but this is not as comprehensive
as would be idea, as getting to the integration commands which are run
is a little more difficult.
Addresses #1107
|
|\
| |
| |
| |
| |
| |
| | |
Split up CONTRIBUTING
Closes #1116
See merge request BuildStream/buildstream!1604
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
|/
|
|
|
|
|
|
| |
Our contributing has got way too big. This patch aims to
split it up into sensible files. These are found in
"Further information".
Closes #1116
|
|\
| |
| |
| |
| | |
Load deps in checkout like we do everywhere else
See merge request BuildStream/buildstream!1599
|
|/
|
|
|
|
|
| |
This patch ensures checkout behaves like the rest of our
commands which support --deps options. That is, we carry
the "deps" string through to the Stream and load the
corresponding PipelineSelection.
|
|\
| |
| |
| |
| | |
Support `--deps all` in `artifact checkout`
See merge request BuildStream/buildstream!1598
|
|/
|
|
|
|
|
|
| |
source checkout supports --deps all, so we should be
consistent.
Additionally, a user may want to have build deps in a
potential chroot so you could rebuild any component.
|
|\
| |
| |
| |
| | |
NEWS: Use past-tense, fix note about YAML cache
See merge request BuildStream/buildstream!1602
|
| |
| |
| |
| |
| | |
Since the majority of the NEWS entries seem to be written in the past
tense, change the handful of entries that aren't.
|
|/
|
|
|
|
|
|
| |
This was incorrectly copy-pasted in the wrong location.
See
https://gitlab.com/BuildStream/buildstream/commit/108a38edd86d9de3ef0ce78cb005041662ed279e#note_216141606
for related discussion.
|
|\
| |
| |
| |
| | |
Ensure `source checkout` is symmetric to `artifact checkout`
See merge request BuildStream/buildstream!1590
|
| |
| |
| |
| |
| |
| | |
The current mentions of `source checkout` in the NEWS file
have been updated so that they explain the recent changes to
`bst source checkout`
|
| | |
|
| | |
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
!1451 introduced the --compression option to bst artifact checkout.
This MR is a step towards making the commands more symmetric,
and introduces the --compression option to bst source checkout
Now we're compressing, we must explicitly close the tarball
once we're done writing to it.
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
This patch introduces a local _handle_compression() function,
which returns a string representing the "mode" of tarfile
compression required by the TarFile object.
In addition to this, _export_artifact() has been changed
so that it now uses this new function
See: https://docs.python.org/3/library/tarfile.html#tarfile.open
|
| | |
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
This MR changes the behaviour of source checkout so that it is more
like artifact checkout. That is, a --directory option can be provided,
rather than a mandatory LOCATION.
In addition to this, the --tar option is no longer a boolean flag,
rather it expects a tarfile name (just like artifact checkout)
The appropriate tests have also been altered so that they support
the new API
|
| | |
|
| |
| |
| |
| |
| | |
_export_artifact() is not a trivial function and thus
deserves a doc comment
|
| |
| |
| |
| |
| |
| | |
In "test_source_checkout_force()" we had a commented out line
of code. This has been removed and a more useful comment has been
added.
|
|/ |
|
|\
| |
| |
| |
| | |
Use distutils plugin from bst-plugins-experimental
See merge request BuildStream/buildstream!1600
|
|/ |
|
|\
| |
| |
| |
| |
| |
| | |
Fail if we try to build a BST_FORMAT_VERSION we do not support anymore
Closes #1136
See merge request BuildStream/buildstream!1603
|