oVirt / vdsm

The Virtual Desktop Server Manager
GNU General Public License v2.0
161 stars 202 forks source link

resourceManager: use f-strings in exceptions #358

Closed aesteve-rh closed 1 year ago

aesteve-rh commented 1 year ago

Fix various exceptions in the resourceManager module, so that it is constructed wiht f-string instead of %-format. Fix ResourceAlreadyAcquired exception that is using a comma to separate instead of %, resulting in bogus printed message.

Before:

vdsm.storage.resourceManager.ResourceAlreadyAcquired: ('%s is already
acquired by %s', '00_storage.a2ad7a23-96c5-4c23-9007-c0d1892ea07d',
'fb7c6873-74b4-4895-876a-a52ab7bfa082')

After:

vdsm.storage.resourceManager.ResourceAlreadyAcquired:
'00_storage.a2ad7a23-96c5-4c23-9007-c0d1892ea07d is already
acquired by fb7c6873-74b4-4895-876a-a52ab7bfa082'

Signed-off-by: Albert Esteve aesteve@redhat.com

mz-pdm commented 1 year ago

/ost

aesteve-rh commented 1 year ago

/ost

aesteve-rh commented 1 year ago

/ost