From 39be15b150352ab2557a3b960caa57bad6502573 Mon Sep 17 00:00:00 2001
From: Thorsten Sick <thorsten.sick@avast.com>
Date: Thu, 10 Jun 2021 08:42:27 +0200
Subject: [PATCH] Extended code documentation

---
 experiment_control.py     | 6 +++---
 plugins/base/machinery.py | 6 +++---
 2 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/experiment_control.py b/experiment_control.py
index f4779a4..51c27ca 100644
--- a/experiment_control.py
+++ b/experiment_control.py
@@ -46,9 +46,9 @@ def create_parser():
     # Sub parser for machine creation
     parser_run = subparsers.add_parser("run", help="run experiments")
     parser_run.set_defaults(func=run)
-    parser_run.add_argument("--configfile", default="experiment.yaml", help="Config file to create from")
-    parser_run.add_argument("--caldera_attack", default=None, help="The id of a specific caldera attack to run")
-    parser_run.add_argument("--caldera_attack_file", default=None, help="The file name containing a list of caldera attacks to run")
+    parser_run.add_argument("--configfile", default="experiment.yaml", help="Config file to create the experiment from")
+    parser_run.add_argument("--caldera_attack", default=None, help="The id of a specific caldera attack to run, will override experiment configuration for attacks")
+    parser_run.add_argument("--caldera_attack_file", default=None, help="The file name containing a list of caldera attacks to run, will override experiment configuration for attacks")
 
     return parser
 
diff --git a/plugins/base/machinery.py b/plugins/base/machinery.py
index 060ab25..5252e84 100644
--- a/plugins/base/machinery.py
+++ b/plugins/base/machinery.py
@@ -105,12 +105,12 @@ class MachineryPlugin(BasePlugin):
         raise NotImplementedError
 
     def get_playground(self):
-        """ path where all the attack tools will be copied to on a client. Your specific machine plugin can overwrite it. """
+        """ Path on the machine  where all the attack tools will be copied to. """
 
         return self.config.get_playground()
 
     def get_vm_name(self):
-        """ Get the name of the machine """
+        """ Get the specific name of the machine """
 
         return self.config.vmname()
 
@@ -122,7 +122,7 @@ class MachineryPlugin(BasePlugin):
         raise NotImplementedError
 
     def get_machine_path_external(self):
-        """  The path external to the vm where specific data is stored """
+        """  The path on the controlling host where vm specific data is stored """
         return os.path.join(self.config.vagrantfilepath(), self.config.machinepath())
 
     ###############