I didn't inspect the change originally, but I have looked at it now and all
parts of it look good to me.
I also agree with the analysis.
Thanks!
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
@avg-I Thank you!
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/openzfs/openzfs/pull/616#issuecomment-388874355
--
openzfs: openzfs-developer
Permalink:
avg-I approved this pull request.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/openzfs/openzfs/pull/616#pullrequestreview-119876549
--
openzfs:
prakashsurya approved this pull request.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/openzfs/openzfs/pull/639#pullrequestreview-119863248
--
openzfs:
jwk404 commented on this pull request.
> @@ -15,6 +15,6 @@
.PARALLEL: $(SUBDIRS)
-SUBDIRS:sh = find ./* -maxdepth 0 -type d
+SUBDIRS:sh = ls */Makefile 2>/dev/null | sed 's/\/Makefile//g'
This is part of a Delphix fix that solved 7710 a little differently. It
shouldn't be upstreamed.
>
ahrens commented on this pull request.
> + } else if (rs->rs_end - range_tree_min(segs) >
+ *max_alloc) {
+ /*
+* This additional segment would extend past
+
Closed #554 via fd2f0a6c091309a6113dc26c7b11c0399b12e806.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/openzfs/openzfs/pull/554#event-1625514017
--
openzfs:
ikozhukhov commented on this pull request.
> @@ -176,6 +176,8 @@ export KEEP="^$(echo $KEEP | sed 's/ /$|^/g')\$"
num_disks=$(echo $DISKS | awk '{print NF}')
[[ $num_disks -lt 3 ]] && fail "Not enough disks to run ZFS Test Suite"
+sudo -k coreadm -e process
i think will be better to save
@jwk404 could you take a look at @yuripv's comments?
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/openzfs/openzfs/pull/571#issuecomment-388901334
--
openzfs:
yuripv commented on this pull request.
> @@ -176,6 +176,8 @@ export KEEP="^$(echo $KEEP | sed 's/ /$|^/g')\$"
num_disks=$(echo $DISKS | awk '{print NF}')
[[ $num_disks -lt 3 ]] && fail "Not enough disks to run ZFS Test Suite"
+sudo -k coreadm -e process
Or that, I just don't like
@lundman pushed 1 commit.
1fa5148 Add support for decryption faults in zinject
--
You are receiving this because you are subscribed to this thread.
View it on GitHub:
11 matches
Mail list logo