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.

Sign up for GitHub

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

Jump to bottom

pool.GrowPhysicalDevice doesn't trigger change notification for blockdev.TotalPhysicalSize #3371

Closed
mvollmer opened this issue Jul 6, 2023 · 4 comments · Fixed by #3372
Closed

pool.GrowPhysicalDevice doesn't trigger change notification for blockdev.TotalPhysicalSize #3371

mvollmer opened this issue Jul 6, 2023 · 4 comments · Fixed by #3372
Assignees
@mulkieran @bgurney-rh

Comments

@mvollmer
Copy link
Collaborator

mvollmer commented Jul 6, 2023

Subject says it all. :-)
@mvollmer
Copy link
Collaborator Author

mvollmer commented Jul 6, 2023

See cockpit-project/cockpit#19079 for my efforts to support growing of block devices in Cockpit. It's working fine, except for the missing notification.

@mulkieran mulkieran self-assigned this Jul 6, 2023
@mulkieran
Copy link
Member

mulkieran commented Jul 6, 2023 •
edited
Loading

@mvollmer Could you give us version of stratisd and D-Bus revision that you're using? Err, revision is r3.

@mvollmer mvollmer mentioned this issue Jul 6, 2023
Merged
@mvollmer
Copy link
Collaborator Author

mvollmer commented Jul 6, 2023

@mvollmer Could you give us version of stratisd and D-Bus revision that you're using? Err, revision is r3.

Upps, sorry. Yes, revision is "r3", with stratisd-3.5.7-1.fc38.x86_64.

@mulkieran
Copy link
Member

Reproduced.

@mulkieran mulkieran mentioned this issue Jul 10, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mulkieran mulkieran

@bgurney-rh bgurney-rh

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Send blockdev change signal on grow physical mulkieran/stratisd
3 participants
@mulkieran @mvollmer @bgurney-rh