Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Run processes serially on Buildkite #343

Merged
merged 1 commit into from
Feb 2, 2021
Merged

Conversation

72636c
Copy link
Member

@72636c 72636c commented Feb 2, 2021

This is sad but it's not uncommon to run into issues with typical instance sizing at SEEK. This should also alleviate contention on the shared publishing agents.

If we want to be clever in future we could inspect BUILDKITE_AGENT_META_DATA_AWS_INSTANCE_TYPE.

@72636c 72636c requested a review from a team as a code owner February 2, 2021 22:54
@changeset-bot
Copy link

changeset-bot bot commented Feb 2, 2021

🦋 Changeset detected

Latest commit: 9d54a88

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
skuba Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@72636c
Copy link
Member Author

72636c commented Feb 2, 2021

Longer term I'd like to explore running these in-process rather than spawning child processes. Unfortunately these tools generally don't provide programmatic APIs that match the convenience of their CLIs.

@72636c 72636c merged commit 45ca22d into master Feb 2, 2021
@72636c 72636c deleted the not-concurrently-on-buildkite branch February 2, 2021 23:00
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.

None yet

2 participants