Re: Proposal: Limiting builds for branches pushed to ceph-ci

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Sounds like a great idea Eric! Greg, Eric's already made a change in the ceph-build PR linked, that's where this logic occurs.

A yaml file in the branch seems like a reasonable way to go.

On Thu, Feb 2, 2023 at 9:40 AM Gregory Farnum <gfarnum@xxxxxxxxxx> wrote:
This seems like a great idea.

...I don't actually know what bit of code handles these builds,
though. I thought it was shaman, but looking at the github repo I'm
not sure?
-Greg

On Thu, Feb 2, 2023 at 9:34 AM J. Eric Ivancich <ivancich@xxxxxxxxxx> wrote:
>
> There are occasions when I would like a branch pushed to ceph-ci to only trigger one build, say centos9+x86_64. I created a PR for that specific case[1], but I’m wondering how feasible it would be to create a more generalized solution. It feels wasteful to launch, say, 7 builds when you’re only interested in 1, and it may slow down other developers.
>
> My thoughts aren’t fully concrete yet, in part I don’t the details of the code that interprets those .yml files, but here’s a sketch…
>
> 1. Those .yml files look for a tag in the branch name along the lines of “build-file” using the existing regex mechanism already leveraged in ceph-dev-new-trigger.yml.
>
> 2. When that tag is found in the branch, use wget to pull a specific file from the top-level of that branch. Say the file is called “ceph-build.config”, I believe a command along the lines of the following would pull that one file without cloning the whole repository:
>
>         wget https://raw.githubusercontent.com/ceph/ceph-ci/${GIT_BRANCH}/ceph-build.config
>
> 3. That file would then define the values of DISTROS, ARCHS, FLAVOR, and maybe also FORCE. BRANCH would be defined by the existing variable ${GIT_BRANCH} variable. I don’t know which format for that file would be easiest — .yml, key/value pairs, etc.
>
> 4. The build process would then continue as it does now.
>
> Perhaps there are hurdles that I’m not seeing. But I thought it’d be worth putting the idea out there. Thanks for considering,
>
> Eric
>
> [1] https://github.com/ceph/ceph-build/pull/2104
> _______________________________________________
> Dev mailing list -- dev@xxxxxxx
> To unsubscribe send an email to dev-leave@xxxxxxx
_______________________________________________
Dev mailing list -- dev@xxxxxxx
To unsubscribe send an email to dev-leave@xxxxxxx
_______________________________________________
Dev mailing list -- dev@xxxxxxx
To unsubscribe send an email to dev-leave@xxxxxxx

[Index of Archives]     [CEPH Users]     [Ceph Devel]     [Ceph Large]     [Information on CEPH]     [Linux BTRFS]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux