Skip to content
This repository has been archived by the owner on Jul 26, 2022. It is now read-only.

CRD empty properties breaking intellij parsing (cause of empty properties) #753

Closed
joggeli34 opened this issue May 21, 2021 · 0 comments · Fixed by #758
Closed

CRD empty properties breaking intellij parsing (cause of empty properties) #753

joggeli34 opened this issue May 21, 2021 · 0 comments · Fixed by #758

Comments

@joggeli34
Copy link

Hi,

When including the CRD for parsing in the intellij, it breaks parsing as the CRD is having empty properties which are not supported there in openApiSchema.

The parsing will be fixed to not break in the intellij following the issue:
https://youtrack.jetbrains.com/issue/IDEA-269698

Probably the empty properties could just be omitted? (I don't know openApi well enough)

                    oneOf:
                      - properties:
                          key:
                          name:
                          property:
                          isBinary:
                          secretType:
                        required:
                          - key
                          - name
                      - properties:
                          path:
                          recursive:
                        required:
                          - path
davidholsgrove added a commit to davidholsgrove/kubernetes-external-secrets that referenced this issue May 24, 2021
Provide types for each property in oneOf

The empty properties here also produces invalid json schema when used with https://github.com/yannh/kubeconform
Flydiverny pushed a commit that referenced this issue Jun 3, 2021
* Closes #753

Provide types for each property in oneOf

The empty properties here also produces invalid json schema when used with https://github.com/yannh/kubeconform

* Remove properties
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant