Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support e2e tests against multiple NetBox versions #167

Closed

Conversation

henrybear327
Copy link
Collaborator

@henrybear327 henrybear327 commented Dec 12, 2024

Since we are introducing the e2e test and running against real NetBox instances, it makes sense to test against different versions of NetBox, so we can flag issues early on.

CI will now run all e2e tests against the supported versions.

The supported NetBox versions are (the latest patch version for the major.minor release):

  • 3.7.8
  • 4.0.11
  • 4.1.7 (4.1.8 not yet supported by the NetBox Helm chart yet)

Instead of create-kind, we now have create-kind-3.7.8, create-kind-4.0.11, and create-kind-4.1.7. The same goes for make test-e2e.

The patching mechanism is used to apply database schema fixes and demo data version changes. When we introduce support for new versions, usually only the database schema would need to be changed.

@henrybear327 henrybear327 self-assigned this Dec 12, 2024
@henrybear327 henrybear327 marked this pull request as draft December 12, 2024 20:11
@henrybear327
Copy link
Collaborator Author

There might be an impact on netbox-community/netbox-chart#397 if the SQL data-loading mechanism is leveraged, as I might be splitting up the SQL file to support different NetBox versions.

@henrybear327 henrybear327 force-pushed the e2e/add_multi_version_support branch 4 times, most recently from ce87756 to b2c0462 Compare December 12, 2024 21:02
@henrybear327 henrybear327 force-pushed the e2e/add_multi_version_support branch 3 times, most recently from 9358c31 to 71b1543 Compare December 12, 2024 23:48
@henrybear327 henrybear327 force-pushed the e2e/add_multi_version_support branch 2 times, most recently from 4ce5415 to b152472 Compare December 13, 2024 00:49
Increase timeout
Fix misc.
@henrybear327 henrybear327 force-pushed the e2e/add_multi_version_support branch from b152472 to 884c1d9 Compare December 13, 2024 01:01
@henrybear327 henrybear327 marked this pull request as ready for review December 13, 2024 01:02
It's observed that the cluster might take 1+ min to apply and have the CR finalized...
Makefile Show resolved Hide resolved
kind/local-env.sh Show resolved Hide resolved
@henrybear327
Copy link
Collaborator Author

Blocked by #175

@henrybear327 henrybear327 deleted the branch ci/tidy_local-env.sh December 16, 2024 12:01
@henrybear327 henrybear327 deleted the e2e/add_multi_version_support branch December 16, 2024 12:02
@henrybear327 henrybear327 restored the e2e/add_multi_version_support branch December 16, 2024 12:02
@henrybear327
Copy link
Collaborator Author

Recreated at #176

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.

2 participants