Skip to content

Commit

Permalink
Merge pull request #566 from openmeterio/fix-benthos-collector
Browse files Browse the repository at this point in the history
ci: fix benthos collector dockerfile
  • Loading branch information
sagikazarmark authored Jan 24, 2024
2 parents 8d205f6 + 0de9858 commit 880669b
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 2 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/artifacts.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -214,7 +214,7 @@ jobs:
uses: docker/build-push-action@4a13e500e55cf31b7a5d59a38ab2040ab0f42f56 # v5.1.0
with:
context: .
target: ${{ matrix.target }}
file: benthos-collector.Dockerfile
build-args: |
VERSION=${{ fromJSON(steps.meta.outputs.json).labels['org.opencontainers.image.version'] }}
platforms: linux/amd64,linux/arm64 # The confluent library and UBI8 don't support ARMv7
Expand Down
1 change: 1 addition & 0 deletions ci/build.go
Original file line number Diff line number Diff line change
Expand Up @@ -191,6 +191,7 @@ func (m *Binary) build(platform Platform, version string, pkg string) *File {

return goModule().
WithSource(m.Source).
WithPlatform(string(platform)).
Build(GoWithSourceBuildOpts{
Name: "benthos",
Pkg: pkg,
Expand Down
2 changes: 1 addition & 1 deletion ci/release.go
Original file line number Diff line number Diff line change
Expand Up @@ -94,7 +94,7 @@ func (m *Ci) binaryArchive(version string, platform Platform) *File {
}

return archiver.Archive(
fmt.Sprintf("benthos_%s", strings.ReplaceAll(string(platform), "/", "_")),
fmt.Sprintf("benthos-collector_%s", strings.ReplaceAll(string(platform), "/", "_")),
dag.Directory().
WithFile("", m.Build().Binary().benthosCollector(platform, version)).
WithFile("", m.Source.File("README.md")).
Expand Down

0 comments on commit 880669b

Please sign in to comment.