Storage

Data last fetched 2026-09-29 12:56 UTC · 54 open PR(s) across 9 repo(s) · 9480a5

Back to index | Grouped by repo | Flat view

storage — Direct Review (18 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.#132 feat(linstor): implement vdi_revert on linstor	152	klmp200	Millefeuille42 review pending Nambrok review pending Storage review pending	91.5stale	0	reviewers	Waiting for reviewers. Last action: klmp200 pushed new commits. ⚠ There have been pushes since the last review(s).
#173 fix(LVMSR): isMaster wasn't always set	6	Nambrok	Wescoeur changes requested Storage review pending	16.9fresh	4	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.#161 Cannot be merged: this pull request has conflicts that must be resolved fix(linstor): handle volume creation failure in clone rollback	25	Kuruyia	Wescoeur review pending Storage review pending	49.8moderate	4	author + reviewers	Waiting for reviewers. Last action: Kuruyia pushed new commits. ⊘ Conflicts must be resolved before this can merge. ⚠ 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	25	DatBewar	Storage review pending	41.9moderate	5	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
#697 tests/ha add automatic tests scenarios for the HA	20	DatBewar	gduperrey changes requested glehmann commented gthvn1 commented rzr commented Storage review pending	32.8moderate	08:50 UTC	author	gduperrey requested changes. Last action: gthvn1 left a review comment.
This pull request needs your attention.#722 Check that revert feature is enabled on SR when testing vdi_revert	1	klmp200	OS Platform & Release review pending Storage review pending	3.5fresh	1	reviewers	Waiting for initial review. Last action: klmp200 pushed new commits.
This pull request needs your attention.#714 large volumes: a few fixes and enhancements	7	glehmann	OS Platform & Release review pending Storage review pending	20.7fresh	7	reviewers	Waiting for initial review. Last action: glehmann requested review.
#313 Cannot be merged: this pull request has conflicts that must be resolved Add host failure test to verify VM resiliency and SR stability	495	rushikeshjadhav	Wescoeur review pending glehmann commented stormi commented ydirson commented Storage review pending	98.4stale	8	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.#19 Add LINSTOR backend	52	Wescoeur	Storage review pending	69.5stale	52	reviewers	Waiting for initial review. Last action: Wescoeur requested review.
#10 Add JSON-RPC client/server modules + API	91	Wescoeur	Kuruyia commented Storage review pending	82.4stale	66	author	Reviewer(s) commented. Last action: Kuruyia requested review.
#15 Add XCP storage RPC daemon	73	Wescoeur	DatBewar approved Kuruyia commented Storage review pending	77.3stale	68	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	73	Wescoeur	Storage review pending	77.7stale	71	reviewers	Waiting for initial review. Last action: Wescoeur pushed new commits.
This pull request needs your attention.#14 Add semgrep exec path prefix check	73	Wescoeur	Storage review pending	77.7stale	73	reviewers	Waiting for initial review. Last action: Wescoeur pushed new commits.
This pull request needs your attention.#13 Add XAPI SMAPIv3 API stubs	77	Wescoeur	Storage review pending	78.9stale	73	reviewers	Waiting for initial review. Last action: Wescoeur pushed new commits.
This pull request needs your attention.#12 Add LINSTOR controller and satellite helpers	77	Wescoeur	Storage review pending	78.9stale	73	reviewers	Waiting for initial review. Last action: Wescoeur pushed new commits.
This pull request needs your attention.#6 Add TCP client/server modules	96	Wescoeur	Millefeuille42 review pending Storage review pending	82.4stale	74	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.#5 Add socket module	99	Wescoeur	Millefeuille42 approved Storage review pending	83.9stale	74	reviewers	Partially approved. Last action: Wescoeur pushed new commits. ⚠ There have been pushes since the last review(s).
This pull request needs your attention.#3 Add iptables helpers for future drivers	100	Wescoeur	Millefeuille42 approved Storage review pending	84.0stale	74	reviewers	Partially approved. Last action: Wescoeur pushed new commits. ⚠ There have been pushes since the last review(s).

Team Member Involvement (36 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	96	AnthoineB	guzu commented	66.2stale	25	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	174	Millefeuille42		93.5stale	173	reviewers	Waiting for initial review. Last action: Millefeuille42 pushed new commits.
#784 Use serial instead of SCSI ID for USB devices	293	Wescoeur	MarkSymsCtx changes requested	96.6stale	250	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
#23 tapdisk: ctrl: grow output buffer for large list responses	3	guzu	AnthoineB review pending Nambrok review pending	10.0fresh	3	reviewers	Waiting for reviewers. Last action: guzu left a PR comment.
#22 Narrow blkif locking when parsing requests	11	guzu	AnthoineB review pending	23.5fresh	6	reviewers	Waiting for reviewers. Last action: guzu left a PR comment.
#21 tapdisk: fixes to 3.55.5-11.2 qcow2	13	AnthoineB	guzu approved	33.1moderate	13	missing review request	⚠ Partially approved, but no other reviewer. Last action: guzu approved.
#13 Fix unit tests with recent cmocka	83	guzu	AnthoineB review pending	80.5stale	28	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
#176 fix(linstor): health check helper broken due to API changes	0	Wescoeur	Kuruyia approved Millefeuille42 approved Nambrok approved	0.0fresh	07:35 UTC	approved	All approvals satisfied. Last action: Nambrok approved.
#128 feat(lvmsr): support `vdi_revert` on lvm	187	klmp200	Lankou66 approved Millefeuille42 approved Nambrok review pending	85.8stale	0	reviewers	Partially approved. Last action: klmp200 pushed new commits. ⚠ There have been pushes since the last review(s).
#124 feat(filesr): support vdi_revert	213	klmp200	AnthoineB review pending Kuruyia commented Lankou66 approved Millefeuille42 approved Nambrok review pending Ythogtha approved imperosol review pending	86.6stale	1	author	Reviewer(s) commented. Last action: klmp200 pushed new commits. ⚠ There have been pushes since the last review(s).
#127 Add QCOW2 support	193	Nambrok	Millefeuille42 commented Wescoeur changes requested klmp200 changes requested	61.1stale	4	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	203	Wescoeur	Lankou66 approved Millefeuille42 approved Nambrok review pending	85.7stale	5	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).
#170 Cannot be merged: this pull request has conflicts that must be resolved feat(qcow2util): Support Qcow2 subclusters	12	AnthoineB	Kuruyia commented Wescoeur changes requested	25.7fresh	7	author	Wescoeur requested changes. Last action: Kuruyia left a review comment. ⊘ Conflicts must be resolved before this can merge.
#169 Cannot be merged: this pull request has conflicts that must be resolved fix(blktap2): stop using lsof to find the tapdisk pid	13	ppomes	Nambrok approved Wescoeur approved	32.2moderate	12	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	320	Lankou66	Nambrok review pending Wescoeur approved	97.0stale	27	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	98	guzu	AnthoineB approved Nambrok review pending	67.9stale	29	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
#542 docs: add troubleshooting page for ZFS SR not imported after reboot	12	webdawg	Ythogtha changes requested freezed commented stormi review pending thomas-dkmt changes requested	31.3moderate	6	author	thomas-dkmt requested changes. Last action: thomas-dkmt left a review comment.
#520 Cannot be merged: this pull request has conflicts that must be resolved doc: Add pull request guidelines	49	stormi	bleader commented dinhngtu commented freezed review pending lmolina commented rzr commented thomas-dkmt commented	47.5moderate	10	author	Reviewer(s) commented. Last action: dinhngtu left a review comment. ⊘ Conflicts must be resolved before this can merge.
#502 Add some details in docs related to NFS SR settings	63	freezed	Kuruyia approved olivierlambert commented stormi review pending thomas-dkmt approved	50.4moderate	11	author	Reviewer(s) commented. Last action: thomas-dkmt approved. ⚠ The author has been active since the comment, but has not re-requested a review.
#486 Add a glossary plugin with terms	103	freezed	stormi review pending thomas-dkmt changes requested	58.3moderate	13	author	thomas-dkmt requested changes. Last action: thomas-dkmt requested changes mentioning freezed.

xcp-ng/xcp-ng-tests

xcp-ng/xcp-ng-tests: open pull requests
PR	Created	Author	Reviewers	Staleness	Last Activity	Waiting For	Rationale
#460 Cannot be merged: this pull request has conflicts that must be resolved misc type related enhancements	175	glehmann	Lankou66 approved dinhngtu approved olivierh-pro approved rzr approved semarie approved vxgmichel approved ydirson changes requested	49.1moderate	09:40 UTC	author	ydirson requested changes. Last action: ydirson requested changes. ⊘ Conflicts must be resolved before this can merge.
#359 Cannot be merged: this pull request has conflicts that must be resolved Replace manual process handling by local_cmd()	347	guzu → glehmann	gthvn1 commented rzr commented stormi review pending ydirson review pending	58.3moderate	09:10 UTC	author	Reviewer(s) commented. Last action: gthvn1 left a review comment. ⊘ Conflicts must be resolved before this can merge.
#726 linstor: add test for healthcheck in linstor-manager	0	Nambrok	Kuruyia commented Wescoeur changes requested OS Platform & Release review pending	0.0fresh	0	author	Wescoeur requested changes. Last action: Kuruyia left a review comment.
#627 [waiting for sm feature] Create tests for vdi_revert on file based SRs	81	klmp200	Lankou66 approved Millefeuille42 approved glehmann approved gthvn1 commented	39.4moderate	1	author	Reviewer(s) commented. Last action: klmp200 left a PR comment. ⚠ The author has been active since the comment, but has not re-requested a review.
#476 Add ty type checker configuration	166	glehmann	Wescoeur approved olivierh-pro approved rzr approved semarie approved stormi approved ydirson changes requested Windows Guest Tools review pending Hypervisor & Kernel review pending	77.1stale	3	author	ydirson requested changes. Last action: glehmann pushed new commits. ⚠ There have been pushes since the last review(s).
#705 lib: replace PackageManagerEnum with PackageManager	14	semarie	Nambrok review pending gduperrey review pending glehmann commented ydirson changes requested XAPI & Network review pending Hypervisor & Kernel review pending	30.9moderate	10	author	ydirson requested changes. Last action: semarie left a PR comment.
#665 quicktest: split tests into SR specific and common ones	47	gthvn1	Wescoeur approved glehmann approved last-genius commented stormi commented	58.4moderate	26	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	83	DatBewar	Ythogtha approved glehmann approved stormi changes requested	70.6stale	39	author	stormi requested changes. Last action: stormi requested changes mentioning DatBewar, glehmann.
#668 storage: force SR scan before coalesce to ensure GC completed	45	glehmann	Nambrok approved gduperrey approved rzr approved	65.7stale	45	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	322	guzu → ydirson	dinhngtu changes requested glehmann commented	97.0stale	84	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	294	ydirson	Wescoeur approved gthvn1 approved rzr approved	96.6stale	98	approved	All approvals satisfied. Last action: bleader mentioned ydirson.

xcp-ng/xcp-storage

xcp-ng/xcp-storage: open pull requests
PR	Created	Author	Reviewers	Staleness	Last Activity	Waiting For	Rationale
#1 Add project config files, GH workflow, tests, JSON-RPC support...	196	Wescoeur	DatBewar approved Lankou66 review pending Millefeuille42 review pending Nambrok review pending Ythogtha review pending klmp200 approved	86.1stale	27	reviewers	Partially approved. Last action: klmp200 approved.
#17 Add DRBD backend function `get_openers`	70	Kuruyia	Millefeuille42 review pending Wescoeur changes requested	75.0stale	61	author	Wescoeur requested changes. Last action: Kuruyia left a PR comment.
#11 Add DRBD helpers	90	Wescoeur	Kuruyia commented	78.7stale	67	author	Reviewer(s) commented. Last action: Wescoeur mentioned Millefeuille42. ⚠ The author has been active since the comment, but has not re-requested a review.
#4 Add service helpers for future drivers	100	Wescoeur	Millefeuille42 approved Ythogtha approved	84.0stale	70	approved	All approvals satisfied. Last action: Ythogtha approved.

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 `bvar.sa_to`	105	DatBewar	edwintorok commented	40.0moderate	07:44 UTC	author	Reviewer(s) commented. Last action: DatBewar mentioned lindig. ⚠ The author has been active since the comment, but has not re-requested a review.