)]}'
{"Dockerfile":[{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"46ad88621cfd9879142d135fe8a3493b160cf6ef","unresolved":false,"context_lines":[{"line_number":24,"context_line":"RUN /output/install-from-bindep"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"# 11000 matches what we are running on eavesdrop"},{"line_number":27,"context_line":"RUN addgroup gerritbot --gid 11000 --system \\"},{"line_number":28,"context_line":"  \u0026\u0026 adduser \\"},{"line_number":29,"context_line":"    --system \\"},{"line_number":30,"context_line":"    --uid 11000 \\"}],"source_content_type":"text/x-dockerfile","patch_set":1,"id":"ed005097_c02343df","line":27,"updated":"2021-12-09 19:27:12.000000000","message":"Should actually be able to pass the --group option below instead of using --ingroup gerritbot, and so entirely avoid the separate addgroup invocation. But this is also fine.","commit_id":"ee566eb49932729aaeb7c0a0cf0cc69c8e1462a0"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"f866a5a7a4515da68509f109b3ee84af8206473e","unresolved":true,"context_lines":[{"line_number":24,"context_line":"RUN /output/install-from-bindep"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"# 11000 matches what we are running on eavesdrop"},{"line_number":27,"context_line":"RUN addgroup gerritbot --gid 11000 --system \\"},{"line_number":28,"context_line":"  \u0026\u0026 adduser \\"},{"line_number":29,"context_line":"    --system \\"},{"line_number":30,"context_line":"    --uid 11000 \\"}],"source_content_type":"text/x-dockerfile","patch_set":1,"id":"76dc5526_470b7073","line":27,"in_reply_to":"88e9d300_ccdfb598","updated":"2021-12-13 23:19:14.000000000","message":"This block comes from gitea\u0027s image which we seem to have reused elsewhere (zuul). I think being consistent is better than fancier.\n\nAs far as abstracting into a yaml file goes that is what kolla does and it seems like nothing but pain because you have to run an entire entra tool before you can run docker image builds. Yes it i repetitive but it rarely changes and self contained is nice so that anyone can build the image easily.","commit_id":"ee566eb49932729aaeb7c0a0cf0cc69c8e1462a0"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"3135496801bba58b2b02f75dedaa55fb89aec508","unresolved":true,"context_lines":[{"line_number":24,"context_line":"RUN /output/install-from-bindep"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"# 11000 matches what we are running on eavesdrop"},{"line_number":27,"context_line":"RUN addgroup gerritbot --gid 11000 --system \\"},{"line_number":28,"context_line":"  \u0026\u0026 adduser \\"},{"line_number":29,"context_line":"    --system \\"},{"line_number":30,"context_line":"    --uid 11000 \\"}],"source_content_type":"text/x-dockerfile","patch_set":1,"id":"88e9d300_ccdfb598","line":27,"in_reply_to":"ed005097_c02343df","updated":"2021-12-13 22:59:32.000000000","message":"Maybe if we\u0027re sick of fiddling around with differences in such things we could write a tool where you figure this out once and abstract it with a simple entry in a YAML file ... ;)","commit_id":"ee566eb49932729aaeb7c0a0cf0cc69c8e1462a0"}]}