| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-01-23 | |||
| 23:42:59 | mriedem | dansmith: looks like grenade isn't happy with https://review.openstack.org/#/c/424252/ | |
| 23:43:42 | dansmith | mriedem: isn't that the dep thing? | |
| 23:44:31 | dansmith | mriedem: yeah, that's the dep thing.. I've been waiting all day to get a run of my patch with this | |
| 23:44:38 | dansmith | it just merged and I rechecked mine -- just rechecked this one too | |
| 23:45:12 | mriedem | dansmith: i'm not sure what dep thing you're talking about | |
| 23:45:19 | mriedem | you changed the d-g patch to not call discover_hosts for grenade | |
| 23:46:03 | dansmith | mriedem: https://review.openstack.org/#/c/424323/ | |
| 23:50:44 | openstackgerrit | OpenStack Proposal Bot proposed openstack/os-vif: Updated from global requirements https://review.openstack.org/395347 | |
| 23:51:01 | mriedem | dansmith: ok i wasn't aware of that | |
| 23:56:16 | openstackgerrit | Sylvain Bauza proposed openstack/nova: WIP: Scheduler calling the Placement API https://review.openstack.org/417961 | |
| 23:57:23 | bauzas | Yippee ki-yay mother ^ | |
| 23:57:44 | bauzas | hoping Jenkins to be green after that one | |
| 23:57:51 | bauzas | now, calling it a day | |
| #openstack-nova - 2017-01-24 | |||
| 00:04:29 | takashin | bauzas: Could you review https://review.openstack.org/#/c/408955/ again and check my comments? | |
| 00:04:56 | bauzas | takashin: hola | |
| 00:05:07 | bauzas | takashin: sorry, it's 1:05am for me now | |
| 00:05:14 | bauzas | takashin: I'm just about to visit my bed | |
| 00:05:41 | bauzas | takashin: but I'll try to look at it by tomorrow in case my above change is not getting a new problme | |
| 00:05:59 | takashin | bauzas: Thank you. | |
| 00:06:24 | bauzas | takashin: FWIW, opening a tab to remember it | |
| 00:06:39 | bauzas | takashin: but ping me later if you wish (I'm UTC+1) | |
| 00:07:45 | takashin | bauzas: I will ping you later. | |
| 00:08:54 | openstackgerrit | Claudiu Belu proposed openstack/nova: Hyper-V PCI Passthrough https://review.openstack.org/420614 | |
| 00:12:16 | openstackgerrit | Ken'ichi Ohmichi proposed openstack/nova: Add unit tests for JSON-Schema' items behavior https://review.openstack.org/415333 | |
| 00:16:17 | takashin | oomichi: Are you around? | |
| 00:17:56 | oomichi | takashin: yep | |
| 00:18:08 | takashin | oomichi: Could you review https://review.openstack.org/#/c/408964/ ? | |
| 00:18:09 | oomichi | may I help | |
| 00:18:18 | oomichi | takashin: ok, will review it soon | |
| 00:18:32 | takashin | oomichi: Thank you. | |
| 00:19:12 | oomichi | takashin: oh, huge one ;) | |
| 00:19:30 | oomichi | maybe alex_xu also will help | |
| 00:22:57 | takashin | oomichi: I understand that. | |
| 00:31:36 | amrith | andreykurilin, I will try | |
| 00:47:22 | bauzas | mriedem: dansmith: FWIW, I think we probably have a grenade issue for the scheduler upcall change http://logs.openstack.org/61/417961/19/check/gate-grenade-dsvm-neutron-multinode-ubuntu-xenial/b497772/logs/new/screen-n-sch.txt.gz?level=TRACE#_2017-01-24_00_30_31_476 | |
| 00:47:58 | bauzas | I'm getting a MissingAuthPlugin exception from Keystone... | |
| 00:56:28 | openstackgerrit | Dan Smith proposed openstack/nova: Move instance creation to conductor https://review.openstack.org/319379 | |
| 00:56:58 | dansmith | mriedem: bauzas: melwitt: ^ all three patches are passing now, just updated to remove the WIP from the last one | |
| 00:57:28 | bauzas | dansmith: I'll look at it tomorrow morning my time | |
| 00:57:33 | dansmith | bauzas: kthx | |
| 00:58:01 | bauzas | dansmith: in the meantime, it seems we don't provide the placement opts for grenade | |
| 00:58:11 | dansmith | bauzas: yeah I saw above, have't looked yet tho | |
| 00:58:18 | bauzas | dansmith: ie. http://logs.openstack.org/61/417961/19/check/gate-tempest-dsvm-cells-ubuntu-xenial/e99ce7c/logs/etc/nova/nova.conf.txt.gz has a placement section | |
| 00:58:29 | bauzas | dansmith: vs. http://logs.openstack.org/61/417961/19/check/gate-grenade-dsvm-neutron-multinode-ubuntu-xenial/b497772/logs/etc/nova/nova.conf.txt.gz not having it | |
| 00:58:58 | bauzas | dansmith: so my WIP change is currently +1 for all the dsvm jobs but not for grenade | |
| 00:59:31 | bauzas | possibly because we miss https://github.com/openstack-dev/devstack/blob/master/lib/placement#L102 in grenade | |
| 01:00:21 | dansmith | yeah | |
| 01:01:55 | bauzas | so the problem is that we have not the above for the Newton cloud | |
| 01:02:18 | bauzas | which means that it's not really working in grenade | |
| 01:02:25 | bauzas | I mean, the placement job | |
| 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 | |