[PR #308] [MERGED] New script: NetBox LXC #2481

Closed
opened 2025-11-20 05:30:15 -05:00 by saavagebueno · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/community-scripts/ProxmoxVE/pull/308
Author: @bvdberg01
Created: 11/17/2024
Status: Merged
Merged: 11/18/2024
Merged by: @MickLesk

Base: mainHead: main


📝 Commits (3)

📊 Changes

3 files changed (+272 additions, -0 deletions)

View changed files

ct/netbox.sh (+114 -0)
install/netbox-install.sh (+119 -0)
json/netbox.json (+39 -0)

📄 Description

Note

We are meticulous when it comes to merging code into the main branch, so please understand that we may reject pull requests that do not meet the project's standards. It's never personal. Also, game-related scripts have a lower chance of being merged.

Description

Adding script for NetBox as requested in discussion (#50) and (#1327) in tteck repository.

Tested on my Proxmox installation and have it running for a few days. Update script is also tested by installing older version from netbox on install (changing line 46 in install/netbox-install.sh to RELEASE="4.1.0") and after install running the update script.

Type of change

Please check the relevant option(s):

  • Bug fix (non-breaking change that resolves an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (a fix or feature that would cause existing functionality to change unexpectedly)
  • New script (a fully functional and thoroughly tested script or set of scripts.)

Prerequisites

The following efforts must be made for the PR to be considered. Please check when completed:

  • Self-review performed (I have reviewed my code, ensuring it follows established patterns and conventions)
  • Testing performed (I have tested my changes, ensuring everything works as expected)
  • Documentation updated (I have updated any relevant documentation)

Additional Information (optional)

Installed NetBox LXC:
image

Running update script:
image
image

Running update script on latest version:
image

Running NetBox:
image


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/community-scripts/ProxmoxVE/pull/308 **Author:** [@bvdberg01](https://github.com/bvdberg01) **Created:** 11/17/2024 **Status:** ✅ Merged **Merged:** 11/18/2024 **Merged by:** [@MickLesk](https://github.com/MickLesk) **Base:** `main` ← **Head:** `main` --- ### 📝 Commits (3) - [`515eb30`](https://github.com/community-scripts/ProxmoxVE/commit/515eb303f096bcfe575cecca3d2cad233a73286a) new scripts for NetBox - [`60da4ee`](https://github.com/community-scripts/ProxmoxVE/commit/60da4eee924884c0459a9757a7ee50a58bf9ff47) Update json/netbox.json - [`a0590c0`](https://github.com/community-scripts/ProxmoxVE/commit/a0590c0be3dec36f4ebc23d28d9113b8508a9037) Fixed issues after pr ### 📊 Changes **3 files changed** (+272 additions, -0 deletions) <details> <summary>View changed files</summary> ➕ `ct/netbox.sh` (+114 -0) ➕ `install/netbox-install.sh` (+119 -0) ➕ `json/netbox.json` (+39 -0) </details> ### 📄 Description > [!NOTE] > We are meticulous when it comes to merging code into the main branch, so please understand that we may reject pull requests that do not meet the project's standards. It's never personal. Also, game-related scripts have a lower chance of being merged. ## Description Adding script for [NetBox](https://github.com/netbox-community/netbox) as requested in discussion [(#50)](https://github.com/tteck/Proxmox/discussions/50#discussioncomment-11068137) and [(#1327)](https://github.com/tteck/Proxmox/discussions/1327) in tteck repository. Tested on my Proxmox installation and have it running for a few days. Update script is also tested by installing older version from netbox on install (changing line 46 in install/netbox-install.sh to RELEASE="4.1.0") and after install running the update script. ## Type of change Please check the relevant option(s): - [ ] Bug fix (non-breaking change that resolves an issue) - [ ] New feature (non-breaking change that adds functionality) - [ ] Breaking change (a fix or feature that would cause existing functionality to change unexpectedly) - [X] New script (a fully functional and thoroughly tested script or set of scripts.) ## Prerequisites The following efforts must be made for the PR to be considered. Please check when completed: - [X] Self-review performed (I have reviewed my code, ensuring it follows established patterns and conventions) - [X] Testing performed (I have tested my changes, ensuring everything works as expected) - [X] Documentation updated (I have updated any relevant documentation) ## Additional Information (optional) **Installed NetBox LXC:** ![image](https://github.com/user-attachments/assets/9de9e5e9-cd0d-41e7-98c7-40a3a1cba4d2) **Running update script:** ![image](https://github.com/user-attachments/assets/310ee62f-d8ea-4b92-8bbe-7a15aae48ad4) ![image](https://github.com/user-attachments/assets/b40faa2e-caad-4e03-95b2-e78947ac12b9) **Running update script on latest version:** ![image](https://github.com/user-attachments/assets/60bbab1e-4914-4de7-b783-1ac569bc2788) **Running NetBox:** ![image](https://github.com/user-attachments/assets/9ed96fb3-5fba-4805-a305-87e02f3329ca) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
saavagebueno added the pull-request label 2025-11-20 05:30:15 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: SVI/ProxmoxVE#2481