Skip to content

docs(superpowers): preempt #25 monolith OTel to Operator native#425

Merged
panicboat merged 1 commit into
mainfrom
docs/otel-25-preempt-spec
May 17, 2026
Merged

docs(superpowers): preempt #25 monolith OTel to Operator native#425
panicboat merged 1 commit into
mainfrom
docs/otel-25-preempt-spec

Conversation

@panicboat

@panicboat panicboat commented May 17, 2026

Copy link
Copy Markdown
Owner

Summary

引き継ぎ事項 #25 (= OTel Operator Ruby auto-injection native support) の stance を "blocked upstream" → "preempted to Operator native" に切り替えるための spec doc + phase 6 closure doc update。

2026-05-17 再確認で upstream の direction が positive:

これに合わせて monorepo 側で deployment.yaml の env vars 6 個 + initializer.rb を 今 削除instrumentation.opentelemetry.io/inject-ruby annotation を追加して Operator merge + chart upgrade で auto 復活する状態に preempt。 短期 monolith trace 消失を許容。

Files

Related

  • monorepo PR (= 別途作成): services/monolith/kubernetes/base/deployment.yaml + services/monolith/workspace/config/initializers/opentelemetry.rb の実 file 変更

Test plan

Summary by CodeRabbit

  • Documentation
    • Updated Phase 6 closure design with revised OpenTelemetry Operator upgrade strategy
    • Added new design specification for OpenTelemetry modernization plan, including architecture guidance and validation checklist for native Ruby auto-injection integration

Review Change Stack

OTel Operator upstream の Ruby auto-instrumentation 動向を 2026-05-17 再
確認、 ruby-contrib#1384 が APPROVED + 直近 active と判明。 stance を
"blocked upstream" から "preempted to Operator native" に切り替え、 monorepo
側で deployment.yaml の env vars 6 個 + initializer.rb を削除 + inject-ruby
annotation を追加して Operator merge を anticipate。 短期 monolith trace 消
失を許容、 Operator merge + chart upgrade で auto 復活する path。

Files:
- docs/superpowers/specs/2026-05-17-monolith-otel-preempt-design.md (新規)
- docs/superpowers/specs/2026-05-13-eks-production-phase-6-closure-design.md
  (#25 entry update + 本 spec への link)

Related: monorepo PR (= deployment.yaml + initializer 修正) で実装
Signed-off-by: panicboat <panicboat@gmail.com>
@coderabbitai

coderabbitai Bot commented May 17, 2026

Copy link
Copy Markdown

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 15b7faa6-5fe6-4439-83fd-2f2d8c068dba

📥 Commits

Reviewing files that changed from the base of the PR and between 6784619 and 253bc7f.

📒 Files selected for processing (2)
  • docs/superpowers/specs/2026-05-13-eks-production-phase-6-closure-design.md
  • docs/superpowers/specs/2026-05-17-monolith-otel-preempt-design.md

📝 Walkthrough

Walkthrough

This PR updates documentation for the monolith's OpenTelemetry instrumentation transition. The Phase 6 closure timeline is updated to reflect a preempted operator-native approach for Ruby auto-injection, and a detailed design specification document is introduced to describe the removal of hardcoded OTel configuration and adoption of Operator-driven annotation-based injection.

Changes

Monolith OTel Preemption Planning

Layer / File(s) Summary
Phase 6 closure timeline update
docs/superpowers/specs/2026-05-13-eks-production-phase-6-closure-design.md
Phase 7+ item #25 plan shifts from "upstream-blocked" to "preempted operator-native"; monolith OTel workaround details are replaced with inject-ruby annotation strategy, upstream status is updated, and follow-up steps are revised to watch for operator merge/chart release before applying configuration changes.
Detailed OTel preemption design specification
docs/superpowers/specs/2026-05-17-monolith-otel-preempt-design.md
New specification document outlines immediate removal of six OTel environment variables and the initializer from monolith Kubernetes deployment, adoption of instrumentation.opentelemetry.io/inject-ruby annotation, related repo changes and out-of-scope follow-ups (chart version, Instrumentation CR, gem additions), and comprehensive risk/mitigation and validation checklists for both immediate and post-Operator-upgrade phases.

Possibly related PRs

  • panicboat/platform#381: Both PRs edit the same EKS Phase 6 closure documentation file, adjusting Phase 7+ remaining-item counts and sections.

Suggested labels

deploy:opentelemetry

Poem

🐰 From hardcoded traces to annotations so fine,
The monolith sheds its OTel design,
An Operator native shall take up the call,
Auto-injection flows—no burden at all!


🎯 2 (Simple) | ⏱️ ~10 minutes


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

Comment @coderabbitai help to get the list of available commands and usage tips.

@panicboat panicboat marked this pull request as ready for review May 17, 2026 00:53
@panicboat panicboat merged commit 2c32a68 into main May 17, 2026
7 checks passed
@panicboat panicboat deleted the docs/otel-25-preempt-spec branch May 17, 2026 00:53
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