| PR | Created | Author | Reviewers | Staleness | Last Activity | Waiting For | Rationale |
|---|---|---|---|---|---|---|---|
| This pull request needs your attention.#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. |
| PR | Created | Author | Reviewers | Staleness | Last Activity | Waiting For | Rationale |
|---|---|---|---|---|---|---|---|
| This pull request needs your attention.#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. |
| This pull request needs your attention.#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). |
| PR | Created | Author | Reviewers | Staleness | Last Activity | Waiting For | Rationale |
|---|---|---|---|---|---|---|---|
| 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.#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). |
| PR | Created | Author | Reviewers | Staleness | Last Activity | Waiting For | Rationale |
|---|---|---|---|---|---|---|---|
| This pull request needs your attention.#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. |
| This pull request needs your attention.#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). |
| This pull request needs your attention.#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). |
| This pull request needs your attention.#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). |
| This pull request needs your attention.#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. |
| 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.#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. |
| 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. |
| 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. |
| 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. |
| 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. |
| PR | Created | Author | Reviewers | Staleness | Last Activity | Waiting For | Rationale |
|---|---|---|---|---|---|---|---|
| #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). |
| 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. |
| #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. |
| #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). |
| #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). |
| 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. |
| 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. |
| #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. |
| #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. |
| 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. |
| PR | Created | Author | Reviewers | Staleness | Last Activity | Waiting For | Rationale |
|---|---|---|---|---|---|---|---|
| #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). |
| #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). |
| #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. |
| #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. |
| #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. |
| #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). |
| #19 Add LINSTOR backend | 73 | Wescoeur | Storage review pending | 77.7stale | 73 | reviewers | Waiting for initial review. Last action: Wescoeur requested review. |
| 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. |
| 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. |