-
Notifications
You must be signed in to change notification settings - Fork 2
Fix fluentd build on arm64 #129
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Fix fluentd build on arm64 by changing the arm64 alpine base image to multi-arch debian based image.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR fixes the fluentd build on arm64 by migrating from an Alpine-based image (fluent/fluentd:v1.17.1-1.0) to a multi-arch Debian-based image (fluent/fluentd:v1.17.1-debian-1.0). This change addresses ARM64 compatibility issues by leveraging Debian's better multi-architecture support.
Key changes:
- Switched base image from Alpine to Debian variant
- Converted all package manager commands from
apktoapt-getwith appropriate Debian package names - Updated copyright header to simplified two-line format consistent with other files in the codebase
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
hippogr
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I leave a comment but approve it anyway.
**Description** Merge bug fixes from v1.4 to dev branch. **Major Revisions** * ModelProxy: fix uncommit changes (#122) * Fix cicd errors when branch creation and building all imges (#123) * Bugfix - fix bug of image regex deployment (#125) * Bugfix - fix bugs in alert manager test and fix kusto alert query issue (#126) * Fix openpai runtime build on arm64 (#128) * Fix fluentd build on arm64 (#129) * Bugfix - bug fix for baremetal support(#127) * Doc - Add Release note for v1.4.0 (#130) --------- Co-authored-by: zhogu <57975490+zhogu@users.noreply.github.com> Co-authored-by: Yifan Xiong <yifan.xiong@microsoft.com>
Fix fluentd build on arm64 by changing the amd64 alpine base image to multi-arch debian base image.