Skip to content

chore(scripts): remove stale Kind reload helper - #2458

Draft
kevkantes wants to merge 1 commit into
mainfrom
prebuild/chore/remove-stale-kind-reload
Draft

chore(scripts): remove stale Kind reload helper#2458
kevkantes wants to merge 1 commit into
mainfrom
prebuild/chore/remove-stale-kind-reload

Conversation

@kevkantes

Copy link
Copy Markdown
Collaborator

⚠️ NEEDS DISCUSSION

@Thun78, this removes the standalone Kind image-reload helper introduced in commit 3d573c3. Please confirm there is no remaining supported local workflow that relies on it.

References #2444 (does not close it).

Description

Removes scripts/dev-kind-reload.sh, an unreferenced one-off helper.

Audit findings:

  • No source, documentation, CI, or other repository caller references the script.
  • Its first-time setup instruction points to missing charts/ai-platform-engineering/values-local-images.yaml.
  • It supports only caipe-ui and autonomous-agents; the current documented and CI-validated Kind deployment workflow uses setup-caipe.sh.
  • The original commit author resolves to @Thun78. The associated merged PR was feat(autonomous-agents): add scheduled and event-triggered task execution #2139.

No Compose, Helm, or runtime configuration changes are included.

Type of Change

  • Other (stale script removal)

Pre-release Helm Charts (Optional)

Not applicable: this PR does not change Helm charts.

Checklist

  • I have read the contributing guidelines
  • Existing issues have been referenced (where applicable)
  • I have verified this change is not present in other open pull requests
  • Functionality is documented (not applicable; removed an undocumented helper)
  • All code style checks pass (git diff --check)
  • New code contribution is covered by automated tests (not applicable)
  • All new and existing tests pass (not run; deletion-only change)

Signed-off-by: kevkantes <kkantesa@cisco.com>
@github-actions

Copy link
Copy Markdown
Contributor

✅ No proprietary content detected. This PR is clear for review!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Todo

Development

Successfully merging this pull request may close these issues.

2 participants