diff options
author | Zuul <zuul@review.opendev.org> | 2023-04-28 17:36:37 +0000 |
---|---|---|
committer | Gerrit Code Review <review@openstack.org> | 2023-04-28 17:36:37 +0000 |
commit | 04082fe6fd4daf6de4df09716075635c686ef2ae (patch) | |
tree | ae94d22dc0a277b273a2cf3a9a6baf1640ce3e56 /test | |
parent | b61602b70c74a4fca67910cb1cc98639e6f52451 (diff) | |
parent | b8f0a0ed5cf9dc79ed29073c15c9996cb55e4efb (diff) | |
download | swift-04082fe6fd4daf6de4df09716075635c686ef2ae.tar.gz |
Merge "ring: Centralize device normalization"
Diffstat (limited to 'test')
-rw-r--r-- | test/unit/common/ring/test_ring.py | 16 |
1 files changed, 10 insertions, 6 deletions
diff --git a/test/unit/common/ring/test_ring.py b/test/unit/common/ring/test_ring.py index 0f7e58e0c..55f45862e 100644 --- a/test/unit/common/ring/test_ring.py +++ b/test/unit/common/ring/test_ring.py @@ -68,8 +68,10 @@ class TestRingData(unittest.TestCase): def test_attrs(self): r2p2d = [[0, 1, 0, 1], [0, 1, 0, 1]] - d = [{'id': 0, 'zone': 0, 'region': 0, 'ip': '10.1.1.0', 'port': 7000}, - {'id': 1, 'zone': 1, 'region': 1, 'ip': '10.1.1.1', 'port': 7000}] + d = [{'id': 0, 'zone': 0, 'region': 0, 'ip': '10.1.1.0', 'port': 7000, + 'replication_ip': '10.1.1.0', 'replication_port': 7000}, + {'id': 1, 'zone': 1, 'region': 1, 'ip': '10.1.1.1', 'port': 7000, + 'replication_ip': '10.1.1.1', 'replication_port': 7000}] s = 30 rd = ring.RingData(r2p2d, d, s) self.assertEqual(rd._replica2part2dev_id, r2p2d) @@ -88,10 +90,12 @@ class TestRingData(unittest.TestCase): pickle.dump(rd, f, protocol=p) meta_only = ring.RingData.load(ring_fname, metadata_only=True) self.assertEqual([ - {'id': 0, 'zone': 0, 'region': 1, 'ip': '10.1.1.0', - 'port': 7000}, - {'id': 1, 'zone': 1, 'region': 1, 'ip': '10.1.1.1', - 'port': 7000}, + {'id': 0, 'zone': 0, 'region': 1, + 'ip': '10.1.1.0', 'port': 7000, + 'replication_ip': '10.1.1.0', 'replication_port': 7000}, + {'id': 1, 'zone': 1, 'region': 1, + 'ip': '10.1.1.1', 'port': 7000, + 'replication_ip': '10.1.1.1', 'replication_port': 7000}, ], meta_only.devs) # Pickled rings can't load only metadata, so you get it all self.assert_ring_data_equal(rd, meta_only) |