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"
@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
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
--
@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
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 --
@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:
@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:
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
@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
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
--
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
--
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:
12 matches
Mail list logo