In OpenStack default AZ is called 'nova', thus device_owner is filled
with 'compute:nova', and that's ok for deployments within one (default)
AZ.

Filtering by device_owner however, will make function getAttachedPorts
to return zero ports for provided node. In this patch, filtering by
device_owner is simply removed.

…ion, add more logs (kubernetes#2296)

* [cinder]: avoid unnecessary memory allocation with more logs

* Fix pagination parsing

This commit makes sure that in all instances of reading Nova instance ID
from the `.Spec.ProviderID` field, we're using
`instanceIDFromProviderID()` function which is designed for that
purpose.

Matt noticed several optimizations that can be done in
`applyNodeSecurityGroupIDForLB()` function. I added a few more
refactorings. In particular:

* We don't need to lookup servers by name as we have `.spec.providerID`
  in the nodes passed to the function.
* We don't need to lookup servers at all when we have the ID, we can
  fetch ports by `deviceID`.
* This means we don't need a Nova client in the LB instance anymore.
* `slices.Contains()` from K8s utils can be used to check if SG is in
  the list.
* It's better to tag the port with SG before we actually apply it. This
  will protect us from situations when the SG is applied, but tagging
  the port failed and now when deleting the SG CPO would be unable to
  find the port and therefore delete the SG.

…nto master