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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

add e2e for new ippool feature #2981

Merged
merged 1 commit into from Jun 28, 2023
Merged

Conversation

zhangzujian
Copy link
Member

@zhangzujian zhangzujian commented Jun 25, 2023

What type of this PR

  • Tests

Which issue(s) this PR fixes:

Fixes #(issue-number)

WHAT

馃 Generated by Copilot at fc7d4ed

This pull request adds and updates the test packages and methods for the IPPool and namespace annotation features in the kube-ovn project. It also enhances the framework package with new packages, methods, and error handling. Additionally, it renames some PatchPod methods to Patch for consistency. The affected files are test/e2e/framework/deployment.go, test/e2e/framework/util.go, test/e2e/kube-ovn/ipam/ipam.go, pkg/internal/big_int.go, test/e2e/framework/ippool.go, test/e2e/framework/namespace.go, test/e2e/framework/pod.go, and test/e2e/kube-ovn/qos/qos.go.

馃 Generated by Copilot at fc7d4ed

The pull request has many changes
To test the IPPool and its ranges
It adds new methods and files
To the framework and its styles
And patches the pods with some patches

HOW

馃 Generated by Copilot at fc7d4ed

  • Add methods to create, update, patch, delete, and validate IPPools in the framework package (link)
  • Add methods to get, create, patch, and delete namespaces in the framework package (link)
  • Add methods to patch and wait for deployment rollout in the framework package (link, link)
  • Add method to check if BigInt is equal to int64 in the internal package (link)
  • Rename PatchPod method to Patch in the framework and qos packages (link, link)
  • Use strset package to generate and manipulate random IPs in the framework package (link, link)
  • Use ipam package to define and manipulate IP and IPRange objects in the ipam package (link)
  • Add test case to verify IPPool feature in the ipam package (link)
  • Add and modify imports and variables in the framework, ipam, and qos packages (link, link, link, link, link, link)
  • Add step to delete ippool in the ipam package (link)
  • Use sort package to sort IPs by numerical order in the ipam package (link)

@zhangzujian zhangzujian added the test automation tests label Jun 25, 2023
@zhangzujian zhangzujian force-pushed the ippool-e2e branch 3 times, most recently from 695e91e to 6d577e1 Compare June 25, 2023 13:17
@zhangzujian zhangzujian marked this pull request as ready for review June 25, 2023 14:23
@zhangzujian zhangzujian merged commit 30cd09e into kubeovn:master Jun 28, 2023
58 checks passed
@zhangzujian zhangzujian deleted the ippool-e2e branch June 28, 2023 02:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
test automation tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants