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

Fix/floating devices #1800

Merged
merged 5 commits into from Nov 3, 2016
Merged

Conversation

julsemaan
Copy link
Collaborator

Description

Fix floating devices in active/active clusters
Make Juniper switch module use Net::SSH2 for floating devices

Impacts

Floating devices

Delete branch after merge

YES

Enhancements

  • Switched floating devices SSH commands to Net::SSH2 in Juniper EX2200 module

Bug Fixes

  • Fixed floating devices in active/active clusters

my $port = $ifIndex;

my $command_mac_limit = "set ethernet-switching-options secure-access-port interface $port mac-limit 16383";
my $command_disconnect_flap = "delete protocols dot1x authenticator interface $port mac-radius flap-on-disconnect";

if ($@) {
Copy link
Contributor

Choose a reason for hiding this comment

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

Remove me

my $logger = pf::log::get_logger;

my $client = pf::api::queue->new(queue => 'general');
$client->notify( '_enableMABFloating', %postdata );
Copy link
Contributor

Choose a reason for hiding this comment

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

Is the "_" on purpose ?

my $logger = pf::log::get_logger;

my $client = pf::api::queue->new(queue => 'general');
$client->notify( '_disableMABFloating', %postdata );
Copy link
Contributor

Choose a reason for hiding this comment

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

Is the "_" on purpose ?

@julsemaan
Copy link
Collaborator Author

Fixed issues

@dwlfrth dwlfrth merged commit edc6e52 into inverse-inc:devel Nov 3, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants