Gerrit Promises alpha

Promises made in code review — “will fix in a follow-up” — and whether they were kept

3changes

14open

4in flight

16to check

23kept

3set aside

18in resolved threads

Details	Ticket	Change	Promises	Scanned	Actions
	LU-12669	LU-12669 ec: recover data from parity 62757 merged added by lefthy	8 open · 3 in flight · 0 to check · 21 kept 5 in resolved threads	5 h ago	Promises
Kept Fix LU-19536 (change 64359) which removed code needed for EC transient-page IO, causing crashes, instead of this patch Marc Vef (change-level) Kept Fix or file tickets for the mmap-read hang and the intermittent wrong-data reconstruction bugs in the disabled EC subtests Gerrit AI review for Lustre lustre/tests/sanity-ec.sh:29 Open Ensure EC read reconstruction falls back to waiting for OST recovery instead of returning -EIO; tracked/merged via another patch Zhenyu Xu COMMIT_MSG:38 Kept Add bounds check/LASSERT on k against MAX_OST_UNAVAIL to prevent stack buffer overflow Zhenyu Xu lustre/lov/lov_io.c:1867 Kept Fix reference leak during read reconstruction causing client OOM Marc Vef (change-level) Kept Verify that NULL cp_vmpage on parity pages is safe through all downstream cl_page code paths Patrick Farrell lustre/obdclass/cl_page.c:375 Open Fix CWARN/return value mismatch (rc=-EIO vs -EAGAIN) when patch is refreshed Marc Vef lustre/lov/lov_io.c:3011 Kept EAGAIN path can silently return EOF instead of error on unreadable stripe Zhenyu Xu lustre/lov/lov_io.c:2858 Kept Fix lov_io_rw_iter_init() LASSERTF that never actually validates CIT_EC_RD iter_init path Zhenyu Xu COMMIT_MSG:48 In flight Track FLR-EC DIO+AIO reconstruction work via filed critical ticket LU-20566 for 2.18.0 +1 repeat Andreas Dilger lustre/llite/file.c:2213 Open fix kmap_local unmap ordering/LIFO violation and KM_MAX_IDX overrun risk Zhenyu Xu lustre/lov/lov_io.c:2759 Kept Fix or document that true AIO O_DIRECT reads on EC files get no reconstruction at all Zhenyu Xu lustre/llite/file.c:2182 all 33 on the change's page
	LU-19548	LU-19548 lfs: update mirror split for EC support 64620 merged added by lefthy	6 open · 1 in flight · 2 to check · 2 kept 4 in resolved threads	10 h ago	Promises
Open Free and re-fetch layout per file in multi-file mirror split/delete so resync decisions use the correct file's layout Marc Vef lustre/utils/lfs.c:6287 Open Resolve mirror id once for -I/--pool paths so collect_mirror_id() doesn't wrongly count the mirror being split as a surviving copy Marc Vef lustre/utils/lfs.c:6289 In flight Enforce on MDS that a parity-only victim file cannot be created Marc Vef lustre/mdd/mdd_object.c:2042 Open Support merging data+parity split victim in mirror extend follow-up patch Maximilian Dilger Documentation/man1/lfs-mirror-split.1:85 Open Document --no-warn option in commit message if refreshed Gerrit AI review for Lustre COMMIT_MSG:28 Open Update man page synopsis to show -d/-f as mutually exclusive Gerrit AI review for Lustre Documentation/man1/lfs-mirror-split.1:14 Unclear File small follow-up patch for remaining issue Keguang Xu (change-level) Unclear Fix minor link-id handling issues in follow-up patch Marc Vef (change-level) Kept Fix resync-stale-mirror-before-delete-last-copy before final release +1 repeat Andreas Dilger (change-level) Not needed Document new -f/-d usage-error interaction in commit message Andreas Dilger COMMIT_MSG:28 Open Decide whether multi-mirror split should iterate instead of erroring, possibly in follow-on patch wc-checkpatch lustre/lod/lod_object.c:3759 Kept Add a -q option to quiet the parity-split warning Andreas Dilger lustre/tests/sanity-ec.sh:2203 all 13 on the change's page
	LU-19852	LU-19852 lod: raidset aware stripe allocator 63809 merged added by lefthy	0 open · 14 to check · 0 kept 9 in resolved threads	12 h ago	Promises
Not checked Fix possible ERR_PTR dereference of 'd' in lod_comp_prep_create() +13 repeats Ronnie Sahlberg lustre/lod/lod_qos.c:3581 Not checked Handle mismatch between on-disk EC layout and new raid-set derivation for existing files +1 repeat Ronnie Sahlberg include/uapi/linux/lustre/lustre_user.h:3171 Not checked Relax overly stringent stripe-count requirement and add -c -1 EC test coverage for production readiness Andreas Dilger (change-level) Not checked Track this issue via filed ticket LU-20526 +1 repeat Ronnie Sahlberg lustre/lod/lod_qos.c:2201 Not checked Add tests for cross-raidset parity reuse and forced RR allocation path Ronnie Sahlberg lustre/tests/sanity-ec.sh:3189 Not checked Check available OST count in RR case before returning error Ronnie Sahlberg lustre/lod/lod_qos.c:409 Not checked Clarify/fix minimum stripe count logic for overstriped vs widely-striped files Andreas Dilger lustre/lod/lod_qos.c:430 Not checked Renumber new tests to avoid collision with tests already added by LU-12668 series Patrick Farrell (change-level) Not checked Add exception list or self-skip in sanity.sh test_27D for insufficient OSTs Andreas Dilger (change-level) Not checked Clarify/verify whether while loop is needed given nfound may be smaller than stripe_count kg.xu lustre/lod/lod_qos.c:944 Not checked Move parameter to previous line if code is refreshed later kg.xu lustre/lod/lod_qos.c:1221 Not checked Fix incorrect comment describing raidset conversion behaviour Ronnie Sahlberg lustre/lod/lod_qos.c:1181 all 14 on the change's page

Verdicts are AI output, checked against the code they cite — verify before acting on one.