Skip to content

Conversation

kayrus
Copy link
Contributor

@kayrus kayrus commented Oct 9, 2024

Partially fixes the #3204

@kayrus kayrus requested a review from a team October 9, 2024 11:53
EmilienM
EmilienM previously approved these changes Oct 9, 2024
@github-actions github-actions bot added edit:sharedfilesystems This PR updates sharedfilesystems code semver:minor Backwards-compatible change labels Oct 9, 2024
@coveralls
Copy link

coveralls commented Oct 9, 2024

Coverage Status

coverage: 78.726%. remained the same
when pulling f8007ee on kayrus:share-group-id
into e0d4de7 on gophercloud:master.

@kayrus kayrus requested a review from EmilienM October 9, 2024 18:39
@kayrus kayrus added the backport-v2 This PR will be backported to v2 label Oct 9, 2024
@kayrus
Copy link
Contributor Author

kayrus commented Oct 10, 2024

@EmilienM can you approve this?

@kayrus kayrus requested a review from a team October 10, 2024 15:03
@kayrus
Copy link
Contributor Author

kayrus commented Oct 14, 2024

@pierreprinetti @mdbooth @EmilienM kindly ping

Copy link
Contributor

@mdbooth mdbooth left a comment

Choose a reason for hiding this comment

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

Marshalling looks correct. Does this need tests?

/lgtm

@@ -40,6 +40,8 @@ type CreateOpts struct {
SnapshotID string `json:"snapshot_id,omitempty"`
// Determines whether or not the share is public
IsPublic *bool `json:"is_public,omitempty"`
// The UUID of the share group. Available starting from the microversion 2.31
ShareGroupID string `json:"share_group_id,omitempty"`
Copy link
Contributor

Choose a reason for hiding this comment

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

@@ -54,6 +54,8 @@ type Share struct {
ShareType string `json:"share_type"`
// The name of the share type.
ShareTypeName string `json:"share_type_name"`
// The UUID of the share group. Available starting from the microversion 2.31
ShareGroupID string `json:"share_group_id"`
Copy link
Contributor

Choose a reason for hiding this comment

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

Copy link
Contributor

@EmilienM EmilienM left a comment

Choose a reason for hiding this comment

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

can't we have tests?

@kayrus
Copy link
Contributor Author

kayrus commented Oct 16, 2024

can't we have tests?

in order to have tests we need to implement a sharegroups package. we don't have it yet. so far this is more useful if you create a share group once using openstack CLI.

@kayrus kayrus merged commit c332e3e into gophercloud:master Oct 16, 2024
20 checks passed
@kayrus kayrus deleted the share-group-id branch October 16, 2024 13:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-v2 This PR will be backported to v2 edit:sharedfilesystems This PR updates sharedfilesystems code semver:minor Backwards-compatible change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants