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

Fix catalog access #537

Merged
merged 10 commits into from
Jan 9, 2023
Merged

Fix catalog access #537

merged 10 commits into from
Jan 9, 2023

Conversation

dataclouder
Copy link
Contributor

@dataclouder dataclouder commented Jan 2, 2023

  • Remove URL checks from subscribed catalog creation Issue #967
  • Add client methods to retrieve catalogs and admin catalogs (Issue #960)
  • Add method to retrieve vApp template record (Issue #960)
  • Add test for catalog retrieval with client methods
  • Add test for catalog URL errors

Giuseppe Maxia added 5 commits January 2, 2023 11:01
These methods do not require a full Organization object
* GetAdminCatalogByHref
* GetAdminCatalogById
* GetAdminCatalogByName

Signed-off-by: Giuseppe Maxia <[email protected]>
These methods do not require a full Organization object
* GetCatalogByHref
* GetCatalogById
* GetCatalogByName

Signed-off-by: Giuseppe Maxia <[email protected]>
Given a vApp template object, this method retrieves the corresponding
query record

Signed-off-by: Giuseppe Maxia <[email protected]>
Signed-off-by: Giuseppe Maxia <[email protected]>
@dataclouder dataclouder marked this pull request as ready for review January 3, 2023 11:38
Copy link
Collaborator

@adambarreiro adambarreiro left a comment

Choose a reason for hiding this comment

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

LGTM!

Giuseppe Maxia added 2 commits January 4, 2023 14:20
Signed-off-by: Giuseppe Maxia <[email protected]>
Signed-off-by: Giuseppe Maxia <[email protected]>
Copy link
Collaborator

@Didainius Didainius left a comment

Choose a reason for hiding this comment

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

Thanks. catalog (tag) tests passed on 3 VCD versions.
Left just a few tiny comments

Giuseppe Maxia added 2 commits January 4, 2023 16:00
Signed-off-by: Giuseppe Maxia <[email protected]>
Signed-off-by: Giuseppe Maxia <[email protected]>
@dataclouder dataclouder merged commit 6153ff1 into vmware:main Jan 9, 2023
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.

5 participants