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

Feature request - support --profile (consistency with aws cli) #503

Closed
eranation opened this issue Aug 3, 2018 · 4 comments · Fixed by #523
Closed

Feature request - support --profile (consistency with aws cli) #503

eranation opened this issue Aug 3, 2018 · 4 comments · Fixed by #523

Comments

@eranation
Copy link

In order to switch a profile, in aws cli one can either use environment variables or use the --profile parameter. For consistency it would be nice if cdk cli would have the same support.

@rix0rrr
Copy link
Contributor

rix0rrr commented Aug 6, 2018

Duplicate of #480.

@eladb eladb mentioned this issue Aug 8, 2018
@rix0rrr rix0rrr closed this as completed Aug 8, 2018
RomainMuller pushed a commit that referenced this issue Aug 8, 2018
* Bump
* Added a `bump.sh` script ;-)

Fixes #467 (via aws/jsii#139)
Fixes #503 (via #517)
Fixes #502 (via #490)
@kennu
Copy link

kennu commented Sep 3, 2018

How do you use this option? These commands both result in the default profile being quietly used instead of the specified my-profile:

cdk --profile my-profile deploy
cdk deploy --profile my-profile

Meanwhile this works using the same profile:

AWS_PROFILE=my-profile cdk deploy

Version: 0.8.2 (build 934ca5f)

@rix0rrr
Copy link
Contributor

rix0rrr commented Nov 26, 2018

Can you post a trace of running with -v?

@kennu
Copy link

kennu commented Dec 2, 2018

@rix0rrr I tested now with AWS CDK version 0.18.1 (build 9f7af21) and it's working correctly with cdk --profile my-profile, thanks.

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

Successfully merging a pull request may close this issue.

3 participants