Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-01-24
01:08:46 bauzas dansmith: so, we first use the devstack stable branch for running the cloud, then applying the master branch
01:08:47 bauzas right?
01:09:02 bauzas I'm not very expert of grenade unfortunatelty
01:09:45 bauzas nevermind, seeing https://github.com/openstack-dev/grenade#basic-flow
01:10:21 bauzas okay, I'll try to investigate then
01:11:13 dansmith yes, that's how it works
01:12:29 bauzas dansmith: okay, thanks
01:12:35 bauzas just stopping today for me
01:12:40 bauzas bye
01:12:59 dansmith o/
01:27:48 openstackgerrit Ken'ichi Ohmichi proposed openstack/nova: Add unit tests for JSON-Schema' items behavior https://review.openstack.org/415333
01:33:23 openstackgerrit Zhenyu Zheng proposed openstack/nova: Fix device taging after API microversion 2.37 https://review.openstack.org/423952
01:37:57 openstackgerrit Zhenyu Zheng proposed openstack/nova: Add sort_key white list for server list/detail https://review.openstack.org/415142
02:07:13 openstackgerrit Zhenyu Zheng proposed openstack/nova: Don't delete neutron port when attach failed https://review.openstack.org/403568
02:33:06 openstackgerrit Takashi NATSUME proposed openstack/nova: Enable cold migration with target host(1/2) https://review.openstack.org/408955
03:05:25 openstackgerrit gengchc2 proposed openstack/nova: Improved-the-judgment-of-the-service-status-in-memcache https://review.openstack.org/423754
03:24:25 openstackgerrit Zhenyu Zheng proposed openstack/nova: Fix device taging after API microversion 2.37 https://review.openstack.org/423952
03:47:29 openstackgerrit Takashi NATSUME proposed openstack/nova: Enable cold migration with target host(2/2) https://review.openstack.org/408964
04:12:01 gmann Kevin_Zheng: few comments on tagging fix
04:13:30 Kevin_Zheng I added those test files just for functional testing, do you have any suggestion? cos I found that only one create-req file could exist for one microversion
04:14:05 gmann Kevin_Zheng: actually common req resp file should be fine as with 2.37 one
04:14:34 gmann Kevin_Zheng: all other version req etc should be ok with those till they do not add mandatory field ion req or resp
04:15:21 Kevin_Zheng I will have a closer check after lunch, thanks alot
04:15:54 openstackgerrit Huan Xie proposed openstack/nova: Fix live migrate with XenServer https://review.openstack.org/424428
04:15:56 gmann 2.37 was added because it add networks filed as mandatory in req
04:17:14 gmann alex_xu: johnthetubaguy oomichi: what do you think whether it will break users - https://review.openstack.org/423952
04:17:51 gmann if someone using wrong tag pattern then yes otherwise no change on tag length etc
05:52:23 openstackgerrit Pushkar Umaranikar proposed openstack/nova: Remove unused columns from BuildRequest table in nova_api db https://review.openstack.org/422259
06:07:45 openstackgerrit Pushkar Umaranikar proposed openstack/nova: Remove unused columns from BuildRequest table in nova_api db https://review.openstack.org/422259
06:07:47 openstackgerrit gengchc2 proposed openstack/nova: Improved-the-judgment-of-the-service-status-in-memcache https://review.openstack.org/423754
06:19:58 Kevin_Zheng gmann: are you still around?
06:20:21 gmann Kevin_Zheng: yea
06:20:32 Kevin_Zheng gmann: hi
06:20:56 Kevin_Zheng gmann: I understand what you are saying in the comment
06:21:56 Kevin_Zheng what I want to do is to test adding tags for virt devices for version 2.37
06:22:30 Kevin_Zheng because we did it wrong in version 237 before
06:23:18 gmann Kevin_Zheng: yea, in 2.37 file it is fine
06:23:42 gmann Kevin_Zheng: and you can add tests with higher version also by using the same 2.37 json file
06:24:02 gmann 2.40 json file is not needed i think
06:24:15 Kevin_Zheng hmm, maybe I can create two servers in 2.37 json?
06:24:35 gmann Kevin_Zheng: 2 servers means ?
06:24:42 Kevin_Zheng because in 237 we actually tested using "auto" for networks,
06:24:58 Kevin_Zheng we cannot add tag to network parameter
06:25:09 Kevin_Zheng thats why I added another one
06:25:45 Kevin_Zheng and it seems the test structure can only read file name with server-create-req.json
06:25:53 gmann Kevin_Zheng: you can change that auto to what you want to tests - https://github.com/openstack/nova/blob/master/nova/tests/functional/api_sample_tests/api_samples/servers/server-create-req-v237.json.tpl
06:26:30 gmann Kevin_Zheng: under 2.37, auto bits are being tested alone - https://github.com/openstack/nova/blob/master/nova/tests/functional/api_sample_tests/api_samples/servers/v2.37/server-create-req.json.tpl
06:26:53 Kevin_Zheng yeah, I did that in the last patchset
06:27:03 Kevin_Zheng no before the last
06:27:11 gmann Kevin_Zheng: ok
06:27:21 Kevin_Zheng but the compuete node seems not supported tagging
06:27:57 Kevin_Zheng could you have a check and maybe show me how to fix it? because I tried alittle but failed so I tried this way
06:28:22 Kevin_Zheng patchset 5
06:28:22 gmann Kevin_Zheng: sure, ll look
06:34:39 openstackgerrit Moshe Levi proposed openstack/nova: Remove mox in network/test_linux_net.py https://review.openstack.org/316427
06:39:52 openstackgerrit Wang Qing wu proposed openstack/nova: WIP(6): Introduce base PowerVM driver (ssp drv) https://review.openstack.org/422594
06:43:29 satyar Hi jaypipes
06:44:03 satyar jaypipes around
06:46:29 gmann Kevin_Zheng: that is strange for me, functional tests runs with fake driver
06:51:52 Kevin_Zheng gmann: yeah, Thanks
06:52:49 alex_xu Kevin_Zheng: just checked https://review.openstack.org/#/c/415330/17, that is what I expected.
06:53:15 satyar Kevin_Zheng is the nova_api_cell0 changed to nova_cell0 now?
06:53:22 Kevin_Zheng alex_xu: thanks, then lets work on the tests
06:53:40 alex_xu Kevin_Zheng: thanks for working on it
06:53:44 Kevin_Zheng satyar: sorry I think you asked the wrong person
06:53:54 satyar :P sorry
06:54:10 satyar alex_xu
06:54:51 satyar i saw some changes from Dan Smith
06:54:59 satyar not sure whats the handler here
06:55:47 alex_xu satyar: dansmith is right person to answer that, but I saw this patch https://review.openstack.org/#/c/420437/
06:57:07 satyar Thanks alex_xu
06:57:28 alex_xu satyar: np
06:59:32 satyar Hi dansmith
07:00:51 openstackgerrit Béla Vancsics proposed openstack/nova: Reduce code duplication https://review.openstack.org/342187
07:08:28 openstackgerrit Zhenyu Zheng proposed openstack/nova: Add new policy for server list/detail with all_tenants https://review.openstack.org/415330
07:13:04 moshele alex_xu: hi can you review https://review.openstack.org/#/c/423608/ ?
07:46:10 alex_xu moshele: just add it to my list
07:46:34 moshele alex_xu: thanks :)
07:47:35 alex_xu moshele: np
08:26:44 alex_xu gmann: just prepare to send a mail about the device tag
08:28:40 gmann alex_xu: nice, thanks.
08:29:10 gmann alex_xu: looks like some issue in functional tests also. debugging that
08:31:04 alex_xu gmann: let us put all the info in the mail and discuss at here
08:31:08 alex_xu s/here/there/
08:32:07 gmann alex_xu: +1
08:54:44 snikitin alex_xu: Hi, Alex! I changed a file name in diagnostics patch and lost your +2 :( Can you look again please? Also I left a comment about your proposal for docs https://review.openstack.org/#/c/355540/
08:55:13 alex_xu snikitin: ok, I will check that
08:55:58 snikitin thanks!
09:13:41 openstackgerrit Ildiko Vancsa proposed openstack/nova: Implement new attach/detach Cinder flow https://review.openstack.org/330285
09:14:34 openstackgerrit Ildiko Vancsa proposed openstack/nova: Implement new attach/detach Cinder flow https://review.openstack.org/330285
09:14:36 openstackgerrit Ildiko Vancsa proposed openstack/nova: Remove check_attach https://review.openstack.org/335358
09:31:10 moshele sfinucan: hi can you review this https://review.openstack.org/#/c/423608/
09:31:33 sfinucan moshele: Sure
09:32:01 openstackgerrit Hironori Shiina proposed openstack/nova: Ironic: Add soft power off support to Ironic driver. https://review.openstack.org/407977
09:42:58 sfinucan moshele: Done
09:45:35 moshele sfinucan: thanks
09:47:11 sfinucan bauzas: Could you take a look at https://review.openstack.org/#/c/419479 when you have a moment?
09:47:34 bauzas sfinucan: currently under the water because of the scheduler upcall, but I'll try
09:47:55 sfinucan bauzas: No panic at all. Priorities :)
09:50:35 bauzas sfinucan: I'm just investigating a grenade issue, hopefully to sort it out soonish

Earlier   Later