metalsmith_instances module cleanups

This change includes fixes required for RPM packaging, and addresses
review comments from the commit which added this module.

Change-Id: I80e28ab3f279f4284a20d44e23edfe720ff1db02

metalsmith_ansible/ansible_plugins/modules/metalsmith_instances.py

@@ -1,4 +1,3 @@
-#!/usr/bin/python
 # Copyright 2020 Red Hat, Inc.
 # All Rights Reserved.
 #
@@ -14,9 +13,6 @@
 # License for the specific language governing permissions and limitations
 # under the License.
 
-from __future__ import absolute_import
-__metaclass__ = type
-
 from concurrent import futures
 import io
 import logging
@@ -33,18 +29,10 @@ from metalsmith import sources
 import yaml
 
 
-ANSIBLE_METADATA = {
-    'metadata_version': '1.1',
-    'status': ['preview'],
-    'supported_by': 'community'
-}
-
-
 DOCUMENTATION = '''
 ---
 module: metalsmith_instances
 short_description: Manage baremetal instances with metalsmith
-version_added: "2.9"
 author: "Steve Baker (@stevebaker)"
 description:
   - Provision and unprovision ironic baremetal instances using metalsmith,
@@ -180,7 +168,7 @@ options:
       - An integer value representing the number of seconds to wait for the
         node provision to complete.
     type: int
-    default: 3660
+    default: 3600
   concurrency:
     description:
       - Maximum number of instances to provision at once. Set to 0 to have no