Skip to content

fix: correct Metal quickstart docs for selectorless services#173

Merged
Defilan merged 1 commit intomainfrom
fix/metal-selectorless-service-docs
Feb 25, 2026
Merged

fix: correct Metal quickstart docs for selectorless services#173
Defilan merged 1 commit intomainfrom
fix/metal-selectorless-service-docs

Conversation

@Defilan
Copy link
Member

@Defilan Defilan commented Feb 22, 2026

Summary

Fixes #166

  • Updated Metal quickstart README to clarify that Metal services are selectorless (no pods on K8s), so kubectl port-forward does not work
  • Replaced port-forward instructions with in-cluster DNS access examples and a temporary curl pod approach
  • Fixed missing trailing newline in pkg/agent/registry_test.go

Test plan

  • Documentation-only change for README
  • Trailing newline fix is cosmetic

Signed-off-by: Christopher Maher chris@mahercode.io

Metal services are selectorless (no pods on K8s), so kubectl port-forward
does not work. Updated README to show in-cluster DNS access and a temporary
curl pod approach instead. Also fix missing trailing newline in registry_test.

Signed-off-by: Christopher Maher <chris@mahercode.io>
@Defilan Defilan force-pushed the fix/metal-selectorless-service-docs branch from 6446dbd to 735857d Compare February 22, 2026 09:08
@Defilan
Copy link
Member Author

Defilan commented Feb 22, 2026

Hey @matiasinsaurralde , do you mind checking this to see if that makes sense and works for you before we close this one out?

@matiasinsaurralde
Copy link
Contributor

This works @Defilan, thanks!

@Defilan Defilan merged commit 89471ec into main Feb 25, 2026
4 checks passed
@Defilan Defilan deleted the fix/metal-selectorless-service-docs branch February 25, 2026 15:32
This was referenced Feb 25, 2026
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.

[BUG] Metal quickstart not working as expected

2 participants