Merge lp:~seyeongkim/charms/trusty/ceph-radosgw/lp1353246 into lp:~openstack-charmers-archive/charms/trusty/ceph-radosgw/next

[Seyeong Kim]

[Edward Hope-Morley (hopem)]

Hmm, fairly sure this used to be fine. Will this not break backwards compatibility with older versions of Ceph? Also, the ceph charm is expecting this to be client.radosgw.gateway. I guess we have two choices here, either conditionally change files/www/s3gw.fcgi based on what version of ceph is in use or conditionally set this value.

[Edward Hope-Morley (hopem)]

Just deployed havana+ceph 0.67 and it works just fine. Was there anything in particular you had to do to hit the issue you are getting?

review: Needs Information

[Edward Hope-Morley (hopem)]

Ok so, I have looked at this some more. When you are running /var/www/s3gw.fcgi by hand you are getting an error because the client name used in that script is inconsistent with the one configured in /etc/ceph/ceph.conf. When apache runs that script it does:

FastCgiExternalServer /var/www/s3gw.fcgi -socket /tmp/radosgw.sock

So it is providing a socket (incidentally the same as the one in /etc/ceph/ceph.conf). So, i think this is a real bug but it is not likely the soure of your original problem since because apache provide a --socket the command does not error out.

I think we should still fix /var/www/s3gw.fcgi to use the correct ceph client (client.radosgw.gateway) as opposed to your patch which modifes ceph.conf.

review: Needs Fixing

[lp:~seyeongkim/charms/trusty/ceph-radosgw/lp1353246]

26. By Seyeong Kim on 2014-08-07

s3gw.fcgi should have same configuration name in ceph.conf

=== modified file 'files/www/s3gw.fcgi'
--- files/www/s3gw.fcgi	2012-10-08 15:58:16 +0000
+++ files/www/s3gw.fcgi	2014-08-07 10:10:15 +0000
@@ -1,2 +1,2 @@
 #!/bin/sh
-exec /usr/bin/radosgw -c /etc/ceph/ceph.conf -n client.rados.gateway
\ No newline at end of file
+exec /usr/bin/radosgw -c /etc/ceph/ceph.conf -n client.radosgw.gateway

=== modified file 'templates/ceph.conf'
--- templates/ceph.conf	2014-07-24 09:48:17 +0000
+++ templates/ceph.conf	2014-08-07 10:10:15 +0000
@@ -26,4 +26,4 @@
  rgw keystone token cache size = {{ cache_size }}
  rgw keystone revocation interval = {{ revocation_check_interval }}
 #nss db path = /var/lib/ceph/nss
-{% endif %}
\ No newline at end of file
+{% endif %}