Skip to content

Conversation

@ShashwatDash23
Copy link
Contributor

@ShashwatDash23 ShashwatDash23 commented Oct 20, 2025

This commit adds support for blackout windows via the cockroach_blackout_window resource. It also adds a cockroach_blackout_windows data source for listing blackout windows.

Commit checklist

  • Changelog
  • Doc gen (make generate)
  • Integration test(s)
  • Acceptance test(s)
  • Example(s)

@ShashwatDash23 ShashwatDash23 force-pushed the shashwatdash23/bow-resource branch 3 times, most recently from 480f3d3 to 56f9ba1 Compare October 20, 2025 20:30
## Example Usage

```terraform
# List all blackout windows (upto 100)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: Where does the number 100 come from here? It may be confusing for the user. Is this the default page size for the api?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The number is defined here: https://github.com/cockroachlabs/managed-service/blob/master/pkg/serverutils/pagination.go#L27-L28. It’s the default page size used when no limit is set.

It’s applied here:: https://github.com/cockroachlabs/managed-service/blob/master/pkg/serverutils/pagination.go#L298C3-L302C112

Do you suggest that we should avoid mentioning the specific value (100) to users?

Copy link
Collaborator

@fantapop fantapop left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this looks good to me. I left one question as a nit.

… data source

This commit adds support for blackout windows via the
`cockroach_blackout_window` resource. It also adds a
`cockroach_blackout_windows` data source for listing
blackout windows.
@ShashwatDash23 ShashwatDash23 force-pushed the shashwatdash23/bow-resource branch from 56f9ba1 to 9e8465f Compare October 23, 2025 15:34
@ShashwatDash23 ShashwatDash23 merged commit 7be4134 into main Oct 23, 2025
4 checks passed
@ShashwatDash23 ShashwatDash23 deleted the shashwatdash23/bow-resource branch October 23, 2025 15:39
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