[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