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 property upload_any_file to vcd_catalog_media #1123

Closed
wants to merge 2 commits into from

Conversation

dataclouder
Copy link
Contributor

@dataclouder dataclouder commented Oct 1, 2023

Setting this property to true we can upload any file. The default, without using this property, only allows .ISO files

Replaced by PR #1124

Giuseppe Maxia added 2 commits October 1, 2023 21:35
Setting this property to true we can upload any file. The default,
without using this property, only allows .ISO files

Signed-off-by: Giuseppe Maxia <[email protected]>
Signed-off-by: Giuseppe Maxia <[email protected]>
@@ -65,3 +65,7 @@ require (
google.golang.org/grpc v1.56.0 // indirect
google.golang.org/protobuf v1.30.0 // indirect
)

replace github.com/vmware/go-vcloud-director/v2 => github.com/dataclouder/go-vcloud-director/v2 v2.17.0-alpha.3.0.20231001193006-c092d978c551
Copy link
Collaborator

Choose a reason for hiding this comment

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

Please remove this module replacement before you merge this PR,
once you completely finish your code changes and the PR is ready to be merged.
You don't need to resolve this conversation, it will be automatically removed once
go.mod is in a correct state.
Thanks! 😸

@dataclouder dataclouder changed the title Add property upload_any_file to vcd_catalog_media Add property upload_any_file to vcd_catalog_media Oct 1, 2023
@dataclouder dataclouder marked this pull request as ready for review October 1, 2023 20:02
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.

Test passed on 10.5.0 for me, thanks!

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.

4 participants