Skip to content

Conversation

@henrybear327
Copy link
Collaborator

@henrybear327 henrybear327 commented Dec 11, 2024

There are several aspects to be updated:

  • In the local-env.sh script, the images listed in Images would need to be changed, as updating the helm chart will cause the required image versions to change, and we need to pre-load them into the kind cluster since it can't pull images directly from the outside world (I checked kubectl describe/logs to see what images failed to load, and update the version strings accordingly)
  • If the major/minor version is changed, we need to update the NetBox demo data link [1] in the load-data.sh script

Based on the experience of migrating from 4.0.x to 4.1.x, the database schema might change. I dumped the database out by executing kubectl exec pod/netbox-db-0 -- bash -c "pg_dump --clean -U postgres netbox" > database.sql before and after the upgrade to compare the difference.

Reference:
[1] https://github.com/netbox-community/netbox-demo-data/blob/master/sql/netbox-demo-v4.1.sql

@henrybear327 henrybear327 requested a review from bruelea December 11, 2024 10:59
@henrybear327 henrybear327 self-assigned this Dec 11, 2024
@henrybear327 henrybear327 changed the title Fix netbox image version Upgrade NetBox image version Dec 11, 2024
@henrybear327 henrybear327 force-pushed the ci/tidy_local-env.sh branch 2 times, most recently from f95caa0 to 2b335de Compare December 12, 2024 12:26
@henrybear327 henrybear327 marked this pull request as draft December 12, 2024 12:54
@henrybear327 henrybear327 force-pushed the ci/tidy_local-env.sh branch 3 times, most recently from f892988 to a29e761 Compare December 12, 2024 17:35
There are several aspects to be updated:
- In the `local-env.sh` script, the images listed in `Images` would need
to be changed, as updating the helm chart will cause the required image
versions to change, and we need to pre-load them into the kind cluster
since it can't pull images directly from the outside world (I checked
`kubectl describe/logs` to see what images failed to load, and update
the version strings accordingly)
- If the major/minor version is changed, we need to update the NetBox
demo data link [1] in the `load-data.sh` script

Based on the experience of migrating from 4.0.x to 4.1.x, the database
schema might change. I dumped the database out by executing
`kubectl exec pod/netbox-db-0 -- bash -c "pg_dump --clean -U postgres netbox" > database.sql`
before and after the upgrade to compare the difference.

Reference:
[1] https://github.com/netbox-community/netbox-demo-data/blob/master/sql/netbox-demo-v4.1.sql
@henrybear327 henrybear327 marked this pull request as ready for review December 12, 2024 17:56
@henrybear327 henrybear327 requested a review from faebr December 12, 2024 17:56
Copy link
Collaborator

@alexandernorth alexandernorth left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, thanks

@henrybear327 henrybear327 merged commit 119d4a0 into main Dec 13, 2024
6 checks passed
@henrybear327 henrybear327 deleted the ci/tidy_local-env.sh branch December 16, 2024 12:01
vaishutin pushed a commit to vaishutin/netbox-operator that referenced this pull request Aug 17, 2025
Upgrade NetBox version

There are several aspects to be updated:
- In the `local-env.sh` script, the images listed in `Images` would need
to be changed, as updating the helm chart will cause the required image
versions to change, and we need to pre-load them into the kind cluster
since it can't pull images directly from the outside world (I checked
`kubectl describe/logs` to see what images failed to load, and update
the version strings accordingly)
- If the major/minor version is changed, we need to update the NetBox
demo data link [1] in the `load-data.sh` script

Based on the experience of migrating from 4.0.x to 4.1.x, the database
schema might change. I dumped the database out by executing
`kubectl exec pod/netbox-db-0 -- bash -c "pg_dump --clean -U postgres netbox" > database.sql`
before and after the upgrade to compare the difference.

Reference:
[1] https://github.com/netbox-community/netbox-demo-data/blob/master/sql/netbox-demo-v4.1.sql
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.

3 participants