Nambrok

Data last fetched 2026-11-11 19:12 UTC · 51 open PR(s) across 7 repo(s) · d04688

Back to index | Grouped by repo | Flat view

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

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.#23 tapdisk: ctrl: grow output buffer for large list responses	47	guzu	AnthoineB review pending Nambrok review pending	66.9stale	1	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.#27 tapdisk: wait for the NBD requests in flight before closing an image	34	guzu	AnthoineB review pending Nambrok review pending	58.2moderate	19	reviewers	Waiting for reviewers. Last action: guzu undrafted the PR.
This pull request needs your attention.#29 tapdisk: nbdserver: reuse the data buffer of the requests	34	guzu	AnthoineB review pending Nambrok review pending	58.2moderate	27	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
This pull request needs your attention.#198 fix(cleanup): untag relinking on leaves moved during coalesce	2	Lankou66	Nambrok review pending Wescoeur review pending Storage review pending	6.8fresh	2	reviewers	Waiting for reviewers. Last action: Lankou66 pushed new commits. ⚠ There have been pushes since the last review(s).
This pull request needs your attention.#108 implement the capability to desactivate CBT on all vdi snapshot child	364	Lankou66	Nambrok review pending Wescoeur approved	97.5stale	2	reviewers	Partially approved. Last action: Lankou66 pushed new commits. ⚠ There have been pushes since the last review(s).
This pull request needs your attention.#170 feat(qcow2util): Support Qcow2 subclusters	56	AnthoineB	Kuruyia approved Nambrok review pending Wescoeur approved	51.0moderate	2	reviewers	Partially approved. Last action: Wescoeur approved.
This pull request needs your attention.#145 qcow2_helper: add support for compressed images	141	guzu	AnthoineB approved Nambrok review pending	83.4stale	2	reviewers	Partially approved. Last action: guzu pushed new commits. ⚠ There have been pushes since the last review(s).
This pull request needs your attention.#128 feat(lvmsr): support `vdi_revert` on lvm	230	klmp200	Lankou66 approved Millefeuille42 approved Nambrok review pending	91.9stale	2	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 feat(filesr): support vdi_revert	257	klmp200	AnthoineB review pending Kuruyia review pending Lankou66 approved Millefeuille42 approved Nambrok review pending Wescoeur review pending Ythogtha approved imperosol review pending	73.6stale	2	reviewers	Partially approved. Last action: klmp200 pushed new commits. ⚠ There have been pushes since the last review(s).
This pull request needs your attention.#127 Add QCOW2 support	236	Nambrok	Millefeuille42 commented Wescoeur changes requested klmp200 changes requested	83.5stale	47	author	Wescoeur requested changes. Last action: Nambrok pushed new commits. ⚠ 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	34	guzu	Nambrok review pending OS Platform & Release review pending	44.9moderate	13	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	44	Nambrok	Kuruyia commented Wescoeur changes requested glehmann approved	65.2stale	43	author	Wescoeur requested changes. Last action: glehmann approved.

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

xapi-project/sm

xapi-project/sm: open pull requests
PR	Created	Author	Reviewers	Staleness	Last Activity	Waiting For	Rationale
#802 Install relative symlinks	138	ydirson	Nambrok approved Wescoeur approved rzr approved	90.0stale	6	approved	All approvals satisfied. Last action: Nambrok approved.

xcp-ng/sm

xcp-ng/sm: open pull requests
PR	Created	Author	Reviewers	Staleness	Last Activity	Waiting For	Rationale
#195 fix(linstor): clear FLAG_TYPE_ABORT in linstor-manager:abortGc	5	Ythogtha	Kuruyia commented Millefeuille42 changes requested Nambrok review pending Wescoeur changes requested	11.0fresh	1	author	Wescoeur requested changes. Last action: Millefeuille42 requested changes.
#132 feat(linstor): implement vdi_revert on linstor	196	klmp200	Millefeuille42 commented Nambrok review pending	94.6stale	2	author	Reviewer(s) commented. Last action: klmp200 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	247	Wescoeur	Lankou66 approved Millefeuille42 approved Nambrok review pending	91.9stale	48	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).

