Grant(or remove) permission to lambda function for invoke from CloudWatch Events #4

unasuke · 2017-03-09T12:09:43Z

Goal

Invoke lambda function from CloudWatch Events

What should I to reach the goal?

Grant permission to lambda function for invoke from cloudwatch events when create a lambda function.
- http://docs.aws.amazon.com/AmazonCloudWatch/latest/events/RunLambdaSchedule.html
- When delete lambda function from rule target, should delete its permission.

check lambda function permission

to grant permission to invoke lambda from cloudwatch events

houndci-bot · 2017-03-09T12:09:52Z

structs.go

+	AwsSourceArn string `json:"AWS:SourceArn"`
+}
+
+type PolicyPrincipal struct {


exported type PolicyPrincipal should have comment or be unexported

houndci-bot · 2017-03-09T12:09:52Z

structs.go

+	ArnLike *PolicyArnLike `json:"ArnLike"`
+}
+
+type PolicyArnLike struct {


exported type PolicyArnLike should have comment or be unexported

houndci-bot · 2017-03-09T12:09:52Z

structs.go

+	Action      string           `json:"Action"`
+}
+
+type PolicyCondition struct {


exported type PolicyCondition should have comment or be unexported

houndci-bot · 2017-03-09T12:09:52Z

structs.go

+type PolicyStatement struct {
+	Resource    string           `json:"Resource"`
+	Condition   *PolicyCondition `json:"Condition"`
+	StatementId string           `json:"Sid"`


struct field StatementId should be StatementID

houndci-bot · 2017-03-09T12:09:52Z

structs.go

+	Statement *[]PolicyStatement `json:"Statement"`
+}
+
+type PolicyStatement struct {


exported type PolicyStatement should have comment or be unexported

houndci-bot · 2017-03-09T12:09:52Z

structs.go

+
+type LambdaPolicy struct {
+	Version   string             `json:"Version"`
+	Id        string             `json:"Id"`


struct field Id should be ID

houndci-bot · 2017-03-09T12:09:52Z

structs.go

@@ -30,3 +30,30 @@ type Target struct {
 	NeedUpdate   bool
 	NeedDelete   bool
 }
+
+type LambdaPolicy struct {


exported type LambdaPolicy should have comment or be unexported

houndci-bot · 2017-03-09T12:09:52Z

lambda.go

+	return strings.HasPrefix(arn, "arn:aws:lambda")
+}
+
+func LambdaFunctionNameFromArn(arn string) string {


exported function LambdaFunctionNameFromArn should have comment or be unexported

houndci-bot · 2017-03-09T12:09:52Z

lambda.go

+	return false, nil
+}
+
+func IsLambdaFunction(arn string) bool {


exported function IsLambdaFunction should have comment or be unexported

houndci-bot · 2017-03-09T12:26:29Z

structs.go

+	AwsSourceArn string `json:"AWS:SourceArn"`
+}
+
+// part of the LambdaPolicy


comment on exported type PolicyPrincipal should be of the form "PolicyPrincipal ..." (with optional leading article)

houndci-bot · 2017-03-09T12:26:29Z

structs.go

+	ArnLike *PolicyArnLike `json:"ArnLike"`
+}
+
+// part of the LambdaPolicy


comment on exported type PolicyArnLike should be of the form "PolicyArnLike ..." (with optional leading article)

houndci-bot · 2017-03-09T12:26:29Z

structs.go

+	Action      string           `json:"Action"`
+}
+
+// part of the LambdaPolicy


comment on exported type PolicyCondition should be of the form "PolicyCondition ..." (with optional leading article)

houndci-bot · 2017-03-09T12:26:29Z

structs.go

+type PolicyStatement struct {
+	Resource    string           `json:"Resource"`
+	Condition   *PolicyCondition `json:"Condition"`
+	StatementId string           `json:"Sid"`


struct field StatementId should be StatementID

houndci-bot · 2017-03-09T12:26:29Z

structs.go

+	Statement *[]PolicyStatement `json:"Statement"`
+}
+
+// part of the LambdaPolicy


comment on exported type PolicyStatement should be of the form "PolicyStatement ..." (with optional leading article)

houndci-bot · 2017-03-09T12:26:29Z

structs.go

+// struct for JSON that return from Lambda.GetPolicy
+type LambdaPolicy struct {
+	Version   string             `json:"Version"`
+	Id        string             `json:"Id"`


struct field Id should be ID

houndci-bot · 2017-03-09T12:26:29Z

structs.go

@@ -30,3 +33,35 @@ type Target struct {
 	NeedUpdate   bool
 	NeedDelete   bool
 }
+
+// struct for JSON that return from Lambda.GetPolicy


comment on exported type LambdaPolicy should be of the form "LambdaPolicy ..." (with optional leading article)

houndci-bot · 2017-03-09T12:26:29Z

structs.go

@@ -21,6 +23,7 @@ type Rule struct {
 	NeedDelete         bool
 }

+// struct for expression CloudWatch Events Target


comment on exported type Target should be of the form "Target ..." (with optional leading article)

houndci-bot · 2017-03-09T12:26:29Z

structs.go

 type Rules struct {
 	Rules []Rule
 }

+// struct for expression CloudWatch Events Rule


comment on exported type Rule should be of the form "Rule ..." (with optional leading article)

houndci-bot · 2017-03-09T12:26:29Z

structs.go

@@ -4,10 +4,12 @@ import (
 	cwe "github.com/aws/aws-sdk-go/service/cloudwatchevents"
 )

+// struct for store unmarshalized configuration yaml


comment on exported type Rules should be of the form "Rules ..." (with optional leading article)

houndci-bot · 2017-03-09T12:26:30Z

structs.go

+	AwsSourceArn string `json:"AWS:SourceArn"`
+}
+
+// part of the LambdaPolicy


comment on exported type PolicyPrincipal should be of the form "PolicyPrincipal ..." (with optional leading article)

houndci-bot · 2017-03-09T12:26:30Z

structs.go

+	ArnLike *PolicyArnLike `json:"ArnLike"`
+}
+
+// part of the LambdaPolicy


