Skip to content

feat: support crossplane 2.3 api types#128

Open
floric wants to merge 6 commits into
crossplane:masterfrom
floric:feat/new-api
Open

feat: support crossplane 2.3 api types#128
floric wants to merge 6 commits into
crossplane:masterfrom
floric:feat/new-api

Conversation

@floric

@floric floric commented May 29, 2026

Copy link
Copy Markdown

Description of your changes

Add compatibility to Crossplane 2.3 by supporting new types and keeping backwards compatibility.

I have:

  • Read and followed Crossplane's contribution process.
  • Run make reviewable test to ensure this PR is ready for review.

How has this code been tested

Tested on provider-gitlab.

floric added 4 commits May 29, 2026 10:10
Signed-off-by: Florian Fa Richter <florian.fa.richter@deutschebahn.com>
Signed-off-by: Florian Fa Richter <florian.fa.richter@deutschebahn.com>
Signed-off-by: Florian Fa Richter <florian.fa.richter@deutschebahn.com>
Signed-off-by: Florian Fa Richter <florian.fa.richter@deutschebahn.com>
@floric floric changed the title Feat/new api feat: support crossplane 2.3 api types May 29, 2026
Signed-off-by: Florian Fa Richter <florian.fa.richter@deutschebahn.com>
@floric

floric commented May 29, 2026

Copy link
Copy Markdown
Author

@negz @jbw976 are you open for this change? Should be needed for all migrations to Crossplane 2.3 if I understood it correctly.

@BoxBoxJason

Copy link
Copy Markdown

Hey there,

Yes, the angryjet generation of the usual methodsets for 2.3 resources is currently broken.
This would be the exact fix required for it to work again.

@jbw976 jbw976 moved this to In Review in Crossplane Roadmap Jun 25, 2026

@adamwg adamwg left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I'm new to this codebase and not a maintainer, but this looks correct to me.

Comment thread cmd/angryjet/main.go Outdated
Signed-off-by: Florian Fa Richter <florian.fa.richter@deutschebahn.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: In Review

Development

Successfully merging this pull request may close these issues.

5 participants