integration test: API allow IPs with leading zeros

[aojea]

/kind cleanup

What this PR does / why we need it:

Test to verify that IPs with leading zeros remain valid.
IIUIC this exercises

kubernetes/pkg/apis/core/validation/validation.go

Lines 6090 to 6098 in 885961c

	// ValidateNonSpecialIP is used to validate Endpoints, EndpointSlices, and
	// external IPs. Specifically, this disallows unspecified and loopback addresses
	// are nonsensical and link-local addresses tend to be used for node-centric
	// purposes (e.g. metadata service).
	//
	// IPv6 references
	// - https://www.iana.org/assignments/iana-ipv6-special-registry/iana-ipv6-special-registry.xhtml
	// - https://www.iana.org/assignments/ipv6-multicast-addresses/ipv6-multicast-addresses.xhtml
	func ValidateNonSpecialIP(ipAddress string, fldPath *field.Path) field.ErrorList {

that is one of the most used function to validate other fields and should be enough to provide signal.

NONE

[k8s-ci-robot]

@aojea: This issue is currently awaiting triage.

If a SIG or subproject determines this is a relevant issue, they will accept it by applying the triage/accepted label and provide further guidance.

The triage/accepted label can be added by org members by writing /triage accepted in a comment.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[aojea]

/assign @liggitt @sttts

I'll need guidance because is not an area I fully understand

[aojea]

if I use the wrong parser the tests that use that data fails

diff --git a/pkg/apis/core/validation/validation.go b/pkg/apis/core/validation/validation.go
index ed7d0ec85c6..e11d43d66e9 100644
--- a/pkg/apis/core/validation/validation.go
+++ b/pkg/apis/core/validation/validation.go
@@ -6097,7 +6097,8 @@ func validateEndpointAddress(address *core.EndpointAddress, fldPath *field.Path)
 // - https://www.iana.org/assignments/ipv6-multicast-addresses/ipv6-multicast-addresses.xhtml
 func ValidateNonSpecialIP(ipAddress string, fldPath *field.Path) field.ErrorList {
        allErrs := field.ErrorList{}
-       ip := netutils.ParseIPSloppy(ipAddress)
+       ip := net.ParseIP(ipAddress)
+       //ip := netutils.ParseIPSloppy(ipAddress)
        if ip == nil {
                allErrs = append(allErrs, field.Invalid(fldPath, ipAddress, "must be a valid IP address"))
                return allErrs

[aojea]

/hold
talked with Stefan offline, will add an specific test instead of modifying current data

[aojea]

/hold talked with Stefan offline, will add an specific test instead of modifying current data

/hold cancel

[aojea]

Kubernetes e2e suite: [sig-node] Pods should run through the lifecycle of Pods and PodStatus [Conformance] expand_more

/test pull-kubernetes-conformance-kind-ga-only-parallel

what happens with this test, is the second time it fails, is conformance and the PR is totally unrelated

[sttts]

/lgtm
/approve

[k8s-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: aojea, sttts

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• test/integration/apiserver/OWNERS [aojea,sttts]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment