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

Add e2e test for Subnet #377

Closed
wants to merge 2 commits into from
Closed

Conversation

jwsui
Copy link
Contributor

@jwsui jwsui commented Oct 31, 2023

Changes include:

  1. Add CRD clientset to perform CRUD of CRDs.
  2. Add SubnetSet test cases.

dantingl
dantingl previously approved these changes Nov 13, 2023
Copy link
Contributor

@dantingl dantingl left a comment

Choose a reason for hiding this comment

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

LGTM

@zhengxiexie
Copy link
Contributor

zhengxiexie
zhengxiexie previously approved these changes Nov 30, 2023
Copy link
Contributor

@zhengxiexie zhengxiexie left a comment

Choose a reason for hiding this comment

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

LGTM

@jwsui jwsui dismissed zhengxiexie’s stale review December 1, 2023 01:55

The merge-base changed after approval.

@jwsui
Copy link
Contributor Author

jwsui commented Dec 1, 2023

/e2e

@jwsui
Copy link
Contributor Author

jwsui commented Dec 1, 2023

/e2e

For cluster-wide resources, `// +genclient:nonNamespaced` is required to
generate CRD client. Otherwise, it will be treated as namespace scope
resource.
Changes include:
1. Add CRD clientset to perform CRUD of CRDs.
2. Add SubnetSet test cases.
@jwsui
Copy link
Contributor Author

jwsui commented Dec 5, 2023

#422

Create a new PR, closing this one.

@jwsui jwsui closed this Dec 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants