all lists on lists.proxmox.com
 help / color / mirror / Atom feed
From: Erik Hollensbe <erik@hollensbe.org>
To: Thomas Lamprecht <t.lamprecht@proxmox.com>
Cc: Proxmox VE development discussion <pve-devel@lists.proxmox.com>
Subject: Re: [pve-devel] OpenAPI converter (similar to JSONSchema generator)
Date: Fri, 18 Dec 2020 03:27:14 -0800	[thread overview]
Message-ID: <CADHGEE+ERMaTOKQAUfJegCceD8iC-2nK3Nf96XMwfgniUkURWw@mail.gmail.com> (raw)
In-Reply-To: <4863e56a-6841-81cd-9d58-b1924d74e7da@proxmox.com>

Thanks for your assistance! I will look into these things over the weekend.

As for the swagger endpoint, yes, more or less, I was going to assume there
could be a /api2/swagger.json or similar route that could resolve to the
API definition.

On Fri, Dec 18, 2020 at 2:02 AM Thomas Lamprecht <t.lamprecht@proxmox.com>
wrote:

> 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
>
>


  reply	other threads:[~2020-12-18 11:27 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
2020-12-18 11:27   ` Erik Hollensbe [this message]
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=CADHGEE+ERMaTOKQAUfJegCceD8iC-2nK3Nf96XMwfgniUkURWw@mail.gmail.com \
    --to=erik@hollensbe.org \
    --cc=pve-devel@lists.proxmox.com \
    --cc=t.lamprecht@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