Initial commit of the AWS CDK Python Quickstarter #956
Conversation
nichtraunzer
requested review from
metmajer,
gerardcl,
tbugfinder and
BraisVQ
nichtraunzer
changed the title
nichtraunzer
changed the title
| @@ -2,6 +2,8 @@ | |||
|
|
|||
| ### Added | |||
|
|
|||
| - inf-cdk-python-aws, provide first iteration of the AWS CDK Quickstarter ([#956](https://github.com/opendevstack/ods-quickstarters/pull/956)) | |||
There was a problem hiding this comment.
just wondering about the naming:
There's also a terraform cdk - how would the name of that be
inf-terraform-cdk-python?
Based on that, this one should be
inf-aws-cdk-python :-D
There was a problem hiding this comment.
actually someone else requested to remove "aws" because its evident ... but I am open to add that again :)
There was a problem hiding this comment.
as mentioned, there's also terraform cdk so it's not evident, imho :-D
| library("ods-jenkins-shared-library@${sharedLibraryRef}") | ||
|
|
||
| odsQuickstarterPipeline( | ||
| imageStreamTag: "${odsNamespace}/jenkins-agent-terraform2:${agentImageTag}", |
There was a problem hiding this comment.
Is this agent picked by design (just asking)?
There was a problem hiding this comment.
since the agent already contains all required libs and tools - yes.
other ideas like a dedicated agent are welcome too ...
| @@ -0,0 +1,5 @@ | |||
| # CDK Python Quickstarter (inf-cdk-python-aws) | |||
| @@ -0,0 +1,62 @@ | |||
| # ODS AWS CDK Python Quickstarter | |||
There was a problem hiding this comment.
here it is AWS CDK Python ....
| If you want to develop locally, make sure those tools are available in your environment, too. To run your code locally execute the following commands: | ||
|
|
||
| ``` | ||
| $ python -mvenv .venv |
There was a problem hiding this comment.
should this be aligned and use Makefile -> make init?
There was a problem hiding this comment.
interesting topic - originally i proposed that but some colleagues do not like a make target approach ...
There was a problem hiding this comment.
should this be aligned and use Makefile -> make init?
Personally I don't like the Makfile approach. It is hard to have that available on a local machine.
interesting topic - originally i proposed that but some colleagues do not like a make target approach ...
Guilty as charged!
| $ cd src | ||
| $ source ../.venv/bin/activate && pip install -r requirements.txt -r requirements-dev.txt | ||
| ``` | ||
| You may want to to verify you code by running ... |
There was a problem hiding this comment.
typo:
You may want to verify your code by running:
| ``` | ||
| $ export AWS_ACCESS_KEY_ID=... | ||
| $ export AWS_SECRET_ACCESS_KEY=... | ||
| $ export AWS_DEFAULT_REGION=us-east-1 |
There was a problem hiding this comment.
I'd use eu-west-1 as the defaults above also direct to eu-west-1.
| @@ -0,0 +1,7 @@ | |||
| --- | |||
| name: AWS CDK (Python) | |||
There was a problem hiding this comment.
here it is AWS CDK (Python)
[Feature] Add Quickstarter for Python CDK (MVP)
Description:
This pull request introduces a new quickstarter called "Quickstarter for Python CDK". The Quickstarter for Python CDK simplifies the process of deploying cloud resources using the AWS Cloud Development Kit (CDK) for Python. It is supposed to be an addition to the existing inf-terraform-aws Quickstarter. CDK Python now offers a a familiar programming language, whereas Terraform is using a declarative language.
This is the first iteration of this long discussed and awaited component, and we're excited to share it with the community.
Key Features:
Dependencies:
Since this PR is an MVP we're looking forward to receiving feedback and suggestions from the community to help improve the Quickstarter for Python CDK. Please feel free to review the changes and provide any comments or recommendations.
Open Topics:
Contributors: