Skip to content

Commit

Permalink
feat: Docker Build for amd64 and arm64 platforms (ytti#2911)
Browse files Browse the repository at this point in the history
* Added buildx platforms argument to build for linux/amd64 and linux/arm64
* Updated README.md documentation to note that the container is built for both platforms.
* Updated CHANGELOG.md to reflect unreleased addition of new docker platforms added to build/publish.

Fixes Issue: ytti#2910, ytti#2977
  • Loading branch information
disaac authored Oct 30, 2023
1 parent ab54ef0 commit 10cfa09
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 2 deletions.
1 change: 1 addition & 0 deletions .github/workflows/publishdocker.yml
Original file line number Diff line number Diff line change
Expand Up @@ -20,3 +20,4 @@ jobs:
username: ${{ secrets.DOCKER_USERNAME }}
password: ${{ secrets.DOCKER_PASSWORD }}
tags: "latest,${{ steps.get_version.outputs.release-version }}"
platforms: linux/amd64,linux/arm64
3 changes: 2 additions & 1 deletion CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/).
- model for Mimosa B11 (@ritzbhuj)
- Added ability to send mail with the Docker container
- Documentation to send mail with hooks
- Added `linux/arm64` and `linux/amd64` platforms to Docker build/publish. (@disaac)

## Changed
- tp-link: fixed enable mode post login entrance (@mirackle-spb)
Expand All @@ -33,7 +34,7 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/).
- fixed prompt for Watchguard FirewareOS not matching the regex when the node is non-master (@netdiver)
- defined 'psych' runtime dependency to resolve 'unsafe_load' error during startup (@MattKobayashi)
- fixed new date/time format with newer RouterOS `# jun/01/2023 12:11:25 by RouterOS 7.9.1` vs `# 2023-06-01 12:16:16 by RouterOS 7.10rc1` (@tim427)
- fixed netscaler backups with hostname set #2828 (@electrocret)
- fixed netscaler backups with hostname set #2828 (@electrocret)
- Do not redirect stderr when fetching opnsense version since default shell (csh) doesn't support it (@spike77453)
- Fix missing configuration for Opengear (@ermuller)
- Fixed fan RPM speeds included in Aruba CX diffs (@danpoltawski)
Expand Down
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -171,7 +171,7 @@ rake install

### Running with Docker

Currently, Docker Hub automatically builds the master branch as [oxidized/oxidized](https://hub.docker.com/r/oxidized/oxidized/), you can make use of this container or build your own.
Currently, Docker Hub automatically builds the master branch for linux/amd64 and linux/arm64 platforms as [oxidized/oxidized](https://hub.docker.com/r/oxidized/oxidized/), you can make use of this container or build your own.

To build your own, clone git repo:

Expand Down

0 comments on commit 10cfa09

Please sign in to comment.