Don't skip an inventory source just because it has a comma (#35002)

* Don't skip an inventory source just because it has a comma, make sure it's also doesn't exist as a path. Fixes #34931

* Add integration test for inventory path with commas

(cherry picked from commit 1ac04ba52e)

lib/ansible/vars/manager.py

@@ -285,7 +285,7 @@ class VariableManager:
                 ''' merges all entities by inventory source '''
                 data = {}
                 for inventory_dir in self._inventory._sources:
-                    if ',' in inventory_dir:  # skip host lists
+                    if ',' in inventory_dir and not os.path.exists(inventory_dir):  # skip host lists
                         continue
                     elif not os.path.isdir(inventory_dir):  # always pass 'inventory directory'
                         inventory_dir = os.path.dirname(inventory_dir)

test/integration/targets/inventory_path_with_comma/aliases

@@ -0,0 +1 @@
+posix/ci/group3

test/integration/targets/inventory_path_with_comma/playbook.yml

@@ -0,0 +1,9 @@
+---
+- hosts: all
+  gather_facts: false
+  tasks:
+    - name: Ensure we can see group_vars from path with comma
+      assert:
+        that:
+          - inventory_var_from_path_with_commas is defined
+          - inventory_var_from_path_with_commas == 'here'

test/integration/targets/inventory_path_with_comma/runme.sh

@@ -0,0 +1,5 @@
+#!/usr/bin/env bash
+
+set -ux
+
+ansible-playbook -i this,path,has,commas/hosts playbook.yml -v "$@"

test/integration/targets/inventory_path_with_comma/this,path,has,commas/group_vars/all.yml

@@ -0,0 +1 @@
+inventory_var_from_path_with_commas: 'here'

test/integration/targets/inventory_path_with_comma/this,path,has,commas/hosts

@@ -0,0 +1 @@
+localhost ansible_connect=local