Merge pull request #8 from wznoinsk/RHOSINFRA-2560

add support for 'disconnect' from Jenkins master cli command - lint fix

.gitignore

@@ -6,3 +6,5 @@
 /dist
 /ChangeLog
 /.eggs
+/.idea/
+.idea/

AUTHORS

@@ -2,3 +2,4 @@ Michal <[email protected]>
 Michal Pryc <[email protected]>
 Sorin Sbarnea <[email protected]>
 Tristan Cacqueray <[email protected]>
+Waldemar Znoinski <[email protected]>

ChangeLog

@@ -1,6 +1,12 @@
 CHANGES
 =======
 
+* add support for 'disconnect' from Jenkins master cli command
+
+0.0.1.dev49
+-----------
+
+* Add IP address to the list column
 * Add dump config options
 * Add directory options to update multiply slaves at once
 * Ensure force reservation is only for not reserved nodes

devnest/lib/cli.py

@@ -176,9 +176,9 @@ def get_base_parser(self):
                                 help=argparse.SUPPRESS)
 
         disconnect_parser = subparsers.add_parser('disconnect',
-                                            parents=[node_parser, nest_parser],
-                                            formatter_class=formatter,
-                                            help='disconnect node(s) from Jenkins master')
+                                                  parents=[node_parser, nest_parser],
+                                                  formatter_class=formatter,
+                                                  help='disconnect node(s) from Jenkins master')
         disconnect_parser.set_defaults(action=Action.DISCONNECT)
 
         list_parser = subparsers.add_parser('list',
@@ -468,7 +468,7 @@ def main(self, argv):
             if reserve_node.get_node_status() == NodeStatus.JOB_RUNNING and \
                not parser_args.force:
                 err_msg = "A CI job is is currently running on %s node. Use --force flag " \
-                          "to reserve the node.\n\tIt will be reserved and available once the currently \ " \
+                          "to reserve the node.\n\tIt will be reserved and available once the currently " \
                           "running CI job finishes\n\tUse:\n\t" \
                           "    $ devnest list -g %s %s\n\tto check if " \
                           "CI job has finished and you can use it, if so the node "\

devnest/lib/node.py

@@ -777,7 +777,7 @@ def disconnect(self, msg='disconnected_by_devnest'):
 
         LOG.info('Disconnecting %s from Jenkins master' % self.node_name)
 
-        self.jenkins.requester.post_and_confirm_status("%s/doDisconnect?offlineMessage=%s" % (node_url, msg), data={} )
+        self.jenkins.requester.post_and_confirm_status("%s/doDisconnect?offlineMessage=%s" % (node_url, msg), data={})
 
     def _update_node_with_node_details(self, node_details):
         """Update node with NodeDetails data.