-
Notifications
You must be signed in to change notification settings - Fork 4
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
SWE-2449 upgrade junos gem #4
base: fix-diff+no-rake
Are you sure you want to change the base?
Conversation
Adding support for l2_interface
vlan delete
lags for gigether interface
'netdev_group' resource support.
solving duplicate declaration, issue Juniper#27
solving rerunning lag_interface
modified addition of tagged vlans
Resolving issue Juniper#35
Resolving qfx issue
Changes part of 1.0
Change GitHub handle
fix below warning from github ``` We found a potential security vulnerability in one of your dependencies. A dependency defined in ./junos-ez-stdlib.gemspec has known security vulnerabilities and should be updated. ```
…lity fix security vulnerability
Fix diff? method for newer JunOS versions
…sions of QFX, REs are named as LOCALRE. This code patch accommodates this deviation in naming convention and returns the version number of LOCALRE.
Support "LOCALRE" as RE name
So, I did some initial testing on this change last night to see how/if it affected narwhal, and I am seeing some interesting behavior we'll probably want to try to hunt down. You'll want to update your narwhal's gemfile to point to this branch: Then, with your local (or dockerized) narwhal up and running (and the lab CSR enrolled), you can run a script like:
Actually, maybe run the script once on an unmodified narwhal, and then change your Gemfile and re-run it, so you can compare before-and-after. Regardless, when I run the script above on an unmodified narwhal, everything works a-ok. When I run it on a narwhal updated to use the new branch, I get a nasty-looking error (though the commit does seem to succeed, still?) |
Rebased our fork's master with original. Then rebased this branch with master.