public inbox for pve-devel@lists.proxmox.com
 help / color / mirror / Atom feed
From: Thomas Lamprecht <t.lamprecht@proxmox.com>
To: Proxmox VE development discussion <pve-devel@lists.proxmox.com>,
	aderumier@odiso.com
Cc: pve-devel <pve-devel@pve.proxmox.com>
Subject: Re: [pve-devel] sdn: looking to unify .cfg files, need opinions about config format
Date: Sun, 18 Apr 2021 19:01:46 +0200	[thread overview]
Message-ID: <8ed4f090-2021-bbe1-4b7b-ceef71d7f4ff@proxmox.com> (raw)
In-Reply-To: <ee2f7206124fc8adb6d6e9b6691b8388f8622aeb.camel@odiso.com>

On 12.01.21 10:19, aderumier@odiso.com wrote:
> Hi,
> 
> I'm looking to unify sdn .cfg files with only 1 file,
> with something different than section config format.
> 
> We have relationship like  zones->vnets->subnets,
> so I was thinking about something like this:
> 
> 
> 
> [zone myzone]
>    type: vxlan
>    option1: xxx
>    option2: xxx
> [[vnet myvnet]]
>    option1: xxx
>    option2: xxx
> [[[subnet 10.0.0.0/8]]]
>    option1: xxx
>    option2: xxx
> 
> 
> [controller  mycontroller]
>    type: evpn
>    option1: xxx
>    option2: xxx
> 
> [dns  mydns]
>    type: powerdns
>    option1: xxx
>    option2: xxx
> 
> 
> What do you think about this ?

That looks like section config, just spelled differently?

But yes, the way section config does schema and types are not ideal when combined
with quite different things.

Maybe we should really just go the simple way and keep it separated for now.

For zones it works good this way, there exist different types and we can use that as
section config type. Subnets and vnets could be combined as vnets are really not that
specially I guess?

We had a mail about what would be Ok to merge, but I do not remember/find it anymore...

> 
> Another way could be a simple yaml config file. (but I think it's not
> really matching currents proxmox configs formats)
> 

I do not like yaml to much, it looks simple first but can do way to much (turing
complete, IIRC) and we do not really use it anywhere atm., so that would mean lots
of new tooling/work to do to handle it sanely and as first-class citizen in PVE
stack...

My goal would be to do a pve-network bump at end of next week, and for that we
need pve-cluster bump.

Currently there we get three new configs:

1. ipams, different management plugins (types), so OK to be own section config
2. dns, different, APIs/DNS servers (types), so OK to be own section config
3. subnets, only one type, or?

hmm, rethinking this now it could be OK to keep as is... While subnets could
possibly be merged into vnets, there's a mediocre benefit there, API could
maybe even get more complicated?

If we'd bump now the biggest thing missing is applying an IP to a VM and CT.

For a CT we can quite easily do it.

For a VM we could even need to support different ways?

* DHCP (?)
* cloudinit
* would the qemu-guest-agent work too?

IMO that's the biggest thing left, or what do you think?




  reply	other threads:[~2021-04-18 17:01 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-01-12  9:19 aderumier
2021-04-18 17:01 ` Thomas Lamprecht [this message]
2021-04-20  7:51   ` alexandre derumier

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=8ed4f090-2021-bbe1-4b7b-ceef71d7f4ff@proxmox.com \
    --to=t.lamprecht@proxmox.com \
    --cc=aderumier@odiso.com \
    --cc=pve-devel@lists.proxmox.com \
    --cc=pve-devel@pve.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal