-
Notifications
You must be signed in to change notification settings - Fork 3
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
What's broken? #17
Comments
for example #15 or #14 or #13 more at https://github.com/kedgeproject/json-schema-generator/issues ;-) |
Condition today: Right now the openapi2jsonschema which is used to generate configs is based off of my branch which has changes added to generate only controllers we care about like deploymentconfig jobspec etc Now I have created a docker image Dockerfile is also changed and is available here. Link to my branch https://github.com/surajssd/openapi2jsonschema/tree/generate-configs-for-kedge Once OpenShift gives you openapi spec and not old swagger then we can remove the whole part where we are doing conversion from swagger v1 to openapi Code to do that is in Dockerfile for installing npm and that tool which does the conversion. And in entrypoint script where we are actually doing the conversion. Once we solve the above problem we don't need the fedora image we can use the upstream image So right now the way to go forward is to generate the configs right now and edit it manually. How do you know if your jsonschema is wrong?Write a kedge json-schema acceptance test framwork. So what this will do is take your jsconschema as one input and another input is the examples in the kedge upstream repo. Since we know all the examples in the kedge repo are correct the jsonschema should validate those configs without error. If jsonschema validation fails then fix it manually. So after you setup this framework, run it on travis so that all changes are checked. |
Thank you so much @surajssd, very helpful!
Link to my branch https://github.com/surajssd/openapi2jsonschema/tree/generate-configs-for-kedge
Dockerfile is also changed and is available here.
Once OpenShift gives you openapi spec and not old swagger then we can remove the whole part where we are doing conversion from swagger v1 to openapi Code to do that is in Dockerfile for installing npm and that tool which does the conversion. And in entrypoint script where we are actually doing the conversion.
Once we solve the above problem we don't need the fedora image we can use the upstream image
How do you know if your jsonschema is wrong?Write a kedge json-schema acceptance test framwork. So what this will do is take your jsconschema as one input and another input is the examples in the kedge upstream repo. Since we know all the examples in the kedge repo are correct the jsonschema should validate those configs without error. If jsonschema validation fails then fix it manually. So after you setup this framework, run it on travis so that all changes are checked.
|
@surajssd @surajnarwade what are currently broken things for us to generate a valid JSON Schema? Would be great to have a consolidated view!
The text was updated successfully, but these errors were encountered: