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

Add missing engine-id-suffix property to /snmp #189

Closed
derdeagle opened this issue Jul 16, 2023 · 0 comments · Fixed by #190
Closed

Add missing engine-id-suffix property to /snmp #189

derdeagle opened this issue Jul 16, 2023 · 0 comments · Fixed by #190

Comments

@derdeagle
Copy link
Contributor

SUMMARY

For the /snmp command the parameter engine-id-suffix is not accepted although it is mentioned in the fine manual. I don't know exactly when it was changed. engine-id was replaced by engine-id-suffix as it seems.

ISSUE TYPE
  • Bug Report
COMPONENT NAME

community.routeros.api_modify

ANSIBLE VERSION
ansible [core 2.15.1]
  config file = /home/username/ansible-mikrotik/ansible.cfg
  configured module search path = ['/home/username/.ansible/plugins/modules', '/usr/share/ansible/plugins/modules']
  ansible python module location = /usr/lib/python3.11/site-packages/ansible
  ansible collection location = /home/username/.ansible/collections:/usr/share/ansible/collections
  executable location = /sbin/ansible
  python version = 3.11.3 (main, Jun  5 2023, 09:32:32) [GCC 13.1.1 20230429] (/usr/bin/python)
  jinja version = 3.1.2
  libyaml = True
COLLECTION VERSION
# /home/username/.ansible/collections/ansible_collections
Collection         Version
------------------ -------
community.routeros 2.8.3  

# /usr/lib/python3.11/site-packages/ansible_collections
Collection         Version
------------------ -------
community.routeros 2.8.2  
CONFIGURATION
CONFIG_FILE() = /home/username/ansible-mikrotik/ansible.cfg
DEFAULT_HOST_LIST(/home/username/ansible-mikrotik/ansible.cfg) = ['/home/username/ansible-mikrotik/hosts']
DEFAULT_VAULT_PASSWORD_FILE(/home/username/ansible-mikrotik/ansible.cfg) = /home/username/ansible-mikrotik/bla
PAGER(env: PAGER) = less
OS / ENVIRONMENT

I don't think this is relevant.

STEPS TO REPRODUCE

Just apply changes with the for /snmp with the parameter engine-id-suffix.

EXPECTED RESULTS

The settings should be applied.

ACTUAL RESULTS
Unknown key \"engine-id-suffix\"
derdeagle added a commit to derdeagle/community.routeros that referenced this issue Jul 16, 2023
derdeagle added a commit to derdeagle/community.routeros that referenced this issue Jul 16, 2023
felixfontein added a commit that referenced this issue Jul 16, 2023
* Fixes #189: Add missing /snmp parameter "engine-id-suffix"

* Fixes #189: Add changelog fragment after getting PR ID

* Update changelogs/fragments/180-fix-engine-id-suffix-in-snmp.yml

Co-authored-by: Felix Fontein <[email protected]>

---------

Co-authored-by: Johannes Münch <[email protected]>
Co-authored-by: Felix Fontein <[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

Successfully merging a pull request may close this issue.

1 participant