)]}'
{"zuul.d/tempest-multinode.yaml":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"febf864bcd63ab3c11d0174acbd5ca79c1ce3aaa","unresolved":true,"context_lines":[{"line_number":193,"context_line":"        neutron-tempest-plugin: https://opendev.org/openstack/neutron-tempest-plugin"},{"line_number":194,"context_line":"      zuul_copy_output:"},{"line_number":195,"context_line":"        \u0027{{ devstack_base_dir }}/data/ovs\u0027: \u0027logs\u0027"},{"line_number":196,"context_line":"        \u0027{{ devstack_base_dir }}/data/ovn\u0027: \u0027logs\u0027"},{"line_number":197,"context_line":"        \u0027/var/log/ovn\u0027: \u0027logs\u0027"},{"line_number":198,"context_line":"      extensions_to_txt:"},{"line_number":199,"context_line":"        db: true"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"873746de_b5eb90da","line":196,"updated":"2021-01-20 22:21:06.000000000","message":"I guess these are supposed to copy the actual DBs?  I don\u0027t see anything in the logs, just empty directories, so maybe aren\u0027t needed?","commit_id":"c9292636c5d5f83403e434460d712c5739f3aa52"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"2d58d70e86f7eddd0a2f99f6d0efb1b6a1252316","unresolved":true,"context_lines":[{"line_number":193,"context_line":"        neutron-tempest-plugin: https://opendev.org/openstack/neutron-tempest-plugin"},{"line_number":194,"context_line":"      zuul_copy_output:"},{"line_number":195,"context_line":"        \u0027{{ devstack_base_dir }}/data/ovs\u0027: \u0027logs\u0027"},{"line_number":196,"context_line":"        \u0027{{ devstack_base_dir }}/data/ovn\u0027: \u0027logs\u0027"},{"line_number":197,"context_line":"        \u0027/var/log/ovn\u0027: \u0027logs\u0027"},{"line_number":198,"context_line":"      extensions_to_txt:"},{"line_number":199,"context_line":"        db: true"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"4eb9bc18_63ef2ff8","line":196,"in_reply_to":"3f56a0aa_f3979603","updated":"2021-01-29 13:40:41.000000000","message":"I just saw Lucas filed a similar change, I pointed him here since there might be additions.  It might answer the question about why the DBs aren\u0027t here.","commit_id":"c9292636c5d5f83403e434460d712c5739f3aa52"},{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"eda84eec2224b283143a2847dddaea604c844252","unresolved":true,"context_lines":[{"line_number":193,"context_line":"        neutron-tempest-plugin: https://opendev.org/openstack/neutron-tempest-plugin"},{"line_number":194,"context_line":"      zuul_copy_output:"},{"line_number":195,"context_line":"        \u0027{{ devstack_base_dir }}/data/ovs\u0027: \u0027logs\u0027"},{"line_number":196,"context_line":"        \u0027{{ devstack_base_dir }}/data/ovn\u0027: \u0027logs\u0027"},{"line_number":197,"context_line":"        \u0027/var/log/ovn\u0027: \u0027logs\u0027"},{"line_number":198,"context_line":"      extensions_to_txt:"},{"line_number":199,"context_line":"        db: true"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"b35f3c13_96c9ace2","line":196,"in_reply_to":"7bbccb63_f965f023","updated":"2021-01-25 08:35:04.000000000","message":"My apologies for my answer not being clear. Seems like now OVN uses default location to store the DB files. The default is different to /opt/stack/data/ovn and that\u0027s why the directory is empty. Not sure when it changed and why you see the DB files there, I have a fresh Ubuntu based devstack box and it\u0027s empty.\n\nAre you saying this should be addressed in this patch too?","commit_id":"c9292636c5d5f83403e434460d712c5739f3aa52"},{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"5066d3d163e33207c780b12abefc3e927942fd2d","unresolved":true,"context_lines":[{"line_number":193,"context_line":"        neutron-tempest-plugin: https://opendev.org/openstack/neutron-tempest-plugin"},{"line_number":194,"context_line":"      zuul_copy_output:"},{"line_number":195,"context_line":"        \u0027{{ devstack_base_dir }}/data/ovs\u0027: \u0027logs\u0027"},{"line_number":196,"context_line":"        \u0027{{ devstack_base_dir }}/data/ovn\u0027: \u0027logs\u0027"},{"line_number":197,"context_line":"        \u0027/var/log/ovn\u0027: \u0027logs\u0027"},{"line_number":198,"context_line":"      extensions_to_txt:"},{"line_number":199,"context_line":"        db: true"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"9b51eed9_832c238e","line":196,"in_reply_to":"873746de_b5eb90da","updated":"2021-01-21 12:03:48.000000000","message":"That\u0027s right, seems like defaults for the DBs location are used.","commit_id":"c9292636c5d5f83403e434460d712c5739f3aa52"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"0a2516045326af661495a35a273e490bf810646d","unresolved":true,"context_lines":[{"line_number":193,"context_line":"        neutron-tempest-plugin: https://opendev.org/openstack/neutron-tempest-plugin"},{"line_number":194,"context_line":"      zuul_copy_output:"},{"line_number":195,"context_line":"        \u0027{{ devstack_base_dir }}/data/ovs\u0027: \u0027logs\u0027"},{"line_number":196,"context_line":"        \u0027{{ devstack_base_dir }}/data/ovn\u0027: \u0027logs\u0027"},{"line_number":197,"context_line":"        \u0027/var/log/ovn\u0027: \u0027logs\u0027"},{"line_number":198,"context_line":"      extensions_to_txt:"},{"line_number":199,"context_line":"        db: true"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"7bbccb63_f965f023","line":196,"in_reply_to":"9b51eed9_832c238e","updated":"2021-01-21 14:26:23.000000000","message":"I guess this didn\u0027t answer my question.  On my destack if I look in ~stack/data/ovn I see the NB and SB DBs.  I didn\u0027t see them in the tempest controller log directory, unless I just missed them?  There were just two empty directories.","commit_id":"c9292636c5d5f83403e434460d712c5739f3aa52"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"459c9526c90b1ae6548cb847f3928c0da0fac24a","unresolved":true,"context_lines":[{"line_number":193,"context_line":"        neutron-tempest-plugin: https://opendev.org/openstack/neutron-tempest-plugin"},{"line_number":194,"context_line":"      zuul_copy_output:"},{"line_number":195,"context_line":"        \u0027{{ devstack_base_dir }}/data/ovs\u0027: \u0027logs\u0027"},{"line_number":196,"context_line":"        \u0027{{ devstack_base_dir }}/data/ovn\u0027: \u0027logs\u0027"},{"line_number":197,"context_line":"        \u0027/var/log/ovn\u0027: \u0027logs\u0027"},{"line_number":198,"context_line":"      extensions_to_txt:"},{"line_number":199,"context_line":"        db: true"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3f56a0aa_f3979603","line":196,"in_reply_to":"b35f3c13_96c9ace2","updated":"2021-01-25 19:36:45.000000000","message":"Maybe it\u0027s easier to just put a link to what I\u0027m talking about.  In the tempest log directories on the controller, there are two empty directories:\n\n./ovn/ovn\n./ovs\n\nhttps://594c7ba7d2e29fea7770-96f2f0e219ea986d39b2d7f15ad9596a.ssl.cf1.rackcdn.com/771658/1/check/neutron-ovn-tempest-ovs-release/2109e55/controller/logs/index.html\n\nI think those are created from the above lines, which is why I wonder if they are needed.\n\nI actually can\u0027t find the DBs anywhere in the logs directory in this run.","commit_id":"c9292636c5d5f83403e434460d712c5739f3aa52"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"5934a282a7bf05cfe2e94c1910d9d2cd7b53fc73","unresolved":true,"context_lines":[{"line_number":194,"context_line":"      zuul_copy_output:"},{"line_number":195,"context_line":"        \u0027{{ devstack_base_dir }}/data/ovs\u0027: \u0027logs\u0027"},{"line_number":196,"context_line":"        \u0027{{ devstack_base_dir }}/data/ovn\u0027: \u0027logs\u0027"},{"line_number":197,"context_line":"        \u0027/var/log/ovn\u0027: \u0027logs\u0027"},{"line_number":198,"context_line":"      extensions_to_txt:"},{"line_number":199,"context_line":"        db: true"},{"line_number":200,"context_line":"      devstack_services:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"6b41fc45_2750b7b5","line":197,"updated":"2021-01-29 13:53:07.000000000","message":"The dbs should be located in /var/lib/ovn by default with packages, I am testing it at https://review.opendev.org/c/openstack/neutron/+/773032","commit_id":"c9292636c5d5f83403e434460d712c5739f3aa52"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"4d5ad02bf677d23db37a0fdb7ecdb7d509919ff8","unresolved":true,"context_lines":[{"line_number":194,"context_line":"      zuul_copy_output:"},{"line_number":195,"context_line":"        \u0027{{ devstack_base_dir }}/data/ovs\u0027: \u0027logs\u0027"},{"line_number":196,"context_line":"        \u0027{{ devstack_base_dir }}/data/ovn\u0027: \u0027logs\u0027"},{"line_number":197,"context_line":"        \u0027/var/log/ovn\u0027: \u0027logs\u0027"},{"line_number":198,"context_line":"      extensions_to_txt:"},{"line_number":199,"context_line":"        db: true"},{"line_number":200,"context_line":"      devstack_services:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"991bb566_7e47f43f","line":197,"in_reply_to":"6b41fc45_2750b7b5","updated":"2021-02-01 09:49:38.000000000","message":"FYI, it works\n\nhttps://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_6ad/773032/2/check/neutron-ovn-tempest-ovs-release/6ad26c1/controller/logs/ovn/index.html","commit_id":"c9292636c5d5f83403e434460d712c5739f3aa52"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"7ae7910263b528c507721da0fd384de84fc5fed4","unresolved":true,"context_lines":[{"line_number":224,"context_line":"        \u0027{{ devstack_base_dir }}/data/ovs\u0027: \u0027logs\u0027"},{"line_number":225,"context_line":"        \u0027{{ devstack_base_dir }}/data/ovn\u0027: \u0027logs\u0027"},{"line_number":226,"context_line":"        \u0027{{ devstack_log_dir }}/ovsdb-server-nb.log\u0027: \u0027logs\u0027"},{"line_number":227,"context_line":"        \u0027{{ devstack_log_dir }}/ovsdb-server-sb.log\u0027: \u0027logs\u0027"},{"line_number":228,"context_line":"      extensions_to_txt:"},{"line_number":229,"context_line":"        db: true"},{"line_number":230,"context_line":"      devstack_services:"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"0ccddb44_461abfea","side":"PARENT","line":227,"updated":"2021-02-05 14:25:36.000000000","message":"Hmmm maybe we do not want to delete those, we may still want to have jobs that compile OVN from source right ?\n\nWe could also change the log location when compiling from source within devstack...\n\nAny opinions ?","commit_id":"09078a9d695ea93f9322ee15fcaf6f15d566c30b"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"fedfee74be235d2b9e2fa0e407ec0f2db1a6c85b","unresolved":false,"context_lines":[{"line_number":224,"context_line":"        \u0027{{ devstack_base_dir }}/data/ovs\u0027: \u0027logs\u0027"},{"line_number":225,"context_line":"        \u0027{{ devstack_base_dir }}/data/ovn\u0027: \u0027logs\u0027"},{"line_number":226,"context_line":"        \u0027{{ devstack_log_dir }}/ovsdb-server-nb.log\u0027: \u0027logs\u0027"},{"line_number":227,"context_line":"        \u0027{{ devstack_log_dir }}/ovsdb-server-sb.log\u0027: \u0027logs\u0027"},{"line_number":228,"context_line":"      extensions_to_txt:"},{"line_number":229,"context_line":"        db: true"},{"line_number":230,"context_line":"      devstack_services:"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"2c54ce3c_6ec14334","side":"PARENT","line":227,"in_reply_to":"09348c42_a00b67f7","updated":"2021-02-09 14:55:25.000000000","message":"I think the ovs-master job (L330) should compile it [0]\n\nBut let\u0027s go ahead with this change and we can fix it in another patch because I also think we should also pass OVN_BUILD_FROM_SOURCE\u003dTrue to it (this flag was introduced in the patch adding package support in DevStack).\n\nhttps://github.com/openstack/neutron/blob/a3dc80b509d72c8d1a3ea007cb657a9e217ba66a/zuul.d/tempest-multinode.yaml#L331-L371","commit_id":"09078a9d695ea93f9322ee15fcaf6f15d566c30b"},{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"c62c013ff9d775fff3a6bd48f1fd872937da8c31","unresolved":true,"context_lines":[{"line_number":224,"context_line":"        \u0027{{ devstack_base_dir }}/data/ovs\u0027: \u0027logs\u0027"},{"line_number":225,"context_line":"        \u0027{{ devstack_base_dir }}/data/ovn\u0027: \u0027logs\u0027"},{"line_number":226,"context_line":"        \u0027{{ devstack_log_dir }}/ovsdb-server-nb.log\u0027: \u0027logs\u0027"},{"line_number":227,"context_line":"        \u0027{{ devstack_log_dir }}/ovsdb-server-sb.log\u0027: \u0027logs\u0027"},{"line_number":228,"context_line":"      extensions_to_txt:"},{"line_number":229,"context_line":"        db: true"},{"line_number":230,"context_line":"      devstack_services:"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"14c0f414_1b45323a","side":"PARENT","line":227,"in_reply_to":"0ccddb44_461abfea","updated":"2021-02-05 14:28:19.000000000","message":"Aha, I thought those were not used anywhere. So how about we have two different zuul_copy_output dicts based on whether we compile or not?","commit_id":"09078a9d695ea93f9322ee15fcaf6f15d566c30b"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"c1c4c454e81291f001a3048f842daaa6b4a43f52","unresolved":true,"context_lines":[{"line_number":224,"context_line":"        \u0027{{ devstack_base_dir }}/data/ovs\u0027: \u0027logs\u0027"},{"line_number":225,"context_line":"        \u0027{{ devstack_base_dir }}/data/ovn\u0027: \u0027logs\u0027"},{"line_number":226,"context_line":"        \u0027{{ devstack_log_dir }}/ovsdb-server-nb.log\u0027: \u0027logs\u0027"},{"line_number":227,"context_line":"        \u0027{{ devstack_log_dir }}/ovsdb-server-sb.log\u0027: \u0027logs\u0027"},{"line_number":228,"context_line":"      extensions_to_txt:"},{"line_number":229,"context_line":"        db: true"},{"line_number":230,"context_line":"      devstack_services:"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"cf9359c3_a2b4db87","side":"PARENT","line":227,"in_reply_to":"0d8ff12c_57b7586e","updated":"2021-02-05 16:11:49.000000000","message":"Thanks Brian! Yeah let\u0027s add those to the master jobs","commit_id":"09078a9d695ea93f9322ee15fcaf6f15d566c30b"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"b0fe3a9c1425540589b5a0fc16617454c4024f0f","unresolved":true,"context_lines":[{"line_number":224,"context_line":"        \u0027{{ devstack_base_dir }}/data/ovs\u0027: \u0027logs\u0027"},{"line_number":225,"context_line":"        \u0027{{ devstack_base_dir }}/data/ovn\u0027: \u0027logs\u0027"},{"line_number":226,"context_line":"        \u0027{{ devstack_log_dir }}/ovsdb-server-nb.log\u0027: \u0027logs\u0027"},{"line_number":227,"context_line":"        \u0027{{ devstack_log_dir }}/ovsdb-server-sb.log\u0027: \u0027logs\u0027"},{"line_number":228,"context_line":"      extensions_to_txt:"},{"line_number":229,"context_line":"        db: true"},{"line_number":230,"context_line":"      devstack_services:"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"d8f07e1b_e1bac23a","side":"PARENT","line":227,"in_reply_to":"14c0f414_1b45323a","updated":"2021-02-05 15:04:45.000000000","message":"That would work as well, do we want to add it to the -master jobs definitions in this patch ?","commit_id":"09078a9d695ea93f9322ee15fcaf6f15d566c30b"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"89f2dc612fcb1ea58aa9f038ef3fae57b66db627","unresolved":false,"context_lines":[{"line_number":224,"context_line":"        \u0027{{ devstack_base_dir }}/data/ovs\u0027: \u0027logs\u0027"},{"line_number":225,"context_line":"        \u0027{{ devstack_base_dir }}/data/ovn\u0027: \u0027logs\u0027"},{"line_number":226,"context_line":"        \u0027{{ devstack_log_dir }}/ovsdb-server-nb.log\u0027: \u0027logs\u0027"},{"line_number":227,"context_line":"        \u0027{{ devstack_log_dir }}/ovsdb-server-sb.log\u0027: \u0027logs\u0027"},{"line_number":228,"context_line":"      extensions_to_txt:"},{"line_number":229,"context_line":"        db: true"},{"line_number":230,"context_line":"      devstack_services:"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"28a13e7c_2aa84f30","side":"PARENT","line":227,"in_reply_to":"2c54ce3c_6ec14334","updated":"2021-02-09 15:04:00.000000000","message":"Here https://review.opendev.org/c/openstack/neutron/+/774655","commit_id":"09078a9d695ea93f9322ee15fcaf6f15d566c30b"},{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"81c0332a31248b12ee9c2e95883aad21618b785b","unresolved":true,"context_lines":[{"line_number":224,"context_line":"        \u0027{{ devstack_base_dir }}/data/ovs\u0027: \u0027logs\u0027"},{"line_number":225,"context_line":"        \u0027{{ devstack_base_dir }}/data/ovn\u0027: \u0027logs\u0027"},{"line_number":226,"context_line":"        \u0027{{ devstack_log_dir }}/ovsdb-server-nb.log\u0027: \u0027logs\u0027"},{"line_number":227,"context_line":"        \u0027{{ devstack_log_dir }}/ovsdb-server-sb.log\u0027: \u0027logs\u0027"},{"line_number":228,"context_line":"      extensions_to_txt:"},{"line_number":229,"context_line":"        db: true"},{"line_number":230,"context_line":"      devstack_services:"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"09348c42_a00b67f7","side":"PARENT","line":227,"in_reply_to":"cf9359c3_a2b4db87","updated":"2021-02-07 14:42:04.000000000","message":"So I found out we don\u0027t compile OVN in Neutron tree at all. Only in OVN Octavia provider driver and neutron-tempest-plugin. I think this change should be good then and I\u0027ll check the patch Brian said is gonna send to OVN Octavia and will update tempest plugin.","commit_id":"09078a9d695ea93f9322ee15fcaf6f15d566c30b"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"a9d70abddf53da05e37b373a4a19f03cdeb9d9b0","unresolved":true,"context_lines":[{"line_number":224,"context_line":"        \u0027{{ devstack_base_dir }}/data/ovs\u0027: \u0027logs\u0027"},{"line_number":225,"context_line":"        \u0027{{ devstack_base_dir }}/data/ovn\u0027: \u0027logs\u0027"},{"line_number":226,"context_line":"        \u0027{{ devstack_log_dir }}/ovsdb-server-nb.log\u0027: \u0027logs\u0027"},{"line_number":227,"context_line":"        \u0027{{ devstack_log_dir }}/ovsdb-server-sb.log\u0027: \u0027logs\u0027"},{"line_number":228,"context_line":"      extensions_to_txt:"},{"line_number":229,"context_line":"        db: true"},{"line_number":230,"context_line":"      devstack_services:"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"0d8ff12c_57b7586e","side":"PARENT","line":227,"in_reply_to":"d8f07e1b_e1bac23a","updated":"2021-02-05 15:54:20.000000000","message":"I guess this explains my empty directory comment in PS1.  We actually do compile OVN from source in the OVN provider repo so I\u0027ll need to do the right thing there.  Might be worth it to put in the master definition below.","commit_id":"09078a9d695ea93f9322ee15fcaf6f15d566c30b"}]}