Skip to content

Commit

Permalink
Include namePrefix in CRD name of CompositionResourceDefinition as CR…
Browse files Browse the repository at this point in the history
…D name must match plural.group

Signed-off-by: Jesper Nylend <jesper.nylend@klp.no>
  • Loading branch information
nylend95 authored and jespernylendklp committed Sep 2, 2024
1 parent 4a23c6b commit e7edfe6
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 5 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -74,7 +74,6 @@ private static void registerOrUpdateCompositeResourceDefinition(CompositeResourc
public static <T extends CustomResource<?, Void>> CompositeResourceDefinition createCompositeResourceDefinition(T compositionDefinition) { //}, Class functionMixin) {

CompositeResourceDefinition compositeResourceDefinition = new CompositeResourceDefinition();
compositeResourceDefinition.setMetadata(CrossplaneMetadataBuilder.createMetadata(compositionDefinition.getCRDName()));

CompositeResourceDefinitionSpec spec = new CompositeResourceDefinitionSpec();
spec.setGroup(compositionDefinition.getGroup());
Expand All @@ -99,10 +98,11 @@ public static <T extends CustomResource<?, Void>> CompositeResourceDefinition cr
Versions versions = new Versions();
versions.setName(compositionDefinition.getVersion());

// This is not 100%. isStorage vs referencable. Need to check the crossplan docs
// This is not 100%. isStorage vs referencable. Need to check the crossplane docs
versions.setReferenceable(compositionDefinition.isStorage());
versions.setServed(compositionDefinition.isServed());

compositeResourceDefinition.setMetadata(CrossplaneMetadataBuilder.createMetadata(namePrefix + compositionDefinition.getCRDName()));

Schema schema = new Schema();
schema.setOpenAPIV3Schema(getOpenAPIV3Schema(compositionDefinition.getClass(), CrossplaneCompositeResourceMixin.class));
Expand Down
6 changes: 3 additions & 3 deletions pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@
<parent>
<groupId>org.springframework.boot</groupId>
<artifactId>spring-boot-starter-parent</artifactId>
<version>3.3.1</version>
<version>3.3.3</version>
</parent>

<groupId>io.crossplane.compositefunctions</groupId>
Expand All @@ -25,8 +25,8 @@
<maven.compiler.source>17</maven.compiler.source>

<!-- Dependency versions -->
<spring-boot.version>3.3.1</spring-boot.version>
<kubernetes-client.version>6.13.0</kubernetes-client.version>
<spring-boot.version>3.3.3</spring-boot.version>
<kubernetes-client.version>6.13.3</kubernetes-client.version>
<os-maven-plugin.version>1.7.1</os-maven-plugin.version>
<protobuf.version>3.25.1</protobuf.version>
<grpc.version>1.63.0</grpc.version>
Expand Down

0 comments on commit e7edfe6

Please sign in to comment.