Skip to content
This repository has been archived by the owner on May 17, 2018. It is now read-only.

Enumerated disk and pool type selection, existing zpool awareness. #22

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ccasey
Copy link

@ccasey ccasey commented Mar 28, 2013

  • diskinfo based enumerated disk selection
  • enumerated selection of common zpool types based on number of
    available disks, including spares
  • aware of existing zpools, will overwrite disks in zpools after warning
  • set dhcp to default network type

- enumerated selection of common zpool types based on number of
  available disks, including spares
- aware of existing zpools, will overwrite disks in zpools after warning
- set dhcp to default network type
@rmustacc
Copy link
Contributor

Thanks for taking a stab at this. One thing that'd be very useful is to actually leverage the disklayout command which can be combined with mkzpool to set the default pool configuration. I'm a bit hesitant to start listing every pool configuration possible as that list can grow pretty large, having to take cache devices, log devices, etc. into account, wanting multiple sets of these, etc. While having some basic set of these defaults (if disklayout's suggestion isn't wanted) may make sense. It would probably be most useful to just provide a way to shell out and manually create the a zpool, resume, verify the zpool exists and create the other necessary datasets.

@ccasey
Copy link
Author

ccasey commented Mar 28, 2013

I had the same thought about the shelling out. I can look into that.

The configs I do provide should be able to be added to after the fact with cache/log devices so they dont preclude them, just don't go all the way from the beginning. I'm not sure the usage of these "extra" zpool components, whether what I've done here handles the vast majority of need or not with the more advanced users already used to the simple installer to do their bidding to that extent, yet. Figured this would give a lot of the people walking in off the street at least a tad more friendly experience for now.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants