Skip to content

Conversation

@pablogarciamiranda
Copy link
Collaborator

@pablogarciamiranda pablogarciamiranda commented Apr 28, 2025

I've adapted the deploy-netbox.sh script as it could be useful to run netbox inside a vCluster for testing purposes.

I've just added a --vcluster to activate the mode. Otherwise, by default, it will deploy to a kind cluster as it's the case now.

How to test?

VCluster Mode (new)

 $ vcluster create test \
  --namespace test \
  --create-namespace

$ ./deploy-netbox.sh test "4.1.8" test --vcluster

Kind Mode (existing)

$ ./deploy-netbox.sh kind "4.1.8" default

@pablogarciamiranda pablogarciamiranda changed the title Adapting deploy-netbox.sh to a vCluster deployment Draft: Adapting deploy-netbox.sh to a vCluster deployment Apr 28, 2025
@pablogarciamiranda pablogarciamiranda marked this pull request as draft April 28, 2025 13:54
@pablogarciamiranda pablogarciamiranda changed the title Draft: Adapting deploy-netbox.sh to a vCluster deployment Adapting deploy-netbox.sh to a vCluster deployment Apr 29, 2025
@pablogarciamiranda pablogarciamiranda marked this pull request as ready for review April 29, 2025 14:35
@pablogarciamiranda
Copy link
Collaborator Author

Fixed @faebr comments. I think this PR could be already merged.

I am preparing another PR with all the changes that @bruelea suggested to be able to allow overriding image registry and Helm chart sources (Still in draft mode, as I need to refactor, but already working #296)

@faebr
Copy link
Collaborator

faebr commented May 2, 2025

Fixed @faebr comments. I think this PR could be already merged.

I am preparing another PR with all the changes that @bruelea suggested to be able to allow overriding image registry and Helm chart sources (Still in draft mode, as I need to refactor, but already working #296)

thanks, looks good to me now. There is still an open conversation with @bruelea which I believe is not resolved yet? There is a mixup between namespace for the vcluster deployment and namespace for the netbox deployment.

@pablogarciamiranda pablogarciamiranda requested a review from bruelea May 5, 2025 09:39
@pablogarciamiranda pablogarciamiranda requested a review from faebr May 5, 2025 11:45
@pablogarciamiranda pablogarciamiranda merged commit aa9a70c into main May 6, 2025
8 checks passed
vaishutin pushed a commit to vaishutin/netbox-operator that referenced this pull request Aug 17, 2025
)

* Adapting deploy-netbox.sh to a vCluster deployment

* Debugging demo data

* Adding steps to load demo-data into NetBox inside the vcluster

* Adding description to script

* Adding kind path

* Adding namespaces

* Removing sleep used for testing purposes

* Fixing script comment

* Adding variable IS_VCLUSTER to easily check in all the if statements

* Refactoring part of loading demo-data into NetBox

* Removing unnecessary code & comments

* Adding comment to clarify that vCluster will be deployed to the NAMESPACE passed as an argument

---------

Co-authored-by: Pablo Garcia Miranda <pablo.garciamiranda@swisscom.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants