[developer] Re: [openzfs/openzfs] 9464 txg_kick() fails to see that we are quiescing, forcing transacti… (#616)

2018-04-13 Thread Rich Ercolani
@avg-I I would presume it's either https://github.com/ahrens/dtrace/blob/master/txg.xd or some later iteration thereof. -- 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#issuecommen

[developer] Re: [openzfs/openzfs] 9464 txg_kick() fails to see that we are quiescing, forcing transacti… (#616)

2018-04-13 Thread Serapheim Dimitropoulos
@avg-I here is the txg.d that I used: https://gist.github.com/sdimitro/23ed816690faa412ddf0b00ae9cd49e8 and here is another ad hoc dtrace script that I make for dynamically tracing txg_kick(): https://gist.github.com/sdimitro/4eac9894d4a109ad5774d0e3b07f20a0 -- You are receiving this because yo

[developer] Re: [openzfs/openzfs] 9466 add JSON output support to channel programs (#619)

2018-04-13 Thread Serapheim Dimitropoulos
sdimitro approved this pull request. Can you add an entry in `usr/src/pkg/manifests/system-test-zfstest.mf` for the new test that you added? Things look good to me overall since I've had a sneak peek of the ZoL review. I'd have @jwk404 take a look at the new test case though just in case. > +#

[developer] Re: [openzfs/openzfs] 9464 txg_kick() fails to see that we are quiescing, forcing transacti… (#616)

2018-04-13 Thread Matthew Ahrens
@avg-I I've updated the description to add the legend. (and yes, it is https://github.com/ahrens/dtrace/blob/master/txg.xd) -- 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#issuecom

[developer] Re: [openzfs/openzfs] 9466 add JSON output support to channel programs (#619)

2018-04-13 Thread Alek P
alek-p commented on this pull request. > +# Common Development and Distribution License ("CDDL"), version 1.0. +# You may only use this file in accordance with the terms of version +# 1.0 of the CDDL. +# +# A full copy of the text of the CDDL should have accompanied this +# source. A copy is of

[developer] Re: [openzfs/openzfs] 9466 add JSON output support to channel programs (#619)

2018-04-13 Thread Alek P
alek-p commented on this pull request. > +# You can obtain a copy of the license at usr/src/OPENSOLARIS.LICENSE +# or http://www.opensolaris.org/os/licensing. +# See the License for the specific language governing permissions +# and limitations under the License. +# +# When distributing Covered

[developer] Re: [openzfs/openzfs] 9466 add JSON output support to channel programs (#619)

2018-04-13 Thread Alek P
alek-p commented on this pull request. > +# You can obtain a copy of the license at usr/src/OPENSOLARIS.LICENSE +# or http://www.opensolaris.org/os/licensing. +# See the License for the specific language governing permissions +# and limitations under the License. +# +# When distributing Covered

[developer] [openzfs/openzfs] Update AMI to OpenIndiana Hipster 2018.04.13 (#620)

2018-04-13 Thread Prakash Surya
You can view, comment on, or merge this pull request online at: https://github.com/openzfs/openzfs/pull/620 -- Commit Summary -- * Update AMI to OpenIndiana Hipster 2018.04.13 -- File Changes -- M Jenkinsfile (2) M jenkins/pipelines/miscellaneous.groovy (2) -- Patch Links -- htt

[developer] Re: [openzfs/openzfs] 9466 add JSON output support to channel programs (#619)

2018-04-13 Thread Serapheim Dimitropoulos
sdimitro commented on this pull request. > +# Common Development and Distribution License ("CDDL"), version 1.0. +# You may only use this file in accordance with the terms of version +# 1.0 of the CDDL. +# +# A full copy of the text of the CDDL should have accompanied this +# source. A copy is

[developer] Re: [openzfs/openzfs] 9466 add JSON output support to channel programs (#619)

2018-04-13 Thread Serapheim Dimitropoulos
sdimitro commented on this pull request. > +# You can obtain a copy of the license at usr/src/OPENSOLARIS.LICENSE +# or http://www.opensolaris.org/os/licensing. +# See the License for the specific language governing permissions +# and limitations under the License. +# +# When distributing Covere

[developer] Re: [openzfs/openzfs] Update AMI to OpenIndiana Hipster 2018.04.13 (#620)

2018-04-13 Thread Prakash Surya
Closed #620 via 7db3b2a0c5c300fdcffea11bfa0e5a71926ecaa6. -- 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/620#event-1573936909 -- openzfs: openzf

[developer] Re: [openzfs/openzfs] Merge remote-tracking branch 'illumos/master' into illumos-sync (#594)

2018-04-13 Thread Prakash Surya
Closed #594. -- 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/594#event-1573939985 -- openzfs: openzfs-developer Permalink: https://openzfs.topic

[developer] Re: [openzfs/openzfs] 5882 Temporary Pool Names (#600)

2018-04-13 Thread Prakash Surya
@avg-I I believe I've fixed the CI failure. Can you rebase onto the latest master to pick up the fix and trigger a new build/test run? -- 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/6

[developer] [openzfs/openzfs] Merge remote-tracking branch 'illumos/master' into illumos-sync (#621)

2018-04-13 Thread zettabot
You can view, comment on, or merge this pull request online at: https://github.com/openzfs/openzfs/pull/621 -- Commit Summary -- * 9231 nightly should create a log/latest/ symlink * 9206 Move getsecs() prototype to stand.h from net.h so it can be used everywhere * 9106 praudit could go

[developer] Re: [openzfs/openzfs] Merge remote-tracking branch 'illumos/master' into illumos-sync (#621)

2018-04-13 Thread Prakash Surya
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/621#pullrequestreview-112170242 -- openzfs: openzfs-

[developer] Re: [openzfs/openzfs] Merge remote-tracking branch 'illumos/master' into illumos-sync (#621)

2018-04-13 Thread Prakash Surya
Merged #621. -- 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/621#event-1574214232 -- openzfs: openzfs-developer Permalink: https://openzfs.topic