[CLOYSTER-165] Generate answer file based on TUI data (#91)

* Fixes an outragous number of bugs and warnings

Signed-off-by: Vinícius Ferrão <[email protected]>

Fixed more outrageous bugs.

The program is now able to at least boot a compute node again

Signed-off-by: Vinícius Ferrão <[email protected]>

Disable cosmetic warnings

Pin spdlog version

Run clang-format

Handle copy to temporary file error

Fix message in CMakeLists.txt

Handle error in answerfile minor version

Fix routing error disabling the VM network

Enable Managed network again

Fix answerfile version handling

Fix copyFile implementation

Fix typos, change variables names and fix throw.

* Fix typos in the documentation.
* Fix typos on LOG messages.
* Change variable names to be more accurate with the context.
* Rethrow should also follow the golden rule of: throw by value and catch by reference.

Signed-off-by: Vinícius Ferrão <[email protected]>

Clearning more warnings

Fix the last warnings

Remove std::exit call, and raise an exception instead

Run clang-format

Added version ranges to conanfile.txt

This change will keep the project more reliable while
still getting updates from upstream on minor versions.

Signed-off-by: Vinícius Ferrão <[email protected]>

Last try to fix clang-format

Signed-off-by: Vinícius Ferrão <[email protected]>

* Add code to dump answerfile information

* Start adding code to dump model (aka cluster) data

* Dump cluster data to answerfile

* Remove the `ini` parameter in most of answerfile functions

This parameter was unneeded, we can use the member m_ini parameter

* Reformat some files

* Dump the answerfile

* presenter: fix issues when counting available nics

* Fix some warnings regarding return statements

One is for the lack of one, the other is code after one

* Fix typing and remote URL

* Make the dump answerfile name passable by parameter

* Fix lint

---------

Signed-off-by: Vinícius Ferrão <[email protected]>
Co-authored-by: Vinícius Ferrão <[email protected]>

Author: arthurmco

include/cloysterhpc/answerfile.h

@@ -16,6 +16,29 @@
 
 using boost::asio::ip::address;
 
+/**
+ * @struct AFNode
+ * @brief Holds individual node settings.
+ *
+ * This structure contains optional settings for individual nodes.
+ */
+struct AFNode {
+    std::optional<std::string> prefix;
+    std::optional<std::string> padding;
+    std::optional<address> start_ip = address();
+    std::optional<std::string> hostname;
+    std::optional<std::string> root_password;
+    std::optional<std::string> mac_address;
+    std::optional<std::string> sockets;
+    std::optional<std::string> cores_per_socket;
+    std::optional<std::string> threads_per_core;
+    std::optional<std::string> bmc_address;
+    std::optional<std::string> bmc_username;
+    std::optional<std::string> bmc_password;
+    std::optional<std::string> bmc_serialport;
+    std::optional<std::string> bmc_serialspeed;
+};
+
 /**
  * @class AnswerFile
  * @brief Manages configuration settings for a cluster environment.
@@ -96,29 +119,6 @@ class AnswerFile {
         std::string kernel;
     };
 
-    /**
-     * @struct AFNode
-     * @brief Holds individual node settings.
-     *
-     * This structure contains optional settings for individual nodes.
-     */
-    struct AFNode {
-        std::optional<std::string> prefix;
-        std::optional<std::string> padding;
-        std::optional<address> start_ip = address();
-        std::optional<std::string> hostname;
-        std::optional<std::string> root_password;
-        std::optional<std::string> mac_address;
-        std::optional<std::string> sockets;
-        std::optional<std::string> cores_per_socket;
-        std::optional<std::string> threads_per_core;
-        std::optional<std::string> bmc_address;
-        std::optional<std::string> bmc_username;
-        std::optional<std::string> bmc_password;
-        std::optional<std::string> bmc_serialport;
-        std::optional<std::string> bmc_serialspeed;
-    };
-
     /**
      * @struct AFNodes
      * @brief Holds settings for multiple nodes.
@@ -156,6 +156,12 @@ class AnswerFile {
     std::filesystem::path m_path;
     inifile m_ini;
 
+    /**
+     * Do the inverse of `loadOptions`, i.e, move the stored settings
+     * into the answerfile.
+     */
+    void dumpOptions();
+
     /**
      * @brief Loads the configuration options from the answer file.
      *
@@ -188,13 +194,17 @@ class AnswerFile {
      */
     void loadExternalNetwork();
 
+    void dumpExternalNetwork();
+
     /**
      * @brief Loads the management network configuration.
      *
      * This function parses and loads the settings for the management network.
      */
     void loadManagementNetwork();
 
+    void dumpManagementNetwork();
+
     /**
      * @brief Loads the service network configuration.
      *
@@ -209,34 +219,44 @@ class AnswerFile {
      */
     void loadApplicationNetwork();
 
+    void dumpApplicationNetwork();
+
     /**
      * @brief Loads the general information settings.
      *
      * This function parses and loads the cluster information settings.
      */
     void loadInformation();
 
+    void dumpInformation();
+
     /**
      * @brief Loads the time-related settings.
      *
      * This function parses and loads the time settings.
      */
     void loadTimeSettings();
 
+    void dumpTimeSettings();
+
     /**
      * @brief Loads the hostname settings.
      *
      * This function parses and loads the hostname settings.
      */
     void loadHostnameSettings();
 
+    void dumpHostnameSettings();
+
     /**
      * @brief Loads the system settings.
      *
      * This function parses and loads the system configuration settings.
      */
     void loadSystemSettings();
 
+    void dumpSystemSettings();
+
     /**
      * @brief Loads the node settings.
      *
@@ -246,7 +266,10 @@ class AnswerFile {
     void loadTools();
     void loadNVHPC();
 
+    void dumpNodes();
+
     void loadPostfix();
+    void dumpPostfix();
 
     bool checkEnabled(const std::string& section);
     /**
@@ -300,10 +323,12 @@ class AnswerFile {
      * @param network The network configuration to load.
      * @param optionalNameservers Indicates if the nameservers are optional.
      */
-    template <typename NetworkType>
-    void loadNetwork(const std::string& networkSection, NetworkType& network,
+    void loadNetwork(const std::string& networkSection, AFNetwork& network,
         bool optionalNameservers = true);
 
+    void dumpNetwork(
+        const AFNetwork& network, const std::string& networkSection);
+
 public:
     AFNetwork external;
     AFNetwork management;
@@ -322,6 +347,7 @@ class AnswerFile {
      * @param path The path to the answer file.
      */
     void loadFile(const std::filesystem::path& path);
+    void dumpFile(const std::filesystem::path& path);
     std::vector<std::shared_ptr<ITool>> getTools();
 
     AnswerFile();

include/cloysterhpc/cluster.h

@@ -236,7 +236,9 @@ class Cluster {
      *
      * @param answerfilePath Path to the answer file.
      */
-    void fillData(const std::string& answerfilePath);
+    void fillData(const std::filesystem::path& answerfilePath);
+
+    void dumpData(const std::filesystem::path& answerfilePath);
 
 #ifndef NDEBUG
     void printNetworks(

include/cloysterhpc/presenter/PresenterNetwork.h

@@ -42,6 +42,8 @@ class NetworkCreator {
     bool checkIfInterfaceRegistered(std::string_view interface);
 
     void saveNetworksToModel(Cluster& model);
+
+    std::size_t getSelectedInterfaces();
 };
 
 class PresenterNetwork : public Presenter {