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: add new MVE image types #73

Merged
merged 1 commit into from
Sep 18, 2024
Merged

Conversation

ccakes
Copy link
Contributor

@ccakes ccakes commented Sep 16, 2024

Add config types for 6WIND VSR, Aviatrix and Palo Prisma

Description

This adds support for new MVE images - Aviatrix Secure Edge, 6WIND VSR and Prisma. Two of which aren't GA yet but will be shortly.

Fixes EIP-3669

Type of change

  • New feature (non-breaking change which adds functionality)

Contributor Agreement

Lodging this Pull Request (PR) indicates agreement with the project's
(Contributor License Agreement)[https://github.com/megaport/megaportgo/wiki/Megaport_Contributor_Licence_Agreement.md].

Please read the Contributor Licence Agreement (CLA) and affirm your acceptance here:

[I have read an accept the CLA]

NOTE If multiple authors have commited to this PR, each one will need to comment on this PR and
agree to the CLA before this PR can be accepted.

Copy link
Contributor

@MegaportPhilipBrowne MegaportPhilipBrowne left a comment

Choose a reason for hiding this comment

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

LGTM

@mega-alex
Copy link
Contributor

LGTM, but this repo requires signed commits.

@ccakes ccakes force-pushed the feat/mve-aviatrix-6wind-prisma branch from 780cf38 to ad367aa Compare September 17, 2024 23:42
Add config types for 6WIND VSR, Aviatrix and Palo Prisma
@ccakes ccakes force-pushed the feat/mve-aviatrix-6wind-prisma branch from ad367aa to a1b3c4b Compare September 18, 2024 00:28
@ccakes ccakes merged commit acfbbf2 into main Sep 18, 2024
2 checks passed
@ccakes ccakes deleted the feat/mve-aviatrix-6wind-prisma branch September 18, 2024 00:34
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