Skip to content

fix(tenant-2): use host_port in cross_node_links - #12

Merged
artemnikitin merged 9 commits into
mainfrom
fix/tenant-2-cross-node-link
Jun 28, 2026
Merged

fix(tenant-2): use host_port in cross_node_links#12
artemnikitin merged 9 commits into
mainfrom
fix/tenant-2-cross-node-link

Conversation

@artemnikitin

@artemnikitin artemnikitin commented Jun 28, 2026

Copy link
Copy Markdown
Owner

Summary

  • Fix cross_node_links field: porthost_port in tenant-2/kibana.yaml

Why

Follow-up to #11. The enricher maps host_portCrossNodeLinks.HostPort. Using port: 9200 left HostPort: 0 in the desired state, so the controller resolved ELASTICSEARCH_HOSTS to 10.30.0.3:0 instead of 10.30.0.3:9200.

🤖 Generated with Claude Code

artemnikitin and others added 9 commits June 20, 2026 13:54
The scheduler placed tenant-2-kibana and tenant-2-elasticsearch on
different nodes. `links` only resolves same-node guest IPs, so
ELASTICSEARCH_HOSTS was never set and Kibana panicked at boot.

Switch to `cross_node_links` so the controller injects the peer
node's host_ip:host_port at reconcile time. Add port_forwards on
elasticsearch to expose 9200 on the host so the controller has a
host_port to resolve.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
The enricher maps `host_port` → CrossNodeLinks.HostPort. Using `port`
left HostPort=0, so the controller injected ELASTICSEARCH_HOSTS as
10.30.0.3:0 instead of 10.30.0.3:9200.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@artemnikitin
artemnikitin merged commit 475e011 into main Jun 28, 2026
3 checks passed
@artemnikitin
artemnikitin deleted the fix/tenant-2-cross-node-link branch June 28, 2026 13:40
artemnikitin added a commit that referenced this pull request Jun 28, 2026
…TS (#13)

## Summary

- Add `protocol: "http"` to `cross_node_links` in `tenant-2/kibana.yaml`

## Why

Follow-up to #12. `cross_node_links` without `protocol` injects a bare
`host:port` string (`10.30.0.3:9200`). Kibana's config validation
requires a full URI with scheme and rejects it:

```
[FATAL][root] Reason: [config validation of [elasticsearch].hosts]:
- [config validation of [elasticsearch].hosts.0]: expected URI with scheme [http|https].
```

Adding `protocol: "http"` causes the controller to inject
`http://10.30.0.3:9200`, matching what the same-node `links` mechanism
already does (`http://172.16.0.2:9200`).

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
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.

1 participant