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

Release 1.7.0 #549

Merged
merged 292 commits into from
May 31, 2017
Merged

Release 1.7.0 #549

merged 292 commits into from
May 31, 2017

Conversation

saichint
Copy link

This PR is for release 1.7.0

chrisvanheuveln and others added 30 commits June 3, 2016 15:22
* partial code commit, no test yet

* partial testing

* range added

*  fix range

* fix default cost

* Added comments

* other method add

* review comments

* partial review comments fixed

* partial review comments fixed

* fix auth and stub

* All review comments fixed

* bug ID added
* Fix regexp to only accept vlan ranges

* Trunk allowed vlan now handles 'vlan add' lines
* modify/delete some repetitive tests

* tested on n9-i4
* Added support for setting/getting YANG config

* revert

* skip all tests on nexus

* cleanup

* method name change

* fixed rubocop errors

* Changes based on feedback

* removed blank line
Added IOS XR support for snmp_user
* Resurrected XR doc. Fixed constant reference.

* split line that was too long

* removed whitespace

* Fixed 6.0.0 version number.

* changes based on review
* Refactor test_snmpcommunity

* Shortened class/testcase names
* Removed redundancy by combining several test sections into a single test
* Added validate_property checks

* Tested on n9-i4

* remove comment line
This PR is for router_ospf_area provider which includes nssa properties.
All tests pass on on platforms except on n8k.
On n8k, commands like "area nssa default-information-originate" and "area nssa default-information-originate route-map " are not getting configured properly after we reset nssa and due to this the relevant tests failed. Bug is filed on nxos platform and skipping these tests only on n8k.

* partial code commit, no test yet

* partial testing

* range added

*  fix range

* fix default cost

* Added comments

* other method add

* review comments

* partial review comments fixed

* partial review comments fixed

* fix auth and stub

* All review comments fixed

* bug ID added

* nssa translate added

* nssa properties added

* comments added

* route-map default is empty

* skip few tests for n8k due to bugs

* Review comments
* Added show_running_yang utility.  Allow retrieval of operational yang data.

* Added show_running_yang utility.  Allow retrieval of operational yang data.

* Updated changelog.  Fixed comment.

* Fixed rubocop error

* Removed extra space.
This PR is for changing the nssa_set method in the ospf_area NU due to the review comments on the puppet side. This simplifies the previous version.

Also changed authentication 'clear_text' to 'cleartext' to keep consistency among all the other providers.
Chris has the context about this PR and so it is better if he reviews this.

* Add feature ospf enable

* review comments
* partial code commit, no test yet

* partial testing

* range added

*  fix range

* fix default cost

* Added comments

* other method add

* review comments

* partial review comments fixed

* partial review comments fixed

* fix auth and stub

* All review comments fixed

* bug ID added

* nssa translate added

* nssa properties added

* comments added

* route-map default is empty

* skip few tests for n8k due to bugs

* Review comments

* first commit

* test code for vls

* test code for vls

* add more tests

* auth added

* cleartext added

* authentication_key_pw added

* message_digest_key added

* default for key_id

* comments add

* CHANGELOG fixed

* comment added

* review coments
saichint and others added 27 commits April 18, 2017 12:25
* Handle version difference for development builds

* remove unnecessary split()

* Update upgrade API and add 'package' API

Removing version check from upgrade API as the version property will be
deprecated.

Adding ‘package’ API which returns the nxos image installed on the
device.

* Modifying tests to handle new upgrade API

* Improving Error Handling for install command failure

* Update Documentation

* Move package api to get full path
* add version checks in yaml

* exclude clause check for new version

* fix ciscotest for regexp

* add doc

* return if val is same

* nv overlay remove for n3k

* vrf_af modify

* fix minor test case error

* fix white space issue in yaml

* add skip for default case

* add schema for os_version

* minor issue with interval fixed

* review comments
This commit enhances the existing ntp_config and ntp_server methods
and adds ntp_auth_key

ntp_config
 - authenticate
 - trusted_key

ntp_server
 - key
 - maxpoll
 - minpoll
 - source_interface
 - vrf

This commit adds ntp_auth_key

ntp_auth_key
  - key
  - algorithm
  - mode
  - password
* Add new obj for SPAN session management

* starting with the test first; span_sesion.rb is incomplete at the
moment

* Add shutdown prop

* Addl tests + getter/setters for span_session

* Use has_key method for validating interface str

* Fix syntax error in span_session

* Fix failing test in lib/span_session.rb

* Fix failing check in test_span_session.rb

* Fix hash align offences

* Remove extraneous blank line

* Various updates/changes from PR feedback

* Update CHANGELOG, fix non-functioning test

* Add key-val to create & destroy

* Remove type in create cmd_ref

* Fix incorrect cmd_ref vars, correct setter calls in test

* Fix refute for removal test

* Add template to cmd_ref, fixes, revised test

* Add missing state key to destination set

* Attempt to fix failing interface-related tests

* Fix input validation for sources + helper method

* source_interface/vlan not iterating properly
* add valid_direction? helper method to validate span source directions (in/out/both)
* first commit

* reset before set

* fix initialize

* minor bug fixes

* doc

* common methods moved to utils

* add comments

* comment clean

* review comments
@mikewiebe
Copy link
Contributor

👍

@mikewiebe mikewiebe merged commit e388a13 into master May 31, 2017
@chrisvanheuveln chrisvanheuveln deleted the release_1.7.0 branch August 13, 2019 17:41
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.

None yet