SIMPLE-6084 Fixed a regression in create_node method to preserve comp…

…atibility with older CML releases

virl2_client/models/lab.py

@@ -537,10 +537,8 @@ def create_node(
  for key in ("image_definition", "configuration"):
  if key not in kwargs:
  kwargs[key] = None
-
- if "compute_id" in kwargs:
- kwargs.pop("compute_id")
  kwargs["resource_pool"] = None
+ kwargs.pop("compute_id", None)
  node = self._create_node_local(node_id, **kwargs)
  return node
 

virl2_client/models/node.py

@@ -420,6 +420,11 @@ def configuration(self) -> str | None:
 
  @configuration.setter
  def configuration(self, value: str | list | dict) -> None:
+ """Set the configuration."""
+ self._set_node_property("configuration", value)
+ self._set_configuration(value)
+
+ def _set_configuration(self, value: str | list | dict) -> None:
  """
  Set the content of:
  - the main configuration file if passed a string,
@@ -430,14 +435,13 @@ def configuration(self, value: str | list | dict) -> None:
 
  :param value: The configuration data in one of three formats.
  """
- self._set_node_property("configuration", value)
  if self._configuration is None:
  self._configuration = []
  if isinstance(value, str):
- if not self._configuration:
- self._configuration.append({"name": "Main", "content": value})
- else:
+ if self._configuration:
  self._configuration[0]["content"] = value
+ else:
+ self._configuration.append({"name": "Main", "content": value})
  return
  new_configs = value if isinstance(value, list) else [value]
  new_configs_by_name = {
@@ -898,7 +902,9 @@ def update(
  node_data = node_data["data"]
 
  for key, value in node_data.items():
- if key == "configuration" and exclude_configurations:
+ if key == "configuration":
+ if not exclude_configurations:
+ self._set_configuration(value)
  continue
  if key == "operational":
  self.sync_operational(node_data)