Skip to content

Fix version parameter autocompletion - #225

Open
dheeraj12347 wants to merge 1 commit into
apache:mainfrom
dheeraj12347:fix-version-autocomplete-224
Open

Fix version parameter autocompletion#225
dheeraj12347 wants to merge 1 commit into
apache:mainfrom
dheeraj12347:fix-version-autocomplete-224

Conversation

@dheeraj12347

Copy link
Copy Markdown
Contributor

Fixes #224

The version parameter autocompletion was incorrectly retrieving
registered Kubernetes versions for APIs such as list hosts and
list routers.

This change makes version completion use the current list API's
version response field when available.

Added regression tests for listHosts and listRouters.

Verified with:

  • go test ./...
  • go vet ./...
  • make
  • git diff --check

@dheeraj12347

Copy link
Copy Markdown
Contributor Author

Hi @nvazquez , @DaanHoogland I’ve worked on CloudMonkey issue #224 regarding the incorrect version parameter autocompletion.

I’ve opened PR #225 with the fix. The change makes version= use the current list API when it exposes a version response field, and adds regression tests for listHosts and listRouters.

I’ve also verified the changes with go test ./..., go vet ./..., and make.

Would appreciate your feedback when you get a chance.

@github-actions

Copy link
Copy Markdown

✅ Build complete for PR #225.

📦 Binary artifacts are available in the workflow run (expires on September 21, 2026).

Note: Download artifacts by clicking on the workflow run link above, then scroll to the "Artifacts" section.
Artifacts from PR builds are for testing only and may contain unreviewed, malicious code.

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.

Autocompletion for version parameter on different APIs always retrieves the registered CKS versions

1 participant