Storage

Data last fetched 2026-10-06 14:01 UTC · 55 open PR(s) across 9 repo(s) · 9793df

Back to index | Grouped by repo | Flat view

storage — Direct Review (15 PRs)

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.#180 doc(linstor): fix wrong comments on _call_local_method	0	Ythogtha	Wescoeur approved Storage review pending	0.0fresh	09:19 UTC	reviewers	Partially approved. Last action: Wescoeur requested review.
#173 fix(LVMSR): isMaster wasn't always set	13	Nambrok	Wescoeur changes requested Storage review pending	33.1moderate	11	author	Wescoeur requested changes. Last action: Nambrok pushed new commits. ⚠ There have been pushes since the last review(s).
This pull request needs your attention.#160 SR.scan: mark gone VDIs missing instead of forgetting	33	DatBewar	Storage review pending	50.9moderate	12	reviewers	Waiting for initial review. Last action: Wescoeur left a PR comment.

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.#714 large volumes: a few fixes and enhancements	14	glehmann	Kuruyia approved OS Platform & Release review pending Storage review pending	21.6fresh	3	reviewers	Partially approved. Last action: Kuruyia requested review.
#697 tests/ha add automatic tests scenarios for the HA	27	DatBewar	gduperrey changes requested glehmann commented gthvn1 commented rzr commented Storage review pending	43.7moderate	7	author	gduperrey requested changes. Last action: gthvn1 left a review 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	502	rushikeshjadhav	Wescoeur review pending glehmann commented stormi commented ydirson commented Storage review pending	98.5stale	15	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.#12 Add LINSTOR controller and satellite helpers	84	Wescoeur	Storage review pending	80.7stale	0	reviewers	Waiting for initial review. Last action: Wescoeur pushed new commits.
This pull request needs your attention.#6 Add TCP client/server modules	103	Wescoeur	Millefeuille42 review pending Storage review pending	83.9stale	0	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	1	Wescoeur	Storage review pending	3.5fresh	0	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.#20 Robustify iptables module	5	Wescoeur	Kuruyia approved Storage review pending	14.2fresh	3	reviewers	Partially approved. Last action: Wescoeur pushed new commits. ⚠ There have been pushes since the last review(s).
This pull request needs your attention.#19 Add LINSTOR backend	59	Wescoeur	Storage review pending	72.6stale	59	reviewers	Waiting for initial review. Last action: Wescoeur requested review.
#15 Add XCP storage RPC daemon	80	Wescoeur	DatBewar approved Kuruyia commented Storage review pending	79.4stale	75	author	Reviewer(s) commented. Last action: Kuruyia requested review.
This pull request needs your attention.#16 Add SMAPIv3 plugins build process + connectors architecture + tapdisk plugin.py skeleton	80	Wescoeur	Storage review pending	79.7stale	78	reviewers	Waiting for initial review. Last action: Wescoeur pushed new commits.
This pull request needs your attention.#14 Add semgrep exec path prefix check	80	Wescoeur	Storage review pending	79.7stale	80	reviewers	Waiting for initial review. Last action: Wescoeur pushed new commits.
This pull request needs your attention.#13 Cannot be merged: this pull request has conflicts that must be resolved Add XAPI SMAPIv3 API stubs	84	Wescoeur	Storage review pending	80.7stale	80	author + reviewers	Waiting for initial review. Last action: Wescoeur pushed new commits. ⊘ Conflicts must be resolved before this can merge.

Team Member Involvement (40 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	103	AnthoineB	guzu commented	70.3stale	32	reviewers	Waiting for reviewers. Last action: AnthoineB dismissed a review. ⚠ There have been pushes since the last review(s).

xapi-project/sm

xapi-project/sm: open pull requests
PR	Created	Author	Reviewers	Staleness	Last Activity	Waiting For	Rationale
#800 refactor: use VHD header to fetch block size	181	Millefeuille42		94.0stale	180	reviewers	Waiting for initial review. Last action: Millefeuille42 pushed new commits.
#784 Use serial instead of SCSI ID for USB devices	300	Wescoeur	MarkSymsCtx changes requested	96.7stale	257	author	MarkSymsCtx requested changes. Last action: MarkSymsCtx requested changes.

xcp-ng/blktap

xcp-ng/blktap: open pull requests
PR	Created	Author	Reviewers	Staleness	Last Activity	Waiting For	Rationale
#24 fixup! CA-429650: add unit tests for blkif request segment validation	5	guzu	AnthoineB review pending Nambrok review pending Wescoeur review pending	15.7fresh	4	reviewers	Waiting for reviewers. Last action: guzu pushed new commits. ⚠ There have been pushes since the last review(s).
#23 tapdisk: ctrl: grow output buffer for large list responses	11	guzu	AnthoineB review pending Nambrok review pending	28.3fresh	10	reviewers	Waiting for reviewers. Last action: guzu left a PR comment.
#22 Narrow blkif locking when parsing requests	18	guzu	AnthoineB review pending	36.9moderate	13	reviewers	Waiting for reviewers. Last action: guzu left a PR comment.
#21 tapdisk: fixes to 3.55.5-11.2 qcow2	21	AnthoineB	guzu approved	45.1moderate	20	missing review request	⚠ Partially approved, but no other reviewer. Last action: guzu approved.
#13 Fix unit tests with recent cmocka	91	guzu	AnthoineB review pending	82.4stale	35	reviewers	Waiting for reviewers. Last action: guzu 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
#124 feat(filesr): support vdi_revert	220	klmp200	AnthoineB review pending Kuruyia commented Lankou66 approved Millefeuille42 approved Nambrok changes requested Ythogtha approved imperosol review pending	87.8stale	13:35 UTC	author	Nambrok requested changes. Last action: Nambrok requested changes.
#132 feat(linstor): implement vdi_revert on linstor	159	klmp200	Millefeuille42 commented Nambrok review pending	92.2stale	13:17 UTC	author	Reviewer(s) commented. Last action: Millefeuille42 left a review comment.
#170 feat(qcow2util): Support Qcow2 subclusters	19	AnthoineB	Kuruyia approved Wescoeur changes requested	19.9fresh	10:30 UTC	author	Wescoeur requested changes. Last action: Kuruyia approved.
#178 fix(linstor) Wrong handling of timeout on LINSTOR HA HTTP server.	5	Ythogtha	Millefeuille42 commented Wescoeur review pending	9.2fresh	1	author	Reviewer(s) commented. Last action: Ythogtha left a PR comment. ⚠ The author has been active since the comment, but has not re-requested a review.
#179 fix(linstor) AbortGc launched even on local.	5	Ythogtha	Millefeuille42 review pending	9.2fresh	1	reviewers	Waiting for reviewers. Last action: Ythogtha requested review from Millefeuille42.
#161 fix(linstor): handle volume creation failure in clone rollback	33	Kuruyia	Millefeuille42 commented Wescoeur review pending klmp200 review pending	28.8fresh	1	author	Reviewer(s) commented. Last action: Kuruyia left a PR comment. ⚠ The author has been active since the comment, but has not re-requested a review.
#177 fix(qcow2util): handle undefined before job started	5	Nambrok	Kuruyia commented	14.2fresh	4	author	Reviewer(s) commented. Last action: Kuruyia left a review comment.
#176 fix(linstor): health check helper broken due to API changes	7	Wescoeur	Kuruyia approved Millefeuille42 approved Nambrok approved	20.7fresh	7	approved	All approvals satisfied. Last action: Nambrok approved.
#128 feat(lvmsr): support `vdi_revert` on lvm	194	klmp200	Lankou66 approved Millefeuille42 approved Nambrok review pending	87.2stale	8	reviewers	Partially approved. Last action: klmp200 pushed new commits. ⚠ There have been pushes since the last review(s).
#127 Add QCOW2 support	200	Nambrok	Millefeuille42 commented Wescoeur changes requested klmp200 changes requested	68.0stale	11	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	210	Wescoeur	Lankou66 approved Millefeuille42 approved Nambrok review pending	87.0stale	12	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).
#169 Cannot be merged: this pull request has conflicts that must be resolved fix(blktap2): stop using lsof to find the tapdisk pid	20	ppomes	Nambrok approved Wescoeur approved	43.1moderate	19	author	All approvals satisfied. Last action: ppomes left a PR comment. ⊘ Conflicts must be resolved before this can merge.
#108 Cannot be merged: this pull request has conflicts that must be resolved implement the capability to desactivate CBT on all vdi snapshot child	328	Lankou66	Nambrok review pending Wescoeur approved	97.1stale	34	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).
#145 Cannot be merged: this pull request has conflicts that must be resolved qcow2_helper: add support for compressed images	105	guzu	AnthoineB approved Nambrok review pending	71.7stale	36	author + reviewers	Partially approved. Last action: guzu left a PR comment. ⊘ Conflicts must be resolved before this can merge.

xcp-ng/xcp-ng-org

xcp-ng/xcp-ng-org: open pull requests
PR	Created	Author	Reviewers	Staleness	Last Activity	Waiting For	Rationale
#520 doc: Add pull request guidelines	56	stormi	bleader commented dinhngtu commented freezed commented lmolina commented rzr commented thomas-dkmt changes requested ydirson commented	38.7moderate	12:49 UTC	author	thomas-dkmt requested changes. Last action: ydirson left a PR comment.
#542 docs: add troubleshooting page for ZFS SR not imported after reboot	19	webdawg	Ythogtha changes requested freezed commented stormi review pending thomas-dkmt changes requested	26.8fresh	4	author	thomas-dkmt requested changes. Last action: stormi mentioned webdawg.
#486 Add a glossary plugin with terms	110	freezed	stormi review pending thomas-dkmt changes requested	47.2moderate	4	author	thomas-dkmt requested changes. Last action: thomas-dkmt mentioned freezed, thomas-dkmt.
#502 Add some details in docs related to NFS SR settings	70	freezed	Kuruyia approved olivierlambert commented stormi review pending thomas-dkmt approved	57.8moderate	18	author	Reviewer(s) commented. Last action: thomas-dkmt approved. ⚠ 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
#699 Cannot be merged: this pull request has conflicts that must be resolved Add a tests about VDI `on-boot=reset`	25	freezed	Kuruyia commented glehmann commented guzu review pending	23.3fresh	0	author	Reviewer(s) commented. Last action: freezed 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.
#722 Check that revert feature is enabled on SR when testing vdi_revert	8	klmp200	Kuruyia approved Wescoeur review pending Ythogtha review pending glehmann approved	23.1fresh	4	reviewers	Partially approved. Last action: glehmann approved.
#726 linstor: add test for healthcheck in linstor-manager	7	Nambrok	Kuruyia commented Wescoeur changes requested glehmann approved	20.7fresh	6	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()	354	guzu → glehmann	gthvn1 commented rzr commented stormi review pending ydirson review pending	65.9stale	7	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	88	klmp200	Lankou66 approved Millefeuille42 approved glehmann approved gthvn1 commented	50.6moderate	8	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.
#665 quicktest: split tests into SR specific and common ones	54	gthvn1	Wescoeur approved glehmann approved last-genius commented stormi commented	63.5stale	33	author	Reviewer(s) commented. Last action: Wescoeur approved. ⚠ 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	90	DatBewar	Ythogtha approved glehmann approved stormi changes requested	74.1stale	46	author	stormi requested changes. Last action: stormi requested changes mentioning DatBewar, glehmann.
#668 storage: force SR scan before coalesce to ensure GC completed	52	glehmann	Nambrok approved gduperrey approved rzr approved	69.5stale	52	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	329	guzu → ydirson	dinhngtu changes requested glehmann commented	97.1stale	91	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	301	ydirson	Wescoeur approved gthvn1 approved rzr approved semarie changes requested	96.7stale	0	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
#11 Add DRBD helpers	97	Wescoeur	Kuruyia commented Millefeuille42 commented	39.2moderate	09:37 UTC	author	Reviewer(s) commented. Last action: Millefeuille42 left a review comment.
#10 Add JSON-RPC client/server modules + API	98	Wescoeur	Kuruyia commented	43.0moderate	0	author	Reviewer(s) commented. Last action: Wescoeur pushed new commits. ⚠ There have been pushes since the last review(s).
#17 Cannot be merged: this pull request has conflicts that must be resolved Add DRBD backend function `get_openers`	77	Kuruyia	Millefeuille42 review pending Wescoeur changes requested	77.4stale	68	author	Wescoeur requested changes. Last action: Kuruyia left a PR comment. ⊘ Conflicts must be resolved before this can merge.

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`	112	DatBewar	edwintorok commented minglumlu approved	51.5moderate	6	author	Reviewer(s) commented. Last action: DatBewar pushed new commits. ⚠ There have been pushes since the last review(s).