Skip to content

Commit

Permalink
Fix LRE/Remote workflow after b44383f (#1441)
Browse files Browse the repository at this point in the history
The previous variant didn't work for pushes on main.
  • Loading branch information
aaronmondal authored Oct 30, 2024
1 parent b44383f commit 399e95b
Showing 1 changed file with 6 additions and 6 deletions.
12 changes: 6 additions & 6 deletions .github/workflows/lre.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -89,9 +89,9 @@ jobs:
- name: Start NativeLink operator
env:
PR_URL: ${{ github.event.pull_request.head.repo.clone_url }}
PR_BRANCH: ${{ github.event.pull_request.head.ref }}
PR_COMMIT: ${{ github.event.pull_request.head.sha }}
REPO_URL: ${{ github.event.pull_request.head.repo.clone_url || github.repository }}
BRANCH: ${{ github.event.pull_request.head.ref || github.ref_name }}
COMMIT: ${{ github.event.pull_request.head.sha || github.sha }}
run: |
nix develop --impure --command bash -c 'cat > kustomization.yaml << EOF
apiVersion: kustomize.config.k8s.io/v1beta1
Expand All @@ -109,13 +109,13 @@ jobs:
- patch: |-
- op: replace
path: /spec/url
value: ${PR_URL}
value: ${REPO_URL}
- op: replace
path: /spec/ref/branch
value: ${PR_BRANCH}
value: ${BRANCH}
- op: replace
path: /spec/ref/commit
value: ${PR_COMMIT}
value: ${COMMIT}
target:
kind: GitRepository
name: nativelink
Expand Down

0 comments on commit 399e95b

Please sign in to comment.