binbashar / leverage

Binbash Leverage CLI intended to orchestrate Leverage Reference Architecture for AWS (www.binbash.co/leverage)
https://pypi.org/project/leverage/
Apache License 2.0
17 stars 2 forks source link

fix: 'AWSCLIContainer' object has no attribute 'common_conf #243

Closed rynkowsg closed 7 months ago

rynkowsg commented 7 months ago

Problem has been explained in the issue HERE.

What?

Why?

References

Before release

Review the checklist here

exequielrafaela commented 7 months ago

@rynkowsg Thanks a lot for this PR, we'll review it ASAP and keep you updated it.

rynkowsg commented 7 months ago

I tested this change before making PR and it solved my problem described in the issue.

rynkowsg commented 7 months ago

Regarding the failing test it looks like secrets were missing for this job for some reason. Looking at the log it looks aws configure doesn't get values. If they were present, they would be masked with * chars.

2024-02-06T14:05:59.3290702Z aws configure set aws_access_key_id  --profile bb-deploymaster
2024-02-06T14:05:59.3291399Z aws configure set aws_secret_access_key  --profile bb-deploymaster
2024-02-06T14:05:59.3292093Z aws configure set region us-east-1 --profile bb-apps-devstg-devops
2024-02-06T14:05:59.3292853Z aws configure set output json --profile bb-apps-devstg-devops
2024-02-06T14:05:59.3293640Z aws configure set role_arn arn:aws:iam:::role/DeployMaster --profile bb-apps-devstg-devops
2024-02-06T14:05:59.3294487Z aws configure set source_profile bb-deploymaster --profile bb-apps-devstg-devops