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 instance tag support #173

Closed
wants to merge 2 commits into from
Closed

Conversation

deoxxa
Copy link
Contributor

@deoxxa deoxxa commented Aug 9, 2014

This relies on mitchellh/goamz#77 and adds the ability to tag instances. I haven't gotten the update part happening yet, but I figured I'd PR this anyway as it's useful as-is.

@deoxxa
Copy link
Contributor Author

deoxxa commented Aug 9, 2014

This is related to #28 as well.

@deoxxa deoxxa mentioned this pull request Aug 9, 2014
@deoxxa
Copy link
Contributor Author

deoxxa commented Aug 10, 2014

I implemented a concept mentioned in #87, where I'm sorting the tags to ensure a consistent order after refresh or apply. One thing with this approach is that tags have to be specified in alphabetical key order in the config... This sounds like a bad thing long-term - please do let me know if there are plans to handle sets/arrays differently in the future.

@deoxxa
Copy link
Contributor Author

deoxxa commented Aug 10, 2014

This is also dependent on mitchellh/goamz#79 now, as I had to add a DeleteTags method to goamz.

add update functionality, ensure canonical ordering

document instance tags

use base tag functions in aws_instance
@deoxxa
Copy link
Contributor Author

deoxxa commented Aug 10, 2014

I rebased and squashed some stuff to make this easier to merge, and to make it easier to merge it separately from any other taggable things that I'll be PR'ing soon.

@deoxxa deoxxa mentioned this pull request Aug 10, 2014
@lgtml
Copy link

lgtml commented Aug 19, 2014

I am using in conjunction with #175 this works pretty good, updates tags some times but that is not offensive.

Tags make terraform usable for me, great work

@blackjid
Copy link

👍

@pearkes
Copy link
Contributor

pearkes commented Aug 20, 2014

Awesome! So, this is still waiting on mitchellh/goamz#79. Also, @mitchellh is working on improvements to handle the ordering of sets. This will allow us to skip the alphabetical solution and let the user specify any order. I would say we should probably wait and adapt for that. I'll let Mitchell describe how to do that once it's implemented.

@pearkes
Copy link
Contributor

pearkes commented Sep 16, 2014

Ditto as this comment, same situation.

Tags are still very important!

@pearkes pearkes closed this Sep 16, 2014
@ghost ghost locked and limited conversation to collaborators May 5, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants