[Notes] [Git][BuildStream/buildstream][328-support-for-downloading-sources-from-mirrors] 26 commits: Add remote source plugin

[Jonathan Maw <gitlab mg gitlab com>]

Title: GitLab

Jonathan Maw pushed to branch 328-support-for-downloading-sources-from-mirrors at BuildStream / buildstream

Commits:

• c049d021
  by Ed Baunton at 2018-07-25T19:43:51Z

  Add remote source plugin
  
  Add a plugin that supports downloading files verbatim from a source with
  an optional overridable filename and destination directory. Bumps bst
  format version to 10.
  
  Fixes #163
  

• bd1196ef
  by Ed Baunton at 2018-07-26T13:22:52Z

  Provide explicit reason for SourceError exception on path
  
  When the user provides a path for the filename parameter, provide a reason
  

• 9a46c16f
  by Ed Baunton at 2018-07-26T14:06:52Z

  Merge branch 'edbaunton/remote-source' into 'master'
  
  Add remote source plugin
  
  Closes #163
  
  See merge request BuildStream/buildstream!541

• 7ceacfea
  by William Salmon at 2018-07-26T14:32:49Z

  Documentation typos and fixes
  
  Daniel went through the Doc's etc and made a list of suggestions.
  These are the result of the most easily patched suggestions and
  one or two thing that I noticed as well.
  

• 8c9fd9e4
  by Javier Jardón at 2018-07-26T15:32:16Z

  Merge branch 'willsalmon/documentation_form_notes' into 'master'
  
  Documentation typos and fixes
  
  See merge request BuildStream/buildstream!569

• 9ebd7fe1
  by Josh Smith at 2018-07-27T04:41:26Z

  _exceptions.py: Modify BstError API to allow optional retry
  
  job.py: Changes to the logic surrounding retry attempts and child process return codes
  element.py, source.py: ElementError and SourceError also implement this change.
  
  These exceptions now have an optional parameter of temporary which defaults to false. This will potentially break
  backwards compatibility where exceptions were previously raised and a retry was intended.
  
  To trigger a retry, one must now raise their SourceError or ElementError with temporary=True.
  
  This aims to fix #397.
  

• ca224425
  by Josh Smith at 2018-07-27T04:41:26Z

  _downloadablefilesource.py: Update network errors to be temporary
  
  This follows the change in 67ecd97a05279a3b7570ad59f05bf0a5973ef04c.
  

• 0a7cfcdf
  by Josh Smith at 2018-07-27T04:41:26Z

  tests/sources: deb, tar and zip all test for retry behaviour
  
  This is following 5d6418a06a3a0262df5f6b9e6e1a36578515fd4a where
  DownloadableFileSource was modified to raise 'temporary'
  errors when relating to file downloading.
  

• 9b2bc78f
  by Josh Smith at 2018-07-27T04:41:26Z

  cascache.py: Update cache push/pull errors to be temporary
  
  Further work needs to be done for the current grpc exceptions which are reraised.
  

• 4986527f
  by Josh Smith at 2018-07-27T04:41:26Z

  plugin.py: Extend Plugin.call() API
  
  Plugin.call() now takes fail_temporarily as an optional parameter,
  when supplied it will cause subsequent failures to trigger temporary
  errors as opposed to permanent errors.
  
  This also extends Plugin.check_output() which makes use of Plugin.call()
  

• a9a1048e
  by Josh Smith at 2018-07-27T04:41:26Z

  git.py: Update git plugin to fail temporarily on clone failures
  
  The git plugin will now make use of the fail_temporarily parameter
  to Plugin.call(), allowing failures to trigger a retry.
  

• aa071cb6
  by Tristan Van Berkom at 2018-07-27T06:13:53Z

  Merge branch 'tristan/temporary-failures' into 'master'
  
  Temorary failures
  
  Closes #397
  
  See merge request BuildStream/buildstream!574

• 4d86e7c5
  by Tristan Van Berkom at 2018-07-27T06:43:48Z

  element.py, source.py, plugin.py: Adjusting API version since annotations.
  
  Since we backported the temporary/permanent failures patch for #397
  into the `bst-1.2` branch, we need to adjust the since versions in master
  down to 1.2.
  

• 110f6cb3
  by Jonathan Maw at 2018-07-27T09:19:28Z

  project: Parse and store mirrors
  

• 784a781a
  by Jonathan Maw at 2018-07-27T09:19:28Z

  Move _ALIAS_SEPARATOR into utils
  
  The separator is useful in source files other than _project.py
  

