Skip to content

Commit

Permalink
Update links to ingest inspector from daily reporter lambda (#1136)
Browse files Browse the repository at this point in the history
* Update link to ingest inspector in daily reporter

* add deployment action for daily reporter
  • Loading branch information
kenoir authored Aug 15, 2024
1 parent 34c2605 commit bbdc459
Show file tree
Hide file tree
Showing 3 changed files with 27 additions and 6 deletions.
26 changes: 26 additions & 0 deletions .github/workflows/deploy-daily-reporter.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,26 @@
name: Deploy Daily Reporter
on:
workflow_dispatch:
push:
branches:
- main

permissions:
id-token: write

jobs:
deploy_ingest_inspector:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
- uses: aws-actions/configure-aws-credentials@v4
with:
aws-region: eu-west-1
role-to-assume: ${{ secrets.DEPLOY_INGEST_INSPECTOR_GHA_ROLE_ARN }}
- name: Deploy Daily Reporter
run: |
pip3 install -r ./builds/requirements.txt
python3 ./builds/publish_lambda_zip.py monitoring/daily_reporter \
--bucket=wellcomecollection-storage-infra \
--key=lambdas/monitoring/daily_reporter.zip
./builds/deploy_lambda_zip.sh monitoring/daily_reporter daily_reporter
2 changes: 1 addition & 1 deletion monitoring/daily_reporter/src/slack.py
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ def _get_slack_message(label, ingests):
if ingests.get(status, []):
result += "\n" + status.title() + ":"
for i in ingests[status][:15]:
result += f"\n- <https://ingest-inspector.wellcomecollection.org/ingests/{i['id']}|`{i['id']}`> – {i['space']}/{i['externalIdentifier']}"
result += f"\n- <https://ingest-inspector.wellcomecollection.org/?ingestId={i['id']}|`{i['id']}`> – {i['space']}/{i['externalIdentifier']}"
if i["version"]:
result += "/" + i["version"]

Expand Down
5 changes: 0 additions & 5 deletions monitoring/ingest_inspector/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -22,8 +22,6 @@ The ingests data isn't sensitive, so we don't need authentication in front of it
It mostly contains user-facing messages from the storage service, or an identifier used by one of Wellcome's catalogue databases.
The user-facing messages can be seen in the publicly-available source code, and the identifiers don't encode any sensitive information.



## Features

The app has a number of features designed to make it as easy as possible for somebody to get the data they need.
Expand Down Expand Up @@ -128,8 +126,6 @@ The app has a number of features designed to make it as easy as possible for som

![A line "callback status:" pending highlighted in blue.](screenshots/pending_callback.png)



## Usage

You can use the app by visiting <https://ingest-inspector.wellcomecollection.org/>.
Expand Down Expand Up @@ -166,4 +162,3 @@ the `ingest_inspector/frontend` directory and run:
```shell
AWS_PROFILE=storage-developer sh deploy.sh
```

0 comments on commit bbdc459

Please sign in to comment.