Skip to content

correctly set ownership during volume read#287

Merged
jmpesp merged 1 commit into
oxidecomputer:mainfrom
jmpesp:correctly_set_ownership
Apr 20, 2022
Merged

correctly set ownership during volume read#287
jmpesp merged 1 commit into
oxidecomputer:mainfrom
jmpesp:correctly_set_ownership

Conversation

@jmpesp

@jmpesp jmpesp commented Apr 15, 2022

Copy link
Copy Markdown
Contributor

A volume "owns" a block if the subvolume owns a block, not if the read
only parent does. Set or propagate ownership information accordingly.

@jmpesp jmpesp requested a review from leftwo April 15, 2022 13:02
A volume "owns" a block if the subvolume owns a block, not if the read
only parent does. Set or propagate ownership information accordingly.
@jmpesp jmpesp force-pushed the correctly_set_ownership branch from 53a7982 to 6a85b67 Compare April 20, 2022 20:27
@jmpesp

jmpesp commented Apr 20, 2022

Copy link
Copy Markdown
Contributor Author

Note: Alan and I discussed this out of band and agreed that this fix should go in separate of the integration tests to avoid merge conflicts with #257. Once that goes in, the integration test work can be done on top of that.

@jmpesp jmpesp merged commit 9b67ce7 into oxidecomputer:main Apr 20, 2022
@jmpesp jmpesp deleted the correctly_set_ownership branch April 20, 2022 21:00
jmpesp added a commit to jmpesp/omicron that referenced this pull request Apr 21, 2022
pull in latest crucible image to grab the volume ownership fix from
oxidecomputer/crucible#287
jmpesp added a commit to jmpesp/propolis that referenced this pull request Apr 21, 2022
pull in latest crucible image to grab the volume ownership fix from
oxidecomputer/crucible#287 plus reconciliation
work, among other things.
jmpesp added a commit to oxidecomputer/propolis that referenced this pull request Apr 21, 2022
pull in latest crucible image to grab the volume ownership fix from
oxidecomputer/crucible#287 plus reconciliation
work, among other things.
jmpesp added a commit to oxidecomputer/omicron that referenced this pull request Apr 25, 2022
pull in latest crucible image to grab the volume ownership fix from
oxidecomputer/crucible#287

This commit updates:
- crucible from a70d4589 to 257032d1
- propolis from 514fa1fc to ec4f3a41
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.

2 participants