Enhancement: Check if a condition is met before the polling interval has elapsed.

[kishen-v]

What happened:

An instant first tick is preferred over long running operations, which may have a particular condition satisfied already.

pvsadm/pkg/utils/utils.go

Lines 59 to 71 in d2d3bd1

	func PollUntil(pollInterval, timeOut <-chan time.Time, condition func() (bool, error)) error {
	for {
	select {
	case <-timeOut:
	return fmt.Errorf("timed out while waiting for job to complete")
	case <-pollInterval:
	if done, err := condition(); err != nil {
	return err
	} else if done {
	return nil
	}
	}
	}

What you expected to happen:
The condition to be checked once when the function is invoked, to cut down on waiting for a condition that may already be achieved.

How to reproduce it (as minimally and precisely as possible):

Anything else we need to know?:
https://stackoverflow.com/questions/32705582/how-to-get-time-tick-to-tick-immediately

Environment:

• pvsadm version (use pvsadm --version):
• Environment - On-Prem or IBM PowerVS:
• OS (e.g: cat /etc/os-release):
• Kernel (e.g. uname -a):
• Others:

[kishen-v]

/good-first-issue

[ppc64le-cloud-bot]

@kishen-v:
This request has been marked as suitable for new contributors.

Please ensure the request meets the requirements listed here.

If this request no longer meets these requirements, the label can be removed
by commenting with the /remove-good-first-issue command.

In response to this:

/good-first-issue

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-sigs/prow repository.

[aman4433]

/assign

[kishen-v]

/remove-kind bug
/kind feature