comment on exported type PolicyArnLike should be of the form "PolicyArnLike ..." (with optional leading article)

houndci-bot · 2017-03-09T12:26:30Z

structs.go

+	Action      string           `json:"Action"`
+}
+
+// part of the LambdaPolicy


comment on exported type PolicyCondition should be of the form "PolicyCondition ..." (with optional leading article)

houndci-bot · 2017-03-09T12:26:30Z

structs.go

+type PolicyStatement struct {
+	Resource    string           `json:"Resource"`
+	Condition   *PolicyCondition `json:"Condition"`
+	StatementId string           `json:"Sid"`


struct field StatementId should be StatementID

houndci-bot · 2017-03-09T12:26:30Z

structs.go

+	Statement *[]PolicyStatement `json:"Statement"`
+}
+
+// part of the LambdaPolicy


comment on exported type PolicyStatement should be of the form "PolicyStatement ..." (with optional leading article)

houndci-bot · 2017-03-09T12:26:30Z

structs.go

+// struct for JSON that return from Lambda.GetPolicy
+type LambdaPolicy struct {
+	Version   string             `json:"Version"`
+	Id        string             `json:"Id"`


struct field Id should be ID

houndci-bot · 2017-03-09T12:26:30Z

structs.go

@@ -30,3 +33,35 @@ type Target struct {
 	NeedUpdate   bool
 	NeedDelete   bool
 }
+
+// struct for JSON that return from Lambda.GetPolicy


comment on exported type LambdaPolicy should be of the form "LambdaPolicy ..." (with optional leading article)

houndci-bot · 2017-03-09T12:26:30Z

structs.go

@@ -21,6 +23,7 @@ type Rule struct {
 	NeedDelete         bool
 }

+// struct for expression CloudWatch Events Target


comment on exported type Target should be of the form "Target ..." (with optional leading article)

houndci-bot · 2017-03-09T12:26:30Z

structs.go

 type Rules struct {
 	Rules []Rule
 }

+// struct for expression CloudWatch Events Rule


comment on exported type Rule should be of the form "Rule ..." (with optional leading article)

houndci-bot · 2017-03-09T12:26:30Z

structs.go

@@ -4,10 +4,12 @@ import (
 	cwe "github.com/aws/aws-sdk-go/service/cloudwatchevents"
 )

+// struct for store unmarshalized configuration yaml


comment on exported type Rules should be of the form "Rules ..." (with optional leading article)

unasuke · 2017-03-09T12:31:42Z

😨

unasuke added 13 commits March 8, 2017 17:40

Create sessionfor lambda

9baa37f

Create structs for lambda policy

2bc637d

Create function IsLambdaFunction

de72dd6

Create function LambdaFunctionNameFromArn

6056865

Create client and store it

66d9cc9

Remame var cweRulesOutput -> cweRulesBeforeApply

9880afa

Reuse error var

a825822

Create function isAlreadyAddPermission

02a3781

check lambda function permission

Create function addPermissionToLambdaFromCloudWatchEvents

f24b263

to grant permission to invoke lambda from cloudwatch events

Add permission to lambda to invoke from CloudWatch Events

9cc3c3a

Return error information

520954a

Create func removePermissonFromLambda

ffc55e8

Remove permission from deleted lambda function

c2d9832

houndci-bot reviewed Mar 9, 2017

View reviewed changes

Add comment to structs

636fda2

houndci-bot reviewed Mar 9, 2017

View reviewed changes

unasuke merged commit 5c89471 into master Mar 9, 2017

unasuke deleted the lambda_permission branch March 9, 2017 12:32

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Grant(or remove) permission to lambda function for invoke from CloudWatch Events #4

Grant(or remove) permission to lambda function for invoke from CloudWatch Events #4

Grant(or remove) permission to lambda function for invoke from CloudWatch Events #4

Grant(or remove) permission to lambda function for invoke from CloudWatch Events #4

Conversation

Goal

What should I to reach the goal?

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment