Skip to content

Commit

Permalink
Remove cluster Python venv stuff (not appropriate in cluster layer)
Browse files Browse the repository at this point in the history
  • Loading branch information
erl-hpe committed Feb 7, 2025
1 parent dfbd22c commit 9d63b43
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 31 deletions.
11 changes: 0 additions & 11 deletions vtds_cluster_kvm/private/cluster.py
Original file line number Diff line number Diff line change
Expand Up @@ -507,14 +507,3 @@ def get_virtual_nodes(self):

def get_virtual_networks(self):
return VirtualNetworks(self.common)

def get_node_venv_path(self):
python_config = self.config.get('python', {})
return python_config.get('node_venv_path', "/root/node-venv")

def get_node_python_executable(self):
# NOTE: do not use path_join() here to construct the path. The
# path here is being constructed for a Linux environment,
# where path separators are always '/' and which might not
# match the system this code is running on.
return "%s/bin/python3" % self.get_node_venv_path()
20 changes: 0 additions & 20 deletions vtds_cluster_kvm/private/config/config.yaml
Original file line number Diff line number Diff line change
@@ -1,24 +1,4 @@
cluster:
# The python information here pertains to a shared Python Virtual
# Environment created by the Cluster layer for use of it and any
# higher layers (application layers) that might need it. Specifically,
# this allows the path to the virtual environment to be specified and
# it allows for the specification of Python modules to be
# pre-installed by the Cluster layer in that virtual environment for
# use by other layers.
python:
node_venv_path: /root/blade-venv
modules:
modules:
example:
# Install
module_name: pyyaml
delete: true # this is an example, so don't really install it
source_type: pypi
metadata:
url: null
version: pyyaml~=6.0

# The 'host_blade_network' specifies the parameters for a
# blade-local network used only among Virtual Nodes that share a
# single blade and the blade itself. The addresses here are
Expand Down

0 comments on commit 9d63b43

Please sign in to comment.