Signed-off-by: Dustin Scott <sdustin@vmware.com>

Signed-off-by: Dustin Scott <sdustin@vmware.com>

This is required when the CLI input is set to
--use-endpoint-slices.

Signed-off-by: Dustin Scott <sdustin@vmware.com>

This check ensures that the version of k8s is compatible when
using the --use-endpoint-slices feature.

Signed-off-by: Dustin Scott <sdustin@vmware.com>

Signed-off-by: Dustin Scott <sdustin@vmware.com>

This refactor allows the parser to select between
'legacy' endpoints and newer endpointslices and
handles them appropriately.

Signed-off-by: Dustin Scott <sdustin@vmware.com>

Signed-off-by: Dustin Scott <sdustin@vmware.com>

Signed-off-by: Dustin Scott <sdustin@vmware.com>

This patch reduces the packages we want coverage
for to pkg/ and internal/ (where all our code lives
now minus scripts and some build tools) and turns
on unit tests during the integration test runs to
capture more coverage data.

[fix] stop status updates on 404

This will ensure the validation of kubernetes version
for using endpoint slices returns appropriately.

Signed-off-by: Dustin Scott <sdustin@vmware.com>

Signed-off-by: Dustin Scott <sdustin@vmware.com>

This moves the validation logic into its own function
for code coverage.

Signed-off-by: Dustin Scott <sdustin@vmware.com>

Signed-off-by: Dustin Scott <sdustin@vmware.com>

In another upcoming change we're switching testing
of older kubernetes releases onto GKE (from KIND)
so that they're running against conformant production
quality cluster configurations. This patch moves
the tests which must remain in kind (for now) into
their own job as a blocker for releases.

This was needed to support running against standard
GKE clusters which have a tiny default upper bound
on the number of Ingress resources that can be present.

For the same reason this patch also reduces the overall
number of Ingresses tested.

This is a step in the direction of making the integration
test suite capable of running in parallel.

Pre-check script would pass and fail in the middle of the
integration test due to missing helm and kubectl binaries.
Added checkes for both and re-organized the print statement
so that it does not run too far to the right in the
user terminal.

Signed-off-by: Dustin Scott <sdustin@vmware.com>

Signed-off-by: Dustin Scott <sdustin@vmware.com>

Rather than create an entirely separate test for endpoint
slices, piggyback on the 2 integration test for PG vs
dbless and use endpoint slices for dbless and endpoints
for PG.  This keeps the codebase simpler while still
providing code coverage for endpoints vs. endpoint slices.

Signed-off-by: Dustin Scott <sdustin@vmware.com>

Previous assumption about the version API was incorrect.
Had to create a new clientset to consume the server version
to ensure that a consumer is not trying to use endpoint
slices against an invalid version.

Signed-off-by: Dustin Scott <sdustin@vmware.com>

This fixes a missing logic assumption that did not
understand that an EndpointSlice object did not match
the exact name of a Service resource like Endpoints do.
Rather, we look at the labels of the EndpointSlice resource
to match the name to ensure that it belongs to the service.
Additionally, added some missing methods that were left out
of prior commits.

Signed-off-by: Dustin Scott <sdustin@vmware.com>

Signed-off-by: Dustin Scott <sdustin@vmware.com>

Signed-off-by: Dustin Scott <sdustin@vmware.com>

Signed-off-by: Dustin Scott <sdustin@vmware.com>

This is required when the CLI input is set to
--use-endpoint-slices.

Signed-off-by: Dustin Scott <sdustin@vmware.com>

This check ensures that the version of k8s is compatible when
using the --use-endpoint-slices feature.

Signed-off-by: Dustin Scott <sdustin@vmware.com>

Signed-off-by: Dustin Scott <sdustin@vmware.com>

This refactor allows the parser to select between
'legacy' endpoints and newer endpointslices and
handles them appropriately.

Signed-off-by: Dustin Scott <sdustin@vmware.com>

Signed-off-by: Dustin Scott <sdustin@vmware.com>

Signed-off-by: Dustin Scott <sdustin@vmware.com>

This will ensure the validation of kubernetes version
for using endpoint slices returns appropriately.

Signed-off-by: Dustin Scott <sdustin@vmware.com>

Signed-off-by: Dustin Scott <sdustin@vmware.com>

This moves the validation logic into its own function
for code coverage.

Signed-off-by: Dustin Scott <sdustin@vmware.com>

Signed-off-by: Dustin Scott <sdustin@vmware.com>

Pre-check script would pass and fail in the middle of the
integration test due to missing helm and kubectl binaries.
Added checkes for both and re-organized the print statement
so that it does not run too far to the right in the
user terminal.

Signed-off-by: Dustin Scott <sdustin@vmware.com>

Signed-off-by: Dustin Scott <sdustin@vmware.com>

Rather than create an entirely separate test for endpoint
slices, piggyback on the 2 integration test for PG vs
dbless and use endpoint slices for dbless and endpoints
for PG.  This keeps the codebase simpler while still
providing code coverage for endpoints vs. endpoint slices.

Signed-off-by: Dustin Scott <sdustin@vmware.com>

Previous assumption about the version API was incorrect.
Had to create a new clientset to consume the server version
to ensure that a consumer is not trying to use endpoint
slices against an invalid version.

Signed-off-by: Dustin Scott <sdustin@vmware.com>

This fixes a missing logic assumption that did not
understand that an EndpointSlice object did not match
the exact name of a Service resource like Endpoints do.
Rather, we look at the labels of the EndpointSlice resource
to match the name to ensure that it belongs to the service.
Additionally, added some missing methods that were left out
of prior commits.

Signed-off-by: Dustin Scott <sdustin@vmware.com>

Signed-off-by: Dustin Scott <sdustin@vmware.com>

Signed-off-by: Dustin Scott <sdustin@vmware.com>

Signed-off-by: Dustin Scott <sdustin@vmware.com>

Signed-off-by: Dustin Scott <sdustin@vmware.com>

Signed-off-by: Dustin Scott <sdustin@vmware.com>

Signed-off-by: Dustin Scott <sdustin@vmware.com>

Removed the hard-coded versioning logic and moved the
logic into a constant.  Use a float rather than nested
integer checks.

Signed-off-by: Dustin Scott <sdustin@vmware.com>

…tion

Signed-off-by: Dustin Scott <sdustin@vmware.com>

Signed-off-by: Dustin Scott <sdustin@vmware.com>

Signed-off-by: Dustin Scott <sdustin@vmware.com>

Signed-off-by: Dustin Scott <sdustin@vmware.com>

Signed-off-by: Dustin Scott <sdustin@vmware.com>