Skip to content
This repository has been archived by the owner on Nov 30, 2021. It is now read-only.

fix(store): change template to return ip address instead of :6789 #3582

Merged
merged 1 commit into from Apr 28, 2015

Conversation

aledbf
Copy link
Contributor

@aledbf aledbf commented Apr 25, 2015

The current template generates mon addr = :6789 instead of a list of ip:port of each ceph node.

@bacongobbler
Copy link
Member

If this still works when there's no mons available then LGTM, though AFAIK gets throws an error when nothing exists

@aledbf
Copy link
Contributor Author

aledbf commented Apr 25, 2015

@bacongobbler good point, but getvs already is used in the line 4 so I think that the error never occurs

@carmstrong
Copy link
Contributor

The current template generates mon addr = :6789 instead of a list of ip:port of each ceph node.

Sounds like the current template is quite broken, then, since the confd template rewrite. I wonder how things are working at all? Enumerating the monitors here enables the clients to connect to all of them, rather that just the one enumerated in initial members. That means that currently, if the single initial member monitor goes down or permanently leaves the cluster, no clients can connect...

@mboersma mboersma added this to the v1.6 milestone Apr 26, 2015
@carmstrong
Copy link
Contributor

LGTM. I manually verified. Before:

root@deis-01:/# cat /etc/ceph/ceph.conf
[global]
fsid = 851c7744-8b42-4c1b-8962-3c7b24078e3f
mon initial members = deis-03
mon host = deis-01,deis-03
mon addr = :6789
auth cluster required = cephx
auth service required = cephx
auth client required = cephx
osd pool default size = 3
osd pool default min_size = 1
osd pool default pg_num = 128
osd pool default pgp_num = 128
osd recovery delay start = 15
log file = /dev/stdout
rgw_frontends = "civetweb port=8888"
rgw_thread_pool_size = 512

[client.radosgw.gateway]
host = deis-store-gateway
keyring = /etc/ceph/ceph.client.radosgw.keyring
rgw socket path = /var/run/ceph/ceph.radosgw.gateway.fastcgi.sock

After:

core@deis-01 ~ $ nse deis-store-monitor
root@deis-01:/# cat /etc/ceph/ceph.conf
[global]
fsid = 851c7744-8b42-4c1b-8962-3c7b24078e3f
mon initial members = deis-03
mon host = deis-01,deis-02,deis-03
mon addr = 172.17.8.100:6789,172.17.8.101:6789,172.17.8.102:6789
auth cluster required = cephx
auth service required = cephx
auth client required = cephx
osd pool default size = 3
osd pool default min_size = 1
osd pool default pg_num = 128
osd pool default pgp_num = 128
osd recovery delay start = 15
log file = /dev/stdout
rgw_frontends = "civetweb port=8888"
rgw_thread_pool_size = 512

[client.radosgw.gateway]
host = deis-store-gateway
keyring = /etc/ceph/ceph.client.radosgw.keyring
rgw socket path = /var/run/ceph/ceph.radosgw.gateway.fastcgi.sock

This is good to merge once we get another Code LGTM.

@mboersma
Copy link
Member

Code LGTM.

aledbf added a commit that referenced this pull request Apr 28, 2015
fix(store): change template to return ip address instead of :6789
@aledbf aledbf merged commit 2f4b965 into deis:master Apr 28, 2015
@mboersma
Copy link
Member

Code LGTM.

@aledbf aledbf deleted the store-ceph-conf-template branch April 29, 2015 05:55
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants