summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* element: Remove __cached fieldrichardmaw/element-cache-state-simplifyRichard Maw2018-09-051-4/+1
| | | | | This can get out of sync with other two cache states, and we can do without it.
* Merge branch 'tpollard/483' into 'master'Tom Pollard2018-09-032-4/+109
|\ | | | | | | | | plugins/git.py: Warn if ref is not in given track See merge request BuildStream/buildstream!564
| * tests/sources/git.py: Add tests for REF_NOT_IN_TRACKtpollard/483Tom Pollard2018-09-031-0/+68
| | | | | | | | | | Add tests that cover assert_ref_in_track & the configurable CoreWarnings REF_NOT_IN_TRACK warnings token.
| * plugins/git.py: Warn if ref is not in given trackTom Pollard2018-09-031-4/+41
|/ | | | | | | | | | | Add a helper function assert_ref_in_track to git.py GitMirror() which is used when staging & initing the source workspace. It checks the element's ref exists in the track (branch/tag) if it has been specified, raising a warning if necessary. The warning makes use of the warning token 'REF_NOT_IN_TRACK' from the configurable CoreWarnings. If the element has been tracked with bst, it is assumed that the value of ref exists in the track as it was generated from it & as such is not asserted.
* Merge branch 'tristan/source-fetcher-changes' into 'master'Tristan Van Berkom2018-09-035-32/+149
|\ | | | | | | | | Source fetcher changes See merge request BuildStream/buildstream!772
| * source.py: Track marked URLs and assert they are marked during ↵tristan/source-fetcher-changesTristan Van Berkom2018-09-021-0/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Plugin.configure() This cannot test for unaliased URLs, as those can be discovered later on outside of user provided element configuration; at least we assert that if an alias was used, we have seen it at load time. This will cause a BUG to occur for a plugin which marks an aliased URL (or attempts to translate one) outside of `Plugin.configure()`, if that URL was not previously seen. This is a part of #620
| * source.py: Added `primary` argument to URL marking APIsTristan Van Berkom2018-09-023-19/+52
| | | | | | | | | | | | | | | | | | | | | | | | | | The Source must now mention whether the marked or translated URL is "primary" or not. Even when a Source may have multiple URLs, the auxilliary URLs are derived from the primary one, not only is this true for git, but it is mandated by our tracking API which assumes there is a primary URL. This adjusts the `git` source and the test `fetch_source.py` source to behave properly and advertize it's primary URL properly. This is a part of #620
| * plugin.py: Added _configure() and _get_configuring() private APIsTristan Van Berkom2018-09-023-2/+29
| | | | | | | | | | | | | | | | | | Keeps track of whether the plugin is currently being configured. Adjusted Element and Source classes to call _configure() in place of calling configure() directly. This is a part of #620
| * source.py: Documenting that marking download URLs is mandatoryTristan Van Berkom2018-09-021-6/+34
| | | | | | | | | | | | | | A download URL must be interpreted by the core at `Plugin.configure()` time, even if only employed later on. This is a part of #620
| * source.py: Fixing docs link formatting to be consistent.Tristan Van Berkom2018-09-021-8/+10
| |
| * source.py: Document Source.get_source_fetchers() to return an iterableTristan Van Berkom2018-09-021-2/+9
|/ | | | | | | | Also highlight the fact that the plugin can rely on the fetcher's fetch() method getting called before consuming the next item in the list, which is the magick behavior that the git plugin relies on. This is a part of #620
* Merge branch 'Qinusty/cache-size-directory' into 'master'Tristan Van Berkom2018-09-011-1/+1
|\ | | | | | | | | Move cache_size.pid.log files into a subdirectory of logs See merge request BuildStream/buildstream!769
| * scheduler.py: Move cache_size logs into folderQinusty/cache-size-directoryJosh Smith2018-08-311-1/+1
|/ | | | | This prevents the cache_size.pid.log files from cluttering the root log directory.
* Merge branch 'Qinusty/retries-should-fail' into 'master'Tristan Van Berkom2018-08-314-10/+6
|\ | | | | | | | | Retries log as failures See merge request BuildStream/buildstream!766
| * tests: Modify tests to ensure retry FAILUREJosh Smith2018-08-313-3/+3
| |
| * job.py: Modify retry messages to be FAILJosh Smith2018-08-311-7/+3
|/ | | | | | | | This adjusts the message handler for the child processes to no longer override the message type. This also removes the ability for unhandled non BstError exceptions to retry.
* _project.py: Adding FIXME comment to address #591 properlyTristan Van Berkom2018-08-311-1/+7
|
* Merge branch 'tpollard/591' into 'master'Tiago Gomes2018-08-301-0/+5
|\ | | | | | | | | buildstream/_project.py: Report if project.conf is missing name See merge request BuildStream/buildstream!680
| * buildstream/_project.py: Report if project.conf is missing nameTom Pollard2018-08-301-0/+5
|/ | | | | | | Explicitly check that project.conf contains a name. This resolves the issue of the provenance check from _yaml.py incorrectly reporting the offending file as the default_config_node projectconfig.yaml when no name key exists in the pre_config_node dict.
* Merge branch 'bzr_fix' into 'master'Phillip Smyth2018-08-301-5/+7
|\ | | | | | | | | Replacing string 'bzr' with value from host tools See merge request BuildStream/buildstream!764
| * Replacing string 'bzr' with value from host toolsbzr_fixknownexus2018-08-301-5/+7
|/
* Merge branch 'tristan/atomic-cache-size-file' into 'master'Tristan Van Berkom2018-08-301-1/+1
|\ | | | | | | | | _artifactcache/artifactcache.py: Write the cache_size file atomically See merge request BuildStream/buildstream!762
| * _artifactcache/artifactcache.py: Write the cache_size file atomicallytristan/atomic-cache-size-fileTristan Van Berkom2018-08-301-1/+1
|/ | | | | | | | This is causing issues while the size file is being read and written simultaneously. The proper fix will be to read/add/save the file atomically and that will require locking, but this fix is a good stop gap to existing crashes.
* Merge branch 'valentindavid/post_tracking_errors' into 'master'Tristan Van Berkom2018-08-302-12/+37
|\ | | | | | | | | | | | | Report processing errors from tracking Closes #533 See merge request BuildStream/buildstream!747
| * Report processing errors from trackingValentin David2018-08-302-12/+37
|/ | | | | | Failures to write files when tracking were not reported. Fixes #533.
* Merge branch 'tristan/source-mirroring-changes' into 'master'Tristan Van Berkom2018-08-302-17/+31
|\ | | | | | | | | Minor code changes revolving around source mirroring See merge request BuildStream/buildstream!758
| * plugins/source/git.py: Fix formatting of url in trackingTristan Van Berkom2018-08-301-1/+3
| | | | | | | | | | This was displaying the aliased URL which is pretty useless, use the translated URL for the timed activity.
| * source.py: Move Source.mark_download_url() to the public methods.Tristan Van Berkom2018-08-301-13/+13
| | | | | | | | | | | | This was sitting in the section for abstract methods, but this is most definitely not an abstract method to be implemented by Sources.
| * source.py: Stylistic changes in Source.__do_fetch()Tristan Van Berkom2018-08-301-3/+15
|/ | | | | | Added some comments to make the flow easier to follow, and removed an annoying 'success' variabled in favor of a for / else loop statement.
* Merge branch 'tristan/538-reenable-ostree-test' into 'master'Tristan Van Berkom2018-08-301-4/+0
|\ | | | | | | | | tests/frontend/mirror.py: Reenable test_mirror_fetch_upstream_absent[ostree] See merge request BuildStream/buildstream!755
| * tests/frontend/mirror.py: Reenable test_mirror_fetch_upstream_absent[ostree]tristan/538-reenable-ostree-testTristan Van Berkom2018-08-301-4/+0
|/ | | | | This test was skipped because of issue #538, but #538 was fixed and the test was still not reenabled.
* Merge branch 'tristan/reduce-filter-tests' into 'master'Tristan Van Berkom2018-08-301-15/+10
|\ | | | | | | | | tests/plugins/filter.py: Don't run redundant tests See merge request BuildStream/buildstream!753
| * tests/plugins/filter.py: Don't run redundant testsTristan Van Berkom2018-08-301-15/+10
|/ | | | | | | | | There is no reason that the filter element codepaths can behave differently depending on the Source implementation used in the test, as the Source implementation does not have any filter specific virtual methods. Removing the redundant tests and just performing these tests with the git source.
* Merge branch 'Qinusty/600-recursive-variables' into 'master'Qinusty2018-08-298-11/+77
|\ | | | | | | | | Add cyclic check within variable resolution See merge request BuildStream/buildstream!712
| * Add tests for cyclic variables checkJosh Smith2018-08-296-7/+46
| | | | | | | | | | Note: This modifies the docker containers used for testing to supply the pytest-timeout package.
| * Add cyclic check within variable resolutionJosh Smith2018-08-292-4/+31
|/ | | | | This aims to address #600, this will raise an exception when a resolved variable contains a reference to the variable.
* Merge branch 'bschubert/log-missed-cache' into 'master'Qinusty2018-08-292-0/+26
|\ | | | | | | | | Log not-found objects in the cache as SKIPPED See merge request BuildStream/buildstream!729
| * Add tests that not-found objects in cache are shown as SKIPPEDbschubert/log-missed-cacheBenjamin Schubert2018-08-291-0/+19
| |
| * Log not-found objects in the cache as SKIPPEDBenjamin Schubert2018-08-291-0/+7
|/
* Merge branch 'valentindavid/roundtripping_only_when_modified' into 'master'Tristan Van Berkom2018-08-291-2/+3
|\ | | | | | | | | Disable round-tripping when element is not modified See merge request BuildStream/buildstream!748
| * Disable round-tripping when element is not modifiedValentin David2018-08-291-2/+3
|/
* Merge branch 'Qinusty/597-non-alias-url-fix' into 'master'Tristan Van Berkom2018-08-292-9/+13
|\ | | | | | | | | | | | | Fix ValueError traceback for URL without alias or URI scheme Closes #597 See merge request BuildStream/buildstream!700
| * Prevent ValueError on URLs without an aliasJosh Smith2018-08-292-9/+13
|/ | | | Fixes #597
* Merge branch 'chandan/setup.py-metadata' into 'master'Tristan Van Berkom2018-08-291-1/+25
|\ | | | | | | | | setup.py: Add more metadata useful for PyPI See merge request BuildStream/buildstream!718
| * setup.py: Add more metadata useful for PyPIChandan Singh2018-08-291-1/+25
|/ | | | | | | | | | | | | | | | | | | - setup.py: List useful links using `project_urls` - setup.py: Specify minumum python vresion using `python_requires` `python_requires` is the new way of specifying requirements for python version, as documented @ https://packaging.python.org/guides/distributing-packages-using-setuptools/#python-requires. This will also make this requirement appear nicely on the PyPI project page. - setup.py: Add project classifiers PyPI will use these classifiers to categorize projects while searching or browsing. Full list of classifiers can be found at https://pypi.org/classifiers.
* Merge branch 'tristan/docs-version-badge' into 'master'Tristan Van Berkom2018-08-299-8/+252
|\ | | | | | | | | Release badges See merge request BuildStream/buildstream!742
| * doc: Adding the release badges to the install page and the semantic ↵Tristan Van Berkom2018-08-294-4/+65
| | | | | | | | | | | | versioning page This is a part of #528
| * README.rst: Adding the release and snapshot badges to the readmeTristan Van Berkom2018-08-291-0/+7
| | | | | | | | This is a part of #528
| * doc: Adding badges.py release badge generatorTristan Van Berkom2018-08-294-4/+180
|/ | | | | | | | This adds a step to the docs generation Makefile to generate release.svg and snapshot.svg badges, modelled after the gitlab badges. This also adds the generated badges directory in docs to .gitignore
* Merge branch 'jjardon/pycodestyle_fixes' into 'master'Tristan Van Berkom2018-08-298-10/+15
|\ | | | | | | | | Some pycodestyle (PEP8) fixes See merge request BuildStream/buildstream!746
View Lorry Controller Status