doc/mgr: svc_name is modified to svc_id

[renhwztetecs]

svc_name is modified to svc_id,
refer to the definition of "static PyObject* get_counter(...)"

Signed-off-by: huanwen ren ren.huanwen@zte.com.cn

[shinobu-x]

jenkins test this please

[jcsp]

This isn't really necessary, because it's a positional argument -- what's currently in the docs just corresponds to what it's called in mgr_module.py.

If you want to update the code to make the C++ side use the same names as the python side for arguments that would probably benefit readability though.

[renhwztetecs]

I think the update code seems to be nothing necessary
I have another doubt，
as follows:
get_metadata(self, svc_type, svc_id) and get_counter(self, svc_type, svc_name, path).

I understand that "svc_id" and "svc_name" are the same, so
I change get_counter(self, svc_type, svc_name, path) to
get_counter(self, svc_type, svc_id, path) , it makes the
document more readable.

thank you

[jcsp]

I just don't think it makes sense to have something in the python module docs that is different from the definition in mgr_module.py for get_counter.

If you want to open a new PR that cleans up the name vs. id terminology across the interface, that would be welcome, but this doc edit I don't think makes sense in its current form.