• cd55eb70
  by Jonathan Maw at 2018-07-27T09:27:48Z

  source: Store the url aliases or use an override
  
  This is part of a later plan to implement mirroring without forcing
  everyone to update their sources. We use the expected calls to
  Source.translate_url() when running Source.configure() to extract the
  aliases from the URL. Multiple aliases must be extracted because
  sources exist that may fetch from multiple aliases (for example, git
  submodules)
  
  Later, we want to substitute another URI where the alias normally reads
  from the project - We accomplish this by re-instantiating the Source
  with the alias overrides passed as an argument to the constructor.
  

• da49bc69
  by Jonathan Maw at 2018-07-27T09:27:48Z

  source: When fetching, try to fetch from mirrors first
  
  **KLUDGE WARNING**: This involves making the source store its "meta"
  object so that it's possible to create a copy of the source inside the
  fetch queue, instead of back when the pipeline was being loaded.
  
  This adds the SourceFetcher class, which is intended for sources that
  fetch from multiple URLs (e.g. the git source and its submodules)
  
  Fix when fetching
  

• 421f1973
  by Jonathan Maw at 2018-07-27T09:27:48Z

  Set default mirror via command-line or user config
  
  In user config (buildstream.conf), it is set with the "default-mirror"
  field.
  
  On the command-line, it is set with "--default-mirror"
  

• d37f3d17
  by Jonathan Maw at 2018-07-27T09:27:48Z

  Add mirrored source tracking
  

• cf1f6cce
  by Jonathan Maw at 2018-07-27T09:27:48Z

  bzr.py: Improve mirror support
  
  This fixes:
  * Bzr repositories pulling from the branch they were created with.
  * Bzr's _ensure_mirror() not actually checking that it successfully
    mirrored the ref.
  

• 0c7e62f5
  by Jonathan Maw at 2018-07-27T10:21:51Z

  git.py: Improve mirror support
  
  When implementing fetching from mirrors, I encountered some problems
  with the git source:
  
  1. The mirror URL was using translate_url()'s output, so if a different
     alias was used, then fetching from the mirror would go to a different
     directory, and be inaccessible.
  2. After fixing that, fetching was unable to pull from a URL other than
     the one used at repository creation, meaning it wouldn't actually
     pull from the mirror.
  

• 86dd7c70
  by Jonathan Maw at 2018-07-27T10:21:51Z

  testutils: Add a helper to copy a testutils repo
  
  This is helpful if you want to test what happens when you have one repo
  that has diverged from another. By copying the repo you're sure they
  start with shared history.
  
  This is especially useful when mirroring.
  

• ad1f034d
  by Jonathan Maw at 2018-07-27T10:21:51Z

  tests: Add mirrored fetching and tracking tests
  

• 687024f9
  by Jonathan Maw at 2018-07-27T10:21:51Z

  doc: Add documentation for source mirroring
  

• 22c1d6e6
  by Jonathan Maw at 2018-07-27T10:21:51Z

  NEWS: Add item for source mirroring
  

• eeca2c30
  by Jonathan Maw at 2018-07-27T10:21:51Z

  doc: Add tutorials for setting up git and tar mirrors
  

30 changed files:

• HACKING.rst
• NEWS
• buildstream/__init__.py
• buildstream/_artifactcache/cascache.py
• buildstream/_context.py
• buildstream/_exceptions.py
• buildstream/_frontend/app.py
• buildstream/_frontend/cli.py
• buildstream/_loader/loader.py
• buildstream/_project.py
• buildstream/_scheduler/jobs/job.py
• buildstream/_versions.py
• buildstream/element.py
• buildstream/plugin.py
• buildstream/plugins/sources/_downloadablefilesource.py
• buildstream/plugins/sources/bzr.py
• buildstream/plugins/sources/git.py
• + buildstream/plugins/sources/remote.py
• buildstream/source.py
• buildstream/utils.py
• doc/source/core_plugins.rst
• doc/source/examples/flatpak-autotools.rst
• + doc/source/examples/git-mirror.rst
• + doc/source/examples/tar-mirror.rst
• doc/source/format_declaring.rst
• doc/source/format_project.rst
• doc/source/install_docker.rst
• doc/source/install_linux_distro.rst
• doc/source/tutorial/autotools.rst
• doc/source/using_config.rst

The diff was not included because it is too large.

—
View it on GitLab.
You're receiving this email because of your account on gitlab.com. If you'd like to receive fewer emails, you can adjust your notification settings.