Skip to content

feat: add Guardrail.bedrock resource - #2152

Merged
erhancagirici merged 3 commits into
crossplane-contrib:mainfrom
erhancagirici:add-bedrock-guardrail-655
Jul 28, 2026
Merged

feat: add Guardrail.bedrock resource#2152
erhancagirici merged 3 commits into
crossplane-contrib:mainfrom
erhancagirici:add-bedrock-guardrail-655

Conversation

@erhancagirici

@erhancagirici erhancagirici commented Jul 22, 2026

Copy link
Copy Markdown
Collaborator

Description of your changes

Depends on #2151

Adds Guardrail.bedrock managed resource

Fixes #2101

I have:

  • Read and followed Crossplane's [contribution process].
  • Run make generate and committed the results (ideally in a separate commit).
  • Not made any manual changes to generated files, and verified this with make check-diff.

How has this code been tested

Uptest-examples/bedrock/cluster/v1beta1/guardrail.yaml

@erhancagirici
erhancagirici force-pushed the add-bedrock-guardrail-655 branch from 5d77fdf to 1f939bc Compare July 22, 2026 16:27
@erhancagirici

Copy link
Copy Markdown
Collaborator Author

/test-examples="examples/bedrock/cluster/v1beta1/guardrail.yaml"

@erhancagirici
erhancagirici force-pushed the add-bedrock-guardrail-655 branch from 1f939bc to 5269321 Compare July 22, 2026 17:17
@erhancagirici

Copy link
Copy Markdown
Collaborator Author

/test-examples="examples/bedrock/cluster/v1beta1/guardrail.yaml"

@erhancagirici
erhancagirici marked this pull request as ready for review July 22, 2026 18:15
@erhancagirici
erhancagirici force-pushed the add-bedrock-guardrail-655 branch from 5269321 to 345258e Compare July 28, 2026 07:32

@jonasz-lasut jonasz-lasut left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thank you @erhancagirici LGTM, left two small nits

Comment thread config/cluster/bedrock/config.go
Comment thread config/namespaced/bedrock/config.go
@erhancagirici
erhancagirici force-pushed the add-bedrock-guardrail-655 branch from 345258e to 1576cee Compare July 28, 2026 10:56
Signed-off-by: Erhan Cagirici <erhan@upbound.io>
Signed-off-by: Erhan Cagirici <erhan@upbound.io>
Signed-off-by: Erhan Cagirici <erhan@upbound.io>
@erhancagirici
erhancagirici force-pushed the add-bedrock-guardrail-655 branch from 1576cee to e91752e Compare July 28, 2026 11:27
@erhancagirici

Copy link
Copy Markdown
Collaborator Author

/test-examples="examples/bedrock/cluster/v1beta1/guardrail.yaml"

@erhancagirici
erhancagirici merged commit 0291084 into crossplane-contrib:main Jul 28, 2026
9 checks passed
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.

Request for Guardrail.bedrock resource

2 participants