Merge branch 'mainline' into mainline

.github/workflows/ci_size_computer.yml

@@ -0,0 +1,97 @@
+name: Compute bin sizes
+
+on:
+ pull_request:
+ types:
+ - opened
+ - edited
+ - synchronize
+
+jobs:
+ binsize:
+ runs-on: ubuntu-latest
+ steps:
+ - name: Set up Go
+ uses: actions/setup-go@v2
+ with:
+ go-version: 1.18
+
+ - name: Set up Node
+ uses: actions/setup-node@v1
+ with:
+ node-version: 16.x
+
+ - name: Get latest release with tag
+ id: latestrelease
+ run: |
+ echo "VERSION_TAG=$(curl -s https://api.github.com/repos/aws/copilot-cli/releases/latest | jq '.tag_name' | sed 's/\"//g')" >> $GITHUB_OUTPUT
+
+ - name: Check out latest release tag
+ uses: actions/checkout@v3
+ with:
+ ref: ${{ steps.latestrelease.outputs.VERSION_TAG }}
+ repository: aws/copilot-cli
+
+ - name: Compute old release binary sizes
+ id: old-bins
+ run: |
+ make release
+ echo "MAC_AMD_KIB=$(du -k ./bin/local/copilot-darwin-amd64 | awk '{ print $1}')" >> $GITHUB_OUTPUT
+ echo "MAC_ARM_KIB=$(du -k ./bin/local/copilot-darwin-arm64 | awk '{ print $1}')" >> $GITHUB_OUTPUT
+ echo "LINUX_AMD_KIB=$(du -k ./bin/local/copilot-linux-amd64 | awk '{ print $1}')" >> $GITHUB_OUTPUT
+ echo "LINUX_ARM_KIB=$(du -k ./bin/local/copilot-linux-arm64 | awk '{ print $1}')" >> $GITHUB_OUTPUT
+ echo "WINDOWS_AMD_KIB=$(du -k ./bin/local/copilot.exe | awk '{ print $1}')" >> $GITHUB_OUTPUT
+
+ - name: Check out PR commit
+ uses: actions/checkout@v3
+ with:
+ ref: ${{ github.event.pull_request.head.sha }}
+
+ - name: Compute new release binary sizes
+ id: new-bins
+ run: |
+ make release
+ echo "MAC_AMD_KIB=$(du -k ./bin/local/copilot-darwin-amd64 | awk '{ print $1}')" >> $GITHUB_OUTPUT
+ echo "MAC_ARM_KIB=$(du -k ./bin/local/copilot-darwin-arm64 | awk '{ print $1}')" >> $GITHUB_OUTPUT
+ echo "LINUX_AMD_KIB=$(du -k ./bin/local/copilot-linux-amd64 | awk '{ print $1}')" >> $GITHUB_OUTPUT
+ echo "LINUX_ARM_KIB=$(du -k ./bin/local/copilot-linux-arm64 | awk '{ print $1}')" >> $GITHUB_OUTPUT
+ echo "WINDOWS_AMD_KIB=$(du -k ./bin/local/copilot.exe | awk '{ print $1}')" >> $GITHUB_OUTPUT
+
+ - name: Save sizes and PR number
+ run: |
+ mkdir artifacts
+ sizes_json="{
+ \"macOS\": {
+ \"amd\": {
+ \"old\": ${{ steps.old-bins.outputs.MAC_AMD_KIB }},
+ \"cur\": ${{ steps.new-bins.outputs.MAC_AMD_KIB }}
+ },
+ \"arm\": {
+ \"old\": ${{ steps.old-bins.outputs.MAC_ARM_KIB }},
+ \"cur\": ${{ steps.new-bins.outputs.MAC_ARM_KIB }}
+ }
+ },
+ \"linux\": {
+ \"amd\": {
+ \"old\": ${{ steps.old-bins.outputs.LINUX_AMD_KIB }},
+ \"cur\": ${{ steps.new-bins.outputs.LINUX_AMD_KIB }}
+ },
+ \"arm\": {
+ \"old\": ${{ steps.old-bins.outputs.LINUX_ARM_KIB }},
+ \"cur\": ${{ steps.new-bins.outputs.LINUX_ARM_KIB }}
+ }
+ },
+ \"windows\": {
+ \"amd\": {
+ \"old\": ${{ steps.old-bins.outputs.WINDOWS_AMD_KIB }},
+ \"cur\": ${{ steps.new-bins.outputs.WINDOWS_AMD_KIB }}
+ }
+ }
+ }"
+ echo ${{ github.event.number }} > artifacts/pr_number
+ echo ${sizes_json} > artifacts/sizes.json
+
+ - uses: actions/upload-artifact@v3
+ with:
+ name: pr_number_and_bin_sizes
+ path: artifacts/

