Thanks @yuripv, no worries! Sorry for the noise.
--
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-389658953
--
openzfs:
I'm sorry this didn't work out, but it really looks like a
distribution-specific configuration issue.
--
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-389646880
Based on the discussion between @yuripv @ikozhukhov and @jwk404, I'm going to
close this out. It sounds like it's not worth pushing this change outside of
Delphix.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
Closed #571.
--
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#event-1630204035
--
openzfs: openzfs-developer
Permalink:
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.
>
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
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
This effectively changes coreadm