[Notes] [Git][BuildStream/buildstream][jennis/migrate_pull_push_commands] 33 commits: sandbox/sandbox.py: Use assertions for programming errors instead of BstErrors.

[James Ennis <gitlab mg gitlab com>]

Title: GitLab

James Ennis pushed to branch jennis/migrate_pull_push_commands at BuildStream / buildstream

Commits:

• 81f7a964
  by Tristan Van Berkom at 2019-01-08T16:55:58Z

  sandbox/sandbox.py: Use assertions for programming errors instead of BstErrors.
  
  When code is faulty, in a plugin or in the core, we should
  get a stack trace and a BUG message.
  

• edcc43ed
  by Tristan Van Berkom at 2019-01-08T19:19:10Z

  Merge branch 'tristan/sandbox-programming-error' into 'master'
  
  sandbox/sandbox.py: Use assertions for programming errors instead of BstErrors.
  
  See merge request BuildStream/buildstream!1046

• 17c5ca2d
  by Valentin David at 2019-01-09T10:45:23Z

  Use relative path to project directory for remote execution certificates/keys
  

• 605f8d11
  by Valentin David at 2019-01-09T10:45:23Z

  Add support for user remote execution configuration
  
  Fixes #631.
  

• 65ea03ab
  by Valentin David at 2019-01-09T10:45:23Z

  Add support for https channel to remote execution and actions servers
  
  Fixes #780.
  

• ba05c4fa
  by Valentin David at 2019-01-09T12:58:59Z

  Merge branch 'valentindavid/remote_execution_configuration' into 'master'
  
  Remote execution configuration: HTTPS and user configuration
  
  Closes #780 and #631
  
  See merge request BuildStream/buildstream!1030

• eb0dbcfc
  by Valentin David at 2019-01-09T13:56:58Z

  buildstream/plugins/elements/script.py: Mark script as BST_VIRTUAL_DIRECTORY
  
  ScriptElement does not use Sandbox.get_directory. It works using it
  with remote execution.
  
  Fixes #850
  

• f7681925
  by Valentin David at 2019-01-09T14:44:22Z

  Merge branch 'valentindavid/script_virtual_directory' into 'master'
  
  buildstream/plugins/elements/script.py: Mark script as BST_VIRTUAL_DIRECTORY
  
  Closes #850
  
  See merge request BuildStream/buildstream!1047

• f87d1c93
  by Jim MacArthur at 2019-01-09T15:01:18Z

  sandboxremote.py: Rename 'instance_name' option to 'instance-name'
  
  It was 'instance-name' in the documentation.
  

• ed8bc5bc
  by Jim MacArthur at 2019-01-09T15:30:28Z

  Merge branch 'jmac/rename_instance_name' into 'master'
  
  Rename 'instance_name' option to 'instance-name'
  
  See merge request BuildStream/buildstream!1048

• f29a0995
  by Tristan Van Berkom at 2019-01-09T19:51:14Z

  Support running test environments in parallel with `detox`
  
  This patch namespaces the test temp directory and the output
  coverage report file with the name of the environment under test,
  such that separately run tests do not access the same files.
  
  When running tests without tox, directly through setup.py,
  then the tmp directory will still be `./tmp`.
  
    * .gitignore: Added new .coverage-reports/ directory
  
    * .gitlab-ci.yml: Rely on tox to combine and report coverage, only
      tell tox about the COVERAGE_PREFIX so that results can be namespaced
      by CI job name.
  
      This change also publishes the sources and final combined `.coverage`
      file in an output gitlab artifact for inspection, and lists some missing
      dependencies to the `coverage` job.
  
    * tox.ini: Add comments and refactor main [testenv] section so that
      other environments dont inherit too much unrelated cruft.
  
      Generate the coverate reports in the respective {envtmpdir} so that
      all per-process coverage files are prefixed with a full path, ensuring
      that concurrent runs don't mix reports and addressing concerns
      raised in #844.
  
      Also implemented new `tox -e coverage` environment to combine
      any found coverage and print a report.
  
    * .coveragerc: Omit .tox/ directory from coverage stats
  
  Fixes issue #844
  

• d364ad02
  by Tristan Van Berkom at 2019-01-09T19:52:22Z

  CONTRIBUTING.rst: Updated to mention new `coverage` tox environment.
  
  Also point out that it is possible to run test environments in
  parallel using the `detox` tool.
  

• 1e352434
  by Tristan Van Berkom at 2019-01-09T20:36:06Z

  Merge branch 'tristan/detox-tests' into 'master'
  
  Allow using detox, and fix/refactor collection of coverage reports
  
  Closes #844
  
  See merge request BuildStream/buildstream!1051

• 116c1070
  by Tristan Van Berkom at 2019-01-09T20:39:15Z

  .coveragerc: Omit some things which are irrelevant to cover.
  
    * Omit versioneer's _version.py
  
    * Omit our __main__.py which is used only internally for generating
      documentation
  

• c8f3616d
  by Tristan Van Berkom at 2019-01-09T21:12:29Z

  Merge branch 'tristan/refine-coverage' into 'master'
  
  .coveragerc: Omit some things which are irrelevant to cover.
  
  See merge request BuildStream/buildstream!1052

