)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":17068,"name":"Jean-Philippe Evrard","email":"openstack@a.spamming.party","username":"evrardjp"},"change_message_id":"ee0db67bc8d121b03b0e857f378c5fc2537b46a9","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     chengli3 \u003ccheng1.li@intel.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2019-07-19 09:44:09 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add nv job for ovs-dpdk"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Georg has created a patch[1] to extend ovs chart with dpdk support. So I"},{"line_number":10,"context_line":"create this patch to add a nv job for dpdk feature."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":10,"id":"7faddb67_7b1d2372","line":7,"range":{"start_line":7,"start_character":4,"end_line":7,"end_character":6},"updated":"2019-07-22 11:10:06.000000000","message":"that\u0027s very confusing, you\u0027re saying nv but this is a voting job, everywhere. Please change the commit message, or change the zuul\u0027s code to make it really non-voting.","commit_id":"eae89f032457b33440ae221ad9e500bb004e7eb0"},{"author":{"_account_id":29668,"name":"cheng li","email":"cheng1.li@intel.com","username":"chengli3"},"change_message_id":"a68451b0e4a92b7b599c4fadd285061718aa61b3","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     chengli3 \u003ccheng1.li@intel.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2019-07-19 09:44:09 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add nv job for ovs-dpdk"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Georg has created a patch[1] to extend ovs chart with dpdk support. So I"},{"line_number":10,"context_line":"create this patch to add a nv job for dpdk feature."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":10,"id":"7faddb67_d0b3d14c","line":7,"range":{"start_line":7,"start_character":4,"end_line":7,"end_character":6},"in_reply_to":"7faddb67_7b1d2372","updated":"2019-07-22 12:20:20.000000000","message":"Done","commit_id":"eae89f032457b33440ae221ad9e500bb004e7eb0"}],"openvswitch/values_overrides/dpdk-ubuntu_bionic.yaml":[{"author":{"_account_id":29668,"name":"cheng li","email":"cheng1.li@intel.com","username":"chengli3"},"change_message_id":"33fd5419e75ad114582dda07eb294af136153ac3","unresolved":false,"context_lines":[{"line_number":9,"context_line":"    ovs:"},{"line_number":10,"context_line":"      vswitchd:"},{"line_number":11,"context_line":"        requests:"},{"line_number":12,"context_line":"          memory: \"2Gi\""},{"line_number":13,"context_line":"          cpu: \"2\""},{"line_number":14,"context_line":"        limits:"},{"line_number":15,"context_line":"          memory: \"2Gi\""}],"source_content_type":"text/x-yaml","patch_set":21,"id":"3fa7e38b_34c7ee66","line":12,"range":{"start_line":12,"start_character":19,"end_line":12,"end_character":22},"updated":"2019-10-15 01:36:14.000000000","message":"Please use less memory to save resource. 500Mi could be good enough.","commit_id":"2691b436939072f64a79c86e5d4f46d1340e634c"},{"author":{"_account_id":29668,"name":"cheng li","email":"cheng1.li@intel.com","username":"chengli3"},"change_message_id":"33fd5419e75ad114582dda07eb294af136153ac3","unresolved":false,"context_lines":[{"line_number":14,"context_line":"        limits:"},{"line_number":15,"context_line":"          memory: \"2Gi\""},{"line_number":16,"context_line":"          cpu: \"2\""},{"line_number":17,"context_line":"          hugepages-1Gi: \"1Gi\""},{"line_number":18,"context_line":"conf:"},{"line_number":19,"context_line":"  ovs_dpdk:"},{"line_number":20,"context_line":"    enabled: true"}],"source_content_type":"text/x-yaml","patch_set":21,"id":"3fa7e38b_d4b69af9","line":17,"range":{"start_line":17,"start_character":10,"end_line":17,"end_character":30},"updated":"2019-10-15 01:36:14.000000000","message":"This should be 2Mi hugepages, as you allocated 2Mi pages instead of 1Gi","commit_id":"2691b436939072f64a79c86e5d4f46d1340e634c"}],"tools/deployment/openstack-support/004-allocate-hugepages.sh":[{"author":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"change_message_id":"8268bf51b0a582393442c62f72f716641abf1abc","unresolved":false,"context_lines":[{"line_number":16,"context_line":""},{"line_number":17,"context_line":"set -xe"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"sudo bash -c \"echo \u0027vm.nr_hugepages\u003d512\u0027 \u003e /etc/sysctl.d/hugepages.conf\""},{"line_number":20,"context_line":"sudo sysctl -p /etc/sysctl.d/hugepages.conf"},{"line_number":21,"context_line":"if [ 512 -ne `cat /sys/kernel/mm/hugepages/hugepages-2048kB/nr_hugepages` ]"},{"line_number":22,"context_line":"then"},{"line_number":23,"context_line":"  echo \"Failed to allocate hugepages\""},{"line_number":24,"context_line":"  exit 1"},{"line_number":25,"context_line":"fi"}],"source_content_type":"text/x-sh","patch_set":11,"id":"7faddb67_9cd5721e","line":25,"range":{"start_line":19,"start_character":0,"end_line":25,"end_character":2},"updated":"2019-08-07 17:37:39.000000000","message":"if we use the \u0027feature gates\u0027 functionaility provide by the over-rides lookup script then we should not need this file, but could include it in the setup-k8s script.","commit_id":"6d5233a026f70dfcde0664c18e1e4b26df47f36f"}],"tools/deployment/openstack-support/060-openvswitch-dpdk.sh":[{"author":{"_account_id":16353,"name":"Georg Kunz","email":"georg.kunz@ericsson.com","username":"georg-kunz"},"change_message_id":"20746289c798c3c8756fcbf623763e6a31f210a4","unresolved":false,"context_lines":[{"line_number":18,"context_line":"tee /tmp/openvswitch.yaml \u003c\u003c EOF"},{"line_number":19,"context_line":"images:"},{"line_number":20,"context_line":"  tags:"},{"line_number":21,"context_line":"    openvswitch_db_server: docker.io/openstackhelm/openvswitch:latest-debian-dpdk"},{"line_number":22,"context_line":"    openvswitch_vswitchd: docker.io/openstackhelm/openvswitch:latest-debian-dpdk"},{"line_number":23,"context_line":"pod:"},{"line_number":24,"context_line":"  resources:"}],"source_content_type":"text/x-sh","patch_set":6,"id":"9fb8cfa7_051f40b7","line":21,"range":{"start_line":21,"start_character":63,"end_line":21,"end_character":81},"updated":"2019-06-11 20:51:48.000000000","message":"just a note: we still need a more robust image for this CI job, i.e., an image which also runs on older hardware.","commit_id":"884b31213dec71cd1ec68e2787bddd44eeeab9cd"},{"author":{"_account_id":29668,"name":"cheng li","email":"cheng1.li@intel.com","username":"chengli3"},"change_message_id":"04c711cbf169077d25c7fbf23e141ff86eca9c65","unresolved":false,"context_lines":[{"line_number":18,"context_line":"tee /tmp/openvswitch.yaml \u003c\u003c EOF"},{"line_number":19,"context_line":"images:"},{"line_number":20,"context_line":"  tags:"},{"line_number":21,"context_line":"    openvswitch_db_server: docker.io/openstackhelm/openvswitch:latest-debian-dpdk"},{"line_number":22,"context_line":"    openvswitch_vswitchd: docker.io/openstackhelm/openvswitch:latest-debian-dpdk"},{"line_number":23,"context_line":"pod:"},{"line_number":24,"context_line":"  resources:"}],"source_content_type":"text/x-sh","patch_set":6,"id":"9fb8cfa7_3f05a946","line":21,"range":{"start_line":21,"start_character":63,"end_line":21,"end_character":81},"in_reply_to":"9fb8cfa7_051f40b7","updated":"2019-06-12 03:08:03.000000000","message":"I also think that the failure is caused by ovs image. Let me try with your image[1]\n\n[1] gkunz/kolla-ovs-dpdk:7.0.0","commit_id":"884b31213dec71cd1ec68e2787bddd44eeeab9cd"},{"author":{"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},"change_message_id":"26058f8350db594cab1df13565eaf6479d2f11b2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"7faddb67_88249eb0","updated":"2019-07-16 15:46:06.000000000","message":"Since there is already a 060-openvswitch.sh file, should we name them as different steps, or perhaps integrate them into the same file and handle the override using parameters?","commit_id":"a0e18b9a2361a7dc808ceb36eb7d6eee54c1e0de"},{"author":{"_account_id":29668,"name":"cheng li","email":"cheng1.li@intel.com","username":"chengli3"},"change_message_id":"c31aeb40cb44a5031ef5799b120fa277264712c3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"7faddb67_cc55aafb","in_reply_to":"7faddb67_88249eb0","updated":"2019-07-18 09:54:17.000000000","message":"Either 060-openvswitch.sh or 060-openvswitch-dpdk.sh can used in deployment. We don\u0027t use both of them at the same time, so I used the same prefix number.","commit_id":"a0e18b9a2361a7dc808ceb36eb7d6eee54c1e0de"},{"author":{"_account_id":21883,"name":"Manuel Buil","email":"mbuil@suse.com","username":"mbuil"},"change_message_id":"7eeb98fd462d01cc0795348f6d51a8cd8cc98522","unresolved":false,"context_lines":[{"line_number":14,"context_line":"#    License for the specific language governing permissions and limitations"},{"line_number":15,"context_line":"#    under the License."},{"line_number":16,"context_line":"set -xe"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"tee /tmp/openvswitch.yaml \u003c\u003c EOF"},{"line_number":19,"context_line":"images:"},{"line_number":20,"context_line":"  tags:"}],"source_content_type":"text/x-sh","patch_set":11,"id":"7faddb67_abb71ff5","line":17,"updated":"2019-08-01 15:51:29.000000000","message":"I\u0027d suggest using:\n\nexport FEATURE_GATES\u003ddpdk\nexport HELM_CHART_ROOT_PATH\u003d\"${HELM_CHART_ROOT_PATH:\u003d\"${OSH_INFRA_PATH:\u003d\"../openstack-helm-infra\"}\"}\"\n: ${OSH_EXTRA_HELM_ARGS_OPENVSWITCH:\u003d\"$(./tools/deployment/common/get-values-overrides.sh openvswitch)\"}\n\nTo refer to the images instead of harcoding them in this script. That way we are aligned with how we do it in the neutron extension for DPDK patch. At the end of the day, we want to the test to be aligned with how we deploy\n\nhttps://review.opendev.org/#/c/643284/31/tools/deployment/developer/dpdk/140-openvswitch.sh","commit_id":"6d5233a026f70dfcde0664c18e1e4b26df47f36f"},{"author":{"_account_id":29668,"name":"cheng li","email":"cheng1.li@intel.com","username":"chengli3"},"change_message_id":"25e320ed496888d3c804dd32d91bdc40ce52fde1","unresolved":false,"context_lines":[{"line_number":14,"context_line":"#    License for the specific language governing permissions and limitations"},{"line_number":15,"context_line":"#    under the License."},{"line_number":16,"context_line":"set -xe"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"tee /tmp/openvswitch.yaml \u003c\u003c EOF"},{"line_number":19,"context_line":"images:"},{"line_number":20,"context_line":"  tags:"}],"source_content_type":"text/x-sh","patch_set":11,"id":"7faddb67_e1643224","line":17,"in_reply_to":"7faddb67_abb71ff5","updated":"2019-08-05 10:11:47.000000000","message":"Good point","commit_id":"6d5233a026f70dfcde0664c18e1e4b26df47f36f"},{"author":{"_account_id":29668,"name":"cheng li","email":"cheng1.li@intel.com","username":"chengli3"},"change_message_id":"c996e4ef319c5f0b2fab9c7b40438c9f0adcbac1","unresolved":false,"context_lines":[{"line_number":14,"context_line":"#    License for the specific language governing permissions and limitations"},{"line_number":15,"context_line":"#    under the License."},{"line_number":16,"context_line":"set -xe"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"tee /tmp/openvswitch.yaml \u003c\u003c EOF"},{"line_number":19,"context_line":"images:"},{"line_number":20,"context_line":"  tags:"}],"source_content_type":"text/x-sh","patch_set":11,"id":"7faddb67_b30f712c","line":17,"in_reply_to":"7faddb67_abb71ff5","updated":"2019-08-05 10:18:54.000000000","message":"emmm, seems I can\u0027t do that, because the get-values-overrides.sh is in openstack-helm project, instead of openstack-helm-infra.","commit_id":"6d5233a026f70dfcde0664c18e1e4b26df47f36f"},{"author":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"change_message_id":"8268bf51b0a582393442c62f72f716641abf1abc","unresolved":false,"context_lines":[{"line_number":14,"context_line":"#    License for the specific language governing permissions and limitations"},{"line_number":15,"context_line":"#    under the License."},{"line_number":16,"context_line":"set -xe"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"tee /tmp/openvswitch.yaml \u003c\u003c EOF"},{"line_number":19,"context_line":"images:"},{"line_number":20,"context_line":"  tags:"}],"source_content_type":"text/x-sh","patch_set":11,"id":"7faddb67_5c113a04","line":17,"in_reply_to":"7faddb67_b30f712c","updated":"2019-08-07 17:37:39.000000000","message":"we should bring this functionality into osh-infra, it was an oversight not to do so in the 1st place.","commit_id":"6d5233a026f70dfcde0664c18e1e4b26df47f36f"},{"author":{"_account_id":29668,"name":"cheng li","email":"cheng1.li@intel.com","username":"chengli3"},"change_message_id":"1334f80a8bce1e67fb275dc7934639f9d6d4dfb2","unresolved":false,"context_lines":[{"line_number":17,"context_line":""},{"line_number":18,"context_line":"tee /tmp/openvswitch.yaml \u003c\u003c EOF"},{"line_number":19,"context_line":": ${OSH_EXTRA_HELM_ARGS_OPENVSWITCH:\u003d\"$(./tools/deployment/common/get-values-overrides.sh openvswitch)\"}"},{"line_number":20,"context_line":"# delete"},{"line_number":21,"context_line":"# images:"},{"line_number":22,"context_line":"#   tags:"},{"line_number":23,"context_line":"#     openvswitch_db_server: docker.io/openstackhelm/openvswitch:latest-ubuntu_bionic-dpdk"},{"line_number":24,"context_line":"#     openvswitch_vswitchd: docker.io/openstackhelm/openvswitch:latest-ubuntu_bionic-dpdk"},{"line_number":25,"context_line":"pod:"},{"line_number":26,"context_line":"  resources:"},{"line_number":27,"context_line":"    enabled: true"}],"source_content_type":"text/x-sh","patch_set":12,"id":"7faddb67_32e38236","line":24,"range":{"start_line":20,"start_character":3,"end_line":24,"end_character":17},"updated":"2019-08-20 11:12:30.000000000","message":"Please delete these lines instead of commenting them out","commit_id":"673daa5d99c1a6fc687350ae91995ee33f161203"},{"author":{"_account_id":16353,"name":"Georg Kunz","email":"georg.kunz@ericsson.com","username":"georg-kunz"},"change_message_id":"7250307cb783b1e01898cb839b14cac3f4983253","unresolved":false,"context_lines":[{"line_number":17,"context_line":"${OSH_EXTRA_HELM_ARGS_OPENVSWITCH:\u003d\"$(./tools/deployment/common/get-values-overrides.sh openvswitch)\"}"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"tee /tmp/openvswitch.yaml \u003c\u003c EOF"},{"line_number":20,"context_line":"pod:"},{"line_number":21,"context_line":"  resources:"},{"line_number":22,"context_line":"    enabled: true"},{"line_number":23,"context_line":"    ovs:"},{"line_number":24,"context_line":"      vswitchd:"},{"line_number":25,"context_line":"        requests:"},{"line_number":26,"context_line":"          memory: \"500Mi\""},{"line_number":27,"context_line":"          cpu: \"1200m\""},{"line_number":28,"context_line":"          hugepages-2Mi: \"1Gi\""},{"line_number":29,"context_line":"        limits:"},{"line_number":30,"context_line":"          memory: \"500Mi\""},{"line_number":31,"context_line":"          cpu: \"1200m\""},{"line_number":32,"context_line":"          hugepages-2Mi: \"1Gi\""},{"line_number":33,"context_line":"conf:"},{"line_number":34,"context_line":"  dpdk:"},{"line_number":35,"context_line":"    enabled: true"},{"line_number":36,"context_line":"    hugepages_mountdir: /dev/hugepages"},{"line_number":37,"context_line":"    socket_memory: 1024"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"EOF"},{"line_number":40,"context_line":""}],"source_content_type":"text/x-sh","patch_set":15,"id":"7faddb67_dbb9a6cb","line":37,"range":{"start_line":20,"start_character":0,"end_line":37,"end_character":23},"updated":"2019-08-23 09:02:01.000000000","message":"since you bring in the get-value-overrides functionality, I believe these parameters should be moved to an overrides file as well.","commit_id":"e2710002c5ad5f7a16bbff42d9a180d95846161a"},{"author":{"_account_id":29668,"name":"cheng li","email":"cheng1.li@intel.com","username":"chengli3"},"change_message_id":"19c572f7831ec19032eb75d7d03cdb5652b1d3ec","unresolved":false,"context_lines":[{"line_number":17,"context_line":"${OSH_EXTRA_HELM_ARGS_OPENVSWITCH:\u003d\"$(./tools/deployment/common/get-values-overrides.sh openvswitch)\"}"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"tee /tmp/openvswitch.yaml \u003c\u003c EOF"},{"line_number":20,"context_line":"pod:"},{"line_number":21,"context_line":"  resources:"},{"line_number":22,"context_line":"    enabled: true"},{"line_number":23,"context_line":"    ovs:"},{"line_number":24,"context_line":"      vswitchd:"},{"line_number":25,"context_line":"        requests:"},{"line_number":26,"context_line":"          memory: \"500Mi\""},{"line_number":27,"context_line":"          cpu: \"1200m\""},{"line_number":28,"context_line":"          hugepages-2Mi: \"1Gi\""},{"line_number":29,"context_line":"        limits:"},{"line_number":30,"context_line":"          memory: \"500Mi\""},{"line_number":31,"context_line":"          cpu: \"1200m\""},{"line_number":32,"context_line":"          hugepages-2Mi: \"1Gi\""},{"line_number":33,"context_line":"conf:"},{"line_number":34,"context_line":"  dpdk:"},{"line_number":35,"context_line":"    enabled: true"},{"line_number":36,"context_line":"    hugepages_mountdir: /dev/hugepages"},{"line_number":37,"context_line":"    socket_memory: 1024"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"EOF"},{"line_number":40,"context_line":""}],"source_content_type":"text/x-sh","patch_set":15,"id":"7faddb67_f8a7afbb","line":37,"range":{"start_line":20,"start_character":0,"end_line":37,"end_character":23},"in_reply_to":"7faddb67_dbb9a6cb","updated":"2019-08-26 03:59:16.000000000","message":"Besides openstack-helm-infra project, the dpdk overrides file may also be used by openstack-helm. As I can see that these configurations can be used with all cases. So I think that it makes sense to move these parameters to the overrides file","commit_id":"e2710002c5ad5f7a16bbff42d9a180d95846161a"}],"zuul.d/project.yaml":[{"author":{"_account_id":17068,"name":"Jean-Philippe Evrard","email":"openstack@a.spamming.party","username":"evrardjp"},"change_message_id":"1cc3afda5b710b19f8d25110b1be1eba63835a96","unresolved":false,"context_lines":[{"line_number":46,"context_line":"      jobs:"},{"line_number":47,"context_line":"        - openstack-helm-infra-tenant-ceph"},{"line_number":48,"context_line":"        - openstack-helm-infra-five-ubuntu"},{"line_number":49,"context_line":"        - openstack-helm-infra-armada-deploy"},{"line_number":50,"context_line":"        - openstack-helm-infra-armada-update-uuid"},{"line_number":51,"context_line":"        - openstack-helm-infra-armada-update-passwords"},{"line_number":52,"context_line":"    experimental:"}],"source_content_type":"text/x-yaml","patch_set":11,"id":"7faddb67_662b9fea","line":49,"range":{"start_line":49,"start_character":10,"end_line":49,"end_character":44},"updated":"2019-08-01 14:29:45.000000000","message":"nit: you don\u0027t add it in periodics? Who/How will you monitor its state? Just testing on changes seem... unreliable to me, but I am fine with this if everyone is.","commit_id":"6d5233a026f70dfcde0664c18e1e4b26df47f36f"},{"author":{"_account_id":17068,"name":"Jean-Philippe Evrard","email":"openstack@a.spamming.party","username":"evrardjp"},"change_message_id":"a7fdf95970cece12e079d3a9e16cb50f21d3ba67","unresolved":false,"context_lines":[{"line_number":46,"context_line":"      jobs:"},{"line_number":47,"context_line":"        - openstack-helm-infra-tenant-ceph"},{"line_number":48,"context_line":"        - openstack-helm-infra-five-ubuntu"},{"line_number":49,"context_line":"        - openstack-helm-infra-armada-deploy"},{"line_number":50,"context_line":"        - openstack-helm-infra-armada-update-uuid"},{"line_number":51,"context_line":"        - openstack-helm-infra-armada-update-passwords"},{"line_number":52,"context_line":"    experimental:"}],"source_content_type":"text/x-yaml","patch_set":11,"id":"7faddb67_42c89353","line":49,"range":{"start_line":49,"start_character":10,"end_line":49,"end_character":44},"in_reply_to":"7faddb67_53615dd2","updated":"2019-08-06 09:49:34.000000000","message":"Well, I expect if there is something to watch over, it would be a periodic, not a check job. But if you\u0027re checking the check jobs, that\u0027s fine for me.","commit_id":"6d5233a026f70dfcde0664c18e1e4b26df47f36f"},{"author":{"_account_id":29668,"name":"cheng li","email":"cheng1.li@intel.com","username":"chengli3"},"change_message_id":"08b8066235a1dbdf679ccdc4d7fba096adb1ba8d","unresolved":false,"context_lines":[{"line_number":46,"context_line":"      jobs:"},{"line_number":47,"context_line":"        - openstack-helm-infra-tenant-ceph"},{"line_number":48,"context_line":"        - openstack-helm-infra-five-ubuntu"},{"line_number":49,"context_line":"        - openstack-helm-infra-armada-deploy"},{"line_number":50,"context_line":"        - openstack-helm-infra-armada-update-uuid"},{"line_number":51,"context_line":"        - openstack-helm-infra-armada-update-passwords"},{"line_number":52,"context_line":"    experimental:"}],"source_content_type":"text/x-yaml","patch_set":11,"id":"7faddb67_53615dd2","line":49,"range":{"start_line":49,"start_character":10,"end_line":49,"end_character":44},"in_reply_to":"7faddb67_662b9fea","updated":"2019-08-05 10:24:29.000000000","message":"Actually, I just had a view of the jobs. No check job or gate job exists in periodic job list. So I wonder if it\u0027s reasonable to do that for dpdk job","commit_id":"6d5233a026f70dfcde0664c18e1e4b26df47f36f"},{"author":{"_account_id":29668,"name":"cheng li","email":"cheng1.li@intel.com","username":"chengli3"},"change_message_id":"25e320ed496888d3c804dd32d91bdc40ce52fde1","unresolved":false,"context_lines":[{"line_number":46,"context_line":"      jobs:"},{"line_number":47,"context_line":"        - openstack-helm-infra-tenant-ceph"},{"line_number":48,"context_line":"        - openstack-helm-infra-five-ubuntu"},{"line_number":49,"context_line":"        - openstack-helm-infra-armada-deploy"},{"line_number":50,"context_line":"        - openstack-helm-infra-armada-update-uuid"},{"line_number":51,"context_line":"        - openstack-helm-infra-armada-update-passwords"},{"line_number":52,"context_line":"    experimental:"}],"source_content_type":"text/x-yaml","patch_set":11,"id":"7faddb67_217ecaae","line":49,"range":{"start_line":49,"start_character":10,"end_line":49,"end_character":44},"in_reply_to":"7faddb67_662b9fea","updated":"2019-08-05 10:11:47.000000000","message":"Thanks for reminding, I will add it into the periodic job list.","commit_id":"6d5233a026f70dfcde0664c18e1e4b26df47f36f"}]}