Skip to content

Conversation

@Supreeth095
Copy link
Contributor

What type of PR is this?

Which issue does this PR fix?:

What does this PR do / Why do we need it?:

Testing done on this change:

Will this PR introduce any new dependencies?:

Will this break upgrades or downgrades? Has updating a running cluster been tested?:

Does this change require updates to the CNI daemonset config files to work?:

Does this PR introduce any user-facing change?:


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@Supreeth095 Supreeth095 requested a review from a team as a code owner August 15, 2025 07:28
instanceType: c5.xlarge
desiredCapacity: 3
amiFamily: UbuntuPro2404
minSize: 3
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: not sure why set min as max here?

@jaydeokar
Copy link
Collaborator

Closing this as we have an open PR

@jaydeokar jaydeokar closed this Aug 20, 2025
@Supreeth095 Supreeth095 reopened this Oct 2, 2025
START=$SECONDS

GINKGO_TEST_BUILD="$SCRIPT_DIR/../test/build"
TEST_IMAGE_REGISTRY=${TEST_IMAGE_REGISTRY:-"617930562442.dkr.ecr.us-west-2.amazonaws.com"}
Copy link
Contributor

Choose a reason for hiding this comment

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Updated change to point to public ecr..

@github-actions
Copy link
Contributor

github-actions bot commented Dec 2, 2025

This pull request is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 14 days

@github-actions github-actions bot added the stale Issue or PR is stale label Dec 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

stale Issue or PR is stale

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants