)]}'
{"neutron/templates/configmap-etc.yaml":[{"author":{"_account_id":6678,"name":"Hyunsun Moon","email":"hyunsun.moon@gmail.com","username":"hyunsun-moon"},"change_message_id":"843de146de9592ac7568f1ecb38874e2cf1a2276","unresolved":false,"context_lines":[{"line_number":119,"context_line":"{{- end -}}"},{"line_number":120,"context_line":"{{- end -}}"},{"line_number":121,"context_line":""},{"line_number":122,"context_line":"{{- if empty .Values.conf.plugins.ml2_conf.ml2.mechanism_drivers -}}"},{"line_number":123,"context_line":"{{- if eq .Values.network.backend \"ovs\" -}}"},{"line_number":124,"context_line":"{{- set .Values.conf.plugins.ml2_conf.ml2 \"mechanism_drivers\" \"openvswitch,l2population\" | quote | trunc 0 -}}"},{"line_number":125,"context_line":"{{- end -}}"}],"source_content_type":"text/x-yaml","patch_set":12,"id":"df7087c5_f01b4618","line":122,"range":{"start_line":122,"start_character":13,"end_line":122,"end_character":64},"updated":"2018-03-09 11:53:58.000000000","message":"I\u0027m unsure hiding default values to template unless if it\u0027s unavoidable. anyway, we can simply set default mechanism drivers includes both openvswitch and linuxbridge in values as devstack does.","commit_id":"340a9ec434ac376d0c852155a2edd4ca6ad85fc6"},{"author":{"_account_id":6678,"name":"Hyunsun Moon","email":"hyunsun.moon@gmail.com","username":"hyunsun-moon"},"change_message_id":"843de146de9592ac7568f1ecb38874e2cf1a2276","unresolved":false,"context_lines":[{"line_number":124,"context_line":"{{- set .Values.conf.plugins.ml2_conf.ml2 \"mechanism_drivers\" \"openvswitch,l2population\" | quote | trunc 0 -}}"},{"line_number":125,"context_line":"{{- end -}}"},{"line_number":126,"context_line":"{{- if eq .Values.network.backend \"linuxbridge\" -}}"},{"line_number":127,"context_line":"{{- set .Values.conf.plugins.ml2_conf.ml2 \"mechanism_drivers\" \"linuxbridge,l2population\" | quote | trunc 0 -}}"},{"line_number":128,"context_line":"{{- end -}}"},{"line_number":129,"context_line":"{{- end -}}"},{"line_number":130,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":12,"id":"df7087c5_30221e54","line":127,"range":{"start_line":127,"start_character":75,"end_line":127,"end_character":87},"updated":"2018-03-09 11:53:58.000000000","message":"this conflicts with l3_ha, which is enabled by default.","commit_id":"340a9ec434ac376d0c852155a2edd4ca6ad85fc6"},{"author":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"change_message_id":"ce4eac586790557aa0db8133948b5f3e651a96ce","unresolved":false,"context_lines":[{"line_number":124,"context_line":"{{- set .Values.conf.plugins.ml2_conf.ml2 \"mechanism_drivers\" \"openvswitch,l2population\" | quote | trunc 0 -}}"},{"line_number":125,"context_line":"{{- end -}}"},{"line_number":126,"context_line":"{{- if eq .Values.network.backend \"linuxbridge\" -}}"},{"line_number":127,"context_line":"{{- set .Values.conf.plugins.ml2_conf.ml2 \"mechanism_drivers\" \"linuxbridge,l2population\" | quote | trunc 0 -}}"},{"line_number":128,"context_line":"{{- end -}}"},{"line_number":129,"context_line":"{{- end -}}"},{"line_number":130,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":12,"id":"df7087c5_db2bc37f","line":127,"range":{"start_line":127,"start_character":75,"end_line":127,"end_character":87},"in_reply_to":"df7087c5_30221e54","updated":"2018-03-09 12:06:22.000000000","message":"good - catch, these were brought directly over from the gate configs - we should fix.","commit_id":"340a9ec434ac376d0c852155a2edd4ca6ad85fc6"}],"neutron/values.yaml":[{"author":{"_account_id":6678,"name":"Hyunsun Moon","email":"hyunsun.moon@gmail.com","username":"hyunsun-moon"},"change_message_id":"843de146de9592ac7568f1ecb38874e2cf1a2276","unresolved":false,"context_lines":[{"line_number":112,"context_line":"    openstack token issue"},{"line_number":113,"context_line":""},{"line_number":114,"context_line":"dependencies:"},{"line_number":115,"context_line":"  dynamic:"},{"line_number":116,"context_line":"    targeted:"},{"line_number":117,"context_line":"      ovs:"},{"line_number":118,"context_line":"        dhcp:"},{"line_number":119,"context_line":"          daemonset:"},{"line_number":120,"context_line":"            - neutron-ovs-agent"},{"line_number":121,"context_line":"        l3:"},{"line_number":122,"context_line":"          daemonset:"},{"line_number":123,"context_line":"            - neutron-ovs-agent"},{"line_number":124,"context_line":"        metadata:"},{"line_number":125,"context_line":"          daemonset:"},{"line_number":126,"context_line":"            - neutron-ovs-agent"},{"line_number":127,"context_line":"      linuxbridge:"},{"line_number":128,"context_line":"        dhcp:"},{"line_number":129,"context_line":"          daemonset:"},{"line_number":130,"context_line":"            - neutron-lb-agent"},{"line_number":131,"context_line":"        l3:"},{"line_number":132,"context_line":"          daemonset:"},{"line_number":133,"context_line":"            - neutron-lb-agent"},{"line_number":134,"context_line":"        metadata:"},{"line_number":135,"context_line":"          daemonset:"},{"line_number":136,"context_line":"            - neutron-lb-agent"},{"line_number":137,"context_line":"  static:"},{"line_number":138,"context_line":"    bootstrap:"},{"line_number":139,"context_line":"      services:"}],"source_content_type":"text/x-yaml","patch_set":12,"id":"df7087c5_15d6ecf2","line":136,"range":{"start_line":115,"start_character":0,"end_line":136,"end_character":30},"updated":"2018-03-09 11:53:58.000000000","message":"multiple mechanisms can be used simultaneously in a cluster. so the dependency is actually be decided mechanism driver plus what kind of L2 agent is enabled in each node. nobody does this but just for example I could label \"openvswitch\" to controller-01 and \"linuxbridge\" to controller-02. in that case, the agents on controller-01 should have dependency on neutron-ovs-agent and the other agents on controller-02 should have neutron-lb-agent.\n\nthe same can be applied to nova-compute dependency as well, which is more tricky to set correct value.","commit_id":"340a9ec434ac376d0c852155a2edd4ca6ad85fc6"},{"author":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"change_message_id":"ce4eac586790557aa0db8133948b5f3e651a96ce","unresolved":false,"context_lines":[{"line_number":112,"context_line":"    openstack token issue"},{"line_number":113,"context_line":""},{"line_number":114,"context_line":"dependencies:"},{"line_number":115,"context_line":"  dynamic:"},{"line_number":116,"context_line":"    targeted:"},{"line_number":117,"context_line":"      ovs:"},{"line_number":118,"context_line":"        dhcp:"},{"line_number":119,"context_line":"          daemonset:"},{"line_number":120,"context_line":"            - neutron-ovs-agent"},{"line_number":121,"context_line":"        l3:"},{"line_number":122,"context_line":"          daemonset:"},{"line_number":123,"context_line":"            - neutron-ovs-agent"},{"line_number":124,"context_line":"        metadata:"},{"line_number":125,"context_line":"          daemonset:"},{"line_number":126,"context_line":"            - neutron-ovs-agent"},{"line_number":127,"context_line":"      linuxbridge:"},{"line_number":128,"context_line":"        dhcp:"},{"line_number":129,"context_line":"          daemonset:"},{"line_number":130,"context_line":"            - neutron-lb-agent"},{"line_number":131,"context_line":"        l3:"},{"line_number":132,"context_line":"          daemonset:"},{"line_number":133,"context_line":"            - neutron-lb-agent"},{"line_number":134,"context_line":"        metadata:"},{"line_number":135,"context_line":"          daemonset:"},{"line_number":136,"context_line":"            - neutron-lb-agent"},{"line_number":137,"context_line":"  static:"},{"line_number":138,"context_line":"    bootstrap:"},{"line_number":139,"context_line":"      services:"}],"source_content_type":"text/x-yaml","patch_set":12,"id":"df7087c5_db50a30d","line":136,"range":{"start_line":115,"start_character":0,"end_line":136,"end_character":30},"in_reply_to":"df7087c5_15d6ecf2","updated":"2018-03-09 12:06:22.000000000","message":"please see here for the extension to this paradigm that adds support for what you want i hope: https://review.openstack.org/#/c/547890/","commit_id":"340a9ec434ac376d0c852155a2edd4ca6ad85fc6"},{"author":{"_account_id":6678,"name":"Hyunsun Moon","email":"hyunsun.moon@gmail.com","username":"hyunsun-moon"},"change_message_id":"843de146de9592ac7568f1ecb38874e2cf1a2276","unresolved":false,"context_lines":[{"line_number":946,"context_line":"      router_auto_schedule: True"},{"line_number":947,"context_line":"      #(NOTE)portdirect: if unset this is populated dyanmicly from the value in"},{"line_number":948,"context_line":"      # \u0027network.backend\u0027 to sane defaults."},{"line_number":949,"context_line":"      interface_driver: null"},{"line_number":950,"context_line":"    oslo_concurrency:"},{"line_number":951,"context_line":"      lock_path: /var/lib/neutron/tmp"},{"line_number":952,"context_line":"    database:"}],"source_content_type":"text/x-yaml","patch_set":12,"id":"df7087c5_95bbdc70","line":949,"range":{"start_line":949,"start_character":24,"end_line":949,"end_character":28},"updated":"2018-03-09 11:53:58.000000000","message":"here applies the same.","commit_id":"340a9ec434ac376d0c852155a2edd4ca6ad85fc6"}]}