*: Delineate between unit and integration tests

[chrisseto]

This commit blocks the k3d package from being used unless the integration or acceptance build tags have been provided. The primary goal of which is to reduce the total iteration time for running unit tests.

Additionally, this commit makes the test name assertions of the SkipIfNot{Integration,Acceptance} helpers run regardless of build tags to ensure consistent naming across the codebase.

[chrisseto]

The slowest test when running task test:unit is ~30s in the redpanda controllers package. Massive improvement IMO.

[RafalKorepta]

I think schema should have Prometheus and Cert-Manager CRDs added as per

redpanda_controller.go:709: "level"=0 "msg"="Skipping unknown GVK" "controller"="redpanda" "controllerGroup"="cluster.redpanda.com" "controllerKind"="Redpanda" "Redpanda"={"name"="rp-b3x99m" "namespace"="testenv-9z9pt"} "namespace"="testenv-9z9pt" "name"="rp-b3x99m" "reconcileID"="8e576e8c-dfa4-42b1-bbf7-e0529f9f0cc9" "gvk"="*v1.PodMonitorList"
redpanda_controller.go:709: "level"=0 "msg"="Skipping unknown GVK" "controller"="redpanda" "controllerGroup"="cluster.redpanda.com" "controllerKind"="Redpanda" "Redpanda"={"name"="rp-b3x99m" "namespace"="testenv-9z9pt"} "namespace"="testenv-9z9pt" "name"="rp-b3x99m" "reconcileID"="8e576e8c-dfa4-42b1-bbf7-e0529f9f0cc9" "gvk"="*v1.ServiceMonitorList"

https://buildkite.com/redpanda/redpanda-operator/builds/4082#01947119-fe4a-4a19-b7e3-a7e3e0575d7b/232-1239

[RafalKorepta]

Interesting that 2 tests reported that generation ID was changed by something ...

--
  | redpanda_controller_test.go:684:
  | Error Trace:	/work/operator/internal/controller/redpanda/redpanda_controller_test.go:684
  | /work/operator/internal/controller/redpanda/redpanda_controller_test.go:1045
  | /work/operator/internal/controller/redpanda/redpanda_controller_test.go:705
  | Error:      	Not equal:
  | expected: 1
  | actual  : 2
  | Test:       	TestIntegrationRedpandaController/TestIntegrationStableUIDAndGeneration
  | Messages:   	*v1.StatefulSet "rp-tmpvwu"'s Generation changed (Something changed .Spec)

https://buildkite.com/redpanda/redpanda-operator/builds/4082#01947119-fe4a-4a19-b7e3-a7e3e0575d7b/232-826

--
  | redpanda_controller_test.go:684:
  | Error Trace:	/work/operator/internal/controller/redpanda/redpanda_controller_test.go:684
  | /work/operator/internal/controller/redpanda/redpanda_controller_test.go:1045
  | /work/operator/internal/controller/redpanda/redpanda_controller_test.go:705
  | Error:      	Not equal:
  | expected: 1
  | actual  : 2
  | Test:       	TestIntegrationRedpandaController/TestIntegrationStableUIDAndGeneration
  | Messages:   	*v1.StatefulSet "rp-b3x99m"'s Generation changed (Something changed .Spec)

https://buildkite.com/redpanda/redpanda-operator/builds/4082#01947119-fe4a-4a19-b7e3-a7e3e0575d7b/232-850

[RafalKorepta]

Another problem with the name length

===
--
  | \| FATA[0000] Failed Cluster Configuration Validation:  provided cluster name 'testintegrationclientfactorytlslisteners' does not match requirements: Cluster name must be <= 32 characters, but has 40
  | Wraps: (3) exit status 1

https://buildkite.com/redpanda/redpanda-operator/builds/4082#01947119-fe4a-4a19-b7e3-a7e3e0575d7b/232-2110

[chrisseto]

I'm going to go ahead and merge this rather than trying to fix all the integration issues as those tests we're just not being run previously and they don't currently block merged.