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

Add InstanceName parameter #1088

Merged

Conversation

fd-jonathanlinn
Copy link
Contributor

Adds the ability to customise the Name tag on the EC2 instances spawned by the ASG, defaults to "buildkite-agent" for backwards compatibility.

Might want to add a MinLength/MaxLength to this, happy to do so based on preferences.

@moskyb
Copy link
Contributor

moskyb commented Jan 30, 2023

re: min and max length, i think we're fine to leave it as is and have that end of things enforced by EC2 - i'm sure that there are reasonable limits in place there. Given that the failure dynamics are largely the same - the cfn stack won't launch in either case - i don't think there's much point in enforcing it on our end 🤷

again though, keen to hear your thoughts on this :)

@fd-jonathanlinn
Copy link
Contributor Author

Yeah agreed on length we could remove those, was mostly trying to play it safe. If we end up referencing StackName it's a moot point since it won't launch like you said

@moskyb moskyb enabled auto-merge February 1, 2023 21:26
@moskyb moskyb merged commit c05953f into buildkite:master Feb 1, 2023
@fd-jonathanlinn fd-jonathanlinn deleted the feat/parameterise-ec2-instance-name branch March 13, 2023 09:55
@ChrisLane
Copy link

ChrisLane commented Mar 16, 2023

Thanks @fd-jonathanlinn! Somehow I didn't see your mention of my issue but I'm glad to see that I can do this now.

@moskyb could https://buildkite.com/docs/agent/v3/elastic-ci-aws/parameters be updated please?
I notice this option isn't mentioned in the docs and since I build the stack using AWS CDK, I don't often get to see the actual template parameters in CloudFormation UI.

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