Skip to content

Commit

Permalink
Change the way the build arguments are being parsed in the github flo…
Browse files Browse the repository at this point in the history
…ws to make sure they dont get seen as 1 argument
  • Loading branch information
Robert Ragas committed Feb 13, 2024
1 parent f949c12 commit d858a50
Show file tree
Hide file tree
Showing 4 changed files with 8 additions and 2 deletions.
4 changes: 3 additions & 1 deletion .github/workflows/build-solr.yml
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,9 @@ jobs:
id: docker_build
uses: docker/build-push-action@v4
with:
build-args: VERSION=${{ matrix.version }} CONFIG_VERSION=${{ matrix.config_version }}
build-args: |
"VERSION=${{ matrix.version }}"
"CONFIG_VERSION=${{ matrix.config_version }}"
context: "{{defaultContext}}:${{ env.SUBDIRECTORY }}"
push: true
tags: ${{ env.REGISTRY }}/${{ env.IMAGE_NAME }}:${{ matrix.version }}
Expand Down
4 changes: 3 additions & 1 deletion .github/workflows/build-tugboat-solr.yml
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,9 @@ jobs:
id: docker_build
uses: docker/build-push-action@v4
with:
build-args: VERSION=${{ matrix.version }} CONFIG_VERSION=${{ matrix.config_version }}
build-args: |
"VERSION=${{ matrix.version }}"
"CONFIG_VERSION=${{ matrix.config_version }}"
context: "{{defaultContext}}:${{ env.SUBDIRECTORY }}"
push: true
tags: ${{ env.REGISTRY }}/${{ env.IMAGE_NAME }}:${{ matrix.version }}
Expand Down
1 change: 1 addition & 0 deletions solr/Dockerfile
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
ARG VERSION=latest
ARG CONFIG_VERSION=9

FROM solr:$VERSION

Expand Down
1 change: 1 addition & 0 deletions tugboat-solr/Dockerfile
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
ARG VERSION=latest
ARG CONFIG_VERSION=9

FROM tugboatqa/solr:$VERSION

Expand Down

0 comments on commit d858a50

Please sign in to comment.