From: Thomas Lamprecht <t.lamprecht@proxmox.com>
To: Proxmox VE development discussion <pve-devel@lists.proxmox.com>,
Erik Hollensbe <erik@hollensbe.org>
Subject: Re: [pve-devel] OpenAPI converter (similar to JSONSchema generator)
Date: Fri, 18 Dec 2020 11:02:35 +0100 [thread overview]
Message-ID: <4863e56a-6841-81cd-9d58-b1924d74e7da@proxmox.com> (raw)
In-Reply-To: <CADHGEEKK-nzX6QDmu-4eqTDNhoQd54iLHL7891mJ4Arj6C7OMQ@mail.gmail.com>
Hi,
first, thanks for your interest in working on and improving Proxmox VE!
On 17/12/2020 05:45, Erik Hollensbe wrote:
> The specification is here and has similar semantics and goals to
> jsonschema, just really good codegen support:
> https://swagger.io/specification/
>
> What I am proposing is a module that sits alongside the JSONSchema package
> and generates an openapi.json/swagger.json on http request, so that the
> client can either be dynamically configured or generated from a request to
> a server. It could also be served on proxmox's website for simpler
> generation.
You mean, you'd like to add a new API endpoint returning the schema in
openAPI format?
>
> It looks like this is doable with the separation of concerns in JSONSchema
> and RESTHandler, since all the former does is extract data from the latter,
> unless I am misreading things. A swagger/openapi generator would do
> something similar. Swagger is just JSON as well, so there's no magic
> formatting or parsing that needs to take place.
JSONSchema and openAPI are closely related, so it should not be really hard
to do, albeit, we're not 100% JSONSchema compatible, meaning that we do not
support all of it and possible use some things which it has no support for
(on the latter I'd need to re-check closely though).
The code dumping it should live along side of JSONSchema in pve-common, but
the actual API endpoint (if that is what you want) has to go in pve-manager,
as only there we have all dependencies available for actual use.
>
> I am comfortable (but rusty) in perl and can read your json schema code
> fine, and am willing to make the patches to generate the specification
> assuming this is OK and acceptable to the powers that be.
We have a wiki giving some pointers about build environment, code-style,
CLA, and sending patches - maybe it helps checking it out:
https://pve.proxmox.com/wiki/Developer_Documentation
>
> The advantages are a healthy ecosystem of code generators (
> https://github.com/OpenAPITools/openapi-generator) and also documentation
> tools, like this one: https://redocly.github.io/redoc/.
>
> I think the package could be delivered in a few weeks assuming a
> development environment is easy enough to get going.
>
See the following readme for some rough instructions, note that just for the
perl development lots
https://git.proxmox.com/?p=pve-common.git;a=blob_plain;f=README.dev;hb=HEAD
cheers,
Thomas
next prev parent reply other threads:[~2020-12-18 10:02 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-12-17 4:45 Erik Hollensbe
2020-12-18 10:02 ` Thomas Lamprecht [this message]
2020-12-18 11:27 ` Erik Hollensbe
2020-12-18 13:12 ` Thomas Lamprecht
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4863e56a-6841-81cd-9d58-b1924d74e7da@proxmox.com \
--to=t.lamprecht@proxmox.com \
--cc=erik@hollensbe.org \
--cc=pve-devel@lists.proxmox.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal