From 83b791223fdd2bae907bf4feb4bc9ca912ba5bf6 Mon Sep 17 00:00:00 2001
From: Tamir Duberstein <tamird@gmail.com>
Date: Wed, 5 Feb 2025 06:22:33 -0500
Subject: [PATCH] gen: remove pointless "update"

libbpf is a submodule, so its version is already effectively pinned.
`xtask codegen` also runs `git submodule update` which reverts the
action of updating to `origin/HEAD`. Remove the cruft.
---
 .github/workflows/gen.yml | 11 +++--------
 1 file changed, 3 insertions(+), 8 deletions(-)

diff --git a/.github/workflows/gen.yml b/.github/workflows/gen.yml
index a5dd3eee..5165ce28 100644
--- a/.github/workflows/gen.yml
+++ b/.github/workflows/gen.yml
@@ -11,14 +11,6 @@ jobs:
         with:
           submodules: recursive
 
-      - name: update libbpf
-        working-directory: xtask/libbpf
-        run: |
-          set -euxo pipefail
-          git fetch origin
-          git checkout origin/HEAD
-          echo "LIBBPF_SHA=$(git rev-parse HEAD)" >> $GITHUB_ENV
-
       - uses: dtolnay/rust-toolchain@nightly
         with:
           components: rustfmt, clippy
@@ -34,6 +26,9 @@ jobs:
       - run: cargo xtask codegen
       - run: cargo xtask public-api --bless
 
+      - run: echo "LIBBPF_SHA=$(git rev-parse HEAD)" >> $GITHUB_ENV
+        working-directory: xtask/libbpf
+
       - uses: peter-evans/create-pull-request@v7
         with:
           branch: create-pull-request/codegen