summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* source.py: don't let StopIteration propagate to silence() contextmanagerbschubert/fix-silence-stopiteration-pep-0479Benjamin Schubert2018-11-091-20/+22
| | | | | | As per PEP 0479 (https://www.python.org/dev/peps/pep-0479/), StopIteration thrown in context managers are not valid starting from Python 3.7.
* Merge branch 'jmac/cas_to_cas_v2' into 'master'Jim MacArthur2018-11-092-94/+652
|\ | | | | | | | | | | | | Direct CAS-to-CAS import Closes #574 See merge request BuildStream/buildstream!911
| * virtual_directory_import.py: New testjmac/cas_to_cas_v2Jim MacArthur2018-11-091-0/+271
| |
| * _casbaseddirectory: Fix order of list_modified_pathsJim MacArthur2018-11-091-9/+26
| | | | | | | | This will now return paths in the same order as list_relative_paths.
| * _casbaseddirectory.py: Enable direct CAS-to-CAS import.Jim MacArthur2018-11-091-85/+355
|/ | | | Fixes issue #574.
* Merge branch 'danielsilverstone-ct/plugin-destroy-debug-to-stderr' into 'master'Valentin David2018-11-091-1/+2
|\ | | | | | | | | plugin.py: Redirect DEBUG from `__del__` to `sys.stderr` See merge request BuildStream/buildstream!944
| * plugin.py: Redirect DEBUG from `__del__` to `sys.stderr`danielsilverstone-ct/plugin-destroy-debug-to-stderrDaniel Silverstone2018-11-091-1/+2
|/ | | | | | | | | Previously, debug messages from `Plugin.__del__()` went to stdout which was a problem for the test suite in extremely rare and hard to predict circumstances. This corrects that by sending the message to stderr where all the rest of the messages tend to go. Signed-off-by: Daniel Silverstone <daniel.silverstone@codethink.co.uk>
* Merge branch 'jennis/quota_declaration_fix' into 'master'Javier Jardón2018-11-083-7/+53
|\ | | | | | | | | | | | | Add local cache expiry documentation and fix misleading error message when specifying a percentage cache quota Closes #700 See merge request BuildStream/buildstream!939
| * artifactcache.py: Fix misleading error message when using % cache quotajennis/quota_declaration_fixJames Ennis2018-11-082-7/+15
| | | | | | | | | | Due to the changed Exception message, this patch also changes the test_parse_size_over_1024T test in misc.py
| * using_config.rst: Add documentation to showing how to impose quotas on the ↵James Ennis2018-11-081-0/+38
|/ | | | | | local cache This patch partially resolves #700
* Merge branch 'aevri/contributing_gitlab' into 'master'Angelos Evripiotis2018-11-081-1/+23
|\ | | | | | | | | Add more to GitLab-relevant parts of contributing See merge request BuildStream/buildstream!935
| * contributing: emphasise the 'why' in commitsAngelos Evripiotis2018-11-081-0/+10
| | | | | | | | | | | | | | | | | | | | | | Explain why it's useful to mention the decisions made for a change and provide links for background, e.g. issue numbers. My general experience of git histories is that folks don't do enough explaining, so it's great we have a contributing section for it. I've found it's easier for folks to stick to rules when they know the practical reasons for them, so provide some.
| * contributing: non-WIP should always be landableAngelos Evripiotis2018-11-081-0/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | I was recently surprised that we don't prefer addressing review comments in 'fixup!' commits. Coming from GitHub, I've found that fixup commits make it easier for reviewers to see what has changed since their last review. The idea is to use '--autosquash' before landing to clean up the history again. It's a pleasant surprise that it's easy to keep track of what changed in merge-requests between pushes, so we can always keep the history clean. Document this, so folks like me can see the light sooner.
| * contributing: WIP again to be kind to reviewersAngelos Evripiotis2018-11-081-1/+7
|/ | | | | | | | As someone coming from GitHub to GitLab, I was pleasantly surprised by the 'filter by WIP status' option. To make sure we get the most out of it, add a guideline to keep the filter clean, to reduce reviewer burden.
* Merge branch 'valentindavid/fix-script-install-root' into 'master'Valentin David2018-11-086-2/+88
|\ | | | | | | | | | | | | Fix bug with root mounted as non-artifact in script plugin. Closes #756 and #749 See merge request BuildStream/buildstream!941
| * Fix bug with root mounted as non-artifact in script plugin.Valentin David2018-11-086-2/+88
|/ | | | The issue was introduced by 6ccfab0b1b25990e406446d5cbe5aee83a5e158a.
* Merge branch 'bschubert/tests-no-chroot-linux' into 'master'Benjamin Schubert2018-11-088-49/+134
|\ | | | | | | | | | | | | test behavior when there is no sandbox on Linux Closes #696 and #736 See merge request BuildStream/buildstream!921
| * Test that helpful messages are raised when missing dependenciesBenjamin Schubert2018-11-085-1/+97
| | | | | | | | | | This adds a `reason` to the SandboxEror thrown in sandboxdummy to be able to understand where the error comes from
| * conftest.py: Ensure platform is not maintained between testsBenjamin Schubert2018-11-081-0/+7
| | | | | | | | | | | | | | | | | | | | | | | | This removes the `_instance` on the platform object that we use for caching and not recreating the object everytime at the start of every test. This is to ensure our tests share the least amount of state. The performance penalty is from 5 to 10% accross the whole test suite. The readings were done 5 times for each before and after the change and on the same computer.
| * Move bwrap checks in platform/linux.pyBenjamin Schubert2018-11-082-48/+30
|/ | | | | | | | Remove the bwraps checks from _site.py and put them in platform.linux which is the only place where they are run. This allows the removal of a double level of caching, making reasoning about tests easier
* Merge branch 'valentindavid/fix-recursive-strip-debug' into 'master'Valentin David2018-11-084-3/+8
|\ | | | | | | | | | | | | Fix infinite recursion in default strip debug command Closes #645 See merge request BuildStream/buildstream!918
| * Fix infinite recursion in default strip debug commandValentin David2018-11-084-3/+8
|/ | | | Fixes #645.
* Merge branch 'valentindavid/script-artifact-corruption' into 'master'richardmaw-codethink2018-11-076-7/+70
|\ | | | | | | | | | | | | Fix cache corruption by scripts when layout and integration commands are used Closes #749 See merge request BuildStream/buildstream!934
| * Fix cache corruption by scripts when layout and integration commands are usedValentin David2018-11-076-7/+70
|/ | | | | | | | Root directory was marked as a non-artifact mount, so not using SafeHardLink. However integration commands executed with write access to the root directory. Fixes #749
* Merge branch 'jmac/make_writable_combination' into 'master'Jim MacArthur2018-11-071-10/+3
|\ | | | | | | | | | | | | Use _force_rmtree instead of custom code. Closes #735 See merge request BuildStream/buildstream!927
| * element.py: Use _force_rmtree instead of custom code.jmac/make_writable_combinationJim MacArthur2018-11-071-10/+3
|/ | | | | There are arguments for and against using onerror - see issue #735 for details.
* Merge branch 'jennis/docs_patch' into 'master'richardmaw-codethink2018-11-073-67/+81
|\ | | | | | | | | Update our instructions/examples regarding how we declare remote caches See merge request BuildStream/buildstream!932
| * using_configuring_artifact_server.rst: Remove examples and redirect to user ↵James Ennis2018-11-071-47/+7
| | | | | | | | config/project config
| * format_project.rst: Update instructions regarding how we declare remote ↵James Ennis2018-11-071-5/+20
| | | | | | | | caches in project.conf
| * using_config.rst: Update our instructions for how we declare caches in user ↵James Ennis2018-11-072-15/+54
|/ | | | config
* Merge branch 'jmac/remote-sandbox-errors' into 'master'Jim MacArthur2018-11-061-4/+1
|\ | | | | | | | | | | | | _sandboxremote.py: Use the standard SandboxError Closes #746 See merge request BuildStream/buildstream!931
| * _sandboxremote.py: Use the standard SandboxError.jmac/remote-sandbox-errorsJim MacArthur2018-11-061-4/+1
|/ | | | | Replaces the custom one which was erroneously added during development. Fixes #746.
* Merge branch 'Qinusty/unit-test-utils' into 'master'richardmaw-codethink2018-11-063-3/+78
|\ | | | | | | | | Fix issue with _pretty_size with large numbers of bytes See merge request BuildStream/buildstream!799
| * Add regression test for _pretty_sizeJosh Smith2018-11-062-0/+74
| |
| * utils: Fix _pretty_size() for sizes > 1024TJosh Smith2018-11-061-3/+4
|/
* Merge branch 'juerg/cas' into 'master'Jürg Billeter2018-11-0514-429/+572
|\ | | | | | | | | | | | | Split up artifact cache and CAS cache Closes #659 See merge request BuildStream/buildstream!922
| * Split up artifact cache and CAS cacheJürg Billeter2018-11-059-403/+550
| | | | | | | | | | | | | | | | This changes CASCache from a subclass to a delegate object of ArtifactCache. As the lower layer, CASCache no longer deals with elements or projects. Fixes #659.
| * element.py: Do not include type name of artifact cache in cache keyJürg Billeter2018-11-051-1/+1
| | | | | | | | | | | | | | The artifact cache backend does not affect build outputs and we anyway no longer have pluggable artifact cache backends. This hardcodes CASCache instead of removing the entry completely to avoid cache key changes.
| * _project.py: Remove unused artifacts parameter from load_elements()Jürg Billeter2018-11-054-9/+8
| |
| * element.py: Remove unused artifacts parameter from _new_from_meta()Jürg Billeter2018-11-053-6/+5
| |
| * _project.py: Remove unused artifacts parameter from create_element()Jürg Billeter2018-11-052-3/+2
| |
| * _elementfactory.py: Remove unused artifacts parameter from create()Jürg Billeter2018-11-052-4/+3
| |
| * element.py: Remove artifacts parameter from constructorJürg Billeter2018-11-052-3/+3
|/ | | | Get the artifact cache from the context instead.
* Merge branch 'richardmaw/pytest-dep-update' into 'master'richardmaw-codethink2018-11-051-1/+1
|\ | | | | | | | | dev-requirements: Update pytest dependency to 3.8 See merge request BuildStream/buildstream!930
| * dev-requirements: Update pytest dependency to 3.8Richard Maw2018-11-051-1/+1
|/ | | | | "_frontend/app.py: Set correct element-path in interactive bst-init" added a dependency on the tmp_path test fixture, which was added by 3.8.
* Merge branch 'valentindavid/limit_fetchers_in_overnight_build' into 'master'Javier Jardón2018-11-051-0/+6
|\ | | | | | | | | | | | | Use less fetchers in overnight build to work-around limitations from servers. Closes #742 See merge request BuildStream/buildstream!929
| * Use less fetchers in overnight build to work-around limitations from servers.Valentin David2018-11-051-0/+6
|/ | | | | | Issue happens with sourceware.org server. Fixes #742
* Merge branch 'juerg/remote-execution' into 'master'Jürg Billeter2018-11-051-0/+1
|\ | | | | | | | | element.py: Pass bare_directory also to SandboxRemote() See merge request BuildStream/buildstream!928
| * element.py: Pass bare_directory also to SandboxRemote()juerg/remote-executionJürg Billeter2018-11-031-0/+1
|/ | | | Fixes: 825963b5 ("element: Make "--sysroot" take a bare directory")
* Merge branch 'chiaratolentino/fix-pip-source-regex' into 'master'Chandan Singh2018-11-029-32/+203
|\ | | | | | | | | plugins/sources/pip.py: Accomodate characters '-','.','_' for packages See merge request BuildStream/buildstream!914