)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"580f994c098ab221d4d92ab518c382dea2a7dca0","unresolved":true,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This change is a partial revert of"},{"line_number":10,"context_line":"Ibf8dca4bd57b3bddb39955b53cc03564506f5754"},{"line_number":11,"context_line":"to reintoduce a try excpet which is required for"},{"line_number":12,"context_line":"some non standard hardware."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"On the Cavium ThunderX plathform its possible to have"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"be2409fb_928ea87a","line":11,"range":{"start_line":11,"start_character":20,"end_line":11,"end_character":26},"updated":"2021-05-25 16:46:36.000000000","message":"try-except","commit_id":"209cc60393368a9c6a11232c6d41ef912934b286"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"a0f6d6d44de4bfa6cfb4caf93f931b841c2bdd9c","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This change is a partial revert of"},{"line_number":10,"context_line":"Ibf8dca4bd57b3bddb39955b53cc03564506f5754"},{"line_number":11,"context_line":"to reintoduce a try excpet which is required for"},{"line_number":12,"context_line":"some non standard hardware."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"On the Cavium ThunderX plathform its possible to have"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"b371f1bf_1e6d13dc","line":11,"range":{"start_line":11,"start_character":20,"end_line":11,"end_character":26},"in_reply_to":"be2409fb_928ea87a","updated":"2021-06-28 12:48:00.000000000","message":"Done","commit_id":"209cc60393368a9c6a11232c6d41ef912934b286"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"580f994c098ab221d4d92ab518c382dea2a7dca0","unresolved":true,"context_lines":[{"line_number":11,"context_line":"to reintoduce a try excpet which is required for"},{"line_number":12,"context_line":"some non standard hardware."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"On the Cavium ThunderX plathform its possible to have"},{"line_number":15,"context_line":"Virutal function which are netdevs which are not associated"},{"line_number":16,"context_line":"to a PF. This causes the PF name lookup to fail."},{"line_number":17,"context_line":"Prior to Ibf8dca4bd57b3bddb39955b53cc03564506f5754"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"01e31b3e_2eab222b","line":14,"range":{"start_line":14,"start_character":23,"end_line":14,"end_character":32},"updated":"2021-05-25 16:46:36.000000000","message":"platform, it\u0027s","commit_id":"209cc60393368a9c6a11232c6d41ef912934b286"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"a0f6d6d44de4bfa6cfb4caf93f931b841c2bdd9c","unresolved":false,"context_lines":[{"line_number":11,"context_line":"to reintoduce a try excpet which is required for"},{"line_number":12,"context_line":"some non standard hardware."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"On the Cavium ThunderX plathform its possible to have"},{"line_number":15,"context_line":"Virutal function which are netdevs which are not associated"},{"line_number":16,"context_line":"to a PF. This causes the PF name lookup to fail."},{"line_number":17,"context_line":"Prior to Ibf8dca4bd57b3bddb39955b53cc03564506f5754"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"3428932e_15d9cd94","line":14,"range":{"start_line":14,"start_character":23,"end_line":14,"end_character":32},"in_reply_to":"01e31b3e_2eab222b","updated":"2021-06-28 12:48:00.000000000","message":"Done","commit_id":"209cc60393368a9c6a11232c6d41ef912934b286"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"580f994c098ab221d4d92ab518c382dea2a7dca0","unresolved":true,"context_lines":[{"line_number":12,"context_line":"some non standard hardware."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"On the Cavium ThunderX plathform its possible to have"},{"line_number":15,"context_line":"Virutal function which are netdevs which are not associated"},{"line_number":16,"context_line":"to a PF. This causes the PF name lookup to fail."},{"line_number":17,"context_line":"Prior to Ibf8dca4bd57b3bddb39955b53cc03564506f5754"},{"line_number":18,"context_line":"when the lookup failed it was caught and we skipped"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"b30ebfa1_ad3e2bb7","line":15,"range":{"start_line":15,"start_character":0,"end_line":15,"end_character":16},"updated":"2021-05-25 16:46:36.000000000","message":"virtual functions","commit_id":"209cc60393368a9c6a11232c6d41ef912934b286"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"a0f6d6d44de4bfa6cfb4caf93f931b841c2bdd9c","unresolved":false,"context_lines":[{"line_number":12,"context_line":"some non standard hardware."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"On the Cavium ThunderX plathform its possible to have"},{"line_number":15,"context_line":"Virutal function which are netdevs which are not associated"},{"line_number":16,"context_line":"to a PF. This causes the PF name lookup to fail."},{"line_number":17,"context_line":"Prior to Ibf8dca4bd57b3bddb39955b53cc03564506f5754"},{"line_number":18,"context_line":"when the lookup failed it was caught and we skipped"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"13f8feb4_a9f04a28","line":15,"range":{"start_line":15,"start_character":0,"end_line":15,"end_character":16},"in_reply_to":"b30ebfa1_ad3e2bb7","updated":"2021-06-28 12:48:00.000000000","message":"Done","commit_id":"209cc60393368a9c6a11232c6d41ef912934b286"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"580f994c098ab221d4d92ab518c382dea2a7dca0","unresolved":false,"context_lines":[{"line_number":14,"context_line":"On the Cavium ThunderX plathform its possible to have"},{"line_number":15,"context_line":"Virutal function which are netdevs which are not associated"},{"line_number":16,"context_line":"to a PF. This causes the PF name lookup to fail."},{"line_number":17,"context_line":"Prior to Ibf8dca4bd57b3bddb39955b53cc03564506f5754"},{"line_number":18,"context_line":"when the lookup failed it was caught and we skipped"},{"line_number":19,"context_line":"populating the parent PF interface name."},{"line_number":20,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"764aae43_20ea6a53","line":17,"range":{"start_line":17,"start_character":9,"end_line":17,"end_character":50},"updated":"2021-05-25 16:46:36.000000000","message":"Good. This was you, not me 😄","commit_id":"209cc60393368a9c6a11232c6d41ef912934b286"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"8df85add7b4ddc51d425916da70a4b9ea543a077","unresolved":false,"context_lines":[{"line_number":14,"context_line":"On the Cavium ThunderX plathform its possible to have"},{"line_number":15,"context_line":"Virutal function which are netdevs which are not associated"},{"line_number":16,"context_line":"to a PF. This causes the PF name lookup to fail."},{"line_number":17,"context_line":"Prior to Ibf8dca4bd57b3bddb39955b53cc03564506f5754"},{"line_number":18,"context_line":"when the lookup failed it was caught and we skipped"},{"line_number":19,"context_line":"populating the parent PF interface name."},{"line_number":20,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"22732b71_9e813b13","line":17,"range":{"start_line":17,"start_character":9,"end_line":17,"end_character":50},"in_reply_to":"764aae43_20ea6a53","updated":"2021-06-28 11:35:14.000000000","message":"yep it was me.","commit_id":"209cc60393368a9c6a11232c6d41ef912934b286"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"580f994c098ab221d4d92ab518c382dea2a7dca0","unresolved":true,"context_lines":[{"line_number":18,"context_line":"when the lookup failed it was caught and we skipped"},{"line_number":19,"context_line":"populating the parent PF interface name."},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"This change restores that behavior."},{"line_number":22,"context_line":"Closes-Bug: #1915255"},{"line_number":23,"context_line":"Change-Id: Ia10ccdd9fbed3870d0592e3cbbff17f292651dd2"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"6a0be5fb_64fc8602","line":21,"updated":"2021-05-25 16:46:36.000000000","message":"nit: missing newline","commit_id":"209cc60393368a9c6a11232c6d41ef912934b286"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"a0f6d6d44de4bfa6cfb4caf93f931b841c2bdd9c","unresolved":false,"context_lines":[{"line_number":18,"context_line":"when the lookup failed it was caught and we skipped"},{"line_number":19,"context_line":"populating the parent PF interface name."},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"This change restores that behavior."},{"line_number":22,"context_line":"Closes-Bug: #1915255"},{"line_number":23,"context_line":"Change-Id: Ia10ccdd9fbed3870d0592e3cbbff17f292651dd2"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"d9902e08_d785ed02","line":21,"in_reply_to":"6a0be5fb_64fc8602","updated":"2021-06-28 12:48:00.000000000","message":"Done","commit_id":"209cc60393368a9c6a11232c6d41ef912934b286"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"526b10f4828f6f9917b0a393faa63fd856d8a94c","unresolved":true,"context_lines":[{"line_number":12,"context_line":"some non standard hardware."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"On the Cavium ThunderX platform, it\u0027s possible to have"},{"line_number":15,"context_line":"virutal functions which are netdevs which are not associated"},{"line_number":16,"context_line":"to a PF. This causes the PF name lookup to fail."},{"line_number":17,"context_line":"Prior to Ibf8dca4bd57b3bddb39955b53cc03564506f5754"},{"line_number":18,"context_line":"when the lookup failed it was caught and we skipped"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"60d060e5_1bab9527","line":15,"range":{"start_line":15,"start_character":0,"end_line":15,"end_character":7},"updated":"2021-06-29 15:57:04.000000000","message":"virtual","commit_id":"a569a51fedd058fdae2eb0066e087c37688987f8"}],"nova/virt/libvirt/driver.py":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"c7a7b38698788e31293b0992f0093a5f6aa1e13c","unresolved":true,"context_lines":[{"line_number":7143,"context_line":"                        # NOTE(sean-k-mooney): we ignore this error as it"},{"line_number":7144,"context_line":"                        # is expected when the virtual function is not a NIC"},{"line_number":7145,"context_line":"                        # or the VF does not have a parent PF with a netdev."},{"line_number":7146,"context_line":"                        pass"},{"line_number":7147,"context_line":"                    return result"},{"line_number":7148,"context_line":""},{"line_number":7149,"context_line":"            return {\u0027dev_type\u0027: fields.PciDeviceType.STANDARD}"}],"source_content_type":"text/x-python","patch_set":2,"id":"b1cedc89_2a5a7df9","line":7146,"updated":"2021-03-09 16:37:26.000000000","message":"Can you reduce the area covered by the try-except and add a debug logging statement to help admins?\n\n  ifname \u003d None\n  if device.name() in net_dev_parents:\n      try:\n          ifname \u003d pci_utils.get_ifname_by_pci_address(\n              pci_address, pf_interface\u003dTrue))\n      except exception.PciDeviceNotFoundById:\n          LOG.debug(...)\n\n      result[\u0027parent_ifname\u0027] \u003d parent_ifname\n\nWe might also want to make the setting of \u0027parent_ifname\u0027 in the result dependent on it being not-none, but I checked and the only user of this value, \u0027nova.pci.stats\u0027, has a truth\u0027y guard so that isn\u0027t really necessary.","commit_id":"0a3e03aee7b812ec5a36d70d72a62e63c0b92f44"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"5fae70fcf04def22aada800a60d77b449e0156dd","unresolved":true,"context_lines":[{"line_number":7143,"context_line":"                        # NOTE(sean-k-mooney): we ignore this error as it"},{"line_number":7144,"context_line":"                        # is expected when the virtual function is not a NIC"},{"line_number":7145,"context_line":"                        # or the VF does not have a parent PF with a netdev."},{"line_number":7146,"context_line":"                        pass"},{"line_number":7147,"context_line":"                    return result"},{"line_number":7148,"context_line":""},{"line_number":7149,"context_line":"            return {\u0027dev_type\u0027: fields.PciDeviceType.STANDARD}"}],"source_content_type":"text/x-python","patch_set":2,"id":"824d467e_7068459d","line":7146,"in_reply_to":"b1cedc89_2a5a7df9","updated":"2021-03-09 16:59:39.000000000","message":"parent_ifname should not be none correct but i guess i could reduce the scope.\nthere is nothing that an admin can do in the cavium case to fix this so it would just be spam.\nthe except is pass now because its techincally valid for the lookup to fail and there is nothing we can do to workaround it so the log is probably not useful.\n\nthis is called in a perodic too so im not sure we should log this even at debug as it would just be noise.","commit_id":"0a3e03aee7b812ec5a36d70d72a62e63c0b92f44"}],"nova/virt/libvirt/host.py":[{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"9e732a201319a25d140e7bda56b30f281a980c24","unresolved":true,"context_lines":[{"line_number":1285,"context_line":"                            # NIC or the VF does not have a parent PF with a"},{"line_number":1286,"context_line":"                            # netdev. We do not log here as this is called"},{"line_number":1287,"context_line":"                            # in a periodic task and that would be noisy at"},{"line_number":1288,"context_line":"                            # debug level."},{"line_number":1289,"context_line":"                            pass"},{"line_number":1290,"context_line":"                    if device.name() in vdpa_parents:"},{"line_number":1291,"context_line":"                        result[\u0027dev_type\u0027] \u003d fields.PciDeviceType.VDPA"}],"source_content_type":"text/x-python","patch_set":3,"id":"4d107b59_916a486c","line":1288,"updated":"2021-06-18 15:26:38.000000000","message":"I foresee a bug that if such VFs are used together QoS min bw policies in the direct port then the scheduling will fail as it will not find any VFs based on the parent_ifname.","commit_id":"209cc60393368a9c6a11232c6d41ef912934b286"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"a0f6d6d44de4bfa6cfb4caf93f931b841c2bdd9c","unresolved":true,"context_lines":[{"line_number":1285,"context_line":"                            # NIC or the VF does not have a parent PF with a"},{"line_number":1286,"context_line":"                            # netdev. We do not log here as this is called"},{"line_number":1287,"context_line":"                            # in a periodic task and that would be noisy at"},{"line_number":1288,"context_line":"                            # debug level."},{"line_number":1289,"context_line":"                            pass"},{"line_number":1290,"context_line":"                    if device.name() in vdpa_parents:"},{"line_number":1291,"context_line":"                        result[\u0027dev_type\u0027] \u003d fields.PciDeviceType.VDPA"}],"source_content_type":"text/x-python","patch_set":3,"id":"f152da0a_98c2a463","line":1288,"in_reply_to":"4d107b59_916a486c","updated":"2021-06-28 12:48:00.000000000","message":"its not a bug.\nthis is unsupported since the platform does not conform to the minium requirements for usign\nthat feature.\n\nwe have noted this in the release note already, as noted in the top level comment we shoudl eventrually \ntry to remove teh use of the pf name on the neutorn side also and just use the pci adress if we want to support qos on this plathform in the future.\n\nbefore Ibf8dca4bd57b3bddb39955b53cc03564506f5754 we  ignored this error so this is just restoring that  pre-existing behaviour.","commit_id":"209cc60393368a9c6a11232c6d41ef912934b286"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"987ea7a0f6c25695496498228e919fc37cf923b6","unresolved":true,"context_lines":[{"line_number":1285,"context_line":"                            # NIC or the VF does not have a parent PF with a"},{"line_number":1286,"context_line":"                            # netdev. We do not log here as this is called"},{"line_number":1287,"context_line":"                            # in a periodic task and that would be noisy at"},{"line_number":1288,"context_line":"                            # debug level."},{"line_number":1289,"context_line":"                            pass"},{"line_number":1290,"context_line":"                    if device.name() in vdpa_parents:"},{"line_number":1291,"context_line":"                        result[\u0027dev_type\u0027] \u003d fields.PciDeviceType.VDPA"}],"source_content_type":"text/x-python","patch_set":3,"id":"6ba85a16_f5b55c35","line":1288,"in_reply_to":"f152da0a_98c2a463","updated":"2021-06-28 13:03:38.000000000","message":"Thanks Sean. I\u0027ve also pushed a patch to Neutron doc to document this limitation https://review.opendev.org/c/openstack/neutron/+/798294","commit_id":"209cc60393368a9c6a11232c6d41ef912934b286"}],"releasenotes/notes/fix-pci-passthrough-for-cavium-thunderx-8fbd1c40718569e2.yaml":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"c7a7b38698788e31293b0992f0093a5f6aa1e13c","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"fixes:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    On some hardware platform a sriov virtual function for a nic port may"},{"line_number":5,"context_line":"    exist without being associated to a parent physical function that has"},{"line_number":6,"context_line":"    an assocatied netdev. In such a case the the PF interface name lookup"},{"line_number":7,"context_line":"    will fail. As the ``PciDeviceNotFoundById`` excption was not handeled"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"5e889cb4_ed6f4656","line":4,"range":{"start_line":4,"start_character":61,"end_line":4,"end_character":64},"updated":"2021-03-09 16:37:26.000000000","message":"NIC","commit_id":"0a3e03aee7b812ec5a36d70d72a62e63c0b92f44"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"c7a7b38698788e31293b0992f0093a5f6aa1e13c","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"fixes:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    On some hardware platform a sriov virtual function for a nic port may"},{"line_number":5,"context_line":"    exist without being associated to a parent physical function that has"},{"line_number":6,"context_line":"    an assocatied netdev. In such a case the the PF interface name lookup"},{"line_number":7,"context_line":"    will fail. As the ``PciDeviceNotFoundById`` excption was not handeled"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"583097d9_52eaefe8","line":4,"range":{"start_line":4,"start_character":32,"end_line":4,"end_character":37},"updated":"2021-03-09 16:37:26.000000000","message":"SR-IOV\n\ny u h8 caps? 😄","commit_id":"0a3e03aee7b812ec5a36d70d72a62e63c0b92f44"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"c7a7b38698788e31293b0992f0093a5f6aa1e13c","unresolved":true,"context_lines":[{"line_number":4,"context_line":"    On some hardware platform a sriov virtual function for a nic port may"},{"line_number":5,"context_line":"    exist without being associated to a parent physical function that has"},{"line_number":6,"context_line":"    an assocatied netdev. In such a case the the PF interface name lookup"},{"line_number":7,"context_line":"    will fail. As the ``PciDeviceNotFoundById`` excption was not handeled"},{"line_number":8,"context_line":"    this would prevent the nova compute agent from starting on affected"},{"line_number":9,"context_line":"    hardware, see: https://bugs.launchpad.net/nova/+bug/1915255 for more"},{"line_number":10,"context_line":"    details. This edgecase has now been adressed however features"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"29da5511_dd798b6c","line":7,"range":{"start_line":7,"start_character":48,"end_line":7,"end_character":56},"updated":"2021-03-09 16:37:26.000000000","message":"exception","commit_id":"0a3e03aee7b812ec5a36d70d72a62e63c0b92f44"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"c7a7b38698788e31293b0992f0093a5f6aa1e13c","unresolved":true,"context_lines":[{"line_number":4,"context_line":"    On some hardware platform a sriov virtual function for a nic port may"},{"line_number":5,"context_line":"    exist without being associated to a parent physical function that has"},{"line_number":6,"context_line":"    an assocatied netdev. In such a case the the PF interface name lookup"},{"line_number":7,"context_line":"    will fail. As the ``PciDeviceNotFoundById`` excption was not handeled"},{"line_number":8,"context_line":"    this would prevent the nova compute agent from starting on affected"},{"line_number":9,"context_line":"    hardware, see: https://bugs.launchpad.net/nova/+bug/1915255 for more"},{"line_number":10,"context_line":"    details. This edgecase has now been adressed however features"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"615bb0a4_dfc8c339","line":7,"range":{"start_line":7,"start_character":65,"end_line":7,"end_character":73},"updated":"2021-03-09 16:37:26.000000000","message":"handled","commit_id":"0a3e03aee7b812ec5a36d70d72a62e63c0b92f44"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"c7a7b38698788e31293b0992f0093a5f6aa1e13c","unresolved":true,"context_lines":[{"line_number":6,"context_line":"    an assocatied netdev. In such a case the the PF interface name lookup"},{"line_number":7,"context_line":"    will fail. As the ``PciDeviceNotFoundById`` excption was not handeled"},{"line_number":8,"context_line":"    this would prevent the nova compute agent from starting on affected"},{"line_number":9,"context_line":"    hardware, see: https://bugs.launchpad.net/nova/+bug/1915255 for more"},{"line_number":10,"context_line":"    details. This edgecase has now been adressed however features"},{"line_number":11,"context_line":"    that depend on the PF name such as minium bandwith based QOS cannot"},{"line_number":12,"context_line":"    be supported on these platforms."}],"source_content_type":"text/x-yaml","patch_set":2,"id":"7c20af6e_e1f0301f","line":9,"range":{"start_line":9,"start_character":12,"end_line":9,"end_character":15},"updated":"2021-03-09 16:37:26.000000000","message":". S","commit_id":"0a3e03aee7b812ec5a36d70d72a62e63c0b92f44"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"c7a7b38698788e31293b0992f0093a5f6aa1e13c","unresolved":true,"context_lines":[{"line_number":8,"context_line":"    this would prevent the nova compute agent from starting on affected"},{"line_number":9,"context_line":"    hardware, see: https://bugs.launchpad.net/nova/+bug/1915255 for more"},{"line_number":10,"context_line":"    details. This edgecase has now been adressed however features"},{"line_number":11,"context_line":"    that depend on the PF name such as minium bandwith based QOS cannot"},{"line_number":12,"context_line":"    be supported on these platforms."}],"source_content_type":"text/x-yaml","patch_set":2,"id":"bfc9b728_5204a88a","line":11,"range":{"start_line":11,"start_character":61,"end_line":11,"end_character":64},"updated":"2021-03-09 16:37:26.000000000","message":"QoS","commit_id":"0a3e03aee7b812ec5a36d70d72a62e63c0b92f44"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"c7a7b38698788e31293b0992f0093a5f6aa1e13c","unresolved":true,"context_lines":[{"line_number":8,"context_line":"    this would prevent the nova compute agent from starting on affected"},{"line_number":9,"context_line":"    hardware, see: https://bugs.launchpad.net/nova/+bug/1915255 for more"},{"line_number":10,"context_line":"    details. This edgecase has now been adressed however features"},{"line_number":11,"context_line":"    that depend on the PF name such as minium bandwith based QOS cannot"},{"line_number":12,"context_line":"    be supported on these platforms."}],"source_content_type":"text/x-yaml","patch_set":2,"id":"b1342e5e_0bee5f91","line":11,"range":{"start_line":11,"start_character":39,"end_line":11,"end_character":45},"updated":"2021-03-09 16:37:26.000000000","message":"minimum","commit_id":"0a3e03aee7b812ec5a36d70d72a62e63c0b92f44"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"580f994c098ab221d4d92ab518c382dea2a7dca0","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"fixes:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    On some hardware platform a SR-IOV virtual function for a NIC port may"},{"line_number":5,"context_line":"    exist without being associated to a parent physical function that has"},{"line_number":6,"context_line":"    an assocatied netdev. In such a case the the PF interface name lookup"},{"line_number":7,"context_line":"    will fail. As the ``PciDeviceNotFoundById`` exception was not handled"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"a300a5d5_5ccc9560","line":4,"range":{"start_line":4,"start_character":21,"end_line":4,"end_character":29},"updated":"2021-05-25 16:46:36.000000000","message":"platforms, an","commit_id":"209cc60393368a9c6a11232c6d41ef912934b286"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"a0f6d6d44de4bfa6cfb4caf93f931b841c2bdd9c","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"fixes:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    On some hardware platform a SR-IOV virtual function for a NIC port may"},{"line_number":5,"context_line":"    exist without being associated to a parent physical function that has"},{"line_number":6,"context_line":"    an assocatied netdev. In such a case the the PF interface name lookup"},{"line_number":7,"context_line":"    will fail. As the ``PciDeviceNotFoundById`` exception was not handled"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"3e3b240d_0b440c64","line":4,"range":{"start_line":4,"start_character":21,"end_line":4,"end_character":29},"in_reply_to":"a300a5d5_5ccc9560","updated":"2021-06-28 12:48:00.000000000","message":"Done","commit_id":"209cc60393368a9c6a11232c6d41ef912934b286"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"580f994c098ab221d4d92ab518c382dea2a7dca0","unresolved":true,"context_lines":[{"line_number":2,"context_line":"fixes:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    On some hardware platform a SR-IOV virtual function for a NIC port may"},{"line_number":5,"context_line":"    exist without being associated to a parent physical function that has"},{"line_number":6,"context_line":"    an assocatied netdev. In such a case the the PF interface name lookup"},{"line_number":7,"context_line":"    will fail. As the ``PciDeviceNotFoundById`` exception was not handled"},{"line_number":8,"context_line":"    this would prevent the nova compute agent from starting on affected"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"84870a8c_c4bf8263","line":5,"range":{"start_line":5,"start_character":35,"end_line":5,"end_character":37},"updated":"2021-05-25 16:46:36.000000000","message":"with ?","commit_id":"209cc60393368a9c6a11232c6d41ef912934b286"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"a0f6d6d44de4bfa6cfb4caf93f931b841c2bdd9c","unresolved":false,"context_lines":[{"line_number":2,"context_line":"fixes:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    On some hardware platform a SR-IOV virtual function for a NIC port may"},{"line_number":5,"context_line":"    exist without being associated to a parent physical function that has"},{"line_number":6,"context_line":"    an assocatied netdev. In such a case the the PF interface name lookup"},{"line_number":7,"context_line":"    will fail. As the ``PciDeviceNotFoundById`` exception was not handled"},{"line_number":8,"context_line":"    this would prevent the nova compute agent from starting on affected"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"8dc6720f_61daea97","line":5,"range":{"start_line":5,"start_character":35,"end_line":5,"end_character":37},"in_reply_to":"84870a8c_c4bf8263","updated":"2021-06-28 12:48:00.000000000","message":"Done","commit_id":"209cc60393368a9c6a11232c6d41ef912934b286"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"580f994c098ab221d4d92ab518c382dea2a7dca0","unresolved":true,"context_lines":[{"line_number":7,"context_line":"    will fail. As the ``PciDeviceNotFoundById`` exception was not handled"},{"line_number":8,"context_line":"    this would prevent the nova compute agent from starting on affected"},{"line_number":9,"context_line":"    hardware. See: https://bugs.launchpad.net/nova/+bug/1915255 for more"},{"line_number":10,"context_line":"    details. This edgecase has now been addressed however features"},{"line_number":11,"context_line":"    that depend on the PF name such as minimum bandwidth based QoS cannot"},{"line_number":12,"context_line":"    be supported on these platforms."}],"source_content_type":"text/x-yaml","patch_set":3,"id":"f588a69a_4c5ac2b8","line":10,"range":{"start_line":10,"start_character":49,"end_line":10,"end_character":58},"updated":"2021-05-25 16:46:36.000000000","message":", however,","commit_id":"209cc60393368a9c6a11232c6d41ef912934b286"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"a0f6d6d44de4bfa6cfb4caf93f931b841c2bdd9c","unresolved":false,"context_lines":[{"line_number":7,"context_line":"    will fail. As the ``PciDeviceNotFoundById`` exception was not handled"},{"line_number":8,"context_line":"    this would prevent the nova compute agent from starting on affected"},{"line_number":9,"context_line":"    hardware. See: https://bugs.launchpad.net/nova/+bug/1915255 for more"},{"line_number":10,"context_line":"    details. This edgecase has now been addressed however features"},{"line_number":11,"context_line":"    that depend on the PF name such as minimum bandwidth based QoS cannot"},{"line_number":12,"context_line":"    be supported on these platforms."}],"source_content_type":"text/x-yaml","patch_set":3,"id":"979b2c56_3f82977c","line":10,"range":{"start_line":10,"start_character":49,"end_line":10,"end_character":58},"in_reply_to":"f588a69a_4c5ac2b8","updated":"2021-06-28 12:48:00.000000000","message":"Done","commit_id":"209cc60393368a9c6a11232c6d41ef912934b286"}]}