)]}'
{".zuul.yaml":[{"author":{"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},"change_message_id":"cd2e29a68c21ac4f8d55a1bc43f9a564e7946b12","unresolved":false,"context_lines":[{"line_number":272,"context_line":"    required-projects:"},{"line_number":273,"context_line":"      - openstack/openstack-helm"},{"line_number":274,"context_line":""},{"line_number":275,"context_line":"- job:"},{"line_number":276,"context_line":"    name: openstack-helm-infra-airship-divingbell"},{"line_number":277,"context_line":"    vars:"},{"line_number":278,"context_line":"      zuul_osh_relative_path: ../airship-divingbell/"},{"line_number":279,"context_line":"    parent: openstack-helm-infra"},{"line_number":280,"context_line":"    nodeset: openstack-helm-single-node"},{"line_number":281,"context_line":"    run: playbooks/airship-divingbell-check.yaml"},{"line_number":282,"context_line":"    required-projects:"},{"line_number":283,"context_line":"      - openstack/airship-divingbell"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3f79a3b5_b7279ec8","line":283,"range":{"start_line":275,"start_character":0,"end_line":283,"end_character":36},"updated":"2018-10-01 06:30:48.000000000","message":"This job doesn\u0027t get run as it is not listed in the check.  Also OSH-infra does not have a playbooks/airship-divingbell-check.yaml, should this job not be part of divingbell so it depends on helm-toolkit from OSH-infra?","commit_id":"3638a9f8e342d69603c08019b363df521c641da3"}],"/COMMIT_MSG":[{"author":{"_account_id":17591,"name":"Steve Wilkerson","email":"wilkers.steve@gmail.com","username":"srwilkers"},"change_message_id":"0e5b99176d95ea595ec36b706d060b5c88581a8b","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Anderson, Craig (ca846m) \u003ccraig.anderson@att.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2018-11-20 10:26:13 -0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add divingbell to zuul for test of htk overrides"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I1e5cec885b4ae20f34733e6aaf1ee0a8a170ac32"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":12,"id":"3f79a3b5_1308c37c","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":48},"updated":"2018-11-26 20:26:24.000000000","message":"Can we add a description here that includes a quick synopsis of why the daemonset overrides needed to be updated as well? Something that summarizes Tin\u0027s comment and your response would suffice","commit_id":"e0ece08591af02face042ba2cde82d7a586aa017"}],"helm-toolkit/templates/utils/_daemonset_overrides.tpl":[{"author":{"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},"change_message_id":"73363711f03f27249e104dfd0544795c770e432f","unresolved":false,"context_lines":[{"line_number":37,"context_line":"              iteration of the daemonset */}}"},{"line_number":38,"context_line":"              {{- $current_dict :\u003d dict }}"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"              {{/* set daemonset name */}}"},{"line_number":41,"context_line":"              {{/* Note: long hostnames can cause the 63 char name limit to be"},{"line_number":42,"context_line":"              exceeded. Truncate the hostname if hostname \u003e 20 char */}}"},{"line_number":43,"context_line":"              {{- if gt (len $host_data.name) 20 }}"},{"line_number":44,"context_line":"                {{- $_ :\u003d set $current_dict \"name\" (substr 0 20 $host_data.name) }}"},{"line_number":45,"context_line":"              {{- else }}"},{"line_number":46,"context_line":"                {{- $_ :\u003d set $current_dict \"name\" $host_data.name }}"},{"line_number":47,"context_line":"              {{- end }}"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"              {{/* apply overrides */}}"},{"line_number":50,"context_line":"              {{- $override_conf_copy :\u003d $host_data.conf }}"}],"source_content_type":"text/x-smarty","patch_set":12,"id":"3f79a3b5_6cfbad15","line":47,"range":{"start_line":40,"start_character":0,"end_line":47,"end_character":24},"updated":"2018-11-20 20:16:33.000000000","message":"Should this be in a separate PS? Thought this patch set (based on commitmsg) is to test integration between HTK and Divingbell - not also to add in daemonset override changes?","commit_id":"e0ece08591af02face042ba2cde82d7a586aa017"},{"author":{"_account_id":26686,"name":"Craig Anderson","email":"craig.cigar.anderson@gmail.com","username":"canderson"},"change_message_id":"4130828fd789e25f1b5683326b0afd2bb6436045","unresolved":false,"context_lines":[{"line_number":37,"context_line":"              iteration of the daemonset */}}"},{"line_number":38,"context_line":"              {{- $current_dict :\u003d dict }}"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"              {{/* set daemonset name */}}"},{"line_number":41,"context_line":"              {{/* Note: long hostnames can cause the 63 char name limit to be"},{"line_number":42,"context_line":"              exceeded. Truncate the hostname if hostname \u003e 20 char */}}"},{"line_number":43,"context_line":"              {{- if gt (len $host_data.name) 20 }}"},{"line_number":44,"context_line":"                {{- $_ :\u003d set $current_dict \"name\" (substr 0 20 $host_data.name) }}"},{"line_number":45,"context_line":"              {{- else }}"},{"line_number":46,"context_line":"                {{- $_ :\u003d set $current_dict \"name\" $host_data.name }}"},{"line_number":47,"context_line":"              {{- end }}"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"              {{/* apply overrides */}}"},{"line_number":50,"context_line":"              {{- $override_conf_copy :\u003d $host_data.conf }}"}],"source_content_type":"text/x-smarty","patch_set":12,"id":"3f79a3b5_87762240","line":47,"range":{"start_line":40,"start_character":0,"end_line":47,"end_character":24},"in_reply_to":"3f79a3b5_6cfbad15","updated":"2018-11-20 20:43:01.000000000","message":"I added it here because it was a requirement for making the override test pass in this environment, which has a very long hostname that exceeds helm\u0027s 63 character limit for name length (see logs from last zuul failure)","commit_id":"e0ece08591af02face042ba2cde82d7a586aa017"},{"author":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"change_message_id":"32d5cf00315c7296a51589d7e54f90f72778f844","unresolved":false,"context_lines":[{"line_number":37,"context_line":"              iteration of the daemonset */}}"},{"line_number":38,"context_line":"              {{- $current_dict :\u003d dict }}"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"              {{/* set daemonset name */}}"},{"line_number":41,"context_line":"              {{/* Note: long hostnames can cause the 63 char name limit to be"},{"line_number":42,"context_line":"              exceeded. Truncate the hostname if hostname \u003e 20 char */}}"},{"line_number":43,"context_line":"              {{- if gt (len $host_data.name) 20 }}"},{"line_number":44,"context_line":"                {{- $_ :\u003d set $current_dict \"name\" (substr 0 20 $host_data.name) }}"},{"line_number":45,"context_line":"              {{- else }}"},{"line_number":46,"context_line":"                {{- $_ :\u003d set $current_dict \"name\" $host_data.name }}"},{"line_number":47,"context_line":"              {{- end }}"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"              {{/* apply overrides */}}"},{"line_number":50,"context_line":"              {{- $override_conf_copy :\u003d $host_data.conf }}"}],"source_content_type":"text/x-smarty","patch_set":12,"id":"3f79a3b5_368a8d12","line":47,"range":{"start_line":40,"start_character":0,"end_line":47,"end_character":24},"in_reply_to":"3f79a3b5_87762240","updated":"2018-11-26 21:03:14.000000000","message":"I agree with tin - can we split this out.","commit_id":"e0ece08591af02face042ba2cde82d7a586aa017"}],"zuul.d/project.yaml":[{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"change_message_id":"0da80351a3f58fcd2a66c6f912b0f455bf79305b","unresolved":false,"context_lines":[{"line_number":13,"context_line":"# See the License for the specific language governing permissions and"},{"line_number":14,"context_line":"# limitations under the License."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"- project:"},{"line_number":17,"context_line":"    templates:"},{"line_number":18,"context_line":"      - publish-openstack-docs-pti"},{"line_number":19,"context_line":"    check:"},{"line_number":20,"context_line":"      jobs:"},{"line_number":21,"context_line":"        - openstack-helm-lint"},{"line_number":22,"context_line":"        - openstack-helm-infra-five-ubuntu"},{"line_number":23,"context_line":"        - openstack-helm-infra-aio-logging:"},{"line_number":24,"context_line":"            voting: false"},{"line_number":25,"context_line":"        - openstack-helm-infra-aio-monitoring:"},{"line_number":26,"context_line":"            voting: false"},{"line_number":27,"context_line":"        - openstack-helm-infra-aio-network-policy:"},{"line_number":28,"context_line":"            voting: false"},{"line_number":29,"context_line":"        - openstack-helm-infra-openstack-support"},{"line_number":30,"context_line":"        - openstack-helm-infra-kubernetes-keystone-auth"},{"line_number":31,"context_line":"        # some testing performed here to check for any break of host/label"},{"line_number":32,"context_line":"        # override functionality"},{"line_number":33,"context_line":"        - openstack-helm-infra-airship-divingbell:"},{"line_number":34,"context_line":"    gate:"},{"line_number":35,"context_line":"      jobs:"},{"line_number":36,"context_line":"        - openstack-helm-lint"},{"line_number":37,"context_line":"        - openstack-helm-infra-five-ubuntu"},{"line_number":38,"context_line":"        - openstack-helm-infra-openstack-support"},{"line_number":39,"context_line":"        - openstack-helm-infra-kubernetes-keystone-auth"},{"line_number":40,"context_line":"    experimental:"},{"line_number":41,"context_line":"      jobs:"},{"line_number":42,"context_line":"        #NOTE(srwilkers): Make fedora job experimental until issues resolved"},{"line_number":43,"context_line":"        - openstack-helm-infra-five-fedora"},{"line_number":44,"context_line":"        #NOTE(srwilkers): Make centos job experimental until issues resolved"},{"line_number":45,"context_line":"        - openstack-helm-infra-five-centos"}],"source_content_type":"text/x-yaml","patch_set":9,"id":"3f79a3b5_15ca4a18","line":45,"range":{"start_line":16,"start_character":2,"end_line":45,"end_character":0},"updated":"2018-11-20 07:13:10.000000000","message":"expected str for dictionary value @ data[\u0027check\u0027][\u0027jobs\u0027][7][\u0027openstack-helm-infra-airship-divingbell\u0027]","commit_id":"c8912eca7210c37f04a1fff1c46e3365327ccd6c"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"change_message_id":"9dd2ac248972a7726dc2d094998ac1bf84c76c90","unresolved":false,"context_lines":[{"line_number":13,"context_line":"# See the License for the specific language governing permissions and"},{"line_number":14,"context_line":"# limitations under the License."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"- project:"},{"line_number":17,"context_line":"    templates:"},{"line_number":18,"context_line":"      - publish-openstack-docs-pti"},{"line_number":19,"context_line":"    check:"},{"line_number":20,"context_line":"      jobs:"},{"line_number":21,"context_line":"        - openstack-helm-lint"},{"line_number":22,"context_line":"        - openstack-helm-infra-five-ubuntu"},{"line_number":23,"context_line":"        - openstack-helm-infra-aio-logging:"},{"line_number":24,"context_line":"            voting: false"},{"line_number":25,"context_line":"        - openstack-helm-infra-aio-monitoring:"},{"line_number":26,"context_line":"            voting: false"},{"line_number":27,"context_line":"        - openstack-helm-infra-aio-network-policy:"},{"line_number":28,"context_line":"            voting: false"},{"line_number":29,"context_line":"        - openstack-helm-infra-openstack-support"},{"line_number":30,"context_line":"        - openstack-helm-infra-kubernetes-keystone-auth"},{"line_number":31,"context_line":"        # some testing performed here to check for any break of host/label"},{"line_number":32,"context_line":"        # override functionality"},{"line_number":33,"context_line":"        - openstack-helm-infra-airship-divingbell:"},{"line_number":34,"context_line":"    gate:"},{"line_number":35,"context_line":"      jobs:"},{"line_number":36,"context_line":"        - openstack-helm-lint"},{"line_number":37,"context_line":"        - openstack-helm-infra-five-ubuntu"},{"line_number":38,"context_line":"        - openstack-helm-infra-openstack-support"},{"line_number":39,"context_line":"        - openstack-helm-infra-kubernetes-keystone-auth"},{"line_number":40,"context_line":"    experimental:"},{"line_number":41,"context_line":"      jobs:"},{"line_number":42,"context_line":"        #NOTE(srwilkers): Make fedora job experimental until issues resolved"},{"line_number":43,"context_line":"        - openstack-helm-infra-five-fedora"},{"line_number":44,"context_line":"        #NOTE(srwilkers): Make centos job experimental until issues resolved"},{"line_number":45,"context_line":"        - openstack-helm-infra-five-centos"}],"source_content_type":"text/x-yaml","patch_set":10,"id":"3f79a3b5_95273ac9","line":45,"range":{"start_line":16,"start_character":2,"end_line":45,"end_character":0},"updated":"2018-11-20 07:35:24.000000000","message":"expected str for dictionary value @ data[\u0027check\u0027][\u0027jobs\u0027][7][\u0027openstack-helm-infra-airship-divingbell\u0027]","commit_id":"84ccadbd72db775b45da28ef10482718613036ff"}]}