Skip to content

Conversation

@glennmatthews
Copy link
Member

  • Rename vxlan_global to overlay_global per discussion.
  • As some properties are dependent on 'nv overlay evpn' and others are not, move the feature logic out of the initialize() method and into the appropriate getters and setters
  • More thorough testing

I need to validate this against a I2 image still but here it is against an I3 image:

Node under test:
  - name  - n9k-108
  - type  - N9K-C9396PX
  - image - bootflash:///nxos.7.0.3.I3.0.278.bin

TestOverlayGlobal#test_dup_host_ip_addr_detection = 5.25 s = .
TestOverlayGlobal#test_dup_host_mac_detection = 3.02 s = .
TestOverlayGlobal#test_anycast_gateway_mac = 3.99 s = .

Fabulous run in 12.253600s, 0.2448 runs/s, 1.6322 assertions/s.

3 runs, 20 assertions, 0 failures, 0 errors, 0 skips

Glenn Matthews added 3 commits February 9, 2016 16:06
- dup_host_mac_detection doesn't require 'nv overlay evpn'
- dup_host_ip_addr_detection requires 'nv overlay evpn'
  - getter returns nil if feature not enabled (instead of an error)
  - setter enables the feature if needed
- anycast_gateway_mac requires 'nv overlay evpn'
  - getter returns nil if feature not enabled (instead of an error)
  - setter enables the feature if needed
@glennmatthews
Copy link
Member Author

Node under test:
  - name  - n9k-108
  - type  - N9K-C9396PX
  - image - bootflash:///nxos.7.0.3.I2.1.bin

TestOverlayGlobal#test_anycast_gateway_mac = 9.02 s = .
TestOverlayGlobal#test_dup_host_mac_detection = 12.93 s = .
TestOverlayGlobal#test_dup_host_ip_addr_detection = 3.54 s = .

Fabulous run in 25.486725s, 0.1177 runs/s, 0.7847 assertions/s.

3 runs, 20 assertions, 0 failures, 0 errors, 0 skips

@glennmatthews
Copy link
Member Author

Test passes against both I2 and I3 on N9k.

@mikewiebe mikewiebe self-assigned this Feb 10, 2016
config_set('feature', 'fabric_forwarding')
config_set('feature', 'nv_overlay_evpn')
rescue Cisco::CliError
CiscoLogger.debug 'feature fabric-forwarding CLI was rejected'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit It's actually feature fabric forwarding, no -

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed.

@mikewiebe
Copy link
Contributor

@glennmatthews Minor nit 👍

@glennmatthews glennmatthews force-pushed the rel120/vxlan_global_fix branch from 2ec9134 to ebf67f7 Compare February 10, 2016 14:58
@glennmatthews
Copy link
Member Author

n3k I3:

Node under test:
  - name  - n3k-105
  - type  - N3K-C3048TP-1GE
  - image - bootflash:///nxos.7.0.3.I3.0.285.bin

TestOverlayGlobal#test_anycast_gateway_mac = 3.11 s = S
TestOverlayGlobal#test_dup_host_ip_addr_detection = 1.06 s = S
TestOverlayGlobal#test_dup_host_mac_detection = 2.55 s = .

Fabulous run in 6.721668s, 0.4463 runs/s, 1.6365 assertions/s.

  1) Skipped:
TestOverlayGlobal#test_anycast_gateway_mac [/home/glmatthe/cisco-network-node-utils/tests/basetest.rb:103]:
Feature 'feature', attribute 'fabric_forwarding', operation 'config_set' is unsupported on this node


  2) Skipped:
TestOverlayGlobal#test_dup_host_ip_addr_detection [/home/glmatthe/cisco-network-node-utils/tests/basetest.rb:103]:
Feature 'feature', attribute 'fabric_forwarding', operation 'config_set' is unsupported on this node

3 runs, 11 assertions, 0 failures, 0 errors, 2 skips

glennmatthews pushed a commit that referenced this pull request Feb 10, 2016
vxlan_global --> overlay_global + fixes
@glennmatthews glennmatthews merged commit 186f030 into release_1.2.0 Feb 10, 2016
@glennmatthews glennmatthews deleted the rel120/vxlan_global_fix branch February 10, 2016 17:11
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 this pull request may close these issues.

3 participants