Skip to content

test/e2e-framework: support kind cluster topology and image customization#48684

Open
AlexanderYastrebov wants to merge 1 commit intomainfrom
alexander.yastrebov/test/e2e-framework/kind-customization
Open

test/e2e-framework: support kind cluster topology and image customization#48684
AlexanderYastrebov wants to merge 1 commit intomainfrom
alexander.yastrebov/test/e2e-framework/kind-customization

Conversation

@AlexanderYastrebov
Copy link
Copy Markdown
Contributor

What does this PR do?

Add support for customizing kind cluster topology and pre-loading Docker images into the cluster before the agent is deployed.

Motivation

Local e2e tests for Spot Scheduling #47429

Describe how you validated your changes

I am testing spot scheduling and extracted these chnages into a separate PR.

Additional Notes

@AlexanderYastrebov AlexanderYastrebov requested a review from a team as a code owner March 31, 2026 15:52
@dd-octo-sts dd-octo-sts bot added internal Identify a non-fork PR team/agent-devx labels Mar 31, 2026
@github-actions github-actions bot added the medium review PR review might take time label Mar 31, 2026
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 8c28176bd9

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

return fmt.Sprintf("kind load docker-image %s --name %s", img, name)
}).(pulumi.StringOutput),
},
utils.PulumiDependsOn(kindCluster),
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Re-run image preload when Kind cluster is replaced

The image preload commands only use DependsOn(kindCluster) for ordering, but they have no trigger tied to cluster recreation, so Pulumi treats them as unchanged when the Kind cluster is replaced with the same name (for example after topology/version changes or stack updates in dev mode). In that case the new cluster starts without the preloaded images, and agent pods configured with imagePullPolicy: Never can fail to start because the image load step is skipped.

Useful? React with 👍 / 👎.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Claude suggested to add

					// Trigger re-run when the cluster is recreated. KubeConfig changes on
					// every new cluster (new TLS certs), so this ensures the image is
					// reloaded into the replacement cluster even when the cluster name and
					// image haven't changed.
					Triggers: pulumi.Array{kindCluster.KubeConfig},

which I did.

@AlexanderYastrebov AlexanderYastrebov added qa/done QA done before merge and regressions are covered by tests changelog/no-changelog No changelog entry needed labels Mar 31, 2026
@agent-platform-auto-pr
Copy link
Copy Markdown
Contributor

agent-platform-auto-pr bot commented Mar 31, 2026

Files inventory check summary

File checks results against ancestor 082be224:

Results for datadog-agent_7.79.0~devel.git.311.4122e59.pipeline.105299919-1_amd64.deb:

No change detected

…tion

Add support for customizing kind cluster topology and pre-loading Docker
images into the cluster before the agent is deployed.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@AlexanderYastrebov AlexanderYastrebov force-pushed the alexander.yastrebov/test/e2e-framework/kind-customization branch from 8c28176 to 4122e59 Compare March 31, 2026 16:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog/no-changelog No changelog entry needed internal Identify a non-fork PR medium review PR review might take time qa/done QA done before merge and regressions are covered by tests team/agent-devx

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant