I tried each tell_* functions in wrapper class, I found there were redundant character

Hi <a class="user-mention notranslate" data-hovercard-type="user" data-hovercard-url="

redundant character '/' before tell GET calls about python-cephclient HOT 3 CLOSED

dmsimard commented on July 3, 2024

redundant character '/' before tell GET calls

from python-cephclient.

Comments (3)

dmsimard commented on July 3, 2024

Hi @firetaker, can you give me an explicit example of where there are extra characters ?

Thanks.

from python-cephclient.

firetaker commented on July 3, 2024

in https://github.com/dmsimard/python-cephclient/blob/master/cephclient/wrapper.py
at line from 551~566:
# tell GET calls
###
def tell_debug_dump_missing(self, id, filename, *_kwargs):
return self.get('/tell/{0}/debug_dump_missing?filename={1}'.format(
id, filename), *_kwargs)

def tell_dump_pg_recovery_stats(self, id, **kwargs):
    return self.get('/tell/{0}/dump_pg_recovery_stats'.format(id), **kwargs)

def tell_list_missing(self, id, offset, **kwargs):
    return self.get('/tell/{0}/list_missing?offset={1}'.format(
        id, offset), **kwargs)

def tell_query(self, id, **kwargs):
    return self.get('/tell/{0}/query'.format(id), **kwargs)

def tell_version(self, id, **kwargs):
    return self.get('/tell/{0}/version'.format(id), **kwargs)

from python-cephclient.

dmsimard commented on July 3, 2024

You're right ! I wasn't looking at the right place.
This is fixed and pushed to master in 107f42a.

Thanks !

from python-cephclient.

Recommend Projects