)]}'
{"jenkins/jobs/projects.yaml":[{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"0cf3587bfe61ea71ce48f86b4f88e607aaa432f8","unresolved":false,"context_lines":[{"line_number":5780,"context_line":"          pipeline: gate"},{"line_number":5781,"context_line":"          node: ubuntu-xenial-2-node"},{"line_number":5782,"context_line":"          node-release: ubuntu-xenial"},{"line_number":5783,"context_line":"          suffix: \u0027\u0027"},{"line_number":5784,"context_line":"          branch-override: default"},{"line_number":5785,"context_line":"      - \u0027{pipeline}-ironic-dsvm-standalone-{node}{suffix}\u0027:"},{"line_number":5786,"context_line":"          pipeline: gate"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"7f231b9d_50cff428","line":5783,"updated":"2017-06-19 21:49:44.000000000","message":"this doesn\u0027t need to change, right? It just means that \u0027-nv\u0027 *could* be a suffix value for this? (cuz eg L5777 and that is voting)\n\nNot that I\u0027m saying not to change it, just that I am somewhat confused or most likely, I don\u0027t understand...","commit_id":"1f2154d590c9d66197aa426d143fedebe24c2bce"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"45adc3c30c1d2d9efedbacc1151ee9f1097e142f","unresolved":false,"context_lines":[{"line_number":5780,"context_line":"          pipeline: gate"},{"line_number":5781,"context_line":"          node: ubuntu-xenial-2-node"},{"line_number":5782,"context_line":"          node-release: ubuntu-xenial"},{"line_number":5783,"context_line":"          suffix: \u0027\u0027"},{"line_number":5784,"context_line":"          branch-override: default"},{"line_number":5785,"context_line":"      - \u0027{pipeline}-ironic-dsvm-standalone-{node}{suffix}\u0027:"},{"line_number":5786,"context_line":"          pipeline: gate"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"5f201791_b834f5df","line":5783,"in_reply_to":"5f201791_15a3b451","updated":"2017-06-20 13:30:50.000000000","message":"This is strange indeed Oo I\u0027ll leave it up for someone from infra to comment on it.","commit_id":"1f2154d590c9d66197aa426d143fedebe24c2bce"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"5c41f19159fea103a2a3c1ba7205838ba72b3040","unresolved":false,"context_lines":[{"line_number":5780,"context_line":"          pipeline: gate"},{"line_number":5781,"context_line":"          node: ubuntu-xenial-2-node"},{"line_number":5782,"context_line":"          node-release: ubuntu-xenial"},{"line_number":5783,"context_line":"          suffix: \u0027\u0027"},{"line_number":5784,"context_line":"          branch-override: default"},{"line_number":5785,"context_line":"      - \u0027{pipeline}-ironic-dsvm-standalone-{node}{suffix}\u0027:"},{"line_number":5786,"context_line":"          pipeline: gate"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"5f201791_f7f5d384","line":5783,"in_reply_to":"5f201791_17696fbd","updated":"2017-06-20 18:17:02.000000000","message":"Ruby, please ping me on #openstack-infra and let\u0027s talk. \n\nL3550 does not state voting/non-voting.\n\nWe have a global regex that makes all jobs ending in -nv, non-voting - and the others are voting by default unless explicitely overwritten. And that\u0027s not the case.\n\nso, a job called bla-nv will be non-voting!","commit_id":"1f2154d590c9d66197aa426d143fedebe24c2bce"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"0d211256793c0199a7e90bff973812c14ce6060c","unresolved":false,"context_lines":[{"line_number":5780,"context_line":"          pipeline: gate"},{"line_number":5781,"context_line":"          node: ubuntu-xenial-2-node"},{"line_number":5782,"context_line":"          node-release: ubuntu-xenial"},{"line_number":5783,"context_line":"          suffix: \u0027\u0027"},{"line_number":5784,"context_line":"          branch-override: default"},{"line_number":5785,"context_line":"      - \u0027{pipeline}-ironic-dsvm-standalone-{node}{suffix}\u0027:"},{"line_number":5786,"context_line":"          pipeline: gate"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"5f201791_17696fbd","line":5783,"in_reply_to":"5f201791_548245ca","updated":"2017-06-20 17:59:01.000000000","message":"I took a closer look. in zuul/layout.yaml [0], the *grenade-dsvm-ironic-* job is non-voting @ L3550 (stable/newton), @L3885-3886 (openstack-dev/grenade), but voting @ L10994-10995 and L11010-11011\n\nSo my guess is that the suffix is to indicate what the possible values are.\n\n@John -- should we add the multinode-multitenant job to some of the other non-ironic projects?\n\n[0] https://github.com/openstack-infra/project-config/blob/d1cf20a9a6b355e56a5554615704f43242241bb7/zuul/layout.yaml","commit_id":"1f2154d590c9d66197aa426d143fedebe24c2bce"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"7722e619fcbc883de836cf21a8fa9cb2217b415e","unresolved":false,"context_lines":[{"line_number":5780,"context_line":"          pipeline: gate"},{"line_number":5781,"context_line":"          node: ubuntu-xenial-2-node"},{"line_number":5782,"context_line":"          node-release: ubuntu-xenial"},{"line_number":5783,"context_line":"          suffix: \u0027\u0027"},{"line_number":5784,"context_line":"          branch-override: default"},{"line_number":5785,"context_line":"      - \u0027{pipeline}-ironic-dsvm-standalone-{node}{suffix}\u0027:"},{"line_number":5786,"context_line":"          pipeline: gate"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"5f201791_548245ca","line":5783,"in_reply_to":"5f201791_b834f5df","updated":"2017-06-20 17:42:45.000000000","message":"Why do you say the -nv job is voting? It should not.","commit_id":"1f2154d590c9d66197aa426d143fedebe24c2bce"},{"author":{"_account_id":14760,"name":"John L. Villalovos","email":"openstack.org@sodarock.com","username":"jlvillal"},"change_message_id":"58951ce909fbbaaac8cb1983c2143c2d9313c353","unresolved":false,"context_lines":[{"line_number":5780,"context_line":"          pipeline: gate"},{"line_number":5781,"context_line":"          node: ubuntu-xenial-2-node"},{"line_number":5782,"context_line":"          node-release: ubuntu-xenial"},{"line_number":5783,"context_line":"          suffix: \u0027\u0027"},{"line_number":5784,"context_line":"          branch-override: default"},{"line_number":5785,"context_line":"      - \u0027{pipeline}-ironic-dsvm-standalone-{node}{suffix}\u0027:"},{"line_number":5786,"context_line":"          pipeline: gate"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"7f231b9d_ab16d9f0","line":5783,"in_reply_to":"7f231b9d_50cff428","updated":"2017-06-19 22:03:14.000000000","message":"I looked at how other jobs changed it and they changed the suffix to \u0027\u0027 from \u0027-nv\u0027\n\nBut I\u0027ll be honest and admit I don\u0027t know if it is required or not. Though it appears to be not required based on what you pointed out.","commit_id":"1f2154d590c9d66197aa426d143fedebe24c2bce"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"bd17946b18e94aefec99448ea760af9320b145cc","unresolved":false,"context_lines":[{"line_number":5780,"context_line":"          pipeline: gate"},{"line_number":5781,"context_line":"          node: ubuntu-xenial-2-node"},{"line_number":5782,"context_line":"          node-release: ubuntu-xenial"},{"line_number":5783,"context_line":"          suffix: \u0027\u0027"},{"line_number":5784,"context_line":"          branch-override: default"},{"line_number":5785,"context_line":"      - \u0027{pipeline}-ironic-dsvm-standalone-{node}{suffix}\u0027:"},{"line_number":5786,"context_line":"          pipeline: gate"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"7f231b9d_ca3585aa","line":5783,"in_reply_to":"7f231b9d_ab16d9f0","updated":"2017-06-20 07:30:00.000000000","message":"Ruby, this is the template to generate jobs. If we keep -nv here, only a job with the -nv suffix will be generated. And the -nv suffix makes a job implicitly non-voting.","commit_id":"1f2154d590c9d66197aa426d143fedebe24c2bce"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"a9b22bf24de21b0eba8bea95d9508a38bb1ffcf5","unresolved":false,"context_lines":[{"line_number":5780,"context_line":"          pipeline: gate"},{"line_number":5781,"context_line":"          node: ubuntu-xenial-2-node"},{"line_number":5782,"context_line":"          node-release: ubuntu-xenial"},{"line_number":5783,"context_line":"          suffix: \u0027\u0027"},{"line_number":5784,"context_line":"          branch-override: default"},{"line_number":5785,"context_line":"      - \u0027{pipeline}-ironic-dsvm-standalone-{node}{suffix}\u0027:"},{"line_number":5786,"context_line":"          pipeline: gate"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"5f201791_15a3b451","line":5783,"in_reply_to":"7f231b9d_ca3585aa","updated":"2017-06-20 13:22:51.000000000","message":"That\u0027s what I had thought too, but if that is the case, then why does the grenade-dsvm-ironic-* test (L5772) have suffix: \u0027-nv\u0027 (L5777), yet it is a voting job? That leads me to think that there is 1. a bug, or 2. that it just indicates the possible values.","commit_id":"1f2154d590c9d66197aa426d143fedebe24c2bce"}],"zuul/layout.yaml":[{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"0cf3587bfe61ea71ce48f86b4f88e607aaa432f8","unresolved":false,"context_lines":[{"line_number":10971,"context_line":"      - ironic-coverage-ubuntu-xenial"},{"line_number":10972,"context_line":"      - gate-tempest-dsvm-ironic-multitenant-network-ubuntu-xenial"},{"line_number":10973,"context_line":"      - gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ipmitool-tinyipa-multinode-ubuntu-xenial"},{"line_number":10974,"context_line":"      - gate-grenade-dsvm-ironic-multinode-multitenant-ubuntu-xenial"},{"line_number":10975,"context_line":"    post:"},{"line_number":10976,"context_line":"      - ironic-coverage-ubuntu-trusty"},{"line_number":10977,"context_line":"      - ironic-coverage-ubuntu-xenial"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"7f231b9d_f0c6004a","line":10974,"updated":"2017-06-19 21:49:44.000000000","message":"just a nit although there doesn\u0027t seem to be any order here, but it would be nice to group this with the other grenade tests.","commit_id":"1f2154d590c9d66197aa426d143fedebe24c2bce"},{"author":{"_account_id":14760,"name":"John L. Villalovos","email":"openstack.org@sodarock.com","username":"jlvillal"},"change_message_id":"58951ce909fbbaaac8cb1983c2143c2d9313c353","unresolved":false,"context_lines":[{"line_number":10971,"context_line":"      - ironic-coverage-ubuntu-xenial"},{"line_number":10972,"context_line":"      - gate-tempest-dsvm-ironic-multitenant-network-ubuntu-xenial"},{"line_number":10973,"context_line":"      - gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ipmitool-tinyipa-multinode-ubuntu-xenial"},{"line_number":10974,"context_line":"      - gate-grenade-dsvm-ironic-multinode-multitenant-ubuntu-xenial"},{"line_number":10975,"context_line":"    post:"},{"line_number":10976,"context_line":"      - ironic-coverage-ubuntu-trusty"},{"line_number":10977,"context_line":"      - ironic-coverage-ubuntu-xenial"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"7f231b9d_2bb9a9e8","line":10974,"in_reply_to":"7f231b9d_f0c6004a","updated":"2017-06-19 22:03:14.000000000","message":"Done","commit_id":"1f2154d590c9d66197aa426d143fedebe24c2bce"}]}