Skip to content

ci-fairy: make boto3 optional (minio/s3 features disabled)

Eric Engestrom requested to merge eric/ci-templates:boto3-optional into master

With boto3 installed:

$ python tools/ci_fairy.py --help
Usage: ci_fairy.py [OPTIONS] COMMAND [ARGS]...

Options:
  -v, --verbose      increase verbosity
  --gitlab-url TEXT  GitLab URL with transport protocol, e.g.
                     https://gitlab.freedesktop.org
  --authfile TEXT    Path to a file containing the gitlab auth token string
  --help             Show this message and exit.

Commands:
  check-commits        Check a commit range for some properties.
  check-merge-request  Checks the given merge request in the project for...
  delete-image         Delete images from the container registry.
  generate-template    Generate a file based on the given Jinja2 template...
  lint                 Run the .gitlab-ci.yml through the gitlab...
  minio
  s3cp                 basically a wrapper around (when dst is an s3...
  wait-for-pipeline    Waits for a pipeline to complete, printing the...

Without boto3:

$ python tools/ci_fairy.py --help
Usage: ci_fairy.py [OPTIONS] COMMAND [ARGS]...

Options:
  -v, --verbose      increase verbosity
  --gitlab-url TEXT  GitLab URL with transport protocol, e.g.
                     https://gitlab.freedesktop.org
  --authfile TEXT    Path to a file containing the gitlab auth token string
  --help             Show this message and exit.

Commands:
  check-commits        Check a commit range for some properties.
  check-merge-request  Checks the given merge request in the project for...
  delete-image         Delete images from the container registry.
  generate-template    Generate a file based on the given Jinja2 template...
  lint                 Run the .gitlab-ci.yml through the gitlab...
  wait-for-pipeline    Waits for a pipeline to complete, printing the...

Closes: #25 (closed)

Note for reviewers: I recommend using the "hide whitespace changes" option in the web ui or --ignore-all-space in git show, since literally 98% (319/327) of lines changes are just indentation changes 🙃

Merge request reports