[llvm] [Github] Fix windows container push (PR #119916)

via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 13 18:52:53 PST 2024


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-github-workflow

Author: Aiden Grossman (boomanaiden154)

<details>
<summary>Changes</summary>

The windows container push was not tested in the pull request and had a couple of typos that prevented it from functioning. This patch fixes that so we can actually push the container to GHCR.

---
Full diff: https://github.com/llvm/llvm-project/pull/119916.diff


1 Files Affected:

- (modified) .github/workflows/build-ci-container-windows.yml (+4-4) 


``````````diff
diff --git a/.github/workflows/build-ci-container-windows.yml b/.github/workflows/build-ci-container-windows.yml
index 898cdce097e97e..bba34066a97cd2 100644
--- a/.github/workflows/build-ci-container-windows.yml
+++ b/.github/workflows/build-ci-container-windows.yml
@@ -68,8 +68,8 @@ jobs:
           name: container
       - name: Push Container
         run: |
-          docker load -i ${{ needs.build-ci-container.outptus.container-filename }}
-          docker tag ${{ steps.vars.outputs.container-name-tag }} ${{ steps.vars.outputs.container-name }}:latest
+          docker load -i ${{ needs.build-ci-container-windows.outputs.container-filename }}
+          docker tag ${{ needs.build-ci-container-windows.outputs.container-name-tag }} ${{ needs.build-ci-container-windows.outputs.container-name }}:latest
           docker login -u ${{ github.actor }} -p $env:GITHUB_TOKEN ghcr.io
-          docker push ${{ needs.build-ci-container.outputs.container-name-tag }}
-          docker push ${{ needs.build-ci-container.outputs.container-name }}:latest
+          docker push ${{ needs.build-ci-container-windows.outputs.container-name-tag }}
+          docker push ${{ needs.build-ci-container-windows.outputs.container-name }}:latest

``````````

</details>


https://github.com/llvm/llvm-project/pull/119916


More information about the llvm-commits mailing list