Skip to content

feat(observability): qualify telemetry across managed tenants - #908

Open
Baylem wants to merge 15 commits into
stack/880-network-and-relayfrom
feat/880-managed-namespace-monitoring
Open

Baylem wants to merge 15 commits into
stack/880-network-and-relayfrom
feat/880-managed-namespace-monitoring

Conversation

@Baylem

@Baylem Baylem commented Nov 11, 2026 •

Copy link
Copy Markdown
Member

Tracks #880 under #867. The upstream Palworld 1.3.0 module now exposes the dedicated server’s observed serverfps as a frame-rate metric. Module PR #60 · Merge 0dcdb48b

Gameplane’s shared catalog delivery prerequisite is #926. The maintainer updated its gitlink to f9e55aa1 after the ARK: Survival Evolved 2.0.1 safeguard merged. This pin includes both this metric and the stopped-copy backup declarations for ARK: Survival Ascended and Project Zomboid. Installations using OCI catalogs need a release containing this revision. Catalog pin and CI, #926

The PodMonitors, workload rules and inventory-based missing-target alert use the operator's managed namespaces. The missing-target alert compares running, labeled containers against discovery. The scrape-failed alert covers discovered targets that stop responding. Relay alerts and dashboards consume the counters from #905, and the memory dashboard shows process RSS and cAdvisor working set separately.

The dedicated Cilium/Prometheus CI fixture now runs identically named games in two namespaces, removes and restores one discovery target, verifies a cross-tenant denial, invalidates one player query, and allocates memory in one game process. The chart rendered both tenant selectors and the alert. The Linux E2E sources compiled. The Palworld engine fixture recorded 59 FPS on dedicated server v1.0.5.102999, Steam build 25247047. Two-tenant Cilium fixture · Relay saturation trials · Prometheus rule fixtures. The live Prometheus/Cilium test passed in 382.65 seconds. The Cilium legacy-to-strict upgrade drill and promtool rule fixtures passed. The complete CI run passed with 51 successful jobs and 12 skipped.

Merge order. #905 has merged. #894 supplies tenant-network isolation and depends on #886. The signed base stack/880-network-and-relay at d3facc68 includes the current #894 implementation and the merged relay instrumentation. The prospective monitoring change spans nine files. GitHub currently shows ten files for review, including the shared changelog index. Once #894 lands, retarget this PR to master, inspect the resulting files and run CI on that merge tree. #926 delivers the bundled Palworld metric described above.

…rt on missing targets

Signed-off-by: Baylem <zelathis@gmail.com>
Signed-off-by: Baylem <zelathis@gmail.com>
Signed-off-by: Baylem <zelathis@gmail.com>
Signed-off-by: Baylem <zelathis@gmail.com>
@Baylem Baylem added type: feature New capability or user-facing addition area: chart Helm chart and deployment area: e2e kind-based end-to-end suite labels Nov 11, 2026
@coderabbitai

coderabbitai Bot commented Nov 11, 2026 •

Copy link
Copy Markdown
Pre-merge checks: Passed 1 · Inconclusive 1

Failed Failed checks: 1 inconclusive

Check name Status Explanation Resolution
Title check Pending This assessment has not completed. The check will be retried on the next pre-merge check run.
Passed Passed checks: 1 passed

Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

@github-actions

github-actions Bot commented Nov 11, 2026 •

Copy link
Copy Markdown

CI Report - b8f443f

run 42988763991 - attempt 1

0 failed - 21 passed - 12 skipped - 151m26s elapsed

Test selection: partial - per-suite table

Coverage

no coverage/* commit statuses found on b8f443f

Changed

Area Files +/-
charts/ 5 +133/-14
test/ 2 +95/-18
docs/ 1 +46/-1

Slowest jobs

Job Duration
e2e telemetry / amd64 (kind) 23m11s
e2e telemetry / arm64 (kind) 22m2s
build e2e images 15m48s
e2e restricted-workloads / amd64 (kind) 14m38s
e2e restricted-workloads / arm64 (kind) 14m36s

e2e buckets run

api-agent, api-auth, api-mods, api-rbac, api-roles, backup-preparation, bot-fast, management-placement, monitoring, multicluster, network-isolation, operator, optional-components, postgres-availability, ratelimit, recovery, restricted-workloads, telemetry, upgrade, wake-hostport

Signed-off-by: Baylem <zelathis@gmail.com>
@Baylem
Baylem marked this pull request as ready for review November 11, 2026 06:25
@Baylem
Baylem requested a review from ValgulNecron as a code owner November 11, 2026 06:25
Signed-off-by: Baylem <zelathis@gmail.com>
Signed-off-by: Baylem <zelathis@gmail.com>
@ValgulNecron

Copy link
Copy Markdown
Collaborator

Hey, could you please add changelog entry?

Signed-off-by: Baylem <zelathis@gmail.com>
Signed-off-by: Baylem <zelathis@gmail.com>
@Baylem

Baylem commented Nov 11, 2026

Copy link
Copy Markdown
Member Author

Added the requested unreleased changelog entry.

CI passed on b8f443f5. The diff through ea991243 changes only the changelog entry and its index link. I canceled the repeat CI run because this update changes only Markdown.

This branch has not been deployed

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

Labels

area: chart Helm chart and deployment area: e2e kind-based end-to-end suite type: feature New capability or user-facing addition

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat(observability): discover tenant metrics and alert on missing game signals

2 participants