Skip to content

Commit

Permalink
Update CI to use fixed Fluentd version (#10)
Browse files Browse the repository at this point in the history
  • Loading branch information
stevehipwell authored May 19, 2021
1 parent 8bf5b46 commit ab346fb
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 12 deletions.
10 changes: 0 additions & 10 deletions .github/workflows/publish.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -15,14 +15,6 @@ jobs:
id: tag
uses: dawidd6/action-get-tag@v1

- name: Set variables
id: vars
run: |
fluentd_version_lookup="$(echo "${{ steps.tag.outputs.tag }}" | grep -ioP 'v\d+\.\d+')"
fluentd_version_lookup="FLUENTD_${fluentd_version_lookup/./_}"
fluentd_version_lookup="${fluentd_version_lookup/v/V}"
echo "::set-output name=fluentd_version_lookup::${fluentd_version_lookup}"
- name: Docker meta
id: docker_meta
uses: crazy-max/ghaction-docker-meta@v1
Expand Down Expand Up @@ -61,8 +53,6 @@ jobs:
push: true
tags: ${{ steps.docker_meta.outputs.tags }}
labels: ${{ steps.docker_meta.outputs.labels }}
build-args: |
FLUENTD_VERSION=${{ secrets[steps.vars.outputs.fluentd_version_lookup] }}
# - name: Update repo description
# uses: peter-evans/dockerhub-description@v2
# with:
Expand Down
3 changes: 1 addition & 2 deletions Dockerfile
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
ARG FLUENTD_VERSION="v1.12"
FROM fluent/fluentd:${FLUENTD_VERSION}
FROM fluent/fluentd:v1.12.3-1.0

# Use root account to use apk
USER root
Expand Down

0 comments on commit ab346fb

Please sign in to comment.