Skip to content

Development: Add Iris Feature Flag#183

Merged
Mtze merged 2 commits intomainfrom
feat/iris-feature-flag
Feb 4, 2026
Merged

Development: Add Iris Feature Flag#183
Mtze merged 2 commits intomainfrom
feat/iris-feature-flag

Conversation

@bensofficial
Copy link
Copy Markdown
Member

@bensofficial bensofficial commented Feb 3, 2026

ls1intum/Artemis#12007

Summary by CodeRabbit

  • Chores
    • Updated deployment configuration to explicitly enable Iris integration when it is configured.
    • Ensures the Iris-related service is reliably initialized and connected during deployment, preventing missed integrations and improving runtime behavior.

@bensofficial bensofficial requested a review from bassner February 3, 2026 10:40
@bensofficial bensofficial self-assigned this Feb 3, 2026
@bensofficial bensofficial requested a review from a team as a code owner February 3, 2026 10:40
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Feb 3, 2026

Walkthrough

When Iris is defined in Artemis templates, the change explicitly enables it by adding an enabled: true YAML property and exporting ARTEMIS_IRIS_ENABLED='true' in the environment template.

Changes

Cohort / File(s) Summary
YAML Configuration
roles/artemis/templates/application-prod.yml.j2
Added iris.enabled: true inside the existing Iris conditional block.
Environment Configuration
roles/artemis/templates/artemis.env.j2
Added ARTEMIS_IRIS_ENABLED='true' environment variable inside the existing Iris conditional block (appears in two locations).

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 A tiny flag, a gentle cue,
enabled true — Iris peeks through,
YAML hums and envs agree,
Artemis wakes, as it should be. 🌱

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly describes the main change: adding an Iris feature flag to the configuration templates.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feat/iris-feature-flag

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.

@bensofficial bensofficial added this to the v2.3.0 milestone Feb 4, 2026
@Mtze Mtze merged commit 6c121c7 into main Feb 4, 2026
2 checks passed
@Mtze Mtze deleted the feat/iris-feature-flag branch February 4, 2026 18:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants