)]}'
{"id":"openstack%2Fopenstack-helm-infra~784536","triplet_id":"openstack%2Fopenstack-helm-infra~master~I1582d008217b8848103579b826fae065c538aaf0","project":"openstack/openstack-helm-infra","branch":"master","topic":"htk-brain-hurty-shenanigans","hashtags":[],"change_id":"I1582d008217b8848103579b826fae065c538aaf0","subject":"[htk] Jobs; put labels only in the template spec","status":"MERGED","created":"2021-04-02 06:32:14.000000000","updated":"2021-04-05 19:48:16.000000000","submitted":"2021-04-05 19:46:58.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":8,"unresolved_comment_count":7,"has_review_started":true,"submission_id":"784536-htk-brain-hurty-shenanigans","meta_rev_id":"5f11e65391fdeee9c7e072f56398163b0911e8d0","_number":784536,"virtual_id_number":784536,"owner":{"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"value":0,"_account_id":30449,"name":"jinyuanliu","email":"liujinyuan@inspur.com","username":"jinyuanliu"},{"value":0,"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},{"value":0,"_account_id":23140,"name":"Jess Egler","email":"jess.egler@gmail.com","username":"jessegler"},{"value":0,"_account_id":28372,"name":"chinasubbareddy mallavarapu","email":"cr3938@att.com","username":"chinasubbareddy"},{"value":0,"_account_id":18250,"name":"Roy Tang","email":"roy.s.tang@att.com","username":"rt7380"},{"value":0,"_account_id":24780,"name":"Sangeet Gupta","email":"sg774j@att.com","username":"sgupta"},{"value":0,"_account_id":22636,"name":"Cliff Parsons","email":"cliffhparsons@aol.com","username":"cliffparsons"},{"tag":"autogenerated:zuul:gate","value":2,"date":"2021-04-05 19:46:58.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":28719,"name":"Phil Sphicas","email":"phil.sphicas@att.com","username":"ps3910"},{"value":0,"_account_id":18511,"name":"Brian Wickersham","email":"bkw86@bellsouth.net","username":"bw6938"},{"value":0,"_account_id":19391,"name":"Chi Lo","email":"cl566n@att.com","username":"cl566n"},{"value":0,"_account_id":30495,"name":"Nafiz Haider","email":"nafizhaider22@gmail.com","username":"nafh27"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":28372,"name":"chinasubbareddy mallavarapu","email":"cr3938@att.com","username":"chinasubbareddy"},"recommended":{"_account_id":23140,"name":"Jess Egler","email":"jess.egler@gmail.com","username":"jessegler"},"all":[{"value":1,"date":"2021-04-05 10:51:50.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":30449,"name":"jinyuanliu","email":"liujinyuan@inspur.com","username":"jinyuanliu"},{"value":0,"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},{"value":1,"date":"2021-04-05 14:47:06.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":23140,"name":"Jess Egler","email":"jess.egler@gmail.com","username":"jessegler"},{"value":2,"date":"2021-04-05 16:51:40.000000000","_account_id":28372,"name":"chinasubbareddy mallavarapu","email":"cr3938@att.com","username":"chinasubbareddy"},{"value":2,"date":"2021-04-04 19:25:27.000000000","_account_id":18250,"name":"Roy Tang","email":"roy.s.tang@att.com","username":"rt7380"},{"value":1,"date":"2021-04-05 12:02:34.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":24780,"name":"Sangeet Gupta","email":"sg774j@att.com","username":"sgupta"},{"value":1,"date":"2021-04-05 14:33:56.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":22636,"name":"Cliff Parsons","email":"cliffhparsons@aol.com","username":"cliffparsons"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":1,"date":"2021-04-04 06:53:51.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":28719,"name":"Phil Sphicas","email":"phil.sphicas@att.com","username":"ps3910"},{"value":1,"date":"2021-04-05 13:21:06.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":18511,"name":"Brian Wickersham","email":"bkw86@bellsouth.net","username":"bw6938"},{"value":1,"date":"2021-04-05 14:30:14.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":19391,"name":"Chi Lo","email":"cl566n@att.com","username":"cl566n"},{"value":1,"date":"2021-04-05 14:45:55.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":30495,"name":"Nafiz Haider","email":"nafizhaider22@gmail.com","username":"nafh27"}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","value":1,"default_value":0,"optional":true},"Workflow":{"approved":{"_account_id":28372,"name":"chinasubbareddy mallavarapu","email":"cr3938@att.com","username":"chinasubbareddy"},"all":[{"value":0,"_account_id":30449,"name":"jinyuanliu","email":"liujinyuan@inspur.com","username":"jinyuanliu"},{"value":0,"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},{"value":0,"_account_id":23140,"name":"Jess Egler","email":"jess.egler@gmail.com","username":"jessegler"},{"value":1,"date":"2021-04-05 16:51:40.000000000","_account_id":28372,"name":"chinasubbareddy mallavarapu","email":"cr3938@att.com","username":"chinasubbareddy"},{"value":0,"_account_id":18250,"name":"Roy Tang","email":"roy.s.tang@att.com","username":"rt7380"},{"value":0,"_account_id":24780,"name":"Sangeet Gupta","email":"sg774j@att.com","username":"sgupta"},{"value":0,"_account_id":22636,"name":"Cliff Parsons","email":"cliffhparsons@aol.com","username":"cliffparsons"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":28719,"name":"Phil Sphicas","email":"phil.sphicas@att.com","username":"ps3910"},{"value":0,"_account_id":18511,"name":"Brian Wickersham","email":"bkw86@bellsouth.net","username":"bw6938"},{"value":0,"_account_id":19391,"name":"Chi Lo","email":"cl566n@att.com","username":"cl566n"},{"value":0,"_account_id":30495,"name":"Nafiz Haider","email":"nafizhaider22@gmail.com","username":"nafh27"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[{"_account_id":31746,"name":"Oleksandr Kozachenko","email":"okozachenko1203@gmail.com","username":"okozachenko"}],"reviewers":{"REVIEWER":[{"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},{"_account_id":18250,"name":"Roy Tang","email":"roy.s.tang@att.com","username":"rt7380"},{"_account_id":18511,"name":"Brian Wickersham","email":"bkw86@bellsouth.net","username":"bw6938"},{"_account_id":19391,"name":"Chi Lo","email":"cl566n@att.com","username":"cl566n"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":22636,"name":"Cliff Parsons","email":"cliffhparsons@aol.com","username":"cliffparsons"},{"_account_id":23140,"name":"Jess Egler","email":"jess.egler@gmail.com","username":"jessegler"},{"_account_id":24780,"name":"Sangeet Gupta","email":"sg774j@att.com","username":"sgupta"},{"_account_id":28372,"name":"chinasubbareddy mallavarapu","email":"cr3938@att.com","username":"chinasubbareddy"},{"_account_id":28719,"name":"Phil Sphicas","email":"phil.sphicas@att.com","username":"ps3910"},{"_account_id":30449,"name":"jinyuanliu","email":"liujinyuan@inspur.com","username":"jinyuanliu"},{"_account_id":30495,"name":"Nafiz Haider","email":"nafizhaider22@gmail.com","username":"nafh27"},{"_account_id":31746,"name":"Oleksandr Kozachenko","email":"okozachenko1203@gmail.com","username":"okozachenko"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2021-04-02 06:34:06.000000000","updated_by":{"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},"reviewer":{"_account_id":28719,"name":"Phil Sphicas","email":"phil.sphicas@att.com","username":"ps3910"},"state":"REVIEWER"},{"updated":"2021-04-02 06:34:06.000000000","updated_by":{"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},"reviewer":{"_account_id":31746,"name":"Oleksandr Kozachenko","email":"okozachenko1203@gmail.com","username":"okozachenko"},"state":"REVIEWER"},{"updated":"2021-04-02 07:30:03.000000000","updated_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"reviewer":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2021-04-02 21:17:05.000000000","updated_by":{"_account_id":18250,"name":"Roy Tang","email":"roy.s.tang@att.com","username":"rt7380"},"reviewer":{"_account_id":18250,"name":"Roy Tang","email":"roy.s.tang@att.com","username":"rt7380"},"state":"REVIEWER"},{"updated":"2021-04-02 21:35:52.000000000","updated_by":{"_account_id":24780,"name":"Sangeet Gupta","email":"sg774j@att.com","username":"sgupta"},"reviewer":{"_account_id":24780,"name":"Sangeet Gupta","email":"sg774j@att.com","username":"sgupta"},"state":"REVIEWER"},{"updated":"2021-04-05 10:51:50.000000000","updated_by":{"_account_id":30449,"name":"jinyuanliu","email":"liujinyuan@inspur.com","username":"jinyuanliu"},"reviewer":{"_account_id":30449,"name":"jinyuanliu","email":"liujinyuan@inspur.com","username":"jinyuanliu"},"state":"REVIEWER"},{"updated":"2021-04-05 13:21:06.000000000","updated_by":{"_account_id":18511,"name":"Brian Wickersham","email":"bkw86@bellsouth.net","username":"bw6938"},"reviewer":{"_account_id":18511,"name":"Brian Wickersham","email":"bkw86@bellsouth.net","username":"bw6938"},"state":"REVIEWER"},{"updated":"2021-04-05 14:30:14.000000000","updated_by":{"_account_id":19391,"name":"Chi Lo","email":"cl566n@att.com","username":"cl566n"},"reviewer":{"_account_id":19391,"name":"Chi Lo","email":"cl566n@att.com","username":"cl566n"},"state":"REVIEWER"},{"updated":"2021-04-05 14:33:56.000000000","updated_by":{"_account_id":22636,"name":"Cliff Parsons","email":"cliffhparsons@aol.com","username":"cliffparsons"},"reviewer":{"_account_id":22636,"name":"Cliff Parsons","email":"cliffhparsons@aol.com","username":"cliffparsons"},"state":"REVIEWER"},{"updated":"2021-04-05 14:45:55.000000000","updated_by":{"_account_id":30495,"name":"Nafiz Haider","email":"nafizhaider22@gmail.com","username":"nafh27"},"reviewer":{"_account_id":30495,"name":"Nafiz Haider","email":"nafizhaider22@gmail.com","username":"nafh27"},"state":"REVIEWER"},{"updated":"2021-04-05 14:47:06.000000000","updated_by":{"_account_id":23140,"name":"Jess Egler","email":"jess.egler@gmail.com","username":"jessegler"},"reviewer":{"_account_id":23140,"name":"Jess Egler","email":"jess.egler@gmail.com","username":"jessegler"},"state":"REVIEWER"},{"updated":"2021-04-05 16:51:40.000000000","updated_by":{"_account_id":28372,"name":"chinasubbareddy mallavarapu","email":"cr3938@att.com","username":"chinasubbareddy"},"reviewer":{"_account_id":28372,"name":"chinasubbareddy mallavarapu","email":"cr3938@att.com","username":"chinasubbareddy"},"state":"REVIEWER"}],"messages":[{"id":"3b9a6528be81c55e53b29121051d763c569774e0","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},"date":"2021-04-02 06:32:14.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"c99a91ffffc3705432e70ba440998d576113a0b9","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},"date":"2021-04-02 06:37:08.000000000","message":"Uploaded patch set 2: Commit message was updated.","accounts_in_message":[],"_revision_number":2},{"id":"128f50fa264e696a798b1d3a486868069c2520c3","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-04-02 07:30:03.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/67c9ccb150014876bbf713c5e52d9c84 : SUCCESS in 4m 59s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/ff996939da034916aa2e2758d235809f : SUCCESS in 4m 58s\n- openstack-helm-lint https://zuul.opendev.org/t/openstack/build/4345d8df514b4d318ce47fd49ddaee8c : SUCCESS in 4m 28s\n- openstack-helm-infra-aio-logging https://zuul.opendev.org/t/openstack/build/aca3a915b8f24d029918987644f1b96a : SUCCESS in 38m 16s\n- openstack-helm-infra-aio-monitoring https://zuul.opendev.org/t/openstack/build/46184791443a432bafb457d630ed0e00 : SUCCESS in 33m 04s\n- openstack-helm-infra-openstack-support https://zuul.opendev.org/t/openstack/build/982d70517097491ea547e7db0878700f : SUCCESS in 47m 22s\n- openstack-helm-infra-openstack-support-ssl https://zuul.opendev.org/t/openstack/build/a02e39d545264238b135cd79f3b79e44 : SUCCESS in 46m 39s\n- openstack-helm-infra-metacontroller https://zuul.opendev.org/t/openstack/build/76ad693d6aa04159ab17defe41e7047f : SUCCESS in 14m 22s","accounts_in_message":[],"_revision_number":2},{"id":"ac6586a71437df3fb804f6249c83bc7553b40fe4","author":{"_account_id":28719,"name":"Phil Sphicas","email":"phil.sphicas@att.com","username":"ps3910"},"date":"2021-04-02 15:13:33.000000000","message":"Patch Set 2: Code-Review-1\n\n(2 comments)\n\nI\u0027d prefer to see the top-level .metadata.labels key removed from the jobs, and any additional labels added to .spec.template.metadata.labels.\n\nWhen I test creating a job, I see the job-name and controller-uid labels automatically added in both places, but only if .metadata.labels does not exist. I think expecting those labels to exist and match between the job and the created pods is reasonable.\n\nCompare:\n\n    kubectl create job expected-labels --image\u003dbusybox --dry-run -o json -- sleep 60 |\n      jq \u0027.spec.template.metadata.labels.my_label\u003d\"test\"\u0027 |\n        kubectl create -f- -o json |\n          jq \u0027{ job_labels: .metadata.labels, pod_labels: .spec.template.metadata.labels }\u0027\n    {\n      \"job_labels\": {\n        \"controller-uid\": \"4f9889c4-f09d-442c-8adc-a1a0d8b15304\",\n        \"job-name\": \"expected-labels\",\n        \"my_label\": \"test\"\n      },\n      \"pod_labels\": {\n        \"controller-uid\": \"4f9889c4-f09d-442c-8adc-a1a0d8b15304\",\n        \"job-name\": \"expected-labels\",\n        \"my_label\": \"test\"\n      }\n    }\n    \n    kubectl create job weird-labels --image\u003dbusybox --dry-run -o json -- sleep 60 |\n      jq \u0027.metadata.labels.my_label\u003d\"test\"\u0027 |\n        kubectl create -f- -o json |\n          jq \u0027{ job_labels: .metadata.labels, pod_labels: .spec.template.metadata.labels }\u0027\n    {\n      \"job_labels\": {\n        \"my_label\": \"test\"\n      },\n      \"pod_labels\": {\n        \"controller-uid\": \"d02c92ad-cfa2-481b-8fb9-31abb222492b\",\n        \"job-name\": \"weird-labels\"\n      }\n    }","accounts_in_message":[],"_revision_number":2},{"id":"0565d16b276660e679032d4babe841ce0683fee3","author":{"_account_id":28719,"name":"Phil Sphicas","email":"phil.sphicas@att.com","username":"ps3910"},"date":"2021-04-02 20:51:06.000000000","message":"Patch Set 2:\n\n(1 comment)","accounts_in_message":[],"_revision_number":2},{"id":"ce87638882a064876e821b94908e234e406ee2d5","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},"date":"2021-04-02 21:11:30.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"9c9362881e42f621048123fc5c675c7bfd2b89e6","author":{"_account_id":28719,"name":"Phil Sphicas","email":"phil.sphicas@att.com","username":"ps3910"},"date":"2021-04-02 21:13:18.000000000","message":"Patch Set 3: Code-Review+1","accounts_in_message":[],"_revision_number":3},{"id":"5c62d62eed692c017c851d16c049e09e1ffc7bb9","author":{"_account_id":18250,"name":"Roy Tang","email":"roy.s.tang@att.com","username":"rt7380"},"date":"2021-04-02 21:17:05.000000000","message":"Patch Set 3: Code-Review+2","accounts_in_message":[],"_revision_number":3},{"id":"c4c33dda42a0573f5ce308cb5aff1777978e02d7","author":{"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},"date":"2021-04-02 21:17:14.000000000","message":"Patch Set 3: Code-Review-1\n\n(1 comment)","accounts_in_message":[],"_revision_number":3},{"id":"0b1f207ff41bab7eee6e832e2b91903e8ab14f4d","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},"date":"2021-04-02 21:17:37.000000000","message":"Uploaded patch set 4: Commit message was updated.","accounts_in_message":[],"_revision_number":4},{"id":"921dac1fe93bf50d51e8d0ee64794ff9513a4c01","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},"date":"2021-04-02 21:20:44.000000000","message":"Uploaded patch set 5: Commit message was updated.","accounts_in_message":[],"_revision_number":5},{"id":"73b61dcc76eb7db4e3209c4a196e9d43e045fed5","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},"date":"2021-04-02 21:23:33.000000000","message":"Uploaded patch set 6: Commit message was updated.","accounts_in_message":[],"_revision_number":6},{"id":"89d27e3db28db4df873e98d6ed146e86af950926","author":{"_account_id":24780,"name":"Sangeet Gupta","email":"sg774j@att.com","username":"sgupta"},"date":"2021-04-02 21:35:52.000000000","message":"Patch Set 6: Code-Review+1","accounts_in_message":[],"_revision_number":6},{"id":"20ec2a613ff1d0a6555906e7e4eedf8b07e6a402","author":{"_account_id":28719,"name":"Phil Sphicas","email":"phil.sphicas@att.com","username":"ps3910"},"date":"2021-04-02 21:38:32.000000000","message":"Patch Set 6:\n\n(2 comments)\n\nnits, if you want","accounts_in_message":[],"_revision_number":6},{"id":"cb1e48c885bf37f253915747d527e63740203b2a","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},"date":"2021-04-02 21:49:59.000000000","message":"Uploaded patch set 7: Commit message was updated.","accounts_in_message":[],"_revision_number":7},{"id":"754aa7057cbac23de62e3fe01c8443b7ee35c9fc","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},"date":"2021-04-02 21:52:44.000000000","message":"Uploaded patch set 8: Commit message was updated.","accounts_in_message":[],"_revision_number":8},{"id":"4c9854e7246b9395ad103adcdcd648c5f8863f9a","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},"date":"2021-04-02 21:54:11.000000000","message":"Uploaded patch set 9: Commit message was updated.","accounts_in_message":[],"_revision_number":9},{"id":"8d13488e594889bc438ba2771049824173217ab2","author":{"_account_id":28719,"name":"Phil Sphicas","email":"phil.sphicas@att.com","username":"ps3910"},"date":"2021-04-02 21:59:57.000000000","message":"Patch Set 9:\n\n(2 comments)\n\nnit again","accounts_in_message":[],"_revision_number":9},{"id":"cd37c361bf6dccd00bc1803bee7f3c55b6cbc0fd","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-04-02 22:48:40.000000000","message":"Patch Set 9: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/d7fc9ed8718c4a46847f9df3b070d8bf : SUCCESS in 4m 20s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/247280ed688c495ba11006ce0a3bc115 : SUCCESS in 4m 07s\n- openstack-helm-lint https://zuul.opendev.org/t/openstack/build/bdba4ad183434a3eaf8bfe939cd158a8 : SUCCESS in 4m 55s\n- openstack-helm-infra-aio-logging https://zuul.opendev.org/t/openstack/build/e7af6ce12da24a58b86b1a92f6376909 : SUCCESS in 37m 42s\n- openstack-helm-infra-aio-monitoring https://zuul.opendev.org/t/openstack/build/0fc9143e0ceb4a08abac70070532ba7a : SUCCESS in 30m 34s\n- openstack-helm-infra-openstack-support https://zuul.opendev.org/t/openstack/build/27a5ab8a1ea64a96986a2333a9b2aea5 : SUCCESS in 46m 46s\n- openstack-helm-infra-openstack-support-ssl https://zuul.opendev.org/t/openstack/build/0621b0ea500e44a2b709a388dc5e8145 : SUCCESS in 48m 35s\n- openstack-helm-infra-metacontroller https://zuul.opendev.org/t/openstack/build/4d2a1afa91f04f579790b6f3bc45fff6 : SUCCESS in 14m 26s","accounts_in_message":[],"_revision_number":9},{"id":"43de346adf116428408150c3e166eab7bcecedce","author":{"_account_id":28719,"name":"Phil Sphicas","email":"phil.sphicas@att.com","username":"ps3910"},"date":"2021-04-04 06:53:51.000000000","message":"Patch Set 9: Code-Review+1","accounts_in_message":[],"_revision_number":9},{"id":"84dedd2a519d391ae741e34c234b9f5965e60d65","author":{"_account_id":18250,"name":"Roy Tang","email":"roy.s.tang@att.com","username":"rt7380"},"date":"2021-04-04 19:25:27.000000000","message":"Patch Set 9: Code-Review+2","accounts_in_message":[],"_revision_number":9},{"id":"1947c21916b7f317a0376cd5e4013d207f99718e","author":{"_account_id":30449,"name":"jinyuanliu","email":"liujinyuan@inspur.com","username":"jinyuanliu"},"date":"2021-04-05 10:51:50.000000000","message":"Patch Set 9: Code-Review+1","accounts_in_message":[],"_revision_number":9},{"id":"23853a1ffd8fcad082fce628cb81cd314fe07dfb","author":{"_account_id":24780,"name":"Sangeet Gupta","email":"sg774j@att.com","username":"sgupta"},"date":"2021-04-05 12:02:34.000000000","message":"Patch Set 9: Code-Review+1","accounts_in_message":[],"_revision_number":9},{"id":"c807dededec723d5b05cbc699560b1c766e7ed95","author":{"_account_id":18511,"name":"Brian Wickersham","email":"bkw86@bellsouth.net","username":"bw6938"},"date":"2021-04-05 13:21:06.000000000","message":"Patch Set 9: Code-Review+1","accounts_in_message":[],"_revision_number":9},{"id":"2869969699a1197c077003ae5e630bbae1a2a84f","author":{"_account_id":19391,"name":"Chi Lo","email":"cl566n@att.com","username":"cl566n"},"date":"2021-04-05 14:30:14.000000000","message":"Patch Set 9: Code-Review+1","accounts_in_message":[],"_revision_number":9},{"id":"e15638087d5062b80c25c774d508a17fa0e92b84","author":{"_account_id":22636,"name":"Cliff Parsons","email":"cliffhparsons@aol.com","username":"cliffparsons"},"date":"2021-04-05 14:33:56.000000000","message":"Patch Set 9: Code-Review+1","accounts_in_message":[],"_revision_number":9},{"id":"22780bf6d55d68ba94e44b389f7053d0816e0cf3","author":{"_account_id":30495,"name":"Nafiz Haider","email":"nafizhaider22@gmail.com","username":"nafh27"},"date":"2021-04-05 14:45:55.000000000","message":"Patch Set 9: Code-Review+1","accounts_in_message":[],"_revision_number":9},{"id":"5b3e855ed43ce0fdf6f442ad9f39622f970efd74","author":{"_account_id":23140,"name":"Jess Egler","email":"jess.egler@gmail.com","username":"jessegler"},"date":"2021-04-05 14:47:06.000000000","message":"Patch Set 9: Code-Review+1","accounts_in_message":[],"_revision_number":9},{"id":"be23562f6a4f07fa421e93746582203c6be2aeee","author":{"_account_id":28372,"name":"chinasubbareddy mallavarapu","email":"cr3938@att.com","username":"chinasubbareddy"},"date":"2021-04-05 16:51:40.000000000","message":"Patch Set 9: Code-Review+2 Workflow+1","accounts_in_message":[],"_revision_number":9},{"id":"965bca499598dab4a4c6b40147eb6c74885fdd33","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-04-05 16:51:57.000000000","message":"Patch Set 9: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":9},{"id":"29605079faa4bf717618831a2ad247504225e053","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-04-05 17:43:27.000000000","message":"Patch Set 9: Verified-2\n\nBuild failed (gate pipeline).  For information on how to proceed, see\nhttps://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing\n\n\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/d7f2e6fe21364a9e8132395e94f5c620 : SUCCESS in 5m 34s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/32f12916e106485fbeddffeedcfacbf5 : SUCCESS in 5m 48s\n- openstack-helm-lint https://zuul.opendev.org/t/openstack/build/f9dc424a47764192a369f34042d79ec6 : SUCCESS in 5m 29s\n- openstack-helm-infra-aio-logging https://zuul.opendev.org/t/openstack/build/e0d0f5c1e2bc4d5c9d00c14b5712c20c : SUCCESS in 36m 15s\n- openstack-helm-infra-aio-monitoring https://zuul.opendev.org/t/openstack/build/e08a581aba1944e08f6a9995f5a10edd : SUCCESS in 31m 07s\n- openstack-helm-infra-openstack-support https://zuul.opendev.org/t/openstack/build/42f8239ba4f749e39e493bdfa030a8b2 : FAILURE in 36m 14s\n- openstack-helm-infra-openstack-support-ssl https://zuul.opendev.org/t/openstack/build/46f30d0984e746b7b2f674deb0e9c9f5 : SUCCESS in 50m 17s","accounts_in_message":[],"_revision_number":9},{"id":"9a40c99fab87cddeb0a7c760f16f98acccbaf595","author":{"_account_id":24780,"name":"Sangeet Gupta","email":"sg774j@att.com","username":"sgupta"},"date":"2021-04-05 17:44:32.000000000","message":"Patch Set 9:\n\nrecheck","accounts_in_message":[],"_revision_number":9},{"id":"068d6c56a08b1044e02305f82f2d269a1deb4a15","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-04-05 18:44:08.000000000","message":"Patch Set 9: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/80c6409d56be45bf9330537d07ae0a3a : SUCCESS in 4m 41s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/677f982b9f9247dba2eb1e4c2cc764b4 : SUCCESS in 3m 32s\n- openstack-helm-lint https://zuul.opendev.org/t/openstack/build/dcb989b38598408cab63d6b13814e3bc : SUCCESS in 4m 05s\n- openstack-helm-infra-aio-logging https://zuul.opendev.org/t/openstack/build/14974cf72ac744d1936216bde1381d14 : SUCCESS in 35m 01s\n- openstack-helm-infra-aio-monitoring https://zuul.opendev.org/t/openstack/build/d5f5ff87a285409ab25e819fa7761855 : SUCCESS in 26m 24s\n- openstack-helm-infra-openstack-support https://zuul.opendev.org/t/openstack/build/b494b8eec7be4b8b8fa9205ac0b0cd3b : SUCCESS in 55m 08s\n- openstack-helm-infra-openstack-support-ssl https://zuul.opendev.org/t/openstack/build/8242d981a4804167b0f288819544585d : SUCCESS in 46m 18s\n- openstack-helm-infra-metacontroller https://zuul.opendev.org/t/openstack/build/e7a4df9aa0c944109f0713c40b086cf2 : SUCCESS in 13m 47s","accounts_in_message":[],"_revision_number":9},{"id":"3603231737725cd097d87a9b3f2c183f71a728cf","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-04-05 18:44:24.000000000","message":"Patch Set 9: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":9},{"id":"9ff72c32a3b7f5adc7672f5264cf13438379a34d","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-04-05 19:46:58.000000000","message":"Patch Set 9: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/4761f020070c416397e149522438c09f : SUCCESS in 3m 31s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/d3445e76c82e48339d929b7474dbbed6 : SUCCESS in 4m 26s\n- openstack-helm-lint https://zuul.opendev.org/t/openstack/build/68927bae0d3049d0a444eca058c21b83 : SUCCESS in 4m 34s\n- openstack-helm-infra-aio-logging https://zuul.opendev.org/t/openstack/build/48a528a927d7451dbc45503162dd6793 : SUCCESS in 32m 23s\n- openstack-helm-infra-aio-monitoring https://zuul.opendev.org/t/openstack/build/9010a517bd2141daa909b9d175685a1a : SUCCESS in 28m 05s\n- openstack-helm-infra-openstack-support https://zuul.opendev.org/t/openstack/build/fcade4ff175b4b00a23d74dd74ad21fd : SUCCESS in 1h 00m 49s\n- openstack-helm-infra-openstack-support-ssl https://zuul.opendev.org/t/openstack/build/db6c6b93cef244a39d9fee4608abf774 : SUCCESS in 57m 35s","accounts_in_message":[],"_revision_number":9},{"id":"90efe73fdba9d04592e2b8bb116cbad30b35e155","tag":"autogenerated:gerrit:merged","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-04-05 19:46:58.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":9},{"id":"5f11e65391fdeee9c7e072f56398163b0911e8d0","tag":"autogenerated:zuul:promote","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-04-05 19:48:16.000000000","message":"Patch Set 9:\n\nBuild succeeded (promote pipeline).\n\n- promote-openstack-tox-docs https://zuul.opendev.org/t/openstack/build/77e835fbd0ed410b8e9128faf5c9a22a : SUCCESS in 56s\n- promote-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/0164b3e6f3f14062b5a7a9be5614c166 : SUCCESS in 49s","accounts_in_message":[],"_revision_number":9}],"current_revision_number":9,"current_revision":"20cf2db961a05285a751b3bc96b8fa4e51e3b412","revisions":{"6050696ef6ea7a6401e01a1f3341ec576808c142":{"kind":"REWORK","_number":1,"created":"2021-04-02 06:32:14.000000000","uploader":{"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},"ref":"refs/changes/36/784536/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-helm-infra","ref":"refs/changes/36/784536/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/36/784536/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/36/784536/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/36/784536/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-helm-infra refs/changes/36/784536/1"}}},"commit":{"parents":[{"commit":"7351586a7d4700e27dceea3b2004d535a48c6ae6","subject":"Merge \"Allow Ceph RBD pool job to leave failed pods\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/7351586a7d4700e27dceea3b2004d535a48c6ae6"}]}],"author":{"name":"Chris Wedgwood","email":"cw@f00f.org","date":"2021-04-02 06:02:36.000000000","tz":-300},"committer":{"name":"Chris Wedgwood","email":"cw@f00f.org","date":"2021-04-02 06:31:56.000000000","tz":-300},"subject":"[htk] Explicitly propagate template labels into job labels","message":"[htk] Explicitly propagate template labels into job labels\n\nThis is an update to address a behavior change introduced with\n0ae8f4d21ac2a091f1612e50f4786da5065d4398.\n\nPrior to that change we did not use explicit labels in job objects.\nWhen no labels are given Kubernetes will propagate the labels up from\nthe template.\n\nIf (any) labels are specified on the job we do not get this behavior,\nwe only get the specified labels.\n\nSpecifically:\n\n    apiVersion: batch/v1\n    kind: Job\n    metadata:\n      # no \"labels:\" here\n      name: placement-db-init\n      namespace: openstack\n    spec:\n      template:\n        metadata:\n          labels:\n            application: placement\n            component: db-init\n            job-name: placement-db-init\n            release_group: placement\n        spec:\n          containers:\n          # do stuffs\n\nends up:\n\n    apiVersion: batch/v1\n    kind: Job\n    metadata:\n      # k8s did this for us!\n      labels:\n        application: placement\n        component: db-init\n        job-name: placement-db-init\n        release_group: placement\n      name: placement-db-init\n      namespace: openstack\n    spec:\n      template:\n        metadata:\n          labels:\n            application: placement\n            component: db-init\n            job-name: placement-db-init\n            release_group: placement\n        spec:\n          containers:\n          # do stuffs\n\nThe aforementioned causes objects to look like:\n\n    apiVersion: batch/v1\n    kind: Job\n    metadata:\n      # k8s did this for us!\n      labels:\n        application: placement\n\t# nothing else!\n      name: placement-db-init\n      namespace: openstack\n    spec:\n      template:\n        metadata:\n          labels:\n            application: placement\n            component: db-init\n            job-name: placement-db-init\n            release_group: placement\n        spec:\n          containers:\n          # do stuffs\n\nMany users have (unintentionally) relied on this behavior for a long\ntime.  Some deployment systems use job labels for synchronization,\nthose labels were only specified in the template and propagated up.\n\nThis should preserve functionality added recently as well as provide\nthe job labels explicitly so that provisioning systems which make use\nof them can continue to do so.\n\nAdditionally we no longer rely on the (non-obvious) behavior where if\nno job labels are given the are taken from the template.\n\nThe explicit \"application\" label is no longer needed as the\nhelm-toolkit.snippets.kubernetes_metadata_labels macro provides it.\n\nChange-Id: I1582d008217b8848103579b826fae065c538aaf0\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/6050696ef6ea7a6401e01a1f3341ec576808c142"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/6050696ef6ea7a6401e01a1f3341ec576808c142"}]},"branch":"refs/heads/master"},"660056f4dbab2b07adc5a88e7654c8c7ce4ee64f":{"kind":"NO_CODE_CHANGE","_number":2,"created":"2021-04-02 06:37:08.000000000","uploader":{"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},"ref":"refs/changes/36/784536/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-helm-infra","ref":"refs/changes/36/784536/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/36/784536/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/36/784536/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/36/784536/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-helm-infra refs/changes/36/784536/2"}}},"commit":{"parents":[{"commit":"7351586a7d4700e27dceea3b2004d535a48c6ae6","subject":"Merge \"Allow Ceph RBD pool job to leave failed pods\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/7351586a7d4700e27dceea3b2004d535a48c6ae6"}]}],"author":{"name":"Chris Wedgwood","email":"cw@f00f.org","date":"2021-04-02 06:02:36.000000000","tz":-300},"committer":{"name":"Chris Wedgwood","email":"cw@f00f.org","date":"2021-04-02 06:36:55.000000000","tz":-300},"subject":"[RFC/DNM] [htk] Explicitly propagate template labels into job labels","message":"[RFC/DNM] [htk] Explicitly propagate template labels into job labels\n\nThis is an update to address a behavior change introduced with\n0ae8f4d21ac2a091f1612e50f4786da5065d4398.\n\nPrior to that change we did not use explicit labels in job objects.\nWhen no labels are given Kubernetes will propagate the labels up from\nthe template.\n\nIf (any) labels are specified on the job we do not get this behavior,\nwe only get the specified labels.\n\nSpecifically:\n\n    apiVersion: batch/v1\n    kind: Job\n    metadata:\n      # no \"labels:\" here\n      name: placement-db-init\n      namespace: openstack\n    spec:\n      template:\n        metadata:\n          labels:\n            application: placement\n            component: db-init\n            job-name: placement-db-init\n            release_group: placement\n        spec:\n          containers:\n          # do stuffs\n\nends up:\n\n    apiVersion: batch/v1\n    kind: Job\n    metadata:\n      # k8s did this for us!\n      labels:\n        application: placement\n        component: db-init\n        job-name: placement-db-init\n        release_group: placement\n      name: placement-db-init\n      namespace: openstack\n    spec:\n      template:\n        metadata:\n          labels:\n            application: placement\n            component: db-init\n            job-name: placement-db-init\n            release_group: placement\n        spec:\n          containers:\n          # do stuffs\n\nThe aforementioned causes objects to look like:\n\n    apiVersion: batch/v1\n    kind: Job\n    metadata:\n      # k8s did this for us!\n      labels:\n        application: placement\n\t# nothing else!\n      name: placement-db-init\n      namespace: openstack\n    spec:\n      template:\n        metadata:\n          labels:\n            application: placement\n            component: db-init\n            job-name: placement-db-init\n            release_group: placement\n        spec:\n          containers:\n          # do stuffs\n\nMany users have (unintentionally) relied on this behavior for a long\ntime.  Some deployment systems use job labels for synchronization,\nthose labels were only specified in the template and propagated up.\n\nThis should preserve functionality added recently as well as provide\nthe job labels explicitly so that provisioning systems which make use\nof them can continue to do so.\n\nAdditionally we no longer rely on the (non-obvious) behavior where if\nno job labels are given the are taken from the template.\n\nThe explicit \"application\" label is no longer needed as the\nhelm-toolkit.snippets.kubernetes_metadata_labels macro provides it.\n\nChange-Id: I1582d008217b8848103579b826fae065c538aaf0\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/660056f4dbab2b07adc5a88e7654c8c7ce4ee64f"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/660056f4dbab2b07adc5a88e7654c8c7ce4ee64f"}]},"branch":"refs/heads/master"},"ab391d92acf42be11463815d8447034106a9a3ab":{"kind":"REWORK","_number":3,"created":"2021-04-02 21:11:30.000000000","uploader":{"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},"ref":"refs/changes/36/784536/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-helm-infra","ref":"refs/changes/36/784536/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/36/784536/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/36/784536/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/36/784536/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-helm-infra refs/changes/36/784536/3"}}},"commit":{"parents":[{"commit":"7351586a7d4700e27dceea3b2004d535a48c6ae6","subject":"Merge \"Allow Ceph RBD pool job to leave failed pods\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/7351586a7d4700e27dceea3b2004d535a48c6ae6"}]}],"author":{"name":"Chris Wedgwood","email":"cw@f00f.org","date":"2021-04-02 06:02:36.000000000","tz":-300},"committer":{"name":"Chris Wedgwood","email":"cw@f00f.org","date":"2021-04-02 21:09:01.000000000","tz":-300},"subject":"[htk] Jobs; put labels only in the template spec","message":"[htk] Jobs; put labels only in the template spec\n\nThis is an update to address a behavior change introduced with\n0ae8f4d21ac2a091f1612e50f4786da5065d4398.\n\nJob labels if empty/unspecified are taken from the template.  If (any)\nlabels are specified on the job we do not get this behavior.\n\nSpecifically:\n\n    apiVersion: batch/v1\n    kind: Job\n    metadata:\n      # no \"labels:\" here\n      name: placement-db-init\n      namespace: openstack\n    spec:\n      template:\n        metadata:\n          labels:\n            application: placement\n            component: db-init\n            job-name: placement-db-init\n            release_group: placement\n        spec:\n          containers:\n          # do stuffs\n\nends up:\n\n    apiVersion: batch/v1\n    kind: Job\n    metadata:\n      # k8s did this for us!\n      labels:\n        application: placement\n        component: db-init\n        job-name: placement-db-init\n        release_group: placement\n      name: placement-db-init\n      namespace: openstack\n    spec:\n      template:\n        metadata:\n          labels:\n            application: placement\n            component: db-init\n            job-name: placement-db-init\n            release_group: placement\n        spec:\n          containers:\n          # do stuffs\n\nThe aforementioned causes objects to look like:\n\n    apiVersion: batch/v1\n    kind: Job\n    metadata:\n      # k8s did this for us!\n      labels:\n        application: placement\n\t# nothing else!\n      name: placement-db-init\n      namespace: openstack\n    spec:\n      template:\n        metadata:\n          labels:\n            application: placement\n            component: db-init\n            job-name: placement-db-init\n            release_group: placement\n        spec:\n          containers:\n          # do stuffs\n\nMany users have rely on this behavior.  Some deployment systems use\njob labels for synchronization, those labels were only specified in\nthe template and propagated up.\n\nThis should preserve functionality added recently as well as restore\nthe prevoius behavior.\n\nThe explicit \"application\" label is no longer needed as the\nhelm-toolkit.snippets.kubernetes_metadata_labels macro provides it.\n\nChange-Id: I1582d008217b8848103579b826fae065c538aaf0\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/ab391d92acf42be11463815d8447034106a9a3ab"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/ab391d92acf42be11463815d8447034106a9a3ab"}]},"branch":"refs/heads/master"},"a04cedc01bfeb6e27249d2e5ae7a4234e874e259":{"kind":"NO_CODE_CHANGE","_number":4,"created":"2021-04-02 21:17:37.000000000","uploader":{"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},"ref":"refs/changes/36/784536/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-helm-infra","ref":"refs/changes/36/784536/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/36/784536/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/36/784536/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/36/784536/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-helm-infra refs/changes/36/784536/4"}}},"commit":{"parents":[{"commit":"7351586a7d4700e27dceea3b2004d535a48c6ae6","subject":"Merge \"Allow Ceph RBD pool job to leave failed pods\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/7351586a7d4700e27dceea3b2004d535a48c6ae6"}]}],"author":{"name":"Chris Wedgwood","email":"cw@f00f.org","date":"2021-04-02 06:02:36.000000000","tz":-300},"committer":{"name":"Chris Wedgwood","email":"cw@f00f.org","date":"2021-04-02 21:17:24.000000000","tz":-300},"subject":"[htk] Jobs; put labels only in the template spec","message":"[htk] Jobs; put labels only in the template spec\n\nThis is an update to address a behavior change introduced with\n0ae8f4d21ac2a091f1612e50f4786da5065d4398.\n\nJob labels if empty/unspecified are taken from the template.  If (any)\nlabels are specified on the job we do not get this behavior.\n\nSpecifically:\n\n    apiVersion: batch/v1\n    kind: Job\n    metadata:\n      # no \"labels:\" here\n      name: placement-db-init\n      namespace: openstack\n    spec:\n      template:\n        metadata:\n          labels:\n            application: placement\n            component: db-init\n            job-name: placement-db-init\n            release_group: placement\n        spec:\n          containers:\n          # do stuffs\n\nends up:\n\n    apiVersion: batch/v1\n    kind: Job\n    metadata:\n      # k8s did this for us!\n      labels:\n        application: placement\n        component: db-init\n        job-name: placement-db-init\n        release_group: placement\n      name: placement-db-init\n      namespace: openstack\n    spec:\n      template:\n        metadata:\n          labels:\n            application: placement\n            component: db-init\n            job-name: placement-db-init\n            release_group: placement\n        spec:\n          containers:\n          # do stuffs\n\nThe aforementioned causes objects to look like:\n\n    apiVersion: batch/v1\n    kind: Job\n    metadata:\n      # k8s did this for us!\n      labels:\n        application: placement\n        # nothing else!\n      name: placement-db-init\n      namespace: openstack\n    spec:\n      template:\n        metadata:\n          labels:\n            application: placement\n            component: db-init\n            job-name: placement-db-init\n            release_group: placement\n        spec:\n          containers:\n          # do stuffs\n\nMany users have rely on this behavior.  Some deployment systems use\njob labels for synchronization, those labels were only specified in\nthe template and propagated up.\n\nThis should preserve functionality added recently as well as restore\nthe prevoius behavior.\n\nThe explicit \"application\" label is no longer needed as the\nhelm-toolkit.snippets.kubernetes_metadata_labels macro provides it.\n\nChange-Id: I1582d008217b8848103579b826fae065c538aaf0\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/a04cedc01bfeb6e27249d2e5ae7a4234e874e259"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/a04cedc01bfeb6e27249d2e5ae7a4234e874e259"}]},"branch":"refs/heads/master"},"2fe9a2e39cb0607371c6212ee6a90131581cdbe7":{"kind":"NO_CODE_CHANGE","_number":5,"created":"2021-04-02 21:20:44.000000000","uploader":{"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},"ref":"refs/changes/36/784536/5","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-helm-infra","ref":"refs/changes/36/784536/5","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/36/784536/5 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/36/784536/5 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/36/784536/5 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-helm-infra refs/changes/36/784536/5"}}},"commit":{"parents":[{"commit":"7351586a7d4700e27dceea3b2004d535a48c6ae6","subject":"Merge \"Allow Ceph RBD pool job to leave failed pods\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/7351586a7d4700e27dceea3b2004d535a48c6ae6"}]}],"author":{"name":"Chris Wedgwood","email":"cw@f00f.org","date":"2021-04-02 06:02:36.000000000","tz":-300},"committer":{"name":"Chris Wedgwood","email":"cw@f00f.org","date":"2021-04-02 21:19:55.000000000","tz":-300},"subject":"[htk] Jobs; put labels only in the template spec","message":"[htk] Jobs; put labels only in the template spec\n\nThis is an update to address a behavior change introduced with\n0ae8f4d21ac2a091f1612e50f4786da5065d4398.\n\nJob labels if empty/unspecified are taken from the template.  If (any)\nlabels are specified on the job we do not get this behavior.\n\nSpecifically:\n\n    apiVersion: batch/v1\n    kind: Job\n    metadata:\n      # no \"labels:\" here\n      name: placement-db-init\n      namespace: openstack\n    spec:\n      template:\n        metadata:\n          labels:\n            application: placement\n            component: db-init\n            job-name: placement-db-init\n            release_group: placement\n        spec:\n          containers:\n          # do stuffs\n\nends up:\n\n    apiVersion: batch/v1\n    kind: Job\n    metadata:\n      # k8s did this for us!\n      labels:\n        application: placement\n        component: db-init\n        job-name: placement-db-init\n        release_group: placement\n      name: placement-db-init\n      namespace: openstack\n    spec:\n      template:\n        metadata:\n          labels:\n            application: placement\n            component: db-init\n            job-name: placement-db-init\n            release_group: placement\n        spec:\n          containers:\n          # do stuffs\n\nThe aforementioned causes objects to look like:\n\n    apiVersion: batch/v1\n    kind: Job\n    metadata:\n      # k8s did this for us!\n      labels:\n        application: placement\n        # nothing else!\n      name: placement-db-init\n      namespace: openstack\n    spec:\n      template:\n        metadata:\n          labels:\n            application: placement\n            component: db-init\n            job-name: placement-db-init\n            release_group: placement\n        spec:\n          containers:\n          # do stuffs\n\nCurrent users rely on this behavior and deployment systems use job\nlabels for synchronization, those labels being only specified in the\ntemplate and propagated up.\n\nThis should preserve functionality added recently as well as restore\nthe prevoius behavior.\n\nThe explicit \"application\" label is no longer needed as the\nhelm-toolkit.snippets.kubernetes_metadata_labels macro provides it.\n\nChange-Id: I1582d008217b8848103579b826fae065c538aaf0\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/2fe9a2e39cb0607371c6212ee6a90131581cdbe7"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/2fe9a2e39cb0607371c6212ee6a90131581cdbe7"}]},"branch":"refs/heads/master"},"987ba64fb241a4177777a03c561910a4b03d6767":{"kind":"NO_CODE_CHANGE","_number":6,"created":"2021-04-02 21:23:33.000000000","uploader":{"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},"ref":"refs/changes/36/784536/6","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-helm-infra","ref":"refs/changes/36/784536/6","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/36/784536/6 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/36/784536/6 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/36/784536/6 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-helm-infra refs/changes/36/784536/6"}}},"commit":{"parents":[{"commit":"7351586a7d4700e27dceea3b2004d535a48c6ae6","subject":"Merge \"Allow Ceph RBD pool job to leave failed pods\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/7351586a7d4700e27dceea3b2004d535a48c6ae6"}]}],"author":{"name":"Chris Wedgwood","email":"cw@f00f.org","date":"2021-04-02 06:02:36.000000000","tz":-300},"committer":{"name":"Chris Wedgwood","email":"cw@f00f.org","date":"2021-04-02 21:22:11.000000000","tz":-300},"subject":"[htk] Jobs; put labels only in the template spec","message":"[htk] Jobs; put labels only in the template spec\n\nThis is an update to address a behavior change introduced with\n0ae8f4d21ac2a091f1612e50f4786da5065d4398.\n\nJob labels if empty/unspecified are taken from the template.  If (any)\nlabels are specified on the job we do not get this behavior.\n\nSpecifically:\n\n    apiVersion: batch/v1\n    kind: Job\n    metadata:\n      # no \"labels:\" here\n      name: placement-db-init\n      namespace: openstack\n    spec:\n      template:\n        metadata:\n          labels:\n            application: placement\n            component: db-init\n            job-name: placement-db-init\n            release_group: placement\n        spec:\n          containers:\n          # do stuffs\n\nends up:\n\n    apiVersion: batch/v1\n    kind: Job\n    metadata:\n      # k8s did this for us!\n      labels:\n        application: placement\n        component: db-init\n        job-name: placement-db-init\n        release_group: placement\n      name: placement-db-init\n      namespace: openstack\n    spec:\n      template:\n        metadata:\n          labels:\n            application: placement\n            component: db-init\n            job-name: placement-db-init\n            release_group: placement\n        spec:\n          containers:\n          # do stuffs\n\nThe aforementioned causes objects to look like:\n\n    apiVersion: batch/v1\n    kind: Job\n    metadata:\n      # k8s did this for us!\n      labels:\n        application: placement\n        # nothing else!\n      name: placement-db-init\n      namespace: openstack\n    spec:\n      template:\n        metadata:\n          labels:\n            application: placement\n            component: db-init\n            job-name: placement-db-init\n            release_group: placement\n        spec:\n          containers:\n          # do stuffs\n\nCurrent users rely on this behavior and deployment systems use job\nlabels for synchronization, those labels being only specified in the\ntemplate and propagating to the job.\n\nThis change preserves functionality added recently and restores the\nprevious behavior.\n\nThe explicit \"application\" label is no longer needed as the\nhelm-toolkit.snippets.kubernetes_metadata_labels macro provides it.\n\nChange-Id: I1582d008217b8848103579b826fae065c538aaf0\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/987ba64fb241a4177777a03c561910a4b03d6767"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/987ba64fb241a4177777a03c561910a4b03d6767"}]},"branch":"refs/heads/master"},"65f7689a715feaf5d5e39624aa910a0aa840de0e":{"kind":"NO_CODE_CHANGE","_number":7,"created":"2021-04-02 21:49:59.000000000","uploader":{"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},"ref":"refs/changes/36/784536/7","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-helm-infra","ref":"refs/changes/36/784536/7","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/36/784536/7 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/36/784536/7 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/36/784536/7 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-helm-infra refs/changes/36/784536/7"}}},"commit":{"parents":[{"commit":"7351586a7d4700e27dceea3b2004d535a48c6ae6","subject":"Merge \"Allow Ceph RBD pool job to leave failed pods\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/7351586a7d4700e27dceea3b2004d535a48c6ae6"}]}],"author":{"name":"Chris Wedgwood","email":"cw@f00f.org","date":"2021-04-02 06:02:36.000000000","tz":-300},"committer":{"name":"Chris Wedgwood","email":"cw@f00f.org","date":"2021-04-02 21:47:19.000000000","tz":-300},"subject":"[htk] Jobs; put labels only in the template spec","message":"[htk] Jobs; put labels only in the template spec\n\nThis is an update to address a behavior change introduced with\n0ae8f4d21ac2a091f1612e50f4786da5065d4398.\n\nJob labels if empty/unspecified are taken from the template.  If (any)\nlabels are specified on the job we do not get this behavior.\n\nSpecifically:\n\n    apiVersion: batch/v1\n    kind: Job\n    metadata:\n      # no \"labels:\" here\n      name: placement-db-init\n      namespace: openstack\n    spec:\n      template:\n        metadata:\n          labels:\n            application: placement\n            component: db-init\n            release_group: placement\n        spec:\n          containers:\n          # do stuffs\n\nends up:\n\n    apiVersion: batch/v1\n    kind: Job\n    metadata:\n      # k8s did this for us!\n      labels:\n        application: placement\n        component: db-init\n        release_group: placement\n      name: placement-db-init\n      namespace: openstack\n    spec:\n      template:\n        metadata:\n          labels:\n            application: placement\n            component: db-init\n            release_group: placement\n        spec:\n          containers:\n          # do stuffs\n\nThe aforementioned change causes objects to look like:\n\n    apiVersion: batch/v1\n    kind: Job\n    metadata:\n      # k8s did this for us!\n      labels:\n        application: placement\n        # nothing else!\n      name: placement-db-init\n      namespace: openstack\n    spec:\n      template:\n        metadata:\n          labels:\n            application: placement\n            component: db-init\n            release_group: placement\n        spec:\n          containers:\n          # do stuffs\n\nCurrent users rely on this behavior and deployment systems use job\nlabels for synchronization, those labels being only specified in the\ntemplate and propagating to the job.\n\nThis change preserves functionality added recently and restores the\nprevious behavior.\n\nThe explicit \"application\" label is no longer needed as the\nhelm-toolkit.snippets.kubernetes_metadata_labels macro provides it.\n\nChange-Id: I1582d008217b8848103579b826fae065c538aaf0\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/65f7689a715feaf5d5e39624aa910a0aa840de0e"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/65f7689a715feaf5d5e39624aa910a0aa840de0e"}]},"branch":"refs/heads/master"},"99fa00149089967d95d903ba1a8385c24291cc76":{"kind":"NO_CODE_CHANGE","_number":8,"created":"2021-04-02 21:52:44.000000000","uploader":{"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},"ref":"refs/changes/36/784536/8","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-helm-infra","ref":"refs/changes/36/784536/8","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/36/784536/8 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/36/784536/8 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/36/784536/8 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-helm-infra refs/changes/36/784536/8"}}},"commit":{"parents":[{"commit":"7351586a7d4700e27dceea3b2004d535a48c6ae6","subject":"Merge \"Allow Ceph RBD pool job to leave failed pods\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/7351586a7d4700e27dceea3b2004d535a48c6ae6"}]}],"author":{"name":"Chris Wedgwood","email":"cw@f00f.org","date":"2021-04-02 06:02:36.000000000","tz":-300},"committer":{"name":"Chris Wedgwood","email":"cw@f00f.org","date":"2021-04-02 21:50:44.000000000","tz":-300},"subject":"[htk] Jobs; put labels only in the template spec","message":"[htk] Jobs; put labels only in the template spec\n\nThis is an update to address a behavior change introduced with\n0ae8f4d21ac2a091f1612e50f4786da5065d4398.\n\nJob labels if empty/unspecified are taken from the template.  If (any)\nlabels are specified on the job we do not get this behavior.\n\nSpecifically if we *apply*:\n\n    apiVersion: batch/v1\n    kind: Job\n    metadata:\n      # no \"labels:\" here\n      name: placement-db-init\n      namespace: openstack\n    spec:\n      template:\n        metadata:\n          labels:\n            application: placement\n            component: db-init\n            release_group: placement\n        spec:\n          containers:\n          # do stuffs\n\nthen *query* we see:\n\n    apiVersion: batch/v1\n    kind: Job\n    metadata:\n      # k8s did this for us!\n      labels:\n        application: placement\n        component: db-init\n\tjob-name: placement-db-init\n        release_group: placement\n      name: placement-db-init\n      namespace: openstack\n    spec:\n      template:\n        metadata:\n          labels:\n            application: placement\n            component: db-init\n            release_group: placement\n        spec:\n          containers:\n          # do stuffs\n\nThe aforementioned change causes objects we apply and query to look\nlike:\n\n    apiVersion: batch/v1\n    kind: Job\n    metadata:\n      # k8s did this for us!\n      labels:\n        application: placement\n        # nothing else!\n      name: placement-db-init\n      namespace: openstack\n    spec:\n      template:\n        metadata:\n          labels:\n            application: placement\n            component: db-init\n            release_group: placement\n        spec:\n          containers:\n          # do stuffs\n\nCurrent users rely on this behavior and deployment systems use job\nlabels for synchronization, those labels being only specified in the\ntemplate and propagating to the job.\n\nThis change preserves functionality added recently and restores the\nprevious behavior.\n\nThe explicit \"application\" label is no longer needed as the\nhelm-toolkit.snippets.kubernetes_metadata_labels macro provides it.\n\nChange-Id: I1582d008217b8848103579b826fae065c538aaf0\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/99fa00149089967d95d903ba1a8385c24291cc76"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/99fa00149089967d95d903ba1a8385c24291cc76"}]},"branch":"refs/heads/master"},"20cf2db961a05285a751b3bc96b8fa4e51e3b412":{"kind":"NO_CODE_CHANGE","_number":9,"created":"2021-04-02 21:54:11.000000000","uploader":{"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},"ref":"refs/changes/36/784536/9","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-helm-infra","ref":"refs/changes/36/784536/9","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/36/784536/9 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/36/784536/9 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/36/784536/9 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-helm-infra refs/changes/36/784536/9"}}},"commit":{"parents":[{"commit":"7351586a7d4700e27dceea3b2004d535a48c6ae6","subject":"Merge \"Allow Ceph RBD pool job to leave failed pods\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/7351586a7d4700e27dceea3b2004d535a48c6ae6"}]}],"author":{"name":"Chris Wedgwood","email":"cw@f00f.org","date":"2021-04-02 06:02:36.000000000","tz":-300},"committer":{"name":"Chris Wedgwood","email":"cw@f00f.org","date":"2021-04-02 21:54:03.000000000","tz":-300},"subject":"[htk] Jobs; put labels only in the template spec","message":"[htk] Jobs; put labels only in the template spec\n\nThis is an update to address a behavior change introduced with\n0ae8f4d21ac2a091f1612e50f4786da5065d4398.\n\nJob labels if empty/unspecified are taken from the template.  If (any)\nlabels are specified on the job we do not get this behavior.\n\nSpecifically if we *apply*:\n\n    apiVersion: batch/v1\n    kind: Job\n    metadata:\n      # no \"labels:\" here\n      name: placement-db-init\n      namespace: openstack\n    spec:\n      template:\n        metadata:\n          labels:\n            application: placement\n            component: db-init\n            release_group: placement\n        spec:\n          containers:\n          # do stuffs\n\nthen *query* we see:\n\n    apiVersion: batch/v1\n    kind: Job\n    metadata:\n      # k8s did this for us!\n      labels:\n        application: placement\n        component: db-init\n        job-name: placement-db-init\n        release_group: placement\n      name: placement-db-init\n      namespace: openstack\n    spec:\n      template:\n        metadata:\n          labels:\n            application: placement\n            component: db-init\n            release_group: placement\n        spec:\n          containers:\n          # do stuffs\n\nThe aforementioned change causes objects we apply and query to look\nlike:\n\n    apiVersion: batch/v1\n    kind: Job\n    metadata:\n      # k8s did this for us!\n      labels:\n        application: placement\n        # nothing else!\n      name: placement-db-init\n      namespace: openstack\n    spec:\n      template:\n        metadata:\n          labels:\n            application: placement\n            component: db-init\n            release_group: placement\n        spec:\n          containers:\n          # do stuffs\n\nCurrent users rely on this behavior and deployment systems use job\nlabels for synchronization, those labels being only specified in the\ntemplate and propagating to the job.\n\nThis change preserves functionality added recently and restores the\nprevious behavior.\n\nThe explicit \"application\" label is no longer needed as the\nhelm-toolkit.snippets.kubernetes_metadata_labels macro provides it.\n\nChange-Id: I1582d008217b8848103579b826fae065c538aaf0\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/20cf2db961a05285a751b3bc96b8fa4e51e3b412"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/20cf2db961a05285a751b3bc96b8fa4e51e3b412"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[{"status":"CLOSED","labels":[{"label":"Verified","status":"OK","applied_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}},{"label":"Code-Review","status":"OK","applied_by":{"_account_id":28372,"name":"chinasubbareddy mallavarapu","email":"cr3938@att.com","username":"chinasubbareddy"}},{"label":"Workflow","status":"OK","applied_by":{"_account_id":28372,"name":"chinasubbareddy mallavarapu","email":"cr3938@att.com","username":"chinasubbareddy"}}]}],"submit_requirements":[{"name":"Verified","status":"SATISFIED","is_legacy":true,"submittability_expression_result":{"expression":"label:Verified\u003dCustom-Rule","fulfilled":true,"status":"PASS","passing_atoms":["label:Verified\u003dCustom-Rule"],"failing_atoms":[],"atom_explanations":{}}},{"name":"Workflow","status":"SATISFIED","is_legacy":true,"submittability_expression_result":{"expression":"label:Workflow\u003dCustom-Rule","fulfilled":true,"status":"PASS","passing_atoms":["label:Workflow\u003dCustom-Rule"],"failing_atoms":[],"atom_explanations":{}}},{"name":"Code-Review","status":"SATISFIED","is_legacy":true,"submittability_expression_result":{"expression":"label:Code-Review\u003dCustom-Rule","fulfilled":true,"status":"PASS","passing_atoms":["label:Code-Review\u003dCustom-Rule"],"failing_atoms":[],"atom_explanations":{}}}]}
