Skip to content

Commit

Permalink
break out into new file per monitor
Browse files Browse the repository at this point in the history
  • Loading branch information
douglance committed Aug 16, 2024
1 parent bb3f859 commit 58439b7
Show file tree
Hide file tree
Showing 5 changed files with 21 additions and 5 deletions.
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
name: 6-Hour Monitoring
name: Assertion Monitoring

on:
push:
Expand All @@ -11,9 +11,8 @@ jobs:
strategy:
matrix:
chain: [core, orbit]
monitor: [assertion, batch-poster]
uses: ./.github/workflows/monitoring.yml
with:
chain: ${{ matrix.chain }}
monitor: ${{ matrix.monitor }}
monitor: assertion
secrets: inherit
18 changes: 18 additions & 0 deletions .github/workflows/monitoring/batch-poster-monitoring.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,18 @@
name: Batch Poster Monitoring

on:
push:
workflow_dispatch:
schedule:
- cron: "0 */6 * * *" # Run every 6 hours

jobs:
run-monitoring:
strategy:
matrix:
chain: [core, orbit]
uses: ./.github/workflows/monitoring.yml
with:
chain: ${{ matrix.chain }}
monitor: batch-poster
secrets: inherit
File renamed without changes.
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,6 @@ jobs:
with:
repository: OffchainLabs/arbitrum-monitoring
path: arbitrum-monitoring
ref: main

- name: Setup Node and install dependencies
uses: actions/setup-node@v4
Expand All @@ -60,7 +59,7 @@ jobs:
BATCH_POSTER_MONITORING: ${{ inputs.monitor == 'batch-poster' }}

- name: Copy chains JSON to Arbitrum Monitoring
run: cd ./arbitrum-monitoring/packages && ls && cp ./packages/arb-token-bridge-ui/public/${{ steps.config.outputs.config_file }} ./arbitrum-monitoring/packages/${{ inputs.monitor }}-monitor/config.json
run: cp ./packages/arb-token-bridge-ui/public/${{ steps.config.outputs.config_file }} ./arbitrum-monitoring/packages/${{ inputs.monitor }}-monitor/config.json

- name: Run monitoring command
run: cd ./arbitrum-monitoring && yarn ${{ inputs.monitor }}-monitor --enableAlerting

0 comments on commit 58439b7

Please sign in to comment.