Skip to content

Commit 2ddb78e

Browse files
matias-pizarroLKaemmerling
authored andcommitted
Fix servers.client._create_image error in 'labels' parameter handling (#58)
1 parent 8ddcb0a commit 2ddb78e

File tree

3 files changed

+8
-3
lines changed

3 files changed

+8
-3
lines changed

CHANGELOG.rst

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,11 @@
22
History
33
=======
44

5+
master (XXXX-XX-XX)
6+
-------------------
7+
8+
* Fix: ServersClient.create_image fails when specifying the `labels`
9+
510
1.4.1 (2019-08-19)
611
------------------
712

hcloud/servers/client.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -670,7 +670,7 @@ def create_image(self, server, description=None, type=None, labels=None):
670670
data.update({"type": type})
671671

672672
if labels is not None:
673-
data.update({"type": labels})
673+
data.update({"labels": labels})
674674

675675
response = self._client.request(url="/servers/{server_id}/actions/create_image".format(server_id=server.id),
676676
method="POST", json=data)

tests/unit/servers/test_client.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -651,8 +651,8 @@ def test_disable_rescue(self, servers_client, server, generic_action):
651651
@pytest.mark.parametrize("server", [Server(id=1), BoundServer(mock.MagicMock(), dict(id=1))])
652652
def test_create_image(self, servers_client, server, response_server_create_image):
653653
servers_client._client.request.return_value = response_server_create_image
654-
response = servers_client.create_image(server, description="my image", type="snapshot")
655-
servers_client._client.request.assert_called_with(url="/servers/1/actions/create_image", method="POST", json={"description": "my image", "type": "snapshot"})
654+
response = servers_client.create_image(server, description="my image", type="snapshot", labels={"key": "value"})
655+
servers_client._client.request.assert_called_with(url="/servers/1/actions/create_image", method="POST", json={"description": "my image", "type": "snapshot", "labels": {"key": "value"}})
656656

657657
assert response.action.id == 1
658658
assert response.action.progress == 0

0 commit comments

Comments
 (0)