Merge pull request #7 from ryansb/develop
Context.exec_role_arn was returning the invoke role
Showing
3 changed files
with
4 additions
and
4 deletions
... | @@ -55,8 +55,8 @@ def main(config=None, debug=False, command=None): | ... | @@ -55,8 +55,8 @@ def main(config=None, debug=False, command=None): |
55 | ctx.delete() | 55 | ctx.delete() |
56 | click.echo('...done') | 56 | click.echo('...done') |
57 | elif command == 'add-event-source': | 57 | elif command == 'add-event-source': |
58 | - click.echo('Adding event source ...') | 58 | + click.echo('Adding event sources ...') |
59 | - ctx.add_event_source() | 59 | + ctx.add_event_sources() |
60 | click.echo('...done') | 60 | click.echo('...done') |
61 | 61 | ||
62 | 62 | ... | ... |
... | @@ -57,7 +57,7 @@ class Context(object): | ... | @@ -57,7 +57,7 @@ class Context(object): |
57 | 57 | ||
58 | @property | 58 | @property |
59 | def exec_role_arn(self): | 59 | def exec_role_arn(self): |
60 | - return self._stack.invoke_role_arn | 60 | + return self._stack.exec_role_arn |
61 | 61 | ||
62 | @property | 62 | @property |
63 | def invoke_role_arn(self): | 63 | def invoke_role_arn(self): | ... | ... |
... | @@ -89,7 +89,7 @@ class S3EventSource(EventSource): | ... | @@ -89,7 +89,7 @@ class S3EventSource(EventSource): |
89 | 'CloudFunctionConfiguration': { | 89 | 'CloudFunctionConfiguration': { |
90 | 'Id': self._make_notification_id(function.name), | 90 | 'Id': self._make_notification_id(function.name), |
91 | 'Events': [e for e in self._config['events']], | 91 | 'Events': [e for e in self._config['events']], |
92 | - 'CloudFunction': function.arn(), | 92 | + 'CloudFunction': function.arn, |
93 | 'InvocationRole': self._context.invoke_role_arn}} | 93 | 'InvocationRole': self._context.invoke_role_arn}} |
94 | try: | 94 | try: |
95 | response = self._s3.put_bucket_notification( | 95 | response = self._s3.put_bucket_notification( | ... | ... |
-
Please register or login to post a comment