Storage — Team Reviews Pending (35 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	27	Wescoeur	klmp200 review pending rzr approved Storage review pending	47.6moderate	2	reviewers	Partially approved. Last action: AnthoineB requested review from klmp200.

xcp-ng/sm

xcp-ng/sm: open pull requests
PR	Created	Author	Reviewers	Staleness	Last Activity	Waiting For	Rationale
#196 fix(linstor): retry volume building on volume manager instantiation	5	Kuruyia	Wescoeur changes requested Storage review pending	15.7fresh	2	author	Wescoeur requested changes. Last action: Wescoeur requested review.
This pull request needs your attention.#179 fix(linstor) AbortGc launched even on local.	41	Ythogtha	Millefeuille42 approved Wescoeur review pending Storage review pending	61.8stale	2	reviewers	Partially approved. Last action: AnthoineB requested review from Wescoeur.
This pull request needs your attention.#192 Fix stale dm nodes	6	AnthoineB	Lankou66 review pending Storage review pending	18.3fresh	2	reviewers	Waiting for reviewers. Last action: AnthoineB requested review from Lankou66.
This pull request needs your attention.#160 SR.scan: mark gone VDIs missing instead of forgetting	69	DatBewar	Wescoeur review pending Storage review pending	73.9stale	15	reviewers	Waiting for reviewers. Last action: DatBewar requested review from Wescoeur.

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.#788 storage/linstor: use JSON output for LINSTOR resource listing	1	Kuruyia	rzr approved Storage review pending	3.5fresh	1	reviewers	Partially approved. Last action: rzr approved.
#724 conftest.py: add fail_with_9() fixture	44	semarie	glehmann commented rzr approved ydirson changes requested XAPI & Network review pending Hypervisor & Kernel review pending Storage review pending	32.4moderate	1	author	ydirson requested changes. Last action: ydirson left a review comment.
This pull request needs your attention.#749 Cannot be merged: this pull request has conflicts that must be resolved tests/storage: Test the NBD endpoint exposed by XAPI	29	last-genius	AnthoineB review pending OS Platform & Release review pending Storage review pending	49.9moderate	2	author + reviewers	Waiting for reviewers. Last action: AnthoineB requested review from AnthoineB. ⊘ Conflicts must be resolved before this can merge.
This pull request needs your attention.#759 Cannot be merged: this pull request has conflicts that must be resolved Graceful interrupt via Ctrl+C	22	vxgmichel	glehmann approved XAPI & Network review pending Hypervisor & Kernel review pending Storage review pending	40.3moderate	7	author + reviewers	Partially approved. Last action: vxgmichel pushed new commits. ⊘ Conflicts must be resolved before this can merge. ⚠ There have been pushes since the last review(s).
#313 Cannot be merged: this pull request has conflicts that must be resolved Add host failure test to verify VM resiliency and SR stability	539	rushikeshjadhav	Wescoeur review pending glehmann commented stormi commented ydirson commented Storage review pending	98.7stale	51	author	Reviewer(s) commented. Last action: Wescoeur mentioned Lankou66. ⊘ 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
This pull request needs your attention.#75 lvm.py, service.py: sanitize plugin arguments	16	DatBewar	Lankou66 review pending guzu review pending semarie approved Storage review pending	38.1moderate	2	reviewers	Partially approved. Last action: AnthoineB requested review from Lankou66.

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	95	Wescoeur	Storage review pending	83.3stale	0	reviewers	Waiting for initial review. Last action: Wescoeur pushed new commits.
#22 Add LINSTOR controller and satellite helpers	34	Wescoeur	Kuruyia commented Storage review pending	58.2moderate	5	author	Reviewer(s) commented. Last action: Kuruyia requested review.
This pull request needs your attention.#46 feat(tests/core): add `FifoLock` tests + more for `FileLock`	5	Wescoeur	Storage review pending	15.7fresh	5	reviewers	Waiting for initial review. Last action: Wescoeur requested review.
This pull request needs your attention.#45 Add more semgrep checks	5	Wescoeur	Storage review pending	15.7fresh	5	reviewers	Waiting for initial review. Last action: Wescoeur requested review.
This pull request needs your attention.#44 fix(tests/core): simplify `LinstorController.get_uri` tests	5	Wescoeur	Storage review pending	15.7fresh	5	reviewers	Waiting for initial review. Last action: Wescoeur pushed new commits.
This pull request needs your attention.#43 Robustify pytest usage: fixtures instead of @patch, autospec, fake FS...	7	Wescoeur	Storage review pending	20.7fresh	5	reviewers	Waiting for initial review. Last action: Wescoeur pushed new commits.
This pull request needs your attention.#42 feat(tests/core): add `log` tests	7	Wescoeur	Storage review pending	20.7fresh	7	reviewers	Waiting for initial review. Last action: Wescoeur requested review.
This pull request needs your attention.#41 feat(core): add `ThreadGroup` helper	8	Wescoeur	Storage review pending	23.1fresh	7	reviewers	Waiting for initial review. Last action: Wescoeur pushed new commits.
This pull request needs your attention.#40 feat(core): add `kill_process` helper	8	Wescoeur	Storage review pending	23.1fresh	7	reviewers	Waiting for initial review. Last action: Wescoeur pushed new commits.
This pull request needs your attention.#39 feat(core): use a new helper `scoped_factory` to prevent leaks	8	Wescoeur	Storage review pending	23.1fresh	7	reviewers	Waiting for initial review. Last action: Wescoeur pushed new commits.
This pull request needs your attention.#38 feat(core): simplify code using `Clock`/`Timer` helpers	8	Wescoeur	Storage review pending	23.1fresh	7	reviewers	Waiting for initial review. Last action: Wescoeur pushed new commits.
This pull request needs your attention.#37 chore(tests): add Python 3.6 test packages for XCP-ng 8.3	12	Wescoeur	Storage review pending	31.3moderate	7	reviewers	Waiting for initial review. Last action: Wescoeur pushed new commits.
This pull request needs your attention.#36 Robustify semgrep rules	13	Wescoeur	Storage review pending	33.1moderate	7	reviewers	Waiting for initial review. Last action: Wescoeur pushed new commits.
This pull request needs your attention.#35 Rework GH workflow	13	Wescoeur	Storage review pending	33.1moderate	8	reviewers	Waiting for initial review. Last action: Wescoeur pushed new commits.
This pull request needs your attention.#34 Add `FifoLock`/`ReadWriteFifoLock` helpers	15	Wescoeur	Storage review pending	36.5moderate	8	reviewers	Waiting for initial review. Last action: Wescoeur pushed new commits.
This pull request needs your attention.#33 Add `close_fd` helper	18	Wescoeur	Storage review pending	41.1moderate	8	reviewers	Waiting for initial review. Last action: Wescoeur pushed new commits.
This pull request needs your attention.#32 fix(core): robustify socket helpers + add UNIX support	18	Wescoeur	Storage review pending	41.1moderate	8	reviewers	Waiting for initial review. Last action: Wescoeur pushed new commits.
This pull request needs your attention.#21 feat(core): add `FileLock`/`ReadWriteFileLock` helpers	37	Wescoeur	Storage review pending	60.5stale	8	reviewers	Waiting for initial review. Last action: Wescoeur pushed new commits.
This pull request needs your attention.#16 Add SMAPIv3 plugins build process + connectors architecture + tapdisk plugin.py skeleton	117	Wescoeur	Storage review pending	87.2stale	8	reviewers	Waiting for initial review. Last action: Wescoeur pushed new commits.
This pull request needs your attention.#31 Add more Protocol/RPC tests	21	Wescoeur	Storage review pending	45.1moderate	8	reviewers	Waiting for initial review. Last action: Wescoeur pushed new commits.
This pull request needs your attention.#30 refactor(core): simplify `iptables` commands	21	Wescoeur	Storage review pending	45.1moderate	8	reviewers	Waiting for initial review. Last action: Wescoeur pushed new commits.
This pull request needs your attention.#14 Add new semgrep rules + enforce conventions	117	Wescoeur	Storage review pending	87.2stale	8	reviewers	Waiting for initial review. Last action: Wescoeur pushed new commits.
This pull request needs your attention.#13 Add XAPI SMAPIv3 API stubs	120	Wescoeur	Storage review pending	87.7stale	8	reviewers	Waiting for initial review. Last action: Wescoeur pushed new commits.
This pull request needs your attention.#6 Add TCP client/server modules	139	Wescoeur	Millefeuille42 review pending Storage review pending	89.5stale	8	reviewers	Waiting for reviewers. Last action: Wescoeur pushed new commits. ⚠ There have been pushes since the last review(s).