Storage

Data last fetched 2026-10-20 16:21 UTC · 71 open PR(s) across 15 repo(s) · 6a459c

Back to index | Grouped by repo | Flat view

storage — Direct Review (15 PRs)

xcp-ng/http-nbd-transfer

xcp-ng/http-nbd-transfer: open pull requests
PR	Created	Author	Reviewers	Staleness	Last Activity	Waiting For	Rationale
This pull request needs your attention.#20 Robustify code	5	Wescoeur	rzr approved Storage review pending	15.7fresh	0	reviewers	Partially approved. Last action: Wescoeur pushed new commits. ⚠ There have been pushes since the last review(s).
#19 Fix and modernize code	6	Wescoeur	rzr commented Storage review pending	18.3fresh	0	author	Reviewer(s) commented. Last action: Wescoeur pushed new commits. ⚠ There have been pushes since the last review(s).

xcp-ng/sm

xcp-ng/sm: open pull requests
PR	Created	Author	Reviewers	Staleness	Last Activity	Waiting For	Rationale
This pull request needs your attention.#185 feat(linstor): set a max_workers for linstor scan	0	Nambrok	Storage review pending	0.0fresh	14:07 UTC	reviewers	Waiting for initial review. Last action: Nambrok pushed new commits.
This pull request needs your attention.#183 fix(cleanup): catch exception to clean up the running file	4	AnthoineB	Storage review pending	12.9fresh	4	reviewers	Waiting for initial review. Last action: AnthoineB requested review.
This pull request needs your attention.#160 SR.scan: mark gone VDIs missing instead of forgetting	47	DatBewar	Storage review pending	62.7stale	6	reviewers	Waiting for initial review. Last action: DatBewar pushed new commits.
This pull request needs your attention.#168 feat: add LVM metadata backup	35	Kuruyia	Millefeuille42 review pending Storage review pending	45.3moderate	8	reviewers	Waiting for reviewers. Last action: AnthoineB requested review from Millefeuille42.

xcp-ng/xcp-ng-tests

xcp-ng/xcp-ng-tests: open pull requests
PR	Created	Author	Reviewers	Staleness	Last Activity	Waiting For	Rationale
This pull request needs your attention.#749 tests/storage: Test the NBD endpoint exposed by XAPI	7	last-genius	OS Platform & Release review pending Storage review pending	9.7fresh	14:04 UTC	reviewers	Waiting for initial review. Last action: last-genius left a PR comment.
#313 Cannot be merged: this pull request has conflicts that must be resolved Add host failure test to verify VM resiliency and SR stability	517	rushikeshjadhav	Wescoeur review pending glehmann commented stormi commented ydirson commented Storage review pending	98.6stale	29	author	Reviewer(s) commented. Last action: Wescoeur mentioned Lankou66. ⊘ Conflicts must be resolved before this can merge.

xcp-ng/xcp-storage

xcp-ng/xcp-storage: open pull requests
PR	Created	Author	Reviewers	Staleness	Last Activity	Waiting For	Rationale
This pull request needs your attention.#14 Add new semgrep rules + enforce conventions	95	Wescoeur	Storage review pending	83.3stale	1	reviewers	Waiting for initial review. Last action: Wescoeur pushed new commits.
This pull request needs your attention.#13 Add XAPI SMAPIv3 API stubs	98	Wescoeur	Storage review pending	83.9stale	1	reviewers	Waiting for initial review. Last action: Wescoeur pushed new commits.
This pull request needs your attention.#22 Add LINSTOR controller and satellite helpers	12	Wescoeur	Storage review pending	31.3moderate	1	reviewers	Waiting for initial review. Last action: Wescoeur pushed new commits.
This pull request needs your attention.#6 Add TCP client/server modules	117	Wescoeur	Millefeuille42 review pending Storage review pending	86.4stale	1	reviewers	Waiting for reviewers. Last action: Wescoeur pushed new commits. ⚠ There have been pushes since the last review(s).
This pull request needs your attention.#21 Cannot be merged: this pull request has conflicts that must be resolved feat(core): add `FileLock`/`ReadWriteFileLock` helpers	15	Wescoeur	Storage review pending	36.5moderate	15	author + reviewers	Waiting for initial review. Last action: Wescoeur pushed new commits. ⊘ Conflicts must be resolved before this can merge.
This pull request needs your attention.#19 Add LINSTOR backend	73	Wescoeur	Storage review pending	77.7stale	73	reviewers	Waiting for initial review. Last action: Wescoeur requested review.
This pull request needs your attention.#16 Cannot be merged: this pull request has conflicts that must be resolved Add SMAPIv3 plugins build process + connectors architecture + tapdisk plugin.py skeleton	94	Wescoeur	Storage review pending	83.1stale	93	author + reviewers	Waiting for initial review. Last action: Wescoeur pushed new commits. ⊘ Conflicts must be resolved before this can merge.

Team Member Involvement (56 PRs)

xapi-project/blktap

xapi-project/blktap: open pull requests
PR	Created	Author	Reviewers	Staleness	Last Activity	Waiting For	Rationale
#442 Switch blktap licensing to GPLv2	117	AnthoineB	guzu approved	49.2moderate	5	reviewers	⚠ Partially approved, but no other reviewer. Last action: AnthoineB mentioned MarkSymsCtx.

xapi-project/sm

xapi-project/sm: open pull requests
PR	Created	Author	Reviewers	Staleness	Last Activity	Waiting For	Rationale
#802 Install relative symlinks	116	ydirson	Nambrok review pending Wescoeur approved rzr approved	87.1stale	13:35 UTC	reviewers	Partially approved. Last action: Wescoeur approved.
#800 refactor: use VHD header to fetch block size	195	Millefeuille42		94.5stale	194	reviewers	Waiting for initial review. Last action: Millefeuille42 pushed new commits.
#784 Use serial instead of SCSI ID for USB devices	314	Wescoeur	MarkSymsCtx changes requested	96.9stale	272	author	MarkSymsCtx requested changes. Last action: MarkSymsCtx requested changes.

xapi-project/xen-api

xapi-project/xen-api: open pull requests
PR	Created	Author	Reviewers	Staleness	Last Activity	Waiting For	Rationale
#7326 CP-313992: allow SM to specify the VBD datapath backend	8	MarkSymsCtx	AnthoineB review pending LunfanZhang review pending cplaursen review pending last-genius review pending minglumlu approved	12.7fresh	1	reviewers	Partially approved. Last action: MarkSymsCtx left a PR comment.

xcp-ng-rpms/http-nbd-transfer

xcp-ng-rpms/http-nbd-transfer: open pull requests
PR	Created	Author	Reviewers	Staleness	Last Activity	Waiting For	Rationale
#10 Add tests subpackage that adapts upstream's test suite	11	rzr	Wescoeur approved glehmann approved	13.8fresh	08:00 UTC	approved	All approvals satisfied. Last action: rzr resolved a review thread.

xcp-ng-rpms/nbdkit

xcp-ng-rpms/nbdkit: open pull requests
PR	Created	Author	Reviewers	Staleness	Last Activity	Waiting For	Rationale
#1 8.3: nbdkit rebuild on updated gnutls	29	rzr	Wescoeur approved glehmann approved	33.6moderate	4	approved	All approvals satisfied. Last action: Wescoeur approved.

xcp-ng-rpms/xha

xcp-ng-rpms/xha: open pull requests
PR	Created	Author	Reviewers	Staleness	Last Activity	Waiting For	Rationale
#8 Update our patches on 25.2.0-2	1	DatBewar	Wescoeur approved rzr approved OS Platform & Release review pending	3.5fresh	1	reviewers	Partially approved. Last action: Wescoeur withdrew the review request from storage.

xcp-ng/blktap

xcp-ng/blktap: open pull requests
PR	Created	Author	Reviewers	Staleness	Last Activity	Waiting For	Rationale
#29 tapdisk: nbdserver: reuse the data buffer of the requests	11	guzu	AnthoineB review pending Nambrok review pending	29.4fresh	5	reviewers	Waiting for reviewers. Last action: guzu pushed new commits. ⚠ There have been pushes since the last review(s).
#26 Cannot be merged: this pull request has conflicts that must be resolved tapdisk: nbdserver: fix the handshake error paths spinning on EBADF	12	guzu	AnthoineB review pending Nambrok review pending	31.3moderate	11	author + reviewers	Waiting for reviewers. Last action: guzu requested review from Nambrok. ⊘ Conflicts must be resolved before this can merge.
#23 Cannot be merged: this pull request has conflicts that must be resolved tapdisk: ctrl: grow output buffer for large list responses	25	guzu	AnthoineB review pending Nambrok review pending	49.8moderate	25	author + reviewers	Waiting for reviewers. Last action: guzu left a PR comment. ⊘ Conflicts must be resolved before this can merge.
#22 Cannot be merged: this pull request has conflicts that must be resolved Narrow blkif locking when parsing requests	32	guzu	AnthoineB review pending	54.1moderate	27	author + reviewers	Waiting for reviewers. Last action: guzu left a PR comment. ⊘ Conflicts must be resolved before this can merge.
#13 Cannot be merged: this pull request has conflicts that must be resolved Fix unit tests with recent cmocka	105	guzu	AnthoineB review pending	85.2stale	50	author + reviewers	Waiting for reviewers. Last action: guzu pushed new commits. ⊘ Conflicts must be resolved before this can merge. ⚠ There have been pushes since the last review(s).

