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

Allow custom tags to be added to EC2 instances (in AWS) #15

Open
brettcave opened this issue Jul 12, 2020 · 2 comments
Open

Allow custom tags to be added to EC2 instances (in AWS) #15

brettcave opened this issue Jul 12, 2020 · 2 comments

Comments

@brettcave
Copy link
Contributor

The plugin sets some default tags on instances. Allow additional custom tags to be set.

@brettcave brettcave changed the title Allow custom tags to be added to instances Allow custom tags to be added to EC2 instances (in AWS) Jul 12, 2020
@brettcave
Copy link
Contributor Author

brettcave@25c93bf allows custom tags. There's no test for this and no strong input validation though.

@GregYeo
Copy link

GregYeo commented Aug 5, 2020

brettcave/GoCD-EC2-Elastic-Agent-Plugin@25c93bf LGTM.

I'd recommend you'd make Pull Request instead of putting Issue to be progressed.

Tags are important resource for big size company. They are used for checking billing and grouping resource for DevOps.
In terms of the operation, manageable tags are great idea.

If you could, I'd like to attach additional controlling for the tags that basic tags below

https://github.com/brettcave/GoCD-EC2-Elastic-Agent-Plugin/blob/25c93bf684bdde6f0fe9f969b66e77b09c06d022/src/main/java/com/continuumsecurity/elasticagent/ec2/Ec2Instance.java#L148-L157

  • tagName
  • tagType
  • tagPipelineName
  • tagPipelineCounter
  • tagPipelineLabel
  • tagStageName
  • tagStageCounter
  • tagJobName
  • tagJobId
  • tagJsonJobIdentifier

are overwritable by adding same tag key in Input Form or removable by selecting Checkbox.
I'm not sure your code is already able to overwrite. I did not confirm it.

Sometimes, Unnecessary tags or uncontrollable tags are increasing management cost.

Although you do not implement it, i think brettcave/GoCD-EC2-Elastic-Agent-Plugin@25c93bf is still nice implementation.

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

No branches or pull requests

2 participants