Hi,

and thanks to the both of you!

Can you please resend the mail to our mailing list so that it is available for everyone please? Another tiny thing, we don't use "From: Lou Lecrivain <lou.lecriv...@wdz.de>" but "Signed-off-by: Lou Lecrivain <lou.lecriv...@wdz.de>" which indicates that contribution was created in whole or in part by yourself and you have the right to submit it under our open source license. jonathan crystall can use the "Co-developed-by: Jonatan Crystall <jonatan.cryst...@gwdg.de>" to make it clear that he contributed the code.

Best wishes,
Hannes Dürr

On 3/6/25 12:08, lou.lecriv...@orange.fr wrote:
From: Lou Lecrivain <lou.lecriv...@wdz.de>

Following review from Hannes,

I have made some improvements to the form of the code for the pve-network part.
No functional changes.

Related changes to pve-manager and pve-docs are also included the email thread.

BR

Lou Lecrivain (3):
   ipam: nautobot: base plugin
   ipam: nautobot: add testing for nautobot plugin
   ipam: nautobot: add checks for prefix deletion

  src/PVE/API2/Network/SDN/Ipams.pm             |   1 +
  src/PVE/Network/SDN/Ipams.pm                  |   3 +
  src/PVE/Network/SDN/Ipams/Makefile            |   2 +-
  src/PVE/Network/SDN/Ipams/NautobotPlugin.pm   | 491 ++++++++++++++++++
  src/test/ipams/nautobot/expected.add_ip       |  11 +
  .../ipams/nautobot/expected.add_ip_notgateway |  11 +
  .../ipams/nautobot/expected.add_next_freeip   |  11 +
  src/test/ipams/nautobot/expected.add_subnet   |  11 +
  src/test/ipams/nautobot/expected.del_ip       |  11 +
  src/test/ipams/nautobot/expected.update_ip    |  11 +
  src/test/ipams/nautobot/ipam_config           |  24 +
  src/test/ipams/nautobot/sdn_config            |  20 +
  src/test/ipams/netbox/ipam_config             |   8 +-
  src/test/ipams/phpipam/ipam_config            |   8 +-
  14 files changed, 620 insertions(+), 3 deletions(-)
  create mode 100644 src/PVE/Network/SDN/Ipams/NautobotPlugin.pm
  create mode 100644 src/test/ipams/nautobot/expected.add_ip
  create mode 100644 src/test/ipams/nautobot/expected.add_ip_notgateway
  create mode 100644 src/test/ipams/nautobot/expected.add_next_freeip
  create mode 100644 src/test/ipams/nautobot/expected.add_subnet
  create mode 100644 src/test/ipams/nautobot/expected.del_ip
  create mode 100644 src/test/ipams/nautobot/expected.update_ip
  create mode 100644 src/test/ipams/nautobot/ipam_config
  create mode 100644 src/test/ipams/nautobot/sdn_config



_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Reply via email to