Add authorized_users as one of the required params alongside root_pass and authorized_keys

Author: ezilber-akamai

linode_api4/groups/linode.py

@@ -165,6 +165,7 @@ def instance_create(
         root_pass: Optional[str] = None,
         kernel: Optional[str] = None,
         boot_size: Optional[int] = None,
+        authorized_users: Optional[List[str]] = None,
         **kwargs,
     ):
         """
@@ -175,7 +176,7 @@ def instance_create(
         To create an Instance from an :any:`Image`, call `instance_create` with
         a :any:`Type`, a :any:`Region`, and an :any:`Image`.  All three of
         these fields may be provided as either the ID or the appropriate object.
-        When an Image is provided, at least one of ``root_pass`` or
+        When an Image is provided, at least one of ``root_pass``, ``authorized_users``, or
         ``authorized_keys`` must also be given.  If ``root_pass`` is provided,
         the Instance and the password are returned as a tuple.
 
@@ -314,6 +315,11 @@ def instance_create(
                                 be a single key, or a path to a file containing
                                 the key.
         :type authorized_keys: list or str
+        :param authorized_users: A list of usernames whose keys should be installed
+                                 as trusted for the root user.  These user's keys
+                                 should already be set up, see :any:`ProfileGroup.ssh_keys`
+                                 for details.
+        :type authorized_users: list[str]
         :param label: The display label for the new Instance
         :type label: str
         :param group: The display group for the new Instance
@@ -366,10 +372,10 @@ def instance_create(
                                          an outdated library.
         """
 
-        if image and not root_pass and not authorized_keys:
+        if image and not root_pass and not authorized_keys and not authorized_users:
             raise ValueError(
                 "When creating an Instance from an Image, at least one of "
-                "root_pass or authorized_keys must be provided."
+                "root_pass, authorized_users, or authorized_keys must be provided."
             )
 
         params = {
@@ -378,6 +384,7 @@ def instance_create(
             "image": image,
             "root_pass": root_pass,
             "authorized_keys": load_and_validate_keys(authorized_keys),
+            "authorized_users": authorized_users,
             # These will automatically be flattened below
             "firewall_id": firewall,
             "backup_id": backup,

linode_api4/objects/linode.py

@@ -1396,7 +1396,7 @@ def disk_create(
                            if creating a disk without an image.
         :param read_only: If True, creates a read-only disk
         :param image: The Image to deploy to the disk.  If provided, at least one of
-                      root_pass or authorized_keys must also be given.
+                      root_pass, authorized_users or authorized_keys must also be given.
         :param root_pass: The password to configure for the root user when deploying an
                           image to this disk.  Not used if image is not given.  If an
                           image is given and root_pass is provided, it will be returned
@@ -1419,10 +1419,10 @@ def disk_create(
         :rtype: Disk or tuple(Disk, str)
         """
 
-        if image and not root_pass and not authorized_keys:
+        if image and not root_pass and not authorized_keys and not authorized_users:
             raise ValueError(
                 "When creating a Disk from an Image, at least one of "
-                "root_pass or authorized_keys must be provided."
+                "root_pass, authorized_users, or authorized_keys must be provided."
             )
 
         authorized_keys = load_and_validate_keys(authorized_keys)
@@ -1586,6 +1586,7 @@ def rebuild(
         disk_encryption: Optional[
             Union[InstanceDiskEncryptionType, str]
         ] = None,
+        authorized_users: Optional[List[str]] = None,
         **kwargs,
     ):
         """
@@ -1598,24 +1599,29 @@ def rebuild(
         :param image: The Image to deploy to this Instance
         :type image: str or Image
         :param root_pass: The root password for the newly rebuilt Instance.  At least
-                          one of root_pass or authorized_keys must be provided.
+                          one of root_pass, authorized_users, or authorized_keys must be provided.
         :type root_pass: str
         :param authorized_keys: The ssh public keys to install in the linode's
                                 /root/.ssh/authorized_keys file.  Each entry may
                                 be a single key, or a path to a file containing
                                 the key.
         :type authorized_keys: list or str
+        :param authorized_users: A list of usernames whose keys should be installed
+                                 as trusted for the root user.  These user's keys
+                                 should already be set up, see :any:`ProfileGroup.ssh_keys`
+                                 for details.
+        :type authorized_users: list[str]
         :param disk_encryption: The disk encryption policy for this Linode.
                                 NOTE: Disk encryption may not currently be available to all users.
         :type disk_encryption: InstanceDiskEncryptionType or str
 
         :returns: The root_pass if provided, otherwise True.
         :rtype: str or bool
         """
-        if not root_pass and not authorized_keys:
+        if not root_pass and not authorized_keys and not authorized_users:
             raise ValueError(
                 "When rebuilding an Instance, at least one of "
-                "root_pass or authorized_keys must be provided."
+                "root_pass, authorized_users, or authorized_keys must be provided."
             )
 
         authorized_keys = load_and_validate_keys(authorized_keys)
@@ -1627,6 +1633,7 @@ def rebuild(
             "disk_encryption": (
                 str(disk_encryption) if disk_encryption else None
             ),
+            "authorized_users": authorized_users,
         }
 
         params.update(kwargs)