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

Add Vdc.GetNsxtAppPortProfileByName and VdcGroup.GetNsxtAppPortProfileByName for convenience #460

Merged
merged 4 commits into from
Apr 6, 2022

Conversation

Didainius
Copy link
Collaborator

@Didainius Didainius commented Apr 5, 2022

This PR adds two additional methods Vdc.GetNsxtAppPortProfileByName and VdcGroup.GetNsxtAppPortProfileByName for easier NSX-T Application Port Profile lookup.

Additionally it improves tests to test out lookup using VDC and VDC Group

This will be used to improve vmware/terraform-provider-vcd#812

Signed-off-by: Dainius Serplis <[email protected]>
Signed-off-by: Dainius Serplis <[email protected]>
@Didainius Didainius changed the title Add Vdc.GetNsxtAppPortProfileByName and VdcGroup.GetNsxtAppPortProfileByName` for convenience Add Vdc.GetNsxtAppPortProfileByName and VdcGroup.GetNsxtAppPortProfileByName for convenience Apr 5, 2022
@Didainius Didainius changed the title Add Vdc.GetNsxtAppPortProfileByName and VdcGroup.GetNsxtAppPortProfileByName for convenience Add Vdc.GetNsxtAppPortProfileByName and VdcGroup.GetNsxtAppPortProfileByName for convenience Apr 5, 2022
@Didainius Didainius marked this pull request as ready for review April 5, 2022 10:09
@Didainius Didainius removed the request for review from dataclouder April 5, 2022 10:09
Signed-off-by: Dainius Serplis <[email protected]>
Copy link
Collaborator

@lvirbalas lvirbalas left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link
Contributor

@mikeletux mikeletux left a comment

Choose a reason for hiding this comment

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

LGTM! Approved!

@Didainius Didainius merged commit cdf3ebc into vmware:main Apr 6, 2022
@Didainius Didainius deleted the apportprofile- branch April 6, 2022 09:10
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.

6 participants