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

Adds Docker Build provider examples using Docker Build Cloud #1633

Merged
merged 12 commits into from
May 13, 2024

Conversation

desteves
Copy link
Contributor

@desteves desteves commented May 7, 2024

Adds two new examples using the Docker Build provider to build a Docker image using Docker Build Cloud (DBC) in Typescript.

dockerbuildcloud-ts - Builds a Docker Image from an NGINX local Dockerfile. This template prompts the user for an existing DBC builder.

aws-ts-containers-dockerbuildcloud - This is a slight variation of the existing aws-ts-containers but with the new Docker Build provider and Docker Build Cloud.

These examples are (will be) referenced in an upcoming blogpost.

@desteves desteves marked this pull request as ready for review May 7, 2024 21:22
@desteves desteves changed the title Adds a Docker Build provider example Adds a Docker Build provider examples May 8, 2024
@desteves desteves changed the title Adds a Docker Build provider examples Adds Docker Build provider examples using DBC May 8, 2024
@desteves desteves requested review from blampe, tejavar, isaac-pulumi, a team and jkodroff May 8, 2024 18:39
Copy link
Member

@jkodroff jkodroff left a comment

Choose a reason for hiding this comment

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

Suggest renaming aws-ts-containers-dbc aws-ts-ecr-dockerbuild or something? The Docker Build provider part needs to pop IMO since that's the key technology and I'm not sure that dbc is something people would know.

Leaving this review as comment in case I'm off-base with the request change to remove CTAs.

aws-ts-containers-dbc/index.ts Outdated Show resolved Hide resolved
aws-ts-containers-dbc/README.md Outdated Show resolved Hide resolved
aws-ts-containers-dbc/README.md Outdated Show resolved Hide resolved
aws-ts-containers-dbc/README.md Outdated Show resolved Hide resolved
aws-ts-containers-dbc/README.md Outdated Show resolved Hide resolved
dockerbuild-ts-dbc/README.md Outdated Show resolved Hide resolved
dockerbuild-ts-dbc/README.md Outdated Show resolved Hide resolved
dockerbuild-ts-dbc/README.md Outdated Show resolved Hide resolved
dockerbuild-ts-dbc/index.ts Outdated Show resolved Hide resolved
dockerbuild-ts-dbc/package.json Outdated Show resolved Hide resolved
desteves and others added 3 commits May 10, 2024 13:10
Co-authored-by: Josh Kodroff <jkodroff@users.noreply.github.com>
Co-authored-by: Josh Kodroff <jkodroff@users.noreply.github.com>
Co-authored-by: Josh Kodroff <jkodroff@users.noreply.github.com>
@desteves
Copy link
Contributor Author

desteves commented May 10, 2024

re - aws-ts-containers-dbc

This was to align with the current aws-ts-containers but yes I agree dbc is a rather unknown abbreviation. I'm hesitant about aws-ts-ecr-dockerbuild because it also deploys a fargate service. Maybe aws-ts-serverless-dockerbuildcloud ?

I went with aws-ts-containers-dockerbuildcloud (I'm not in love with it, but it's to align with current examples.)

@desteves desteves changed the title Adds Docker Build provider examples using DBC Adds Docker Build provider examples using Docker Build Cloud May 10, 2024
@desteves desteves requested a review from jkodroff May 10, 2024 21:52
Copy link
Member

@jkodroff jkodroff left a comment

Choose a reason for hiding this comment

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

Don't block on my naming feedback.

@desteves desteves merged commit 5817bbe into master May 13, 2024
50 of 51 checks passed
@desteves desteves deleted the dockerbuild branch May 13, 2024 18:07
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 this pull request may close these issues.

None yet

2 participants