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

Nested Services for New-NSXTServiceDefinition and Get-NSXTInfraScope Path Fix #16

Merged
merged 1 commit into from
Mar 7, 2022

Conversation

ezrahill
Copy link
Contributor

@ezrahill ezrahill commented Mar 7, 2022

Improvements to New-NSXTServiceDefinition to support nested services.

Example

PS /Users/ehill> New-NSXTServiceDefinition -Name "Ezra-MyHTTP2" -Service @("HTTP","HTTPS")
Successfully created new NSX-T Service Ezra-MyHTTP2

display_name id                                                                                                                                               ------------ --
Ezra-MyHTTP2 Ezra-MyHTTP2

PS /Users/ehill> Get-NSXTServiceDefinition -Name "Ezra-MyHTTP2"

Name        : Ezra-MyHTTP2
Id          : Ezra-MyHTTP2
Protocol    :
Source      :
Destination :
Services    : {HTTP, HTTPS}
Path        : /infra/services/Ezra-MyHTTP2

Also, fixed a bug when running Get-NSXTInfraScope, the Path property was null

Before

PS /Users/ehill> Get-NSXTInfraScope

Name                     Id                 Path
----                     --                 ----
All Uplinks              cgw-all
Direct Connect Interface cgw-direct-connect
Internet Interface       cgw-public
MGW                      mgw
VPC Interface            cgw-cross-vpc
VPN Tunnel Interface     cgw-vpn

After

PS /Users/ehill> Get-NSXTInfraScope
Name                     Id                 Path
----                     --                 ----
All Uplinks              cgw-all            /infra/labels/cgw-all
Direct Connect Interface cgw-direct-connect /infra/labels/cgw-direct-connect
Internet Interface       cgw-public         /infra/labels/cgw-public
MGW                      mgw                /infra/labels/mgw
VPC Interface            cgw-cross-vpc      /infra/labels/cgw-cross-vpc
VPN Tunnel Interface     cgw-vpn            /infra/labels/cgw-vpn

@lamw lamw merged commit 05401af into lamw:master Mar 7, 2022
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