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.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add kinesis video service modules #8278

Merged
merged 2 commits into from Apr 12, 2019
Merged

Add kinesis video service modules #8278

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
ac667f4
Add kinesis video service modules
teraken0509 Apr 11, 2019
1efd830
Add kinesisvideo to the available customizable service names
teraken0509 Apr 12, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
3 changes: 3 additions & 0 deletions aws/config.go
View file
Open in desktop
Expand Up @@ -74,6 +74,7 @@ import (
"github.com/aws/aws-sdk-go/service/kinesis"
"github.com/aws/aws-sdk-go/service/kinesisanalytics"
"github.com/aws/aws-sdk-go/service/kinesisanalyticsv2"
"github.com/aws/aws-sdk-go/service/kinesisvideo"
"github.com/aws/aws-sdk-go/service/kms"
"github.com/aws/aws-sdk-go/service/lambda"
"github.com/aws/aws-sdk-go/service/lexmodelbuildingservice"
Expand Down Expand Up @@ -223,6 +224,7 @@ type AWSClient struct {
kinesisanalyticsconn *kinesisanalytics.KinesisAnalytics
kinesisanalyticsv2conn *kinesisanalyticsv2.KinesisAnalyticsV2
kinesisconn *kinesis.Kinesis
kinesisvideoconn *kinesisvideo.KinesisVideo
kmsconn *kms.KMS
lambdaconn *lambda.Lambda
lexmodelconn *lexmodelbuildingservice.LexModelBuildingService
Expand Down Expand Up @@ -394,6 +396,7 @@ func (c *Config) Client() (interface{}, error) {
kinesisanalyticsconn: kinesisanalytics.New(sess.Copy(&aws.Config{Endpoint: aws.String(c.Endpoints["kinesisanalytics"])})),
kinesisanalyticsv2conn: kinesisanalyticsv2.New(sess.Copy(&aws.Config{Endpoint: aws.String(c.Endpoints["kinesisanalytics"])})),
kinesisconn: kinesis.New(sess.Copy(&aws.Config{Endpoint: aws.String(c.Endpoints["kinesis"])})),
kinesisvideoconn: kinesisvideo.New(sess.Copy(&aws.Config{Endpoint: aws.String(c.Endpoints["kinesisvideo"])})),
kmsconn: kms.New(sess.Copy(&aws.Config{Endpoint: aws.String(c.Endpoints["kms"])})),
lambdaconn: lambda.New(sess.Copy(&aws.Config{Endpoint: aws.String(c.Endpoints["lambda"])})),
lexmodelconn: lexmodelbuildingservice.New(sess.Copy(&aws.Config{Endpoint: aws.String(c.Endpoints["lexmodels"])})),
Expand Down
1 change: 1 addition & 0 deletions aws/provider.go
View file
Open in desktop
Expand Up @@ -910,6 +910,7 @@ func init() {
"kinesis_analytics",
"kinesis",
"kinesisanalytics",
"kinesisvideo",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you please also add this to the available customizable service names in website/docs/guides/custom-service-endpoints.html.md? Thanks so much.

The CONTRIBUTING documentation surrounding this should be available soon: https://github.com/terraform-providers/terraform-provider-aws/pull/8277/files#diff-8cc7b2b0d78dd2501610391c086a8516R288

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bflad Thanks. Fixed it.

"kms",
"lambda",
"lexmodels",
Expand Down