summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* tests/testutils/artifactshare.py: Add SIGTERM handler to subprocessjuerg/artifactshareJürg Billeter2019-09-161-0/+6
| | | | | | | | 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.
* Merge branch 'jennis/frontend_documentation' into 'master'bst-marge-bot2019-09-163-0/+99
|\ | | | | | | | | Add UI section in CONTRIBUTING See merge request BuildStream/buildstream!1584
| * CONTRIBUTING.rst: Add UI sectionJames Ennis2019-09-162-0/+79
| | | | | | | | | | | | Now that the frontend has been mostly reworked/standardized, this patch attempts to put our some guidelines/information in around UI contributions.
| * using_commands.rst: Add internal X-referencesJames Ennis2019-09-161-0/+6
| | | | | | | | | | This patch adds internal cross references for the artifact and source commands.
| * using_commands.rst: Add show and list-contentsJames Ennis2019-09-161-0/+14
|/ | | | | This patch ensures that we document the recently introduced artifact subcommands "show" and "list-contents".
* Merge branch 'coldtom/filter-element-improvements' into 'master'bst-marge-bot2019-09-1613-1/+158
|\ | | | | | | | | | | | | 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
| * tests/integration: Add test for filter elementcoldtom/filter-element-improvementsThomas Coldrick2019-09-164-4/+73
| | | | | | | | | | Adds an integration test to check the `pass-integration` option for filter elements works.
| * element.py: Abstract not producing artifactsThomas Coldrick2019-09-164-6/+16
| | | | | | | | | | | | | | | | 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.
| * filter.py: Fail if parent element is a stackThomas Coldrick2019-09-164-0/+22
| | | | | | | | | | | | | | | | | | | | | | 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
| * filter.py: Allow passing integration commandsThomas Coldrick2019-09-166-1/+57
|/ | | | | | | | | | | | | 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
* Merge branch 'jennis/split_contributing' into 'master'bst-marge-bot2019-09-1311-1879/+1915
|\ | | | | | | | | | | | | Split up CONTRIBUTING Closes #1116 See merge request BuildStream/buildstream!1604
| * coding_guidelines.rst: Inline comments require 2 whitespacesjennis/split_contributingJames Ennis2019-09-131-1/+1
| |
| * coding_guidelines.rst: Fix typo in frobiliciouslyJames Ennis2019-09-131-2/+2
| |
| * grpc_protocol.rst: Rename title to be more descriptiveJames Ennis2019-09-131-2/+2
| |
| * writing_documentation.rst: Rename title to be more descriptiveJames Ennis2019-09-131-2/+2
| |
| * using_the_testsuite.rst: Rename title to be more descriptiveJames Ennis2019-09-131-2/+2
| |
| * CONTRIBUTING.rst: Split up CONTRIBUTING into smaller filesJames Ennis2019-09-1311-1879/+1915
|/ | | | | | | | 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
* Merge branch 'jennis/load_deps_consistently' into 'master'bst-marge-bot2019-09-132-13/+10
|\ | | | | | | | | Load deps in checkout like we do everywhere else See merge request BuildStream/buildstream!1599
| * _stream.py: Load the appropriate PipelineSelection in checkoutJames Ennis2019-09-132-13/+10
|/ | | | | | | 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.
* Merge branch 'jennis/add_deps_all_to_checkout' into 'master'bst-marge-bot2019-09-132-5/+5
|\ | | | | | | | | Support `--deps all` in `artifact checkout` See merge request BuildStream/buildstream!1598
| * cli.py: Allow checkout to handle --deps allJames Ennis2019-09-132-5/+5
|/ | | | | | | | 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.
* Merge branch 'chandan/fix-news' into 'master'bst-marge-bot2019-09-121-8/+8
|\ | | | | | | | | NEWS: Use past-tense, fix note about YAML cache See merge request BuildStream/buildstream!1602
| * NEWS: Try to use past tense consistentlyChandan Singh2019-09-121-5/+5
| | | | | | | | | | Since the majority of the NEWS entries seem to be written in the past tense, change the handful of entries that aren't.
| * NEWS: Move note about YAML cache next to note about YAML APIChandan Singh2019-09-121-3/+3
|/ | | | | | | | This was incorrectly copy-pasted in the wrong location. See https://gitlab.com/BuildStream/buildstream/commit/108a38edd86d9de3ef0ce78cb005041662ed279e#note_216141606 for related discussion.
* Merge branch 'jennis/update_source_checkout' into 'master'bst-marge-bot2019-09-125-49/+160
|\ | | | | | | | | Ensure `source checkout` is symmetric to `artifact checkout` See merge request BuildStream/buildstream!1590
| * NEWS: Amend source checkout NEWS entriesjennis/update_source_checkoutJames Ennis2019-09-121-3/+6
| | | | | | | | | | | | The current mentions of `source checkout` in the NEWS file have been updated so that they explain the recent changes to `bst source checkout`
| * _stream.py: Inform user of source checkout locationJames Ennis2019-09-121-0/+2
| |
| * source_checkout.py: Add --compression tests for source checkoutJames Ennis2019-09-121-0/+34
| |
| * cli.py: Add --compression option to source checkoutJames Ennis2019-09-122-8/+20
| | | | | | | | | | | | | | | | | | !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.
| * _stream.py: Handle tar compression mode in separate functionJames Ennis2019-09-121-7/+21
| | | | | | | | | | | | | | | | | | | | | | 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
| * source_checkout.py: Add test for --tar/--directory conflictJames Ennis2019-09-121-0/+17
| |
| * cli.py: Make source checkout more consistent with artifact checkoutJames Ennis2019-09-123-28/+42
| | | | | | | | | | | | | | | | | | | | | | | | 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
| * _stream.py: Include all args in doc comment of source_checkout()James Ennis2019-09-121-1/+4
| |
| * _stream.py: Add doc comment to _export_artifacts()James Ennis2019-09-121-0/+12
| | | | | | | | | | _export_artifact() is not a trivial function and thus deserves a doc comment
| * source_checkout.py: Remove commented out touch() and add commentJames Ennis2019-09-121-1/+1
| | | | | | | | | | | | 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.
| * cli.py: Fix typo in source checkout's doc stringJames Ennis2019-09-121-1/+1
|/
* Merge branch 'jjardon/distutils' into 'master'bst-marge-bot2019-09-126-102/+3
|\ | | | | | | | | Use distutils plugin from bst-plugins-experimental See merge request BuildStream/buildstream!1600
| * Use distutils plugin from bst-plugins-experimentalJavier Jardón2019-09-126-102/+3
|/
* Merge branch 'jjardon/BST_FORMAT_VERSION' into 'master'Javier Jardón2019-09-1213-12/+26
|\ | | | | | | | | | | | | Fail if we try to build a BST_FORMAT_VERSION we do not support anymore Closes #1136 See merge request BuildStream/buildstream!1603
| * Fail if we try to build a BST_FORMAT_VERSION we do not support anymorejjardon/BST_FORMAT_VERSIONJavier Jardón2019-09-1213-12/+26
|/ | | | | bst-1.x support BST_FORMAT_VERSION == 17, as that is not supported by master I think is ok to set BST_FORMAT_VERSION_MIN = 18
* Merge branch 'build-all-option' into 'master'bst-marge-bot2019-09-124-4/+34
|\ | | | | | | | | Add 'dependencies' option to 'build' user config See merge request BuildStream/buildstream!1469
| * tests/remoteexecution/partial.py: Test building all dependenciesbuild-all-optionRaoul Hidalgo Charman2019-09-121-2/+12
| |
| * _context.py: Add 'dependencies' option to 'build' user configRaoul Hidalgo Charman2019-09-123-2/+22
|/ | | | | This option sets the default value for the `--deps` option of `bst build`.
* Prepare NEWS for 1.3.1.dev1 releaseChandan Singh2019-09-121-132/+164
| | | | | | | | | | | Since the development branch of BuildStream hasn't had a release in a while, the number of entries in the NEWS file is rather large. So, to make it easier to digest, split it into topics like CLI, API, Plugins etc. Under each topic, I have attempted to sort entries by their severity. Despite being long, the list is quite likely still incomplete.
* Merge branch 'juerg/cache-usage' into 'master'bst-marge-bot2019-09-102-2/+36
|\ | | | | | | | | | | | | cascache.py: Fix cache usage monitor on Python older than 3.7 Closes #1131 See merge request BuildStream/buildstream!1597
| * tests/artifactcache/expiry.py: Add test for cache usage monitorjuerg/cache-usageJürg Billeter2019-09-101-0/+33
| |
| * cascache.py: Fix cache usage monitor on Python older than 3.7Jürg Billeter2019-09-101-2/+3
|/ | | | | | | Older Python versions don't support the typecode 'q', see https://bugs.python.org/issue19896 Fixes #1131.
* Merge branch 'traveltissues/none-childdata' into 'master'bst-marge-bot2019-09-101-3/+4
|\ | | | | | | | | Check result is not None in tracking queue done() See merge request BuildStream/buildstream!1595
| * Check result is not None in tracking queue done()Darius Makovsky2019-09-101-3/+4
|/ | | | | | The tracking queue is the only queue implementing `done()` which requires the result to be iterable. If there is no result from the job this will hit a TypeError.
* Merge branch 'tpollard/sandboxmessage' into 'master'bst-marge-bot2019-09-101-2/+2
|\ | | | | | | | | _message.py: Use bool for sandbox Message() parameter See merge request BuildStream/buildstream!1592
View Lorry Controller Status