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

Building with docker #125

Merged
merged 6 commits into from
Feb 10, 2019
Merged

Building with docker #125

merged 6 commits into from
Feb 10, 2019

Conversation

jmattheis
Copy link
Member

@jmattheis jmattheis commented Feb 9, 2019

The arm7 docker image wasn't really used and the base image gotify/server used got deprecated.

@eternal-flame-AD Could you have a look at this PR and gotify/website#6?

@jmattheis
Copy link
Member Author

As for the template, it will only be possible to build plugins against the master branch of server. I think we should support building against a tag, or even a commit. You can see the .travis.yml on my netlify repo for an example implementation.

#110 (comment)

Do you mean this? Currently this has only impact on the golang version. What exactly do you mean with building agains a tag/commit. That the gomod-cap check will be executed with the go.mod files from these commits?

@codecov
Copy link

codecov bot commented Feb 9, 2019

Codecov Report

Merging #125 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff          @@
##           master   #125   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files          39     39           
  Lines        1400   1400           
=====================================
  Hits         1400   1400

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 a1204a5...6e668cc. Read the comment docs.

@jmattheis jmattheis added this to the v1.3.0 milestone Feb 9, 2019
@eternal-flame-AD
Copy link
Member

Do you mean this? Currently this has only impact on the golang version. What exactly do you mean with building agains a tag/commit. That the gomod-cap check will be executed with the go.mod files from these commits?

Yes.

Could you have a look at this PR and gotify/website#6?

Okay, I will take a close look at that and leave my review tomorrow :)

@jmattheis
Copy link
Member Author

Thanks! Would you mind if I add you as a outside collaborator? Then I could add you as reviewer.

With the plugin-template something like this should already be possible:

$ make GOTIFY_VERSION=v1.2.1 check

Is that what you meant?`

@eternal-flame-AD
Copy link
Member

eternal-flame-AD commented Feb 9, 2019

Thanks! Would you mind if I add you as a outside collaborator? Then I could add you as reviewer.

That sounds great:) Thanks

With the plugin-template something like this should already be possible:

Let's track this in gotify/plugin-template#2

Yeah it does have a check-go-mod that checks the dependency against a server version, but it cannot update go.mod automatically to fit another version of server, so practically using only prepared commands in Makefile you cannot update go.mod to fit another server version in the build script other than what was committed in the VCS. In my netlify repo .
travis.yml loops through a list of predefined targets and updates go.mod for each of them before building for that target. The workflow looks like this:

make GOTIFY_VERSION=v1.3.0 update-go-mod
make build # build for 1.3.0
make GOTIFY_VERSION=v1.4.0 update-go-mod
make build # build for 1.4.0

Copy link
Member

@eternal-flame-AD eternal-flame-AD left a comment

Choose a reason for hiding this comment

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

Sounds great to me:)

Since go 1.12 is very likely to be coming out this week I am considering whether to use it for the 1.3.0

@jmattheis
Copy link
Member Author

In gotify/build@43beb35 I've added automatic building of gotiy/build docker images. They now are tagged like this gotify/build:{GO_VERSION}-{GOOS}-{GOARCH}

@jmattheis jmattheis merged commit 09c1516 into master Feb 10, 2019
@jmattheis jmattheis deleted the building-with-docker branch February 10, 2019 18:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants