lp:~koalinux/charms/trusty/neutron-contrail/vhost-phy

Branch merges

Propose for merging

No branches dependent on this one.

Rejected for merging into lp:~sdn-charmers/charms/trusty/neutron-contrail/trunk

Robert Ayres (community): Disapprove on 2017-11-08

Diff: 43 lines (+4/-4)

2 files modified

hooks/neutron_contrail_hooks.py (+1/-1)
hooks/neutron_contrail_utils.py (+3/-3)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Related blueprints

73. By José Pekkarinen on 2017-11-06

  Use vhost-interface instead of vhost-phy to define the local interface.
  vhost-phy is defined in dpdk installations but referenced always in vrouter_ctx,
  producing tracebacks like the one blelow in case you don't use dpdk:

  2017-11-02 09:45:46 DEBUG upgrade-charm Traceback (most recent call last):
  2017-11-02 09:45:46 DEBUG upgrade-charm File "/var/lib/juju/agents/unit-neutron-contrail-0/charm/hooks/upgrade-charm", line 463, in <module>
  2017-11-02 09:45:46 DEBUG upgrade-charm main()
  2017-11-02 09:45:46 DEBUG upgrade-charm File "/var/lib/juju/agents/unit-neutron-contrail-0/charm/hooks/upgrade-charm", line 444, in main
  2017-11-02 09:45:46 DEBUG upgrade-charm hooks.execute(sys.argv)
  2017-11-02 09:45:46 DEBUG upgrade-charm File "/var/lib/juju/agents/unit-neutron-contrail-0/charm/hooks/charmhelpers/core/hookenv.py", line 715, in execute
  2017-11-02 09:45:46 DEBUG upgrade-charm self._hooks[hook_name]()
  2017-11-02 09:45:46 DEBUG upgrade-charm File "/var/lib/juju/agents/unit-neutron-contrail-0/charm/hooks/upgrade-charm", line 450, in upgrade_charm
  2017-11-02 09:45:46 DEBUG upgrade-charm write_vrouter_config()
  2017-11-02 09:45:46 DEBUG upgrade-charm File "/var/lib/juju/agents/unit-neutron-contrail-0/charm/hooks/neutron_contrail_utils.py", line 548, in write_vrouter_config
  2017-11-02 09:45:46 DEBUG upgrade-charm ctx.update(vrouter_ctx())
  2017-11-02 09:45:46 DEBUG upgrade-charm File "/var/lib/juju/agents/unit-neutron-contrail-0/charm/hooks/neutron_contrail_utils.py", line 510, in vrouter_ctx
  2017-11-02 09:45:46 DEBUG upgrade-charm "vhost_physical": config["vhost-phy"] }
  2017-11-02 09:45:46 DEBUG upgrade-charm KeyError: 'vhost-phy'

72. By Robert Ayres on 2017-09-20

Support setting huge pages kernel parameters

71. By Robert Ayres on 2017-08-03

Support other huge page sizes

70. By Robert Ayres on 2017-07-27

Minor fix for DPDK bonded interfaces

69. By Robert Ayres on 2017-07-22

Minor fix

68. By Robert Ayres on 2017-07-21

Restore network routes when bringing up dpdk vhost0

67. By Robert Ayres on 2017-07-19

Fix leaky file descriptor

66. By Robert Ayres on 2017-07-17

Add DPDK vRouter support

65. By Robert Ayres on 2017-03-10

Merge Ante Karamatic's changes (with modifications)

64. By Robert Ayres on 2017-02-15

Remove juju bridge by default