ci: Test Envoy sync in PRs

Signed-off-by: Ryan Northey <[email protected]>

Author: phlax

.github/workflows/_sync.yml

@@ -0,0 +1,39 @@
+name: CI/sync
+
+on:
+  workflow_call:
+    secrets:
+      app-id:
+      app-key:
+    inputs:
+      sync:
+        type: boolean
+        default: true
+
+jobs:
+  sync:
+    runs-on: ubuntu-22.04
+    steps:
+    - id: appauth
+      if: ${{ inputs.sync }}
+      uses: envoyproxy/toolshed/gh-actions/[email protected]
+      with:
+        key: ${{ secrets.app-key }}
+        app_id: ${{ secrets.app-id }}
+    - name: 'Checkout go-control-plane repository'
+      uses: actions/checkout@v4
+      with:
+        path: go-control-plane
+        fetch-depth: ${{ ! inputs.sync && 1 || 0 }}
+        token: ${{ steps.appauth.outputs.token || secrets.GITHUB_TOKEN }}
+    - name: 'Checkout Envoy repository'
+      uses: actions/checkout@v4
+      with:
+        repository: envoyproxy/envoy
+        fetch-depth: ${{ ! inputs.sync && 1 || 0 }}
+        path: envoy
+    - run: ci/sync_envoy.sh
+      env:
+        ENVOY_SRC_DIR: ../envoy
+        NO_COMMIT_CHANGES: ${{ ! inputs.sync && '1' || '' }}
+      working-directory: go-control-plane

.github/workflows/ci.yaml

@@ -8,8 +8,13 @@ on:
 
 jobs:
   build:
-    runs-on: ubuntu-latest
+    runs-on: ubuntu-22.04
     steps:
       - uses: actions/checkout@v4
       - name: build and test
-        run: make docker_tests
+        run: make docker_tests
+
+  sync:
+    uses: ./.github/workflows/_sync.yml
+    with:
+      sync: false

.github/workflows/envoy-sync.yaml

@@ -1,50 +1,25 @@
 name: Sync Envoy
 
+permissions:
+  contents: read
+
 on:
   workflow_dispatch:
 
 concurrency:
   group: ${{ github.workflow }}
   cancel-in-progress: true
 
-permissions:
-  contents: read
 
 jobs:
   sync:
     runs-on: ubuntu-22.04
-    permissions:
-      contents: write
     if: |
       ${{
           !contains(github.actor, '[bot]')
           || github.actor == 'sync-envoy[bot]'
       }}
-    steps:
-    - id: appauth
-      uses: envoyproxy/toolshed/gh-actions/[email protected]
-      with:
-        key: ${{ secrets.ENVOY_CI_UPDATE_BOT_KEY }}
-        app_id: ${{ secrets.ENVOY_CI_UPDATE_APP_ID }}
-
-    # Checkout the repo
-    - name: 'Checkout Repository'
-      uses: actions/checkout@v4
-      with:
-        ref: main
-        fetch-depth: 0
-        token: ${{ steps.appauth.outputs.token }}
-
-    # Checkout the Envoy repo
-    - name: 'Checkout Repository'
-      uses: actions/checkout@v4
-      with:
-        repository: envoyproxy/envoy
-        ref: main
-        fetch-depth: 0
-        path: upstream
-
-    - run: mv upstream ../envoy
-    - run: ci/sync_envoy.sh
-      env:
-        ENVOY_SRC_DIR: ../envoy
+    uses: ./.github/workflows/_sync.yml
+    secrets:
+      app-id: ${{ secrets.ENVOY_CI_UPDATE_APP_ID }}
+      app-key: ${{ secrets.ENVOY_CI_UPDATE_BOT_KEY }}

ci/sync_envoy.sh

@@ -73,4 +73,6 @@ commit_changes () {
 
 build_protos
 sync_protos
-commit_changes
+if [[ -z "$NO_COMMIT_CHANGES" ]]; then
+    commit_changes
+fi