Skip to content

new file: lib/oxidized/model/aerohive.rb#3671

Open
daniel-franca wants to merge 1 commit intoytti:masterfrom
daniel-franca:switch-aerohive
Open

new file: lib/oxidized/model/aerohive.rb#3671
daniel-franca wants to merge 1 commit intoytti:masterfrom
daniel-franca:switch-aerohive

Conversation

@daniel-franca
Copy link

Pre-Request Checklist

  • Passes rubocop code analysis (try rubocop --auto-correct)
  • Tests added or adapted (try rake test)
  • Changes are reflected in the documentation
  • User-visible changes appended to CHANGELOG.md

Description

@daniel-franca
Copy link
Author

Tested on production site

Copy link
Collaborator

@robertcheramy robertcheramy left a comment

Choose a reason for hiding this comment

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

Pleas also document the change in CHANGELOG.md and the new model in Supported-OS-Types.md.

@@ -0,0 +1,20 @@
class Aerohive < Oxidized::Model
prompt /(\([\w.@()-]+\)\s[#>]\s?)$/
Copy link
Collaborator

Choose a reason for hiding this comment

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

Use a prompt beginning at ^ if possible

class Aerohive < Oxidized::Model
prompt /(\([\w.@()-]+\)\s[#>]\s?)$/

cmd 'enable'
Copy link
Collaborator

Choose a reason for hiding this comment

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


cmd 'enable'

cmd 'terminal length 0'
Copy link
Collaborator

Choose a reason for hiding this comment

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


cmd 'show running-config' do |cfg|
# Remove o próprio comando da saída
cfg.gsub! /^show running-config/, ''
Copy link
Collaborator

Choose a reason for hiding this comment

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

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.

2 participants