-
Notifications
You must be signed in to change notification settings - Fork 5
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
Extend e2e chainsaw tests #249
Conversation
…nged back when we have the leaselocker deadlocks/timeouts under control)
…is OOMKilled with current limits of 750m/768Mi)
…created during the chainsaw test (description e2e/...)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very nice these tests, I like that it checks all fields and we also make sure not to accidentally change a status field. Only small comments, I did not repeat them for all tests but obviously they apply to all the tests of the same type. Will you add the update of custom fields still in this mr?
tests/e2e/Prefix/IPv4/prefixclaim-ipv4-parentprefixselector-restore/chainsaw-test.yaml
Outdated
Show resolved
Hide resolved
tests/e2e/Prefix/IPv4/prefixclaim-ipv4-parentprefixselector-restore/chainsaw-test.yaml
Show resolved
Hide resolved
tests/e2e/Prefix/IPv4/prefixclaim-ipv4-restore/chainsaw-test.yaml
Outdated
Show resolved
Hide resolved
tests/e2e/Prefix/IPv6/prefixclaim-ipv6-apply-update/netbox_v1_prefixclaim-update.yaml
Show resolved
Hide resolved
tests/e2e/ipaddress/ipv4/ipaddressclaim-ipv4-apply-update/chainsaw-test.yaml
Show resolved
Hide resolved
tests/e2e/iprange/ipv4/iprangeclaim-ipv4-invalid-size/chainsaw-test.yaml
Outdated
Show resolved
Hide resolved
tests/e2e/iprange/ipv4/iprangeclaim-ipv4-invalid-tenant/chainsaw-test.yaml
Outdated
Show resolved
Hide resolved
…store/chainsaw-test.yaml Co-authored-by: Fabian <30692464+faebr@users.noreply.github.com>
…-test.yaml Co-authored-by: Fabian <30692464+faebr@users.noreply.github.com>
…aw-test.yaml Co-authored-by: Fabian <30692464+faebr@users.noreply.github.com>
Thanks for the thorough review. I'd keep the todos since they'd require some adjustments to the Controllers and/or some troubleshooting. |
Looks good to me, just a small question, do you know why this error is here: https://github.com/netbox-community/netbox-operator/actions/runs/13636803517/job/38117487010?pr=249#step:7:1007 |
its during cleanup from the cleanup script so all good |
Adds new e2e tests for IpAddressClaim and IpRangeClaim and aligns naming of all test resources (folder names, chainsaw test names and prefixes for k8s CRs).
Changes:
netboxOperatorRestorationHash
from assertions everywhere except in the restoration and parentPrefixSelector tests.metadata.name
compatible name. This name is used in the path of the file in git, in the.metadata.name
of the chainsaw Test yaml and as name (prefix) for all CRs involved (e.g. PrefixClaim, IpAddressClaim, IpRangeClaim)description
to Chainsaw Test resources to allow for further explanation (What does the test do?)Closes #145