Skip to content

Conversation

@dkijania
Copy link
Member

@dkijania dkijania commented Dec 2, 2025

We hit maximum length of step when I added additional drityWhen filter to one of the jobs:

https://buildkite.com/o-1-labs-2/mina-o-1-labs/builds/39708#019ade81-a113-44fb-b6f6-23a8c004c234

My solution is to delegate logic which decide if job should be triggered or not based to bash script. Thanks to that we can shorten step code.

P.S. I choose bash as it is most natural and most commonly used in CI. If we would like to use different language we need to install it on toolchain first etc. Currently its to big lift .

@dkijania dkijania requested a review from a team as a code owner December 2, 2025 13:35
@dkijania
Copy link
Member Author

dkijania commented Dec 2, 2025

!ci-build-me

6 similar comments
@dkijania
Copy link
Member Author

dkijania commented Dec 2, 2025

!ci-build-me

@dkijania
Copy link
Member Author

dkijania commented Dec 2, 2025

!ci-build-me

@dkijania
Copy link
Member Author

dkijania commented Dec 2, 2025

!ci-build-me

@dkijania
Copy link
Member Author

dkijania commented Dec 2, 2025

!ci-build-me

@dkijania
Copy link
Member Author

dkijania commented Dec 2, 2025

!ci-build-me

@dkijania
Copy link
Member Author

dkijania commented Dec 2, 2025

!ci-build-me

@dkijania dkijania force-pushed the dkijania/move_triaging_to_bash branch from 4dcc752 to a6aa7c1 Compare December 2, 2025 15:18
@dkijania
Copy link
Member Author

dkijania commented Dec 2, 2025

!ci-build-me

@dkijania
Copy link
Member Author

dkijania commented Dec 2, 2025

!ci-nightly-me

@dkijania
Copy link
Member Author

dkijania commented Dec 2, 2025

@SanabriaRusso
Copy link
Member

Nightly CI: https://buildkite.com/o-1-labs-2/mina-end-to-end-nightlies/builds/4021

Waiting for nightly

@dkijania dkijania force-pushed the dkijania/move_triaging_to_bash branch from a6aa7c1 to 3abda6b Compare December 3, 2025 18:41
@dkijania
Copy link
Member Author

dkijania commented Dec 3, 2025

!ci-build-me

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.

3 participants