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

Added BuildkiteAgentCancelGracePeriod option to linux stack #1258

Merged
merged 1 commit into from
Nov 7, 2023

Conversation

njgrisafi
Copy link
Contributor

Following expectations from docs here https://buildkite.com/docs/agent/v3/configuration

We should allow users to configure the cancellation grace period.

Added

  • BuildkiteAgentCancelGracePeriod to aws-stack.yml (default 60)

@njgrisafi njgrisafi force-pushed the configurable-cancel-grace-period branch from b8d1ca7 to 38f7a7e Compare November 3, 2023 20:10
@moskyb moskyb force-pushed the configurable-cancel-grace-period branch from a084bff to 054fec6 Compare November 7, 2023 22:23
@moskyb
Copy link
Contributor

moskyb commented Nov 7, 2023

@njgrisafi i've done a wee bit of massaging + rebasing on this PR - just fixing a typo. thanks for your contribution!

@moskyb moskyb merged commit a3de71d into buildkite:main Nov 7, 2023
1 check passed
@moskyb
Copy link
Contributor

moskyb commented Nov 7, 2023

worth noting: there's a difference between the windows and linux stacks here; linux stacks were defaulting to 60s for cancel-grace-period, but windows stacks didn't set it, which means that it would be the agent's default of 10s. i didn't wanna chuck it in there willy-nilly, so i'll add it in in another PR.

@njgrisafi njgrisafi deleted the configurable-cancel-grace-period branch November 8, 2023 21:41
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

3 participants