Skip to content

feat: add minio_audit_webhook resource #798

Merged
felladrin merged 5 commits intomainfrom
feature/728-audit-webhook-resource
Mar 4, 2026
Merged

feat: add minio_audit_webhook resource #798
felladrin merged 5 commits intomainfrom
feature/728-audit-webhook-resource

Conversation

@aminueza
Copy link
Copy Markdown
Owner

@aminueza aminueza commented Mar 4, 2026

Summary

Test Plan

  • Unit tests for config string building (TestBuildAuditWebhookCfgData)
  • Acceptance test: create + import (TestAccMinioAuditWebhook_basic)
  • Acceptance test: update queue/batch size (TestAccMinioAuditWebhook_update)
  • Full test suite passes (docker compose run --rm test — 0 failures)

Closes #728

aminueza added 4 commits March 4, 2026 10:37
Add a typed resource for managing MinIO audit webhook configurations.
This provides a user-friendly interface over the config KV API for
configuring HTTP endpoints that receive audit log events, supporting
endpoint URL, authentication, mTLS, batching, and queue settings.

Closes #728
Unit tests verify config string building with various field
combinations. Acceptance tests cover basic create/import and
update scenarios using disabled webhooks with dummy endpoints.
Add Terraform example showing Splunk and backup webhook targets,
documentation template with import instructions, and generate
the rendered docs page.
MinIO GetConfigKV does not return 'enable' in the audit_webhook
response. Skip reading it in Read (retain user value from state,
same pattern as sensitive fields). Mark queue_size and batch_size
as Computed to accept server defaults without false diffs.
@aminueza aminueza marked this pull request as ready for review March 4, 2026 10:19
@aminueza aminueza requested a review from felladrin as a code owner March 4, 2026 10:19
@felladrin felladrin merged commit b20e6a3 into main Mar 4, 2026
16 checks passed
@felladrin felladrin deleted the feature/728-audit-webhook-resource branch March 4, 2026 12:03
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.

Add resource for managing audit logging webhooks and Kafka targets

2 participants