Amazing job, thanks Ed! The new workflow has split the old check into a few smaller checks, which will be helpful for debugging. The total duration time for checks has been reduced from 2~3 hours to 35min~, which has a significant acceleration. We can now use tags like [skip ci] to skip the checks for some file changes, like /docs, which is greatly helpful.
More highlighted is the `Build Job Summary` report for an overview - https://github.com/apache/cloudberry/actions/runs/11925461028?pr=714. BTW, I bring up one idea here for your information. We can try to add the comments commands support, like `/ok-to-test` to enable running tests once the PR is ready, which can be triggered by committers or PR writers, `/merge`, `/rebase`, and more. See https://prow.k8s.io/command-help. Best, Dianjin Wang On Wed, Nov 20, 2024 at 1:25 PM Ed Espino <esp...@apache.org> wrote: > Hello Dev Cloudberries, > > I need two review approvals for PR #714 "ci: add core Apache Cloudberry > (incubating) build and test workflow" > <https://github.com/apache/cloudberry/pull/714>. This PR represents our > first step in re-establishing updated Continuous Integration (CI) for the > project. > > Please note: This is just the beginning. Over the coming weeks, we'll be > working to: > > - Improve overall test coverage > - Address disabled tests > - Re-introduce previous test schedules > - And more > > Your review would be greatly appreciated. Please do not hesitate to bring > up discussions around this and other developer and project topics. Your > input, no matter how large or small, will make the world of difference for > all of us. > Thank you, -=e > > > -- > Ed Espino > Apache Cloudberry (incubating) & MADlib >
