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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update alpine to 3.7 #3121

Merged
merged 1 commit into from
Dec 10, 2017
Merged

Update alpine to 3.7 #3121

merged 1 commit into from
Dec 10, 2017

Conversation

sapk
Copy link
Member

@sapk sapk commented Dec 8, 2017

Alpine 3.7 has been release 馃槃

@ethantkoenig
Copy link
Member

LGTM (CI failure seems unrelated)

@tboerger tboerger added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Dec 8, 2017
Copy link
Contributor

@0rzech 0rzech left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Golang based on Alpine 3.7 is still in beta. Maybe it would be better to wait for its release, so the same version of Alpine is used for compilation and deployment?

@sapk
Copy link
Member Author

sapk commented Dec 8, 2017

That go1.10 that is in beta but alpine 3.7 does have go1.9 as golang version and we don't even have go inside the final docker container.

@0rzech
Copy link
Contributor

0rzech commented Dec 8, 2017

Sorry, the golang image itself is rc, not beta.

Official Golang Docker images do not use Go from Alpine repositories (though there is Go 1.9 in Alpine 3.7 community repo) and yes, we do not have Go installed in deployment image, but we do use specific Go version and link to system libraries when building Gitea in build image.

Btw. This is one of the reasons for Docker's existence: to minimize differences between development and production environments. Also, Alpine 3.7 is absolutely brand new, so maybe it would be better to let it settle a bit?

Maybe it's nothing, but IMHO there's no reason to pursue novelty yet. Especially when there's #2927 just around the corner.

Just my $0.02. I'm not blocking anything. ;)

@sapk
Copy link
Member Author

sapk commented Dec 9, 2017

@0rzech I understand and I will update #2927 accordingly. ;-)

edit: or the opposite if #2927 is merge before XD

@techknowlogick
Copy link
Member

LGTM

@tboerger tboerger added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Dec 9, 2017
@codecov-io
Copy link

Codecov Report

Merging #3121 into master will increase coverage by 0.22%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3121      +/-   ##
==========================================
+ Coverage   34.62%   34.85%   +0.22%     
==========================================
  Files         274      274              
  Lines       40026    40026              
==========================================
+ Hits        13861    13952      +91     
+ Misses      24183    24075     -108     
- Partials     1982     1999      +17
Impacted Files Coverage 螖
models/repo.go 41.47% <0%> (+0.18%) 猬嗭笍
models/repo_list.go 67.18% <0%> (+1.56%) 猬嗭笍
models/repo_indexer.go 51.45% <0%> (+2.91%) 猬嗭笍
models/lfs.go 31.25% <0%> (+6.25%) 猬嗭笍
modules/lfs/server.go 35.01% <0%> (+14.32%) 猬嗭笍
modules/lfs/content_store.go 43.75% <0%> (+35.93%) 猬嗭笍

Continue to review full report at Codecov.

Legend - Click here to learn more
螖 = absolute <relative> (impact), 酶 = not affected, ? = missing data
Powered by Codecov. Last update b8e45b1...4d0f81e. Read the comment docs.

@sapk
Copy link
Member Author

sapk commented Dec 9, 2017

Rebase and now tests are passing. \o/
Ready to be merge.

@lunny lunny added this to the 1.4.0 milestone Dec 10, 2017
@lunny lunny added the topic/build PR changes how Gitea is built, i.e. regarding Docker or the Makefile label Dec 10, 2017
@lunny lunny merged commit 23ffdf4 into go-gitea:master Dec 10, 2017
@sapk sapk deleted the update-alpine-37 branch December 10, 2017 10:29
@go-gitea go-gitea locked and limited conversation to collaborators Nov 23, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. topic/build PR changes how Gitea is built, i.e. regarding Docker or the Makefile
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants