-
Notifications
You must be signed in to change notification settings - Fork 4
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
- Loading branch information
1 parent
49151d0
commit 458cfb2
Showing
25 changed files
with
903 additions
and
18 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,3 @@ | ||
| golang-version=1.23 | ||
| kind-version=v0.25.0 | ||
| kind-image=kindest/node:v1.31.2 |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,73 @@ | ||
| # Modified from https://github.com/prometheus-operator/prometheus-operator/blob/main/.github/workflows/e2e-feature-gated.yaml | ||
| name: e2e-tests | ||
| on: | ||
| push: | ||
| branches: | ||
| - main | ||
| tags: | ||
| - v* | ||
| pull_request: | ||
| concurrency: | ||
| group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.ref }} | ||
| cancel-in-progress: true | ||
| env: | ||
| NETBOX_HOST: demo.netbox.dev | ||
| AUTH_TOKEN: 0123456789abcdef0123456789abcdef01234567 | ||
| POD_NAMESPACE: default | ||
| HTTPS_ENABLE: true | ||
| NETBOX_RESTORATION_HASH_FIELD_NAME: netboxOperatorRestorationHash | ||
| jobs: | ||
| e2e-tests: | ||
| name: E2E tests for netbox operator | ||
| runs-on: ubuntu-latest | ||
| steps: | ||
| - uses: actions/checkout@692973e3d937129bcbf40652eb9f2f61becf3332 # v4.1.7 | ||
| - uses: actions/setup-go@41dfa10bad2bb2ae585af6ee5bb4d7d973ad74ed # v5.1.0 | ||
| with: | ||
| go-version: 1.23.4 | ||
| - name: Import environment variables from file | ||
| run: | | ||
| cat ".github/env" >> "$GITHUB_ENV" | ||
| echo "E2E_DIAGNOSTIC_DIRECTORY=$(mktemp -d)" >> "$GITHUB_ENV" | ||
| - name: Start kind cluster | ||
| uses: helm/kind-action@0025e74a8c7512023d06dc019c617aa3cf561fde # v1.10.0 | ||
| with: | ||
| version: ${{ env.kind-version }} | ||
| node_image: ${{ env.kind-image }} | ||
| wait: 300s | ||
| config: ./tests/e2e/kind-config.yaml | ||
| cluster_name: e2e | ||
| - name: Wait for cluster to finish bootstraping | ||
| run: | | ||
| echo "Waiting for all nodes to be ready..." | ||
| kubectl wait --for=condition=Ready nodes --all --timeout=120s | ||
| kubectl get nodes | ||
| echo "Waiting for all pods to be ready..." | ||
| kubectl wait --for=condition=Ready pods --all --all-namespaces --timeout=300s | ||
| kubectl get pods -A | ||
| echo "Cluster information" | ||
| kubectl cluster-info | ||
| - name: Setup kind cluster with required software such as NetBox | ||
| run: | | ||
| make create-kind | ||
| - name: Deploy NetBox operator to the kind cluster | ||
| run: | | ||
| make deploy-kind | ||
| - name: Run tests | ||
| env: | ||
| E2E_DIAGNOSTIC_DIRECTORY: ${{ env.E2E_DIAGNOSTIC_DIRECTORY }} | ||
| run: | | ||
| # # Very straight forward way of implementing a test and checking the result | ||
| # kubectl apply -f config/samples/netbox_v1_prefixclaim.yaml | ||
| # kubectl get prefixclaim,prefix,ipaddressclaim,ipaddress,iprange,iprangeclaim | ||
| # kubectl wait --for=condition=ready --timeout=30s prefixclaim.netbox.dev/prefixclaim-sample | ||
| # Use Chainsaw | ||
| make test-e2e | ||
| - name: Upload diagnostics artifact | ||
| if: ${{ failure() }} | ||
| uses: actions/upload-artifact@b4b15b8c7c6ac21ea08fcf65892d2ee8f75cf882 # v4.4.3 | ||
| with: | ||
| name: cluster-state | ||
| path: ${{ env.E2E_DIAGNOSTIC_DIRECTORY }} | ||
| retention-days: 15 |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -50,6 +50,16 @@ install-$(GO_PACKAGE_NAME_GOLANGCI_LINT): | |
| echo "$(GO_PACKAGE_NAME_GOLANGCI_LINT) is installed" ; \ | ||
| fi | ||
|
|
||
| # check if chainsaw is installed or not | ||
| GO_PACKAGE_NAME_CHAINSAW := chainsaw | ||
| install-$(GO_PACKAGE_NAME_CHAINSAW): | ||
| @if [ ! -x "$(GOBIN)/$(GO_PACKAGE_NAME_CHAINSAW)" ]; then \ | ||
| echo "Installing $(GO_PACKAGE_NAME_CHAINSAW)..." ; \ | ||
| go install github.com/kyverno/[email protected] ; \ | ||
| else \ | ||
| echo "$(GO_PACKAGE_NAME_CHAINSAW) is installed" ; \ | ||
| fi | ||
|
|
||
| .PHONY: all | ||
| all: build | ||
|
|
||
|
|
@@ -223,6 +233,10 @@ envtest: $(ENVTEST) ## Download envtest-setup locally if necessary. | |
| $(ENVTEST): $(LOCALBIN) | ||
| test -s $(LOCALBIN)/setup-envtest || GOBIN=$(LOCALBIN) go install sigs.k8s.io/controller-runtime/tools/setup-envtest@87bcfec | ||
|
|
||
| generate_mocks: # TODO: auto install go install go.uber.org/mock/mockgen@latest | ||
| generate_mocks: ## TODO: auto install go install go.uber.org/mock/mockgen@latest | ||
| mkdir -p ${GEN_DIR} | ||
| mockgen -destination ${GEN_DIR}/${NETBOX_MOCKS_OUTPUT_FILE} -source=${INTERFACE_DEFITIONS_DIR} | ||
|
|
||
| .PHONY: test-e2e | ||
| test-e2e: install-$(GO_PACKAGE_NAME_CHAINSAW) | ||
| chainsaw test --namespace e2e | ||
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
95 changes: 95 additions & 0 deletions
95
...s/e2e/Prefix/IPv4/GivenPrefixClaimWhenAppliedThenFailedPrefixExhausted/chainsaw-test.yaml
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,95 @@ | ||
| apiVersion: chainsaw.kyverno.io/v1alpha1 | ||
| kind: Test | ||
| metadata: | ||
| name: prefixclaim-ipv4-apply-exhausted | ||
| spec: | ||
| steps: | ||
| - name: Install CR 1 | ||
| description: Apply prefix claim CR 1 | ||
| try: | ||
| - apply: | ||
| file: netbox_v1_prefixclaim_1.yaml | ||
| - assert: | ||
| resource: | ||
| apiVersion: netbox.dev/v1 | ||
| kind: PrefixClaim | ||
| metadata: | ||
| name: prefixclaim-ipv4-apply-prefixexhausted-1 | ||
| spec: | ||
| comments: your comments | ||
| description: some description | ||
| parentPrefix: 2.1.0.0/24 | ||
| prefixLength: /25 | ||
| preserveInNetbox: false | ||
| site: DM-Akron | ||
| tenant: Dunder-Mifflin, Inc. | ||
| status: | ||
| parentPrefix: 2.1.0.0/24 | ||
| prefix: 2.1.0.0/25 | ||
| prefixName: prefixclaim-ipv4-apply-prefixexhausted-1 | ||
| - assert: | ||
| resource: | ||
| apiVersion: netbox.dev/v1 | ||
| kind: Prefix | ||
| metadata: | ||
| name: prefixclaim-ipv4-apply-prefixexhausted-1 | ||
| spec: | ||
| comments: your comments | ||
| description: some description | ||
| prefix: 2.1.0.0/25 | ||
| site: DM-Akron | ||
| tenant: Dunder-Mifflin, Inc. | ||
| - name: Install CR 2 | ||
| description: Apply prefix claim CR 2 | ||
| try: | ||
| - apply: | ||
| file: netbox_v1_prefixclaim_2.yaml | ||
| - assert: | ||
| resource: | ||
| apiVersion: netbox.dev/v1 | ||
| kind: PrefixClaim | ||
| metadata: | ||
| name: prefixclaim-ipv4-apply-prefixexhausted-2 | ||
| spec: | ||
| comments: your comments | ||
| description: some description | ||
| parentPrefix: 2.1.0.0/24 | ||
| prefixLength: /25 | ||
| preserveInNetbox: false | ||
| site: DM-Akron | ||
| tenant: Dunder-Mifflin, Inc. | ||
| status: | ||
| parentPrefix: 2.1.0.0/24 | ||
| prefix: 2.1.0.128/25 | ||
| prefixName: prefixclaim-ipv4-apply-prefixexhausted-2 | ||
| - assert: | ||
| resource: | ||
| apiVersion: netbox.dev/v1 | ||
| kind: Prefix | ||
| metadata: | ||
| name: prefixclaim-ipv4-apply-prefixexhausted-2 | ||
| spec: | ||
| comments: your comments | ||
| description: some description | ||
| prefix: 2.1.0.128/25 | ||
| site: DM-Akron | ||
| tenant: Dunder-Mifflin, Inc. | ||
| - name: Install CR 3 | ||
| description: Apply prefix claim CR 3 | ||
| try: | ||
| - apply: | ||
| file: netbox_v1_prefixclaim_3.yaml | ||
| - assert: | ||
| resource: | ||
| apiVersion: v1 | ||
| count: 1 | ||
| kind: Event | ||
| type: Warning | ||
| reason: PrefixCRNotCreated | ||
| source: | ||
| component: prefix-claim-controller | ||
| message: Failed to fetch new Prefix from NetBox. parent prefix exhausted, will restart the parent prefix selection process | ||
| involvedObject: | ||
| apiVersion: netbox.dev/v1 | ||
| kind: PrefixClaim | ||
| name: prefixclaim-ipv4-apply-prefixexhausted-3 |
15 changes: 15 additions & 0 deletions
15
...ix/IPv4/GivenPrefixClaimWhenAppliedThenFailedPrefixExhausted/netbox_v1_prefixclaim_1.yaml
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,15 @@ | ||
| apiVersion: netbox.dev/v1 | ||
| kind: PrefixClaim | ||
| metadata: | ||
| labels: | ||
| app.kubernetes.io/name: netbox-operator | ||
| app.kubernetes.io/managed-by: kustomize | ||
| name: prefixclaim-ipv4-apply-prefixexhausted-1 | ||
| spec: | ||
| tenant: "Dunder-Mifflin, Inc." | ||
| site: "DM-Akron" | ||
| description: "some description" | ||
| comments: "your comments" | ||
| preserveInNetbox: false | ||
| parentPrefix: "2.1.0.0/24" | ||
| prefixLength: "/25" |
15 changes: 15 additions & 0 deletions
15
...ix/IPv4/GivenPrefixClaimWhenAppliedThenFailedPrefixExhausted/netbox_v1_prefixclaim_2.yaml
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,15 @@ | ||
| apiVersion: netbox.dev/v1 | ||
| kind: PrefixClaim | ||
| metadata: | ||
| labels: | ||
| app.kubernetes.io/name: netbox-operator | ||
| app.kubernetes.io/managed-by: kustomize | ||
| name: prefixclaim-ipv4-apply-prefixexhausted-2 | ||
| spec: | ||
| tenant: "Dunder-Mifflin, Inc." | ||
| site: "DM-Akron" | ||
| description: "some description" | ||
| comments: "your comments" | ||
| preserveInNetbox: false | ||
| parentPrefix: "2.1.0.0/24" | ||
| prefixLength: "/25" |
15 changes: 15 additions & 0 deletions
15
...ix/IPv4/GivenPrefixClaimWhenAppliedThenFailedPrefixExhausted/netbox_v1_prefixclaim_3.yaml
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,15 @@ | ||
| apiVersion: netbox.dev/v1 | ||
| kind: PrefixClaim | ||
| metadata: | ||
| labels: | ||
| app.kubernetes.io/name: netbox-operator | ||
| app.kubernetes.io/managed-by: kustomize | ||
| name: prefixclaim-ipv4-apply-prefixexhausted-3 | ||
| spec: | ||
| tenant: "Dunder-Mifflin, Inc." | ||
| site: "DM-Akron" | ||
| description: "some description" | ||
| comments: "your comments" | ||
| preserveInNetbox: false | ||
| parentPrefix: "2.1.0.0/24" | ||
| prefixLength: "/25" |
47 changes: 47 additions & 0 deletions
47
...Pv4/GivenPrefixClaimWithIPv4ParentPrefixSelectorWhenAppliedThenSucceed/chainsaw-test.yaml
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,47 @@ | ||
| apiVersion: chainsaw.kyverno.io/v1alpha1 | ||
| kind: Test | ||
| metadata: | ||
| name: prefixclaim-ipv4-parentprefixselector-apply-succeed | ||
| spec: | ||
| steps: | ||
| - name: Apply prefix claim CR and go through each field and check for equality | ||
| try: | ||
| - apply: | ||
| file: netbox_v1_prefixclaim.yaml | ||
| - assert: | ||
| resource: | ||
| apiVersion: netbox.dev/v1 | ||
| kind: PrefixClaim | ||
| metadata: | ||
| name: prefixclaim-ipv4-parentprefixselector-sample | ||
| spec: | ||
| comments: your comments | ||
| description: some description | ||
| parentPrefixSelector: | ||
| environment: Production | ||
| family: IPv4 | ||
| poolName: Pool 1 | ||
| site: DM-Buffalo | ||
| tenant: MY_TENANT | ||
| prefixLength: /31 | ||
| preserveInNetbox: true | ||
| site: DM-Akron | ||
| tenant: MY_TENANT | ||
| status: | ||
| parentPrefix: 3.0.1.0/24 | ||
| prefix: 3.0.1.0/31 | ||
| prefixName: prefixclaim-ipv4-parentprefixselector-sample | ||
| - assert: | ||
| resource: | ||
| apiVersion: netbox.dev/v1 | ||
| kind: Prefix | ||
| metadata: | ||
| name: prefixclaim-ipv4-parentprefixselector-sample | ||
| spec: | ||
| comments: your comments | ||
| description: some description | ||
| prefix: 3.0.1.0/31 | ||
| preserveInNetbox: true | ||
| tenant: MY_TENANT | ||
| customFields: | ||
| netboxOperatorRestorationHash: b0d7d37281d2f2735ff188bdfa4eda469e55b684 |
Oops, something went wrong.