search

lae / ansible-role-proxmox

IaC for Proxmox VE clusters.
MIT License
516 stars 146 forks source link

Better documentation for Ceph #264

Open willifehler opened 2 months ago

willifehler commented 2 months ago

Hey there,

could we please add better examples how to specify a list of hosts for?

pve_ceph_mon_group: "{{ pve_group }}" # Host group containing all Ceph monitor hosts
pve_ceph_mgr_group: "{{ pve_ceph_mon_group }}" # Host group containing all Ceph manager hosts
pve_ceph_mds_group: "{{ pve_group }}" # Host group containing all Ceph metadata server hosts

Let's say I only want to have 5 monitors, 5 mgrs and 5 mds but my pve group has 25 hosts.

Cheers - Willi

lae commented 2 months ago

not to sound like a broken record, but pull requests welcome :slightly_smiling_face:

the excerpt you've provided are just what the defaults are set to, and does not seem confusing to me so you probably have a better idea than I do on what to add to the documentation. they default to pve_group as a safeguard, but otherwise have no reason to be related to pve_group if you specify other group names.

lae commented 2 months ago

Also this request might overlap with/be a duplicate of #68, given the title.