)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":32553,"name":"Sven Kieske","email":"sven_oss@posteo.de","username":"skieske"},"change_message_id":"c1fc74a8ae0691ef7ed609cd65e9c9049785d92a","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     Kevin Honka \u003ck.honka@aditsystems.de\u003e"},{"line_number":5,"context_line":"CommitDate: 2024-03-13 13:42:58 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Force IPv4 only for ansible_host and ansible_ssh_host"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"By default the openstack inventory fetches the first fixed ip"},{"line_number":10,"context_line":"address for \"ansible_host\" and \"ansible_ssh_host\"."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"02db739a_61ef8910","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":2},"updated":"2024-03-13 16:56:12.000000000","message":"I would rather summarize this change as:\n`add an option to use only IPv4 addresses for ansible_host and ansible_ssh_host`\n\nbecause it\u0027s not really forced, is it?","commit_id":"a973a7d87707bfcc9581f7be162e6f4d59f43c36"},{"author":{"_account_id":36886,"name":"Kevin Honka","email":"k.honka@aditsystems.de","username":"khonka"},"change_message_id":"6c9f9c858a02d107270531e3f7da51677da9eb72","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Kevin Honka \u003ck.honka@aditsystems.de\u003e"},{"line_number":5,"context_line":"CommitDate: 2024-03-13 13:42:58 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Force IPv4 only for ansible_host and ansible_ssh_host"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"By default the openstack inventory fetches the first fixed ip"},{"line_number":10,"context_line":"address for \"ansible_host\" and \"ansible_ssh_host\"."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"b3f613e0_0ac80cff","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":2},"in_reply_to":"02db739a_61ef8910","updated":"2024-04-05 09:35:26.000000000","message":"Acknowledged","commit_id":"a973a7d87707bfcc9581f7be162e6f4d59f43c36"},{"author":{"_account_id":32553,"name":"Sven Kieske","email":"sven_oss@posteo.de","username":"skieske"},"change_message_id":"5573f6f18a08f48d324162771ef28fb711186f0d","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Kevin Honka \u003ck.honka@aditsystems.de\u003e"},{"line_number":5,"context_line":"CommitDate: 2024-03-13 13:42:58 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Force IPv4 only for ansible_host and ansible_ssh_host"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"By default the openstack inventory fetches the first fixed ip"},{"line_number":10,"context_line":"address for \"ansible_host\" and \"ansible_ssh_host\"."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"a82788cc_8c0760a0","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":2},"in_reply_to":"02db739a_61ef8910","updated":"2024-03-14 08:59:02.000000000","message":"Done","commit_id":"a973a7d87707bfcc9581f7be162e6f4d59f43c36"},{"author":{"_account_id":32553,"name":"Sven Kieske","email":"sven_oss@posteo.de","username":"skieske"},"change_message_id":"c1fc74a8ae0691ef7ed609cd65e9c9049785d92a","unresolved":true,"context_lines":[{"line_number":13,"context_line":"where Ansible runs in a ipv4 only setup,"},{"line_number":14,"context_line":"but OpenStack is already ipv6 enabled."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"To prevent this, I added a new option called \"ipv4_only\","},{"line_number":17,"context_line":"which forces the inventory plugin to use only fixed ipv4 addresses."},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Closes-Bug: #2051249"},{"line_number":20,"context_line":"Change-Id: I3aa9868c9299705d4b0dcbf9b9cb561c0855c11c"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"6dfa8fbb_1b8c9dbd","line":17,"range":{"start_line":16,"start_character":0,"end_line":17,"end_character":67},"updated":"2024-03-13 16:56:12.000000000","message":"nit: In general it is discouraged to use \"I\" in documentation, however I must admit that it\u0027s not strictly documented for commit messages. For reference see:\n\nhttps://docs.openstack.org/doc-contrib-guide/writing-style/general-writing-guidelines.html#write-in-second-person\n\nWe could rephrase this as:\n\n```\nTo prevent this, an option called `ipv4_only` was added, which forces[...]\n```\n\nNotice also that you should in general quote variables in docs, text etc with backticks (in rst formatted text in double backticks).","commit_id":"a973a7d87707bfcc9581f7be162e6f4d59f43c36"},{"author":{"_account_id":36886,"name":"Kevin Honka","email":"k.honka@aditsystems.de","username":"khonka"},"change_message_id":"6c9f9c858a02d107270531e3f7da51677da9eb72","unresolved":false,"context_lines":[{"line_number":13,"context_line":"where Ansible runs in a ipv4 only setup,"},{"line_number":14,"context_line":"but OpenStack is already ipv6 enabled."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"To prevent this, I added a new option called \"ipv4_only\","},{"line_number":17,"context_line":"which forces the inventory plugin to use only fixed ipv4 addresses."},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Closes-Bug: #2051249"},{"line_number":20,"context_line":"Change-Id: I3aa9868c9299705d4b0dcbf9b9cb561c0855c11c"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"b0862e8f_a61ff14f","line":17,"range":{"start_line":16,"start_character":0,"end_line":17,"end_character":67},"in_reply_to":"6dfa8fbb_1b8c9dbd","updated":"2024-04-05 09:35:26.000000000","message":"Done","commit_id":"a973a7d87707bfcc9581f7be162e6f4d59f43c36"},{"author":{"_account_id":32553,"name":"Sven Kieske","email":"sven_oss@posteo.de","username":"skieske"},"change_message_id":"5573f6f18a08f48d324162771ef28fb711186f0d","unresolved":false,"context_lines":[{"line_number":13,"context_line":"where Ansible runs in a ipv4 only setup,"},{"line_number":14,"context_line":"but OpenStack is already ipv6 enabled."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"To prevent this, I added a new option called \"ipv4_only\","},{"line_number":17,"context_line":"which forces the inventory plugin to use only fixed ipv4 addresses."},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Closes-Bug: #2051249"},{"line_number":20,"context_line":"Change-Id: I3aa9868c9299705d4b0dcbf9b9cb561c0855c11c"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"d48c39df_70b36c14","line":17,"range":{"start_line":16,"start_character":0,"end_line":17,"end_character":67},"in_reply_to":"6dfa8fbb_1b8c9dbd","updated":"2024-03-14 08:59:02.000000000","message":"Done","commit_id":"a973a7d87707bfcc9581f7be162e6f4d59f43c36"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":32553,"name":"Sven Kieske","email":"sven_oss@posteo.de","username":"skieske"},"change_message_id":"5a7dd50861ae66af33131f54715795e67350adf2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"4ea56ebf_c0128bfc","updated":"2024-03-13 13:42:09.000000000","message":"recheck broken CI should be fixed now by https://review.opendev.org/c/openstack/ansible-collections-openstack/+/912684","commit_id":"b839ec23b44ae676a77a7c18731b2ab0fdfa0da5"},{"author":{"_account_id":36886,"name":"Kevin Honka","email":"k.honka@aditsystems.de","username":"khonka"},"change_message_id":"6c9f9c858a02d107270531e3f7da51677da9eb72","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"fbe8fd89_9248dd72","in_reply_to":"4ea56ebf_c0128bfc","updated":"2024-04-05 09:35:26.000000000","message":"Done.","commit_id":"b839ec23b44ae676a77a7c18731b2ab0fdfa0da5"},{"author":{"_account_id":32553,"name":"Sven Kieske","email":"sven_oss@posteo.de","username":"skieske"},"change_message_id":"c1fc74a8ae0691ef7ed609cd65e9c9049785d92a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"40fec638_acf1ff40","updated":"2024-03-13 16:56:12.000000000","message":"Welcome and thanks for your first contribution!\n\nI have made some comments below, the change itself looks imho good, but could use some small adjustments.","commit_id":"a973a7d87707bfcc9581f7be162e6f4d59f43c36"}],"plugins/inventory/openstack.py":[{"author":{"_account_id":36886,"name":"Kevin Honka","email":"k.honka@aditsystems.de","username":"khonka"},"change_message_id":"6c9f9c858a02d107270531e3f7da51677da9eb72","unresolved":false,"context_lines":[{"line_number":391,"context_line":"                None)"},{"line_number":392,"context_line":""},{"line_number":393,"context_line":"            if self.get_option(\u0027ipv4_only\u0027):"},{"line_number":394,"context_line":"              fixed_ip \u003d next("},{"line_number":395,"context_line":"                  (address[\u0027addr\u0027] for address in addresses"},{"line_number":396,"context_line":"                  if (address[\u0027OS-EXT-IPS:type\u0027] \u003d\u003d \u0027fixed\u0027 and"},{"line_number":397,"context_line":"                      address[\u0027version\u0027] \u003d\u003d 4)),"}],"source_content_type":"text/x-python","patch_set":1,"id":"70f98071_b73ff17d","line":394,"in_reply_to":"175bbbc1_69c82268","updated":"2024-04-05 09:35:26.000000000","message":"\u003e linters_2_12: E111 indentation is not a multiple of 4\n\nPlease fix.","commit_id":"d62f8ebf016a63050501c626d64cdd223e563ad0"},{"author":{"_account_id":36886,"name":"Kevin Honka","email":"k.honka@aditsystems.de","username":"khonka"},"change_message_id":"6c9f9c858a02d107270531e3f7da51677da9eb72","unresolved":false,"context_lines":[{"line_number":391,"context_line":"                None)"},{"line_number":392,"context_line":""},{"line_number":393,"context_line":"            if self.get_option(\u0027ipv4_only\u0027):"},{"line_number":394,"context_line":"              fixed_ip \u003d next("},{"line_number":395,"context_line":"                  (address[\u0027addr\u0027] for address in addresses"},{"line_number":396,"context_line":"                  if (address[\u0027OS-EXT-IPS:type\u0027] \u003d\u003d \u0027fixed\u0027 and"},{"line_number":397,"context_line":"                      address[\u0027version\u0027] \u003d\u003d 4)),"}],"source_content_type":"text/x-python","patch_set":1,"id":"812cfc40_a496cb67","line":394,"in_reply_to":"2feb3997_5044ac38","updated":"2024-04-05 09:35:26.000000000","message":"\u003e linters_latest: E111 indentation is not a multiple of 4\n\nPlease fix.","commit_id":"d62f8ebf016a63050501c626d64cdd223e563ad0"},{"author":{"_account_id":36886,"name":"Kevin Honka","email":"k.honka@aditsystems.de","username":"khonka"},"change_message_id":"6c9f9c858a02d107270531e3f7da51677da9eb72","unresolved":false,"context_lines":[{"line_number":391,"context_line":"                None)"},{"line_number":392,"context_line":""},{"line_number":393,"context_line":"            if self.get_option(\u0027ipv4_only\u0027):"},{"line_number":394,"context_line":"              fixed_ip \u003d next("},{"line_number":395,"context_line":"                  (address[\u0027addr\u0027] for address in addresses"},{"line_number":396,"context_line":"                  if (address[\u0027OS-EXT-IPS:type\u0027] \u003d\u003d \u0027fixed\u0027 and"},{"line_number":397,"context_line":"                      address[\u0027version\u0027] \u003d\u003d 4)),"}],"source_content_type":"text/x-python","patch_set":1,"id":"e60a9040_79c570c5","line":394,"in_reply_to":"ef90c16c_4256741b","updated":"2024-04-05 09:35:26.000000000","message":"\u003e pep8: E111 indentation is not a multiple of 4\n\nPlease fix.","commit_id":"d62f8ebf016a63050501c626d64cdd223e563ad0"},{"author":{"_account_id":36886,"name":"Kevin Honka","email":"k.honka@aditsystems.de","username":"khonka"},"change_message_id":"6c9f9c858a02d107270531e3f7da51677da9eb72","unresolved":false,"context_lines":[{"line_number":393,"context_line":"            if self.get_option(\u0027ipv4_only\u0027):"},{"line_number":394,"context_line":"              fixed_ip \u003d next("},{"line_number":395,"context_line":"                  (address[\u0027addr\u0027] for address in addresses"},{"line_number":396,"context_line":"                  if (address[\u0027OS-EXT-IPS:type\u0027] \u003d\u003d \u0027fixed\u0027 and"},{"line_number":397,"context_line":"                      address[\u0027version\u0027] \u003d\u003d 4)),"},{"line_number":398,"context_line":"                  None)"},{"line_number":399,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"fb8cb336_7319daf4","line":396,"in_reply_to":"192ce9fa_98f987a7","updated":"2024-04-05 09:35:26.000000000","message":"\u003e linters_latest: W504 line break after binary operator\n\nPlease fix.","commit_id":"d62f8ebf016a63050501c626d64cdd223e563ad0"},{"author":{"_account_id":36886,"name":"Kevin Honka","email":"k.honka@aditsystems.de","username":"khonka"},"change_message_id":"6c9f9c858a02d107270531e3f7da51677da9eb72","unresolved":false,"context_lines":[{"line_number":393,"context_line":"            if self.get_option(\u0027ipv4_only\u0027):"},{"line_number":394,"context_line":"              fixed_ip \u003d next("},{"line_number":395,"context_line":"                  (address[\u0027addr\u0027] for address in addresses"},{"line_number":396,"context_line":"                  if (address[\u0027OS-EXT-IPS:type\u0027] \u003d\u003d \u0027fixed\u0027 and"},{"line_number":397,"context_line":"                      address[\u0027version\u0027] \u003d\u003d 4)),"},{"line_number":398,"context_line":"                  None)"},{"line_number":399,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"f7db5d00_d1963841","line":396,"in_reply_to":"7307df1b_cbe1e64f","updated":"2024-04-05 09:35:26.000000000","message":"\u003e linters_latest: E128 continuation line under-indented for visual indent\n\nPlease fix.","commit_id":"d62f8ebf016a63050501c626d64cdd223e563ad0"},{"author":{"_account_id":36886,"name":"Kevin Honka","email":"k.honka@aditsystems.de","username":"khonka"},"change_message_id":"6c9f9c858a02d107270531e3f7da51677da9eb72","unresolved":false,"context_lines":[{"line_number":393,"context_line":"            if self.get_option(\u0027ipv4_only\u0027):"},{"line_number":394,"context_line":"              fixed_ip \u003d next("},{"line_number":395,"context_line":"                  (address[\u0027addr\u0027] for address in addresses"},{"line_number":396,"context_line":"                  if (address[\u0027OS-EXT-IPS:type\u0027] \u003d\u003d \u0027fixed\u0027 and"},{"line_number":397,"context_line":"                      address[\u0027version\u0027] \u003d\u003d 4)),"},{"line_number":398,"context_line":"                  None)"},{"line_number":399,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"933a7770_4e9b7e13","line":396,"in_reply_to":"7dffafd9_d862dc44","updated":"2024-04-05 09:35:26.000000000","message":"\u003e linters_2_12: E128 continuation line under-indented for visual indent\n\nPlease fix.","commit_id":"d62f8ebf016a63050501c626d64cdd223e563ad0"},{"author":{"_account_id":36886,"name":"Kevin Honka","email":"k.honka@aditsystems.de","username":"khonka"},"change_message_id":"6c9f9c858a02d107270531e3f7da51677da9eb72","unresolved":false,"context_lines":[{"line_number":393,"context_line":"            if self.get_option(\u0027ipv4_only\u0027):"},{"line_number":394,"context_line":"              fixed_ip \u003d next("},{"line_number":395,"context_line":"                  (address[\u0027addr\u0027] for address in addresses"},{"line_number":396,"context_line":"                  if (address[\u0027OS-EXT-IPS:type\u0027] \u003d\u003d \u0027fixed\u0027 and"},{"line_number":397,"context_line":"                      address[\u0027version\u0027] \u003d\u003d 4)),"},{"line_number":398,"context_line":"                  None)"},{"line_number":399,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"dba91afb_7bedd4a1","line":396,"in_reply_to":"95a6da22_935708e1","updated":"2024-04-05 09:35:26.000000000","message":"\u003e pep8: E128 continuation line under-indented for visual indent\n\nPlease fix.","commit_id":"d62f8ebf016a63050501c626d64cdd223e563ad0"},{"author":{"_account_id":36886,"name":"Kevin Honka","email":"k.honka@aditsystems.de","username":"khonka"},"change_message_id":"6c9f9c858a02d107270531e3f7da51677da9eb72","unresolved":false,"context_lines":[{"line_number":393,"context_line":"            if self.get_option(\u0027ipv4_only\u0027):"},{"line_number":394,"context_line":"              fixed_ip \u003d next("},{"line_number":395,"context_line":"                  (address[\u0027addr\u0027] for address in addresses"},{"line_number":396,"context_line":"                  if (address[\u0027OS-EXT-IPS:type\u0027] \u003d\u003d \u0027fixed\u0027 and"},{"line_number":397,"context_line":"                      address[\u0027version\u0027] \u003d\u003d 4)),"},{"line_number":398,"context_line":"                  None)"},{"line_number":399,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"37cbc9fe_6f740437","line":396,"in_reply_to":"a194ae3c_abc6f70c","updated":"2024-04-05 09:35:26.000000000","message":"\u003e linters_2_12: W504 line break after binary operator\n\nPlease fix.","commit_id":"d62f8ebf016a63050501c626d64cdd223e563ad0"},{"author":{"_account_id":36886,"name":"Kevin Honka","email":"k.honka@aditsystems.de","username":"khonka"},"change_message_id":"6c9f9c858a02d107270531e3f7da51677da9eb72","unresolved":false,"context_lines":[{"line_number":393,"context_line":"            if self.get_option(\u0027ipv4_only\u0027):"},{"line_number":394,"context_line":"              fixed_ip \u003d next("},{"line_number":395,"context_line":"                  (address[\u0027addr\u0027] for address in addresses"},{"line_number":396,"context_line":"                  if (address[\u0027OS-EXT-IPS:type\u0027] \u003d\u003d \u0027fixed\u0027 and"},{"line_number":397,"context_line":"                      address[\u0027version\u0027] \u003d\u003d 4)),"},{"line_number":398,"context_line":"                  None)"},{"line_number":399,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"9422aa70_1b1d49b4","line":396,"in_reply_to":"b79f421b_70e03467","updated":"2024-04-05 09:35:26.000000000","message":"\u003e pep8: W504 line break after binary operator\n\nPlease fix.","commit_id":"d62f8ebf016a63050501c626d64cdd223e563ad0"},{"author":{"_account_id":36886,"name":"Kevin Honka","email":"k.honka@aditsystems.de","username":"khonka"},"change_message_id":"6c9f9c858a02d107270531e3f7da51677da9eb72","unresolved":false,"context_lines":[{"line_number":398,"context_line":"                  None)"},{"line_number":399,"context_line":""},{"line_number":400,"context_line":"            else:"},{"line_number":401,"context_line":"              fixed_ip \u003d next("},{"line_number":402,"context_line":"                  (address[\u0027addr\u0027] for address in addresses"},{"line_number":403,"context_line":"                  if address[\u0027OS-EXT-IPS:type\u0027] \u003d\u003d \u0027fixed\u0027),"},{"line_number":404,"context_line":"                  None)"}],"source_content_type":"text/x-python","patch_set":1,"id":"3e8da019_60ea130a","line":401,"in_reply_to":"55b876e9_3cc0f730","updated":"2024-04-05 09:35:26.000000000","message":"\u003e linters_2_12: E111 indentation is not a multiple of 4\n\nPlease fix.","commit_id":"d62f8ebf016a63050501c626d64cdd223e563ad0"},{"author":{"_account_id":36886,"name":"Kevin Honka","email":"k.honka@aditsystems.de","username":"khonka"},"change_message_id":"6c9f9c858a02d107270531e3f7da51677da9eb72","unresolved":false,"context_lines":[{"line_number":398,"context_line":"                  None)"},{"line_number":399,"context_line":""},{"line_number":400,"context_line":"            else:"},{"line_number":401,"context_line":"              fixed_ip \u003d next("},{"line_number":402,"context_line":"                  (address[\u0027addr\u0027] for address in addresses"},{"line_number":403,"context_line":"                  if address[\u0027OS-EXT-IPS:type\u0027] \u003d\u003d \u0027fixed\u0027),"},{"line_number":404,"context_line":"                  None)"}],"source_content_type":"text/x-python","patch_set":1,"id":"79f3b4e7_bc2abf45","line":401,"in_reply_to":"6d9a95f0_745068c4","updated":"2024-04-05 09:35:26.000000000","message":"\u003e pep8: E111 indentation is not a multiple of 4\n\nPlease fix.","commit_id":"d62f8ebf016a63050501c626d64cdd223e563ad0"},{"author":{"_account_id":36886,"name":"Kevin Honka","email":"k.honka@aditsystems.de","username":"khonka"},"change_message_id":"6c9f9c858a02d107270531e3f7da51677da9eb72","unresolved":false,"context_lines":[{"line_number":398,"context_line":"                  None)"},{"line_number":399,"context_line":""},{"line_number":400,"context_line":"            else:"},{"line_number":401,"context_line":"              fixed_ip \u003d next("},{"line_number":402,"context_line":"                  (address[\u0027addr\u0027] for address in addresses"},{"line_number":403,"context_line":"                  if address[\u0027OS-EXT-IPS:type\u0027] \u003d\u003d \u0027fixed\u0027),"},{"line_number":404,"context_line":"                  None)"}],"source_content_type":"text/x-python","patch_set":1,"id":"caedc349_1753eabc","line":401,"in_reply_to":"b6f47944_6414a067","updated":"2024-04-05 09:35:26.000000000","message":"\u003e linters_latest: E111 indentation is not a multiple of 4\n\nPlease fix.","commit_id":"d62f8ebf016a63050501c626d64cdd223e563ad0"},{"author":{"_account_id":36886,"name":"Kevin Honka","email":"k.honka@aditsystems.de","username":"khonka"},"change_message_id":"6c9f9c858a02d107270531e3f7da51677da9eb72","unresolved":false,"context_lines":[{"line_number":400,"context_line":"            else:"},{"line_number":401,"context_line":"              fixed_ip \u003d next("},{"line_number":402,"context_line":"                  (address[\u0027addr\u0027] for address in addresses"},{"line_number":403,"context_line":"                  if address[\u0027OS-EXT-IPS:type\u0027] \u003d\u003d \u0027fixed\u0027),"},{"line_number":404,"context_line":"                  None)"},{"line_number":405,"context_line":""},{"line_number":406,"context_line":"            ip \u003d floating_ip if floating_ip is not None and not self.get_option(\u0027private\u0027) else fixed_ip"}],"source_content_type":"text/x-python","patch_set":1,"id":"bd5cd252_2656cc8c","line":403,"in_reply_to":"87a17f39_6824ff3d","updated":"2024-04-05 09:35:26.000000000","message":"\u003e linters_latest: E128 continuation line under-indented for visual indent\n\nPlease fix.","commit_id":"d62f8ebf016a63050501c626d64cdd223e563ad0"},{"author":{"_account_id":36886,"name":"Kevin Honka","email":"k.honka@aditsystems.de","username":"khonka"},"change_message_id":"6c9f9c858a02d107270531e3f7da51677da9eb72","unresolved":false,"context_lines":[{"line_number":400,"context_line":"            else:"},{"line_number":401,"context_line":"              fixed_ip \u003d next("},{"line_number":402,"context_line":"                  (address[\u0027addr\u0027] for address in addresses"},{"line_number":403,"context_line":"                  if address[\u0027OS-EXT-IPS:type\u0027] \u003d\u003d \u0027fixed\u0027),"},{"line_number":404,"context_line":"                  None)"},{"line_number":405,"context_line":""},{"line_number":406,"context_line":"            ip \u003d floating_ip if floating_ip is not None and not self.get_option(\u0027private\u0027) else fixed_ip"}],"source_content_type":"text/x-python","patch_set":1,"id":"8a48f0da_3fbd1d21","line":403,"in_reply_to":"8d11e9bc_badacace","updated":"2024-04-05 09:35:26.000000000","message":"\u003e linters_2_12: E128 continuation line under-indented for visual indent\n\nPlease fix.","commit_id":"d62f8ebf016a63050501c626d64cdd223e563ad0"},{"author":{"_account_id":36886,"name":"Kevin Honka","email":"k.honka@aditsystems.de","username":"khonka"},"change_message_id":"6c9f9c858a02d107270531e3f7da51677da9eb72","unresolved":false,"context_lines":[{"line_number":400,"context_line":"            else:"},{"line_number":401,"context_line":"              fixed_ip \u003d next("},{"line_number":402,"context_line":"                  (address[\u0027addr\u0027] for address in addresses"},{"line_number":403,"context_line":"                  if address[\u0027OS-EXT-IPS:type\u0027] \u003d\u003d \u0027fixed\u0027),"},{"line_number":404,"context_line":"                  None)"},{"line_number":405,"context_line":""},{"line_number":406,"context_line":"            ip \u003d floating_ip if floating_ip is not None and not self.get_option(\u0027private\u0027) else fixed_ip"}],"source_content_type":"text/x-python","patch_set":1,"id":"8b7b5a30_78bafa24","line":403,"in_reply_to":"cafa33ae_001575a8","updated":"2024-04-05 09:35:26.000000000","message":"\u003e pep8: E128 continuation line under-indented for visual indent\n\nPlease fix.","commit_id":"d62f8ebf016a63050501c626d64cdd223e563ad0"},{"author":{"_account_id":36886,"name":"Kevin Honka","email":"k.honka@aditsystems.de","username":"khonka"},"change_message_id":"6c9f9c858a02d107270531e3f7da51677da9eb72","unresolved":false,"context_lines":[{"line_number":393,"context_line":"            if self.get_option(\u0027ipv4_only\u0027):"},{"line_number":394,"context_line":"                fixed_ip \u003d next("},{"line_number":395,"context_line":"                    (address[\u0027addr\u0027] for address in addresses"},{"line_number":396,"context_line":"                    if (address[\u0027OS-EXT-IPS:type\u0027] \u003d\u003d \u0027fixed\u0027 and address[\u0027version\u0027] \u003d\u003d 4)),"},{"line_number":397,"context_line":"                    None)"},{"line_number":398,"context_line":""},{"line_number":399,"context_line":"            else:"}],"source_content_type":"text/x-python","patch_set":3,"id":"7f120daf_7064e2d7","line":396,"in_reply_to":"1dc6a77c_75e64b2a","updated":"2024-04-05 09:35:26.000000000","message":"\u003e linters_latest: E128 continuation line under-indented for visual indent\n\nPlease fix.","commit_id":"e65e4e09610ad6e41d836dae7da5898230cccae7"},{"author":{"_account_id":36886,"name":"Kevin Honka","email":"k.honka@aditsystems.de","username":"khonka"},"change_message_id":"6c9f9c858a02d107270531e3f7da51677da9eb72","unresolved":false,"context_lines":[{"line_number":393,"context_line":"            if self.get_option(\u0027ipv4_only\u0027):"},{"line_number":394,"context_line":"                fixed_ip \u003d next("},{"line_number":395,"context_line":"                    (address[\u0027addr\u0027] for address in addresses"},{"line_number":396,"context_line":"                    if (address[\u0027OS-EXT-IPS:type\u0027] \u003d\u003d \u0027fixed\u0027 and address[\u0027version\u0027] \u003d\u003d 4)),"},{"line_number":397,"context_line":"                    None)"},{"line_number":398,"context_line":""},{"line_number":399,"context_line":"            else:"}],"source_content_type":"text/x-python","patch_set":3,"id":"8e3029e1_1deb5539","line":396,"in_reply_to":"7ccc9ebb_36b3c988","updated":"2024-04-05 09:35:26.000000000","message":"\u003e linters_2_12: E128 continuation line under-indented for visual indent\n\nPlease fix.","commit_id":"e65e4e09610ad6e41d836dae7da5898230cccae7"},{"author":{"_account_id":36886,"name":"Kevin Honka","email":"k.honka@aditsystems.de","username":"khonka"},"change_message_id":"6c9f9c858a02d107270531e3f7da51677da9eb72","unresolved":false,"context_lines":[{"line_number":393,"context_line":"            if self.get_option(\u0027ipv4_only\u0027):"},{"line_number":394,"context_line":"                fixed_ip \u003d next("},{"line_number":395,"context_line":"                    (address[\u0027addr\u0027] for address in addresses"},{"line_number":396,"context_line":"                    if (address[\u0027OS-EXT-IPS:type\u0027] \u003d\u003d \u0027fixed\u0027 and address[\u0027version\u0027] \u003d\u003d 4)),"},{"line_number":397,"context_line":"                    None)"},{"line_number":398,"context_line":""},{"line_number":399,"context_line":"            else:"}],"source_content_type":"text/x-python","patch_set":3,"id":"4052a0dc_1078a3e2","line":396,"in_reply_to":"f4d31ecd_7a111280","updated":"2024-04-05 09:35:26.000000000","message":"\u003e pep8: E128 continuation line under-indented for visual indent\n\nPlease fix.","commit_id":"e65e4e09610ad6e41d836dae7da5898230cccae7"},{"author":{"_account_id":32553,"name":"Sven Kieske","email":"sven_oss@posteo.de","username":"skieske"},"change_message_id":"c1fc74a8ae0691ef7ed609cd65e9c9049785d92a","unresolved":true,"context_lines":[{"line_number":96,"context_line":"        only."},{"line_number":97,"context_line":"    type: bool"},{"line_number":98,"context_line":"    default: false"},{"line_number":99,"context_line":"  ipv4_only:"},{"line_number":100,"context_line":"    description:"},{"line_number":101,"context_line":"      - Use only ipv4 addresses for ansible_host and ansible_ssh_host."},{"line_number":102,"context_line":"      - Using I(ipv4_only) helps when running Ansible in a ipv4 only setup."}],"source_content_type":"text/x-python","patch_set":6,"id":"4cf5af78_e036cb8f","line":99,"range":{"start_line":99,"start_character":2,"end_line":99,"end_character":11},"updated":"2024-03-13 16:56:12.000000000","message":"nit: the ordering of the words seems to be different for other options, so we should keep this consistent, e.g. there is `only_clouds`, not `clouds_only`\n\nso imho this option should be called `only_ipv4` I guess? I\u0027m not that familiar with this part of the code base though, so maybe someone more experienced can chime in.","commit_id":"a973a7d87707bfcc9581f7be162e6f4d59f43c36"},{"author":{"_account_id":32553,"name":"Sven Kieske","email":"sven_oss@posteo.de","username":"skieske"},"change_message_id":"5573f6f18a08f48d324162771ef28fb711186f0d","unresolved":false,"context_lines":[{"line_number":96,"context_line":"        only."},{"line_number":97,"context_line":"    type: bool"},{"line_number":98,"context_line":"    default: false"},{"line_number":99,"context_line":"  ipv4_only:"},{"line_number":100,"context_line":"    description:"},{"line_number":101,"context_line":"      - Use only ipv4 addresses for ansible_host and ansible_ssh_host."},{"line_number":102,"context_line":"      - Using I(ipv4_only) helps when running Ansible in a ipv4 only setup."}],"source_content_type":"text/x-python","patch_set":6,"id":"5495c318_86dba8a3","line":99,"range":{"start_line":99,"start_character":2,"end_line":99,"end_character":11},"in_reply_to":"4cf5af78_e036cb8f","updated":"2024-03-14 08:59:02.000000000","message":"Done","commit_id":"a973a7d87707bfcc9581f7be162e6f4d59f43c36"},{"author":{"_account_id":36886,"name":"Kevin Honka","email":"k.honka@aditsystems.de","username":"khonka"},"change_message_id":"6c9f9c858a02d107270531e3f7da51677da9eb72","unresolved":false,"context_lines":[{"line_number":96,"context_line":"        only."},{"line_number":97,"context_line":"    type: bool"},{"line_number":98,"context_line":"    default: false"},{"line_number":99,"context_line":"  ipv4_only:"},{"line_number":100,"context_line":"    description:"},{"line_number":101,"context_line":"      - Use only ipv4 addresses for ansible_host and ansible_ssh_host."},{"line_number":102,"context_line":"      - Using I(ipv4_only) helps when running Ansible in a ipv4 only setup."}],"source_content_type":"text/x-python","patch_set":6,"id":"7c541589_68caff2d","line":99,"range":{"start_line":99,"start_character":2,"end_line":99,"end_character":11},"in_reply_to":"4cf5af78_e036cb8f","updated":"2024-04-05 09:35:26.000000000","message":"Done","commit_id":"a973a7d87707bfcc9581f7be162e6f4d59f43c36"}]}