Skip to content

Commit

Permalink
Document GitHub actions AWS integration
Browse files Browse the repository at this point in the history
  • Loading branch information
fajpunk committed Nov 19, 2024
1 parent ecc54f8 commit f6a6441
Show file tree
Hide file tree
Showing 3 changed files with 12 additions and 3 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/ci.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -57,9 +57,9 @@ jobs:
python-version: ${{ matrix.python }}
tox-envs: "py,typing"
env:
LTD_TEST_AWS_ID: ${{ secrets.LTD_TEST_AWS_ID }}
LTD_TEST_AWS_ID: ${{ vars.LTD_TEST_AWS_ID }}
LTD_TEST_AWS_SECRET: ${{ secrets.LTD_TEST_AWS_SECRET }}
LTD_TEST_BUCKET: ${{ secrets.LTD_TEST_BUCKET }}
LTD_TEST_BUCKET: ${{ vars.LTD_TEST_BUCKET }}

docs:

Expand Down
1 change: 0 additions & 1 deletion changelog.d/20241118_133228_danfuchs_DM_47580.md
Original file line number Diff line number Diff line change
@@ -1,4 +1,3 @@
### Bug fixes

- Change Slack channel references to refer to channels in the Rubin Slack instance.
- Get non-sensitive CI vars from GitHub variables instead of secrets
10 changes: 10 additions & 0 deletions docs/development.rst
Original file line number Diff line number Diff line change
Expand Up @@ -29,3 +29,13 @@ Releases are made by creating a Git tag with a semantic version and pushing to G
git push --tags
Travis CI creates the PyPI release itself and `setuptools_scm <https://github.com/pypa/setuptools_scm/>`_ ensures the PyPI version matches the Git tag.

GitHub CI
=========

The GitHub CI action needs AWS creds and an existing S3 bucket.
These creds and the bucket name are injected via repository-scoped Actions `secrets <https://github.com/lsst-sqre/ltd-conveyor/settings/secrets/actions>`_ and `variables <https://github.com/lsst-sqre/ltd-conveyor/settings/variables/actions>`_

These creds are attached to the `ltd-conveyor-tests IAM user <https://us-east-1.console.aws.amazon.com/iam/home?region=us-west-2#/users/details/ltd-conveyor-tests?section=permissions>`_.
This user has an attached `ltd-conveyor-tests policy <https://us-east-1.console.aws.amazon.com/iam/home?region=us-west-2#/policies/details/arn%3Aaws%3Aiam%3A%3A039289279626%3Apolicy%2Fltd-conveyor-tests?section=permissions>`_.
This policy grants access to the `lsst-the-docs-test S3 bucket <https://us-west-2.console.aws.amazon.com/s3/buckets/lsst-the-docs-test?region=us-west-2&bucketType=general&tab=objects>`_.

0 comments on commit f6a6441

Please sign in to comment.