Handle error return if portString parsing has error

[tedyu]

What type of PR is this?
/kind bug

What this PR does / why we need it:
In componentStatusStorage#serversToValidate, the error return from strconv.Atoi is ignored.

When portString parsing has error, we should handle it.

NONE

Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.:

[k8s-ci-robot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: tedyu
To complete the pull request process, please assign deads2k
You can assign the PR to them by writing /assign @deads2k in a comment when ready.

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

Needs approval from an approver in each of these files:

• pkg/registry/OWNERS

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

[tedyu]

/test pull-kubernetes-integration

[whs-dot-hk]

Hi, @tedyu

looks like etcdUrl.Host not contains colon will have

null port and address now

[aojea]

Hi, @tedyu

looks like etcdUrl.Host not contains colon will have

null port and address now

yeah, if the etcdUrl.Hot does not contain a colon the if is not matching the err variable generated but the string conversion.
also,I guess that if there is an error parsing the port we should handle the error instead of using the default port,

[tedyu]

@aojea @whs-dot-hk
Please take another look.

[whs-dot-hk]

Yes, it looks fine now

but I am not a member of kubernetes to approve

[tedyu]

/assign @liggitt

[liggitt]

Thanks for the PR, but https://github.com/kubernetes/kubernetes/pull/82680/files#diff-e735088e40a414f069432ee1deacd83eR361 is already open sweeping all strconv use for port parsing

[liggitt]

/close

[k8s-ci-robot]

@liggitt: Closed this PR.

In response to this:

/close

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.