ansible-collections / community.kubernetes

Kubernetes Collection for Ansible
https://galaxy.ansible.com/community/kubernetes
GNU General Public License v3.0
265 stars 106 forks source link

Two new CI failures in sanity and molecule tests #323

Closed geerlingguy closed 3 years ago

geerlingguy commented 3 years ago
SUMMARY

We are now hitting two new errors in CI causing failed builds: https://github.com/ansible-collections/community.kubernetes/actions/runs/405144301

First, in the sanity test:

ERROR: Found 3 validate-modules issue(s) which need to be resolved:
ERROR: plugins/modules/k8s.py:0:0: doc-default-does-not-match-spec: Argument 'append_hash' in argument_spec defines default as (False) but documentation defines default as (None)
ERROR: plugins/modules/k8s.py:0:0: doc-default-does-not-match-spec: Argument 'apply' in argument_spec defines default as (False) but documentation defines default as (None)
ERROR: plugins/modules/k8s_service.py:0:0: doc-default-does-not-match-spec: Argument 'apply' in argument_spec defines default as (False) but documentation defines default as (None)

Second, in the molecule test:

  Warning: : running playbook inside collection community.kubernetes
  Warning: :  * Failed to parse
  /home/runner/.cache/molecule/kubernetes/default/inventory/ansible_inventory.yml
  with ansible_collections.community.kubernetes.plugins.inventory.k8s plugin:
  Incorrect plugin name in file: none found
ISSUE TYPE
COMPONENT NAME

CI

Akasurde commented 3 years ago
  Warning: : running playbook inside collection kubernetes.core
  Warning: :  * Failed to parse
  /home/runner/.cache/molecule/core/default/inventory/ansible_inventory.yml with
  ansible_collections.kubernetes.core.plugins.inventory.k8s plugin: Incorrect
  plugin name in file: none found
  Warning: : Unable to parse
  /home/runner/.cache/molecule/core/default/inventory/ansible_inventory.yml as an
  inventory source
  Warning: : Invalid characters were found in group names but not replaced, use
  -vvvv to see details
  Warning: : Could not match supplied host pattern, ignoring:
  namespace_inventory_pods
geerlingguy commented 3 years ago

@Akasurde - All right, looks like those PRs have fixed up all the current failures, and I'm seeing passing tests now, so I am happy to close this out now :)