| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-01-24 | |||
| 01:02:38 | bauzas | looks terrible to me | |
| 01:04:50 | bauzas | dansmith: I need direction here before going to bed | |
| 01:05:11 | bauzas | dansmith: because tomorrow morning I won't be knowing what to do | |
| 01:05:33 | dansmith | bauzas: if we have something we have to do in order to upgrade, that has to go into grenade | |
| 01:05:45 | bauzas | dansmith: when we have a config option missed by the old version but needed by the new version, what to do ? | |
| 01:05:56 | dansmith | bauzas: however, I think you could just make the grenade job do whatever we do in newton to enable placement, so that you don't make a grenade change | |
| 01:06:01 | bauzas | just adding it straight to grenade ? | |
| 01:06:12 | dansmith | which config option? | |
| 01:06:29 | bauzas | dansmith: I guess those two https://github.com/openstack-dev/devstack/blob/master/lib/placement#L102-L103 | |
| 01:06:37 | dansmith | it's possible to start placement in newton | |
| 01:06:42 | dansmith | you mean devstack/newton can't? | |
| 01:06:58 | bauzas | maybe, lemme look at the stable branch | |
| 01:07:26 | dansmith | bauzas: if not, I think it's going to be kinda ugly.. starting that in grenade, because you'll have to alter the devstack config and because what we *want* is people to start placement in newton | |
| 01:07:30 | bauzas | dansmith: mmm, weirdo https://github.com/openstack-dev/devstack/blob/stable/newton/lib/placement#L110-L111 | |
| 01:08:09 | bauzas | dansmith: just remembering how grenade works | |
| 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 | gmann | Kevin_Zheng: sure, ll look | |
| 06:28:22 | Kevin_Zheng | patchset 5 | |
| 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 | |