[developer] [openzfs/openzfs] Build failures may "leak" the EC2 instance (#443)

2017-08-07 Thread Prakash Surya
Since integrating the following change: commit ffa59080374b94e49ea6df62df0d8753cf5dbad8 Author: Prakash Surya Date: Fri Jun 23 16:41:41 2017 -0700 Use EC2 "Spot Instances" with Jenkins Automation It's possible for an EC2 instance to be "leaked"

[developer] Re: [openzfs/openzfs] 8567 Inconsistent return value in zpool_read_label (#442)

2017-08-07 Thread Alan Somers
@andy-js the more common convention in libzfs is to return -1 on error. There are about five times as many `return (-1)` statements as `return (E*)`. Plus, I'm not sure what the correct errno would be for the case where all system calls succeed, but the label is not intelligible. -- You are

[developer] Re: [openzfs/openzfs] 8567 Inconsistent return value in zpool_read_label (#442)

2017-08-07 Thread Andrew Stormont
Perhaps zpool_read_label should just return an errno? -- 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/442#issuecomment-320784745 --

[developer] Re: [openzfs/openzfs] Native data and metadata encryption for zfs (#124)

2017-08-07 Thread Attila Fülöp
@ahrens I tried an earlier version of this patch, which crashed in debug builds. The tests ran fine on a non-debug build though. See https://github.com/openzfs/openzfs/pull/124#issuecomment-288516088 ff and https://github.com/openzfs/openzfs/pull/124#issuecomment-285545199 ff I'm planing to

[developer] [openzfs/openzfs] 8567 Inconsistent return value in zpool_read_label (#442)

2017-08-07 Thread Alan Somers
Make zpool_read_label's return value consistent, and adjust its consumers accordingly. You can view, comment on, or merge this pull request online at: https://github.com/openzfs/openzfs/pull/442 -- Commit Summary -- * 8567 Inconsistent return value in zpool_read_label -- File Changes --

[developer] Re: [openzfs/openzfs] Limit the uptime of Jenkins slaves to 12 hours (#441)

2017-08-07 Thread Prakash Surya
@prakashsurya pushed 1 commit. 27d63fa Revert "testing: modify slave uptime limit to 1 hour" -- You are receiving this because you are subscribed to this thread. View it on GitHub:

[developer] Re: [openzfs/openzfs] Limit the uptime of Jenkins slaves to 12 hours (#441)

2017-08-07 Thread Prakash Surya
@prakashsurya pushed 1 commit. a9f19cc testing: modify slave uptime limit to 1 hour -- You are receiving this because you are subscribed to this thread. View it on GitHub:

[developer] [openzfs/openzfs] Limit the uptime of Jenkins slaves to 12 hours (#441)

2017-08-07 Thread Prakash Surya
To try and prevent EC2 instance "leaks", this patch adds an additional daemon that is managed by supervisord. The purpose of this new daemon is to shutdown the system after it has been running for a specified amount of time. As a result of the shutdown, the instance should be automatically

[developer] Re: [openzfs/openzfs] Native data and metadata encryption for zfs (#124)

2017-08-07 Thread Matthew Ahrens
@ilovezfs I'd like to see the encryption changes merged to ZoL first, since that's where they originated and they've gotten the most testing and code review on Linux. https://github.com/zfsonlinux/zfs/pull/5769 We also need to figure out the failing tests (which might or might not be the

[developer] Re: [openzfs/openzfs] 8552 ZFS LUA code uses floating point math (#437)

2017-08-07 Thread Prakash Surya
landed: https://github.com/openzfs/openzfs/commit/916c8d8811 -- 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/437#issuecomment-320653793 --

[developer] Re: [openzfs/openzfs] 8521 nvlist memory leak in get_clones_stat() and spa_load_best() (#430)

2017-08-07 Thread Prakash Surya
landed: https://github.com/openzfs/openzfs/commit/7d3000f774 -- 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/430#issuecomment-320653620 --

[developer] Re: [openzfs/openzfs] 8521 nvlist memory leak in get_clones_stat() and spa_load_best() (#430)

2017-08-07 Thread Prakash Surya
Closed #430. -- 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/430#event-1195333086 -- openzfs-developer Archives: