Hi, On 2022-11-19 14:22:20 -0600, Justin Pryzby wrote: > On Sun, Oct 02, 2022 at 01:52:01PM -0700, Andres Freund wrote: > > > > +# To avoid unnecessarily spinning up a lot of VMs / containers for entirely > > +# broken commits, have a very minimal test that all others depend on. > > +task: > > + name: SanityCheck > > Maybe this should be named 00-SanityCheck, so it sorts first in cfbot ?
Hm. Perhaps cfbot could just use the sorting from cirrus? I don't really like the idea of making the names more confusing with numbered prefixes, particularly when only used for some but not other tasks. > Also, if CompilerWarnings doesn't depend on Linux, that means those two > tasks will normally start and run simultaneously, which means a single > branch will use all 8 of the linux CPUs available from cirrus. Is that > intentional? I don't think that'd really make anything worse. But perhaps we could just reduce the CPU count for linux autoconf by 1? I suspect that even with asan enabled it'd still be roughly even with the rest. I'll try to repost a version of the ubsan/asan patch together with the sanitycheck patch and see how that looks. Greetings, Andres Freund
