charm needs to implement required_relations (block until control plane is ready)

Bug #1638772 reported by Ryan Beisner
18
This bug affects 2 people
Affects Status Importance Assigned to Milestone
OpenStack Nova Compute Proxy Charm
Won't Fix
Medium
Unassigned

Bug Description

Per policies and best practices, the charm should deploy to a non-error state (blocked is ok). However, when deployed without config, or before the remote compute host is ready, the install proxy charm hook fails.

We should check that the remote host is up before firing the proxy installation, and block until it is reachable.

We should also implement require_relations and block until that is satisfied.

Logs and artifacts:

https://openstack-ci-reports.ubuntu.com/artifacts/test_charm_pipeline/openstack/charm-nova-compute-proxy/393024/2/1119/index.html

Tags: uosci
Ryan Beisner (1chb1n)
description: updated
description: updated
Revision history for this message
Ryan Beisner (1chb1n) wrote :

This issue affects usability of the charm in production and testing as well. If a bundle is deployed, the nova-compute-proxy proxy_install routines will most likely try to execute long before glance and other relations are ready.

summary: - charm single fails
+ charm single fails (need to use blocked status, wait for config and
+ relations)
Revision history for this message
Ryan Beisner (1chb1n) wrote : Re: charm single fails (need to use blocked status, wait for config and relations)

Specifically, instantiation of a REMOTEProxy object should not raise an exception when configs are missing.

unit-nova-compute-proxy-0[2569]: 2016-11-03 16:33:24 INFO unit.nova-compute-proxy/0.juju-log server.go:268 Registered config file: /var/lib/charm/nova-compute-proxy/etc/nova/nova.conf
unit-nova-compute-proxy-0[2569]: 2016-11-03 16:33:24 INFO unit.nova-compute-proxy/0.config-changed logger.go:40 Traceback (most recent call last):
unit-nova-compute-proxy-0[2569]: 2016-11-03 16:33:24 INFO unit.nova-compute-proxy/0.config-changed logger.go:40 File "/var/lib/juju/agents/unit-nova-compute-proxy-0/charm/hooks/config-changed", line 48, in <module>
unit-nova-compute-proxy-0[2569]: 2016-11-03 16:33:24 INFO unit.nova-compute-proxy/0.config-changed logger.go:40 password=config('remote-password'))
unit-nova-compute-proxy-0[2569]: 2016-11-03 16:33:24 INFO unit.nova-compute-proxy/0.config-changed logger.go:40 File "/var/lib/juju/agents/unit-nova-compute-proxy-0/charm/hooks/nova_compute_proxy.py", line 75, in __init__
unit-nova-compute-proxy-0[2569]: 2016-11-03 16:33:24 INFO unit.nova-compute-proxy/0.config-changed logger.go:40 raise Exception('Missing configuration')
unit-nova-compute-proxy-0[2569]: 2016-11-03 16:33:24 INFO unit.nova-compute-proxy/0.config-changed logger.go:40 Exception: Missing configuration
unit-nova-compute-proxy-0[2569]: 2016-11-03 16:33:24 ERROR juju.worker.uniter.operation runhook.go:107 hook "config-changed" failed: exit status 1

Vance Morris (vmorris)
Changed in charm-nova-compute-proxy:
status: New → Confirmed
Ryan Beisner (1chb1n)
description: updated
description: updated
summary: - charm single fails (need to use blocked status, wait for config and
- relations)
+ charm needs to implement required_relations (block until control plane
+ is ready)
Changed in charm-nova-compute-proxy:
importance: Undecided → Medium
Ryan Beisner (1chb1n)
Changed in charm-nova-compute-proxy:
status: Confirmed → Won't Fix
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

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