Skip to content

feat(agent_association): add configurable description input for notification channels#439

Merged
agustincelentano merged 1 commit into
mainfrom
feat/agent-association-channel-description
Jul 15, 2026
Merged

feat(agent_association): add configurable description input for notification channels#439
agustincelentano merged 1 commit into
mainfrom
feat/agent-association-channel-description

Conversation

@agustincelentano

Copy link
Copy Markdown
Collaborator

What

Exposes a new optional description input in the three agent association modules, wiring it into the nullplatform_notification_channel resource:

  • parameter_storage_definition_agent_association
  • scope_definition_agent_association
  • service_definition_agent_association

The variable is identical across the three (type = string, default = "") for consistency.

Why

The channel description was only available (hardcoded) in the parameter storage module, and absent in scope/service. This lets consumers set a meaningful, per-channel description from the module call (e.g. driven from a for_each catalog).

Backward compatibility

⚠️ The parameter storage module previously hardcoded its description ("Notification channel to handle parameter storage and retrieval"). It now reads from var.description with default = "". Consumers relying on the old text must set it explicitly after upgrading, otherwise the channel description becomes empty.

Scope and service modules had no description before, so for them this is purely additive.

Testing

Validated end-to-end against a real nullplatform account (implementation-aws binding) by pointing the four channel modules at this branch:

  • tofu validate / tofu fmt
  • tofu apply: 9 channels updated in-place, 0 destroyed — the API accepted the description PATCH.
  • Verified via API that the description persisted on the channels (scope, service and parameter storage).

READMEs (terraform-docs ## Inputs tables) updated manually to include the new input, respecting the alphabetical sort configured in .terraform-docs.yml.

🤖 Generated with Claude Code

…ication channels

Expose a new optional `description` variable in the parameter storage, scope
and service agent association modules, wiring it into the
nullplatform_notification_channel resource.

The parameter storage module previously hardcoded its channel description;
it now reads from var.description (default "") for consistency across the
three modules. Consumers relying on the old hardcoded text must set it
explicitly after upgrading.
@agustincelentano
agustincelentano merged commit 7663877 into main Jul 15, 2026
44 checks passed
@agustincelentano
agustincelentano deleted the feat/agent-association-channel-description branch July 15, 2026 13:40
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.

2 participants