[Backport 2.5.3][yugabyte#7619] Platform: Properly detect python versions when activating virtualenv

Summary:
D10896 did not pass in the executables as an array. This diff fixes it so the python executable
is passed as an array and is properly looped over.

Test Plan:
bin/install_python_requirements.sh --create_package
bin/install_python_requirements.sh --use_package

Reviewers: daniel

Reviewed By: daniel

Subscribers: yugaware

Differential Revision: https://phabricator.dev.yugabyte.com/D10900

Author: WesleyW

managed/devops/bin/common.sh

@@ -78,8 +78,8 @@ get_timestamp() {
 }
 
 check_python_executables() {
-  executables="$1"
-  for py_executable in $executables; do
+  executables=("$@")
+  for py_executable in "${executables[@]}"; do
     if which "$py_executable" > /dev/null 2>&1; then
       PYTHON_EXECUTABLE="$py_executable"
       return 0
@@ -91,9 +91,9 @@ check_python_executables() {
 set_python_vars() {
   # Prioritize python3 over python2.
   if [ -z "$YB_MANAGED_DEVOPS_USE_PYTHON3" ]; then
-    if check_python_executables "$PYTHON3_EXECUTABLES"; then
+    if check_python_executables "${PYTHON3_EXECUTABLES[@]}"; then
       YB_MANAGED_DEVOPS_USE_PYTHON3="1"
-    elif check_python_executables "$PYTHON2_EXECUTABLES"; then
+    elif check_python_executables "${PYTHON2_EXECUTABLES[@]}"; then
       YB_MANAGED_DEVOPS_USE_PYTHON3="0"
     fi
 
@@ -123,7 +123,7 @@ set_python_vars() {
       if [[ "$YB_MANAGED_DEVOPS_USE_PYTHON3" == "0" ]]; then
         POSSIBLE_EXECUTABLES="$PYTHON2_EXECUTABLES"
       fi
-      check_python_executables "$POSSIBLE_EXECUTABLES"
+      check_python_executables "${POSSIBLE_EXECUTABLES[@]}"
     fi
   fi
 

managed/devops/opscli/ybops/cloud/aws/utils.py

@@ -210,7 +210,7 @@ def __init__(self, metadata, host_vpc_id, host_vpc_region):
         self._validate_cidr_overlap()
 
     def _validate_cidr_overlap(self):
-        region_networks = [ip_network(cidr.decode('utf-8')) for cidr in self.region_cidrs.values()]
+        region_networks = [ip_network(cidr) for cidr in self.region_cidrs.values()]
         all_networks = region_networks
         for i in range(len(all_networks)):
             for j in range(i + 1, len(all_networks)):