fix: Fix non-boolean conditions

[martinpitt]

Ansible 2.19 demands conditions (when: or assert:) to have a boolean type. Fix list-value conditions with an explicit length check.

Also fix the pcp_accounts default value in roles/pcp/tasks/main.yml -- that variable is a list (of objects), not an object.

---

This fixes the Ansible 2.19 specific errors like

[ERROR]: Task failed: Action failed: {'failed': True, 'exception': '(traceback unavailable)', 'msg': 'Task failed: Conditional result was "['cyrus-sasl-lib', 'cyrus-sasl-scram']" of type 'list', which evaluates to True. Conditionals must have a boolean result.', 'changed': False, '_ansible_no_log': False}

I tested them against https://github.com/linux-system-roles/metrics, I'll post a draft PR soon. Note that this still leaves one Fedora 42 failure in tests_verify_from_spark.yml which is unrelated, it's an SELinux rejection.

@richm FYI

[martinpitt]

Running system-role test with this change here: linux-system-roles/metrics#242

[martinpitt]

The F42 testing farm run passed, and the GitHub workflow run failed tests_verify_from_spark.yml as expected.

[richm]

we really need to fix the ansible-lint action to exclude other collections . . .