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

investigate bsd issue (refs #16282) #17801

Closed
wants to merge 11 commits into from

Conversation

timotheecour
Copy link
Member

@timotheecour timotheecour commented Apr 20, 2021

@Araq I'm trying to understand the BSD issue mentioned in #16282 (comment)

how do i make bsd CI run in this PR?

(BSD CI is running in #16282 but not in this PR which is identical except for last commit)

@Araq
Copy link
Member

Araq commented Apr 20, 2021

how do i make bsd CI run in this PR?

You cannot anymore. Only my PRs are checked for BSDs as Sourcehut complained about the tremendous resource consumptions that the Nim project causes. No, I'm not kidding.

@timotheecour
Copy link
Member Author

You cannot anymore. Only my PRs are checked for BSDs as Sourcehut complained about the tremendous resource consumptions that the Nim project causes. No, I'm not kidding.

I'm aware of that issue, but the fact that only 1 person can run BSD tests is not good; there's a huge difference between running bsd CI in all PR's on each commit, vs allowing an opt-in [ci bsd] or equivalent that would probably involve < 1% of all PRs/commits.

I'm fairly certain @ddevault would be ok with that? /cc @ddevault

@ddevault
Copy link

Hm, not really. [ci bsd] does not jive with how we designed builds.sr.ht. We could accomodate you on git.sr.ht, but probably not through the dispatch portal to github.

@timotheecour
Copy link
Member Author

timotheecour commented Apr 20, 2021

So how do we proceed on our end? All that's needed is an opt-in way to allow running CI against our builds.sr.ht pipelines; it's expected to not be frequently used so shouldn't result in large resource use.

If [ci bsd] doesn't work, how about a .builds/openbsd.yml.disabled that PR authors would temporarily enable while working on a fix involving BSD, and that would run whenver the .disabled extension is removed?

@ddevault
Copy link

It's not about the resource use, it's about the design and maintenance cost of an on-demand feature.

@ringabout
Copy link
Member

ringabout commented Apr 21, 2021

@timotheecour

Only my PRs are checked for BSDs

Not really, the members of Nim-lang can create a new branch, then the PR can be checked on BSDs too.

So you need to clone the nim-lang/Nim directly and make a new branch then push it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants