Skip to content
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

Fix Docker Build (issue with go-sqlite3) #259

Merged
merged 1 commit into from
Dec 25, 2023

Conversation

OstlerDev
Copy link

@OstlerDev OstlerDev commented Dec 20, 2023

Bug: In a new version of Alpine they updated an internal library which then broke the Go library mattn/go-sqlite3. Golang automatically pulled in this new version of alpine when rebuilding the 1.20 release, which is causing the builds to now fail.

Short Term Fix: By pinning the image to 1.20.0-alpine instead of just 1.20-alpine we are grabbing an older built image that does not experience this issue. This maintains compatibility with all current code and requires no other changes.

Long Term Fix: Update the library version for mattn/go-sqlite3 to one that includes the patch.

See here for more detail: mattn/go-sqlite3#1177 (comment)

Changelog

Fixed

  • Dockerfile is now able to build

Changed

  • Pin the golang image we build with to golang:1.20.0-alpine.

@jemand771
Copy link
Member

latest failure is my fault, I updated @actions/upload-artifact in my custom action without checking its breaking changes - whoops

@jemand771 jemand771 added Bug Something isn't working docker Things related to container builds, container images Build build-related changes labels Dec 25, 2023
@jemand771 jemand771 merged commit aa88498 into statping-ng:dev Dec 25, 2023
17 checks passed
@OstlerDev OstlerDev deleted the fix-docker-build-gosqlite3 branch January 3, 2024 18:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working Build build-related changes docker Things related to container builds, container images
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants