charm deprecation needs a user-visible warning
Bug #1474093 reported by
Mike McCracken
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| quantum-gateway (Juju Charms Collection) |
Fix Released
|
High
|
Corey Bryant | ||
Bug Description
As the next release switches to neutron-gateway, a user who does a 'juju deploy quantum-gateway' needs to have some way of knowing that it's deprecated code.
I wouldn't necessarily expect that the user has read the most recent README for the charm (especially if they're using a bundle or the openstack-installer package)
I'm not sure what all the options available are, but perhaps using the juju workload-status API would be a good start?
Alternatively, if the plan is to just redirect quantum-gateway to neutron-gateway at the charm store level so it's impossible to freshly deploy stale code, that'd be great.
Related branches
lp:~corey.bryant/charms/trusty/quantum-gateway/end-of-life
- James Page: Approve
-
Diff: 15436 lines (+344/-14080)138 files modified.bzrignore (+0/-1)
.coveragerc (+0/-6)
.project (+0/-17)
.pydevproject (+0/-9)
Makefile (+1/-13)
README.md (+1/-207)
actions.yaml (+0/-2)
actions/git_reinstall.py (+0/-45)
charm-helpers-hooks.yaml (+0/-9)
charm-helpers-tests.yaml (+0/-5)
files/NeutronAgentMon (+0/-155)
files/neutron-ha-monitor.conf (+0/-4)
files/neutron-ha-monitor.py (+0/-436)
hooks/charmhelpers/contrib/__init__.py (+0/-15)
hooks/charmhelpers/contrib/charmsupport/__init__.py (+0/-15)
hooks/charmhelpers/contrib/charmsupport/nrpe.py (+0/-360)
hooks/charmhelpers/contrib/charmsupport/volumes.py (+0/-175)
hooks/charmhelpers/contrib/hahelpers/__init__.py (+0/-15)
hooks/charmhelpers/contrib/hahelpers/apache.py (+0/-82)
hooks/charmhelpers/contrib/hahelpers/cluster.py (+0/-272)
hooks/charmhelpers/contrib/network/__init__.py (+0/-15)
hooks/charmhelpers/contrib/network/ip.py (+0/-450)
hooks/charmhelpers/contrib/network/ovs/__init__.py (+0/-96)
hooks/charmhelpers/contrib/network/ufw.py (+0/-276)
hooks/charmhelpers/contrib/openstack/__init__.py (+0/-15)
hooks/charmhelpers/contrib/openstack/alternatives.py (+0/-33)
hooks/charmhelpers/contrib/openstack/amulet/__init__.py (+0/-15)
hooks/charmhelpers/contrib/openstack/amulet/deployment.py (+0/-146)
hooks/charmhelpers/contrib/openstack/amulet/utils.py (+0/-294)
hooks/charmhelpers/contrib/openstack/context.py (+0/-1328)
hooks/charmhelpers/contrib/openstack/files/__init__.py (+0/-18)
hooks/charmhelpers/contrib/openstack/files/check_haproxy.sh (+0/-32)
hooks/charmhelpers/contrib/openstack/files/check_haproxy_queue_depth.sh (+0/-30)
hooks/charmhelpers/contrib/openstack/ip.py (+0/-146)
hooks/charmhelpers/contrib/openstack/neutron.py (+0/-322)
hooks/charmhelpers/contrib/openstack/templates/__init__.py (+0/-18)
hooks/charmhelpers/contrib/openstack/templates/ceph.conf (+0/-15)
hooks/charmhelpers/contrib/openstack/templates/git.upstart (+0/-17)
hooks/charmhelpers/contrib/openstack/templates/haproxy.cfg (+0/-58)
hooks/charmhelpers/contrib/openstack/templates/openstack_https_frontend (+0/-24)
hooks/charmhelpers/contrib/openstack/templates/openstack_https_frontend.conf (+0/-24)
hooks/charmhelpers/contrib/openstack/templates/section-keystone-authtoken (+0/-9)
hooks/charmhelpers/contrib/openstack/templates/section-rabbitmq-oslo (+0/-22)
hooks/charmhelpers/contrib/openstack/templates/section-zeromq (+0/-14)
hooks/charmhelpers/contrib/openstack/templating.py (+0/-295)
hooks/charmhelpers/contrib/openstack/utils.py (+0/-642)
hooks/charmhelpers/contrib/python/__init__.py (+0/-15)
hooks/charmhelpers/contrib/python/debug.py (+0/-56)
hooks/charmhelpers/contrib/python/packages.py (+0/-96)
hooks/charmhelpers/contrib/python/rpdb.py (+0/-58)
hooks/charmhelpers/contrib/python/version.py (+0/-34)
hooks/charmhelpers/contrib/storage/__init__.py (+0/-15)
hooks/charmhelpers/contrib/storage/linux/__init__.py (+0/-15)
hooks/charmhelpers/contrib/storage/linux/ceph.py (+0/-444)
hooks/charmhelpers/contrib/storage/linux/loopback.py (+0/-78)
hooks/charmhelpers/contrib/storage/linux/lvm.py (+0/-105)
hooks/charmhelpers/contrib/storage/linux/utils.py (+0/-70)
hooks/charmhelpers/core/hookenv.py (+232/-38)
hooks/charmhelpers/core/host.py (+56/-12)
hooks/charmhelpers/core/services/base.py (+43/-19)
hooks/charmhelpers/core/services/helpers.py (+2/-2)
hooks/charmhelpers/fetch/__init__.py (+0/-439)
hooks/charmhelpers/fetch/archiveurl.py (+0/-161)
hooks/charmhelpers/fetch/bzrurl.py (+0/-78)
hooks/charmhelpers/fetch/giturl.py (+0/-71)
hooks/charmhelpers/payload/__init__.py (+0/-17)
hooks/charmhelpers/payload/execd.py (+0/-66)
hooks/quantum_contexts.py (+0/-193)
hooks/quantum_hooks.py (+7/-327)
hooks/quantum_utils.py (+0/-1153)
metadata.yaml (+2/-0)
setup.cfg (+0/-5)
templates/ext-port.conf (+0/-16)
templates/folsom/dhcp_agent.ini (+0/-10)
templates/folsom/l3_agent.ini (+0/-8)
templates/folsom/metadata_agent.ini (+0/-12)
templates/folsom/nova.conf (+0/-26)
templates/folsom/ovs_quantum_plugin.ini (+0/-8)
templates/folsom/quantum.conf (+0/-15)
templates/git/cron.d/neutron-dhcp-agent-netns-cleanup (+0/-4)
templates/git/cron.d/neutron-l3-agent-netns-cleanup (+0/-4)
templates/git/cron.d/neutron-lbaas-agent-netns-cleanup (+0/-4)
templates/git/neutron_sudoers (+0/-4)
templates/git/upstart/neutron-agent.upstart (+0/-25)
templates/git/upstart/neutron-ovs-cleanup.upstart (+0/-13)
templates/git/upstart/neutron-server.upstart (+0/-22)
templates/grizzly/nova.conf (+0/-22)
templates/grizzly/quantum.conf (+0/-11)
templates/havana/dhcp_agent.ini (+0/-27)
templates/havana/dnsmasq.conf (+0/-3)
templates/havana/fwaas_driver.ini (+0/-7)
templates/havana/l3_agent.ini (+0/-24)
templates/havana/lbaas_agent.ini (+0/-8)
templates/havana/metadata_agent.ini (+0/-18)
templates/havana/metering_agent.ini (+0/-10)
templates/havana/neutron.conf (+0/-16)
templates/havana/nova.conf (+0/-27)
templates/havana/ovs_neutron_plugin.ini (+0/-14)
templates/havana/vpn_agent.ini (+0/-8)
templates/icehouse/metadata_agent.ini (+0/-19)
templates/icehouse/ml2_conf.ini (+0/-35)
templates/icehouse/neutron.conf (+0/-18)
templates/juno/l3_agent.ini (+0/-25)
templates/juno/ml2_conf.ini (+0/-36)
templates/kilo/fwaas_driver.ini (+0/-8)
templates/kilo/lbaas_agent.ini (+0/-13)
templates/kilo/neutron.conf (+0/-25)
templates/kilo/nova.conf (+0/-33)
templates/kilo/vpn_agent.ini (+0/-9)
templates/os-charm-phy-nic-mtu.conf (+0/-22)
templates/parts/database (+0/-1)
templates/parts/rabbitmq (+0/-21)
tests/00-setup (+0/-11)
tests/014-basic-precise-icehouse (+0/-11)
tests/015-basic-trusty-icehouse (+0/-9)
tests/016-basic-trusty-juno (+0/-11)
tests/017-basic-trusty-kilo (+0/-11)
tests/018-basic-utopic-juno (+0/-9)
tests/019-basic-vivid-kilo (+0/-9)
tests/050-basic-trusty-icehouse-git (+0/-9)
tests/051-basic-trusty-juno-git (+0/-12)
tests/README (+0/-53)
tests/basic_deployment.py (+0/-656)
tests/charmhelpers/__init__.py (+0/-38)
tests/charmhelpers/contrib/__init__.py (+0/-15)
tests/charmhelpers/contrib/amulet/__init__.py (+0/-15)
tests/charmhelpers/contrib/amulet/deployment.py (+0/-93)
tests/charmhelpers/contrib/amulet/utils.py (+0/-323)
tests/charmhelpers/contrib/openstack/__init__.py (+0/-15)
tests/charmhelpers/contrib/openstack/amulet/__init__.py (+0/-15)
tests/charmhelpers/contrib/openstack/amulet/deployment.py (+0/-146)
tests/charmhelpers/contrib/openstack/amulet/utils.py (+0/-294)
unit_tests/__init__.py (+0/-4)
unit_tests/test_actions_git_reinstall.py (+0/-105)
unit_tests/test_quantum_contexts.py (+0/-274)
unit_tests/test_quantum_hooks.py (+0/-370)
unit_tests/test_quantum_utils.py (+0/-1077)
unit_tests/test_utils.py (+0/-100)
Revision history for this message
| Ryan Beisner (1chb1n) wrote | #1 |
Revision history for this message
| Mike McCracken (mikemc) wrote Re: [Bug 1474093] Re: charm deprecation needs a user-visible warning | #2 |
| Changed in quantum-gateway (Juju Charms Collection): | |
| status: | New → Fix Committed |
| assignee: | nobody → Corey Bryant (corey.bryant) |
| status: | Fix Committed → Confirmed |
| Changed in quantum-gateway (Juju Charms Collection): | |
| status: | Confirmed → Fix Committed |
| Changed in quantum-gateway (Juju Charms Collection): | |
| importance: | Undecided → High |
| milestone: | none → 15.10 |
| Changed in quantum-gateway (Juju Charms Collection): | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
Right, not everyone reads the release notes. We sent a ~1mo. reminder to the juju mailing list, and should probably send another @ freeze.
One of the options discussed was to have the quantum-gateway name space contain a dummy charm >= 15.07, with juju status and debug log deprecation messages, and otherwise no charm functionality (ie. deploy would just fail).