Merge pull request #323 from fabric-testbed/321.remove-redundant-node…

…-delete Remove redundant `Node.delete()`
fabric-testbed · May 8, 2024 · af03216 · af03216
2 parents be3c7b7 + 1cca8ed
commit af03216
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 5 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -10,8 +10,9 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ### Fixed
 - Add Facility Port to allow adding multiple interfaces (Issue [#289](https://github.com/fabric-testbed/fabrictestbed-extensions/issues/289))
-- validate_config errors out when config directory does not exist (Issue [#299](https://github.com/fabric-testbed/fabrictestbed-extensions/issues/299)
-- create_ssh_config adds extra indentation (Issue [#300](https://github.com/fabric-testbed/fabrictestbed-extensions/issues/300)
+- validate_config errors out when config directory does not exist (Issue [#299](https://github.com/fabric-testbed/fabrictestbed-extensions/issues/299))
+- create_ssh_config adds extra indentation (Issue [#300](https://github.com/fabric-testbed/fabrictestbed-extensions/issues/300))
+- Remove duplicate Node.delete() method (Issue [#321](https://github.com/fabric-testbed/fabrictestbed-extensions/issues/321))
 
 ## [1.6.4] - 2024-03-05
 

diff --git a/fabrictestbed_extensions/fablib/node.py b/fabrictestbed_extensions/fablib/node.py
@@ -322,9 +322,6 @@ def render_template(self, input_string, skip: List[str] = ["ssh_command"]):
 
         return output_string
 
-    def delete(self):
-        self.get_slice().get_fim_topology().remove_node(name=self.get_name())
-
     def show(
         self, fields=None, output=None, quiet=False, colors=False, pretty_names=True
     ):