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

nxos_snmp_traps: fix 'group: all' for N35 platforms (#55995) #56229

Merged
merged 1 commit into from May 21, 2019

Conversation

Projects
None yet
4 participants
@trishnaguha
Copy link
Member

commented May 8, 2019

SUMMARY
  • nxos_snmp_traps: fix 'group: all' for N35 platforms
  • group: all attempts to enable traps for all features defined in the module's feature_list

  • N35 platforms do not support snmp-server enable traps bfd; so removing bfd from the feature_list for that platform

  • Minor cleanup in sanity.yaml test file

  • whitespace lint fix

(cherry picked from commit 0e0c2a7)

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME
ADDITIONAL INFORMATION

nxos_snmp_traps: fix 'group: all' for N35 platforms (#55995)
* nxos_snmp_traps: fix 'group: all' for N35 platforms

- `group: all` attempts to enable traps for all features defined in the module's `feature_list`

- `N35` platforms do not support `snmp-server enable traps bfd`; so removing `bfd` from the `feature_list` for that platform

- Minor cleanup in `sanity.yaml` test file

* whitespace lint fix

(cherry picked from commit 0e0c2a7)
@ansibot

This comment has been minimized.

@ansibot ansibot added the stale_ci label May 16, 2019

@abadger abadger merged commit 46f185c into ansible:stable-2.8 May 21, 2019

1 check passed

Shippable Run 122187 status is SUCCESS.
Details
@abadger

This comment has been minimized.

Copy link
Member

commented May 21, 2019

Merged for the 2.8.1 release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.