search

asterisk / dahdi-linux

This is the official dahdi-linux repository. All issues and PR should be raised here.
GNU General Public License v2.0
47 stars 69 forks source link

GitHub ci #24

Closed duncanpatterson closed 1 year ago

duncanpatterson commented 1 year ago

This commit adds basic kernel compilation tests. This tests against

At present the tests are run on a Monday morning at 0200

InterLinked1 commented 1 year ago

At present the tests are run on a Monday morning at 0200

Hi Duncan, just one question, wouldn't it make more sense to:

a) Run the checks daily, rather than weekly? (The Asterisk repos run all the tests on a daily basis). b) Also run the CI against any PR that is submitted (not just on a schedule)

GitHub CI is free for public projects, so at least I'm not sure what the downside would be to more frequent tests.

duncanpatterson commented 1 year ago

Good morning

I agree. This is a work in progress at the moment as I learn how github actions work. There is more coming including mandatory checks on PRs and bundling everything into a new release.

The good news is that running these tests on my fork was successful and the master branch currently builds against all current kernel versions (with the work that Pushkar has been merging and a few extra bits) .

I'll be working on more over the next few days and any input is welcome.

Duncan. Duncan Patterson Director of Engineering

On Mon, 14 Aug 2023 at 17:07, InterLinked1 @.***> wrote:

At present the tests are run on a Monday morning at 0200

Hi Duncan, just one question, wouldn't it make more sense to:

a) Run the checks daily, rather than weekly? (The Asterisk repos run all the tests on a daily basis). b) Also run the CI against any PR that is submitted (not just on a schedule)

GitHub CI is free for public projects, so at least I'm not sure what the downside would be to more frequent tests.

— Reply to this email directly, view it on GitHub https://github.com/asterisk/dahdi-linux/pull/24#issuecomment-1677625504, or unsubscribe https://github.com/notifications/unsubscribe-auth/AKAPJQ36SIAA4YIGGIM2NC3XVJEM3ANCNFSM6AAAAAA3A4EK3U . You are receiving this because you modified the open/close state.Message ID: @.***>

InterLinked1 commented 1 year ago

Good morning I agree. This is a work in progress at the moment as I learn how github actions work. There is more coming including mandatory checks on PRs and bundling everything into a new release. The good news is that running these tests on my fork was successful and the master branch currently builds against all current kernel versions (with the work that Pushkar has been merging and a few extra bits) . I'll be working on more over the next few days and any input is welcome.

Thanks Duncan for the reply, and sounds good! Great to see everything is working here and thanks for your work on this, this is looking great so far!

My only other recommendation so far is I noticed that there are a bunch of merge commits in the Git history now (e.g. "Merge branch master into foobar" which makes it look a bit messy. I see the actual commits further down, but this adds some confusion. If you look at the asterisk repo, you don't see this although they might have a more sophisticated merge mechanism; but I think it is essentially just "merge and squash" in the GitHub merge UI, rather than the default option. If we could do that going forward, that would keep the history a bit cleaner as we do rely on that to be able to make sense of what and when things are changing.