Upload size report to the metrics service.

Author: yifanyang

.github/workflows/test-all.yml

@@ -1,6 +1,6 @@
 name: Run All Tests
 
-on: 
+on:
   pull_request:
     branches:
       - master
@@ -40,8 +40,6 @@ jobs:
         github-token: ${{ secrets.GITHUB_TOKEN }}
         path-to-lcov: ./lcov-all.info
       continue-on-error: true
-    - name: Generate Size Report
-      run: yarn size-report
   deploy:
     name: Canary Deploy
     runs-on: ubuntu-latest

.github/workflows/test-changed.yml

@@ -28,5 +28,27 @@ jobs:
       run: yarn build
     - name: Run tests on changed packages
       run: xvfb-run yarn test:changed
-    - name: Generate Size Report
-      run: yarn size-report
+
+  health-metrics-test:
+    name: Health Metrics Test
+    runs-on: ubuntu-latest
+    env:
+      METRICS_SERVICE_URL: ${{ secrets.METRICS_SERVICE_URL }}
+      GITHUB_PULL_REQUEST_NUMBER: ${{ github.event.pull_request.number }}
+      GITHUB_PULL_REQUEST_BASE_SHA: ${{ github.event.pull_request.base.sha }}
+    steps:
+      - uses: actions/checkout@v2
+      - uses: actions/setup-node@v1
+        with:
+          node-version: 10.x
+      - uses: GoogleCloudPlatform/github-actions/setup-gcloud@master
+        with:
+          service_account_key: ${{ secrets.GCP_SA_KEY }}
+      - run: cp config/ci.config.json config/project.json
+      - run: yarn install
+      - run: yarn build
+
+      - name: Run health-metrics/binary-size test
+        run: yarn size-report
+
+      # TODO(yifany): Enable startup times testing on CI.

scripts/report_binary_size.js

@@ -1,29 +1,31 @@
 const { resolve } = require('path');
 const fs = require('fs');
 const { execSync } = require('child_process');
+const https = require('https');
 
 const repoRoot = resolve(__dirname, '..');
 
-const commitHash = process.env.GITHUB_SHA || execSync('git rev-parse HEAD').toString();
 const runId = process.env.GITHUB_RUN_ID || 'local-run-id';
 
+const METRICS_SERVICE_URL = process.env.METRICS_SERVICE_URL;
+
 // CDN scripts
 function generateReportForCDNScripts() {
     const reports = [];
     const firebaseRoot = resolve(__dirname, '../packages/firebase');
     const pkgJson = require(`${firebaseRoot}/package.json`);
-    
+
     const special_files = [
         'firebase-performance-standalone.es2017.js',
         'firebase-performance-standalone.js',
         'firebase.js'
     ];
-    
+
     const files = [
-        ...special_files.map(file => `${firebaseRoot}/${file}`), 
+        ...special_files.map(file => `${firebaseRoot}/${file}`),
         ...pkgJson.components.map(component => `${firebaseRoot}/firebase-${component}.js`)
     ];
-    
+
     for (const file of files) {
         const { size } = fs.statSync(file);
         const fileName = file.split('/').slice(-1)[0];
@@ -83,13 +85,64 @@ function generateSizeReport() {
         console.log(r.sdk, r.type, r.value);
     }
 
-    console.log(`Github Action URL: https://github.com/firebase/firebase-js-sdk/actions/runs/${runId}`);
+    console.log(`Github Action URL: https://github.com/${process.env.GITHUB_REPOSITORY}/actions/runs/${runId}`);
 
     return {
-        log: `https://github.com/firebase/firebase-js-sdk/actions/runs/${runId}`,
+        log: `https://github.com/${process.env.GITHUB_REPOSITORY}/actions/runs/${runId}`,
         metric: "BinarySize",
         results: reports
     };
 }
 
-generateSizeReport();
+function constructRequestPath() {
+    const repo = process.env.GITHUB_REPOSITORY;
+    const commit = process.env.GITHUB_SHA;
+    let path = `/repos/${repo}/commits/${commit}/reports`;
+    if (process.env.GITHUB_EVENT_NAME === 'pull_request') {
+        const pullRequestNumber = process.env.GITHUB_PULL_REQUEST_NUMBER;
+        const pullRequestBaseSha = process.env.GITHUB_PULL_REQUEST_BASE_SHA;
+        path += `?pull_request=${pullRequestNumber}&base_commit=${pullRequestBaseSha}`;
+    }
+    return path;
+}
+
+function constructRequestOptions(path) {
+    const accessToken = execSync('gcloud auth print-identity-token', { encoding: 'utf8' }).trim();
+    return {
+        path: path,
+        method: 'POST',
+        headers: {
+            'Authorization': `Bearer ${accessToken}`,
+            'Content-Type': 'application/json'
+        }
+    };
+}
+
+function upload(report) {
+    if (!process.env.GITHUB_ACTIONS) {
+        console.log('Metrics upload is only enabled on CI.');
+        return;
+    }
+
+    const path = constructRequestPath();
+    const options = constructRequestOptions(path);
+
+    console.log(`${report.metric} report:`, report);
+    console.log(`Posting to metrics service endpoint: ${path} ...`);
+
+    const request = https.request(METRICS_SERVICE_URL, options, response => {
+        response.setEncoding('utf8');
+        console.log(`Response status code: ${response.statusCode}`);
+        response.on('data', console.log);
+        response.on('end', () => {
+            if (response.statusCode !== 202) {
+                process.exit(1);
+            }
+        })
+    });
+    request.write(JSON.stringify(report));
+    request.end();
+}
+
+const report = generateSizeReport();
+upload(report);