Skip to content

chore: update Terraform reference: v0.10.0 notification config - #448

Merged
mbevc1 merged 1 commit into
mainfrom
chore/terraform-v0.10.0
Oct 5, 2026
Merged

mbevc1 merged 1 commit into
mainfrom
chore/terraform-v0.10.0

Conversation

@mintlify

@mintlify mintlify Bot commented Oct 5, 2026

Copy link
Copy Markdown
Contributor

Summary

Sync Terraform reference docs with terraform-provider-kosli v0.10.0.

Changes

  • Add kosli_notification_config resource page (example, targets, import, schema)
  • Add kosli_notification_config data source page
  • Add both pages to navigation
  • kosli_action: add Triggers section documenting all six values, correct ON_SCALED_ARTIFACT description, update examples and schema text

Context

Release v0.10.0 (October 1, 2026), upstream PR #273 and GH-267.

@mintlify
mintlify Bot requested a review from a team as a code owner October 5, 2026 09:09
@mintlify

mintlify Bot commented Oct 5, 2026 •

Copy link
Copy Markdown
Contributor Author

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated
kosli 🟢 Ready View Preview Oct 5, 2026, 9:09 AM

@claude

claude Bot commented Oct 5, 2026 •

Copy link
Copy Markdown
Contributor

Claude finished @mintlify[bot]'s task in 42s —— View job


Doc review: PR #448

I checked the changed pages against integrations/kosli_actions.md, administration/authentication/api_key_rotation.md and changelog/index.mdx. Both new pages are in config/navigation.json, sitting next to the other Terraform resources and data sources. All internal links resolve, including /terraform-reference/resources/action#triggers. No pages were renamed or removed, so no redirects are needed. The six kosli_action triggers and the corrected ON_SCALED_ARTIFACT description match integrations/kosli_actions.md:16-27.

terraform-reference/resources/notification_config.mdx and terraform-reference/data-sources/notification_config.mdx

Improvement: "the recipients Kosli derives" isn't explained, and the page that explains it isn't linked

  • Location: terraform-reference/resources/notification_config.mdx:7 and terraform-reference/data-sources/notification_config.mdx:9
  • Issue: Both pages say the targets replace "the recipients Kosli would otherwise derive from what the notification is about", but they never say who those recipients are. administration/authentication/api_key_rotation.md:41-52 does: by default the org's admins and the key's creator get the warning, the override applies only to service account keys, and personal keys always go to their owner. A Terraform user can't tell what they are replacing, or what destroying the resource brings back.
  • Recommendation: Add a link to /administration/authentication/api_key_rotation#expiry-warnings on both pages, for example after the first paragraph. Optionally add one line saying the defaults are the org's admins plus the key's creator. These pages are agent-synced from kosli-dev/terraform-provider-kosli, so make the same change in that repo's resource/data-source descriptions or doc templates too, or the next sync will drop it.
  • Fix this →

1 Improvement, 0 Critical, 0 Suggestions. Fine to merge as is; this is advisory only.

@mbevc1 mbevc1 changed the title chore: Update Terraform reference: v0.10.0 notification config chore: update Terraform reference: v0.10.0 notification config Oct 5, 2026
@mbevc1
mbevc1 enabled auto-merge (squash) October 5, 2026 10:46
@mbevc1
mbevc1 merged commit 0414faf into main Oct 5, 2026
7 of 8 checks passed
@mbevc1
mbevc1 deleted the chore/terraform-v0.10.0 branch October 5, 2026 10:46

This branch was successfully deployed

1 active deployment
staging — bfa9a79a Deployed Oct 5, 2026 by mintlify[bot]
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