• c87bb592
  by Jürg Billeter at 2019-01-10T09:43:37Z

  _platform/platform.py: Add canonicalize_arch() method
  

• 26e33346
  by Jürg Billeter at 2019-01-10T12:50:15Z

  _options/optionarch.py: Accept architecture aliases
  
  Accept common architecture aliases for arch options instead of only
  accepting the canonicalized, OS-independent architecture name. This
  restores compatibility with existing projects and may simplify option
  handling for projects that only target a single OS (and thus, do not
  need OS-independent architecture names).
  

• 06deb4c4
  by Jürg Billeter at 2019-01-10T12:50:15Z

  element.py: Accept architecture aliases for sandbox config
  
  Accept common architecture aliases for the sandbox config for
  consistency with arch options.
  

• 0c2a66b3
  by Jürg Billeter at 2019-01-10T13:06:02Z

  tests/format/optionarch.py: Add tests for architecture aliases
  

• f86bc760
  by Jürg Billeter at 2019-01-10T13:34:56Z

  Merge branch 'juerg/arch' into 'master'
  
  Accept architecture aliases
  
  See merge request BuildStream/buildstream!1034

• 347eb34e
  by Angelos Evripiotis at 2019-01-10T14:38:23Z

  contributing: fix 'oprtation' and some other typos
  

• 6bc27bc1
  by Angelos Evripiotis at 2019-01-10T15:07:46Z

  Merge branch 'aevri/contrib-typos' into 'master'
  
  contributing: fix 'oprtation' and some other typos
  
  See merge request BuildStream/buildstream!1055

• 901f90ba
  by James Ennis at 2019-01-15T15:22:50Z

  _context.py: Doc fix: get_toplevel_project() returns object not list
  

• 5ccf9fdd
  by James Ennis at 2019-01-15T15:22:51Z

  _context.py: Add documentation to get_workspaces() command
  

• ef811882
  by James Ennis at 2019-01-15T15:22:51Z

  Move push and pull to the new artifact subcommand group
  
  This commit also ensures that if we try to use the 'old' commands,
  BuildStream will fail and instruct the user to use the new command.
  

• 5cd26627
  by Richard Maw at 2019-01-15T15:22:51Z

  cli: Split classify_artifacts helper up
  
  Classifying artifact refs is useful for glob expansion
  even in cases where an element target name would be inappropriate.
  

• 0ba54bc4
  by James Ennis at 2019-01-15T15:22:51Z

  artifactcache.py: Add new method ArtifactCache.create_remote()
  
  This allows for the creation of a CASRemote object using the
  ArtifactCache API
  

• 9ec8e00c
  by James Ennis at 2019-01-15T15:24:05Z

  artifactcache.py: Allow ArtifactCache to handle pulling/pushing refs
  
  With the addition of the artifact subcommands, which are able to
  process artifact refs, we should not have to rely on CASCache
  internals to handle artifacts. This commit ensures that the
  pulling/pushing of elements (and their cache keys) is handled
  by ArtifactCache.
  

• 26180a20
  by James Ennis at 2019-01-15T15:24:05Z

  artifactcache.py: API Cleanup - Add contains_ref() method
  
  This ensures that we can determine whether an artifact is locally cached
  (as well as an element) using the ArtifactCache API.
  

• 6dad79de
  by James Ennis at 2019-01-15T16:19:12Z

  WIP: Ensure push/pull can deal with artifact refs
  

• 87ebe430
  by James Ennis at 2019-01-15T16:42:33Z

  cli.py: Handle the pulling of an artifact's buildtree
  

• 037278b4
  by James Ennis at 2019-01-15T16:43:31Z

  cli.py: Handle the --remote option for pulling artifacts
  

• e2509897
  by James Ennis at 2019-01-15T16:48:19Z

  WIP: cli.py: Spawn pull jobs in another process to isolate gRPC calls
  

30 changed files:

• .coveragerc
• .gitignore
• .gitlab-ci.yml
• CONTRIBUTING.rst
• buildstream/_artifactcache/artifactcache.py
• buildstream/_artifactcache/cascache.py
• buildstream/_context.py
• buildstream/_frontend/cli.py
• buildstream/_options/optionarch.py
• buildstream/_platform/platform.py
• buildstream/_project.py
• buildstream/element.py
• buildstream/plugins/elements/script.py
• buildstream/sandbox/_sandboxremote.py
• buildstream/sandbox/sandbox.py
• doc/source/format_project.rst
• doc/source/using_config.rst
• tests/artifactcache/config.py
• tests/artifactcache/junctions.py
• tests/completions/completions.py
• + tests/format/option-arch-alias/element.bst
• + tests/format/option-arch-alias/project.conf
• + tests/format/option-arch-unknown/element.bst
• + tests/format/option-arch-unknown/project.conf
• tests/format/optionarch.py
• tests/frontend/help.py
• tests/frontend/pull.py
• tests/frontend/push.py
• tests/frontend/workspace.py
• tests/integration/build-tree.py

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.