Nambrok

Data last fetched 2026-10-20 16:26 UTC · 34 open PR(s) across 6 repo(s) · cc5b7a

Back to index | Grouped by repo | Flat view

It looks like they're waiting for me (13 PRs)

xapi-project/sm

xapi-project/sm: open pull requests
PR	Created	Author	Reviewers	Staleness	Last Activity	Waiting For	Rationale
This pull request needs your attention.#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.

xcp-ng/blktap

xcp-ng/blktap: open pull requests
PR	Created	Author	Reviewers	Staleness	Last Activity	Waiting For	Rationale
This pull request needs your attention.#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).
This pull request needs your attention.#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.
This pull request needs your attention.#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.

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.#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).
This pull request needs your attention.#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.
This pull request needs your attention.#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.
This pull request needs your attention.#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.
This pull request needs your attention.#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).
This pull request needs your attention.#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).
This pull request needs your attention.#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-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.#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).
This pull request needs your attention.#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.

It looks like I'm waiting on others (7 PRs)

xcp-ng/sm

xcp-ng/sm: open pull requests
PR	Created	Author	Reviewers	Staleness	Last Activity	Waiting For	Rationale
#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.
#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).
#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.
#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).

xcp-ng/xcp-ng-tests

xcp-ng/xcp-ng-tests: open pull requests
PR	Created	Author	Reviewers	Staleness	Last Activity	Waiting For	Rationale
#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.
#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.

Storage — Team Reviews Pending (14 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.#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	95	Wescoeur	Storage review pending	83.3stale	93	author + reviewers	Waiting for initial review. Last action: Wescoeur pushed new commits. ⊘ Conflicts must be resolved before this can merge.