Skip to content

[STORY-3495] doc: add CRs examples#36

Merged
sc-david-voisin merged 3 commits intomainfrom
doc/story-3495/provide-crs-examples
Mar 27, 2026
Merged

[STORY-3495] doc: add CRs examples#36
sc-david-voisin merged 3 commits intomainfrom
doc/story-3495/provide-crs-examples

Conversation

@sc-david-voisin
Copy link
Copy Markdown
Contributor

@sc-david-voisin sc-david-voisin commented Mar 17, 2026

Provide custom resources that:

  • are easy to use in Prod environments
  • demonstrate firewall change rules
  • demonstrate plan change

@sc-david-voisin sc-david-voisin self-assigned this Mar 17, 2026
@sc-david-voisin sc-david-voisin changed the title doc: add CRs examples [STORY-3495] doc: add CRs examples Mar 17, 2026
@notion-workspace
Copy link
Copy Markdown

@sc-david-voisin sc-david-voisin marked this pull request as ready for review March 17, 2026 13:52

Example:
`config/samples/databases_v1alpha1_postgresql.yaml`
Examples:
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

In the PR you add three files, but only two in the list, did you miss one? (the custom resource is not in the list)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I did on purpose, I expect these examples to grow in the future, and I've listed the most significant ones.

@sc-david-voisin sc-david-voisin force-pushed the doc/story-3495/provide-crs-examples branch from d4e3182 to 59f242f Compare March 19, 2026 11:17
Comment on lines +30 to +31
- type: "managed_range"
range_id: "man-osc-fr1-egress"
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

What's the point of this range if we already allow all ?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

There is no point. This is a remain from my very first sample tests.
Fixed in c053a43

Comment on lines +30 to +31
- type: "managed_range"
range_id: "man-osc-fr1-egress"
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Same here

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Same response as above.

@sc-david-voisin sc-david-voisin merged commit 9244235 into main Mar 27, 2026
5 checks passed
@sc-david-voisin sc-david-voisin deleted the doc/story-3495/provide-crs-examples branch March 27, 2026 09:00
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.

3 participants