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.

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

sam deploy missing RegionParameter #4

Closed
DALDEI opened this issue Nov 24, 2018 · 4 comments
Closed

sam deploy missing RegionParameter #4

DALDEI opened this issue Nov 24, 2018 · 4 comments

Comments

@DALDEI
Copy link
Contributor

DALDEI commented Nov 24, 2018

running npm run deploy, there is an error when running sam deploy.
There needs be an additional parameter overide RegionParameter=$REGION, or a default set in the template.

Manually running the failed command with the region argument appears to work
(not tested yet, but the stack did run)

sam deploy --region us-west-2 --template-file /moji/github-cognito-openid-wrapper/scripts/../serverless-output.yml --stack-name xxxx --parameter-overrides GitHubClientIdParameter=xxx GitHubClientSecretParameter=xxxx CognitoRedirectUriParameter=xxxx RegionParameter=$REGION --capabilities CAPABILITY_IAM

@TimothyJones
Copy link
Owner

Ah, whoops. I have a default region set in my AWS config, which might be why it worked on mine. Thanks for the report!

@TimothyJones
Copy link
Owner

I see the region parameter there in the deploy script:

https://github.com/TimothyJones/github-cognito-openid-wrapper/blob/master/scripts/deploy.sh#L24

Am I missing somewhere else that I've missed the parameter?

@TimothyJones
Copy link
Owner

Closing because I think this is fixed, but please feel free to reopen if I missed something.

@TimothyJones
Copy link
Owner

I see what I missed now! Fixed by #10.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants