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

check license from nsxt side #496

Merged
merged 1 commit into from
Feb 21, 2024
Merged

check license from nsxt side #496

merged 1 commit into from
Feb 21, 2024

Conversation

TaoZou1
Copy link
Contributor

@TaoZou1 TaoZou1 commented Jan 22, 2024

While init, it will check the license from nsxt side. If CONTAINER license is disable, it will reboot. If DFW license is disable, security policy will be only response for DELETE operation. It will check license periodically.

Test Done:
no CONTAINER license
1. if no CONTAINER license, nsx-operator should reset
CONTAINER license enable, DFW disable
1. nsx-operator could bootup
2. security policy failed to create or update
3. security policy could be deleted
CONTAINER license enable, DFW enable -> CONTAINER/DFW disable
1. nsx-operator restart due to DFW changed

@TaoZou1 TaoZou1 force-pushed the license branch 2 times, most recently from 210374d to 35a18bf Compare January 22, 2024 02:53
@TaoZou1
Copy link
Contributor Author

TaoZou1 commented Jan 22, 2024

/e2e

cmd/main.go Show resolved Hide resolved
cmd/main.go Outdated Show resolved Hide resolved
cmd/main.go Outdated Show resolved Hide resolved
cmd/main.go Show resolved Hide resolved
pkg/metrics/metrics.go Outdated Show resolved Hide resolved
@TaoZou1 TaoZou1 force-pushed the license branch 2 times, most recently from 60a5653 to 4c6c7a8 Compare January 26, 2024 09:17
pkg/nsx/util/license.go Outdated Show resolved Hide resolved
pkg/nsx/client.go Outdated Show resolved Hide resolved
cmd/main.go Outdated Show resolved Hide resolved
pkg/nsx/cluster.go Outdated Show resolved Hide resolved
pkg/nsx/cluster.go Outdated Show resolved Hide resolved
pkg/nsx/util/license.go Outdated Show resolved Hide resolved
pkg/nsx/services/securitypolicy/firewall.go Show resolved Hide resolved
@TaoZou1 TaoZou1 force-pushed the license branch 5 times, most recently from 3d85843 to cd94db8 Compare February 6, 2024 01:21
pkg/config/config.go Outdated Show resolved Hide resolved
cmd/main.go Outdated Show resolved Hide resolved
@TaoZou1 TaoZou1 force-pushed the license branch 3 times, most recently from 42e0fa8 to 996bc0e Compare February 19, 2024 02:31
cmd/main.go Show resolved Hide resolved
cmd/main.go Outdated Show resolved Hide resolved
pkg/config/config.go Outdated Show resolved Hide resolved
pkg/nsx/cluster.go Outdated Show resolved Hide resolved
pkg/nsx/services/vpc/vpc.go Show resolved Hide resolved
pkg/nsx/util/license.go Outdated Show resolved Hide resolved
@TaoZou1 TaoZou1 force-pushed the license branch 3 times, most recently from 9006ec3 to eeb58ff Compare February 19, 2024 10:25
cmd/main.go Show resolved Hide resolved
pkg/nsx/cluster.go Show resolved Hide resolved
pkg/nsx/services/vpc/vpc.go Outdated Show resolved Hide resolved
pkg/nsx/util/license.go Outdated Show resolved Hide resolved
While init, it will check the license from nsxt side. If CONTAINER
license is disable, it will reboot. If DFW license is disable,
security policy will be only response for DELETE operation.
It will run a routine to check license periodically. If there is
no DFW license, it will check license more frequently

SecurityPolicy controller will check if error is invalid license error.

Test Done:
no CONTAINER license
1. if no CONTAINER license, nsx-operator should reset
CONTAINER license enable, DFW disable
1. nsx-operator could bootup
2. security policy failed to create or update
3. security policy could be deleted
CONTAINER license enable, DFW enable -> CONTAINER/DFW disable
1. nsx-operator restart due to DFW changed
no DFW license, but nsx-operator try to create security policy
1. nsx-operator restart due to invalid license error
@TaoZou1 TaoZou1 merged commit 83bbf26 into vmware-tanzu:main Feb 21, 2024
2 checks passed
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