Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement proposal #31 for better plugin mechanism and cleanup Json schema #41

Merged
merged 9 commits into from
May 5, 2020

Conversation

davidfestal
Copy link
Collaborator

@davidfestal davidfestal commented Apr 24, 2020

What does this PR do?

This PR implements in the proposal #31 that was agreed on,
especially the overriding mechanism for parent devfiles and plugins
based on strategic merge patch (a-la-kustomize).

This PR also enhances the build mechanism that derives JSON schemas from
the generated K8S CRD OpenApi schema. Especially it:

  • removes some K8S-specific fields (union discriminators and kubernetes extensions)
  • forbids additional non-validated properties everywhere, unless it is explicitly requested by the source model
  • makes required the fields that should be mandatory but are set optional in the K8S model in order to allow easier overriding

As part of this enhancement, this PR also upgrades the version of the operator-sdk CLI tools used to generate the CRDs to 0.17.0

What issues does this PR fix or reference?

This PR implements issue #31

Is your PR tested? Consider putting some instruction how to test your changes

Yes

to define Spring boot example

Signed-off-by: David Festal <[email protected]>
Signed-off-by: David Festal <[email protected]>
.. and update the parent override mechanism to use a
common strategic merge patch approach

Signed-off-by: David Festal <[email protected]>
Signed-off-by: David Festal <[email protected]>
Signed-off-by: David Festal <[email protected]>
... that are derived from the CRD Openapi schema

And also upgrade the operator-sdk used for CRD generation to 0.17.0

Signed-off-by: David Festal <[email protected]>
Signed-off-by: David Festal <[email protected]>
@che-osio-probot-app
Copy link

Open Developer Workspace:
Contribute

@davidfestal davidfestal requested a review from l0rd April 24, 2020 18:21
@davidfestal
Copy link
Collaborator Author

After confirmation from @l0rd, let me merge this PR and provide remaining changes in new PRs.

@davidfestal davidfestal merged commit db7d735 into master May 5, 2020
@davidfestal davidfestal linked an issue Jun 23, 2020 that may be closed by this pull request
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.

Better plugin mechanism
1 participant