)]}'
{"neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/ovsdb_monitor.py":[{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"bf9a512ecc6588596dd683c924407a279ef7f47e","unresolved":false,"context_lines":[{"line_number":51,"context_line":"    import collections"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"    from ovs.db import data"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"    def IndexEntryClass(table):"},{"line_number":56,"context_line":"        def defaults_uuid_to_row(atom, base):"},{"line_number":57,"context_line":"            return atom.value"}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_f9994f6b","line":54,"updated":"2020-10-02 09:01:36.000000000","message":"Maybe we can document this is a copy of https://github.com/openvswitch/ovs/blob/4357b6bca4e5cec47919fba584b7acd20b8f2e17/python/ovs/db/custom_index.py#L127 so no modifications shall be done here?","commit_id":"9680c9b0edd845a8320fd6459fba44778acea110"},{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"bf9a512ecc6588596dd683c924407a279ef7f47e","unresolved":false,"context_lines":[{"line_number":65,"context_line":"        return cls"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"if not hasattr(idlutils, \u0027frozen_row\u0027):"},{"line_number":68,"context_line":"    def frozen_row(row):"},{"line_number":69,"context_line":"        IndexEntry \u003d IndexEntryClass(row._table)"},{"line_number":70,"context_line":"        return IndexEntry("},{"line_number":71,"context_line":"            uuid\u003drow.uuid,"}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_b9b197e3","line":68,"updated":"2020-10-02 09:01:36.000000000","message":"Can we just bump the requirements.txt for ovsdbapp once the patch is merged instead?","commit_id":"9680c9b0edd845a8320fd6459fba44778acea110"},{"author":{"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},"change_message_id":"a7fcaf1192ada39b99adc950c312964686bb0a41","unresolved":false,"context_lines":[{"line_number":65,"context_line":"        return cls"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"if not hasattr(idlutils, \u0027frozen_row\u0027):"},{"line_number":68,"context_line":"    def frozen_row(row):"},{"line_number":69,"context_line":"        IndexEntry \u003d IndexEntryClass(row._table)"},{"line_number":70,"context_line":"        return IndexEntry("},{"line_number":71,"context_line":"            uuid\u003drow.uuid,"}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_f06022d2","line":68,"in_reply_to":"9f560f44_b9b197e3","updated":"2020-10-05 18:13:59.000000000","message":"Yeah, seems like the way to go. I could also put the IndexEntryClass handling in ovsdbapp as well. Initially put this patch up because successive patches needed it for tests to pass and getting releases made sometimes takes a while. But getting anything through CI right now takes a while, so ... :p","commit_id":"9680c9b0edd845a8320fd6459fba44778acea110"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"d69055fcc97c45afbeffd4da5321e254d1edb181","unresolved":false,"context_lines":[{"line_number":35,"context_line":"from neutron.conf.plugins.ml2.drivers.ovn import ovn_conf"},{"line_number":36,"context_line":"from neutron.db import ovn_hash_ring_db"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"CONF \u003d cfg.CONF"},{"line_number":40,"context_line":"LOG \u003d log.getLogger(__name__)"},{"line_number":41,"context_line":""}],"source_content_type":"text/x-python","patch_set":10,"id":"5f681702_e2a4210b","line":38,"updated":"2020-10-19 14:06:21.000000000","message":"Just a note, without a backward compatibility with older versions of the ovsdbapp it will be difficult to backport this fix to stable branches (due to the global-requirement cap). Is that acceptable ?\n\nJust want to ask before voting cause I think it\u0027s important to have in mind.","commit_id":"af9265bf65159dfe395300a8563697e904b7c891"},{"author":{"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},"change_message_id":"71359a7966822258a3e0c6550acf567bf661721e","unresolved":false,"context_lines":[{"line_number":35,"context_line":"from neutron.conf.plugins.ml2.drivers.ovn import ovn_conf"},{"line_number":36,"context_line":"from neutron.db import ovn_hash_ring_db"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"CONF \u003d cfg.CONF"},{"line_number":40,"context_line":"LOG \u003d log.getLogger(__name__)"},{"line_number":41,"context_line":""}],"source_content_type":"text/x-python","patch_set":10,"id":"5f681702_e5f26b06","line":38,"in_reply_to":"5f681702_e2a4210b","updated":"2020-10-19 14:51:52.000000000","message":"The ovsdbapp backports of the fix have mostly been merged. just train remains https://review.opendev.org/#/c/755857/4. So after the stable bugfix releases, I think we just need to bump the lower-constraint to point to the bugfix release. If we need to copy stuff in-tree for a while we can, though.","commit_id":"af9265bf65159dfe395300a8563697e904b7c891"}]}