Skip to content

feat(albwaf): add the ability to manage mrs groups - #1863

Merged
SerseusWasTaken merged 11 commits into
mainfrom
feat/modify-mrs-groups-in-terraform
Nov 3, 2026
Merged

SerseusWasTaken merged 11 commits into
mainfrom
feat/modify-mrs-groups-in-terraform

Conversation

@SerseusWasTaken

@SerseusWasTaken SerseusWasTaken commented Oct 30, 2026 •

Copy link
Copy Markdown
Contributor

Description

Adds the configured_rules attribute to configure rules in managed rule sets

relates to #1795 and STACKITTPR-998

Checklist

  • Issue was linked above
  • Code format was applied: make fmt
  • Examples were added / adjusted (see examples/ directory)
  • Docs are up-to-date: make generate-docs (will be checked by CI)
  • Unit tests got implemented or updated
  • Acceptance tests got implemented or updated (see e.g. here)
  • Unit tests are passing: make test (will be checked by CI)
  • No linter issues: make lint (will be checked by CI)

omarsilva1 and others added 6 commits October 2, 2026 17:07
stackit_alb_waf_managed_rule_set exposed the mode of every rule only as
a read-only attribute, and Update returned "not updatable", so a CRS
rule could be set to log-only or disabled only through the API, and was
lost whenever the rule set was recreated.

rule_modes is an optional map from rule ID to mode. It is applied with
PatchManagedRuleSet after create and on update, in place; a rule removed
from the map goes back to MODE_ENABLED. Read refreshes only the listed
rules, so a mode changed outside Terraform shows as drift. On update the
plan carries the groups inventory from state with the planned modes
applied, rather than marking the whole inventory unknown.

The data source keeps its schema: the resource uses ResourceModel, which
embeds the shared Model and adds rule_modes.

relates to #1794

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@SerseusWasTaken SerseusWasTaken changed the title Feat/modify mrs groups in terraform feat(albwaf): add the ability to manage mrs groups Oct 30, 2026
@SerseusWasTaken
SerseusWasTaken marked this pull request as ready for review October 30, 2026 12:14
@SerseusWasTaken
SerseusWasTaken requested a review from a team as a code owner October 30, 2026 12:14
@SerseusWasTaken
SerseusWasTaken force-pushed the feat/modify-mrs-groups-in-terraform branch from 6425b35 to 5a9441d Compare October 30, 2026 12:18
@SerseusWasTaken
SerseusWasTaken force-pushed the feat/modify-mrs-groups-in-terraform branch from 5a9441d to f486d38 Compare November 2, 2026 13:59
Co-authored-by: Alexander Dahmen <alexander.dahmen@inovex.de>
Fyusel
Fyusel previously approved these changes Nov 3, 2026
@SerseusWasTaken
SerseusWasTaken force-pushed the feat/modify-mrs-groups-in-terraform branch from d712891 to 2203ddf Compare November 3, 2026 13:34
@SerseusWasTaken
SerseusWasTaken merged commit 2e721e9 into main Nov 3, 2026
5 checks passed
@SerseusWasTaken
SerseusWasTaken deleted the feat/modify-mrs-groups-in-terraform branch November 3, 2026 13:39
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.

3 participants