.github/workflows/ci_size_writer.yml

@@ -0,0 +1,110 @@
+name: Comment PR with bin sizes
+
+on:
+ workflow_run:
+ workflows: ["Compute bin sizes"]
+ types:
+ - completed
+
+jobs:
+ comment_bin_size:
+ runs-on: ubuntu-latest
+ steps:
+ - name: Download artifacts
+ uses: actions/github-script@v6
+ with:
+ script: |
+ let allArtifacts = await github.rest.actions.listWorkflowRunArtifacts({
+ owner: context.repo.owner,
+ repo: context.repo.repo,
+ run_id: context.payload.workflow_run.id,
+ });
+ let matchArtifact = allArtifacts.data.artifacts.filter((artifact) => {
+ return artifact.name == "pr_number_and_bin_sizes"
+ })[0];
+ let download = await github.rest.actions.downloadArtifact({
+ owner: context.repo.owner,
+ repo: context.repo.repo,
+ artifact_id: matchArtifact.id,
+ archive_format: 'zip',
+ });
+ let fs = require('fs');
+ fs.writeFileSync(`${process.env.GITHUB_WORKSPACE}/artifacts.zip`, Buffer.from(download.data));
+ 
+ - name: Unzip artifacts
+ run: unzip artifacts.zip
+
+ - name: Calculate size change
+ id: diff
+ uses: actions/github-script@v6
+ with:
+ result-encoding: string
+ script: |
+ let fs = require('fs');
+ let sizes_raw = fs.readFileSync('sizes.json');
+ let sizes = JSON.parse(sizes_raw)
+
+ const diff = (old, cur) => {
+ const res = (cur-old)/old * 100;
+ return res.toFixed(2); // 2 decimal places.
+ };
+
+ const lastRelease = '${{ steps.latestrelease.outputs.VERSION_TAG }}';
+ let msg = `
+ 🍕 Here are the new binary sizes!
+
+ | Name | New size (kiB) | ${lastRelease} size (kiB) | Delta (%) |
+ |:-- |:--- |:--- |:-- |
+ `;
+
+ let shouldAcknowledgeSize = false;
+ for (const os of Object.keys(sizes)) {
+ for (const arch of Object.keys(sizes[os])) {
+ const delta = diff(sizes[os][arch].old, sizes[os][arch].cur);
+ if (delta > 10) {
+ shouldAcknowledgeSize = true;
+ }
+
+ let symbol = "❤️ ";
+ if (delta > 5) {
+ symbol = "😭 +";
+ } else if (delta > 1) {
+ symbol = "🥺 +"
+ } else if (delta > 0) {
+ symbol = "+";
+ }
+ msg += `| ${os} (${arch}) | ${sizes[os][arch].cur} | ${sizes[os][arch].old} | ${symbol}${delta} |` + "\n";
+ }
+ }
+
+ if (shouldAcknowledgeSize) {
+ msg += "**The binary size increased more than expected! Applying the `do-not-merge` label.**\n"
+ github.rest.issues.addLabels({
+ issue_number: context.issue.number,
+ owner: context.repo.owner,
+ repo: context.repo.repo,
+ labels: ['do-not-merge']
+ });
+ }
+ return msg;
+ 
+ - name: Find PR number
+ id: find-pr-number
+ run: |
+ echo "PR_NUMBER=$(cat pr_number)" >> $GITHUB_OUTPUT
+ 
+ - name: Find comment
+ uses: peter-evans/find-comment@v2
+ id: fc
+ with:
+ issue-number: ${{ steps.find-pr-number.outputs.PR_NUMBER }}
+ comment-author: 'github-actions[bot]'
+ body-includes: 'Here are the new binary sizes'
+
+ - name: Create or update comment
+ uses: peter-evans/create-or-update-comment@v2
+ with:
+ comment-id: ${{ steps.fc.outputs.comment-id }}
+ issue-number: ${{ steps.find-pr-number.outputs.PR_NUMBER }}
+ body: ${{ steps.diff.outputs.result }}
+ edit-mode: replace