Michael Barrett
Michael Barrett
Ok, I've done a little bit of cleaning up - but I think this is close. @GarisonLotus can you add some tests for setting tags with the new config setting?
Thanks for this - I'm not sure how I didn't notice it till now. Let me take a look - there's a merge conflict, but if everything looks good I'll...
Ok, I took a look - one of the things I think would be good is if it only did the 'tail' of events if you added that flag. It...
Ahh, ok - I did misunderstand. I think the last option is the right one, just so we don't change the current functionality on anyone. Sorry for taking so long...
Hey @etuttle - we actually had someone ask about this change on the cloudtools-dev mailing list. I realize this probably fell off your plate in all this time, but just...
NP Ethan - if it proves to be too much of a pain let me know and I'll see what I can do. Thanks!
This looks good from my first pass - I like the way it's laid out. @ejholmes can you take a quick look at the plan/graph stuff, just to be sure...
Hey @xiaket - thanks for this PR! A couple of things: 1. The reason you are getting the error in tests is because your stacker fork master branch is very...
Hey @xiaket - sorry for not getting back to you sooner. This PR actually made me realize that we potentially have a bit bigger of an issue - mostly that...
Thanks @gabriellhrn - can you give an idea of how you would expect this to work in stacker? Would it be a configuration setting on a stack by stack basis?...