feat: add all remaining resources#10
Open
DominikPildner wants to merge 12 commits into
Open
Conversation
DominikPildner
requested review from
johannes-kirchner and
philippeckel
as code owners
Resolved add/add and content conflicts between the all-resources-generated branch (complete resource set) and main (which added observability, mariadb, opensearch, mongodbflex, sqlserverflex incrementally). - Keep our generated files (zz_*.go, zz_register.go, zz_setup.go) as they include all resources - Merge config/provider.go and config/external_name.go keeping all entries - Keep our fixed examples (observability planName, sqlserverflex flavor/roles) - Adopt main's observability config helper pattern for addInstanceReference Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Take main's v1alpha1/ example layout. Convert main's instanceIdRef references to instanceIdSelector with matchLabels to keep the selector-by-label convention used across the branch's examples. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Remove flat-path duplicates (mongodbflex, mariadb, opensearch) left over from the merge with main. Replace placeholder content (xxxxxxxx-..., XXX.XXX, class: class) in v1alpha1/ paths with the branch's pre-merge tested values across postgresflex, sqlserverflex, mongodbflex, mariadb, opensearch, objectstorage, observability, rabbitmq, redis, secretsmanager. Mirror each cluster v1alpha1/ to its namespaced/ equivalent with the .stackit.m.crossplane.io apiVersion. Use <PROJECT-ID> placeholder consistently and swap real public CIDRs (193.148.160.0/19, 45.129.40.0/21, 45.135.244.0/22) for RFC 5737 documentation prefixes (192.0.2.0/24, 198.51.100.0/24, 203.0.113.0/24) in ACL / sgwAcl / ipAcl fields. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Extract shortGroup constant in compute/network configurators to satisfy goconst, remove a double blank line in network/config.go and re-align the compute block in external_name.go. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.