Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Aireos is not supported with network_cli as we move to collections in 2.10 #119

Closed
tom0010 opened this issue Sep 24, 2020 · 1 comment
Closed

Comments

@tom0010
Copy link

tom0010 commented Sep 24, 2020

SUMMARY

Note: I have been doing this on 2.9.11, but actively trying to migrate to collections ready for 2.10.

As we go to 2.10, the new aireos_config is now in the community.network collection, as per the documentation: https://docs.ansible.com/ansible/latest/collections/community/network/aireos_config_module.html

This leads us to: https://github.com/ansible-collections/community.network#supported-connections
Which states that only network_cli or httpapi can be used.

Inside of https://docs.ansible.com/ansible/latest/network/user_guide/platform_index.html there is not aireos, so I think this has been missed when migrating to this new collection and out of Ansible base.

ISSUE TYPE
  • Bug Report
COMPONENT NAME

community.network
community.network.aireos_config

ANSIBLE VERSION
2.9.11
CONFIGURATION
COLLECTIONS_PATHS(/Users/user/scripts/testing/ansible.cfg) = ['/Users/user/scripts/testing']
DEFAULT_NO_LOG(/Users/user/scripts/testing/ansible.cfg) = False
DEFAULT_STRATEGY(/Users/user/scripts/testing/ansible.cfg) = free
DISPLAY_SKIPPED_HOSTS(/Users/user/scripts/testing/ansible.cfg) = True
GALAXY_IGNORE_CERTS(/Users/user/scripts/testing/ansible.cfg) = True
HOST_KEY_CHECKING(/Users/user/scripts/testing/ansible.cfg) = False
PARAMIKO_HOST_KEY_AUTO_ADD(/Users/user/scripts/testing/ansible.cfg) = True
PARAMIKO_LOOK_FOR_KEYS(/Users/user/scripts/testing/ansible.cfg) = False
RETRY_FILES_ENABLED(/Users/user/scripts/testing/ansible.cfg) = False

OS / ENVIRONMENT

Ansible AWX 14.1 - Centos 8
macOS 10.15.6

STEPS TO REPRODUCE

site.yaml:

  gather_facts: false
  hosts: "wlc"
  collections:
    - "community.network"
  tasks:
    - name: "Import Wireless LAN Controller tasks"
      import_tasks: "roles/wlc/tasks/main.yml"

main.yml:

- name: "TASK 5"
  block:
  - name: "TASK 5 -"
    community.network.aireos_config:
      lines:
        - "mgmtuser password localadmin {{ new_password }}"
        - "ap mgmtuser add username localadmin password {{ new_password }} secret {{ new_password }} all"
      save_when: "changed"
    vars:
      ansible_network_os: "aireos"  
EXPECTED RESULTS

The playbook to be executed correctly.

ACTUAL RESULTS
"msg": "invalid connection specified, expected connection=local, got ansible.netcommon.network_cli"
ganeshrn added a commit to ganeshrn/community.network that referenced this issue Sep 25, 2020
Fixes:
ansible-collections#119

*  For platforms using persistent allow network_cli connection type
   In ansible 2.6 `network_cli` connection was added as first class
   connection, however some of the community platform code was not
   updated to use `network_cli`.
*  Deprecate local connection type for platforms that use persistent
   connection framework and have cliconf and terminal plugins
   implemented.
@ganeshrn
Copy link
Member

ganeshrn commented Oct 1, 2020

@t1hom7as Can you test the
linked PR to confirm if it fixes the issue for you.

patchback bot pushed a commit that referenced this issue Oct 15, 2020
* Allow network_cli type for platforms using persistent connection

Fixes:
#119

*  For platforms using persistent allow network_cli connection type
   In ansible 2.6 `network_cli` connection was added as first class
   connection, however some of the community platform code was not
   updated to use `network_cli`.
*  Deprecate local connection type for platforms that use persistent
   connection framework and have cliconf and terminal plugins
   implemented.

* add changelog

* Fix review comments

* minor updates

* remove unwanted space

* FIx more santiy issues

* fix review comments

(cherry picked from commit b5629d7)
felixfontein pushed a commit that referenced this issue Oct 15, 2020
… (#133)

* Allow network_cli type for platforms using persistent connection

Fixes:
#119

*  For platforms using persistent allow network_cli connection type
   In ansible 2.6 `network_cli` connection was added as first class
   connection, however some of the community platform code was not
   updated to use `network_cli`.
*  Deprecate local connection type for platforms that use persistent
   connection framework and have cliconf and terminal plugins
   implemented.

* add changelog

* Fix review comments

* minor updates

* remove unwanted space

* FIx more santiy issues

* fix review comments

(cherry picked from commit b5629d7)

Co-authored-by: Ganesh Nalawade <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants