-
-
Notifications
You must be signed in to change notification settings - Fork 375
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
Feature/debianize #1339
Feature/debianize #1339
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #1339 +/- ##
==========================================
+ Coverage 74.51% 74.87% +0.36%
==========================================
Files 150 150
Lines 16994 16999 +5
==========================================
+ Hits 12663 12728 +65
+ Misses 3322 3258 -64
- Partials 1009 1013 +4
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
ce5334d
to
c533df8
Compare
01917ea
to
a5a0b56
Compare
I have a few commits in https://github.com/iofq/aptly master branch that may fit well. Namely added release workflow to gitlab CI (which means we can completely remove goxc as I think it wasn't quite working for this anyway) |
oh, great. goxc is still in bcs of the MacOS, FreeBSD and legacy linux builds... |
Yep here is an example release: https://github.com/iofq/aptly/releases/tag/v1.15.0 Still cleaning it up and trying to figure out a changelog |
bf86188
to
8ea4c6a
Compare
3cffdd4
to
ed02dab
Compare
config rfiles are read in the following order: 1) ~/.aptly.conf 2) /usr/local/etc/aptly.conf 3) /etc/aptly.conf
gpgv: can't allocate lock for '/home/runner/.gnupg/aptlytest.gpg' this forced running local system tessts in /home/runner, as it is in the gitgub actions.
- abort docker scripts on error - generate version in system tests - build debian packages in docker - add make clean target - fix lint
68b8bc3
to
ab243ca
Compare
ab243ca
to
1d80214
Compare
I think CI and release build are OK now :) However, since this is a PR and not on master branch, could you maybe test in your iofq:master branch if the correct version is built, debian release upload and gh-release are running ? Thanks in advance ! |
6d4aae2
to
abe981e
Compare
disable tests if env secrets are empty - detect emtpy aws token - upload: check for aptly creds
abe981e
to
cb82b2c
Compare
606ddee
to
a0eadfe
Compare
a0eadfe
to
c20568f
Compare
thanks a lot ! I added some more checks to not publish a release version to ci and prod... please approve, then I can merge and update documentation 👍 |
this should not build release if pipeline triggered on master and master also has a version tag. avoid building same version twice and uploading to ci and release repos.
32016aa
to
dc7fa1c
Compare
Fixes #1199, #960, #1108, #1287, #934, #425
Description of the Change
Debianize aptly and be compatible with official debian package.
This PR improved building and packaging and also improvides the apt repository structure
APT Repositories
Aptly is now available on repo.aptly.info for:
Repo Key
New CI repo:
New Release repo:
TODO