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

Feat/gaiaos-vsx #2501

Merged
merged 5 commits into from
Nov 7, 2022
Merged

Feat/gaiaos-vsx #2501

merged 5 commits into from
Nov 7, 2022

Conversation

rwxd
Copy link
Contributor

@rwxd rwxd commented Mar 21, 2022

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

This pull request adds virtual system support for checkpoint (gaia os).
If the device has vsx enabled, the configuration for all virtual systems will be pulled and added to the config.
I used the multiple context functionality of the Cisco ASA model as a template.

Solves #2400

@rwxd rwxd mentioned this pull request Apr 2, 2022
@mortzu
Copy link
Collaborator

mortzu commented May 13, 2022

Please fix the merge conflict

@rwxd
Copy link
Contributor Author

rwxd commented May 13, 2022

@mortzu the merge conflict is fixed

@mortzu mortzu merged commit f8f6e1b into ytti:master Nov 7, 2022
@beatlepaul
Copy link

Hi - I am new to Oxidized and just setting this up. How do I apply this VSX 'fix' please? Is it a case of creating a file called gaiaos-vsx.rb with the updated contents?

@rwxd
Copy link
Contributor Author

rwxd commented Mar 26, 2024

Hi - I am new to Oxidized and just setting this up. How do I apply this VSX 'fix' please? Is it a case of creating a file called gaiaos-vsx.rb with the updated contents?

It works with the normal gaiaos model. On your oxidized webui, the device is listed as gaiaos?

When the command show vsx has VSX Enabled in the output, oxidized should iterate over each VS and save the configuration.

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