Juju Charms Collection
swift-proxy package

Charms need config option to set public endpoint address

Bug #1398182 reported by Billy Olsen
24
This bug affects 4 people
Affects Status Importance Assigned to Milestone
Charm Helpers
Fix Released
Medium
Billy Olsen
ceilometer (Juju Charms Collection)
Fix Released
High
Billy Olsen
Juju Charms Collection 15.07
ceph-radosgw (Juju Charms Collection)
Fix Released
High
Billy Olsen
Juju Charms Collection 15.07
cinder (Juju Charms Collection)
Fix Released
High
Billy Olsen
Juju Charms Collection 15.07
glance (Juju Charms Collection)
Fix Released
High
Billy Olsen
Juju Charms Collection 15.07
heat (Juju Charms Collection)
Fix Released
High
Billy Olsen
Juju Charms Collection 15.07
keystone (Juju Charms Collection)
Fix Released
High
Billy Olsen
Juju Charms Collection 15.07
neutron-api (Juju Charms Collection)
Fix Released
High
Billy Olsen
Juju Charms Collection 15.07
nova-cloud-controller (Juju Charms Collection)
Fix Released
High
Billy Olsen
Juju Charms Collection 15.07
swift-proxy (Juju Charms Collection)
Fix Released
High
Billy Olsen
Juju Charms Collection 15.07

Bug Description

Charms determine the public URL based upon the public IP addresses that are available on the os-public-network. In some installations, the public IP is not available on the node itself, but is instead provided by a reverse proxy elsewhere. In these instances, an option should be made available to allow the publicURL to be specified so that the endpoints registered within keystone refer to the public URL specified.

For example, if my swift service is provided externally through https://files.example.com but the files.example.com host does not reside on the swift-proxy unit there is no way via the charm to advertise the publicURL in a publicly consumable manner.

See original description

Related branches

lp://staging/~billy-olsen/charms/trusty/ceilometer/public-endpoint-host
OpenStack Charmers: Pending requested
Diff: 924 lines (+404/-103)
12 files modified
config.yaml (+12/-0)
hooks/charmhelpers/contrib/hahelpers/cluster.py (+25/-0)
hooks/charmhelpers/contrib/openstack/ip.py (+49/-44)
hooks/charmhelpers/contrib/openstack/neutron.py (+10/-5)
hooks/charmhelpers/contrib/openstack/utils.py (+65/-18)
hooks/charmhelpers/contrib/python/packages.py (+28/-5)
hooks/charmhelpers/core/hookenv.py (+147/-10)
hooks/charmhelpers/core/host.py (+1/-1)
hooks/charmhelpers/core/services/base.py (+32/-11)
hooks/charmhelpers/fetch/__init__.py (+1/-1)
hooks/charmhelpers/fetch/giturl.py (+7/-5)
unit_tests/test_ceilometer_hooks.py (+27/-3)
lp://staging/~billy-olsen/charms/trusty/ceph-radosgw/public-endpoint-host
Corey Bryant: Approve
Diff: 1126 lines (+460/-135)
16 files modified
config.yaml (+12/-0)
hooks/charmhelpers/contrib/hahelpers/cluster.py (+25/-0)
hooks/charmhelpers/contrib/openstack/amulet/deployment.py (+15/-6)
hooks/charmhelpers/contrib/openstack/ip.py (+49/-44)
hooks/charmhelpers/contrib/openstack/neutron.py (+10/-5)
hooks/charmhelpers/contrib/openstack/utils.py (+65/-18)
hooks/charmhelpers/contrib/python/packages.py (+28/-5)
hooks/charmhelpers/core/hookenv.py (+147/-10)
hooks/charmhelpers/core/host.py (+1/-1)
hooks/charmhelpers/core/services/base.py (+32/-11)
hooks/charmhelpers/fetch/__init__.py (+1/-1)
hooks/charmhelpers/fetch/giturl.py (+7/-5)
hooks/hooks.py (+4/-15)
tests/charmhelpers/contrib/amulet/utils.py (+8/-1)
tests/charmhelpers/contrib/openstack/amulet/deployment.py (+15/-6)
unit_tests/test_hooks.py (+41/-7)
lp://staging/~billy-olsen/charm-helpers/better-address-endpoint-overrides
Jorge Niedbalski (community): Approve
Felipe Reyes: Pending requested
charmers: Pending requested
Edward Hope-Morley: Pending requested
Diff: 224 lines (+75/-73)
2 files modified
charmhelpers/contrib/openstack/ip.py (+49/-44)
tests/contrib/openstack/test_ip.py (+26/-29)
lp://staging/~billy-olsen/charms/trusty/swift-proxy/public-endpoint-host
Corey Bryant: Approve
Diff: 291 lines (+155/-48)
3 files modified
config.yaml (+12/-0)
hooks/charmhelpers/contrib/openstack/ip.py (+49/-44)
unit_tests/test_swift_hooks.py (+94/-4)
lp://staging/~billy-olsen/charms/trusty/nova-cloud-controller/public-endpoint-host
Corey Bryant: Approve
Diff: 1162 lines (+440/-125)
14 files modified
config.yaml (+11/-0)
hooks/charmhelpers/contrib/hahelpers/cluster.py (+25/-0)
hooks/charmhelpers/contrib/openstack/ip.py (+49/-44)
hooks/charmhelpers/contrib/openstack/neutron.py (+10/-5)
hooks/charmhelpers/contrib/openstack/utils.py (+65/-18)
hooks/charmhelpers/contrib/peerstorage/__init__.py (+2/-0)
hooks/charmhelpers/contrib/python/packages.py (+28/-5)
hooks/charmhelpers/core/hookenv.py (+147/-10)
hooks/charmhelpers/core/host.py (+1/-1)
hooks/charmhelpers/core/services/base.py (+32/-11)
hooks/charmhelpers/fetch/__init__.py (+1/-1)
hooks/charmhelpers/fetch/giturl.py (+7/-5)
unit_tests/test_nova_cc_contexts.py (+8/-4)
unit_tests/test_nova_cc_hooks.py (+54/-21)
lp://staging/~billy-olsen/charms/trusty/neutron-api/public-endpoint-host
Corey Bryant: Approve
Diff: 259 lines (+98/-51)
3 files modified
config.yaml (+12/-0)
hooks/charmhelpers/contrib/openstack/ip.py (+49/-44)
unit_tests/test_neutron_api_hooks.py (+37/-7)
lp://staging/~billy-olsen/charms/trusty/keystone/public-endpoint-host
Corey Bryant: Approve
Diff: 1011 lines (+420/-110)
14 files modified
config.yaml (+12/-0)
hooks/charmhelpers/contrib/hahelpers/cluster.py (+25/-0)
hooks/charmhelpers/contrib/openstack/ip.py (+49/-44)
hooks/charmhelpers/contrib/openstack/neutron.py (+10/-5)
hooks/charmhelpers/contrib/openstack/utils.py (+65/-18)
hooks/charmhelpers/contrib/peerstorage/__init__.py (+2/-0)
hooks/charmhelpers/contrib/python/packages.py (+28/-5)
hooks/charmhelpers/contrib/unison/__init__.py (+5/-4)
hooks/charmhelpers/core/hookenv.py (+147/-10)
hooks/charmhelpers/core/host.py (+1/-1)
hooks/charmhelpers/core/services/base.py (+32/-11)
hooks/charmhelpers/fetch/__init__.py (+1/-1)
hooks/charmhelpers/fetch/giturl.py (+7/-5)
unit_tests/test_keystone_utils.py (+36/-6)
lp://staging/~billy-olsen/charms/trusty/glance/public-endpoint-host
Corey Bryant: Approve
Diff: 761 lines (+321/-87)
9 files modified
config.yaml (+10/-0)
hooks/charmhelpers/contrib/hahelpers/cluster.py (+25/-0)
hooks/charmhelpers/contrib/openstack/ip.py (+49/-44)
hooks/charmhelpers/contrib/openstack/neutron.py (+10/-5)
hooks/charmhelpers/core/hookenv.py (+147/-10)
hooks/charmhelpers/core/host.py (+1/-1)
hooks/charmhelpers/core/services/base.py (+32/-11)
hooks/charmhelpers/fetch/__init__.py (+1/-1)
unit_tests/test_glance_relations.py (+46/-15)
lp://staging/~billy-olsen/charms/trusty/cinder/public-endpoint-host
Corey Bryant: Approve
Diff: 270 lines (+107/-50)
3 files modified
config.yaml (+13/-0)
hooks/charmhelpers/contrib/openstack/ip.py (+49/-44)
unit_tests/test_cinder_hooks.py (+45/-6)
lp://staging/~openstack-charmers-archive/charms/trusty/glance/next
lp://staging/~openstack-charmers-archive/charms/trusty/nova-cloud-controller/next
lp://staging/~openstack-charmers-archive/charms/trusty/neutron-api/next
lp://staging/~openstack-charmers-archive/charms/trusty/ceph-radosgw/next
lp://staging/~openstack-charmers-archive/charms/trusty/keystone/next
lp://staging/~openstack-charmers-archive/charms/trusty/cinder/next
lp://staging/~openstack-charmers-archive/charms/trusty/ceilometer/next
lp://staging/~openstack-charmers-archive/charms/trusty/heat/next
lp://staging/~openstack-charmers-archive/charms/trusty/swift-proxy/next
Billy Olsen (billy-olsen)
tags: added: cts openstack
Billy Olsen (billy-olsen)
tags: added: canonical-bootstack
Revision history for this message
Peter Sabaini (peter-sabaini) wrote :

One additional point re: motivation -- for a public service, you'd also want the SSL certs be issued by a ext. CA. Issuing certs for IP addresses is cumbersome and hard to maintain; ususally we try to stick to our wildcard *.c.c cert for economy

Billy Olsen (billy-olsen)
Changed in swift-proxy (Juju Charms Collection):
assignee: nobody → Billy Olsen (billy-olsen)
status: New → In Progress
Revision history for this message
Ante Karamatić (ivoks) wrote :

Allowing publicURL to be configured is something that would be benefitial for all charms that create endpoints. Most of the time deployment only has private IPs, but people do NAT/proxy to those IPs. Problem is then with keystone who serves private IPs as endpoints.

Ante Karamatić (ivoks)
tags: added: cpec
Changed in swift-proxy (Juju Charms Collection):
importance: Undecided → High
no longer affects: nova-cloud-controller (Ubuntu)
Ante Karamatić (ivoks)
summary: - Cannot set publicURL for public access
+ Charms need to config option for publicURL
summary: - Charms need to config option for publicURL
+ Charms need config option to set publicURL
JuanJo Ciarlante (jjo)
Changed in openstack-dashboard (Juju Charms Collection):
status: New → Invalid
no longer affects: openstack-dashboard (Juju Charms Collection)
Revision history for this message
JuanJo Ciarlante (jjo) wrote : Re: Charms need config option to set publicURL

+1 we need this added to every relevant OS charm with a service
(in keystone catalog).

As a suggestion, please consider adding some minimal auto-templating, like
#1 public_url: https://{{service_name}}.os.example.com:{{service_port}}/{{service_path}}
or
#2 public_url: https://{{service_type}}.os.example.com:{{service_port}}/{{service_path}}

which would expand to e.g.
#1:
- https://keystone.os.example.com:5000/v2.0
- https://swift-proxy.os.example.com:8080/v1/AUTH_xxx
- https://nova-cloud-controller.os.example.com:8774/v2/yyy
#2:
- https://identity.os.example.com:5000/v2.0
- https://object-store.os.example.com:8080/v1/AUTH_xxx
- https://compute.os.example.com:8774/v2/yyy

FYI above would allow us to set a single 'override:' line at juju-deploy config.

Ante Karamatić (ivoks)
Changed in ceilometer (Juju Charms Collection):
importance: Undecided → High
Changed in ceph-radosgw (Juju Charms Collection):
importance: Undecided → High
Changed in cinder (Juju Charms Collection):
importance: Undecided → High
Changed in glance (Juju Charms Collection):
importance: Undecided → High
Changed in heat (Juju Charms Collection):
importance: Undecided → High
Changed in keystone (Juju Charms Collection):
importance: Undecided → High
Changed in neutron-api (Juju Charms Collection):
importance: Undecided → High
Changed in nova-cloud-controller (Juju Charms Collection):
importance: Undecided → High
description: updated
Billy Olsen (billy-olsen)
Changed in ceilometer (Juju Charms Collection):
assignee: nobody → Billy Olsen (billy-olsen)
Changed in ceph-radosgw (Juju Charms Collection):
assignee: nobody → Billy Olsen (billy-olsen)
Changed in cinder (Juju Charms Collection):
assignee: nobody → Billy Olsen (billy-olsen)
Changed in glance (Juju Charms Collection):
assignee: nobody → Billy Olsen (billy-olsen)
Changed in heat (Juju Charms Collection):
assignee: nobody → Billy Olsen (billy-olsen)
Changed in keystone (Juju Charms Collection):
assignee: nobody → Billy Olsen (billy-olsen)
Changed in neutron-api (Juju Charms Collection):
assignee: nobody → Billy Olsen (billy-olsen)
Changed in nova-cloud-controller (Juju Charms Collection):
assignee: nobody → Billy Olsen (billy-olsen)
Changed in ceilometer (Juju Charms Collection):
milestone: none → 15.07
Changed in ceph-radosgw (Juju Charms Collection):
milestone: none → 15.07
Changed in cinder (Juju Charms Collection):
milestone: none → 15.07
Changed in glance (Juju Charms Collection):
milestone: none → 15.07
Changed in heat (Juju Charms Collection):
milestone: none → 15.07
Changed in keystone (Juju Charms Collection):
milestone: none → 15.07
Changed in neutron-api (Juju Charms Collection):
milestone: none → 15.07
Changed in nova-cloud-controller (Juju Charms Collection):
milestone: none → 15.07
Changed in swift-proxy (Juju Charms Collection):
milestone: none → 15.07
Changed in ceilometer (Juju Charms Collection):
status: New → In Progress
Changed in ceph-radosgw (Juju Charms Collection):
status: New → In Progress
Changed in cinder (Juju Charms Collection):
status: New → In Progress
Changed in glance (Juju Charms Collection):
status: New → In Progress
Changed in keystone (Juju Charms Collection):
status: New → In Progress
Changed in neutron-api (Juju Charms Collection):
status: New → In Progress
Changed in nova-cloud-controller (Juju Charms Collection):
status: New → In Progress
Changed in heat (Juju Charms Collection):
status: New → In Progress
Revision history for this message
Billy Olsen (billy-olsen) wrote :

I've provided a merge proposal for various charms which provides an os-public-hostname option allowing for a user to specify what the base hostname portion of the endpoint URLs is. It doesn't allow for a complete re-write of the API endpoint, but it does allow for the primary use cases to be addressed; NAT'ing the private IP and wildcard certificates.

tags: added: backport-potential
Billy Olsen (billy-olsen)
summary: - Charms need config option to set publicURL
+ Charms need config option to set public endpoint address
Jorge Niedbalski (niedbalski)
Changed in charm-helpers:
assignee: nobody → Billy Olsen (billy-olsen)
importance: Undecided → Medium
status: New → Fix Released
Revision history for this message
Corey Bryant (corey.bryant) wrote :

Hi Billy, Do you have an mp for heat?

Billy Olsen (billy-olsen)
Changed in ceilometer (Juju Charms Collection):
status: In Progress → Fix Committed
Changed in ceph-radosgw (Juju Charms Collection):
status: In Progress → Fix Committed
Changed in cinder (Juju Charms Collection):
status: In Progress → Fix Committed
Changed in glance (Juju Charms Collection):
status: In Progress → Fix Committed
Changed in heat (Juju Charms Collection):
status: In Progress → Fix Committed
Changed in keystone (Juju Charms Collection):
status: In Progress → Fix Committed
Changed in neutron-api (Juju Charms Collection):
status: In Progress → Fix Committed
Changed in nova-cloud-controller (Juju Charms Collection):
status: In Progress → Fix Committed
Changed in swift-proxy (Juju Charms Collection):
status: In Progress → Fix Committed
Billy Olsen (billy-olsen)
Changed in ceilometer (Juju Charms Collection):
status: Fix Committed → Fix Released
Changed in ceph-radosgw (Juju Charms Collection):
status: Fix Committed → Fix Released
Changed in cinder (Juju Charms Collection):
status: Fix Committed → Fix Released
Changed in glance (Juju Charms Collection):
status: Fix Committed → Fix Released
Changed in heat (Juju Charms Collection):
status: Fix Committed → Fix Released
Changed in keystone (Juju Charms Collection):
status: Fix Committed → Fix Released
Changed in neutron-api (Juju Charms Collection):
status: Fix Committed → Fix Released
Changed in nova-cloud-controller (Juju Charms Collection):
status: Fix Committed → Fix Released
Changed in swift-proxy (Juju Charms Collection):
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.