xcp-ng/http-nbd-transfer

xcp-ng/http-nbd-transfer: open pull requests
PR	Created	Author	Reviewers	Staleness	Last Activity	Waiting For	Rationale
#18 Improve python scripts performance	6	Wescoeur	Millefeuille42 commented rzr approved	18.3fresh	08:57 UTC	author	Reviewer(s) commented. Last action: Millefeuille42 left a review comment.

xcp-ng/sm

xcp-ng/sm: open pull requests
PR	Created	Author	Reviewers	Staleness	Last Activity	Waiting For	Rationale
#184 fix(linstor): improve `ss` command speed	1	Wescoeur	AnthoineB approved Kuruyia approved Millefeuille42 approved Nambrok approved Ythogtha approved klmp200 approved	3.5fresh	12:11 UTC	approved	All approvals satisfied. Last action: Millefeuille42 approved.
#132 feat(linstor): implement vdi_revert on linstor	174	klmp200	Millefeuille42 commented Nambrok review pending	93.5stale	5	author	Reviewer(s) commented. Last action: klmp200 pushed new commits. ⚠ There have been pushes since the last review(s).
#128 feat(lvmsr): support `vdi_revert` on lvm	208	klmp200	Lankou66 approved Millefeuille42 approved Nambrok review pending	89.4stale	5	reviewers	Partially approved. Last action: klmp200 pushed new commits. ⚠ There have been pushes since the last review(s).
#124 Cannot be merged: this pull request has conflicts that must be resolved feat(filesr): support vdi_revert	235	klmp200	AnthoineB review pending Kuruyia review pending Lankou66 approved Millefeuille42 approved Nambrok review pending Wescoeur review pending Ythogtha approved imperosol review pending	53.9moderate	5	author + reviewers	Partially approved. Last action: klmp200 requested review from Kuruyia. ⊘ Conflicts must be resolved before this can merge.
#177 fix(qcow2util): handle undefined before job started	20	Nambrok	AnthoineB approved Kuruyia commented	43.1moderate	7	author	Reviewer(s) commented. Last action: AnthoineB approved.
#169 fix(blktap2): stop using lsof to find the tapdisk pid	34	ppomes	Nambrok approved Wescoeur approved	38.3moderate	7	approved	All approvals satisfied. Last action: ppomes mentioned Wescoeur, Nambrok.
#173 fix(LVMSR): isMaster wasn't always set	27	Nambrok	Lankou66 approved Wescoeur changes requested	51.9moderate	8	author	Wescoeur requested changes. Last action: Lankou66 approved.
#145 qcow2_helper: add support for compressed images	119	guzu	AnthoineB approved Nambrok review pending	77.4stale	8	reviewers	Partially approved. Last action: guzu pushed new commits. ⚠ There have been pushes since the last review(s).
#179 fix(linstor) AbortGc launched even on local.	19	Ythogtha	Millefeuille42 approved	39.3moderate	13	missing review request	⚠ Partially approved, but no other reviewer. Last action: Millefeuille42 approved.
#161 fix(linstor): handle volume creation failure in clone rollback	47	Kuruyia	Millefeuille42 commented Wescoeur review pending klmp200 review pending	50.8moderate	13	author	Reviewer(s) commented. Last action: Millefeuille42 resolved a review thread.
#170 feat(qcow2util): Support Qcow2 subclusters	34	AnthoineB	Kuruyia approved Wescoeur changes requested	45.8moderate	14	author	Wescoeur requested changes. Last action: Kuruyia approved.
#127 Add QCOW2 support	214	Nambrok	Millefeuille42 commented Wescoeur changes requested klmp200 changes requested	76.0stale	25	author	Wescoeur requested changes. Last action: Nambrok pushed new commits. ⚠ There have been pushes since the last review(s).
#125 Cannot be merged: this pull request has conflicts that must be resolved Stable COW release	225	Wescoeur	Lankou66 approved Millefeuille42 approved Nambrok review pending	89.2stale	26	author	Nambrok requested changes. Last action: Wescoeur pushed new commits. ⊘ Conflicts must be resolved before this can merge. ⚠ There have been pushes since the last review(s).
#108 Cannot be merged: this pull request has conflicts that must be resolved implement the capability to desactivate CBT on all vdi snapshot child	342	Lankou66	Nambrok review pending Wescoeur approved	97.2stale	48	author + reviewers	Partially approved. Last action: Lankou66 pushed new commits. ⊘ Conflicts must be resolved before this can merge. ⚠ There have been pushes since the last review(s).

xcp-ng/xcp

xcp-ng/xcp: open pull requests
PR	Created	Author	Reviewers	Staleness	Last Activity	Waiting For	Rationale
#856 Update XHA maintainer in packages.json	1	DatBewar	Wescoeur approved gduperrey approved rzr approved	3.5fresh	1	approved	All approvals satisfied. Last action: Wescoeur approved.

xcp-ng/xcp-ng-org

xcp-ng/xcp-ng-org: open pull requests
PR	Created	Author	Reviewers	Staleness	Last Activity	Waiting For	Rationale
#520 Cannot be merged: this pull request has conflicts that must be resolved doc: Add pull request guidelines	70	stormi	bleader commented dinhngtu commented freezed commented glehmann commented lmolina commented rzr commented thomas-dkmt changes requested ydirson commented	57.0moderate	1	author	thomas-dkmt requested changes. Last action: glehmann left a review comment. ⊘ Conflicts must be resolved before this can merge.
#542 docs: add troubleshooting page for ZFS SR not imported after reboot	33	webdawg	Ythogtha approved freezed commented stormi review pending thomas-dkmt changes requested	48.7moderate	8	author	thomas-dkmt requested changes. Last action: Ythogtha approved.
#486 Cannot be merged: this pull request has conflicts that must be resolved Add a glossary plugin with terms	124	freezed	stormi review pending thomas-dkmt changes requested	63.2stale	18	author	thomas-dkmt requested changes. Last action: thomas-dkmt mentioned freezed, thomas-dkmt. ⊘ Conflicts must be resolved before this can merge.
#502 Cannot be merged: this pull request has conflicts that must be resolved Add some details in docs related to NFS SR settings	85	freezed	Kuruyia approved olivierlambert commented stormi review pending thomas-dkmt approved	68.0stale	32	author	Reviewer(s) commented. Last action: thomas-dkmt approved. ⊘ Conflicts must be resolved before this can merge. ⚠ The author has been active since the comment, but has not re-requested a review.

xcp-ng/xcp-ng-tests

xcp-ng/xcp-ng-tests: open pull requests
PR	Created	Author	Reviewers	Staleness	Last Activity	Waiting For	Rationale
#737 storage: add test for http-nbd-transfer	13	rzr	Wescoeur review pending glehmann approved stormi commented ydirson review pending	15.5fresh	08:04 UTC	author	Reviewer(s) commented. Last action: rzr pushed new commits. ⚠ There have been pushes since the last review(s).
#697 tests/ha add automatic tests scenarios for the HA	41	DatBewar	Wescoeur commented gduperrey changes requested glehmann approved gthvn1 commented rzr commented	36.5moderate	1	author	gduperrey requested changes. Last action: glehmann approved.
#739 storage: add tapdisk NBD server tests	12	guzu	AnthoineB approved OS Platform & Release review pending	31.3moderate	4	reviewers	Partially approved. Last action: guzu pushed new commits. ⚠ There have been pushes since the last review(s).
#738 storage: add VBD blktap tests	12	guzu	Nambrok review pending OS Platform & Release review pending	31.3moderate	4	reviewers	Waiting for reviewers. Last action: guzu pushed new commits. ⚠ There have been pushes since the last review(s).
#744 Use a reboot-safe symlink in /dev/disk/by-id as path in BlockDeviceInfo	11	vxgmichel	Nambrok review pending glehmann approved rzr commented	20.6fresh	4	author	Reviewer(s) commented. Last action: glehmann approved. ⚠ The author has been active since the comment, but has not re-requested a review.
#746 Add a test for VDI on-boot=reset parameter	8	freezed	Kuruyia approved glehmann approved	20.5fresh	4	approved	All approvals satisfied. Last action: glehmann approved.
#714 large volumes: a few fixes and enhancements	28	glehmann	DatBewar approved Kuruyia approved freezed review pending OS Platform & Release review pending	46.6moderate	8	reviewers	Partially approved. Last action: DatBewar approved.
#722 Check that revert feature is enabled on SR when testing vdi_revert	22	klmp200	Kuruyia approved Wescoeur approved Ythogtha review pending glehmann approved	46.4moderate	13	reviewers	Partially approved. Last action: Wescoeur approved.
#726 linstor: add test for healthcheck in linstor-manager	22	Nambrok	Kuruyia commented Wescoeur changes requested glehmann approved	46.4moderate	21	author	Wescoeur requested changes. Last action: glehmann approved.
#359 Cannot be merged: this pull request has conflicts that must be resolved Replace manual process handling by local_cmd()	368	guzu → glehmann	gthvn1 commented rzr commented stormi review pending ydirson review pending	76.1stale	21	author	Reviewer(s) commented. Last action: gthvn1 left a review comment. ⊘ Conflicts must be resolved before this can merge.
#627 Cannot be merged: this pull request has conflicts that must be resolved [waiting for sm feature] Create tests for vdi_revert on file based SRs	102	klmp200	Lankou66 approved Millefeuille42 approved glehmann approved gthvn1 commented	64.3stale	22	author	Reviewer(s) commented. Last action: klmp200 left a PR comment. ⊘ Conflicts must be resolved before this can merge. ⚠ The author has been active since the comment, but has not re-requested a review.
#625 Add test for rolling pool update with VM availability check on updated host	105	DatBewar	Ythogtha approved glehmann approved stormi changes requested	79.0stale	60	author	stormi requested changes. Last action: stormi requested changes mentioning DatBewar, glehmann.
#668 storage: force SR scan before coalesce to ensure GC completed	67	glehmann	Nambrok approved gduperrey approved rzr approved	75.7stale	67	approved	All approvals satisfied. Last action: rzr approved.
#362 Cannot be merged: this pull request has conflicts that must be resolved Collect logs in case of test failure	344	guzu → ydirson	dinhngtu changes requested glehmann commented	97.2stale	105	author	ydirson requested changes. Last action: stormi transferred ownership to ydirson. ⊘ Conflicts must be resolved before this can merge.

xcp-ng/xcp-ng-xapi-plugins

xcp-ng/xcp-ng-xapi-plugins: open pull requests
PR	Created	Author	Reviewers	Staleness	Last Activity	Waiting For	Rationale
#56 Switch to python3	316	ydirson	Wescoeur approved gthvn1 approved rzr approved semarie changes requested	96.9stale	15	author	semarie requested changes. Last action: semarie requested changes.

xcp-ng/xcp-storage

xcp-ng/xcp-storage: open pull requests
PR	Created	Author	Reviewers	Staleness	Last Activity	Waiting For	Rationale
#20 Robustify iptables module	19	Wescoeur	Kuruyia approved Ythogtha commented	41.7moderate	11:49 UTC	author	Reviewer(s) commented. Last action: Ythogtha left a review comment.
#27 Add a timeout to `drbdsetup` commands to prevent them from getting stuck	1	Wescoeur	Kuruyia approved Millefeuille42 approved	3.5fresh	09:22 UTC	approved	All approvals satisfied. Last action: Wescoeur pushed new commits. ⚠ There have been pushes since the last review(s).
#15 Add XCP storage RPC daemon	95	Wescoeur	DatBewar approved Kuruyia commented	44.3moderate	09:21 UTC	author	Reviewer(s) commented. Last action: Wescoeur pushed new commits. ⚠ There have been pushes since the last review(s).
#17 Add DRBD backend function `get_openers`	91	Kuruyia	Millefeuille42 review pending Wescoeur changes requested	81.4stale	1	author	Wescoeur requested changes. Last action: Kuruyia pushed new commits. ⚠ There have been pushes since the last review(s).
#11 Add DRBD helpers	111	Wescoeur	Kuruyia commented Millefeuille42 commented	59.8moderate	1	author	Reviewer(s) commented. Last action: Wescoeur pushed new commits. ⚠ There have been pushes since the last review(s).
#10 Add JSON-RPC client/server modules + API	112	Wescoeur	Kuruyia commented	60.8stale	1	author	Reviewer(s) commented. Last action: Wescoeur pushed new commits. ⚠ There have been pushes since the last review(s).
#24 chore: add GH commit conventions check	11	Wescoeur	Kuruyia commented	28.3fresh	8	author	Reviewer(s) commented. Last action: Kuruyia resolved a review thread.

xenserver/xha

xenserver/xha: open pull requests
PR	Created	Author	Reviewers	Staleness	Last Activity	Waiting For	Rationale
#33 Fix initialisation of unknown index value which overflows `hbvar.sa_to`	127	DatBewar	edwintorok commented minglumlu approved	53.8moderate	4	author	Reviewer(s) commented. Last action: DatBewar pushed new commits. ⚠ There have been pushes since the last review(s).