Skip to content

Commit 9949abc

Browse files
committed
Fix traits.create call
1 parent d786564 commit 9949abc

File tree

2 files changed

+3
-4
lines changed

2 files changed

+3
-4
lines changed

python/ironic-understack/ironic_understack/tests/test_update_baremetal_port.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@
2424
def test_with_valid_data(mocker):
2525
node_uuid = uuidutils.generate_uuid()
2626
mock_traits = mocker.Mock()
27-
mock_node = mocker.Mock(traits=mock_traits)
27+
mock_node = mocker.Mock(uuid=node_uuid, traits=mock_traits)
2828
mock_task = mocker.Mock(node=mock_node)
2929
mock_port = mocker.Mock(
3030
uuid=uuidutils.generate_uuid(),
@@ -52,5 +52,5 @@ def test_with_valid_data(mocker):
5252

5353
mock_traits.get_trait_names.assert_called_once()
5454
mock_traits.destroy.assert_called_once_with("CUSTOM_NETWORK_SWITCH")
55-
mock_traits.create.assert_called_once_with("CUSTOM_STORAGE_SWITCH")
55+
mock_traits.create.assert_called_once_with(node_uuid, ["CUSTOM_STORAGE_SWITCH"])
5656
mock_node.save.assert_called_once()

python/ironic-understack/ironic_understack/update_baremetal_port.py

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -212,8 +212,7 @@ def _set_node_traits(task, vlan_groups: set[str]):
212212
except openstack.exceptions.NotFoundException:
213213
pass
214214

215-
for trait in traits_to_add:
216-
task.node.traits = task.node.traits.create(trait)
215+
task.node.traits = task.node.traits.create(None, task.node.uuid, list(traits_to_add))
217216

218217
task.node.save()
219218

0 commit comments

Comments
 (0)