Drop wait argument from image create (#622)

This removes a useless parameter that was supposed to be removed on 2.3
release.

doc/source/images.rst

@@ -19,7 +19,7 @@ methods:
 And create through the following methods, there's also a copy method on an
 image:
 
-  - `create(data, public=False, wait=True)` - Create a new image. The first
+  - `create(data, public=False)` - Create a new image. The first
     argument is the binary data of the image itself. If the image is public,
     set `public` to `True`.
   - `create_from_simplestreams(server, alias, public=False, auto_update=False, wait=False)` -

integration/test_images.py

@@ -49,7 +49,7 @@ def test_create(self):
 
         with open(path, "rb") as f:
             data = f.read()
-            image = self.client.images.create(data, wait=True)
+            image = self.client.images.create(data)
 
         self.assertEqual(fingerprint, image.fingerprint)
 

pylxd/models/image.py

@@ -116,7 +116,7 @@ def create(
 
         if wait is False:  # pragma: no cover
             warnings.warn(
-                "Image.create wait parameter ignored and will be removed in " "2.3",
+                "Image.create wait parameter ignored and will be removed",
                 DeprecationWarning,
             )
 

pylxd/models/tests/test_image.py

@@ -109,17 +109,15 @@ def test_all(self):
     def test_create(self):
         """An image is created."""
         fingerprint = hashlib.sha256(b"").hexdigest()
-        a_image = models.Image.create(self.client, b"", public=True, wait=True)
+        a_image = models.Image.create(self.client, b"", public=True)
 
         self.assertIsInstance(a_image, models.Image)
         self.assertEqual(fingerprint, a_image.fingerprint)
 
     def test_create_with_metadata(self):
         """An image with metadata is created."""
         fingerprint = hashlib.sha256(b"").hexdigest()
-        a_image = models.Image.create(
-            self.client, b"", metadata=b"", public=True, wait=True
-        )
+        a_image = models.Image.create(self.client, b"", metadata=b"", public=True)
 
         self.assertIsInstance(a_image, models.Image)
         self.assertEqual(fingerprint, a_image.fingerprint)
@@ -128,7 +126,7 @@ def test_create_with_metadata_streamed(self):
         """An image with metadata is created."""
         fingerprint = hashlib.sha256(b"").hexdigest()
         a_image = models.Image.create(
-            self.client, StringIO(""), metadata=StringIO(""), public=True, wait=True
+            self.client, StringIO(""), metadata=StringIO(""), public=True
         )
 
         self.assertIsInstance(a_image, models.Image)