Skip to content
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

separate daily test metrics #646

Merged
merged 1 commit into from
Jan 21, 2025
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
32 changes: 25 additions & 7 deletions .github/workflows/daily-test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ env:
ROLE_TO_ASSUME: ${{ secrets.ROLE_TO_ASSUME }}
METRIC_NAME: ${{ secrets.METRIC_NAME }}
METRIC_NAMESPACE: ${{ secrets.METRIC_NAMESPACE }}

REGION: us-east-1

permissions:
id-token: write
Expand All @@ -32,33 +32,46 @@ jobs:
matrix:
# Latest 2 Android versions. TODO: Upgrade to Appium 2 for OS 14 and above.
os_list: ["12", "13"]
demo_download_url_list: ["$DEMO_APP_DOWNLOAD_LINK", "$DEMO_APP_DOWNLOAD_LINK_NO_VIDEO_CODECS"]
demo_flavor_list: ["default", "no-video-codecs"]
outputs:
job-status: ${{ job.status }}
continue-on-error: true

steps:
- name: Checkout test package
uses: actions/checkout@v4
if: always()
with:
repository: awslabs/amazon-chime-sdk-apple
token: ${{ secrets.GH_INTEG_READ_ONLY_PAT }}
ref: automated-test-development

- name: Get latest prod demo app
if: always()
run: |
wget -O amazon-chime-sdk-app.apk ${{ matrix.demo_download_url_list }}
if [ "${{ matrix.demo_flavor_list }}" = "default" ]; then
wget -O amazon-chime-sdk-app.apk $DEMO_APP_DOWNLOAD_LINK;
elif [ "${{ matrix.demo_flavor_list }}" = "no-video-codecs" ]; then
wget -O amazon-chime-sdk-app.apk $DEMO_APP_DOWNLOAD_LINK_NO_VIDEO_CODECS;
else
echo "Error: Unsupported demo app flavor: ${{ matrix.demo_flavor_list }}";
exit 1;
fi

- name: Setup Node.js - 15.x
uses: actions/setup-node@v4
if: always()
with:
node-version: 15.x

- name: Install Dependencies
if: always()
run: |
npm install @aws-sdk/client-cloudwatch

- name: Run tests against specified Android versions
id: tests
if: always()
run: |
id=$(curl -F '[email protected]' -F name=amazon-chime-sdk-app.apk -u "${{ secrets.SAUCE_USERNAME }}:${{ secrets.SAUCE_ACCESS_KEY }}" 'https://api.us-west-1.saucelabs.com/v1/storage/upload' |jq '.item.id')
npm install
Expand All @@ -78,25 +91,29 @@ jobs:

- name: Configure AWS Credentials
uses: aws-actions/configure-aws-credentials@v2
if: always()
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: this is required because we want to ensure it refreshs for each flavor?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

required because regardless of whether previous steps fail, we still want to try and send a datapoint to CW

with:
role-to-assume: ${{ secrets.ROLE_TO_ASSUME }}
aws-region: us-east-1
aws-region: ${{ env.REGION }}

- name: Setup Node.js - 18.x
uses: actions/setup-node@v4
if: always()
with:
node-version: 18.x

- name: Install Dependencies
if: always()
run: |
# Required to bydpass issue with canvas dependency: https://github.com/Automattic/node-canvas/issues/1862
sudo apt-get install -y libcairo2-dev libjpeg-dev libpango1.0-dev libgif-dev build-essential g++

- name: Send Metric to CloudWatch
if: always()
run: |
node -e "
const { CloudWatchClient, PutMetricDataCommand } = require('@aws-sdk/client-cloudwatch');
const client = new CloudWatchClient({ region: 'us-east-1' });
const client = new CloudWatchClient({ region: process.env.REGION });

const value = '${{ job.status }}' === 'failure' ? 0 : 1;

Expand All @@ -105,8 +122,9 @@ jobs:
MetricData: [{
MetricName: process.env.METRIC_NAME,
Dimensions: [
{ Name: 'Workflow', Value: 'DailyTest' },
{ Name: 'Platform', Value: 'Android' }
{ Name: 'Platform', Value: 'Android' },
{ Name: 'OS', Value: ${{ matrix.os_list }} },
{ Name: 'Flavor', Value: '${{ matrix.demo_flavor_list }}' }
],
Value: value,
}]
Expand Down
Loading