)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":25733,"name":"Jan Gutter","email":"github@jangutter.com","username":"jangutter"},"change_message_id":"7e949e4a4042ad7e2ecacc265c5aac54e9da402e","unresolved":true,"context_lines":[{"line_number":14,"context_line":""},{"line_number":15,"context_line":"There\u0027s also an easter egg: developers can get Zuul to render the"},{"line_number":16,"context_line":"ARA output for them. It\u0027s not enabled by default because the files"},{"line_number":17,"context_line":"are rather large."},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Change-Id: Id8328e374c9590b1363026fa2b2b24e191183987"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"8976b6e9_68eb6c89","line":17,"updated":"2023-08-11 09:50:53.000000000","message":"Note to self: add the slowdown stats here (there\u0027s a per-task overhead to log to the sqlite backend)","commit_id":"f72fc29aa199ebc8f5371c3313c55d5df80e0f75"},{"author":{"_account_id":25733,"name":"Jan Gutter","email":"github@jangutter.com","username":"jangutter"},"change_message_id":"4cf5847ee56b0ad6efa7280b0937f463d0b6a171","unresolved":false,"context_lines":[{"line_number":14,"context_line":""},{"line_number":15,"context_line":"There\u0027s also an easter egg: developers can get Zuul to render the"},{"line_number":16,"context_line":"ARA output for them. It\u0027s not enabled by default because the files"},{"line_number":17,"context_line":"are rather large."},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Change-Id: Id8328e374c9590b1363026fa2b2b24e191183987"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"475796e9_ec1edcef","line":17,"in_reply_to":"8976b6e9_68eb6c89","updated":"2023-08-11 10:07:53.000000000","message":"Looks like the impact time-wise is between 5-15 minutes for jobs (for the sqlite option) I\u0027ll add this observation in here for any future respin.","commit_id":"f72fc29aa199ebc8f5371c3313c55d5df80e0f75"},{"author":{"_account_id":32553,"name":"Sven Kieske","email":"sven_oss@posteo.de","username":"skieske"},"change_message_id":"dd86055dac2c488d1a3187351038f0940a09a921","unresolved":true,"context_lines":[{"line_number":13,"context_line":"to guide developers on how to view the sqlite file."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"The ARA plugin is installed by default, but not activated. This"},{"line_number":16,"context_line":"is intended to catch a small amount of regressions and"},{"line_number":17,"context_line":"integration failures."},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Developers can enable the plugin by adding the string `#ara`"},{"line_number":20,"context_line":"to their commit message. This avoids extra load on the CI."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"1ab96103_4fe73562","line":17,"range":{"start_line":16,"start_character":15,"end_line":17,"end_character":21},"updated":"2023-09-05 08:41:13.000000000","message":"can we name/link those? are there bugreports for them?\n\nor are this general issues with ara, which can\u0027t be resolved?","commit_id":"73dcad3263b846318d684d717f9238a752158aee"},{"author":{"_account_id":25733,"name":"Jan Gutter","email":"github@jangutter.com","username":"jangutter"},"change_message_id":"0628a4b607b751f18651c1ce3d232093549b38d7","unresolved":true,"context_lines":[{"line_number":13,"context_line":"to guide developers on how to view the sqlite file."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"The ARA plugin is installed by default, but not activated. This"},{"line_number":16,"context_line":"is intended to catch a small amount of regressions and"},{"line_number":17,"context_line":"integration failures."},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Developers can enable the plugin by adding the string `#ara`"},{"line_number":20,"context_line":"to their commit message. This avoids extra load on the CI."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"a21cc726_0659cdc4","line":17,"range":{"start_line":16,"start_character":15,"end_line":17,"end_character":21},"in_reply_to":"1ab96103_4fe73562","updated":"2023-09-05 09:30:54.000000000","message":"The intention is to catch \"failure to install\" issues. When a new series is made and pip fails to install ara using the constraints, it gives early warning that this will break.\n\nIt won\u0027t catch \"failure to run\" issues.","commit_id":"73dcad3263b846318d684d717f9238a752158aee"},{"author":{"_account_id":32553,"name":"Sven Kieske","email":"sven_oss@posteo.de","username":"skieske"},"change_message_id":"b7ae36151dec3cd23181e06e6a5330a29b6b1903","unresolved":false,"context_lines":[{"line_number":13,"context_line":"to guide developers on how to view the sqlite file."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"The ARA plugin is installed by default, but not activated. This"},{"line_number":16,"context_line":"is intended to catch a small amount of regressions and"},{"line_number":17,"context_line":"integration failures."},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Developers can enable the plugin by adding the string `#ara`"},{"line_number":20,"context_line":"to their commit message. This avoids extra load on the CI."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"cb921b63_eeb791b4","line":17,"range":{"start_line":16,"start_character":15,"end_line":17,"end_character":21},"in_reply_to":"a21cc726_0659cdc4","updated":"2023-09-05 10:02:34.000000000","message":"Ack\nSorry, I totally misread that sentence, that there would be a small amount of regressions and failures because ARA is enabled 😂","commit_id":"73dcad3263b846318d684d717f9238a752158aee"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"6acf4c58a426dfee2f8e001293b0ee8aa49b9e47","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"e2b074d0_32ea9331","updated":"2023-08-10 20:04:15.000000000","message":"very interesting, especially the easter egg","commit_id":"f72fc29aa199ebc8f5371c3313c55d5df80e0f75"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"07971d21906f8d28959651b42a823f2d709a8712","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"8f71e60a_715e203e","updated":"2023-08-14 09:45:57.000000000","message":"#generate_ara_html","commit_id":"452983f0192401752351eaaeb11eb9ed20e11c55"},{"author":{"_account_id":25733,"name":"Jan Gutter","email":"github@jangutter.com","username":"jangutter"},"change_message_id":"0ffcffe834ae657deeed7466f331c0e1fc1c3c83","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"fe5c15ca_3b8b6d00","updated":"2023-08-14 10:06:47.000000000","message":"Zuul explicitly considers passing job parameters an anti-pattern, and sending it via a review comment would probably fall under this category. Another way of doing this is by using the \"check experimental\" pipeline (that listens for review comments with \"check experimental\" and also doesn\u0027t accept extra parameters)\n\nHowever, the git commit message _is_ part of the job in the _check_ and _gate_ pipelines:\nhttps://zuul.opendev.org/t/openstack/build/efe7a1de5ce14f7398989c43e63dab45/log/zuul-info/inventory.yaml#80\n\nSo if your git commit message has text in it, you can do \"special\" things... but for obvious reasons every \"special\" thing you do adds a little bit of drift from the pure pipeline. I think it might be tolerable for a \"debug\" flag, but other things are very risky.","commit_id":"452983f0192401752351eaaeb11eb9ed20e11c55"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"c16ab54c6af936e45e3298118558eb746713ae5b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"dc1b1036_1bc68a02","in_reply_to":"8708b4de_bcfb7e64","updated":"2023-08-14 09:55:21.000000000","message":"really? strange CI usage, but ok, will test)","commit_id":"452983f0192401752351eaaeb11eb9ed20e11c55"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"0e3bed432c0c8f8a4c642e749751ce2e67803e59","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"8708b4de_bcfb7e64","in_reply_to":"8f71e60a_715e203e","updated":"2023-08-14 09:54:18.000000000","message":"\u003e #generate_ara_html\n\noh... zuull is testing... lets wait and check again)","commit_id":"452983f0192401752351eaaeb11eb9ed20e11c55"},{"author":{"_account_id":25733,"name":"Jan Gutter","email":"github@jangutter.com","username":"jangutter"},"change_message_id":"836844fc19a6950b56fa8a4a79fda477f0daffef","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"dd544175_472194c7","in_reply_to":"8f71e60a_715e203e","updated":"2023-08-14 09:54:12.000000000","message":"Sadly, it doesn\u0027t work like this, it needs to be in the git comment, not in the review comment.","commit_id":"452983f0192401752351eaaeb11eb9ed20e11c55"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"f28b0b1c10dcd9518a8e53e0950cb0278f178e55","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"937f706b_561b9b80","in_reply_to":"fe5c15ca_3b8b6d00","updated":"2023-08-14 11:07:05.000000000","message":"agree","commit_id":"452983f0192401752351eaaeb11eb9ed20e11c55"},{"author":{"_account_id":9061,"name":"David Moreau Simard","email":"moi@dmsimard.com","username":"dmsimard"},"change_message_id":"cc569759723e88b99d3c7836ebff45f6cc931511","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"647f07ff_a99d2bc1","updated":"2023-09-10 01:08:06.000000000","message":"Hi and thanks for working on this!\n\nI\u0027m the author of ara and have re-joined the #openstack-kolla channel as dmsimard if you happen to have any questions.","commit_id":"73dcad3263b846318d684d717f9238a752158aee"},{"author":{"_account_id":25733,"name":"Jan Gutter","email":"github@jangutter.com","username":"jangutter"},"change_message_id":"0628a4b607b751f18651c1ce3d232093549b38d7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"22fdd70e_02b5ee1e","updated":"2023-09-05 09:30:54.000000000","message":"The intention for this review is to re-add access to a developer-centric tool that,\nin the past, has been a very valuable way of analysing ansible runs.\n\nSince it\u0027s been broken for a while now, it\u0027s clear that the tool has not been used much recently. The intention is not to make it a user-facing tool, but add back the option to enable debugging in CI in a more convenient way.\n\nIt\u0027s definitely not the intention to be a comprehensive user guide on how to use ARA or how to do debugging.","commit_id":"73dcad3263b846318d684d717f9238a752158aee"},{"author":{"_account_id":9061,"name":"David Moreau Simard","email":"moi@dmsimard.com","username":"dmsimard"},"change_message_id":"cc569759723e88b99d3c7836ebff45f6cc931511","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"0a9bc379_5d00036e","in_reply_to":"22fdd70e_02b5ee1e","updated":"2023-09-10 01:08:06.000000000","message":"I would argue that operators can benefit from ara in production but I am biased :)","commit_id":"73dcad3263b846318d684d717f9238a752158aee"}],"doc/source/contributor/ci.rst":[{"author":{"_account_id":32553,"name":"Sven Kieske","email":"sven_oss@posteo.de","username":"skieske"},"change_message_id":"dd86055dac2c488d1a3187351038f0940a09a921","unresolved":true,"context_lines":[{"line_number":17,"context_line":"large amount of execution information and can render it into a browser"},{"line_number":18,"context_line":"friendly format."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"This plugin is not enabled by default because there is a per-task overhead."},{"line_number":21,"context_line":"However, it\u0027s possible to trigger it when trying to debug a failing job."},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"By adding the text `#ara` to the git commit message of the review, the CI jobs"}],"source_content_type":"text/x-rst","patch_set":5,"id":"1ca60300_1728a86e","line":20,"range":{"start_line":20,"start_character":0,"end_line":20,"end_character":1},"updated":"2023-09-05 08:41:13.000000000","message":"the overhead should imho be documented here and not be buried in a discussion in a gerrit review somewhere on the internet.\nPlease add the overhead here, thanks.","commit_id":"73dcad3263b846318d684d717f9238a752158aee"},{"author":{"_account_id":25733,"name":"Jan Gutter","email":"github@jangutter.com","username":"jangutter"},"change_message_id":"0628a4b607b751f18651c1ce3d232093549b38d7","unresolved":true,"context_lines":[{"line_number":17,"context_line":"large amount of execution information and can render it into a browser"},{"line_number":18,"context_line":"friendly format."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"This plugin is not enabled by default because there is a per-task overhead."},{"line_number":21,"context_line":"However, it\u0027s possible to trigger it when trying to debug a failing job."},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"By adding the text `#ara` to the git commit message of the review, the CI jobs"}],"source_content_type":"text/x-rst","patch_set":5,"id":"a96b8923_55ae1abf","line":20,"range":{"start_line":20,"start_character":0,"end_line":20,"end_character":1},"in_reply_to":"1ca60300_1728a86e","updated":"2023-09-05 09:30:54.000000000","message":"The overhead varies greatly depending on tons of factors: it scales with number of nodes (in the CI case that\u0027s not a big issue) but also with number of tasks (so loops get penalised a lot more), the amount of facts and the size of the data collected by the tasks.\n\nThe probability distribution of the slowdown is affected by so many factors that predicting it is not usually worthwhile.\n\nSince we\u0027re only enabling this in the debugging case, is it really valuable to know how much extra overhead a debug flag brings to the table?","commit_id":"73dcad3263b846318d684d717f9238a752158aee"},{"author":{"_account_id":25733,"name":"Jan Gutter","email":"github@jangutter.com","username":"jangutter"},"change_message_id":"796b8f13b39a3f77d58f663c4d82146c0987fe04","unresolved":false,"context_lines":[{"line_number":17,"context_line":"large amount of execution information and can render it into a browser"},{"line_number":18,"context_line":"friendly format."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"This plugin is not enabled by default because there is a per-task overhead."},{"line_number":21,"context_line":"However, it\u0027s possible to trigger it when trying to debug a failing job."},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"By adding the text `#ara` to the git commit message of the review, the CI jobs"}],"source_content_type":"text/x-rst","patch_set":5,"id":"dcbd9de4_dbf3000e","line":20,"range":{"start_line":20,"start_character":0,"end_line":20,"end_character":1},"in_reply_to":"3d570023_98faa3a3","updated":"2023-09-05 10:18:48.000000000","message":"If there\u0027s a respin, how about I add:\n\"Enabling these kind of debugging options are not recommended if the job is already\nunder storage or time constraints.\"","commit_id":"73dcad3263b846318d684d717f9238a752158aee"},{"author":{"_account_id":32553,"name":"Sven Kieske","email":"sven_oss@posteo.de","username":"skieske"},"change_message_id":"b7ae36151dec3cd23181e06e6a5330a29b6b1903","unresolved":false,"context_lines":[{"line_number":17,"context_line":"large amount of execution information and can render it into a browser"},{"line_number":18,"context_line":"friendly format."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"This plugin is not enabled by default because there is a per-task overhead."},{"line_number":21,"context_line":"However, it\u0027s possible to trigger it when trying to debug a failing job."},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"By adding the text `#ara` to the git commit message of the review, the CI jobs"}],"source_content_type":"text/x-rst","patch_set":5,"id":"3d570023_98faa3a3","line":20,"range":{"start_line":20,"start_character":0,"end_line":20,"end_character":1},"in_reply_to":"a96b8923_55ae1abf","updated":"2023-09-05 10:02:34.000000000","message":"well I didn\u0027t ask for hyper accurate scientific studies on the probability of the slowdown ;) . You _did_ already provide numbers, just not - imho - in the right location.\n\nBecause the reason we are hiding this feature behind a commit message tag is the slowdown, so imho the rough amount of slowdown should be documented next to it.\n\nBecause I can tell from experience that some time later a dev will come and ask what the overhead is and if we can\u0027t enable this in the general case, and for these cases it\u0027s just nice to document why something was done next to how it is done.\n\nThe slowdown numbers basically provide additional context for the uninitiated.\n\nSo I\u0027d say it\u0027s valuable, but ymmv.","commit_id":"73dcad3263b846318d684d717f9238a752158aee"},{"author":{"_account_id":32553,"name":"Sven Kieske","email":"sven_oss@posteo.de","username":"skieske"},"change_message_id":"f71c76d32b49d429586923b0585e82eb45b54db9","unresolved":false,"context_lines":[{"line_number":17,"context_line":"large amount of execution information and can render it into a browser"},{"line_number":18,"context_line":"friendly format."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"This plugin is not enabled by default because there is a per-task overhead."},{"line_number":21,"context_line":"However, it\u0027s possible to trigger it when trying to debug a failing job."},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"By adding the text `#ara` to the git commit message of the review, the CI jobs"}],"source_content_type":"text/x-rst","patch_set":5,"id":"2e629364_2d0f6e2c","line":20,"range":{"start_line":20,"start_character":0,"end_line":20,"end_character":1},"in_reply_to":"dcbd9de4_dbf3000e","updated":"2023-09-05 13:08:02.000000000","message":"sounds good to me 😊","commit_id":"73dcad3263b846318d684d717f9238a752158aee"}],"tests/ara-readme.md.j2":[{"author":{"_account_id":25733,"name":"Jan Gutter","email":"github@jangutter.com","username":"jangutter"},"change_message_id":"80894b2182f5ccbc4438cf7827f890ee5859bdee","unresolved":false,"context_lines":[{"line_number":25,"context_line":"For example, rendering a particular log to static html pages:"},{"line_number":26,"context_line":"```"},{"line_number":27,"context_line":"# (Use the \"Download all logs\" script from the Artifacts tab with DOWNLOAD_DIR\u003d~/zuul-logs)"},{"line_number":28,"context_line":"python3 -m pip install --user \"ara[server]\u003d\u003d{{ ara_version.stdout.split(\u0027 \u0027) | last }}\""},{"line_number":29,"context_line":"export ARA_DATABASE_NAME\u003d~/zuul-logs/{{ inventory_hostname }}/ara-report/ansible.sqlite"},{"line_number":30,"context_line":"ara-manage generate ~/zuul-logs/{{ inventory_hostname }}/ara-report/ara-html"},{"line_number":31,"context_line":"```"}],"source_content_type":"text/x-jinja2","patch_set":4,"id":"ccbbf0fb_9b89a7cd","line":28,"range":{"start_line":28,"start_character":47,"end_line":28,"end_character":58},"updated":"2023-08-17 13:27:49.000000000","message":"ara_version is used here","commit_id":"452983f0192401752351eaaeb11eb9ed20e11c55"},{"author":{"_account_id":32553,"name":"Sven Kieske","email":"sven_oss@posteo.de","username":"skieske"},"change_message_id":"dd86055dac2c488d1a3187351038f0940a09a921","unresolved":true,"context_lines":[{"line_number":14,"context_line":"How do I view the output?"},{"line_number":15,"context_line":"-------------------------"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"You need a local installation of ARA: following the"},{"line_number":18,"context_line":"[getting started guide](https://ara.readthedocs.io/en/latest/getting-started.html)"},{"line_number":19,"context_line":"should help you with installing the `ara-manage` command."},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"For small scale operations, you have a choice between using:"},{"line_number":22,"context_line":"* [a small embedded server](https://ara.readthedocs.io/en/latest/cli.html#ara-manage-runserver), or"}],"source_content_type":"text/x-jinja2","patch_set":5,"id":"53b6394f_77404481","line":19,"range":{"start_line":17,"start_character":0,"end_line":19,"end_character":57},"updated":"2023-09-05 08:41:13.000000000","message":"as it\u0027s only a single line, why do we force the user to check another website, which might change?\n\ninstead write:\n\n\"You need a local isntallation of ARA: you can install it via `python3 -m pip install --user $youruser \"ara[server]\"`","commit_id":"73dcad3263b846318d684d717f9238a752158aee"},{"author":{"_account_id":32553,"name":"Sven Kieske","email":"sven_oss@posteo.de","username":"skieske"},"change_message_id":"b7ae36151dec3cd23181e06e6a5330a29b6b1903","unresolved":false,"context_lines":[{"line_number":14,"context_line":"How do I view the output?"},{"line_number":15,"context_line":"-------------------------"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"You need a local installation of ARA: following the"},{"line_number":18,"context_line":"[getting started guide](https://ara.readthedocs.io/en/latest/getting-started.html)"},{"line_number":19,"context_line":"should help you with installing the `ara-manage` command."},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"For small scale operations, you have a choice between using:"},{"line_number":22,"context_line":"* [a small embedded server](https://ara.readthedocs.io/en/latest/cli.html#ara-manage-runserver), or"}],"source_content_type":"text/x-jinja2","patch_set":5,"id":"822fcc71_ab89e3ed","line":19,"range":{"start_line":17,"start_character":0,"end_line":19,"end_character":57},"in_reply_to":"21451482_0a878138","updated":"2023-09-05 10:02:34.000000000","message":"Ack","commit_id":"73dcad3263b846318d684d717f9238a752158aee"},{"author":{"_account_id":25733,"name":"Jan Gutter","email":"github@jangutter.com","username":"jangutter"},"change_message_id":"0628a4b607b751f18651c1ce3d232093549b38d7","unresolved":true,"context_lines":[{"line_number":14,"context_line":"How do I view the output?"},{"line_number":15,"context_line":"-------------------------"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"You need a local installation of ARA: following the"},{"line_number":18,"context_line":"[getting started guide](https://ara.readthedocs.io/en/latest/getting-started.html)"},{"line_number":19,"context_line":"should help you with installing the `ara-manage` command."},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"For small scale operations, you have a choice between using:"},{"line_number":22,"context_line":"* [a small embedded server](https://ara.readthedocs.io/en/latest/cli.html#ara-manage-runserver), or"}],"source_content_type":"text/x-jinja2","patch_set":5,"id":"21451482_0a878138","line":19,"range":{"start_line":17,"start_character":0,"end_line":19,"end_character":57},"in_reply_to":"53b6394f_77404481","updated":"2023-09-05 09:30:54.000000000","message":"In the past ARA changed its install methods. This is why I refer to the upstream documentation as the canonical, and give an example below. If we just link to the upstream documentation, users have to click through. If we just give an example, then if it breaks, users won\u0027t find the upstream documentation.","commit_id":"73dcad3263b846318d684d717f9238a752158aee"},{"author":{"_account_id":32553,"name":"Sven Kieske","email":"sven_oss@posteo.de","username":"skieske"},"change_message_id":"dd86055dac2c488d1a3187351038f0940a09a921","unresolved":true,"context_lines":[{"line_number":25,"context_line":"For example, rendering a particular log to static html pages:"},{"line_number":26,"context_line":"```"},{"line_number":27,"context_line":"# (Use the \"Download all logs\" script from the Artifacts tab with DOWNLOAD_DIR\u003d~/zuul-logs)"},{"line_number":28,"context_line":"python3 -m pip install --user \"ara[server]\u003d\u003d{{ ara_version.stdout.split(\u0027 \u0027) | last }}\""},{"line_number":29,"context_line":"export ARA_DATABASE_NAME\u003d~/zuul-logs/{{ inventory_hostname }}/ara-report/ansible.sqlite"},{"line_number":30,"context_line":"ara-manage generate ~/zuul-logs/{{ inventory_hostname }}/ara-report/ara-html"},{"line_number":31,"context_line":"```"}],"source_content_type":"text/x-jinja2","patch_set":5,"id":"86f92b4a_677d03bc","line":28,"range":{"start_line":28,"start_character":0,"end_line":28,"end_character":22},"updated":"2023-09-05 08:41:13.000000000","message":"this installs the ara server. But on line 18 we told the user to check some other site on how to install it. this is either redundant or not that well structured.","commit_id":"73dcad3263b846318d684d717f9238a752158aee"},{"author":{"_account_id":25733,"name":"Jan Gutter","email":"github@jangutter.com","username":"jangutter"},"change_message_id":"0628a4b607b751f18651c1ce3d232093549b38d7","unresolved":true,"context_lines":[{"line_number":25,"context_line":"For example, rendering a particular log to static html pages:"},{"line_number":26,"context_line":"```"},{"line_number":27,"context_line":"# (Use the \"Download all logs\" script from the Artifacts tab with DOWNLOAD_DIR\u003d~/zuul-logs)"},{"line_number":28,"context_line":"python3 -m pip install --user \"ara[server]\u003d\u003d{{ ara_version.stdout.split(\u0027 \u0027) | last }}\""},{"line_number":29,"context_line":"export ARA_DATABASE_NAME\u003d~/zuul-logs/{{ inventory_hostname }}/ara-report/ansible.sqlite"},{"line_number":30,"context_line":"ara-manage generate ~/zuul-logs/{{ inventory_hostname }}/ara-report/ara-html"},{"line_number":31,"context_line":"```"}],"source_content_type":"text/x-jinja2","patch_set":5,"id":"fbee1de8_68998dd1","line":28,"range":{"start_line":28,"start_character":0,"end_line":28,"end_character":22},"in_reply_to":"86f92b4a_677d03bc","updated":"2023-09-05 09:30:54.000000000","message":"The intention is to have an example that is nearly cut-and-paste. It\u0027s not intended to always work, it\u0027s intended to be an illustrative example.\n\nI did not find it redundant to provide a link to the upstream documentation, and a usable example in the same documentation.\n\nThe intention is to help developers debug something using a tool, not to be a comprehensive document of how the tool works.","commit_id":"73dcad3263b846318d684d717f9238a752158aee"},{"author":{"_account_id":32553,"name":"Sven Kieske","email":"sven_oss@posteo.de","username":"skieske"},"change_message_id":"b7ae36151dec3cd23181e06e6a5330a29b6b1903","unresolved":false,"context_lines":[{"line_number":25,"context_line":"For example, rendering a particular log to static html pages:"},{"line_number":26,"context_line":"```"},{"line_number":27,"context_line":"# (Use the \"Download all logs\" script from the Artifacts tab with DOWNLOAD_DIR\u003d~/zuul-logs)"},{"line_number":28,"context_line":"python3 -m pip install --user \"ara[server]\u003d\u003d{{ ara_version.stdout.split(\u0027 \u0027) | last }}\""},{"line_number":29,"context_line":"export ARA_DATABASE_NAME\u003d~/zuul-logs/{{ inventory_hostname }}/ara-report/ansible.sqlite"},{"line_number":30,"context_line":"ara-manage generate ~/zuul-logs/{{ inventory_hostname }}/ara-report/ara-html"},{"line_number":31,"context_line":"```"}],"source_content_type":"text/x-jinja2","patch_set":5,"id":"f8be8947_9cdb0ff4","line":28,"range":{"start_line":28,"start_character":0,"end_line":28,"end_character":22},"in_reply_to":"fbee1de8_68998dd1","updated":"2023-09-05 10:02:34.000000000","message":"Ack","commit_id":"73dcad3263b846318d684d717f9238a752158aee"},{"author":{"_account_id":24072,"name":"Marcin Juszkiewicz","email":"mjuszkiewicz@redhat.com","username":"hrw"},"change_message_id":"4bc8bfbc297fd5c9d41cfa6426913797ed229b24","unresolved":true,"context_lines":[{"line_number":33,"context_line":"Can I get the CI to generate the HTML for me?"},{"line_number":34,"context_line":"---------------------------------------------"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"Yes! If the commit message contains the string `#ara_verbose` then Zuul will"},{"line_number":37,"context_line":"render the html file and collect it in the logs. Beware: this adds hundreds of"},{"line_number":38,"context_line":"MiB to the log files. (Tip: the `#` shouldn\u0027t be the first character in the"},{"line_number":39,"context_line":"line)"}],"source_content_type":"text/x-jinja2","patch_set":5,"id":"5d3cf379_0a897d7f","line":36,"updated":"2023-09-05 11:04:28.000000000","message":"where is code to handle #ara_verbose? I see only part for #ara.","commit_id":"73dcad3263b846318d684d717f9238a752158aee"},{"author":{"_account_id":25733,"name":"Jan Gutter","email":"github@jangutter.com","username":"jangutter"},"change_message_id":"1377eaedf287d4774a1b57bdc7f3a0fa23f294f7","unresolved":true,"context_lines":[{"line_number":33,"context_line":"Can I get the CI to generate the HTML for me?"},{"line_number":34,"context_line":"---------------------------------------------"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"Yes! If the commit message contains the string `#ara_verbose` then Zuul will"},{"line_number":37,"context_line":"render the html file and collect it in the logs. Beware: this adds hundreds of"},{"line_number":38,"context_line":"MiB to the log files. (Tip: the `#` shouldn\u0027t be the first character in the"},{"line_number":39,"context_line":"line)"}],"source_content_type":"text/x-jinja2","patch_set":5,"id":"237bedaa_347ab43f","line":36,"in_reply_to":"5d3cf379_0a897d7f","updated":"2023-09-05 11:08:22.000000000","message":"It\u0027s in tests/post.yml when collecting the logs","commit_id":"73dcad3263b846318d684d717f9238a752158aee"}],"tests/post.yml":[{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"fa4b138ea20555ba23906a51b71e40311d3f1810","unresolved":true,"context_lines":[{"line_number":79,"context_line":""},{"line_number":80,"context_line":"        - name: get ara version"},{"line_number":81,"context_line":"          command: \"ara --version\""},{"line_number":82,"context_line":"          register: ara_version"},{"line_number":83,"context_line":""},{"line_number":84,"context_line":"        - name: template ara readme"},{"line_number":85,"context_line":"          template:"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"2fa75e31_be74ceec","line":82,"range":{"start_line":82,"start_character":1,"end_line":82,"end_character":31},"updated":"2023-08-17 13:23:29.000000000","message":"do we use it anywhere?","commit_id":"452983f0192401752351eaaeb11eb9ed20e11c55"},{"author":{"_account_id":25733,"name":"Jan Gutter","email":"github@jangutter.com","username":"jangutter"},"change_message_id":"80894b2182f5ccbc4438cf7827f890ee5859bdee","unresolved":false,"context_lines":[{"line_number":79,"context_line":""},{"line_number":80,"context_line":"        - name: get ara version"},{"line_number":81,"context_line":"          command: \"ara --version\""},{"line_number":82,"context_line":"          register: ara_version"},{"line_number":83,"context_line":""},{"line_number":84,"context_line":"        - name: template ara readme"},{"line_number":85,"context_line":"          template:"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"222ff166_aa2d4437","line":82,"range":{"start_line":82,"start_character":1,"end_line":82,"end_character":31},"in_reply_to":"2fa75e31_be74ceec","updated":"2023-08-17 13:27:49.000000000","message":"Yeah, we use this in the README.md template. It renders out like so:\npython3 -m pip install --user \"ara[server]\u003d\u003d1.6.1\"","commit_id":"452983f0192401752351eaaeb11eb9ed20e11c55"},{"author":{"_account_id":24072,"name":"Marcin Juszkiewicz","email":"mjuszkiewicz@redhat.com","username":"hrw"},"change_message_id":"4bc8bfbc297fd5c9d41cfa6426913797ed229b24","unresolved":true,"context_lines":[{"line_number":77,"context_line":"            dest: \"{{ ara_report_local_dir }}/\""},{"line_number":78,"context_line":"            mode: pull"},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"        - name: get ara version"},{"line_number":81,"context_line":"          command: \"ara --version\""},{"line_number":82,"context_line":"          register: ara_version"},{"line_number":83,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":5,"id":"f1bb0624_7de48231","line":80,"range":{"start_line":80,"start_character":20,"end_line":80,"end_character":23},"updated":"2023-09-05 11:04:28.000000000","message":"ARA","commit_id":"73dcad3263b846318d684d717f9238a752158aee"},{"author":{"_account_id":9061,"name":"David Moreau Simard","email":"moi@dmsimard.com","username":"dmsimard"},"change_message_id":"cc569759723e88b99d3c7836ebff45f6cc931511","unresolved":true,"context_lines":[{"line_number":77,"context_line":"            dest: \"{{ ara_report_local_dir }}/\""},{"line_number":78,"context_line":"            mode: pull"},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"        - name: get ara version"},{"line_number":81,"context_line":"          command: \"ara --version\""},{"line_number":82,"context_line":"          register: ara_version"},{"line_number":83,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":5,"id":"768e7460_d7b73c04","line":80,"range":{"start_line":80,"start_character":20,"end_line":80,"end_character":23},"in_reply_to":"49215801_773c35b9","updated":"2023-09-10 01:08:06.000000000","message":"It\u0027s impossible to always get the capitalization right, it\u0027s kind of an inside joke for the project. Just like sometimes it\u0027s Ansible and others it\u0027s ansible.","commit_id":"73dcad3263b846318d684d717f9238a752158aee"},{"author":{"_account_id":25733,"name":"Jan Gutter","email":"github@jangutter.com","username":"jangutter"},"change_message_id":"8047d24f56da4180276ce34d9dd5ce007b1ad174","unresolved":true,"context_lines":[{"line_number":77,"context_line":"            dest: \"{{ ara_report_local_dir }}/\""},{"line_number":78,"context_line":"            mode: pull"},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"        - name: get ara version"},{"line_number":81,"context_line":"          command: \"ara --version\""},{"line_number":82,"context_line":"          register: ara_version"},{"line_number":83,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":5,"id":"49215801_773c35b9","line":80,"range":{"start_line":80,"start_character":20,"end_line":80,"end_character":23},"in_reply_to":"f1bb0624_7de48231","updated":"2023-09-05 11:09:05.000000000","message":"Ack, will fix if a respin is needed.","commit_id":"73dcad3263b846318d684d717f9238a752158aee"},{"author":{"_account_id":25733,"name":"Jan Gutter","email":"github@jangutter.com","username":"jangutter"},"change_message_id":"1377eaedf287d4774a1b57bdc7f3a0fa23f294f7","unresolved":true,"context_lines":[{"line_number":90,"context_line":""},{"line_number":91,"context_line":"        - name: Generate HTML from ARA database"},{"line_number":92,"context_line":"          command: \"ara-manage generate {{ ansible_env.HOME }}/ara-html\""},{"line_number":93,"context_line":"          when: (zuul.change_message | default(\u0027\u0027)) is search(\u0027#ara_verbose\u0027)"},{"line_number":94,"context_line":""},{"line_number":95,"context_line":"        - name: Download ARA HTML"},{"line_number":96,"context_line":"          synchronize:"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"2fdda089_6ff4aaf4","line":93,"range":{"start_line":93,"start_character":10,"end_line":93,"end_character":77},"updated":"2023-09-05 11:08:22.000000000","message":"This bit matches #ara_verbose\n\n(it relies on the fact that `#ara_verbose` also matches `#ara` )","commit_id":"73dcad3263b846318d684d717f9238a752158aee"}],"tests/run.yml":[{"author":{"_account_id":32553,"name":"Sven Kieske","email":"sven_oss@posteo.de","username":"skieske"},"change_message_id":"dd86055dac2c488d1a3187351038f0940a09a921","unresolved":true,"context_lines":[{"line_number":260,"context_line":"          - \"{{ kolla_ansible_src_dir }}\""},{"line_number":261,"context_line":"          - \"ansible-core{{ ansible_core_version_constraint }}\""},{"line_number":262,"context_line":"          - \"ansible{{ ansible_version_constraint }}\""},{"line_number":263,"context_line":"          - \"ara[server]\u003c1.7\""},{"line_number":264,"context_line":"        virtualenv: \"{{ kolla_ansible_venv_path }}\""},{"line_number":265,"context_line":""},{"line_number":266,"context_line":"    - name: install Ansible collections"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"16591f88_90b9a058","line":263,"range":{"start_line":263,"start_character":0,"end_line":263,"end_character":7},"updated":"2023-09-05 08:41:13.000000000","message":"why do we pin ara to versions less than 1.7?","commit_id":"73dcad3263b846318d684d717f9238a752158aee"},{"author":{"_account_id":9061,"name":"David Moreau Simard","email":"moi@dmsimard.com","username":"dmsimard"},"change_message_id":"cc569759723e88b99d3c7836ebff45f6cc931511","unresolved":false,"context_lines":[{"line_number":260,"context_line":"          - \"{{ kolla_ansible_src_dir }}\""},{"line_number":261,"context_line":"          - \"ansible-core{{ ansible_core_version_constraint }}\""},{"line_number":262,"context_line":"          - \"ansible{{ ansible_version_constraint }}\""},{"line_number":263,"context_line":"          - \"ara[server]\u003c1.7\""},{"line_number":264,"context_line":"        virtualenv: \"{{ kolla_ansible_venv_path }}\""},{"line_number":265,"context_line":""},{"line_number":266,"context_line":"    - name: install Ansible collections"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"8d016a53_492149ac","line":263,"range":{"start_line":263,"start_character":0,"end_line":263,"end_character":7},"in_reply_to":"14f9e650_5f8f31b0","updated":"2023-09-10 01:08:06.000000000","message":"For info: I am expecting ara 1.7.0 to be out sometime early next week -- I have used this very patch to test the release candidate and it actually bubbled up an issue which has since been fixed: https://review.opendev.org/c/openstack/kolla-ansible/+/894416\n\nIn case you are curious what it looks like:\n- 1.7.0rc2: https://dev.demo.recordsansible.org/?path\u003dkolla-ansible\n- 1.6.1: https://demo.recordsansible.org/?path\u003dkolla-ansible","commit_id":"73dcad3263b846318d684d717f9238a752158aee"},{"author":{"_account_id":25733,"name":"Jan Gutter","email":"github@jangutter.com","username":"jangutter"},"change_message_id":"0628a4b607b751f18651c1ce3d232093549b38d7","unresolved":true,"context_lines":[{"line_number":260,"context_line":"          - \"{{ kolla_ansible_src_dir }}\""},{"line_number":261,"context_line":"          - \"ansible-core{{ ansible_core_version_constraint }}\""},{"line_number":262,"context_line":"          - \"ansible{{ ansible_version_constraint }}\""},{"line_number":263,"context_line":"          - \"ara[server]\u003c1.7\""},{"line_number":264,"context_line":"        virtualenv: \"{{ kolla_ansible_venv_path }}\""},{"line_number":265,"context_line":""},{"line_number":266,"context_line":"    - name: install Ansible collections"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"ce206599_08361b8d","line":263,"range":{"start_line":263,"start_character":0,"end_line":263,"end_character":7},"in_reply_to":"16591f88_90b9a058","updated":"2023-09-05 09:30:54.000000000","message":"ara isn\u0027t tested using the upper-constraints mechanism for openstack. This means it\u0027s usually wise to manually constrain it to a known working (and installable) range for a project.","commit_id":"73dcad3263b846318d684d717f9238a752158aee"},{"author":{"_account_id":32553,"name":"Sven Kieske","email":"sven_oss@posteo.de","username":"skieske"},"change_message_id":"b7ae36151dec3cd23181e06e6a5330a29b6b1903","unresolved":false,"context_lines":[{"line_number":260,"context_line":"          - \"{{ kolla_ansible_src_dir }}\""},{"line_number":261,"context_line":"          - \"ansible-core{{ ansible_core_version_constraint }}\""},{"line_number":262,"context_line":"          - \"ansible{{ ansible_version_constraint }}\""},{"line_number":263,"context_line":"          - \"ara[server]\u003c1.7\""},{"line_number":264,"context_line":"        virtualenv: \"{{ kolla_ansible_venv_path }}\""},{"line_number":265,"context_line":""},{"line_number":266,"context_line":"    - name: install Ansible collections"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"14f9e650_5f8f31b0","line":263,"range":{"start_line":263,"start_character":0,"end_line":263,"end_character":7},"in_reply_to":"ce206599_08361b8d","updated":"2023-09-05 10:02:34.000000000","message":"Ack","commit_id":"73dcad3263b846318d684d717f9238a752158aee"}]}