Description of problem:
It looks like master node is not queried for nodes when '--cluster-type' option is specified.

Version-Release number of selected component (if applicable):
# Unreleased package
# rpm -q sos sos-collector
sos-4.0-2.el8.noarch
package sos-collector is not installed

How reproducible:
100%

Steps to Reproduce:
# Setup
# ssh-keygen -t rsa -f $HOME/.ssh/id_rsa -N ''
# cat $HOME/.ssh/id_rsa.pub >> $HOME/.ssh/authorized_keys
# chmod og-wx ~/.ssh/authorized_keys
# cat >/usr/share/ovirt-engine/dbscripts/engine-psql.sh <<'EOF'
#!/bin/bash
# /usr/share/ovirt-engine/dbscripts/engine-psql.sh -c "select host_name from vds_static..."

echo "$@" >> /root/engine-psql.debug
EOF
# chmod +x /usr/share/ovirt-engine/dbscripts/engine-psql.sh
# cat /root/engine-psql.debug
cat: /root/engine-psql.debug: No such file or directory

# Repoduce
# echo -e "\n"| sos-collector -vvv --case-id=666 --no-local --cluster-type ovirt --master "127.0.0.1" -c "ovirt.cluster=whatever"
. . .
[ovirt] Query command for ovirt DB set to: SELECT host_name from vds where cluster_id in (select cluster_id FROM cluster WHERE name like 'whatever' and storage_pool_id in (SELECT id FROM storage_pool WHERE name like '%'))
[sos_collector:reduce_node_list] Node list reduced to []
. . .
# cat /root/engine-psql.debug
cat: /root/engine-psql.debug: No such file or directory

Actual results:
engine-psql.sh is not called to query database for nodes. (called through _get_nodes())

Expected results:
engine-psql.sh is not called to query database for nodes. (called through _get_nodes())

This is from sos-collector-1.8-2.el8.noarch for comparison
# Repoduce
# echo -e "\n"| sos-collector -vvv --case-id=666 --no-local --cluster-type ovirt --master "127.0.0.1" -c "ovirt.cluster=whatever"
. . .
[['ovirt']] Query command for ovirt DB set to: SELECT host_name from vds where cluster_id in (select cluster_id FROM cluster WHERE name like 'whatever' and storage_pool_id in (SELECT id FROM storage_pool WHERE name like '%'))
[sweetpig-16.4a2m.lab.eng.bos.redhat.com:run_command] Running command /usr/share/ovirt-engine/dbscripts/engine-psql.sh -c 'SELECT host_name from vds where cluster_id in (select cluster_id FROM cluster WHERE name like '"'"'whatever'"'"' and storage_pool_id in (SELECT id FROM storage_pool WHERE name like '"'"'%'"'"'))'
. . .
# cat engine-psql.debug 
-c SELECT host_name from vds where cluster_id in (select cluster_id FROM cluster WHERE name like 'whatever' and storage_pool_id in (SELECT id FROM storage_pool WHERE name like '%'))


Additional info:
I managed to trace it to the following method:
    def get_nodes_from_cluster(self):
        """Collects the list of nodes from the determined cluster cluster"""
        if self.cluster_type:
            nodes = self.cluster._get_nodes()
            self.log_debug('Node list: %s' % nodes)
            return nodes
        return []


When I change following code:
        if self.opts.cluster_type:
            if self.opts.cluster_type == 'none':
                self.cluster = self.clusters['jbon']
            else:
                self.cluster = self.clusters[self.opts.cluster_type]
            self.cluster.master = self.master

to
        if self.opts.cluster_type:
            if self.opts.cluster_type == 'none':
                self.cluster = self.clusters['jbon']
            else:
                self.cluster = self.clusters[self.opts.cluster_type]
                self.cluster_type = self.opts.cluster_type
            self.cluster.master = self.master

it works as expected. I'm not suggesting that as a patch. The issue might be that the self.cluster_type is no longer set by parsing options implicitly?