Skip to content

chore(ci): Fix the email report #98

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 14 commits into from
May 14, 2025
1 change: 0 additions & 1 deletion .github/workflows/nightly_packaging_report.yml
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,6 @@ jobs:
date: '${{ inputs.date }}'
secrets:
CROSSBOW_GITHUB_TOKEN: ${{ secrets.CROSSBOW_GITHUB_TOKEN }}
CROSSBOW_SMTP_USER: ${{ secrets.CROSSBOW_SMTP_USER }}
CROSSBOW_SMTP_PASSWORD: ${{ secrets.CROSSBOW_SMTP_PASSWORD }}
CROSSBOW_ZULIP_WEBHOOK: ${{ secrets.CROSSBOW_ZULIP_WEBHOOK }}
CROSSBOW_SLACK_WEBHOOK: ${{ secrets.CROSSBOW_SLACK_WEBHOOK }}
1 change: 0 additions & 1 deletion .github/workflows/nightly_release_report.yml
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,6 @@ jobs:
date: '${{ inputs.date }}'
secrets:
CROSSBOW_GITHUB_TOKEN: ${{ secrets.CROSSBOW_GITHUB_TOKEN }}
CROSSBOW_SMTP_USER: ${{ secrets.CROSSBOW_SMTP_USER }}
CROSSBOW_SMTP_PASSWORD: ${{ secrets.CROSSBOW_SMTP_PASSWORD }}
CROSSBOW_ZULIP_WEBHOOK: ${{ secrets.CROSSBOW_ZULIP_WEBHOOK }}
CROSSBOW_SLACK_WEBHOOK: ${{ secrets.CROSSBOW_SLACK_WEBHOOK }}
1 change: 0 additions & 1 deletion .github/workflows/nightly_tests_report.yml
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,6 @@ jobs:
date: '${{ inputs.date }}'
secrets:
CROSSBOW_GITHUB_TOKEN: ${{ secrets.CROSSBOW_GITHUB_TOKEN }}
CROSSBOW_SMTP_USER: ${{ secrets.CROSSBOW_SMTP_USER }}
CROSSBOW_SMTP_PASSWORD: ${{ secrets.CROSSBOW_SMTP_PASSWORD }}
CROSSBOW_ZULIP_WEBHOOK: ${{ secrets.CROSSBOW_ZULIP_WEBHOOK }}
CROSSBOW_SLACK_WEBHOOK: ${{ secrets.CROSSBOW_SLACK_WEBHOOK }}
7 changes: 3 additions & 4 deletions .github/workflows/report.yml
Original file line number Diff line number Diff line change
Expand Up @@ -13,8 +13,6 @@ on:
secrets:
CROSSBOW_GITHUB_TOKEN:
required: true
CROSSBOW_SMTP_USER:
required: true
CROSSBOW_SMTP_PASSWORD:
required: true
CROSSBOW_ZULIP_WEBHOOK:
Expand Down Expand Up @@ -66,10 +64,11 @@ jobs:
set +e
archery crossbow report \
--send \
--smtp-server 'commit-email.info' \
--sender-name Crossbow \
--sender-email 'crossbow@ursacomputing.com' \
--sender-email 'crossbow@commit-email.info' \
--recipient-email '[email protected]' \
--smtp-user ${{ secrets.CROSSBOW_SMTP_USER }} \
--smtp-user crossbow \
--smtp-password ${{ secrets.CROSSBOW_SMTP_PASSWORD }} \
${job_id}
email_exit_code=$?
Expand Down
11 changes: 6 additions & 5 deletions .github/workflows/token_expiration.yml
Original file line number Diff line number Diff line change
Expand Up @@ -8,11 +8,13 @@ on:
required: true
default: '30'
type: string

schedule:
# Mondays
- cron: '0 0 * * 1'

permissions:
contents: read

jobs:
token-expiration-check:
name: Validate token expiration date
Expand All @@ -24,8 +26,6 @@ jobs:
- name: Checkout Crossbow
uses: actions/checkout@v3
with:
# If we use github.token then the push will not trigger CI
token: ${{ secrets.CROSSBOW_GITHUB_TOKEN }}
path: crossbow
fetch-depth: 0
- uses: actions/setup-python@v4
Expand All @@ -42,9 +42,10 @@ jobs:
echo "Checking whether to notify on token expiration date"
archery crossbow notify-token-expiration \
--send \
--smtp-server 'commit-email.info' \
--sender-name Crossbow \
--sender-email 'crossbow@ursacomputing.com' \
--sender-email 'crossbow@commit-email.info' \
--recipient-email '[email protected]' \
--smtp-user ${{ secrets.CROSSBOW_SMTP_USER }} \
--smtp-user crossbow \
--smtp-password ${{ secrets.CROSSBOW_SMTP_PASSWORD }} \
--days ${{ inputs.days || '30'}}