-
Notifications
You must be signed in to change notification settings - Fork 24
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
Improve AWS resource tags handling #1592
Conversation
I thought you wanted the tags in |
@drj11 Yes, this is just a quick fix mainly to enable easier handling of the tags |
looks good, but I haven't tried launching one. I'll do that now. |
raise ConfigError() | ||
|
||
if 'owner' not in tags: | ||
tags['owner'] = os.popen("git config --get user.email").read().rstrip() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like this.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks but I copied this from your code :)
|
||
department: dbmi | ||
environment: dev | ||
project: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
an empty project currently means that using this template without changing it fails.
I think it would be nice if was filled in with something so that it can be used without changing it.
But then I was thinking that maybe it's a good thing that you have to change it.
The error is obscure though:
$ make web-stack
python stack.py > web.json
Configuration saved to s3://refinery-gygexukbykxpm/refinery-config-20170126T1457
TZ=UTC date +refinery-web-%Y%m%dT%H%M > aws-config/stack-name
aws cloudformation create-stack --capabilities CAPABILITY_IAM --stack-name $(cat aws-config/stack-name) --template-body file://web.json
A client error (ValidationError) occurred when calling the CreateStack operation: [/Resources/RDSInstance/Type/Tags/0/Value] 'null' values are not allowed in templates
Makefile:7: recipe for target 'web-stack' failed
make: *** [web-stack] Error 255
launched one. works. Use your skill and judgement to address my comments (I'm happy if you decide to change nothing), then merge. |
No description provided.