Skip to content

[manila] add scheduler_hints to the shares CreateOpts#3211

Merged
pierreprinetti merged 1 commit intogophercloud:masterfrom
kayrus:share-scheduler-hints
Oct 16, 2024
Merged

[manila] add scheduler_hints to the shares CreateOpts#3211
pierreprinetti merged 1 commit intogophercloud:masterfrom
kayrus:share-scheduler-hints

Conversation

@kayrus
Copy link
Copy Markdown
Contributor

@kayrus kayrus commented Oct 10, 2024

Fixes #3204

@kayrus kayrus requested a review from a team October 10, 2024 16:02
@kayrus kayrus added the backport-v2 This PR will be backported to v2 label Oct 10, 2024
@github-actions github-actions bot added edit:sharedfilesystems This PR updates sharedfilesystems code semver:minor Backwards-compatible change labels Oct 10, 2024
@kayrus kayrus force-pushed the share-scheduler-hints branch from 3c0c8e2 to 2b109d7 Compare October 10, 2024 16:11
@coveralls
Copy link
Copy Markdown

Coverage Status

coverage: 78.726%. remained the same
when pulling 2b109d7 on kayrus:share-scheduler-hints
into e0d4de7 on gophercloud:master.

@kayrus
Copy link
Copy Markdown
Contributor Author

kayrus commented Oct 14, 2024

@pierreprinetti @mdbooth @EmilienM kindly ping

Copy link
Copy Markdown
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 to me.

/lgtm

AvailabilityZone string `json:"availability_zone,omitempty"`
// SchedulerHints are hints for the scheduler to select the share backend
// Only available in and beyond API version 2.65
SchedulerHints *SchedulerHints `json:"scheduler_hints,omitempty"`
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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


// SchedulerHints contains options for providing scheduler hints when creating
// a Share.
type SchedulerHints struct {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Member

@pierreprinetti pierreprinetti left a comment

Choose a reason for hiding this comment

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

Thank you! (and thank you @mdbooth for the review)

@pierreprinetti pierreprinetti merged commit b6222a7 into gophercloud:master Oct 16, 2024
@kayrus kayrus deleted the share-scheduler-hints branch October 16, 2024 13:05
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.

[manila] add scheduler_hints field to manila shares

4 participants