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

.github: deployment automation for data-plane-controller and agent-api #1768

Merged
merged 1 commit into from
Nov 15, 2024

Conversation

jgraettinger
Copy link
Member

@jgraettinger jgraettinger commented Nov 11, 2024

Introduce workflow_dispatch actions for building and deploying these control-plane services to Cloud Run, using workload identity federation.

Include Dockerfile infrastructure and entrypoints for placing secrets into exepected locations.

Minor tweaks to data-plane-controller, adjusting defaults to work better with Cloud Run and also to support IPv4 addresses for Ansible hosts, which is required due to our dependency on Cloud Run (which only supports IPv4, paired with Cloud NAT).

Fixes #1767

(Describe the high level scope of new or changed features)

Workflow steps:

(How does one use this feature, and how has it changed)

Documentation links affected:

(list any documentation links that you created, or existing ones that you've identified as needing updates, along with a brief description)

Notes for reviewers:

(anything that might help someone review this PR)


This change is Reviewable

Introduce workflow_dispatch actions for building and deploying these
control-plane services to Cloud Run, using workload identity federation.

Include Dockerfile infrastructure and entrypoints for placing secrets
into exepected locations.

Minor tweaks to data-plane-controller, adjusting defaults to work better
with Cloud Run and also to support IPv4 addresses for Ansible hosts,
which is required due to our dependency on Cloud Run (which only
supports IPv4, paired with Cloud NAT).
@jgraettinger jgraettinger added the change:planned This is a planned change label Nov 11, 2024
Copy link
Member

@mdibaiee mdibaiee left a comment

Choose a reason for hiding this comment

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

LGTM % one question and one small nit

@@ -0,0 +1,31 @@
FROM ubuntu:noble
Copy link
Member

Choose a reason for hiding this comment

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

Does the image here affect our costs in any noticable way? I'm wondering if we could save costs by using a slimmer image. I guess it depends on how frequently this image is going to be run by Cloud Run and how much time is taken by the image to boot up compared to other images 🤔

Copy link
Member Author

Choose a reason for hiding this comment

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

Not especially, but this isn't a huge image (50MB iirc?). I do think it's a good idea to use the same OS base as our GitHub runner environment.

set -o nounset

# Place secrets into expected file locations.
mkdir /root/.aws
Copy link
Member

Choose a reason for hiding this comment

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

We are not placing anything here, is this actually necessary to create here manually?

Copy link
Member Author

Choose a reason for hiding this comment

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

I left it because it's likely we will in the future, if we have agent APIs doing s3 operations (such as for updating storage mappings).

@jgraettinger jgraettinger merged commit dfc5b27 into master Nov 15, 2024
5 checks passed
@jgraettinger jgraettinger deleted the johnny/dpc-cd branch November 15, 2024 20:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
change:planned This is a planned change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

cloud run CD for data-plane-controller and agent-api
2 participants