13:01:26 #startmeeting neutron_db 13:01:27 Meeting started Mon Sep 15 13:01:26 2014 UTC and is due to finish in 60 minutes. The chair is HenryG. Information about MeetBot at http://wiki.debian.org/MeetBot. 13:01:28 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 13:01:30 The meeting name has been set to 'neutron_db' 13:01:59 I need to go afk in 15 minutes. If you have any item for me cna you please move it forward? 13:02:08 Not much left to discuss 13:02:23 Reorganize migrations is merged 13:02:38 Migration unit tests are working 13:02:58 salv-orlando, I have question how can I recheck vmware? 13:03:57 akamyshnikova_: vmware-recheck, but we suspended neutron jobs last weeks. 13:04:02 I resumed them this morning. 13:04:27 salv-orlando, ok, thanks! 13:04:32 jlibosva: I was on PTO - this merged https://review.openstack.org/#/c/117913/ and people said they had no problem updating from havana? 13:05:00 I think they should have had problems… according to your comments? 13:05:12 or did someone else fix the patch for me? 13:05:15 salv-orlando: I/we fixed while you were away 13:05:17 salv-orlando: HenryG used the conditions for operations 13:05:47 jlibosva: thanks guys, this is what I was planning to do 13:05:56 salv-orlando: all credit goes to HenryG :) 13:06:02 but hardly found any time to sit in front of a keyboard last week 13:06:29 salv-orlando: We are here to serve :) 13:07:37 HenryG: salv-orlando does it mean in the end we don't support offline migrations now, right? 13:07:56 jlibosva: Only for Juno 13:08:04 Is it documented somewhere? 13:08:16 Not yet 13:08:34 We need to update the deployer/operator docs 13:09:00 From kilo we will once again support offline migrations 13:09:37 I.e. Juno -> Kilo -> L___ -> etc. 13:10:26 It's only Icehouse -> Juno that needs to be online 13:11:28 #action HenryG to ensure operator docs get updated 13:11:33 correct. It is a suspension in offline migration support. 13:11:46 we already agreed to this when we did the healing script 13:13:11 Yup 13:13:27 Anything other questions for salv-orlando ? 13:14:33 salv-orlando: You may be excused. :) 13:14:50 #topic Migration Unit Tests 13:15:09 akamyshnikova_: These now work great! 13:15:13 We need to mention Juno migration does not support offline mode in the release note. I believe Kyle will cover this :-) 13:16:04 amotoki: thanks, I will get in touch with mestery about that 13:16:31 amotoki: Thanks for the reminder here, I'll make sure to cover that as well. 13:16:44 HenryG: mestery: we check the release note as a community. it is wiki-based :-) 13:16:58 amotoki: Exactly, please feel free to update that yourself! :) 13:17:14 akamyshnikova_, this working ok now :) so it needs review and we are ready to merge :) 13:17:37 akamyshnikova_: I will do a review of those patches later on today. 13:17:49 salv-orlando, thanks! 13:18:06 I have to leave now. Enjoy the rest of meeting. If there is any note for me I’ll read the log when I’ll be back. 13:18:16 akamyshnikova_: I see a comment from Maru about moving it to functional job. I will talk to him about that. 13:18:33 salv-orlando: Thanks for your time. 13:19:27 akamyshnikova_: Since no root or devstack env is required, it does not need to be in the functional job. 13:20:11 HenryG: akamyshnikova_ I don't think the reason is root or devstack but the system interaction 13:20:27 HenryG: doesn't it depend on db back-ends like mysql or postgres? 13:20:52 Yes, it does 13:21:03 HenryG, yes, in fact these tests was created in oslo to be implemented in unittests 13:21:03 But it will skip if they are not found. 13:21:32 HenryG, yes if backend is not available tests just skip 13:22:27 But Maru is the test guru, so we should still listen carefully to his reasoning. 13:22:46 sure 13:23:10 it doesnt require devstack but a specific backend needs to be installed. for some case postgresql should be installed. Can we satisfy the requirement? 13:23:54 amotoki: The unit test requirements are currently python only. 13:24:13 HenryG: yes 13:24:24 in gate we have all necessary backend for this test available(mysql,postgres) 13:24:47 locally you have to install it yourself 13:25:34 The advantage of having the tests in unit tests is that it is easier for developers to test than setting up the environment for the functional tests. 13:25:51 So it will be easier for devs to test locally. 13:26:24 I added information in developer's documentation about this test 13:26:40 Okay, i got the situation. It is a kind of functional tests, but the situation around db is a bit different. thanks. 13:27:04 HenryG: someone will still need to setup db, no? 13:27:38 jlibosva: Yes, but it takes like 30 seconds to do it manually. One time. 13:30:09 However, setting up the functional test environment is not hard either. And moving the tests to the functional job is not a bad thing. 13:30:32 I'm more inclined to move it to functional 13:30:48 I would prefer it to be in unit tests, but I will talk to Maru. 13:32:09 HenryG, I also thinks that it should be unit tests, but of course we need to know why Maru suggest this. 13:32:51 I will be online next 2 hours if you will need me 13:33:08 #topic Open discussion 13:33:27 Anything else? 13:33:57 HenryG, I want to ask what sort of docs should I provide for test, but I think we need to get feedback from Maru first 13:35:41 akamyshnikova_: You already have the information in the comments. It's just a question of where to put them officially. 13:36:33 it won't be added in docs.openstack.org? 13:36:44 automatically 13:37:37 * HenryG knows too little about how documentation is done :( 13:37:54 http://docs-draft.openstack.org/20/76520/71/check/gate-neutron-docs/87432c0/doc/build/html/devref/db_layer.html#module-neutron.tests.unit.db.test_migration 13:38:36 this is result of gate-neutron-docs 13:38:53 Very nice, I did not know that. 13:39:41 :) 13:40:03 akamyshnikova_: We can check with emagana if that is sufficient or if some other steps are needed. 13:40:16 HenryG, great thanks! 13:41:26 akamyshnikova_: In your commit message you can put "DocImpact: developer docs" 13:41:51 HenryG, sure, thanks, for pointing this 13:42:03 http://docs.openstack.org/developer/openstack-projects.html is generated automatically when neutron repo is updated. 13:42:28 DocImpact for developer docs is unnecessary. 13:42:50 amotoki, ok 13:43:23 DocImpact flag is mainly targeted for documentations manually updated like admin guide or something. 13:43:40 amotoki, is there anything else what I should provide? 13:44:06 i think dev doc in neutron repo is enough 13:44:31 as long as it does not affect operators. 13:45:01 amotoki, ok, thanks! 13:45:06 akamyshnikova_: does it make sense? 13:46:04 amotoki, yes, thank you for explanation :) 13:46:45 amotoki: You are talking about doc/source/devref/* ? 13:47:23 HenryG: In my understanding, doc under neutron repo is the developer docs. 13:47:46 amotoki: OK, thanks 13:48:05 There is also TESTING.rst in the root dir 13:48:49 Anyway, we'll figure it out. 13:49:05 Anything else to discuss? 13:50:54 none from me. Thanks for your great work on migrations! 13:55:00 Thanks everyone! 13:55:05 #endmeeting