{"action":"created","issue":{"url":"https://api.github.com/repos/zfsonlinux/zfs/issues/3526","labels_url":"https://api.github.com/repos/zfsonlinux/zfs/issues/3526/labels{/name}","comments_url":"https://api.github.com/repos/zfsonlinux/zfs/issues/3526/comments","events_url":"https://api.github.com/repos/zfsonlinux/zfs/issues/3526/events","html_url":"https://github.com/zfsonlinux/zfs/pull/3526","id":90967334,"number":3526,"title":"Change default cachefile property to 'none'.","user":{"login":"FransUrbo","id":572706,"avatar_url":"https://avatars.githubusercontent.com/u/572706?v=3","gravatar_id":"","url":"https://api.github.com/users/FransUrbo","html_url":"https://github.com/FransUrbo","followers_url":"https://api.github.com/users/FransUrbo/followers","following_url":"https://api.github.com/users/FransUrbo/following{/other_user}","gists_url":"https://api.github.com/users/FransUrbo/gists{/gist_id}","starred_url":"https://api.github.com/users/FransUrbo/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/FransUrbo/subscriptions","organizations_url":"https://api.github.com/users/FransUrbo/orgs","repos_url":"https://api.github.com/users/FransUrbo/repos","events_url":"https://api.github.com/users/FransUrbo/events{/privacy}","received_events_url":"https://api.github.com/users/FransUrbo/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-06-25T13:36:27Z","updated_at":"2015-06-25T19:42:55Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/zfsonlinux/zfs/pulls/3526","html_url":"https://github.com/zfsonlinux/zfs/pull/3526","diff_url":"https://github.com/zfsonlinux/zfs/pull/3526.diff","patch_url":"https://github.com/zfsonlinux/zfs/pull/3526.patch"},"body":"This in preparation to a later complete removal of cache file.\r\n\r\nA cache file is almost never needed, at least not on Linux, so the long-time plan is to remove it altogether. This is the first step.\r\n\r\nI noticed that this affects the <code>import</code> as well as <code>create</code> (which was not my intention - I only wanted it for the <code>create</code> command). That is, if I create a pool with the old code, using defaults (<code>-</code>), export it and then import it with the new code, the <code>cachefile</code> property will be <code>none</code>.\r\n\r\nThat _might_ not be what we want, although I'm for it..."},"comment":{"url":"https://api.github.com/repos/zfsonlinux/zfs/issues/comments/115373473","html_url":"https://github.com/zfsonlinux/zfs/pull/3526#issuecomment-115373473","issue_url":"https://api.github.com/repos/zfsonlinux/zfs/issues/3526","id":115373473,"user":{"login":"FransUrbo","id":572706,"avatar_url":"https://avatars.githubusercontent.com/u/572706?v=3","gravatar_id":"","url":"https://api.github.com/users/FransUrbo","html_url":"https://github.com/FransUrbo","followers_url":"https://api.github.com/users/FransUrbo/followers","following_url":"https://api.github.com/users/FransUrbo/following{/other_user}","gists_url":"https://api.github.com/users/FransUrbo/gists{/gist_id}","starred_url":"https://api.github.com/users/FransUrbo/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/FransUrbo/subscriptions","organizations_url":"https://api.github.com/users/FransUrbo/orgs","repos_url":"https://api.github.com/users/FransUrbo/repos","events_url":"https://api.github.com/users/FransUrbo/events{/privacy}","received_events_url":"https://api.github.com/users/FransUrbo/received_events","type":"User","site_admin":false},"created_at":"2015-06-25T19:42:55Z","updated_at":"2015-06-25T19:42:55Z","body":"> It might be a really good idea to change this default as your suggesting before the tag.\n> \n> This is the kind of change which would be nice to make sooner rather than latter.\n> \nI was kind'a thinking that it might be better to accept it as one of the very first commits after 0.6.5, to get it as much testing as possible and not 'risk' \"normal\" users system.\n\nI can't really see any big problems, but you never know. For those that have huge pools with a lot of devices (which I gather is a lot fewer than those with 'a bunch' of vdevs), it is reasonably easy to \"fix\" the long import (if they see any) they (might!) get the first time with this, to set <code>cachefile=/etc/zfs/zpool.cache</code>, instead of NULL/'-' (which is the current default - 'use default path') or 'none' (this new default).\n\n\nBecause of this, I have no problem what so ever to add it before the tag but if someone sees (or can think of) ANY problem with this, maybe we should wait. Next tag isn't that far of...\n> I'd expect this to resolve the various initramfs issues regarding having an up to date cache file.\n> \nIndeed!! We've talked about this for years, but no-one have had time or interest to deal with it (it being an extremely low-prio thing). I was kind'a bored today, and I figured it would only be a few lines fixes. Turned out to be a couple more than I expected, but it's still small enough :)\n\nOnce we've run with this for a couple of months or so, we can discuss what the next step would be, but maybe this is enough? Maybe we should leave the cachefile property [set to default 'none'] from this PR and not go any further, for those very few instances where IT IS needed (or at least better - as with hundreds/thousands of vdevs)?"}}