Skip to content

Commit 8d06020

Browse files
committed
Removed tailscale and metrics from ci
- We have this complex block for tracking test-time - However, it's been broken for a month and noone noticed - Therefore I don't think this is used - Also note that boot time is tracked in a different way - There are lots of ways to get job time statistics from GHA - So if we do want this, there's probably an easier way
1 parent 784192f commit 8d06020

1 file changed

Lines changed: 0 additions & 45 deletions

File tree

.github/workflows/ci.yml

Lines changed: 0 additions & 45 deletions
Original file line numberDiff line numberDiff line change
@@ -534,9 +534,6 @@ jobs:
534534
with:
535535
timezoneLinux: "America/New_York"
536536

537-
- name: Record start time
538-
run: date +%s > ${{ runner.temp }}/startTime # Get start time for test suite
539-
540537
- name: Set env vars (SQLite)
541538
if: contains(matrix.env.DB, 'sqlite')
542539
run: echo "database__connection__filename=/dev/shm/ghost-test.db" >> $GITHUB_ENV
@@ -553,13 +550,6 @@ jobs:
553550
working-directory: ghost/core
554551
run: yarn test:ci:integration
555552

556-
# Get runtime in seconds for test suite
557-
- name: Record test duration
558-
run: |
559-
startTime="$(cat ${{ runner.temp }}/startTime)"
560-
endTime="$(date +%s)"
561-
echo "test_time=$(($endTime-$startTime))" >> $GITHUB_ENV
562-
563553
- uses: actions/upload-artifact@v4
564554
if: matrix.node == env.NODE_VERSION && contains(matrix.env.DB, 'mysql')
565555
with:
@@ -568,41 +558,6 @@ jobs:
568558
ghost/*/coverage-e2e/cobertura-coverage.xml
569559
ghost/*/coverage-integration/cobertura-coverage.xml
570560
571-
# Continue on error if TailScale service is down
572-
- name: Tailscale Action
573-
timeout-minutes: 2
574-
continue-on-error: true
575-
if: (github.event_name == 'push' && github.repository_owner == 'TryGhost') || (github.event_name == 'pull_request' && startsWith(github.head_ref, 'TryGhost/'))
576-
uses: tailscale/github-action@v1
577-
with:
578-
authkey: ${{ secrets.TAILSCALE_AUTHKEY }}
579-
580-
# Report time taken to metrics service
581-
# Continue on error if previous TailScale step fails
582-
- name: Store test duration
583-
uses: tryghost/actions/actions/trigger-metric@main
584-
timeout-minutes: 1
585-
continue-on-error: true
586-
if: (github.event_name == 'push' && github.repository_owner == 'TryGhost') || (github.event_name == 'pull_request' && startsWith(github.head_ref, 'TryGhost/'))
587-
with:
588-
metricName: 'test-time'
589-
metricValue: ${{ env.test_time }}
590-
configuration: |
591-
{
592-
"metrics": {
593-
"transports": ["elasticsearch"],
594-
"metadata": {
595-
"database": "${{ matrix.env.DB }}",
596-
"node": "${{ matrix.node }}"
597-
}
598-
},
599-
"elasticsearch": {
600-
"host": "${{ secrets.ELASTICSEARCH_HOST }}",
601-
"username": "${{ secrets.ELASTICSEARCH_USERNAME }}",
602-
"password": "${{ secrets.ELASTICSEARCH_PASSWORD }}"
603-
}
604-
}
605-
606561
- uses: tryghost/actions/actions/slack-build@main
607562
if: failure() && github.event_name == 'push' && github.ref == 'refs/heads/main'
608563
with:

0 commit comments

Comments
 (0)