openstack/aodh
Code Issues Proposed changes

Switch back to built-in md5 function

Browse Source 
hashlib.md5 always supports usedforsecurity argument in Python 3.9 and
later. The method in oslo.utils is being deprecated[1].

[1] https://review.opendev.org/c/openstack/oslo.utils/+/930879

Change-Id: I8b426c2803a08c15e078b6f6ea4b462ac3a81f3b
This commit is contained in:
Takashi Kajinami 2024-09-30 20:16:27 +09:00
parent 88c2fd905a
commit 4b8a53f4c1
1 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
aodh/coordination.py
View File

@ -13,12 +13,12 @@
# License for the specific language governing permissions and limitations
# under the License.
import bisect
import hashlib
import struct
from oslo_config import cfg
from oslo_log import log
from oslo_utils import encodeutils
from oslo_utils.secretutils import md5
from oslo_utils import uuidutils
import tenacity
import tooz.coordination
@ -79,7 +79,8 @@ class HashRing(object):
@staticmethod
def _hash(key):
return struct.unpack_from(
'>I', md5(str(key).encode(), usedforsecurity=False).digest())[0]
'>I',
hashlib.md5(str(key).encode(), usedforsecurity=False).digest())[0]
def _get_position_on_ring(self, key):
hashed_key = self._hash(key)