Skip to content

Conversation

@rgibert
Copy link
Member

@rgibert rgibert commented Oct 18, 2024

K8s deploys want to run in parallel instead of the current serial deploys that we do for code. This adds a parallel flag to pipedream that defaults to false to maintain current behaviour but enables parallel execution of all pipelines when wanted.

This also standardizes the naming of the tests to be clear on serial vs parallel and adds vendor/ to gitignore so it doesn't get accidentally checked in.

K8s deploys want to run in parallel instead of the current serial deploys that we do for code.  This adds a parallel flag to pipedream that defaults to false to maintain current behaviour but enables parallel execution of all pipelines when wanted.
@rgibert rgibert requested a review from IanWoodard October 18, 2024 15:02
@rgibert rgibert self-assigned this Oct 18, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

how did we end up writing tests in javascript ? Seems a new level of inconsistency

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤷🏻 We'd need to ask Matt

@fpacifici
Copy link

Then looks good to me. Please fix the jsvscript format and I think it would be a good idea to get a review from @IanWoodard

@IanWoodard IanWoodard merged commit 4db1cd6 into main Oct 30, 2024
2 checks passed
@IanWoodard IanWoodard deleted the rgibert/add-parallel-pipedream-option branch October 30, 2024 17:19
github-merge-queue bot pushed a commit to getsentry/relay that referenced this pull request Oct 16, 2025
* Adds s4s2 as test region
* fix(tests): Fixing pipedream tests by @IanWoodard in
getsentry/gocd-jsonnet#81
* feat: Add support for doing pipedream stages in parallel by @rgibert
in getsentry/gocd-jsonnet#80
phacops pushed a commit to getsentry/snuba that referenced this pull request Oct 20, 2025
* Adds s4s2 as test region
* fix(tests): Fixing pipedream tests by @IanWoodard in
getsentry/gocd-jsonnet#81
* feat: Add support for doing pipedream stages in parallel by @rgibert
in getsentry/gocd-jsonnet#80
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