Skip to content

Replace Go model generation for open-virtual-networking #6428

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

Closed
manusa opened this issue Oct 7, 2024 · 1 comment · Fixed by #6534
Closed

Replace Go model generation for open-virtual-networking #6428

manusa opened this issue Oct 7, 2024 · 1 comment · Fixed by #6534
Assignees
Labels
Milestone

Comments

@manusa
Copy link
Member

manusa commented Oct 7, 2024

Description

Part of #6130

Replace model generation for open-virtual-networking module with new OpenAPI approach.

Note

For extension models there are going to be multiple conflicts with the previously generated code.

Tasks

  • Remove build-helper-maven-plugin
  • Remove maven-antrun-plugin from the generate profile
  • Remove jsonschema2pojo-maven-plugin from the generate profile
  • Consolidate all extension models to the same module (certmanager, tekton, and so on all have multiple model generator modules)
  • Configure the openapi-model-generator-maven-plugin plugin to generate the specific classes.
  • Remove the go-related files and directories (Makefile, cmd)
  • Remove invocation from the generateModel.sh script
@manusa manusa moved this to In Progress in Eclipse JKube Nov 4, 2024
@manusa manusa self-assigned this Nov 4, 2024
@manusa
Copy link
Member Author

manusa commented Nov 4, 2024

github.com/ovn-org/ovn-kubernetes

@manusa manusa added this to the 7.0.0 milestone Nov 4, 2024 — with automated-tasks
@github-project-automation github-project-automation bot moved this from Review to Done in Eclipse JKube Nov 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant