| mriedem | claudiub|2: isn't it 2am for you? | 00:01 |
|---|---|---|
| *** haomaiwa_ has quit IRC | 00:01 | |
| claudiub|2 | mriedem: it is. | 00:01 |
| *** haomaiwang has joined #openstack-nova | 00:01 | |
| claudiub|2 | doing regression testing. fun times. :) | 00:01 |
| mriedem | ah well it is release time | 00:02 |
| mriedem | claudiub|2: don't worry about stable this week then | 00:02 |
| mriedem | i'll hammer garyk in the morning :) | 00:02 |
| claudiub|2 | for example, what amuses me right now is that whenever the network_info we get in the hyper-v driver is accessed, the whole thread freezes -> spawn freezes as well | 00:02 |
| claudiub|2 | so, instances are getting stuck in spawning / deleting. | 00:03 |
| mriedem | hmm, that network_info is the async network info wrapper right? | 00:03 |
| mriedem | that's on a greenthread | 00:03 |
| claudiub|2 | yeah | 00:03 |
| mriedem | and blocks until it's populated | 00:03 |
| mriedem | claudiub|2: did you try a gmr to see what's holding it up? | 00:03 |
| claudiub|2 | it doesn't seem to be populated any time soon.. | 00:04 |
| claudiub|2 | and i'm even more curious as to why i haven't seen this yet in the hyper-v ci. | 00:04 |
| mriedem | something in the neutron hyper-v plugin? | 00:04 |
| mriedem | does the hyper-v ci use nova-network or the neutron hyper-v plugin? | 00:05 |
| *** salv-orl_ has quit IRC | 00:05 | |
| claudiub|2 | it uses neutron hyper-v agent, but that's not the issue | 00:05 |
| claudiub|2 | it's an entirely separate process. | 00:05 |
| claudiub|2 | plus, the neutron hyper-v agent is waiting to see new VM vnics, so it can bind the respective neutron port. | 00:06 |
| *** ivase has joined #openstack-nova | 00:11 | |
| *** ducttape_ has joined #openstack-nova | 00:12 | |
| openstackgerrit | Matt Riedemann proposed openstack/python-novaclient: Use keystoneclient python bindings for testing https://review.openstack.org/296599 | 00:13 |
| *** Sukhdev has quit IRC | 00:13 | |
| *** ivase_ has joined #openstack-nova | 00:15 | |
| *** ivase has quit IRC | 00:15 | |
| *** fawadkhaliq has quit IRC | 00:15 | |
| *** ducttape_ has quit IRC | 00:17 | |
| *** fawadkhaliq has joined #openstack-nova | 00:17 | |
| claudiub|2 | mriedem: until a little while ago, the neutron port was being created on instance spawn, if I'm not mistaken. I remember this, because I saw the nova-compute logs whenever the neutron-hyperv-agent was down (-> Neutron would then say binding_failed). So, by the time the hyper-v driver would get to the spawn, the neutron port was already created. | 00:17 |
| claudiub|2 | it seems that this changed. I assume with that async wrapper? | 00:18 |
| *** achanda has quit IRC | 00:19 | |
| *** unicell has quit IRC | 00:19 | |
| *** Swami has quit IRC | 00:19 | |
| mriedem | claudiub|2: there were some changes with the async wrapper but only for the tests | 00:20 |
| *** vilobhmm11 has joined #openstack-nova | 00:20 | |
| *** ivase_ has quit IRC | 00:21 | |
| mriedem | claudiub|2: this https://github.com/openstack/nova/commit/61fc1b9ee11e416aecbf3a29e1d150a53fc890e8 | 00:21 |
| mriedem | claudiub|2: the hyper-v driver doesn't wait for the vif-plugged event like the libvirt driver right? | 00:22 |
| claudiub|2 | mriedem: not yet. I have an implementation for it, planning to upload it next week. | 00:23 |
| claudiub|2 | mriedem: the bp was approved for mitaka, so I guess it should be re-approved for newton as well. | 00:23 |
| claudiub|2 | https://blueprints.launchpad.net/nova/+spec/hyper-v-spawn-on-neutron-event | 00:24 |
| *** mtanino has quit IRC | 00:24 | |
| claudiub|2 | anyways, I killed the neutron-hyperv-agent on my host. meaning that if I spawn an instance, it should fail because neutron won't be able to bind the port on my host. yet, the spawn still gets to the hyper-v driver and gets stuck when it tries to access network_info | 00:25 |
| claudiub|2 | to get the necessary info to attach the vnics. | 00:26 |
| claudiub|2 | looking on nova list / show, there are is no network on the instance | 00:27 |
| mriedem | claudiub|2: is allocate_for_instance ever completing? | 00:27 |
| mriedem | claudiub|2: as for https://blueprints.launchpad.net/nova/+spec/hyper-v-spawn-on-neutron-event yeah just re-propose the spec | 00:28 |
| *** fawadkhaliq has quit IRC | 00:28 | |
| *** thorst has joined #openstack-nova | 00:29 | |
| claudiub|2 | mriedem: i'm not sure it's even called. let me pun some pdbs. | 00:29 |
| claudiub|2 | as for the https://blueprints.launchpad.net/nova/+spec/hyper-v-spawn-on-neutron-event, in Mitaka, it was approved specless. | 00:29 |
| claudiub|2 | Since it's not a new idea and it's pretty straight-forward. | 00:30 |
| openstackgerrit | Joe Cropper proposed openstack/nova: Add get_host_memory_stats virt driver API https://review.openstack.org/288838 | 00:30 |
| *** thorst has quit IRC | 00:30 | |
| *** fawadkhaliq has joined #openstack-nova | 00:30 | |
| *** thorst has joined #openstack-nova | 00:31 | |
| *** zz_dimtruck is now known as dimtruck | 00:32 | |
| *** y_sawai has joined #openstack-nova | 00:32 | |
| *** fawadkhaliq has quit IRC | 00:34 | |
| *** fawadkhaliq has joined #openstack-nova | 00:35 | |
| *** thorst has quit IRC | 00:36 | |
| *** aunnam has joined #openstack-nova | 00:39 | |
| *** ssurana has quit IRC | 00:39 | |
| *** samueldm1 has quit IRC | 00:39 | |
| *** samueldmq has joined #openstack-nova | 00:39 | |
| *** fawadkhaliq has quit IRC | 00:41 | |
| *** ducttape_ has joined #openstack-nova | 00:42 | |
| claudiub|2 | well, this was a waste of time and effort. | 00:42 |
| claudiub|2 | seems some automation scripts overwrote my nova.conf file, erasing the use_neutron=True config option | 00:43 |
| *** ctrath has joined #openstack-nova | 00:44 | |
| *** yuywz has joined #openstack-nova | 00:47 | |
| *** sheel has quit IRC | 00:47 | |
| *** qeelee has joined #openstack-nova | 00:48 | |
| *** Sukhdev has joined #openstack-nova | 00:49 | |
| *** Sukhdev has quit IRC | 00:49 | |
| *** ivase has joined #openstack-nova | 00:51 | |
| *** ivase_ has joined #openstack-nova | 00:55 | |
| *** haukebruno has joined #openstack-nova | 00:55 | |
| *** akshai_ has quit IRC | 00:56 | |
| *** ivase has quit IRC | 00:56 | |
| *** tbachman has quit IRC | 00:57 | |
| *** woodster_ has quit IRC | 00:57 | |
| *** mylu has quit IRC | 00:58 | |
| mriedem | claudiub|2: ha | 00:59 |
| mriedem | well, at least it's not some monster black hole of a bug :) | 00:59 |
| mriedem | get some sleep | 00:59 |
| claudiub|2 | yeah, that's good. :) and not yet, still work to do. | 01:00 |
| *** tbachman has joined #openstack-nova | 01:00 | |
| *** qeelee has quit IRC | 01:00 | |
| *** haomaiwang has quit IRC | 01:01 | |
| claudiub|2 | but thanks for the concern, I'll try to get you some palinca in austin. :)) | 01:01 |
| claudiub|2 | it's good stuff. :)) | 01:01 |
| *** haomaiwang has joined #openstack-nova | 01:01 | |
| *** ducttape_ has quit IRC | 01:02 | |
| *** ivase_ has quit IRC | 01:02 | |
| *** qeelee has joined #openstack-nova | 01:03 | |
| *** ducttape_ has joined #openstack-nova | 01:03 | |
| *** y_sawai has quit IRC | 01:04 | |
| *** y_sawai has joined #openstack-nova | 01:05 | |
| *** shihanzhang has quit IRC | 01:07 | |
| *** fawadkhaliq has joined #openstack-nova | 01:08 | |
| *** fawadkhaliq has quit IRC | 01:11 | |
| *** yamahata has quit IRC | 01:11 | |
| *** angdraug has quit IRC | 01:13 | |
| *** gongysh_ has joined #openstack-nova | 01:14 | |
| mriedem | tjones_zzzz: vmware nsx ci seems to never want to run on this https://review.openstack.org/#/c/287055/ | 01:15 |
| *** Jeffrey4l has joined #openstack-nova | 01:15 | |
| *** Kevin_Zheng has joined #openstack-nova | 01:15 | |
| *** efried has quit IRC | 01:16 | |
| mriedem | melwitt: Vek: finally got a +2 from jenkins on this https://review.openstack.org/#/c/296599/ | 01:17 |
| mriedem | we need to land and release that before keystone client does their 3.0 release | 01:17 |
| mriedem | stevemar: ^ | 01:17 |
| * melwitt looks | 01:17 | |
| *** y_sawai has quit IRC | 01:18 | |
| *** simondodsley has quit IRC | 01:19 | |
| *** achanda has joined #openstack-nova | 01:19 | |
| *** ivase has joined #openstack-nova | 01:20 | |
| mriedem | oh i think you'll be pleased | 01:23 |
| *** achanda has quit IRC | 01:23 | |
| melwitt | you're right! | 01:24 |
| *** ivase_ has joined #openstack-nova | 01:24 | |
| *** ivase has quit IRC | 01:25 | |
| *** vishwanathj has quit IRC | 01:26 | |
| *** ivase has joined #openstack-nova | 01:30 | |
| *** Daisy has joined #openstack-nova | 01:30 | |
| *** penick has quit IRC | 01:30 | |
| *** ivase_ has quit IRC | 01:31 | |
| *** Daisy has quit IRC | 01:31 | |
| *** Daisy has joined #openstack-nova | 01:32 | |
| *** jwcroppe has quit IRC | 01:33 | |
| *** jwcroppe has joined #openstack-nova | 01:33 | |
| *** achanda has joined #openstack-nova | 01:34 | |
| *** mtanino has joined #openstack-nova | 01:34 | |
| openstackgerrit | Bin Zhou proposed openstack/nova: upgrade connection_info when Ceph mon IP changed https://review.openstack.org/289813 | 01:35 |
| *** ljxiash has joined #openstack-nova | 01:35 | |
| openstackgerrit | Bin Zhou proposed openstack/nova: upgrade connection_info when Ceph mon IP changed https://review.openstack.org/289813 | 01:35 |
| *** ivase has quit IRC | 01:36 | |
| *** Daisy has quit IRC | 01:36 | |
| *** mylu has joined #openstack-nova | 01:36 | |
| *** jwcroppe has quit IRC | 01:38 | |
| *** akshai has joined #openstack-nova | 01:41 | |
| *** baoli has joined #openstack-nova | 01:42 | |
| *** mylu has quit IRC | 01:42 | |
| *** ivase has joined #openstack-nova | 01:42 | |
| *** apoorvad has quit IRC | 01:44 | |
| *** haomaiwang has quit IRC | 01:44 | |
| *** thorst has joined #openstack-nova | 01:46 | |
| *** thorst has quit IRC | 01:47 | |
| *** ivase_ has joined #openstack-nova | 01:47 | |
| *** coreywright has quit IRC | 01:47 | |
| *** ivase has quit IRC | 01:49 | |
| *** efoley has quit IRC | 01:49 | |
| *** houming has joined #openstack-nova | 01:49 | |
| *** hoangcx has joined #openstack-nova | 01:50 | |
| *** Daisy has joined #openstack-nova | 01:51 | |
| *** baoli has quit IRC | 01:52 | |
| *** browne has quit IRC | 01:53 | |
| *** ivase_ has quit IRC | 01:53 | |
| *** mylu has joined #openstack-nova | 01:55 | |
| *** ctrath has quit IRC | 01:55 | |
| *** fawadkhaliq has joined #openstack-nova | 01:56 | |
| *** akshai_ has joined #openstack-nova | 01:57 | |
| *** yamahata has joined #openstack-nova | 01:57 | |
| *** ducttape_ has quit IRC | 01:57 | |
| *** Jeffrey4l has quit IRC | 01:58 | |
| *** baoli has joined #openstack-nova | 01:58 | |
| *** achanda has quit IRC | 01:59 | |
| *** mtreinish has quit IRC | 01:59 | |
| *** egarbade- has quit IRC | 01:59 | |
| *** Daisy has quit IRC | 02:00 | |
| *** akshai has quit IRC | 02:00 | |
| *** Daisy has joined #openstack-nova | 02:00 | |
| *** egarbade- has joined #openstack-nova | 02:01 | |
| *** mtreinish has joined #openstack-nova | 02:01 | |
| *** Daisy has quit IRC | 02:01 | |
| *** Daisy_ has joined #openstack-nova | 02:01 | |
| *** Daisy_ has quit IRC | 02:02 | |
| *** Daisy has joined #openstack-nova | 02:03 | |
| *** akshai_ has quit IRC | 02:04 | |
| *** EinstCrazy has joined #openstack-nova | 02:04 | |
| *** achanda has joined #openstack-nova | 02:05 | |
| *** coreywright has joined #openstack-nova | 02:06 | |
| *** rwmjones has quit IRC | 02:06 | |
| *** ivase has joined #openstack-nova | 02:06 | |
| *** akshai has joined #openstack-nova | 02:07 | |
| *** ivase_ has joined #openstack-nova | 02:10 | |
| *** ivase has quit IRC | 02:11 | |
| *** mriedem has quit IRC | 02:11 | |
| *** mriedem has joined #openstack-nova | 02:11 | |
| *** toure_ is now known as toure | 02:13 | |
| *** baoli has quit IRC | 02:13 | |
| *** Daisy_ has joined #openstack-nova | 02:15 | |
| *** ivase_ has quit IRC | 02:17 | |
| *** ivase has joined #openstack-nova | 02:17 | |
| *** baoli has joined #openstack-nova | 02:18 | |
| *** Daisy has quit IRC | 02:18 | |
| *** ctrath has joined #openstack-nova | 02:19 | |
| *** baoli has quit IRC | 02:20 | |
| *** ivase_ has joined #openstack-nova | 02:22 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Deprecated option 'mkisofs_cmd' for removal https://review.openstack.org/262744 | 02:22 |
| *** ivase has quit IRC | 02:22 | |
| *** fawadkhaliq has quit IRC | 02:27 | |
| *** ivase_ has quit IRC | 02:28 | |
| *** baoli has joined #openstack-nova | 02:29 | |
| *** aunnam has quit IRC | 02:29 | |
| *** baoli has quit IRC | 02:31 | |
| *** browne has joined #openstack-nova | 02:36 | |
| *** tbachman has quit IRC | 02:36 | |
| *** baoli has joined #openstack-nova | 02:39 | |
| *** y_sawai has joined #openstack-nova | 02:40 | |
| *** baoli has quit IRC | 02:41 | |
| *** Daisy has joined #openstack-nova | 02:41 | |
| *** baoli has joined #openstack-nova | 02:42 | |
| *** Daisy_ has quit IRC | 02:43 | |
| *** zenoway has joined #openstack-nova | 02:48 | |
| *** baoli has quit IRC | 02:48 | |
| *** baoli has joined #openstack-nova | 02:51 | |
| *** Daisy has quit IRC | 02:51 | |
| *** Daisy has joined #openstack-nova | 02:52 | |
| *** zenoway has quit IRC | 02:52 | |
| *** baoli has quit IRC | 02:54 | |
| *** y_sawai has quit IRC | 02:59 | |
| *** vishwanathj has joined #openstack-nova | 03:02 | |
| *** Daisy_ has joined #openstack-nova | 03:02 | |
| *** ivase has joined #openstack-nova | 03:04 | |
| *** Daisy has quit IRC | 03:05 | |
| *** ivase_ has joined #openstack-nova | 03:08 | |
| *** ivase has quit IRC | 03:09 | |
| *** y_sawai has joined #openstack-nova | 03:11 | |
| *** y_sawai has quit IRC | 03:11 | |
| *** y_sawai has joined #openstack-nova | 03:11 | |
| *** ivase_ has quit IRC | 03:14 | |
| *** ivase has joined #openstack-nova | 03:16 | |
| *** mriedem has quit IRC | 03:17 | |
| *** haukebruno has quit IRC | 03:18 | |
| *** Daisy_ has quit IRC | 03:19 | |
| *** haukebruno has joined #openstack-nova | 03:20 | |
| *** Daisy has joined #openstack-nova | 03:20 | |
| *** ivase has quit IRC | 03:20 | |
| *** Daisy has quit IRC | 03:21 | |
| *** Daisy has joined #openstack-nova | 03:21 | |
| *** ljxiash has quit IRC | 03:22 | |
| *** ivase has joined #openstack-nova | 03:22 | |
| *** zenoway has joined #openstack-nova | 03:23 | |
| *** hoangcx has quit IRC | 03:25 | |
| *** hoangcx has joined #openstack-nova | 03:26 | |
| *** ctrath has quit IRC | 03:26 | |
| *** links has joined #openstack-nova | 03:27 | |
| *** zenoway has quit IRC | 03:28 | |
| *** coreywright has quit IRC | 03:29 | |
| *** haomaiwang has joined #openstack-nova | 03:29 | |
| *** rfolco_ has quit IRC | 03:30 | |
| *** akshai_ has joined #openstack-nova | 03:40 | |
| *** akshai has quit IRC | 03:42 | |
| *** qeelee has quit IRC | 03:42 | |
| *** salv-orlando has joined #openstack-nova | 03:43 | |
| *** coreywright has joined #openstack-nova | 03:43 | |
| *** zehua has joined #openstack-nova | 03:46 | |
| *** achanda has quit IRC | 03:49 | |
| *** salv-orlando has quit IRC | 03:49 | |
| *** mariojv has quit IRC | 03:53 | |
| *** y_sawai has quit IRC | 03:54 | |
| *** bswartz has quit IRC | 03:55 | |
| *** y_sawai has joined #openstack-nova | 03:55 | |
| *** bswartz has joined #openstack-nova | 03:56 | |
| *** Marga_ has quit IRC | 03:58 | |
| *** Daisy has quit IRC | 03:58 | |
| *** Marga_ has joined #openstack-nova | 03:59 | |
| *** Daisy has joined #openstack-nova | 03:59 | |
| *** Marga_ has quit IRC | 03:59 | |
| *** Marga_ has joined #openstack-nova | 03:59 | |
| *** haomaiwang has quit IRC | 04:01 | |
| *** haomaiwang has joined #openstack-nova | 04:01 | |
| *** rwmjones has joined #openstack-nova | 04:03 | |
| *** Daisy has quit IRC | 04:03 | |
| *** fawadkhaliq has joined #openstack-nova | 04:03 | |
| *** efoley has joined #openstack-nova | 04:05 | |
| *** akshai has joined #openstack-nova | 04:11 | |
| *** akshai_ has quit IRC | 04:14 | |
| *** dimtruck is now known as zz_dimtruck | 04:14 | |
| *** ivase has quit IRC | 04:15 | |
| *** mariojv has joined #openstack-nova | 04:16 | |
| *** ivase has joined #openstack-nova | 04:16 | |
| *** sheel has joined #openstack-nova | 04:18 | |
| zehua | HI | 04:20 |
| *** ivase has quit IRC | 04:23 | |
| *** vishwanathj is now known as vishwanathj_zzz | 04:24 | |
| *** Daisy has joined #openstack-nova | 04:25 | |
| *** fawadkhaliq has quit IRC | 04:27 | |
| *** fawadkhaliq has joined #openstack-nova | 04:27 | |
| *** pwnall has quit IRC | 04:29 | |
| *** tbachman has joined #openstack-nova | 04:31 | |
| *** penick has joined #openstack-nova | 04:33 | |
| *** qeelee has joined #openstack-nova | 04:34 | |
| *** ivase has joined #openstack-nova | 04:39 | |
| *** tbachman_ has joined #openstack-nova | 04:41 | |
| *** tbachman has quit IRC | 04:42 | |
| *** tbachman_ is now known as tbachman | 04:42 | |
| *** dave-mccowan has quit IRC | 04:45 | |
| *** ivase_ has joined #openstack-nova | 04:45 | |
| *** ivase has quit IRC | 04:47 | |
| *** tbachman has quit IRC | 04:47 | |
| *** mylu has quit IRC | 04:48 | |
| *** ivase has joined #openstack-nova | 04:50 | |
| *** y_sawai has quit IRC | 04:50 | |
| *** y_sawai has joined #openstack-nova | 04:51 | |
| *** ivase_ has quit IRC | 04:51 | |
| *** qeelee has quit IRC | 04:51 | |
| *** tbachman has joined #openstack-nova | 04:55 | |
| openstackgerrit | apporc proposed openstack/nova: fix admin context scenarios for cinderclient https://review.openstack.org/296890 | 04:57 |
| *** tanee has quit IRC | 04:58 | |
| *** achanda has joined #openstack-nova | 04:59 | |
| *** haomaiwang has quit IRC | 05:01 | |
| *** haomaiwang has joined #openstack-nova | 05:01 | |
| *** salv-orlando has joined #openstack-nova | 05:02 | |
| *** tanee has joined #openstack-nova | 05:03 | |
| *** libu has joined #openstack-nova | 05:03 | |
| *** EinstCrazy has quit IRC | 05:04 | |
| *** EinstCrazy has joined #openstack-nova | 05:04 | |
| *** sudipto has joined #openstack-nova | 05:05 | |
| *** baoli has joined #openstack-nova | 05:06 | |
| *** mylu has joined #openstack-nova | 05:07 | |
| *** EinstCrazy has quit IRC | 05:09 | |
| *** baoli has quit IRC | 05:11 | |
| *** garyk has joined #openstack-nova | 05:11 | |
| *** penick_ has joined #openstack-nova | 05:16 | |
| *** salv-orlando has quit IRC | 05:17 | |
| *** penick has quit IRC | 05:17 | |
| *** penick_ is now known as penick | 05:17 | |
| *** zenoway has joined #openstack-nova | 05:22 | |
| *** akshai has quit IRC | 05:22 | |
| *** qeelee has joined #openstack-nova | 05:24 | |
| *** ivase_ has joined #openstack-nova | 05:25 | |
| *** zenoway has quit IRC | 05:26 | |
| *** ivase has quit IRC | 05:27 | |
| *** markvoelker has quit IRC | 05:28 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Enhance value check for option notify_on_state_change https://review.openstack.org/242717 | 05:30 |
| *** ivase_ has quit IRC | 05:32 | |
| *** fawadkhaliq has quit IRC | 05:32 | |
| *** gongysh_ has quit IRC | 05:32 | |
| *** fawadkhaliq has joined #openstack-nova | 05:32 | |
| *** ivase has joined #openstack-nova | 05:33 | |
| *** ssurana has joined #openstack-nova | 05:33 | |
| *** penick has quit IRC | 05:35 | |
| *** mtanino has quit IRC | 05:35 | |
| *** tbachman has quit IRC | 05:37 | |
| *** ivase has quit IRC | 05:37 | |
| *** ljxiash has joined #openstack-nova | 05:40 | |
| *** harlowja_at_home has quit IRC | 05:43 | |
| *** vilobhmm11 has quit IRC | 05:44 | |
| openstackgerrit | Jianghua Wang proposed openstack/nova: XenAPI: add unit test for plugin test_pluginlib_nova.py https://review.openstack.org/286383 | 05:45 |
| openstackgerrit | Lianhao Lu proposed openstack/nova-specs: Repropose Need a way to list compute node metric names https://review.openstack.org/296894 | 05:46 |
| *** sridharg has joined #openstack-nova | 05:46 | |
| *** pcaruana has quit IRC | 05:48 | |
| openstackgerrit | Lianhao Lu proposed openstack/nova-specs: Repropose Need a way to list compute node metric names https://review.openstack.org/296894 | 05:48 |
| *** mylu has quit IRC | 05:51 | |
| *** mylu has joined #openstack-nova | 05:52 | |
| *** efoley has quit IRC | 05:55 | |
| *** ivase has joined #openstack-nova | 05:56 | |
| *** Daisy has quit IRC | 05:58 | |
| *** Daisy has joined #openstack-nova | 05:58 | |
| *** ivase_ has joined #openstack-nova | 06:00 | |
| *** ivase has quit IRC | 06:00 | |
| *** haomaiwang has quit IRC | 06:01 | |
| *** haomaiwang has joined #openstack-nova | 06:01 | |
| *** Daisy has quit IRC | 06:03 | |
| *** Daisy has joined #openstack-nova | 06:03 | |
| *** Daisy_ has joined #openstack-nova | 06:04 | |
| *** mylu has quit IRC | 06:04 | |
| *** liverpooler has quit IRC | 06:05 | |
| *** Daisy_ has quit IRC | 06:05 | |
| *** vilobhmm11 has joined #openstack-nova | 06:05 | |
| *** Daisy_ has joined #openstack-nova | 06:05 | |
| *** ivase has joined #openstack-nova | 06:06 | |
| *** ivase_ has quit IRC | 06:06 | |
| *** Daisy has quit IRC | 06:07 | |
| *** Daisy_ has quit IRC | 06:10 | |
| *** ivase_ has joined #openstack-nova | 06:11 | |
| *** ivase has quit IRC | 06:12 | |
| *** jaosorior has joined #openstack-nova | 06:15 | |
| *** ivase has joined #openstack-nova | 06:17 | |
| *** jaosorior has quit IRC | 06:17 | |
| *** ivase_ has quit IRC | 06:18 | |
| *** jaosorior has joined #openstack-nova | 06:18 | |
| *** ekuris_ has joined #openstack-nova | 06:20 | |
| *** salv-orlando has joined #openstack-nova | 06:21 | |
| *** rcernin has joined #openstack-nova | 06:23 | |
| *** gongysh_ has joined #openstack-nova | 06:24 | |
| *** salv-orlando has quit IRC | 06:24 | |
| *** haukebruno has quit IRC | 06:24 | |
| *** sheel has quit IRC | 06:27 | |
| *** ivase_ has joined #openstack-nova | 06:29 | |
| *** markvoelker has joined #openstack-nova | 06:29 | |
| *** sheel has joined #openstack-nova | 06:29 | |
| *** ljxiash has quit IRC | 06:29 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Imported Translations from Zanata https://review.openstack.org/293921 | 06:29 |
| *** cfriesen__ has quit IRC | 06:29 | |
| *** ivase has quit IRC | 06:30 | |
| *** gongysh_ has quit IRC | 06:32 | |
| *** markvoelker has quit IRC | 06:34 | |
| *** ivase has joined #openstack-nova | 06:34 | |
| *** ivase_ has quit IRC | 06:36 | |
| *** y_sawai has quit IRC | 06:39 | |
| openstackgerrit | apporc proposed openstack/nova: fix admin context scenarios for cinderclient https://review.openstack.org/296890 | 06:39 |
| *** y_sawai has joined #openstack-nova | 06:39 | |
| *** ivase has quit IRC | 06:41 | |
| *** Daisy has joined #openstack-nova | 06:41 | |
| openstackgerrit | Ankit Agrawal proposed openstack/nova: Move is_volume_backed_instance to compute.utils https://review.openstack.org/270482 | 06:43 |
| *** ljxiash has joined #openstack-nova | 06:43 | |
| *** Daisy has quit IRC | 06:44 | |
| *** Daisy has joined #openstack-nova | 06:45 | |
| *** ishant has joined #openstack-nova | 06:45 | |
| *** houming_ has joined #openstack-nova | 06:48 | |
| *** moshele has joined #openstack-nova | 06:48 | |
| *** houming has quit IRC | 06:48 | |
| *** houming_ is now known as houming | 06:48 | |
| *** Daisy has quit IRC | 06:49 | |
| *** fawadkhaliq has quit IRC | 06:53 | |
| *** gongysh has quit IRC | 06:54 | |
| openstackgerrit | Dinesh Bhor proposed openstack/nova: Provide user friendly messages for db sync https://review.openstack.org/296920 | 06:54 |
| openstackgerrit | Ankit Agrawal proposed openstack/nova: Remove redundant migration status update https://review.openstack.org/247519 | 06:56 |
| *** gongysh has joined #openstack-nova | 06:56 | |
| *** liverpooler has joined #openstack-nova | 06:56 | |
| *** unicell has joined #openstack-nova | 06:57 | |
| *** qeelee has quit IRC | 07:00 | |
| *** haomaiwang has quit IRC | 07:01 | |
| *** hoangcx has quit IRC | 07:01 | |
| *** mylu has joined #openstack-nova | 07:01 | |
| *** haomaiwa_ has joined #openstack-nova | 07:01 | |
| *** gongysh_ has joined #openstack-nova | 07:02 | |
| *** lpetrut has joined #openstack-nova | 07:02 | |
| *** ssurana has quit IRC | 07:03 | |
| *** browne has quit IRC | 07:03 | |
| *** hoangcx has joined #openstack-nova | 07:04 | |
| *** qeelee has joined #openstack-nova | 07:04 | |
| *** mylu has quit IRC | 07:06 | |
| *** lifeless has quit IRC | 07:06 | |
| *** Daisy has joined #openstack-nova | 07:08 | |
| *** y_sawai has quit IRC | 07:09 | |
| *** houming has quit IRC | 07:10 | |
| *** houming has joined #openstack-nova | 07:11 | |
| openstackgerrit | Srushti Gadadare proposed openstack/nova: Fix invalid uuid warnings in server-group unit tests https://review.openstack.org/293840 | 07:18 |
| *** cristicalin has joined #openstack-nova | 07:19 | |
| *** yamahata has quit IRC | 07:21 | |
| *** Marga_ has quit IRC | 07:27 | |
| *** ihrachys has joined #openstack-nova | 07:29 | |
| *** pcaruana has joined #openstack-nova | 07:32 | |
| *** daemontool has joined #openstack-nova | 07:33 | |
| *** achanda has quit IRC | 07:33 | |
| openstackgerrit | Ankit Agrawal proposed openstack/nova: Fix evaluation of host disk usage by volume-backed instances https://review.openstack.org/200870 | 07:34 |
| *** zenoway has joined #openstack-nova | 07:36 | |
| *** achanda has joined #openstack-nova | 07:37 | |
| openstackgerrit | Anh Tran proposed openstack/nova: Removes some redundant words https://review.openstack.org/296928 | 07:37 |
| *** Marga_ has joined #openstack-nova | 07:40 | |
| *** tesseract has joined #openstack-nova | 07:41 | |
| *** tesseract is now known as Guest27982 | 07:42 | |
| openstackgerrit | Vasyl Saienko proposed openstack/nova: Allow to update resource per single node. https://review.openstack.org/287485 | 07:45 |
| *** ihrachys has quit IRC | 07:45 | |
| *** Marga_ has quit IRC | 07:45 | |
| *** ihrachys has joined #openstack-nova | 07:45 | |
| *** ndipanov has quit IRC | 07:46 | |
| *** scheuran has joined #openstack-nova | 07:48 | |
| *** ihrachys has quit IRC | 07:49 | |
| *** ihrachys has joined #openstack-nova | 07:49 | |
| *** ivase has joined #openstack-nova | 07:51 | |
| *** ivase_ has joined #openstack-nova | 07:55 | |
| *** achanda has quit IRC | 07:56 | |
| *** markvoelker has joined #openstack-nova | 07:56 | |
| *** ivase has quit IRC | 07:56 | |
| *** belmoreira has joined #openstack-nova | 07:57 | |
| *** salv-orlando has joined #openstack-nova | 07:57 | |
| *** haomaiwa_ has quit IRC | 08:01 | |
| *** haomaiwang has joined #openstack-nova | 08:01 | |
| *** salv-orlando has quit IRC | 08:02 | |
| *** ivase_ has quit IRC | 08:02 | |
| *** markvoelker has quit IRC | 08:02 | |
| *** ekuris_ has quit IRC | 08:02 | |
| *** matrohon has joined #openstack-nova | 08:03 | |
| openstackgerrit | Dina Belova proposed openstack/nova: Integrate OSprofiler and Nova https://review.openstack.org/254703 | 08:03 |
| *** ljxiash has quit IRC | 08:04 | |
| *** drankis has joined #openstack-nova | 08:05 | |
| *** ljxiash has joined #openstack-nova | 08:06 | |
| *** cristicalin has quit IRC | 08:07 | |
| *** ljxiash has quit IRC | 08:10 | |
| *** achanda has joined #openstack-nova | 08:10 | |
| *** rdopiera has joined #openstack-nova | 08:12 | |
| openstackgerrit | apporc proposed openstack/nova: fix admin context scenarios for cinderclient https://review.openstack.org/296890 | 08:17 |
| *** ekuris_ has joined #openstack-nova | 08:17 | |
| *** gszasz has joined #openstack-nova | 08:18 | |
| *** mylu has joined #openstack-nova | 08:19 | |
| *** salv-orl_ has joined #openstack-nova | 08:21 | |
| *** mpavone has joined #openstack-nova | 08:23 | |
| *** mylu has quit IRC | 08:23 | |
| *** lpetrut has quit IRC | 08:23 | |
| *** snikitin has joined #openstack-nova | 08:24 | |
| *** PaulMurray has joined #openstack-nova | 08:25 | |
| *** e0ne has joined #openstack-nova | 08:29 | |
| *** jlanoux has joined #openstack-nova | 08:30 | |
| *** jlanoux has quit IRC | 08:31 | |
| *** jlanoux has joined #openstack-nova | 08:32 | |
| *** rk4n has joined #openstack-nova | 08:34 | |
| *** rk4n has quit IRC | 08:34 | |
| markus_z | johnthetubaguy: bauzas: https://bugs.launchpad.net/nova/+bug/1560860 sounds like a regression to me. A patch is on its way. | 08:35 |
| openstack | Launchpad bug 1560860 in OpenStack Compute (nova) "mellanox infiniband SR-IOV(ib_hostdev vif) detach port fails" [Critical,In progress] - Assigned to Moshe Levi (moshele) | 08:35 |
| *** rk4n has joined #openstack-nova | 08:35 | |
| *** sheel has quit IRC | 08:37 | |
| *** ekuris_ has quit IRC | 08:38 | |
| *** qeelee has quit IRC | 08:38 | |
| *** lpetrut has joined #openstack-nova | 08:39 | |
| *** macsz has joined #openstack-nova | 08:40 | |
| *** qeelee has joined #openstack-nova | 08:41 | |
| *** matrohon has quit IRC | 08:42 | |
| *** achanda has quit IRC | 08:45 | |
| openstackgerrit | Madhuri Kumari proposed openstack/nova: Fix API Error on hypervisor-uptime API https://review.openstack.org/289868 | 08:45 |
| *** zehua has quit IRC | 08:46 | |
| *** zehua has joined #openstack-nova | 08:46 | |
| *** lpetrut has quit IRC | 08:49 | |
| bauzas | markus_z: morning | 08:50 |
| *** Jeffrey4l has joined #openstack-nova | 08:50 | |
| markus_z | bauzas: good morning | 08:50 |
| *** inara has quit IRC | 08:50 | |
| bauzas | markus_z: I wouldn't maybe flag this as Critical since it only impacts only one type of deployment | 08:51 |
| *** ekuris_ has joined #openstack-nova | 08:51 | |
| markus_z | bauzas: hm, yeah. | 08:52 |
| bauzas | moshele: around ? | 08:52 |
| bauzas | moshele: I can see https://review.openstack.org/#/c/296305/ being -W | 08:52 |
| markus_z | bauzas: thanks for resetting the importance. I'd learn better how to set proper importance if a reasoning is given in a bug report comment. Just as a hint for future work. | 08:53 |
| bauzas | moshele: so the problem is with suspend AFAIU | 08:54 |
| bauzas | moshele: it doesn't prevent the instance to boot | 08:54 |
| bauzas | I don't want to nitpick against importance type, but I think it's really not an issue for all | 08:54 |
| bauzas | moshele: that tho means that SRIOV instances can't resume, which is a regression, I agree | 08:55 |
| bauzas | markus_z: ^ | 08:55 |
| *** achanda has joined #openstack-nova | 08:56 | |
| markus_z | bauzas: yep, I understand your reasoning now. It was also about spreading that information to others. | 08:57 |
| moshele | bauzas: the problem is with suspend | 08:59 |
| bauzas | moshele: okay, any plan to finish the change so we could review it ? | 09:00 |
| *** haomaiwang has quit IRC | 09:01 | |
| moshele | bauzas: yes I plan to do it today | 09:01 |
| *** haomaiwang has joined #openstack-nova | 09:01 | |
| openstackgerrit | Madhuri Kumari proposed openstack/python-novaclient: Change aggregate-details to aggregate-show https://review.openstack.org/289215 | 09:01 |
| bauzas | moshele: cool, FYI we're planning to ship an RC2 candidate by today, that just means your change would only be targeted for RC3 | 09:02 |
| moshele | bauzas: understood | 09:02 |
| *** ihrachys has quit IRC | 09:03 | |
| bauzas | moshele: are you able to test your change on your local testbed ? I'm very concerned by any functional testing we could do | 09:03 |
| moshele | bauzas: yes, I put W-1 because I have not test on real setup, so now I am preparing SR-IOV setup to test it | 09:04 |
| bauzas | coolness | 09:08 |
| *** lucas-dinner is now known as lucasagomes | 09:11 | |
| bauzas | ttx: morning | 09:14 |
| bauzas | ttx: we plan to deliver RC2 later this morning, but I was hoping to integrate https://review.openstack.org/#/c/296661/2 within the release | 09:15 |
| bauzas | ttx: we can wait for the experimental job to run, but I'd appreciate if you could take a look at the above by today :-) | 09:15 |
| *** garyk has quit IRC | 09:17 | |
| ttx | oops | 09:18 |
| ttx | I think I already approved it | 09:18 |
| ttx | bauzas: ^ | 09:19 |
| *** yassine has joined #openstack-nova | 09:19 | |
| *** yassine is now known as Guest54256 | 09:19 | |
| bauzas | ttx: nah, no worries, it was tested manually in the newton branch | 09:19 |
| bauzas | ttx: thanks | 09:19 |
| bauzas | johnthetubaguy: around ? | 09:20 |
| johnthetubaguy | bauzas: hey | 09:20 |
| bauzas | johnthetubaguy: so, two points | 09:20 |
| bauzas | johnthetubaguy: not sure you saw the backlog but moshele is working on a regression bugfix | 09:21 |
| ttx | bauzas: also have a translations import queued up | 09:21 |
| bauzas | johnthetubaguy: https://review.openstack.org/#/c/296305/1 | 09:21 |
| bauzas | johnthetubaguy: which means RC3 | 09:21 |
| bauzas | (anyway) | 09:21 |
| bauzas | ttx: oh okay | 09:21 |
| johnthetubaguy | bauzas: we need RC3 for translations anyways | 09:21 |
| *** ndipanov has joined #openstack-nova | 09:21 | |
| ttx | Left https://review.openstack.org/#/c/295319/ out though, missing PTL / release liaison +2 | 09:21 |
| bauzas | johnthetubaguy: agreed, hence my anyway :) | 09:21 |
| bauzas | johnthetubaguy: so, second point | 09:22 |
| bauzas | johnthetubaguy: I discussed with tpeoples yesterday on https://bugs.launchpad.net/nova/+bug/1561357 | 09:22 |
| openstack | Launchpad bug 1561357 in OpenStack Compute (nova) "VM deployed with availability-zone (force_hosts) cannot be live migrated to an untargeted host" [Undecided,New] | 09:22 |
| *** hoangcx has quit IRC | 09:23 | |
| bauzas | johnthetubaguy: I don't think it's really a regression, because if the operators forces the destination when booting, he can force the destination when live-migrating (using the host optional field) | 09:23 |
| johnthetubaguy | bauzas: oh, its kinda waiting for your API changes | 09:23 |
| johnthetubaguy | bauzas: does the force host thing work OK? | 09:23 |
| johnthetubaguy | I guess it does, it skips the scheduler | 09:24 |
| *** libu has quit IRC | 09:24 | |
| bauzas | johnthetubaguy: yeah so there is a workaround | 09:24 |
| bauzas | johnthetubaguy: the main problem being with the other methods (resize) not possibly asking a destination | 09:24 |
| johnthetubaguy | hmm, yes | 09:25 |
| *** hoangcx has joined #openstack-nova | 09:25 | |
| bauzas | johnthetubaguy: my first guess on that (that can be newton-ish I think) is that we should provide force_hosts to the scheduler but not persist it | 09:25 |
| johnthetubaguy | if feels like we should drop that force host for any "move" operation, in a way | 09:25 |
| bauzas | johnthetubaguy: TBH, I don't like force_hosts, that's a very poor thing | 09:25 |
| johnthetubaguy | I think we need to persist the request spec fully really, but we should think through that | 09:26 |
| bauzas | johnthetubaguy: but given we support it, let's consider this as non-persistent maybe | 09:26 |
| johnthetubaguy | bauzas: so the better affinity stuff we have bp up for should help replace that | 09:26 |
| johnthetubaguy | right now, you kinda need it way more than you would hope | 09:26 |
| bauzas | yeah | 09:26 |
| bauzas | so, here is the thing | 09:26 |
| *** achanda has quit IRC | 09:26 | |
| bauzas | if people force a destination, that's because they don't trust the scheduler | 09:27 |
| *** vilobhmm11 has quit IRC | 09:27 | |
| johnthetubaguy | bauzas: well, not always, its because there is no way to tell the scheduler what they want | 09:27 |
| johnthetubaguy | but I guess thats really the same thing | 09:28 |
| *** ndipanov has quit IRC | 09:28 | |
| bauzas | I think it's eventually a problem of trusting the scheduler | 09:28 |
| bauzas | if you want to test a specific compute node, you can do that other ways | 09:28 |
| bauzas | like for example using aggregates | 09:28 |
| *** ndipanov has joined #openstack-nova | 09:29 | |
| bauzas | (and flavors or whatever) | 09:29 |
| bauzas | so the thing is, force_hosts is not really a contract we should honor in the future | 09:29 |
| bauzas | ttx: https://review.openstack.org/#/c/295319/ is not a regression, it's just it never worked | 09:32 |
| johnthetubaguy | bauzas: honestly, folks want it | 09:33 |
| johnthetubaguy | bauzas: erm, it surely used to right? | 09:33 |
| bauzas | johnthetubaguy: I agree, do they want to *persist* it ? | 09:33 |
| bauzas | johnthetubaguy: https://review.openstack.org/#/c/295319/ is reported by https://bugzilla.redhat.com/show_bug.cgi?id=1313254 which is OSP6 (Juno) | 09:33 |
| openstack | bugzilla.redhat.com bug 1313254 in openstack-nova "no retry when using 'nova image-list' on next configured glance api_servers endpoint" [High,On_dev] - Assigned to dclarke | 09:33 |
| *** sfinucan has joined #openstack-nova | 09:34 | |
| bauzas | so maybe saying "never" is too hard | 09:34 |
| johnthetubaguy | bauzas: oh, we broke it years ago | 09:34 |
| johnthetubaguy | so fun fact... | 09:34 |
| johnthetubaguy | if you retry, glance will generally give you an error | 09:34 |
| *** houming has quit IRC | 09:34 | |
| johnthetubaguy | so it only works for transient errors that trigger 500s, so we should probably not worry about it | 09:35 |
| bauzas | TBH, I'd be far happier if that round-robin was done in the glanceclient instead of the nova wrapper, but whatsoever | 09:35 |
| markus_z | bauzas: As a heads up, we (Germany) have 2 public holidays tomorrow and Monday. In case anyone asks about me. | 09:36 |
| bauzas | anyway, glance v2 for Nova seems a nice thing to focus on Newton, so I'm not really worried by the fix | 09:36 |
| bauzas | markus_z: WAAAAAT? Germans have more holidays than France ? | 09:36 |
| *** andymaier has joined #openstack-nova | 09:36 | |
| bauzas | :p | 09:36 |
| johnthetubaguy | markus_z: ah, we are the same in the UK actually, good friday and easter monday off | 09:36 |
| markus_z | bauzas: we strike less ;) | 09:36 |
| *** Daisy has quit IRC | 09:37 | |
| *** salv-orl_ has quit IRC | 09:37 | |
| * johnthetubaguy enjoys his last few weeks in the EU (well hopefully not...!!) | 09:37 | |
| markus_z | johnthetubaguy: hm? where are you going? | 09:37 |
| *** Daisy has joined #openstack-nova | 09:37 | |
| bauzas | johnthetubaguy: heh, UK was in EU ? | 09:37 |
| johnthetubaguy | markus_z: its more about what the UK will vote for | 09:37 |
| bauzas | :p | 09:37 |
| *** salv-orlando has joined #openstack-nova | 09:37 | |
| markus_z | oh, that thing | 09:37 |
| bauzas | markus_z: I'm flying from GVA now :p | 09:37 |
| johnthetubaguy | bauzas: we are the cry babies in the corner moaning all the time, hard to miss us | 09:37 |
| markus_z | he he :D | 09:38 |
| markus_z | we like you though | 09:38 |
| bauzas | I stopped departing from LYS for many reasons | 09:38 |
| bauzas | johnthetubaguy: heh | 09:38 |
| bauzas | I think US folks also have holidays | 09:39 |
| bauzas | for Easter | 09:39 |
| johnthetubaguy | bauzas: I was more thinking that host target is always persisted, but always dropped when re-using a stored request spec, otherwise we loose knowing what the original API request was, but I might be overthinking that | 09:41 |
| bauzas | johnthetubaguy: interesting thoughts | 09:41 |
| johnthetubaguy | bauzas: agreed we can probably hold back on that retry patch | 09:41 |
| *** sfinucan has quit IRC | 09:41 | |
| *** ihrachys has joined #openstack-nova | 09:41 | |
| bauzas | johnthetubaguy: so you see the other way | 09:41 |
| bauzas | which is probably better | 09:42 |
| *** Daisy has quit IRC | 09:42 | |
| tojuvone | johnthetubaguy: Btw drafted some maintenance spec already, but it is kinda under review in OPNFV still as looking the requirements/use cases... so did not bring to API yesterday. | 09:42 |
| bauzas | johnthetubaguy: unrelated, I'm monitoring https://review.openstack.org/#/c/296661/2 and will provide a release patch against the merge SHA1 | 09:43 |
| johnthetubaguy | tojuvone: OK, they were busy in the end worrying about microversion docs, so that was lucky in some ways | 09:43 |
| johnthetubaguy | bauzas: cool, looks like we have a good bunch of translations merged in there, which is great | 09:44 |
| markus_z | johnthetubaguy: These openstack specs about config options, what will be the impact? Will they result in a guidance for other projects like cinder and neutron? | 09:44 |
| ildikov | johnthetubaguy: | 09:44 |
| markus_z | johnthetubaguy: IOW, what's the TC going to do with them? | 09:45 |
| ildikov | johnthetubaguy: hi :) | 09:45 |
| bauzas | johnthetubaguy: yeah, hence me waiting for the merge sha1 because the zanate change merging first | 09:45 |
| tojuvone | johnthetubaguy: Yes, that was important topic. Need those microversion docs asap. | 09:45 |
| johnthetubaguy | markus_z: its not really for the TC, its more about projects agreeing on how to improve their config options, I liked ttx's point on the ML, we probably should store "best practice" guides separately | 09:46 |
| ildikov | johnthetubaguy: I just wanted to ask whether you saw the meeting plans for next week regarding the Cinder-Nova API changes we plan to do? | 09:46 |
| johnthetubaguy | ildikov: I did, I think with the time change, I should be able to make the end of it | 09:46 |
| markus_z | johnthetubaguy: ok, got it | 09:47 |
| johnthetubaguy | ildikov: but its a tricky time for me, its on one of my rehearsal nights | 09:47 |
| tojuvone | johnthetubaguy: Meanwhile it might be I wait those OPNFV comments while could otherwise discuss here already. | 09:47 |
| ildikov | johnthetubaguy: uh, sorry :( | 09:47 |
| johnthetubaguy | tojuvone: I would probably go for doing both in parallel really | 09:47 |
| johnthetubaguy | ildikov: no worries, its never going to be a perfect time for everyone | 09:48 |
| ildikov | johnthetubaguy: it's a bit of a challenge to organize a meeting with participants all around the globe, plus time change... :( | 09:48 |
| johnthetubaguy | ildikov: yeah, its never fun, you always leave folks out | 09:48 |
| tojuvone | johnthetubaguy: ok, maybe good because sooner the better. Not the first time I would have redrafted the whole spec to rescope | 09:49 |
| ildikov | johnthetubaguy: I picked this slot as it at least gives a chance in most time zones, even if it's uncomfortable for a few of us | 09:50 |
| ildikov | johnthetubaguy: anyway, if you can make it to the end of the meeting I still appreciate, I will ping ndipanov and mriedem as well if they are interested | 09:51 |
| ildikov | johnthetubaguy: it would be good to have coverage from both teams | 09:51 |
| *** inara has joined #openstack-nova | 09:52 | |
| johnthetubaguy | tojuvone: I would worry about the problem spec the most, at this point, get agreement on that bit | 09:52 |
| johnthetubaguy | ildikov: yeah, ping us Tuesday/Wednesday, to remind us, I mean to read through things before the meeting, and see how stuff is going | 09:53 |
| *** qeelee has quit IRC | 09:53 | |
| tojuvone | johnthetubaguy: Well, maybe I just add it then :) | 09:55 |
| ildikov | johnthetubaguy: sure, tnx much for support | 09:55 |
| *** ishant has quit IRC | 09:57 | |
| *** markvoelker has joined #openstack-nova | 09:58 | |
| openstackgerrit | Jianghua Wang proposed openstack/nova: XenAPI: OVS agent updates the wrong port with Neutron https://review.openstack.org/242846 | 09:59 |
| *** haomaiwang has quit IRC | 10:01 | |
| moshele | ndipanov: hi | 10:01 |
| *** haomaiwang has joined #openstack-nova | 10:01 | |
| ndipanov | hey moshele | 10:02 |
| moshele | ndipanov: can you look at https://review.openstack.org/#/c/296305/ | 10:03 |
| *** markvoelker has quit IRC | 10:03 | |
| moshele | ndipanov: please read the details of the bug https://bugs.launchpad.net/nova/+bug/1560860 | 10:03 |
| openstack | Launchpad bug 1560860 in OpenStack Compute (nova) "mellanox infiniband SR-IOV(ib_hostdev vif) detach port fails" [High,In progress] - Assigned to Moshe Levi (moshele) | 10:03 |
| ndipanov | moshele, I always read the bug :) | 10:04 |
| moshele | ndipanov: :) | 10:04 |
| *** ljxiash has joined #openstack-nova | 10:06 | |
| *** Jeffrey4l has quit IRC | 10:06 | |
| *** ljjjustin has left #openstack-nova | 10:07 | |
| *** ljjjustin has joined #openstack-nova | 10:07 | |
| *** mylu has joined #openstack-nova | 10:07 | |
| ndipanov | moshele, so using vif data instead of libvirt config works in all cases? | 10:08 |
| *** sfinucan has joined #openstack-nova | 10:09 | |
| ndipanov | or I guess the question is - do you think it's more robust then relying on the libvirt config even if it requires different cases for different xml elements | 10:09 |
| ndipanov | moshele, ^ | 10:09 |
| ndipanov | let me ask it on the review | 10:09 |
| moshele | ndipanov: yes this is also an option | 10:10 |
| openstackgerrit | Ryan Moore proposed openstack/nova: Add better help to rdp options https://review.openstack.org/253556 | 10:10 |
| *** ljxiash has quit IRC | 10:10 | |
| *** moorryan has joined #openstack-nova | 10:11 | |
| moshele | ndipanov: what do you think? maybe relaying on the vif is not that good | 10:11 |
| *** mylu has quit IRC | 10:12 | |
| ndipanov | moshele, tbh I am not sure | 10:13 |
| ndipanov | moshele, well actually get_config will get it from the vif anyway right? | 10:14 |
| moshele | ndipanov: a sec looking at the code | 10:16 |
| ndipanov | moshele, yeah found it | 10:17 |
| *** Jeffrey4l has joined #openstack-nova | 10:18 | |
| openstackgerrit | Madhuri Kumari proposed openstack/python-novaclient: Change aggregate-details to aggregate-show https://review.openstack.org/289215 | 10:18 |
| moshele | ndipanov: yes, so I think the fix is ok | 10:18 |
| moshele | ndipanov: and the Ethernet mellanox ci passed so I didn't break it for Ethernet sr-iov | 10:20 |
| ndipanov | ok | 10:20 |
| ndipanov | let me comment | 10:20 |
| *** inara has quit IRC | 10:20 | |
| *** gongysh_ has quit IRC | 10:21 | |
| *** tojuvone has quit IRC | 10:22 | |
| *** tojuvone has joined #openstack-nova | 10:22 | |
| openstackgerrit | Tomi Juvonen proposed openstack/nova-specs: Set the time period when host is under maintenance https://review.openstack.org/296995 | 10:22 |
| *** gongysh_ has joined #openstack-nova | 10:23 | |
| *** sdague has joined #openstack-nova | 10:24 | |
| openstackgerrit | Madhuri Kumari proposed openstack/nova: Fix API Error on hypervisor-uptime API https://review.openstack.org/289868 | 10:29 |
| *** inara has joined #openstack-nova | 10:30 | |
| *** _gryf has quit IRC | 10:30 | |
| *** jaosorior has quit IRC | 10:34 | |
| *** Jeffrey4l has quit IRC | 10:35 | |
| *** ndipanov has quit IRC | 10:38 | |
| *** cdent has joined #openstack-nova | 10:39 | |
| cdent | p!spy | 10:40 |
| *** hoangcx has quit IRC | 10:41 | |
| *** jaosorior has joined #openstack-nova | 10:43 | |
| bauzas | johnthetubaguy: mriedem: https://review.openstack.org/#/c/297006/ | 10:43 |
| *** erlon has joined #openstack-nova | 10:45 | |
| *** daemontool has quit IRC | 10:46 | |
| johnthetubaguy | bauzas: hmm, got reordered in the gate I guess, but that looks good | 10:46 |
| bauzas | johnthetubaguy: nope, I just verified against http://git.openstack.org/cgit/openstack/nova/log/?h=stable/mitaka | 10:46 |
| johnthetubaguy | bauzas: I was expecting the translations to go first for some reason, but its all good | 10:48 |
| bauzas | me too, but zuul left us wondering :p | 10:48 |
| openstackgerrit | Wenzhi Yu proposed openstack/nova-specs: Add network monitor for compute node https://review.openstack.org/297011 | 10:49 |
| *** gongysh_ has quit IRC | 10:51 | |
| *** claudiub|2 has quit IRC | 10:52 | |
| *** _gryf has joined #openstack-nova | 10:55 | |
| *** ociuhandu has quit IRC | 10:57 | |
| *** haomaiwang has quit IRC | 11:00 | |
| *** liunian_ has joined #openstack-nova | 11:03 | |
| *** haomaiwang has joined #openstack-nova | 11:04 | |
| *** matrohon has joined #openstack-nova | 11:06 | |
| *** lifeless has joined #openstack-nova | 11:07 | |
| *** zehua has quit IRC | 11:07 | |
| *** moorryan has quit IRC | 11:08 | |
| *** zehua has joined #openstack-nova | 11:08 | |
| *** liunian_ has quit IRC | 11:08 | |
| *** brad[] has quit IRC | 11:08 | |
| *** liunian_ has joined #openstack-nova | 11:09 | |
| *** brad[] has joined #openstack-nova | 11:09 | |
| *** zehua_ has joined #openstack-nova | 11:11 | |
| *** baoli has joined #openstack-nova | 11:11 | |
| *** zehua has quit IRC | 11:12 | |
| *** liunian_ has quit IRC | 11:13 | |
| *** artom has quit IRC | 11:16 | |
| *** zehua_ has quit IRC | 11:16 | |
| *** ociuhandu has joined #openstack-nova | 11:18 | |
| *** baoli_ has joined #openstack-nova | 11:18 | |
| *** baoli_ has quit IRC | 11:20 | |
| *** baoli_ has joined #openstack-nova | 11:20 | |
| *** baoli has quit IRC | 11:21 | |
| openstackgerrit | John Garbutt proposed openstack/nova: Config options: Centralize resource tracker options https://review.openstack.org/294211 | 11:21 |
| *** rtheis has joined #openstack-nova | 11:23 | |
| *** smatzek has joined #openstack-nova | 11:23 | |
| *** Guest54256 is now known as yassou | 11:24 | |
| openstackgerrit | John Garbutt proposed openstack/nova: Config options: Centralize netconf options https://review.openstack.org/294312 | 11:27 |
| *** mvk_ has quit IRC | 11:28 | |
| pkoniszewski | bauzas: hey, there? | 11:30 |
| *** baoli_ has quit IRC | 11:31 | |
| *** baoli has joined #openstack-nova | 11:31 | |
| *** rdopiera has quit IRC | 11:32 | |
| *** baoli has quit IRC | 11:35 | |
| *** sudipto has quit IRC | 11:35 | |
| *** baoli has joined #openstack-nova | 11:35 | |
| *** ivase has joined #openstack-nova | 11:37 | |
| pkoniszewski | bauzas: nvm, just found out that we are safe with soft-affinity/anti-affinity | 11:39 |
| *** rdopiera has joined #openstack-nova | 11:39 | |
| *** sfinucan has quit IRC | 11:40 | |
| *** ivase_ has joined #openstack-nova | 11:41 | |
| *** ivase has quit IRC | 11:41 | |
| *** thorst has joined #openstack-nova | 11:43 | |
| *** sfinucan has joined #openstack-nova | 11:43 | |
| *** ndipanov has joined #openstack-nova | 11:44 | |
| *** ivase has joined #openstack-nova | 11:46 | |
| openstackgerrit | Chris Dent proposed openstack/nova: Add ResourcePool and ResourcePoolList objects https://review.openstack.org/284963 | 11:47 |
| *** PaulMurray has quit IRC | 11:47 | |
| *** ivase_ has quit IRC | 11:47 | |
| *** aysyd has joined #openstack-nova | 11:51 | |
| *** ivase has quit IRC | 11:53 | |
| openstackgerrit | Chris Dent proposed openstack/nova: Add Allocation and AllocationList objects https://review.openstack.org/282442 | 11:54 |
| openstackgerrit | Chris Dent proposed openstack/nova: Add ResourcePool and ResourcePoolList objects https://review.openstack.org/284963 | 11:54 |
| openstackgerrit | Chris Dent proposed openstack/nova: Add a get_by_uuid for aggregates https://review.openstack.org/284895 | 11:54 |
| openstackgerrit | Chris Dent proposed openstack/nova: Add name to ResourceProvider object https://review.openstack.org/281945 | 11:54 |
| openstackgerrit | Chris Dent proposed openstack/nova: WIP resource-pools/placement api https://review.openstack.org/293104 | 11:54 |
| *** edmondsw has joined #openstack-nova | 11:54 | |
| *** mylu has joined #openstack-nova | 11:55 | |
| cdent | johnthetubaguy: I've reorderd that stack of resource-* stuff so that that allocation item stuff is at the start, since it is associated with the resource-providers spec which is re-approved. Can you take your -2 off https://review.openstack.org/282442 ? | 11:58 |
| *** sheel has joined #openstack-nova | 11:58 | |
| openstackgerrit | Chris Dent proposed openstack/nova: Add ResourcePool and ResourcePoolList objects https://review.openstack.org/284963 | 11:59 |
| *** markvoelker has joined #openstack-nova | 11:59 | |
| *** mylu has quit IRC | 11:59 | |
| *** rfolco_ has joined #openstack-nova | 12:00 | |
| openstackgerrit | Chris Dent proposed openstack/nova: WIP resource-pools/placement api https://review.openstack.org/293104 | 12:00 |
| *** haomaiwang has quit IRC | 12:01 | |
| *** haomaiwang has joined #openstack-nova | 12:01 | |
| *** salv-orl_ has joined #openstack-nova | 12:02 | |
| *** salv-orl_ has quit IRC | 12:02 | |
| cdent | dansmith: What's the current state on the compute-node-inventory stuff, if any? https://review.openstack.org/#/c/279313/ | 12:04 |
| *** salv-orlando has quit IRC | 12:05 | |
| *** ababich has quit IRC | 12:06 | |
| *** daemontool has joined #openstack-nova | 12:07 | |
| *** EinstCrazy has joined #openstack-nova | 12:08 | |
| *** moshele has quit IRC | 12:08 | |
| *** zehua has joined #openstack-nova | 12:10 | |
| *** cdent has quit IRC | 12:11 | |
| *** daemontool has quit IRC | 12:15 | |
| *** dave-mccowan has joined #openstack-nova | 12:17 | |
| *** mriedem has joined #openstack-nova | 12:21 | |
| mriedem | snow day! | 12:21 |
| alaski | melwitt: thanks! http://lists.openstack.org/pipermail/openstack-dev/2016-March/090291.html | 12:22 |
| *** ivase has joined #openstack-nova | 12:23 | |
| *** klkumar has joined #openstack-nova | 12:24 | |
| *** mtanino has joined #openstack-nova | 12:25 | |
| mriedem | alaski: if you click off the comment, you can move around again | 12:26 |
| mriedem | not helpful if you're mid-comment though | 12:26 |
| mriedem | i sometimes open the same review in 2 tabs, one for reference and one for commenting | 12:26 |
| alaski | yeah, I end up doing that as well at times | 12:27 |
| *** ivase_ has joined #openstack-nova | 12:27 | |
| alaski | but the big issue is the opposite, the page moves as I'm commenting | 12:27 |
| alaski | and I end up typing into a comment box that's no longer on screen | 12:27 |
| *** ivase has quit IRC | 12:28 | |
| mriedem | hmm, i don't think i've had that issue | 12:28 |
| mriedem | mdbooth: it looks like the entire development staff at ZTE disagree with us https://review.openstack.org/#/c/289813/:) | 12:28 |
| *** garyk has joined #openstack-nova | 12:31 | |
| garyk | johnthetubaguy: can you please remove your -2 - https://review.openstack.org/#/c/188321/ | 12:31 |
| *** mylu has joined #openstack-nova | 12:31 | |
| *** haukebruno has joined #openstack-nova | 12:32 | |
| *** ivase_ has quit IRC | 12:34 | |
| *** salv-orlando has joined #openstack-nova | 12:34 | |
| *** mdrabe has joined #openstack-nova | 12:34 | |
| *** cdent has joined #openstack-nova | 12:35 | |
| *** mvk_ has joined #openstack-nova | 12:35 | |
| *** claudiub|2 has joined #openstack-nova | 12:36 | |
| cdent | p!spy | 12:36 |
| *** Daisy has joined #openstack-nova | 12:38 | |
| *** EinstCrazy has quit IRC | 12:38 | |
| mriedem | ndipanov: jbernard_: jdurgin: if you have rbd people at red hat, it would be good if they could look at this https://review.openstack.org/#/c/289813/ | 12:39 |
| Kevin_Zheng | johnthetubaguy: can you remove -2 on this one, it was also part of config job https://review.openstack.org/#/c/272949/ | 12:39 |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Just use 'last_seen_up' to check service's status https://review.openstack.org/297085 | 12:39 |
| mriedem | cburgess: also fyi on this rbd thing https://review.openstack.org/#/c/289813/ | 12:39 |
| *** EinstCrazy has joined #openstack-nova | 12:40 | |
| *** Daisy has quit IRC | 12:42 | |
| *** burgerk has joined #openstack-nova | 12:42 | |
| *** efried has joined #openstack-nova | 12:42 | |
| *** knikolla has quit IRC | 12:42 | |
| *** yuywz has quit IRC | 12:44 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: config options: centralize section: "glance" https://review.openstack.org/257153 | 12:45 |
| *** ivase has joined #openstack-nova | 12:46 | |
| *** rwmjones is now known as rwmjones_mtg | 12:47 | |
| *** ivase_ has joined #openstack-nova | 12:50 | |
| *** ivase has quit IRC | 12:51 | |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Make quotas commands more user-friendly https://review.openstack.org/290556 | 12:51 |
| *** ljxiash has joined #openstack-nova | 12:51 | |
| *** ccarmack has joined #openstack-nova | 12:53 | |
| *** tbachman has joined #openstack-nova | 12:54 | |
| *** akshai has joined #openstack-nova | 12:55 | |
| *** lpetrut has joined #openstack-nova | 12:56 | |
| *** mylu has quit IRC | 12:56 | |
| *** ccarmack1 has joined #openstack-nova | 12:56 | |
| *** ivase_ has quit IRC | 12:56 | |
| *** rlrossit has joined #openstack-nova | 12:57 | |
| cdent | bauzas, mriedem, johnthetubaguy, jaypipes, dansmith: I've updated the scheduler section on https://etherpad.openstack.org/p/newton-nova-priorities-tracking to reflect what I know. It is missing information about the compute-node-inventory stuff. dansmith can you add/update that please? | 12:57 |
| mriedem | armax: the port_security_enabled stuff has been a problem in nova for a long time, there is another attempt at a fix here https://review.openstack.org/#/c/284095/ - would be good to get some neutron cores on that | 12:57 |
| *** ccarmack has quit IRC | 12:59 | |
| *** Goneri has joined #openstack-nova | 13:00 | |
| *** klkumar has quit IRC | 13:00 | |
| *** haomaiwang has quit IRC | 13:01 | |
| *** haomaiwang has joined #openstack-nova | 13:01 | |
| *** ivase has joined #openstack-nova | 13:04 | |
| mriedem | ndipanov: the bug for https://review.openstack.org/#/c/296305/ is marked as an rc blocker, do you think we should hold up rc2 for it? | 13:04 |
| ildikov | mriedem: hi | 13:04 |
| ildikov | mriedem: sorry, it was already bedtime for me yesterday, when you pinged | 13:04 |
| mriedem | ildikov: i didn't think you slept | 13:04 |
| ildikov | mriedem: the answer to your question is yes, I plan to re-submit the blueprint for Newton | 13:05 |
| mriedem | ildikov: with probably updates for the detach case? | 13:05 |
| ildikov | mriedem: I'm sick a bit, so this time, I went to bed in time, otherwise you're right, I usually stay up late :) | 13:05 |
| *** vladikr has joined #openstack-nova | 13:06 | |
| ildikov | mriedem: if you saw my mail on the list we plan to have a meeting regarding Cinder-Nova interaction issues | 13:06 |
| ildikov | mriedem: one part of that is multiattach of course | 13:06 |
| ildikov | the plan is to have a cross-project spec to capture the big picture | 13:06 |
| *** eharney has quit IRC | 13:06 | |
| mriedem | ildikov: i don't think i saw that one yet | 13:06 |
| mriedem | meeting at the summit? | 13:06 |
| openstackgerrit | Chris Dent proposed openstack/nova: Add name to ResourceProvider object https://review.openstack.org/281945 | 13:07 |
| *** haomaiwang has quit IRC | 13:07 | |
| ildikov | I thought to update the multiattach spec after we see what will be covered by the API changes | 13:07 |
| *** rodrigods has quit IRC | 13:07 | |
| ildikov | mriedem: IRC meeting next Wednesday 2100UTC | 13:07 |
| ildikov | mriedem: I would like to have a Cinder-Nova session on the Summit too | 13:07 |
| *** ivase_ has joined #openstack-nova | 13:07 | |
| *** rodrigods has joined #openstack-nova | 13:08 | |
| *** ivase has quit IRC | 13:08 | |
| ildikov | mriedem: https://etherpad.openstack.org/p/cinder-nova-api-changes | 13:08 |
| openstackgerrit | Andreas Scheuring proposed openstack/nova: [WIP] libvirt: Update interface xml definition on migration https://review.openstack.org/297100 | 13:10 |
| mriedem | ildikov: can you send me a link to the ML thread? | 13:10 |
| *** sahid has joined #openstack-nova | 13:10 | |
| mriedem | ildikov: nvm, it's in the etherpad | 13:11 |
| *** EinstCrazy has quit IRC | 13:12 | |
| ildikov | mriedem: :) it's here if you would want to check it later still: http://osdir.com/ml/openstack-dev/2016-03/msg02426.html | 13:12 |
| *** ccarmack has joined #openstack-nova | 13:12 | |
| *** ducttape_ has joined #openstack-nova | 13:13 | |
| *** rlrossit has quit IRC | 13:13 | |
| mriedem | ildikov: ok, i'll be there | 13:13 |
| *** zehua has quit IRC | 13:13 | |
| *** ivase_ has quit IRC | 13:14 | |
| ildikov | mriedem: great, thanks much | 13:14 |
| *** klkumar has joined #openstack-nova | 13:14 | |
| mriedem | ildikov: we'll have a cinder/nova session at the summit, but would be good to iron out stuff before that 40 minute slot | 13:14 |
| *** ccarmack2 has joined #openstack-nova | 13:15 | |
| mriedem | johnthetubaguy: bauzas: i'm assuming you guys saw this new rc potential bug already https://bugs.launchpad.net/nova/+bug/1560860 | 13:15 |
| openstack | Launchpad bug 1560860 in OpenStack Compute (nova) "mellanox infiniband SR-IOV(ib_hostdev vif) detach port fails" [High,In progress] - Assigned to Moshe Levi (moshele) | 13:15 |
| *** ccarmack1 has quit IRC | 13:15 | |
| mriedem | johnthetubaguy: bauzas: just wondering if we should release rc2 now since we know we're still going to have an rc3 for translations | 13:15 |
| *** ivase has joined #openstack-nova | 13:15 | |
| ildikov | mriedem: of course, with scottda we're trying to put together a cross-project spec as I said earlier so that we would not need to handle everything in the multiattach spec which does not necessarily belong there on the first place | 13:15 |
| *** zehua has joined #openstack-nova | 13:15 | |
| mriedem | ildikov: you might want to read thingee's thred about cross-project specs | 13:16 |
| mriedem | seems like this would just be a cinder and a nova spec, separately | 13:16 |
| openstackgerrit | Chris Dent proposed openstack/nova: Add Allocation and AllocationList objects https://review.openstack.org/282442 | 13:16 |
| openstackgerrit | Chris Dent proposed openstack/nova: Add name to ResourceProvider object https://review.openstack.org/281945 | 13:16 |
| *** ducttape_ has quit IRC | 13:16 | |
| mriedem | my understanding is cross project specs are things that go into all openstack projects, like logging, api guidelines, et | 13:16 |
| mriedem | *etc | 13:16 |
| ildikov | mriedem: we also organize this meeting for this reason or maybe it will even be a meeting series until we don't get to some agreement | 13:16 |
| *** ccarmack has quit IRC | 13:17 | |
| ildikov | mriedem: from the last discussion with him our understanding was that we can capture the high level goals there and then the specifics go into specs in each project | 13:17 |
| openstackgerrit | Chris Dent proposed openstack/nova: Add a get_by_uuid for aggregates https://review.openstack.org/284895 | 13:17 |
| ildikov | mriedem: but well, if we get there that this (administrational) part will be the biggest issue to sort out, I'll be pretty happy :) | 13:18 |
| *** zehua has quit IRC | 13:18 | |
| bauzas | mriedem: yup, we discussed about the above bug for RC3 | 13:19 |
| *** ivase_ has joined #openstack-nova | 13:19 | |
| mriedem | ildikov: agreed | 13:19 |
| bauzas | mriedem: moshele will update it by today IIUC | 13:19 |
| *** ivase has quit IRC | 13:19 | |
| *** ccarmack has joined #openstack-nova | 13:19 | |
| bauzas | mriedem: and we agreed on providing a RC2 now but saying we'll also have another RC | 13:20 |
| *** zz_dimtruck is now known as dimtruck | 13:20 | |
| *** ccarmack2 has quit IRC | 13:21 | |
| sahid | mriedem: when you have a moment can you re-approve https://blueprints.launchpad.net/nova/+spec/libvirt-clean-driver | 13:22 |
| mriedem | sahid: yeah, sec | 13:22 |
| sahid | since it's long background work i would like to start soon for this release | 13:22 |
| sahid | cool thanks | 13:22 |
| *** zehua has joined #openstack-nova | 13:22 | |
| edleafe | melwitt: just saw your post re: gerrit jumping around. I thought it was just something odd on my machine. | 13:23 |
| mriedem | sahid: done, althought we probably should have created a libvirt-clean-driver-newton blueprint for newton work, john has been doing that for some of these multi-series refactor/cleanup blueprints | 13:23 |
| mriedem | *although | 13:23 |
| mriedem | bauzas: i see some NFV CI even failed on the original change, but it's not clear to me that they failed because of that actual change, which sucks. would be good if the SRIOV CI actually tests suspend | 13:24 |
| garyk | johnthetubaguy: there are a number of patches that ytou have blocked with -2 as it was the end of mitaka. any chance you can remove those? | 13:24 |
| bauzas | mriedem: totally agreed, we're missing some coverage here | 13:25 |
| mriedem | this wouldn't be the reason :) http://144.76.193.39/ci-artifacts/262341/7/Nova-ML2-Sriov/logs/n-sch.log.gz | 13:25 |
| *** artom has joined #openstack-nova | 13:25 | |
| mriedem | huh http://intel-openstack-ci-logs.ovh/compute-ci/refs/changes/41/262341/7/compute-nfv-flavors/20160215_232057/screen/n-sch.log.gz | 13:26 |
| mriedem | same in the intel nfv ci | 13:26 |
| *** ivase_ has quit IRC | 13:26 | |
| *** eharney has joined #openstack-nova | 13:27 | |
| *** ivase has joined #openstack-nova | 13:27 | |
| *** links has quit IRC | 13:28 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova: plugins/xenserver: Add '.py' extension https://review.openstack.org/289431 | 13:29 |
| johnthetubaguy | garyk: I can if the blueprints are approved first, which ones are you thinking about, I ended up having to do a few hundred -2s in the end | 13:29 |
| tbachman | johnthetubaguy (et. al.): was wondering if someone would be willing to have a look at this: https://review.openstack.org/#/c/288076/ | 13:31 |
| garyk | johnthetubaguy: ok, so we need an approved bp to be re-approved? | 13:31 |
| johnthetubaguy | garyk: yes, as with the last few releases, a blueprint is only approved for a specific release | 13:32 |
| garyk | johnthetubaguy: the bp was approved at the beginning of mitaka. no one reviewed the code … so it was blocked at the end of m. | 13:32 |
| garyk | ok. understood. | 13:32 |
| *** EinstCrazy has joined #openstack-nova | 13:32 | |
| *** EinstCrazy has quit IRC | 13:33 | |
| *** krtaylor has quit IRC | 13:33 | |
| johnthetubaguy | mriedem: meant to ask, for the specless stuff from last cycle, I guess you are OK with us doing the re-approve if folks want to keep doing that one? | 13:34 |
| johnthetubaguy | (unless it was blocked for some good reason) | 13:34 |
| *** moshele has joined #openstack-nova | 13:34 | |
| *** ccarmack has quit IRC | 13:34 | |
| *** cleong has joined #openstack-nova | 13:36 | |
| mriedem | johnthetubaguy: yeah, i did one for hyper-v last night (to wait for neutron vif plugged events) and one for sahid and libvirt this morning, although as noted the libvirt driver cleanup blueprint probably should have been closed for mitaka and opened a new one for newton | 13:36 |
| mriedem | since it's a multi-series cleanup effort | 13:36 |
| *** claudiub|2 has quit IRC | 13:39 | |
| *** rlrossit has joined #openstack-nova | 13:41 | |
| *** Daisy has joined #openstack-nova | 13:41 | |
| *** toure is now known as toure_ | 13:41 | |
| *** mylu has joined #openstack-nova | 13:41 | |
| dansmith | mriedem: I replied to all your comments | 13:42 |
| dansmith | I think the only valid one from you and alex is the alignment one, but let me know before I rebase that mondo set | 13:42 |
| edleafe | bauzas: I updated the Nova Scheduler meeting page, and added an Agenda page. Can you check out https://review.openstack.org/#/c/296786/ and give your approval if it looks good? | 13:42 |
| bauzas | edleafe: ack | 13:43 |
| *** dimtruck is now known as zz_dimtruck | 13:44 | |
| johnthetubaguy | mriedem: ah, yeah | 13:44 |
| *** krtaylor has joined #openstack-nova | 13:45 | |
| *** al_indigo has joined #openstack-nova | 13:47 | |
| mriedem | ndipanov: moshele: when booting an instance with an sriov port, do we always have a pci request id when allocating the network? | 13:47 |
| openstackgerrit | Sean Dague proposed openstack/nova: Clarify the ``use_neutron`` option upgrade notes. https://review.openstack.org/297129 | 13:48 |
| openstackgerrit | Thomas Maddox proposed openstack/nova: [WIP - DO NOT MERGE] Testing LXC Gate https://review.openstack.org/274792 | 13:48 |
| mriedem | dansmith: ok, might have to wait until after the meeting | 13:48 |
| *** xenogear has joined #openstack-nova | 13:48 | |
| dansmith | mriedem: okay | 13:49 |
| * tbachman wonders what’s the appropriate way is to have a patch in nova considered by cores | 13:49 | |
| *** achanda has joined #openstack-nova | 13:50 | |
| *** achanda has quit IRC | 13:50 | |
| moshele | mriedem: so when booting sriov ports nova will create pci request to each on see https://github.com/openstack/nova/blob/master/nova/network/neutronv2/api.py#L1052 | 13:51 |
| mriedem | moshele: ah ok, that was what i was missing | 13:52 |
| mriedem | tbachman: first, there are like 900 open patches in nova | 13:52 |
| *** daemontool has joined #openstack-nova | 13:52 | |
| tbachman | :( | 13:52 |
| mriedem | tbachman: second, it's the vmware driver and there is supposed to be a subteam that reviews those first before going to the core team | 13:52 |
| *** mylu has quit IRC | 13:52 | |
| mriedem | tbachman: so garyk is your guy | 13:52 |
| tbachman | mriedem: I think garyk gave it a +1, fwiw | 13:52 |
| mriedem | tbachman: not on the latest revision | 13:53 |
| openstackgerrit | Sean Dague proposed openstack/nova: Clarify the ``use_neutron`` option upgrade notes. https://review.openstack.org/297129 | 13:53 |
| tbachman | mriedem: ah — good point | 13:53 |
| openstackgerrit | Dan Smith proposed openstack/nova: Further hack up the nova.tests.unit.db.fakes module of horribleness https://review.openstack.org/296106 | 13:53 |
| openstackgerrit | Dan Smith proposed openstack/nova: Objectify test_flavors and test_flavors_extra_specs https://review.openstack.org/296105 | 13:53 |
| tbachman | mriedem: thx! | 13:53 |
| openstackgerrit | Dan Smith proposed openstack/nova: Make flavor extra_specs operations work against the API DB https://review.openstack.org/294710 | 13:54 |
| openstackgerrit | Dan Smith proposed openstack/nova: Block flavor creation until main database is empty https://review.openstack.org/295310 | 13:54 |
| openstackgerrit | Dan Smith proposed openstack/nova: Add flavor migration routine https://review.openstack.org/294771 | 13:54 |
| openstackgerrit | Dan Smith proposed openstack/nova: Make Flavor access routines work against API database https://review.openstack.org/294313 | 13:54 |
| openstackgerrit | Dan Smith proposed openstack/nova: Make Flavor create() and destroy() work against API DB https://review.openstack.org/294758 | 13:54 |
| * tbachman goes chasing after garyk | 13:54 | |
| * tbachman also deos the math of 900 patches between 14 cores ::( | 13:54 | |
| tbachman | :( | 13:54 |
| tbachman | s/deos/does | 13:54 |
| tbachman | garyk: when you have a moment, can you review the latest: https://review.openstack.org/#/c/288076/ ? | 13:55 |
| tbachman | garyk: thanks! | 13:56 |
| *** ducttape_ has joined #openstack-nova | 13:56 | |
| garyk | tbachman: i have taken a look and its kosher. there is one rule - you bitch about the review process your patch goes to the back of the queue… | 13:56 |
| tbachman | lol | 13:57 |
| tbachman | garyk: definitely not bitching | 13:57 |
| garyk | :) | 13:57 |
| tbachman | just new to the openstack review process | 13:57 |
| *** sbezverk has joined #openstack-nova | 13:57 | |
| mriedem | yeah, i'm not saying tbachman is bitching, | 13:57 |
| tbachman | I’ve worked on OpenDaylight, but that’s certainly a different (and smaller) community | 13:57 |
| *** pushkaru has joined #openstack-nova | 13:57 | |
| mriedem | i realize he's relatively new, so i'm just explaining why it takes awhile | 13:57 |
| *** jaypipes has joined #openstack-nova | 13:57 | |
| tbachman | so, trying to feel my way around here | 13:57 |
| tbachman | mriedem: much appreciated | 13:58 |
| mriedem | johnthetubaguy: has a nice wiki page somewhere explaining this in more polite terms | 13:58 |
| sarafraj | morning | 13:58 |
| diana_clarke | bauzas: (from earlier today) "glance v2 for Nova seems a nice thing to focus on Newton, so I'm not really worried by the fix" | 13:59 |
| mriedem | tbachman: http://docs.openstack.org/developer/nova/how_to_get_involved.html | 13:59 |
| bauzas | diana_clarke: yup? | 13:59 |
| johnthetubaguy | tbachman: got some tips on things here: http://docs.openstack.org/developer/nova/how_to_get_involved.html and http://docs.openstack.org/developer/nova/process.html#how-do-i-get-my-code-merged | 13:59 |
| diana_clarke | bauzas: Bug 1557584 will also happen in galnce v2 | 13:59 |
| openstack | bug 1557584 in OpenStack Compute (nova) "Broken retry mechanism for 'nova image-list'" [Medium,Fix released] https://launchpad.net/bugs/1557584 - Assigned to Diana Clarke (diana-clarke) | 13:59 |
| * tbachman realizes he’s had a #googlefail | 13:59 | |
| tbachman | mriedem: johnthetubaguy thanks! | 13:59 |
| bauzas | diana_clarke: okay, I just wonder why we should carry all the logic in our wrapper anyway | 14:00 |
| *** Daisy has quit IRC | 14:00 | |
| diana_clarke | bauzas: Some of that discussion happened here: https://review.openstack.org/#/c/293127/ | 14:00 |
| *** mylu has joined #openstack-nova | 14:00 | |
| mriedem | tbachman: we also have a vmware subteam section in this review etherpad here https://etherpad.openstack.org/p/newton-nova-priorities-tracking | 14:00 |
| diana_clarke | bauzas: This is just a bug fix for something that's been that way since 2011/2012 | 14:00 |
| * tbachman starts his morning reading | 14:00 | |
| bauzas | diana_clarke: agreed, just saying it's not a regression | 14:01 |
| bauzas | diana_clarke: I haven't fenced your stable change, it should just been approved once stable/mitaka becomes a real stable branch :) | 14:01 |
| *** ducttape_ has quit IRC | 14:01 | |
| *** ctrath has joined #openstack-nova | 14:01 | |
| *** mgoddard_ has joined #openstack-nova | 14:01 | |
| *** lpetrut has quit IRC | 14:02 | |
| dansmith | alex_xu: if you're still around can you check my replies to your comments? | 14:02 |
| alex_xu | dansmith: yea, i'm reading | 14:02 |
| dansmith | thanks | 14:02 |
| diana_clarke | bauzas: And what happens to the cherry pick in the stable/liberty branch? Is it still possible once mitaka is cut? | 14:02 |
| bauzas | diana_clarke: I feel so | 14:03 |
| bauzas | diana_clarke: not super sure tho, there is some discussion about that | 14:03 |
| *** mtanino has quit IRC | 14:03 | |
| *** moshele has quit IRC | 14:04 | |
| diana_clarke | bauzas: And can "normal" core reviewers +2 stable branch reviews, or is it a limited subset? | 14:04 |
| *** mgoddard has quit IRC | 14:04 | |
| dansmith | not a subset | 14:05 |
| dansmith | different set | 14:05 |
| *** EinstCrazy has joined #openstack-nova | 14:05 | |
| dansmith | is this the glance generator thing? | 14:05 |
| bauzas | diana_clarke: no, only nova-stable-maints | 14:05 |
| diana_clarke | dansmith: yup, the tiny bug that keeps on giving ;) | 14:06 |
| dansmith | I haven't followed the discussion, but I think it's probably appropriate for backport | 14:06 |
| scheuran | Hi markus_z, would be great if you could have a look at my live migration prototype :) https://review.openstack.org/#/c/297100/ | 14:07 |
| sdague | johnthetubaguy / bauzas / mriedem - https://review.openstack.org/297129 - brought up by Sam-I-Am as an issue when install guide testing | 14:07 |
| sdague | it just clarifies that the new option replaces 2 instead of one | 14:07 |
| diana_clarke | dansmith: I'm just worried that if the cherry pick in stable/liberty doesn't get merged soon, that stable/liberty will become stable/old_stable and it won't be eligible anymore. | 14:08 |
| *** Daisy has joined #openstack-nova | 14:08 | |
| mriedem | sdague: ok | 14:08 |
| *** pushkaru has quit IRC | 14:08 | |
| *** tojuvone has quit IRC | 14:08 | |
| markus_z | scheuran: sure, will do | 14:08 |
| dansmith | diana_clarke: yeah, well, that's how it goes, but I don't think that's going to happen super quick | 14:08 |
| *** pushkaru has joined #openstack-nova | 14:08 | |
| *** tojuvone has joined #openstack-nova | 14:08 | |
| diana_clarke | dansmith: Yeah, I'll wait an see. There was some conversation about it earlier, but it was 5am my time so I missed it. | 14:09 |
| *** ducttape_ has joined #openstack-nova | 14:09 | |
| scheuran | markus_z, please don't spend too much time on optimizing the code - there are many findings - review focus should be more about the concept | 14:10 |
| scheuran | markus_z, thx | 14:10 |
| markus_z | scheuran: ok. the format got screwed up btw | 14:10 |
| markus_z | scheuran: It's not possible to find the real diff | 14:11 |
| markus_z | scheuran: could you push a new one? | 14:11 |
| *** woodster_ has joined #openstack-nova | 14:11 | |
| scheuran | markus_z, oh ok | 14:11 |
| *** haplo37 has joined #openstack-nova | 14:12 | |
| scheuran | markus_z, oh, probably this was my new pycharm setup :/ | 14:12 |
| *** zz_dimtruck is now known as dimtruck | 14:14 | |
| * tbachman chuckles on “Why are the reviewers being mean to me?” section of link | 14:15 | |
| *** lpetrut has joined #openstack-nova | 14:15 | |
| *** claudiub|2 has joined #openstack-nova | 14:15 | |
| alex_xu | dansmith: np | 14:16 |
| alex_xu | dansmith: so we can't upgrade any commit between 'create against api db' patch and 'block creation' patch? | 14:17 |
| dansmith | alex_xu: you can, you'd just have to be careful. I would rather land those all at the same time with a small window of "be careful" than to block creation early with no way to get out of it | 14:17 |
| *** EinstCrazy has quit IRC | 14:18 | |
| *** EinstCrazy has joined #openstack-nova | 14:18 | |
| * tbachman realizes he’s already violated one of the principles: “Please don’t ask for core review straight away, especially not directly (IRC or email)” | 14:19 | |
| tbachman | oops :( | 14:19 |
| tbachman | mriedem: johnthetubaguy: sorry :( | 14:20 |
| mriedem | tbachman: it's fine | 14:20 |
| johnthetubaguy | tbachman: hey no worries, you are reaching out and willing to learn, that beats all of those | 14:20 |
| *** ekuris_ has quit IRC | 14:21 | |
| alex_xu | dansmith: ok, got it, thanks. And I didn't realize the id and flavor id problem also, i will try to revisit the patch tomorrow, let me remove -1 first. | 14:21 |
| dansmith | alex_xu: yeah, the flavor.id and flavorid problem is *ridiculous* :( | 14:21 |
| tbachman | johnthetubaguy: mriedem: russellb’s link on average times is helpful — gives perspective on how long it takes cores to work through (realisticaally) on the backlog | 14:22 |
| tbachman | (that was linked on the link you sent me, so thx) | 14:22 |
| *** mgoddard_ has quit IRC | 14:23 | |
| *** mgoddard has joined #openstack-nova | 14:23 | |
| alex_xu | dansmith: heh | 14:23 |
| *** knikolla has joined #openstack-nova | 14:23 | |
| *** aginwala has joined #openstack-nova | 14:24 | |
| *** sridharg has quit IRC | 14:24 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova: nova-manage: Use consistent prints for errors https://review.openstack.org/289309 | 14:24 |
| *** gszasz has quit IRC | 14:25 | |
| openstackgerrit | Artom Lifshitz proposed openstack/nova: Add tag column to vifs and bdm https://review.openstack.org/264014 | 14:28 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: plugins/xenserver: Add '.py' extension https://review.openstack.org/289431 | 14:28 |
| openstackgerrit | Radoslav Gerganov proposed openstack/nova: The 'record' option of the WebSocketProxy should be string https://review.openstack.org/294512 | 14:29 |
| *** aginwala has quit IRC | 14:29 | |
| openstackgerrit | Artom Lifshitz proposed openstack/nova: Add metadata objects for device tagging https://review.openstack.org/264015 | 14:30 |
| openstackgerrit | Andreas Scheuring proposed openstack/nova: [WIP] libvirt: Update interface xml definition on migration https://review.openstack.org/297184 | 14:30 |
| *** cfriesen__ has joined #openstack-nova | 14:31 | |
| openstackgerrit | Andreas Scheuring proposed openstack/nova: [WIP] libvirt: Update interface xml definition on migration https://review.openstack.org/297100 | 14:32 |
| *** vishwanathj_zzz is now known as vishwanathj | 14:33 | |
| tpeoples | bauzas: see https://bugs.launchpad.net/nova/+bug/1561357 for a detailed description of the issue i brought up last night | 14:33 |
| openstack | Launchpad bug 1561357 in OpenStack Compute (nova) "VM deployed with availability-zone (force_hosts) cannot be live migrated to an untargeted host" [Undecided,New] | 14:33 |
| *** ametts has joined #openstack-nova | 14:33 | |
| bauzas | tpeoples: we discussed on that this morning and we'll continue right after the nova meeting | 14:34 |
| scheuran | markus_z, done https://review.openstack.org/#/c/297100/ | 14:34 |
| *** Daisy has quit IRC | 14:34 | |
| tpeoples | bauzas: ah okay, just got on. sounds good, thanks | 14:34 |
| sbezverk | Good morning, any plan to backport this patch to liberty? https://review.openstack.org/#/c/265163/ | 14:35 |
| *** Daisy has joined #openstack-nova | 14:35 | |
| *** aginwala has joined #openstack-nova | 14:35 | |
| *** jaosorior has quit IRC | 14:37 | |
| openstackgerrit | Artom Lifshitz proposed openstack/nova: Libvirt driver implementation of device tagging https://review.openstack.org/264016 | 14:37 |
| *** Daisy has quit IRC | 14:39 | |
| *** klkumar has quit IRC | 14:40 | |
| armax | mriedem: looking | 14:40 |
| *** burgerk has quit IRC | 14:42 | |
| *** _ducttape_ has joined #openstack-nova | 14:44 | |
| cdent | Is there a tentative vague date for newton mid-cycle? | 14:46 |
| *** ducttape_ has quit IRC | 14:47 | |
| * johnthetubaguy hopes its in Q3... | 14:47 | |
| cdent | ditto | 14:47 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova: Add compute manager device tagging support https://review.openstack.org/264017 | 14:47 |
| mriedem | sbezverk: you should probably ask in the #openstack-oslo channel | 14:48 |
| mriedem | cdent: no idea, it's usually around the 3rd milestone | 14:49 |
| mriedem | brb | 14:49 |
| *** mdrabe has quit IRC | 14:49 | |
| sfinucan | I want to integrate cliff [1] into nova-manage (and maybe more). Does that need a spec? [1] http://docs.openstack.org/developer/cliff/ | 14:49 |
| markus_z | scheuran: much less changes :) | 14:49 |
| sfinucan | it's an alternative to this change which seems error prone, IMO https://review.openstack.org/#/c/289309/ | 14:49 |
| sfinucan | *'ERROR: ' prone, heh | 14:50 |
| *** zehua1 has joined #openstack-nova | 14:53 | |
| *** cdent has quit IRC | 14:53 | |
| *** zehua has quit IRC | 14:54 | |
| johnthetubaguy | sfinucan: yeah, that feels like a specless bp I guess | 14:55 |
| *** zehua1 has left #openstack-nova | 14:55 | |
| openstackgerrit | Artom Lifshitz proposed openstack/nova: Device tagging user visible changes https://review.openstack.org/264018 | 14:56 |
| *** haomaiwang has joined #openstack-nova | 14:56 | |
| *** klkumar has joined #openstack-nova | 14:56 | |
| *** zehua has joined #openstack-nova | 14:56 | |
| zehua | HI | 14:57 |
| *** yamahata has joined #openstack-nova | 14:58 | |
| *** harlowja_at_home has joined #openstack-nova | 14:58 | |
| *** haomaiwang has quit IRC | 15:01 | |
| *** haomaiwang has joined #openstack-nova | 15:01 | |
| *** yamahata has quit IRC | 15:02 | |
| *** links has joined #openstack-nova | 15:02 | |
| *** yamahata has joined #openstack-nova | 15:03 | |
| jaypipes | mriedem: sorry, I needed to step away from the Nova IRC meeting. cdent informs me that no new specs will be approved until after the summit. is that correct? :( | 15:04 |
| mriedem | jaypipes: yes | 15:05 |
| jaypipes | mriedem: can I ask why? | 15:05 |
| *** mgoddard_ has joined #openstack-nova | 15:05 | |
| mriedem | jaypipes: as i told cdent, imagine a large clogged toilet | 15:05 |
| mriedem | jaypipes: we're only re-approving previously approved specs/bp's until the summit | 15:06 |
| jaypipes | mriedem: so let the shit sit in the bowl for a month? | 15:06 |
| *** EinstCrazy has quit IRC | 15:06 | |
| mriedem | jaypipes: no, work on plunging through the shit that's in there so it flushes through | 15:06 |
| mriedem | i.e. flush what we have before piling on more shit | 15:06 |
| * mriedem likes this technical discussion | 15:06 | |
| mriedem | jaypipes: we already have at least 20 previously approved specs in newton | 15:07 |
| mriedem | we have plenty of stuff that is carried over from mitaka that needs to get done before we start taking on new things | 15:07 |
| mriedem | which includes resource providers stuff, os-vif, cells v2 and someothers | 15:07 |
| mriedem | i'm going to post something to -dev list shortly | 15:07 |
| *** aloga_ has joined #openstack-nova | 15:08 | |
| *** ssurana has joined #openstack-nova | 15:08 | |
| *** ababich has joined #openstack-nova | 15:08 | |
| jaypipes | mriedem: resource providers wasn't approved in mitaka, unfortunately. | 15:08 |
| *** mdrabe has joined #openstack-nova | 15:08 | |
| *** mgoddard has quit IRC | 15:09 | |
| jaypipes | mriedem: which means waiting another month or more to discuss it, which is kinda frustrating considering a good chunk of the code for it is already up. | 15:09 |
| mriedem | we just re-approved resource-providers yesterday | 15:09 |
| *** klkumar has quit IRC | 15:09 | |
| jaypipes | mriedem: compute-node-inventory, resource-providers-allocations, and generic-resource-pools are the primary ones. resource-providers blueprint was pretty much already completed in Mitaka. | 15:10 |
| *** mylu has quit IRC | 15:11 | |
| jaypipes | mriedem: mostly this is my fault for not having had these blueprints in a good order. | 15:11 |
| sbezverk | mriedem thank you | 15:11 |
| jaypipes | mriedem: I'm just wary of missing even more time on this particular effort. | 15:11 |
| *** lucasagomes_ has joined #openstack-nova | 15:11 | |
| *** Shalmazar has joined #openstack-nova | 15:11 | |
| dansmith | jaypipes: this approach is to try to avoid things we know we need to work on waiting until milestone 3 again | 15:11 |
| *** adreznec_ has joined #openstack-nova | 15:12 | |
| dansmith | jaypipes: instead of approving a bunch of new shit, finishing the shit we previously approved (or thought we approved because there are like eight things) | 15:12 |
| sfinucan | johnthetubaguy: Done. Do I need to get approval for specless BPs, or do I just start work? :) https://blueprints.launchpad.net/nova/+spec/cliff-integration | 15:12 |
| openstackgerrit | John Garbutt proposed openstack/nova: Add prototype feature classification matrix https://review.openstack.org/264719 | 15:12 |
| jaypipes | dansmith: right, but this means that -2s are going to be stuck on all these resource-provider related patches for 5+ more weeks. | 15:12 |
| *** inara` has joined #openstack-nova | 15:12 | |
| mriedem | jaypipes: we need to get the things done that were in progress in mitaka | 15:12 |
| mriedem | people can keep proposing specs, | 15:12 |
| *** aginwala has quit IRC | 15:13 | |
| johnthetubaguy | sfinucan: you do need approval, the downside is we just blocked approvals till post summit, I think | 15:13 |
| *** rlrossit has quit IRC | 15:13 | |
| *** dave-mccowan has quit IRC | 15:13 | |
| *** edmondsw has quit IRC | 15:13 | |
| *** baoli has quit IRC | 15:13 | |
| *** lifeless has quit IRC | 15:13 | |
| *** inara has quit IRC | 15:13 | |
| *** david-lyle has quit IRC | 15:13 | |
| *** fmccrthy has quit IRC | 15:13 | |
| *** jdurgin has quit IRC | 15:13 | |
| *** zzzeek has quit IRC | 15:13 | |
| *** purplerbot has quit IRC | 15:13 | |
| *** lucasagomes has quit IRC | 15:13 | |
| *** adreznec has quit IRC | 15:13 | |
| *** clduser has quit IRC | 15:13 | |
| *** aloga has quit IRC | 15:13 | |
| *** clayton has quit IRC | 15:13 | |
| *** Madasi has quit IRC | 15:13 | |
| *** jschlueter has quit IRC | 15:13 | |
| *** diana_clarke has quit IRC | 15:13 | |
| *** Shalmazar is now known as Madasi | 15:13 | |
| mriedem | but we need to focus review effort on what's been re-approved already | 15:13 |
| *** mylu has joined #openstack-nova | 15:13 | |
| *** mriedem has left #openstack-nova | 15:13 | |
| dansmith | jaypipes: the things we were working on in mitaka and failed to get in should be freed up | 15:13 |
| *** zzzeek has joined #openstack-nova | 15:13 | |
| *** mriedem has joined #openstack-nova | 15:13 | |
| dansmith | I've lost track of all the blueprints and what was approved and not | 15:13 |
| dansmith | but things like the inventory stuff which we were trying to get into mitaka should be good to go now, no? | 15:14 |
| dansmith | seems there's quite a bit of that to do | 15:14 |
| *** zehua has quit IRC | 15:14 | |
| jaypipes | dansmith: pretty much nothing was approved other than the one initial spec for adding the resource-providers database schema and that's already been done. | 15:14 |
| johnthetubaguy | dansmith: right, that stuff we should unblock I think | 15:14 |
| * johnthetubaguy is confused | 15:14 | |
| dansmith | jaypipes: we were trying to land all the inventory stuff and didn't | 15:14 |
| dansmith | jaypipes: so that stuff should be good to go now | 15:14 |
| jaypipes | dansmith: compute-node-inventory was not approved. resource-providers-allocations, not approved. generic-resource-pools, not approved. | 15:14 |
| * dansmith is really frustrated | 15:15 | |
| jaypipes | it's been stuck in endless spec reviews. | 15:15 |
| dansmith | jaypipes: I get that things are broken out into a hundred pieces, and that you know which are approved and I don't | 15:15 |
| dansmith | however | 15:15 |
| *** cdent has joined #openstack-nova | 15:15 | |
| jaypipes | dansmith: Dan, I was asked to split the big spec into multiple pieces. | 15:15 |
| dansmith | I think we can be practical here and continue to try to land the things that may not have been technically approved, but which we were *planning* to land in mitaka | 15:15 |
| *** mgoddard_ has quit IRC | 15:15 | |
| *** mgoddard has joined #openstack-nova | 15:16 | |
| dansmith | mriedem: the spirit of the approach is to flush the queue of things we were working on until we cut off for rc1 yes? | 15:16 |
| mriedem | dansmith: yes | 15:16 |
| cdent | p!hist | 15:17 |
| *** Guest27982 has quit IRC | 15:17 | |
| *** rcernin has quit IRC | 15:17 | |
| jaypipes | rc1> | 15:17 |
| jaypipes | ? | 15:17 |
| *** david-lyle has joined #openstack-nova | 15:17 | |
| openstackgerrit | John Garbutt proposed openstack/nova: Get support matrix notes from the ini file https://review.openstack.org/264763 | 15:18 |
| dansmith | yeah, which would include all the inventory stuff, IMHO | 15:18 |
| *** links has quit IRC | 15:18 | |
| jaypipes | dansmith: you mean milestone 1? | 15:18 |
| dansmith | up to but not including the allocation stuff | 15:18 |
| dansmith | jaypipes: no | 15:18 |
| openstackgerrit | John Garbutt proposed openstack/nova: Add prototype feature classification matrix https://review.openstack.org/264719 | 15:18 |
| dansmith | I mean rc1 | 15:18 |
| dansmith | the things we were trying to land in mitaka | 15:18 |
| jaypipes | I thought we were aiming to get the scheduler REST API (generic-resource-pools) split out in Newton? | 15:19 |
| openstackgerrit | John Garbutt proposed openstack/nova: Get support matrix notes from the ini file https://review.openstack.org/264763 | 15:19 |
| dansmith | jaypipes: I don't even know, I haven't looked that far.. that's newton stuff, I'm talking about things we wanted in mitaka and didn't get | 15:19 |
| *** adreznec_ is now known as adreznec | 15:19 | |
| jaypipes | dansmith: oh, shit, sorry, you're talking about *Mitaka* RC1. | 15:19 |
| dansmith | yes | 15:19 |
| bauzas | jaypipes: I think we can continue discussing on the series in the spec and in the implementation patches so that we reach consensus at a certain point in time and just leave the approvals be there for summit time | 15:19 |
| *** armax has quit IRC | 15:19 | |
| jaypipes | ffs, I thought you were talking about Newton RC1. | 15:19 |
| jaypipes | whcih is why I was freaking out. | 15:20 |
| dansmith | jaypipes: in five minutes we're going to talk about the Z release milestone 2, fasten your seatbelt | 15:20 |
| dansmith | no man | 15:20 |
| bauzas | jaypipes: that doesn't really block progress, right? | 15:20 |
| *** purplerbot has joined #openstack-nova | 15:20 | |
| dansmith | we're trying to get the runway open for things we wanted in mitaka but didn't get | 15:20 |
| *** armax has joined #openstack-nova | 15:20 | |
| dansmith | instead of waiting for summit, defining priorities, etc | 15:20 |
| dansmith | assuming those have an obvious pass to land early | 15:20 |
| jaypipes | bauzas: it leaves a whole bunch of -2s blocking a whole series of patches. | 15:20 |
| dansmith | before we start approving specs for *new* things | 15:20 |
| bauzas | jaypipes: well, I see that as a feature branch | 15:21 |
| bauzas | jaypipes: with a -2 at the bottom of the branch until we approve it | 15:21 |
| jaypipes | dansmith: I understand that. I'd just like some clarity on the three specific specs I listed above, none of which were approved for Mitaka. | 15:21 |
| dansmith | unfortunately, I don't think the specs matched the reality of how we had to land things, and the latter is all I really care about | 15:22 |
| dansmith | so, IMHO, everything up to but not including the allocation stuff should be fair game right now, | 15:22 |
| dansmith | regardless of the position of the specs | 15:22 |
| dansmith | I'll let mriedem make the calls on that, but that's my advice | 15:22 |
| *** liverpooler has quit IRC | 15:22 | |
| mriedem | i honestly don't have a grasp on what's landed, what was in progress, what was close, etc for that effort in mitaka | 15:23 |
| mriedem | i know there is one change for a re-approved spec yesterday | 15:23 |
| mriedem | which is dependent on 2 technically not-approved bp things, | 15:23 |
| mriedem | but as dansmith said, it seems the specs and code changes aren't in the same order | 15:24 |
| mriedem | and in this case i think we have to land dependencies to get to the in-progress mitaka-approved thing | 15:24 |
| dansmith | they can't be, which isn't necessarily a problem | 15:24 |
| dansmith | the specs are in like theoretical order | 15:24 |
| dansmith | or modeling order | 15:24 |
| dansmith | but in reality, we have to land things in a specific order | 15:24 |
| mriedem | right, i'm just using that as an example | 15:24 |
| dansmith | yeah | 15:24 |
| mriedem | this https://review.openstack.org/#/c/282442/ | 15:25 |
| *** jschlueter has joined #openstack-nova | 15:25 | |
| *** eharney has quit IRC | 15:25 | |
| *** fmccrthy has joined #openstack-nova | 15:25 | |
| *** baoli has joined #openstack-nova | 15:25 | |
| mriedem | looking at that today, it's been re-ordered | 15:25 |
| dansmith | that has gotten basically no review at this point, AFAIK | 15:25 |
| *** yamahata has quit IRC | 15:26 | |
| dansmith | I split it out of a larger patch specifically to save it for newton | 15:26 |
| cdent | mriedem: I moved allocation to the start of that stack this morning | 15:26 |
| *** coreywright has quit IRC | 15:26 | |
| mriedem | cdent: ok | 15:26 |
| cdent | because of the different specs things are attached to | 15:26 |
| cdent | (based on the version of this conversation that was had I think yesterday) | 15:26 |
| jaypipes | dansmith: I reviewed that patch yesterday. | 15:26 |
| mriedem | so let me send this info out to the dev list | 15:26 |
| dansmith | it's this stuff that I think gets the pass: https://review.openstack.org/#/c/279313/ | 15:26 |
| dansmith | jaypipes: review in mitaka | 15:26 |
| *** zehua has joined #openstack-nova | 15:26 | |
| openstackgerrit | Sergey Belous proposed openstack/nova: Do not merge. Patch exists for testing os-vif with Nova https://review.openstack.org/286086 | 15:28 |
| mriedem | johnthetubaguy: please drop the -2 from https://review.openstack.org/#/c/282442/ | 15:28 |
| jaypipes | lemme open this in Gertty so I don't want to stab someone with a rusty spork. | 15:28 |
| *** wasmum has quit IRC | 15:29 | |
| *** rlrossit has joined #openstack-nova | 15:29 | |
| *** dave-mccowan has joined #openstack-nova | 15:29 | |
| *** edmondsw has joined #openstack-nova | 15:29 | |
| *** lifeless has joined #openstack-nova | 15:29 | |
| *** jdurgin has joined #openstack-nova | 15:29 | |
| *** clduser has joined #openstack-nova | 15:29 | |
| *** clayton has joined #openstack-nova | 15:29 | |
| *** diana_clarke has joined #openstack-nova | 15:29 | |
| *** zehua has quit IRC | 15:29 | |
| *** Guest27982 has joined #openstack-nova | 15:30 | |
| *** rcernin has joined #openstack-nova | 15:30 | |
| *** lifeless has quit IRC | 15:30 | |
| *** e0ne has quit IRC | 15:30 | |
| *** lifeless has joined #openstack-nova | 15:30 | |
| jaypipes | dansmith: so your compute node inventory is dependent on this patch from me: https://review.openstack.org/#/c/286931/ | 15:30 |
| *** e0ne has joined #openstack-nova | 15:30 | |
| dansmith | jaypipes: correct | 15:31 |
| jaypipes | dansmith: which has had a -2 from johnthetubaguy for weeks and no reviews. | 15:31 |
| dansmith | which should be un-2'd at this point | 15:31 |
| dansmith | jaypipes: he -2d it the day of the rc | 15:31 |
| dansmith | per the plan, and per the discussion | 15:31 |
| *** andrewbogott has quit IRC | 15:32 | |
| jaypipes | dansmith: I'm not arguing that. I'm arguing that if we stuck with the plan of not working on anything that didn't get approved in mitaka, this particular patch would remain -2'd until after the summit. And that's what I'm frustrated about. | 15:32 |
| *** browne has joined #openstack-nova | 15:32 | |
| dansmith | no dude | 15:32 |
| jaypipes | because, again, the spec hasn't been approved: https://review.openstack.org/260048 | 15:32 |
| dansmith | okay, I think I'm done :) | 15:32 |
| jaypipes | dansmith: Dan, please work with me here. | 15:33 |
| jaypipes | I am trying to understand what is and what isn't blocked until the summit. | 15:33 |
| *** andrewbogott has joined #openstack-nova | 15:33 | |
| *** xenogear has quit IRC | 15:34 | |
| mriedem | i see jaypipes' point, but wasn't involved in the mitaka effort so have to defer to dansmith on exceptions in this case | 15:34 |
| dansmith | mriedem: damn you | 15:34 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: move graphic/serial consoles check at pre_live_migration https://review.openstack.org/254709 | 15:35 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: live_migration_flags/block_migration_flags default to 0 https://review.openstack.org/295933 | 15:35 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: add "get_job_info" to Guest's object https://review.openstack.org/250740 | 15:35 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: add method to configure max downtime when migrating https://review.openstack.org/250739 | 15:35 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: add "abort_job" to Guest's object https://review.openstack.org/250738 | 15:35 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: add method "migrate" to Guest's object https://review.openstack.org/250365 | 15:35 |
| *** lucasagomes_ is now known as lucasagomes | 15:36 | |
| *** yamahata has joined #openstack-nova | 15:37 | |
| *** andrewbogott has quit IRC | 15:37 | |
| *** andrewbogott has joined #openstack-nova | 15:37 | |
| bauzas | mriedem: dansmith: johnthetubaguy: tpeoples: resuming our convo about https://bugs.launchpad.net/bugs/1561357 | 15:38 |
| openstack | Launchpad bug 1561357 in OpenStack Compute (nova) "VM deployed with availability-zone (force_hosts) cannot be live migrated to an untargeted host" [Undecided,New] | 15:38 |
| bauzas | if you damn have time :-) | 15:38 |
| johnthetubaguy | mriedem: dropped that -2 | 15:39 |
| mriedem | johnthetubaguy: saw that, thanks | 15:40 |
| mriedem | bauzas: sending this ML thread atm | 15:40 |
| openstackgerrit | Radoslav Gerganov proposed openstack/nova: The 'record' option of the WebSocketProxy should be string https://review.openstack.org/294512 | 15:40 |
| bauzas | mriedem: take your time, we can discuss later | 15:40 |
| bauzas | see, I can understand priorities ? :p | 15:41 |
| *** eharney has joined #openstack-nova | 15:42 | |
| dansmith | jaypipes: mriedem: so if you want to continue I think we should get on a hangout where I can make lewd hand gestures. at the end of the day, I'm just trying to advise this process based on what has been reviewed more heavily, what I wanted us to land in mitaka vs. wait for newton, etc | 15:43 |
| dansmith | I don't really want to get yelled at | 15:43 |
| dansmith | because really all I've tried to do for this set is get it landed as much as possible in the short time window we had | 15:43 |
| jaypipes | dansmith: I'm not yelling at you. | 15:43 |
| dansmith | jaypipes: you are in my head | 15:43 |
| *** lpetrut has quit IRC | 15:44 | |
| jaypipes | dansmith: and I appreciate everything you have done in this series so far, I do. My personal frustration is on the blueprint side. I just want to see these darn specs land. | 15:44 |
| mriedem | dansmith: NO ONE IS YELLING AT YOU | 15:44 |
| dansmith | also, I'm out tomorrow | 15:45 |
| *** mgoddard_ has joined #openstack-nova | 15:45 | |
| dansmith | and I'm going to be digging in the mud and swearing at all you people while I do it | 15:45 |
| *** coreywright has joined #openstack-nova | 15:45 | |
| johnthetubaguy | actually good point, I am out tomorrow and monday, due to holidays in the UK | 15:46 |
| mriedem | i'll be clearing my driveway of snow after i get this email sent | 15:46 |
| mriedem | easter?! | 15:46 |
| mriedem | people get days off for that? | 15:46 |
| johnthetubaguy | yeah | 15:46 |
| *** Swami has joined #openstack-nova | 15:46 | |
| johnthetubaguy | https://www.gov.uk/bank-holidays | 15:46 |
| mriedem | man | 15:46 |
| mriedem | ha | 15:46 |
| mriedem | i might need a day off from the belly full of jelly beans i'm going to consume | 15:47 |
| *** doug-fish has joined #openstack-nova | 15:47 | |
| *** wasmum has joined #openstack-nova | 15:48 | |
| *** mgoddard has quit IRC | 15:49 | |
| cdent | dansmith, jaypipes, johnthetubaguy : since you're all warmed up on this resource-* can you guys provide some input on the questions and comments on patchset 12 on this: https://review.openstack.org/#/c/284895/ It's about where to put the DB methods. I essentially agree with what jay suggests, but didn't think that patch was the place to change the pattern. | 15:49 |
| markus_z | same holidays in Germany, so I'm back on Tuesday | 15:49 |
| bauzas | it's a cabal of european countries for making French people sad, I guess | 15:50 |
| *** angdraug has joined #openstack-nova | 15:51 | |
| johnthetubaguy | cdent: I honestly thought that was just for the api_database stuff at this point, or maybe new objects, seems odd to do half and half in an existing object | 15:51 |
| jaypipes | cdent: ah, yeah, I need to re-post a review on that. basically, I'm good with putting that in the nova.db.api module to keep it consistent with what's already there if doffm plans to overhaul it anyway. | 15:51 |
| cdent | right on, thanks guys | 15:52 |
| johnthetubaguy | did we deprecate the DB driver now, I think... | 15:52 |
| * cdent just wants to gets some of this stuff merged | 15:52 | |
| *** libu has joined #openstack-nova | 15:52 | |
| johnthetubaguy | that should help tidy that stuff up soon | 15:52 |
| *** jdurgin1 has quit IRC | 15:52 | |
| *** belmoreira has quit IRC | 15:53 | |
| johnthetubaguy | mriedem: so we had this approved for mitaka, shall I create one for newton? https://blueprints.launchpad.net/nova/+spec/compute-node-inventory, basically to unblock this series that we were working on: https://review.openstack.org/#/c/279313/ | 15:55 |
| openstackgerrit | Ed Leafe proposed openstack/nova: Move config options from nova/api directory https://review.openstack.org/258180 | 15:55 |
| openstackgerrit | Ed Leafe proposed openstack/nova: Improve the help text for the API options (2) https://review.openstack.org/260587 | 15:55 |
| openstackgerrit | Ed Leafe proposed openstack/nova: Improve the help text for the API options (1) https://review.openstack.org/260586 | 15:55 |
| openstackgerrit | Ed Leafe proposed openstack/nova: Improve the help text for the API options (3) https://review.openstack.org/260588 | 15:55 |
| openstackgerrit | Ed Leafe proposed openstack/nova: Improve the help text for the API options (4) https://review.openstack.org/297237 | 15:55 |
| cdent | johnthetubaguy: huh, I see my name all over a lot of that blueprint page but very little of the code is from me. Perhaps that's part of my confusion... | 16:00 |
| mriedem | johnthetubaguy: yeah maybe, that's that weird case where the bp was approved but the spec wasn't | 16:00 |
| *** haomaiwang has quit IRC | 16:01 | |
| johnthetubaguy | mriedem: yeah, I think we did that in the end to get it moving | 16:01 |
| mriedem | johnthetubaguy: i'm fine with that then, and i think it helps with jaypipes's worries about that one, and keeps dansmith's changes moving | 16:01 |
| *** haomaiwang has joined #openstack-nova | 16:01 | |
| mriedem | at least thats my (limited) understanding | 16:01 |
| johnthetubaguy | mriedem: yeah, the stuff we all seem to agree on, just keep it moving, basically | 16:02 |
| *** Kennan has quit IRC | 16:04 | |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Make quotas commands more user-friendly https://review.openstack.org/290556 | 16:04 |
| *** Kennan has joined #openstack-nova | 16:05 | |
| *** lpetrut has joined #openstack-nova | 16:08 | |
| *** apoorvad has joined #openstack-nova | 16:09 | |
| mriedem | johnthetubaguy: dansmith: so on clearing out https://etherpad.openstack.org/p/newton-nova-priorities-tracking - thoughts on leaving the sub-team headers in there and just wiping the contents, or just drop everything below the top headers? | 16:11 |
| mriedem | i figured i'd leave the section headers but clear the section contents | 16:11 |
| *** e0ne has quit IRC | 16:14 | |
| *** mgoddard_ has quit IRC | 16:14 | |
| *** mgoddard has joined #openstack-nova | 16:14 | |
| *** mvk_ has quit IRC | 16:15 | |
| *** drankis has quit IRC | 16:16 | |
| *** ljxiash has quit IRC | 16:16 | |
| *** nic has joined #openstack-nova | 16:17 | |
| *** mgoddard_ has joined #openstack-nova | 16:18 | |
| *** zenoway has quit IRC | 16:20 | |
| *** zenoway has joined #openstack-nova | 16:21 | |
| mriedem | done https://etherpad.openstack.org/p/newton-nova-priorities-tracking | 16:21 |
| *** mgoddard has quit IRC | 16:21 | |
| *** Marga_ has joined #openstack-nova | 16:21 | |
| *** haomaiwang has quit IRC | 16:22 | |
| *** mriedem is now known as mriedem_snow | 16:22 | |
| *** Marga_ has quit IRC | 16:22 | |
| *** haomaiwang has joined #openstack-nova | 16:22 | |
| *** haomaiwang has quit IRC | 16:23 | |
| *** Marga_ has joined #openstack-nova | 16:23 | |
| *** haomaiwa_ has joined #openstack-nova | 16:23 | |
| *** haomaiwa_ has quit IRC | 16:24 | |
| *** rk4n has quit IRC | 16:25 | |
| *** zenoway has quit IRC | 16:25 | |
| *** haomaiwang has joined #openstack-nova | 16:25 | |
| *** haomaiwang has quit IRC | 16:26 | |
| *** haomaiwa_ has joined #openstack-nova | 16:26 | |
| *** haomaiwa_ has quit IRC | 16:27 | |
| *** haomaiwang has joined #openstack-nova | 16:27 | |
| *** haomaiwang has quit IRC | 16:28 | |
| *** haomaiwa_ has joined #openstack-nova | 16:28 | |
| *** haomaiwa_ has quit IRC | 16:29 | |
| *** haomaiwa_ has joined #openstack-nova | 16:29 | |
| *** haomaiwa_ has quit IRC | 16:30 | |
| *** zzzeek has quit IRC | 16:30 | |
| *** 7F1AAK9L3 has joined #openstack-nova | 16:30 | |
| *** 7F1AAK9L3 has quit IRC | 16:31 | |
| *** drankis has joined #openstack-nova | 16:31 | |
| *** haomaiwa_ has joined #openstack-nova | 16:31 | |
| *** haomaiwa_ has quit IRC | 16:32 | |
| *** pushkaru has quit IRC | 16:32 | |
| *** haomaiwang has joined #openstack-nova | 16:32 | |
| johnthetubaguy | mriedem_snow: that works | 16:32 |
| *** haomaiwang has quit IRC | 16:33 | |
| *** andrearosa has quit IRC | 16:33 | |
| *** zzzeek has joined #openstack-nova | 16:33 | |
| *** pwnall has joined #openstack-nova | 16:33 | |
| *** haomaiwa_ has joined #openstack-nova | 16:33 | |
| *** haomaiwa_ has quit IRC | 16:34 | |
| *** haomaiwang has joined #openstack-nova | 16:34 | |
| *** Bjoern has joined #openstack-nova | 16:34 | |
| *** haomaiwang has quit IRC | 16:35 | |
| *** haomaiwang has joined #openstack-nova | 16:35 | |
| *** haomaiwang has quit IRC | 16:36 | |
| *** haomaiwang has joined #openstack-nova | 16:36 | |
| *** haomaiwang has quit IRC | 16:37 | |
| sahid | johnthetubaguy: some patches here still have your -2, can you removed them when you have a moment https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/libvirt-clean-driver | 16:37 |
| *** 14WAAKC1C has joined #openstack-nova | 16:37 | |
| *** 14WAAKC1C has quit IRC | 16:38 | |
| *** haomaiwa_ has joined #openstack-nova | 16:38 | |
| openstackgerrit | John Garbutt proposed openstack/nova: Migrate compute node resource information to Inventory objects https://review.openstack.org/279313 | 16:38 |
| *** haomaiwa_ has quit IRC | 16:39 | |
| *** haomaiwang has joined #openstack-nova | 16:39 | |
| openstackgerrit | John Garbutt proposed openstack/nova: Make compute_node_statistics() use new schema https://review.openstack.org/286931 | 16:39 |
| *** haomaiwang has quit IRC | 16:40 | |
| openstackgerrit | John Garbutt proposed openstack/nova: Add Service.get_minimum_version_multi() for multiple binaries https://review.openstack.org/279312 | 16:40 |
| *** haomaiwang has joined #openstack-nova | 16:40 | |
| *** haomaiwang has quit IRC | 16:41 | |
| *** haomaiwa_ has joined #openstack-nova | 16:41 | |
| *** haomaiwa_ has quit IRC | 16:42 | |
| *** haomaiwa_ has joined #openstack-nova | 16:42 | |
| *** oomichi has quit IRC | 16:42 | |
| *** haomaiwa_ has quit IRC | 16:43 | |
| *** lpetrut has quit IRC | 16:43 | |
| *** haomaiwa_ has joined #openstack-nova | 16:43 | |
| *** haomaiwa_ has quit IRC | 16:43 | |
| bauzas | mriedem_snow: http://fr.snow-forecast.com/resorts/Les7Laux/6day/mid if you need this | 16:44 |
| *** unicell has quit IRC | 16:44 | |
| bauzas | mriedem_snow: dansmith: johnthetubaguy: tpeoples: so after thinking a bit more of https://bugs.launchpad.net/nova/+bug/1561357 I think it's problematic for unshelve | 16:46 |
| openstack | Launchpad bug 1561357 in OpenStack Compute (nova) "VM deployed with availability-zone (force_hosts) cannot be live migrated to an untargeted host" [High,Confirmed] - Assigned to Sylvain Bauza (sylvain-bauza) | 16:46 |
| *** gokrokve has joined #openstack-nova | 16:46 | |
| bauzas | but not for live-mig and evacuate because we can bypass the scheduler if providing a destination when migrating | 16:46 |
| bauzas | hence me triaging to High and putting myself on it | 16:47 |
| bauzas | the problematic situation being : you force an instance, you shelve it, you unshelve it | 16:47 |
| *** achanda has joined #openstack-nova | 16:47 | |
| *** mgoddard_ has quit IRC | 16:47 | |
| bauzas | in that case, you'll only unshelve to the same host | 16:47 |
| *** mgoddard has joined #openstack-nova | 16:47 | |
| bauzas | mriedem_snow: dansmith: johnthetubaguy: tpeoples: I'm on a call atm, but I'll work on a bugfix we should backport to RC3 | 16:48 |
| johnthetubaguy | bauzas: that almost feels correct, in some ways, but we should probably not do that till there is a way to "fix" it | 16:49 |
| tpeoples | good point bauzas. it's still problematic for us, at least, for live migrate due to the way we've been using force_hosts in the past - we never associated force_hosts meaning "pin this host here" but rather just cared about initial deploy. will your fix affect live migrate at all? | 16:49 |
| bauzas | tpeoples: there is a workaround for live-mig | 16:49 |
| bauzas | tpeoples: you just need to live-mig <instance> <host> | 16:49 |
| bauzas | tpeoples: and it'll be totally fine | 16:50 |
| *** vilobhmm11 has joined #openstack-nova | 16:50 | |
| bauzas | johnthetubaguy: I don't get your point, you mean backporting to RC3 ? sure | 16:50 |
| tpeoples | yeah, i get that there's a valid workaround. but it breaks previous behavior for us where untargeted migration would work after forcing a deploy | 16:50 |
| bauzas | johnthetubaguy: I was talking about aiming the patch I'm going to write to land for RC3 | 16:50 |
| cfriesen__ | when using ceph with librbd, is there a way to adjust the affinity of the threads used by librbd? | 16:51 |
| johnthetubaguy | bauzas: I am just worrying my little head about things, ignore me really, just worried we might want the current "broken" behaviour longer term | 16:51 |
| bauzas | tpeoples: like I said in the past, if you forced a destination, that's because you don't trust the scheduler, right? | 16:51 |
| bauzas | johnthetubaguy: ah | 16:51 |
| bauzas | johnthetubaguy: nah, be sure I'm going to axe it once I have time to do so :p | 16:51 |
| bauzas | johnthetubaguy: I said multiple times I litterally hate the UX for forced_hosts | 16:51 |
| tpeoples | not necessarily. i may just want to deploy to a specific host for whatever reason right now, but later on i don't care where the VM goes (because i trust the scheduler) | 16:51 |
| *** vilobhmm111 has joined #openstack-nova | 16:51 | |
| bauzas | tpeoples: so you can do that other ways that prevent you to be stuck | 16:52 |
| bauzas | tpeoples: like using a maintenance (or target) aggregate | 16:52 |
| bauzas | tpeoples: where you put the host you want to target | 16:52 |
| bauzas | tpeoples: and then either use a flavor/agg relational filter or an image/agg relational filter | 16:53 |
| tpeoples | aggregate would only get me to deploy to a set of hosts, not necessarily a specific one. requiring someone to temporarily move their desired host into a specific aggregate is a bit overkill imo | 16:53 |
| bauzas | tpeoples: I actually care about the reason you want as an admin to boot an instance to a specific host | 16:54 |
| bauzas | tpeoples: because that can't be done for your regular users | 16:54 |
| *** sudipto has joined #openstack-nova | 16:54 | |
| bauzas | tpeoples: so that only works if you're an admin and you want to boot an instance, which is definitely something not cloudy | 16:54 |
| johnthetubaguy | bauzas: yeah, it probably makes sense to create a new UX that makes more sense | 16:54 |
| *** vilobhmm11 has quit IRC | 16:55 | |
| tpeoples | sure, it's not necessarily cloudy but it's behavior we have users using | 16:55 |
| openstackgerrit | Ed Leafe proposed openstack/nova: Improve the help text for the legacy API options https://review.openstack.org/273266 | 16:55 |
| bauzas | tpeoples: you have *admin* users ? | 16:56 |
| *** e0ne has joined #openstack-nova | 16:56 | |
| *** macsz has quit IRC | 16:56 | |
| *** wasmum has quit IRC | 16:56 | |
| *** lpetrut has joined #openstack-nova | 16:56 | |
| tpeoples | yes, not all of them. | 16:57 |
| bauzas | tpeoples: in that case, why are they not trusting the scheduler and ask for a specific target ? | 16:57 |
| bauzas | tpeoples: that's the why I care | 16:57 |
| bauzas | why do they need to be admin-powered ? | 16:58 |
| bauzas | I mean, it requires knowing the topologyu | 16:58 |
| johnthetubaguy | I guess that have a specific placement required we don't currently model in our API? | 16:58 |
| johnthetubaguy | s/that/they/ | 16:58 |
| bauzas | so you prefer expose your topology and leave people hack the destination rather than leaving the scheduler doing its job | 16:58 |
| bauzas | and I need to understand why | 16:59 |
| tpeoples | bauzas: they might want to deploy to a specific host because they know other related VMs are on that host | 16:59 |
| bauzas | like johnthetubaguy, I guess it can be due to a missing feature, but I'd like to know | 16:59 |
| bauzas | tpeoples: so why not using server groups ? | 16:59 |
| tpeoples | i acknowledge there's other ways to do that but this is how we went about implementation in the past | 16:59 |
| bauzas | tpeoples: okay, so that's historical reasons, right? | 17:00 |
| bauzas | tpeoples: not because you need something else | 17:00 |
| *** pushkaru has joined #openstack-nova | 17:00 | |
| bauzas | tpeoples: and again, that means your user know the host the instance belongs to, which is an admin-only thing | 17:00 |
| tpeoples | sure. i'm not asking for something else, i just don't want something that was previously working for us that seemed valid to do to stop working | 17:00 |
| bauzas | tpeoples: you could do this with regular users without exposing your topology | 17:01 |
| *** openstackgerrit has quit IRC | 17:01 | |
| *** openstackgerrit has joined #openstack-nova | 17:01 | |
| *** e0ne_ has joined #openstack-nova | 17:02 | |
| *** scheuran has quit IRC | 17:02 | |
| *** vilobhmm111 has quit IRC | 17:02 | |
| tpeoples | bauzas: yes, most of our users are admins. our deployments aren't used by *that* many people. typically an admin here and maybe a couple other regular users. | 17:02 |
| bauzas | okay, anyway, I'm going to do this the johnthetubaguy's way by persisting the force_host info but not passing it to the scheduler | 17:03 |
| *** e0ne has quit IRC | 17:03 | |
| bauzas | with a big fat comment leaking how I hate doing this :p | 17:04 |
| tpeoples | bauzas: ok. for the record i don't disagree with what you're saying. it's just something we did in the past and the current code will break that feature (targeted deploy) we support so i'd like to figure out a way to ease the pain | 17:05 |
| bauzas | tpeoples: sure :-) | 17:05 |
| bauzas | tpeoples: I should be able to write that patch either tonight my time or tomorrow (EU based) | 17:06 |
| *** pwnall has quit IRC | 17:06 | |
| tpeoples | bauzas: sounds good, i'll keep an eye out for it. thanks for the discussion | 17:06 |
| *** jlanoux has quit IRC | 17:10 | |
| *** mylu has quit IRC | 17:11 | |
| *** wasmum has joined #openstack-nova | 17:12 | |
| *** mylu has joined #openstack-nova | 17:12 | |
| *** lpetrut has quit IRC | 17:14 | |
| *** lpetrut has joined #openstack-nova | 17:14 | |
| *** e0ne_ has quit IRC | 17:17 | |
| *** ljxiash has joined #openstack-nova | 17:17 | |
| *** eil397 has joined #openstack-nova | 17:17 | |
| *** libu has quit IRC | 17:19 | |
| *** sayalilunkad has quit IRC | 17:21 | |
| *** liunian_ has joined #openstack-nova | 17:22 | |
| *** ljxiash has quit IRC | 17:23 | |
| *** jaypipes has quit IRC | 17:23 | |
| *** rdopiera has quit IRC | 17:23 | |
| *** mylu has quit IRC | 17:26 | |
| *** rmevans has joined #openstack-nova | 17:26 | |
| *** rk4n has joined #openstack-nova | 17:26 | |
| *** mylu has joined #openstack-nova | 17:27 | |
| *** lpetrut has quit IRC | 17:27 | |
| openstackgerrit | Pushkar Umaranikar proposed openstack/nova: Config options: Centralize netconf options https://review.openstack.org/294312 | 17:27 |
| *** unicell has joined #openstack-nova | 17:27 | |
| *** lucasagomes is now known as lucas-afk | 17:28 | |
| *** lpetrut has joined #openstack-nova | 17:29 | |
| *** nic has quit IRC | 17:30 | |
| *** mylu has quit IRC | 17:30 | |
| *** mylu has joined #openstack-nova | 17:30 | |
| *** eil397 has quit IRC | 17:30 | |
| *** pradk has joined #openstack-nova | 17:31 | |
| *** oomichi has joined #openstack-nova | 17:31 | |
| *** earlephilhower has joined #openstack-nova | 17:32 | |
| *** rwmjones_mtg is now known as rwmjones | 17:34 | |
| *** lpetrut has quit IRC | 17:35 | |
| *** mylu has quit IRC | 17:35 | |
| *** rcernin has quit IRC | 17:36 | |
| *** wasmum has quit IRC | 17:38 | |
| *** haukebruno has quit IRC | 17:38 | |
| *** wasmum has joined #openstack-nova | 17:40 | |
| *** pcaruana has quit IRC | 17:40 | |
| *** garyk has quit IRC | 17:41 | |
| *** sahid has quit IRC | 17:41 | |
| *** ndipanov has quit IRC | 17:42 | |
| *** liunian_ has quit IRC | 17:42 | |
| *** eil397 has joined #openstack-nova | 17:43 | |
| *** pwnall has joined #openstack-nova | 17:45 | |
| melwitt | mriedem_snow: it looks like "Improve Developer Docs" and "Quotas" subteam got accidentally removed from https://etherpad.openstack.org/p/newton-nova-priorities-tracking ? | 17:46 |
| *** angdraug has quit IRC | 17:47 | |
| *** eil397 has left #openstack-nova | 17:47 | |
| *** nic has joined #openstack-nova | 17:49 | |
| *** wasmum has quit IRC | 17:51 | |
| *** wasmum has joined #openstack-nova | 17:53 | |
| *** matrohon has quit IRC | 17:54 | |
| dansmith | melwitt: were you around for the meeting? | 17:54 |
| *** sfinucan has quit IRC | 17:54 | |
| melwitt | dansmith: no. I take it I should read the log now | 17:54 |
| dansmith | it was discussed that some things would be trimmed from that now and periodically to make sure they're getting regenerated by the subteams | 17:55 |
| dansmith | so I imagine that happened | 17:55 |
| melwitt | ah, okay | 17:55 |
| *** ihrachys has quit IRC | 17:55 | |
| *** mylu has joined #openstack-nova | 17:55 | |
| *** mpavone has quit IRC | 17:56 | |
| *** cdent has quit IRC | 17:57 | |
| *** cdent has joined #openstack-nova | 17:57 | |
| *** cdent has quit IRC | 17:57 | |
| *** salv-orl_ has joined #openstack-nova | 18:02 | |
| *** ctrath has quit IRC | 18:03 | |
| auggy | melwitt: i also missed the meeting and am catching up :) | 18:04 |
| auggy | bestcoast time zone ftw! | 18:04 |
| *** salv-orlando has quit IRC | 18:05 | |
| dansmith | ugh you people | 18:07 |
| *** nic has quit IRC | 18:08 | |
| *** nic has joined #openstack-nova | 18:08 | |
| melwitt | haha :) | 18:08 |
| nic | What do you mean, "you people"? | 18:08 |
| dansmith | "you late-ass sleeping-in giving-west-coasters-a-bad-rap people" | 18:09 |
| auggy | hehe | 18:09 |
| alaski | mriedem_snow: I don't understand why this would cause tests to explode https://review.openstack.org/#/c/263925/25/nova/compute/api.py | 18:09 |
| dansmith | alaski: sometimes mriedem_snow seems to quote the line above the one he means to, so I assume he means the .create() call | 18:10 |
| dansmith | presumably because of a lack of mock or something? | 18:10 |
| alaski | ahh, that makes a bit more sense | 18:11 |
| openstackgerrit | Gleb Stepanov proposed openstack/nova: Add console log offset https://review.openstack.org/292473 | 18:12 |
| thomasem | back | 18:12 |
| *** lpetrut has joined #openstack-nova | 18:14 | |
| *** rmevans has quit IRC | 18:14 | |
| *** wasmum has quit IRC | 18:15 | |
| openstackgerrit | Andrew Laski proposed openstack/nova: Create Instance from BuildRequest if not in a cell https://review.openstack.org/263927 | 18:16 |
| openstackgerrit | Andrew Laski proposed openstack/nova: Populate instance_mappings during boot https://review.openstack.org/263925 | 18:16 |
| openstackgerrit | Andrew Laski proposed openstack/nova: Include CellMapping in InstanceMapping object https://review.openstack.org/286309 | 18:16 |
| dansmith | alaski: good answer | 18:18 |
| *** Bjoern is now known as Bjoern_zZzZzZzZ | 18:18 | |
| alaski | :) | 18:18 |
| *** lpetrut has quit IRC | 18:18 | |
| *** tbachman has quit IRC | 18:19 | |
| mriedem_snow | dansmith: alaski: wrong! | 18:20 |
| *** mriedem_snow is now known as mriedem | 18:20 | |
| *** wasmum has joined #openstack-nova | 18:21 | |
| mriedem | melwitt: i'm pretty sure i left those in? | 18:21 |
| *** tbachman has joined #openstack-nova | 18:21 | |
| mriedem | melwitt: i'm going to say that someone else removed those | 18:22 |
| mriedem | i remember seeing them | 18:22 |
| *** nic has quit IRC | 18:22 | |
| *** nic has joined #openstack-nova | 18:22 | |
| mriedem | alaski: clarified | 18:23 |
| mriedem | alaski: i meant, you're setting inst_mapping.cell_id | 18:23 |
| mriedem | but that's renamed in the dependent change https://review.openstack.org/#/c/286309/10/nova/objects/instance_mapping.py@35 | 18:23 |
| mriedem | so you're setting cell_id which doesn't really do anything | 18:24 |
| auggy | maybe there's someone out there who just really hates quotas and developer docs | 18:24 |
| mriedem | sorry, was clearing our driveway and then lunch | 18:24 |
| openstackgerrit | Sujitha proposed openstack/nova: Config options: Centralize compute options https://review.openstack.org/295938 | 18:24 |
| alaski | mriedem: gotcha, good catch. | 18:25 |
| openstackgerrit | Gleb Stepanov proposed openstack/nova: Improve line continuation https://review.openstack.org/290378 | 18:25 |
| mriedem | alaski: actually feng xi caught it earlier | 18:26 |
| mriedem | i was just reiterating | 18:26 |
| alaski | oops, missed that | 18:27 |
| mriedem | and now because the schools are closed, i have to work while little mermaid: ariel's beginning is blaring in the background | 18:27 |
| dims | mriedem : awwww | 18:27 |
| *** gcb has quit IRC | 18:28 | |
| *** gcb has joined #openstack-nova | 18:29 | |
| alaski | mriedem: I'm not sure that bug will actually cause a failure. The right think would be to set cell_mapping to None but in create() it will do the same thing as not setting it | 18:29 |
| alaski | it's still wrong though | 18:29 |
| *** pushkaru has quit IRC | 18:30 | |
| *** pushkaru has joined #openstack-nova | 18:32 | |
| *** daemontool has quit IRC | 18:32 | |
| dansmith | could have said "some kid movie", but no, had to go all the way | 18:32 |
| dansmith | Title: Subtitle | 18:32 |
| *** ctrath has joined #openstack-nova | 18:33 | |
| *** Sukhdev has joined #openstack-nova | 18:33 | |
| alaski | dansmith: is there a reason objects don't blow up when setting an attribute that's not a defined field? | 18:33 |
| dansmith | alaski: because that's a totally legit thing to do | 18:33 |
| dansmith | and we do it on purpose in places | 18:33 |
| dansmith | fields is just the stuff you need version tracked and RPC'd | 18:33 |
| dansmith | there are things I'd do differently about that if I had to do it all over again, but.. | 18:34 |
| mriedem | dansmith: remember me having that exact same question a few saturdays ago? | 18:34 |
| alaski | okay. I'm not advocating for it, just curious | 18:34 |
| dansmith | mriedem: yep | 18:34 |
| *** itlinux has joined #openstack-nova | 18:34 | |
| itlinux | hello all | 18:34 |
| itlinux | I have a question on host-aggregates metadata | 18:35 |
| itlinux | can someone answer one of my questions thanks | 18:35 |
| itlinux | how can I retrieve the metadata from the instance where I see all the other metadata set in by cloud-init | 18:35 |
| itlinux | thanks | 18:35 |
| openstackgerrit | Anusha Unnam proposed openstack/nova: config options: Centralize xenapi options https://review.openstack.org/297301 | 18:36 |
| *** Sukhdev has quit IRC | 18:39 | |
| dansmith | alaski: so you just removed the lazy load from that instance_mapping patch? | 18:40 |
| alaski | dansmith: yes. cell_mapping should always be explicitly set based on the db result | 18:40 |
| dansmith | okay | 18:40 |
| *** yassou has quit IRC | 18:40 | |
| *** vilobhmm11 has joined #openstack-nova | 18:40 | |
| *** Bjoern_zZzZzZzZ is now known as Bjoern | 18:42 | |
| dansmith | alaski: am I missing where you test the case where we load from DB and the linkage is None? | 18:43 |
| * alaski looks | 18:44 | |
| alaski | nope, did not test that path | 18:44 |
| * alaski goes to typey typey | 18:44 | |
| dansmith | \o/ I found something | 18:45 |
| *** mylu has quit IRC | 18:45 | |
| dansmith | although the bad news is it just prevents you from spending time +2ing my stuff :/ | 18:45 |
| *** mylu has joined #openstack-nova | 18:46 | |
| *** Guest27982 has quit IRC | 18:46 | |
| *** Marga_ has quit IRC | 18:46 | |
| alaski | my plan worked | 18:47 |
| dims | haha | 18:47 |
| dansmith | damn you pesky kids! | 18:47 |
| mriedem | dansmith: so https://review.openstack.org/#/c/294313/10/nova/objects/flavor.py@310 | 18:47 |
| mriedem | "No. The current code has to look up the flavor to get the id, then use the id on the project row. Since we have the object now, we can skip that extra step and just provide it the thing we need." | 18:48 |
| dansmith | mriedem: yeah, so I really hate that, because we eclipse the id builtin | 18:48 |
| dansmith | and my editor screams at me | 18:48 |
| *** gokrokve has quit IRC | 18:48 | |
| dansmith | (and rightly so) | 18:48 |
| *** mtanino has joined #openstack-nova | 18:48 | |
| mriedem | hate 'id' as the name of the formal parameter? | 18:48 |
| dansmith | yeah | 18:48 |
| mriedem | so flavor_id is your way of saying flavor.id, vs flavorid | 18:49 |
| *** wanghao has quit IRC | 18:49 | |
| mriedem | which is itself a field | 18:49 |
| dansmith | yeah | 18:49 |
| mriedem | god | 18:49 |
| dansmith | I mean, not just my way, | 18:49 |
| *** wanghao has joined #openstack-nova | 18:49 | |
| dansmith | it's sprinkled some through the existing code, | 18:49 |
| dansmith | although none of it is fully consistent of course | 18:49 |
| dansmith | all of my new code is consistent, | 18:49 |
| mriedem | i have to check the api db model on flavor project | 18:49 |
| dansmith | and we're killing the old stuff | 18:49 |
| mriedem | ooooo https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api_models.py#L154 | 18:50 |
| mriedem | ding! | 18:50 |
| dansmith | right, that's why I'm doing it, | 18:50 |
| dansmith | because it's the column on the related objects | 18:50 |
| dansmith | that refers to flavor.id, not flavorid | 18:50 |
| dansmith | er, flavor.id not flavor.flavorid | 18:50 |
| mriedem | yeah, just answered my own question in there | 18:51 |
| mriedem | would be nice to have a note about that, but not worth reving if the rest is ok | 18:51 |
| dansmith | okay | 18:52 |
| mriedem | also, can we return project from _flavor_add_project? | 18:52 |
| dansmith | mriedem: we can but we don't use it, so why? | 18:52 |
| dansmith | actually we can't | 18:52 |
| dansmith | well, I mean we can, but we shouldn't | 18:52 |
| dansmith | because it's out of the db session at that point, | 18:52 |
| dansmith | which means if we touch anything that needs to talk to the db again, we'll explode | 18:53 |
| mriedem | well, can we remove the return in the _flavor_add_project staticmethod? | 18:53 |
| mriedem | and _flavor_del_project | 18:53 |
| *** pmacharya has joined #openstack-nova | 18:53 | |
| *** achanda has quit IRC | 18:53 | |
| dansmith | you disagree with my proxy comment? | 18:54 |
| dansmith | I wasn't sure what side you were taking with your "me too" | 18:54 |
| mriedem | 'me too' as in one of those crazy people | 18:56 |
| mriedem | that don't like it | 18:56 |
| mriedem | sure, it's a prox | 18:56 |
| mriedem | *proxy | 18:56 |
| dansmith | I don't get that line of thinking at all | 18:56 |
| mriedem | but if the thing you're calling isn't going to return anything, it's confusing, imo, to have a return from the other method | 18:57 |
| dansmith | it's not a proxy unless it passes everything in and returns everything out | 18:57 |
| mriedem | glorp, ok so it's a black box to you | 18:57 |
| mriedem | i'm thinking of it as a white box | 18:57 |
| mriedem | i don't care all that much | 18:57 |
| dansmith | all of our db_api calls are clean proxies, AFAICT | 18:58 |
| mriedem | still need to review tests, have to handle this whiny dog first | 18:58 |
| *** mylu has quit IRC | 18:58 | |
| dansmith | alaski: where do you stand on this style issue? (answer only required if you agree with me) | 18:59 |
| alaski | dansmith: writing tests and don't have context, which style issue? | 19:00 |
| dansmith | alaski: just say "of course dansmith is right" and you can go back to work | 19:00 |
| *** mylu has joined #openstack-nova | 19:00 | |
| pmacharya | Hi, I was trying to upgrade nova db from Juno to Liberty release. the nova-manage db sync fails saying there are unmigrated flavor records and tells to run nova-manage db migrate_flavor_data. But I cannot find that option. Can someone point me to what am I missing? | 19:01 |
| alaski | dansmith: heh. you're always right Dan | 19:01 |
| * dansmith frames that | 19:01 | |
| mriedem | pmacharya: see the release notes for liberty | 19:01 |
| mriedem | there is a nova-manage command you have to run before you can upgrade your db to liberty | 19:01 |
| dansmith | and IIRC, we had a bug in what we told you to run in that error message, which should be fixed at this point | 19:01 |
| *** zenoway has joined #openstack-nova | 19:02 | |
| mriedem | also, upgrading from juno to liberty in one shot, kudos on the giant balls | 19:02 |
| dansmith | oh, right | 19:02 |
| dansmith | pmacharya: you will have to use kilo code to do that migration | 19:02 |
| dansmith | that's why it's missing | 19:02 |
| dansmith | http://www.danplanet.com/blog/2015/06/26/upgrading-nova-to-kilo-with-minimal-downtime/ | 19:02 |
| *** Kennan has quit IRC | 19:02 | |
| *** Swami has quit IRC | 19:03 | |
| melwitt | dansmith: I agree with returning for a proxy call because the called function may be changed to return something and the proxy should continue to work | 19:03 |
| *** Kennan has joined #openstack-nova | 19:03 | |
| alaski | dansmith: mriedem since python doesn't distinguish between 'return None' and not returning I don't see an issue with returning there | 19:04 |
| * dansmith passes melwitt the twenty under the table | 19:04 | |
| pmacharya | Ah ok. I'll try that out. thanks! | 19:04 |
| * dansmith passes another twenty to alaski | 19:04 | |
| mriedem | you can all go straight to hell | 19:04 |
| dansmith | lol | 19:04 |
| alaski | sometimes it's nice to not return just to indicate that the caller shouldn't expect it, but meh | 19:05 |
| melwitt | this sort of thing became apparent to me when I wrote a decorator and initially didn't return the wrapped function result because the first case I worked on didn't return anything | 19:05 |
| dansmith | yeah, it's really confusing to me, | 19:05 |
| dansmith | when you're modifying the bottom one and the top isn't getting the result | 19:05 |
| *** mylu has quit IRC | 19:05 | |
| dansmith | and an hour later you realize the indirection layer you largely ignore isn't returning your result | 19:05 |
| *** sudipto has quit IRC | 19:05 | |
| dansmith | happened to me one with db_api being an oddball method that didn't return | 19:05 |
| melwitt | and then I put the decorator on something else and boom | 19:05 |
| *** mylu has joined #openstack-nova | 19:06 | |
| dansmith | oh | 19:06 |
| openstackgerrit | Pushkar Umaranikar proposed openstack/nova: Config options: Centralize consoleauth options https://review.openstack.org/293118 | 19:06 |
| dansmith | mriedem: I worked "knee deep in the hoopla" into a conversation the other day.. thought you would be proud | 19:07 |
| *** daemontool has joined #openstack-nova | 19:07 | |
| doffm | This has happened to me also. 30 minutes wasted by a non returning wrapper. Gnughhh. | 19:07 |
| dansmith | you weren't around to enjoy though | 19:07 |
| mriedem | dansmith: that's pretty good | 19:07 |
| dansmith | doffm: I'm out of twenties, sorry | 19:07 |
| mriedem | seems we have another bug because the libvirt driver ignores the user-requested device name on volume attach https://bugs.launchpad.net/nova/+bug/1561246 | 19:08 |
| openstack | Launchpad bug 1561246 in OpenStack Compute (nova) "user cannot choose disk bus when attaching volume to instance" [Medium,Confirmed] | 19:08 |
| dansmith | mriedem: that's never worked, and we fixed it this time | 19:08 |
| dansmith | not a regression, they're just mad because now it's for real | 19:08 |
| mriedem | dansmith: care to comment in the bug? | 19:08 |
| *** mylu has quit IRC | 19:08 | |
| mriedem | mark won't fix or invalid if you want | 19:08 |
| dansmith | oh, hmm | 19:09 |
| dansmith | this is about disk bus | 19:09 |
| dansmith | not just the name not being honored | 19:09 |
| *** mylu has joined #openstack-nova | 19:10 | |
| mriedem | yeah | 19:11 |
| mriedem | clarkb had a bug that infra hit b/c of this too because of something with config drive | 19:11 |
| dansmith | so | 19:11 |
| mriedem | the volume attachment says it's at a given mountpoint but that's actually the config drive mount | 19:11 |
| dansmith | I'm sure this was not intended in the original patch | 19:11 |
| dansmith | the device name has never been honored and is just a source of confusion people | 19:11 |
| dansmith | *for people | 19:12 |
| *** Marga_ has joined #openstack-nova | 19:12 | |
| dansmith | but if it was using that to determine the disk bus somewhere deep, then that's not cool | 19:12 |
| dansmith | the problem is, we'd take the parameter and return failure if they specify an already used one, even though it isn't actually used that way in the guest | 19:12 |
| dansmith | so we'd have vda and vdc in the guest, they'd use vdb thinking that was right, and we'd return failure for no reason | 19:13 |
| dansmith | the hack in the patch is probably okay, but we should probably have one of the originators look | 19:14 |
| artom | The valid uuid thing from https://bugs.launchpad.net/nova/+bug/1557378, are we going to start enforcing that for *all* fake uuids in tests, or just the ones that end up in objects' fields? | 19:14 |
| openstack | Launchpad bug 1557378 in OpenStack Compute (nova) "tox gives invalid uuid warnings in unit test" [Medium,In progress] - Assigned to Srushti Gadadare (srushti-gadadare) | 19:14 |
| dansmith | mriedem: you want me to put a patch up with that in it so it's ready for discussion? | 19:18 |
| mriedem | dansmith: sure | 19:18 |
| mriedem | artom: at least object uuid fields | 19:18 |
| mriedem | given it emits a warning | 19:18 |
| *** baoli has quit IRC | 19:18 | |
| dansmith | mriedem: actually, re-reading this I think I'm remembering some discussion | 19:21 |
| *** angdraug has joined #openstack-nova | 19:21 | |
| artom | mriedem, gotcha | 19:22 |
| dansmith | mriedem: I think we choose the appropriate device type based on, like he says, the image metadata | 19:22 |
| dansmith | mriedem: which seems okay to me as the default attachment type | 19:22 |
| artom | Is it worth it to be consistent and insist future new tests use real uuids, regardless of whether there's a FutureWarning or not? | 19:22 |
| dansmith | if you're booting instances from ide based images you probably already hate performance | 19:22 |
| *** ihrachys has joined #openstack-nova | 19:23 | |
| mriedem | artom: real uuids in new tests are probably good, but i wouldn't -1 for it | 19:23 |
| *** achanda has joined #openstack-nova | 19:24 | |
| mriedem | artom: especially since it's easy with the uuidsentinel module | 19:24 |
| openstackgerrit | Andrew Laski proposed openstack/nova: Create Instance from BuildRequest if not in a cell https://review.openstack.org/263927 | 19:24 |
| openstackgerrit | Andrew Laski proposed openstack/nova: Populate instance_mappings during boot https://review.openstack.org/263925 | 19:24 |
| openstackgerrit | Andrew Laski proposed openstack/nova: Include CellMapping in InstanceMapping object https://review.openstack.org/286309 | 19:24 |
| *** daemontool has quit IRC | 19:25 | |
| artom | mriedem, thanks, that sounds very reasonnable. | 19:25 |
| alaski | johnthetubaguy: another one for you to remove the -2 from https://review.openstack.org/#/c/263927/14 | 19:26 |
| *** ljjjustin has quit IRC | 19:27 | |
| *** yamahata has quit IRC | 19:28 | |
| *** baoli has joined #openstack-nova | 19:29 | |
| *** mgoddard has quit IRC | 19:29 | |
| *** zhangjn has quit IRC | 19:31 | |
| *** zhangjn has joined #openstack-nova | 19:32 | |
| mriedem | dansmith: alaski: melwitt: doffm: +2 on https://review.openstack.org/#/c/294313/ | 19:33 |
| *** ljjjustin has joined #openstack-nova | 19:34 | |
| dansmith | woot | 19:34 |
| alaski | will look as soon as I finish raining on a parade | 19:34 |
| *** liusheng has quit IRC | 19:34 | |
| *** RuiChen has quit IRC | 19:34 | |
| dansmith | oh snap | 19:34 |
| dansmith | "will look as soon as I finish ruining someone's day" | 19:35 |
| * dansmith hopes it's not one of his days | 19:35 | |
| *** davidlenwell has quit IRC | 19:35 | |
| *** gongysh has quit IRC | 19:36 | |
| alaski | nope. maybe drizzling is a better description | 19:36 |
| *** Swami has joined #openstack-nova | 19:37 | |
| *** davidlenwell has joined #openstack-nova | 19:38 | |
| *** wuhao has quit IRC | 19:38 | |
| *** wasmum has quit IRC | 19:39 | |
| mriedem | dansmith: does this require a rebase? https://review.openstack.org/#/c/294710/ | 19:40 |
| alaski | bauzas: not sure if you saw https://review.openstack.org/#/c/294283/. It's not released yet though | 19:40 |
| mriedem | all i can think of now is alaski drizzling on someone's something | 19:40 |
| bauzas | alaski: oh nice | 19:41 |
| *** ociuhandu has quit IRC | 19:41 | |
| *** e0ne has joined #openstack-nova | 19:41 | |
| *** baoli has quit IRC | 19:41 | |
| *** RuiChen has joined #openstack-nova | 19:41 | |
| *** liusheng has joined #openstack-nova | 19:42 | |
| *** rk4n has quit IRC | 19:43 | |
| *** baoli has joined #openstack-nova | 19:43 | |
| doffm | dansmith: What is the issue with static methods and context manager decorators? | 19:43 |
| mriedem | bauzas: so did we ever come to some agreement on https://bugs.launchpad.net/nova/+bug/1561357 ? | 19:43 |
| openstack | Launchpad bug 1561357 in OpenStack Compute (nova) "VM deployed with availability-zone (force_hosts) cannot be live migrated to an untargeted host" [High,Confirmed] - Assigned to Sylvain Bauza (sylvain-bauza) | 19:43 |
| doffm | I will have to pull all my static methods out alsol. | 19:43 |
| doffm | *also | 19:43 |
| mriedem | doffm: see https://review.openstack.org/#/c/294283/ for the oslo.db context manager thing | 19:43 |
| bauzas | mriedem: yeah, I'm working on a patch | 19:43 |
| *** gongysh has joined #openstack-nova | 19:43 | |
| mriedem | doffm: the stub_out thing is a bug with fixtures | 19:43 |
| mriedem | or i thought it was, alaski reported the issue i thought | 19:44 |
| alaski | yeah, there's a stub_out issue with staticmethods and py34 https://github.com/testing-cabal/fixtures/pull/23 | 19:44 |
| alaski | there's an issue with py27 as well but we don't hit it | 19:45 |
| alaski | well, we might need to update for the fix I added. I should look into that | 19:45 |
| *** lpetrut has joined #openstack-nova | 19:46 | |
| *** Marga_ has quit IRC | 19:46 | |
| *** Marga_ has joined #openstack-nova | 19:46 | |
| *** mylu has quit IRC | 19:47 | |
| dansmith | mriedem: why do you think it does? | 19:47 |
| dansmith | mriedem: jenkins just blessed it | 19:47 |
| mriedem | dansmith: hmm, might have been an older patchset | 19:48 |
| bauzas | mriedem: FWIW https://bugs.launchpad.net/nova/+bug/1561357/comments/1 | 19:48 |
| openstack | Launchpad bug 1561357 in OpenStack Compute (nova) "VM deployed with availability-zone (force_hosts) cannot be live migrated to an untargeted host" [High,Confirmed] - Assigned to Sylvain Bauza (sylvain-bauza) | 19:48 |
| *** baoli has quit IRC | 19:48 | |
| *** ociuhandu has joined #openstack-nova | 19:50 | |
| openstackgerrit | Sujitha proposed openstack/nova: Config options: Centralize console options https://review.openstack.org/294261 | 19:53 |
| *** achanda has quit IRC | 19:55 | |
| *** ctrath has quit IRC | 19:55 | |
| *** ctrath has joined #openstack-nova | 19:56 | |
| *** mylu has joined #openstack-nova | 19:58 | |
| *** andymaier has quit IRC | 19:58 | |
| mriedem | dansmith: easy backport to liberty that has a +2 https://review.openstack.org/#/c/295478/ | 20:00 |
| *** mylu has quit IRC | 20:01 | |
| dansmith | mriedem: cool, got it | 20:01 |
| alaski | dansmith: https://review.openstack.org/#/c/294313/11/nova/objects/flavor.py :( | 20:01 |
| dansmith | alaski: sadface is not appropriate for actual problems :) | 20:02 |
| *** cdelatte has quit IRC | 20:02 | |
| *** davidlenwell has quit IRC | 20:02 | |
| dansmith | alaski: if you had asked me not to return from my proxy methods I'd try to shame you out of doing it | 20:02 |
| dansmith | :) | 20:02 |
| *** mylu has joined #openstack-nova | 20:03 | |
| alaski | heh | 20:03 |
| dansmith | er wait | 20:03 |
| dansmith | oh, projects | 20:03 |
| dansmith | okay I guess I'm not positive that you're right there, but I'm going to assume you are | 20:04 |
| alaski | I'm not 100% sure, but fairly certain since this is what I do in InstanceMapping and get a None cell_mapping when joined this way | 20:04 |
| dansmith | okay | 20:05 |
| dansmith | better safe anyway | 20:05 |
| dansmith | alaski: while I fix, you wanna check the others real quick to see if there are other cases? | 20:05 |
| alaski | the rest of this patch looked fine to me | 20:05 |
| mriedem | alaski: i'm assuming jaypipes is away for a good reason? | 20:06 |
| mriedem | was waiting for him to review https://review.openstack.org/#/c/296305/ | 20:06 |
| *** gyee has joined #openstack-nova | 20:06 | |
| *** Bjoern has quit IRC | 20:07 | |
| alaski | mriedem: I believe so. He was in a webinar earlier and had to run for something | 20:07 |
| mriedem | alright | 20:08 |
| mriedem | i think rc3 is next week anyway | 20:08 |
| mriedem | but that's marked for rc | 20:08 |
| dansmith | alaski: so I think the best way to test this is to do it in the functional tests later, like the rest of the db methods | 20:09 |
| dansmith | are you cool with that? | 20:09 |
| dansmith | otherwise I have to mock out sqla or write to the DB, which the unit tests basically don't do | 20:10 |
| *** baoli has joined #openstack-nova | 20:10 | |
| alaski | yeah, that's fine with me | 20:10 |
| dansmith | okay | 20:11 |
| *** jwcroppe has joined #openstack-nova | 20:12 | |
| *** ljjjustin has quit IRC | 20:12 | |
| *** davidlenwell has joined #openstack-nova | 20:12 | |
| dansmith | alaski: I'm also wondering.. we have lots of flavors with no projects, right? | 20:13 |
| dansmith | why haven't we hit this in the other test? | 20:13 |
| dansmith | tests | 20:13 |
| alaski | I know you didn't add this, but the exceptions on deleting things not in the db are starting to bug me | 20:13 |
| alaski | dansmith: we should | 20:13 |
| alaski | I don't actually know | 20:13 |
| dansmith | hmm | 20:15 |
| *** rwmjones is now known as rwmjones_hols | 20:16 | |
| *** Swami_ has joined #openstack-nova | 20:16 | |
| *** Swami__ has joined #openstack-nova | 20:17 | |
| *** ljjjustin has joined #openstack-nova | 20:17 | |
| *** Swami has quit IRC | 20:19 | |
| *** zhiyan has quit IRC | 20:19 | |
| alaski | looking at the flavor create path it doesn't seem possible to populate projects, it can only be done later. so it should be empty for flavors in devstack | 20:19 |
| dansmith | well, and the default flavors are created in the first migration and we clearly don't have projects available there | 20:20 |
| *** ljxiash has joined #openstack-nova | 20:20 | |
| *** zhiyan has joined #openstack-nova | 20:20 | |
| *** Swami_ has quit IRC | 20:21 | |
| *** e0ne has quit IRC | 20:21 | |
| alaski | I wonder if nothing is exercising this path | 20:21 |
| openstackgerrit | Mark Doffman proposed openstack/nova: Make Aggregate metadata functions work with API db https://review.openstack.org/297350 | 20:21 |
| alaski | functionally | 20:22 |
| *** jwcroppe_ has joined #openstack-nova | 20:22 | |
| *** smatzek has quit IRC | 20:23 | |
| doffm | Ok, i'm done. Will stop this now and go work on backlog. | 20:23 |
| *** Sukhdev has joined #openstack-nova | 20:24 | |
| *** ljxiash has quit IRC | 20:25 | |
| *** jwcroppe has quit IRC | 20:25 | |
| dansmith | alaski: I just noticed that something I did broke the functional tests at some point | 20:27 |
| dansmith | and I didn't notice | 20:27 |
| *** sheel has quit IRC | 20:27 | |
| *** ihrachys has quit IRC | 20:28 | |
| *** achanda has joined #openstack-nova | 20:28 | |
| dansmith | OH | 20:29 |
| dansmith | I fell into my own trap | 20:30 |
| dansmith | lol | 20:30 |
| *** ccesario has quit IRC | 20:31 | |
| *** achanda has quit IRC | 20:31 | |
| *** haplo37 has quit IRC | 20:31 | |
| alaski | hmm, no jenkins results past the third patch in the series | 20:31 |
| dansmith | yeah it's confused | 20:32 |
| dansmith | alaski: so I just tested and I get [] in db['projects'] | 20:32 |
| dansmith | if not projects are in the db | 20:32 |
| alaski | hmm, interesting. I need to go double check my patch then | 20:32 |
| dansmith | alaski: http://pastebin.com/0a3iHnFd | 20:33 |
| *** lpetrut has quit IRC | 20:33 | |
| dansmith | gives me | 20:34 |
| dansmith | Projects are: [] | 20:34 |
| dansmith | and a pass on the test | 20:34 |
| alaski | cool. then I'm +2 on it | 20:34 |
| dansmith | woot | 20:35 |
| dansmith | I'll leave the test in there | 20:35 |
| alaski | are you pushing a new rev? | 20:36 |
| dansmith | alaski: if you will +A that I'll push just a change to the create patch which has that test isuse | 20:36 |
| alaski | done | 20:38 |
| lifeless | alaski: hi, dunno if you saw my review on fixtures | 20:40 |
| alaski | lifeless: I did, thanks. that was a good catch, and I updated the patch to address it | 20:40 |
| lifeless | alaski: excellent. I'll look at it first thing tuesday (next working day) | 20:41 |
| alaski | lifeless: thanks | 20:41 |
| dansmith | yes | 20:44 |
| dansmith | oops | 20:44 |
| *** Goneri has quit IRC | 20:44 | |
| *** moshele has joined #openstack-nova | 20:45 | |
| *** rk4n has joined #openstack-nova | 20:46 | |
| openstackgerrit | Andrew Laski proposed openstack/nova: Create Instance from BuildRequest if not in a cell https://review.openstack.org/263927 | 20:46 |
| openstackgerrit | Andrew Laski proposed openstack/nova: Populate instance_mappings during boot https://review.openstack.org/263925 | 20:46 |
| openstackgerrit | Andrew Laski proposed openstack/nova: Include CellMapping in InstanceMapping object https://review.openstack.org/286309 | 20:46 |
| openstackgerrit | Dan Smith proposed openstack/nova: Further hack up the nova.tests.unit.db.fakes module of horribleness https://review.openstack.org/296106 | 20:47 |
| openstackgerrit | Dan Smith proposed openstack/nova: Objectify test_flavors and test_flavors_extra_specs https://review.openstack.org/296105 | 20:47 |
| openstackgerrit | Dan Smith proposed openstack/nova: Make flavor extra_specs operations work against the API DB https://review.openstack.org/294710 | 20:47 |
| openstackgerrit | Dan Smith proposed openstack/nova: Block flavor creation until main database is empty https://review.openstack.org/295310 | 20:47 |
| openstackgerrit | Dan Smith proposed openstack/nova: Add flavor migration routine https://review.openstack.org/294771 | 20:47 |
| openstackgerrit | Dan Smith proposed openstack/nova: Make Flavor create() and destroy() work against API DB https://review.openstack.org/294758 | 20:47 |
| *** openstackgerrit has quit IRC | 20:48 | |
| *** openstackgerrit has joined #openstack-nova | 20:49 | |
| openstackgerrit | Andrew Laski proposed openstack/nova: Create Instance from BuildRequest if not in a cell https://review.openstack.org/263927 | 20:49 |
| openstackgerrit | Andrew Laski proposed openstack/nova: Populate instance_mappings during boot https://review.openstack.org/263925 | 20:49 |
| *** Marga_ has quit IRC | 20:53 | |
| *** Sukhdev has quit IRC | 20:53 | |
| *** Marga_ has joined #openstack-nova | 20:53 | |
| dansmith | mriedem: were you good on that create-instance-mapping patch other than the bad field usage? | 20:55 |
| mriedem | haven't been back to that series yet, trying to check https://review.openstack.org/#/c/284095/ before someone approves it | 20:55 |
| *** moshele has quit IRC | 20:56 | |
| *** doug-fis_ has joined #openstack-nova | 20:57 | |
| *** ihrachys has joined #openstack-nova | 20:58 | |
| *** doug-fish has quit IRC | 20:59 | |
| *** sdague has quit IRC | 20:59 | |
| *** rmevans has joined #openstack-nova | 21:00 | |
| *** doug-fis_ has quit IRC | 21:05 | |
| *** sileht has quit IRC | 21:07 | |
| *** sileht has joined #openstack-nova | 21:08 | |
| mriedem | hmm, is there anything that shows up the security groups applied to a server? nova show isn't showing me that | 21:10 |
| *** rtheis has quit IRC | 21:11 | |
| *** jwcroppe_ has quit IRC | 21:11 | |
| *** pmacharya has quit IRC | 21:12 | |
| *** thorst has quit IRC | 21:12 | |
| dansmith | yeah | 21:12 |
| dansmith | nova secgroup-list | 21:12 |
| dansmith | or list-secgroups | 21:12 |
| dansmith | one lists the security groups, one shows it on a server | 21:12 |
| dansmith | mriedem: ^ | 21:13 |
| melwitt | mriedem: nova show should show it if policy allows you access to the security groups extension | 21:13 |
| *** fawadkhaliq has joined #openstack-nova | 21:13 | |
| mriedem | ah list-secgroup | 21:13 |
| mriedem | melwitt: it's not in the server view details | 21:13 |
| dansmith | I see it on my machine | 21:13 |
| dansmith | in nova show | 21:13 |
| dansmith | | security_groups | default, permissive | 21:13 |
| melwitt | mriedem: yeah, at least on my version I don't see it as a normal user, I do see it as admin | 21:13 |
| *** eharney has quit IRC | 21:14 | |
| mriedem | hmm, i am admin | 21:14 |
| *** thorst has joined #openstack-nova | 21:14 | |
| mriedem | i'm able to list all tenants instances, so i know i'm admin, and created this network | 21:14 |
| mriedem | dansmith: that's server create or server show? | 21:15 |
| *** mylu has quit IRC | 21:16 | |
| fawadkhaliq | folks, quick question on Nova cells. when scheduling instance creation, I see that user can specify hint with target_cell filter to launch instance on a particular cell. Is this an admin only or even tenants can choose to do that? Also, is there a way to limit a particular tenant to only be able to launch Nova instance on a particular cell or a group of cells rather than all? | 21:16 |
| dansmith | mriedem: show | 21:17 |
| mriedem | hmmmm wtf | 21:17 |
| *** mylu has joined #openstack-nova | 21:17 | |
| mriedem | dansmith: oh i think i see | 21:18 |
| mriedem | dansmith: you're using nova-network right? | 21:18 |
| dansmith | yep | 21:18 |
| *** rfolco_ has quit IRC | 21:18 | |
| dansmith | because this machine has to actually work | 21:18 |
| *** thorst has quit IRC | 21:18 | |
| mriedem | yeah, ok, it's different if it's a GET vs POST and if it's nova-net vs neutron | 21:18 |
| mriedem | yay for usability! | 21:18 |
| mriedem | https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/security_groups.py#L452-L475 | 21:19 |
| mriedem | if it's a GET, it doesn't add it | 21:19 |
| mriedem | if it's a POST, it defaults to adding the 'default' security group | 21:19 |
| melwitt | hm, so far this looks like it's only in the legacy_v2 api? https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/legacy_v2/contrib/security_groups.py#L453-L457 | 21:19 |
| mriedem | even if it's not in there | 21:19 |
| mriedem | melwitt: nope | 21:19 |
| melwitt | oh, sorry I just saw your link | 21:20 |
| mriedem | kabam! http://paste.openstack.org/show/491809/ | 21:20 |
| melwitt | well, so the behavior was changed. I wonder if that was intentional | 21:20 |
| mriedem | melwitt: between v2 and v2.1? | 21:22 |
| melwitt | I have known there's a problem with the response for a POST, it always shows "default" no matter what reality is. there was discussion about it on a review where someone was trying to fix the "assuming default security group for neutron" thing | 21:22 |
| mriedem | melwitt: yeah https://review.openstack.org/#/c/173204/ | 21:22 |
| melwitt | mriedem: yeah, v2 is definitely adding the security groups for a GET | 21:22 |
| melwitt | mriedem: that's the one | 21:23 |
| *** baoli has quit IRC | 21:23 | |
| mriedem | ok, it's coming up again here https://review.openstack.org/#/c/284095/ | 21:26 |
| mriedem | just reiterated my -1 | 21:26 |
| melwitt | fawadkhaliq: target_cell filter is controlled by policy | 21:28 |
| *** aysyd has quit IRC | 21:29 | |
| melwitt | fawadkhaliq: for example https://github.com/openstack/nova/blob/master/etc/nova/policy.json#L6 | 21:29 |
| *** efried has quit IRC | 21:30 | |
| *** ihrachys_ has joined #openstack-nova | 21:31 | |
| *** omnipresent has joined #openstack-nova | 21:31 | |
| *** cleong has quit IRC | 21:31 | |
| dansmith | melwitt: the context thing is fine, | 21:32 |
| dansmith | because we've got to have the context already before we call create | 21:32 |
| dansmith | I'll comment | 21:32 |
| *** ctrath has quit IRC | 21:32 | |
| melwitt | fawadkhaliq: I don't think there's a way to limit tenants to a particular cell or group of cells. alaski, do you know? ^ | 21:32 |
| fawadkhaliq | melwitt: thanks, policy.json makes sense. | 21:33 |
| *** ildikov has quit IRC | 21:33 | |
| *** ababich has quit IRC | 21:33 | |
| *** tdurakov_ has joined #openstack-nova | 21:33 | |
| *** browne1 has joined #openstack-nova | 21:33 | |
| fawadkhaliq | melwitt: yes, tenant to cell is the second part I was thinking about. thanks for the response. | 21:33 |
| *** ildikov has joined #openstack-nova | 21:33 | |
| *** ihrachys has quit IRC | 21:34 | |
| *** armax has quit IRC | 21:34 | |
| *** yonglihe has quit IRC | 21:34 | |
| *** junjie_ has joined #openstack-nova | 21:34 | |
| *** oomichi has quit IRC | 21:34 | |
| *** patrickeast has quit IRC | 21:34 | |
| *** sarafraj has quit IRC | 21:34 | |
| *** tdurakov has quit IRC | 21:34 | |
| *** junjie has quit IRC | 21:34 | |
| *** zenoway has quit IRC | 21:34 | |
| *** tdurakov_ is now known as tdurakov | 21:34 | |
| *** junjie_ is now known as junjie | 21:34 | |
| *** harlowja_at_home has quit IRC | 21:34 | |
| *** ayoung has quit IRC | 21:34 | |
| *** carl_baldwin has quit IRC | 21:34 | |
| *** armax has joined #openstack-nova | 21:34 | |
| *** yonglihe has joined #openstack-nova | 21:35 | |
| *** tr3buchet has quit IRC | 21:35 | |
| *** devananda has quit IRC | 21:35 | |
| *** carl_baldwin has joined #openstack-nova | 21:35 | |
| melwitt | dansmith: hm, okay. | 21:35 |
| *** browne1 has quit IRC | 21:35 | |
| *** raorn has quit IRC | 21:35 | |
| *** ctrath has joined #openstack-nova | 21:36 | |
| *** mylu_ has joined #openstack-nova | 21:37 | |
| *** nic has quit IRC | 21:37 | |
| *** browne has quit IRC | 21:37 | |
| melwitt | dansmith: I meant as a standalone private function, I thought it would be using the passed context? | 21:37 |
| *** zhangjn has quit IRC | 21:37 | |
| *** patrickeast has joined #openstack-nova | 21:38 | |
| dansmith | melwitt: you mean it *could* be misused? | 21:38 |
| fawadkhaliq | melwitt: btw, do you know if Horizon allows the target cell hint option similar to region and availability_zones or target cell is only available via the CLI? | 21:38 |
| *** oomichi has joined #openstack-nova | 21:39 | |
| *** sarafraj has joined #openstack-nova | 21:39 | |
| *** knikolla has quit IRC | 21:40 | |
| melwitt | dansmith: not quite thinking about misuse, but thinking about if it makes sense what it's doing, on its own | 21:40 |
| *** openstack has joined #openstack-nova | 21:49 | |
| *** ababich has joined #openstack-nova | 21:50 | |
| dansmith | damn, I can't find any good homer simpson pouting memes | 21:50 |
| melwitt | okay. I trust you. I was just not making assumptions, ignorance is bliss I guess :P | 21:50 |
| *** fawadkhaliq has quit IRC | 21:51 | |
| dansmith | no, there's nothing to trust here, just opinion :) | 21:51 |
| *** mylu has joined #openstack-nova | 21:52 | |
| *** fawadkhaliq has joined #openstack-nova | 21:52 | |
| *** pwnall has quit IRC | 21:52 | |
| *** mdrabe has quit IRC | 21:56 | |
| *** gongysh has joined #openstack-nova | 21:56 | |
| openstackgerrit | Sujitha proposed openstack/nova: config options: Centralize upgrade_levels section https://review.openstack.org/297381 | 21:57 |
| *** mylu has quit IRC | 21:57 | |
| *** ametts has quit IRC | 21:57 | |
| *** pradk has quit IRC | 21:58 | |
| dansmith | melwitt: are you good on that other than that issue? I'm updating for him now | 22:00 |
| *** edmondsw has joined #openstack-nova | 22:00 | |
| *** jdurgin has joined #openstack-nova | 22:00 | |
| *** clduser has joined #openstack-nova | 22:00 | |
| *** clayton has joined #openstack-nova | 22:00 | |
| *** diana_clarke has joined #openstack-nova | 22:00 | |
| *** penick has joined #openstack-nova | 22:01 | |
| *** vladikr has quit IRC | 22:01 | |
| openstackgerrit | Sujitha proposed openstack/nova: config options: Centralize upgrade_levels section https://review.openstack.org/297381 | 22:02 |
| *** DuncanT has quit IRC | 22:03 | |
| dansmith | heh, this is failing unit tests anyway | 22:03 |
| openstackgerrit | Dan Smith proposed openstack/nova: Include CellMapping in InstanceMapping object https://review.openstack.org/286309 | 22:05 |
| *** DuncanT has joined #openstack-nova | 22:06 | |
| *** burt has quit IRC | 22:07 | |
| *** baoli has joined #openstack-nova | 22:09 | |
| *** baoli has quit IRC | 22:09 | |
| *** baoli has joined #openstack-nova | 22:10 | |
| *** thorst has joined #openstack-nova | 22:11 | |
| *** fawadkhaliq has quit IRC | 22:11 | |
| *** fawadkhaliq has joined #openstack-nova | 22:12 | |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Stop providing force_hosts to the scheduler for move ops https://review.openstack.org/297387 | 22:13 |
| bauzas | dansmith: mriedem: alaski: ^ | 22:13 |
| *** omnipresent has quit IRC | 22:13 | |
| bauzas | it's a RC2 regression fix | 22:13 |
| *** wanghao has quit IRC | 22:13 | |
| *** zenoway has joined #openstack-nova | 22:14 | |
| *** wanghao has joined #openstack-nova | 22:14 | |
| *** dave-mccowan has joined #openstack-nova | 22:14 | |
| alaski | melwitt: there is no upstream way that I'm aware of to limit tenants to particular cells | 22:14 |
| *** ctrath has quit IRC | 22:15 | |
| alaski | fawadkhaliq: ^ | 22:15 |
| *** rmevans has quit IRC | 22:15 | |
| fawadkhaliq | alaski: I see. | 22:16 |
| *** thorst has quit IRC | 22:16 | |
| fawadkhaliq | alaski: is this something being considered btw as a useful addition? or exist outside upstreamed version? | 22:16 |
| *** thorst has joined #openstack-nova | 22:16 | |
| alaski | I know that it's been useful to other deployers, who have written their own way to do it. But I don't think it was ever published publicly | 22:17 |
| *** _ducttape_ has quit IRC | 22:17 | |
| alaski | and I no longer have access to it | 22:17 |
| *** zenoway has quit IRC | 22:18 | |
| fawadkhaliq | alaski: yeah, I see something like CERN did with custom scheduler to map tenants to cells. | 22:18 |
| fawadkhaliq | alaski: thanks for the information. its helpful. | 22:18 |
| cburgess | mriedem Oh wow thats a fun bug. Operationally our mon IP's never change so we have never hit this but wow... good bug. | 22:19 |
| *** browne has quit IRC | 22:19 | |
| *** ihrachys_ is now known as ihrachys | 22:20 | |
| alaski | fawadkhaliq: right. in cellsv2 you could write a cell scheduler filter to do it. in cellsv2 you could probably write a scheduler filter to do it | 22:20 |
| alaski | bah, sub first cellsv2 with cellsv1 | 22:20 |
| *** thorst has quit IRC | 22:21 | |
| fawadkhaliq | alaski: really, could you please point me to how that can be done, I would like to try that. That seems reasonable. | 22:24 |
| *** knikolla has joined #openstack-nova | 22:25 | |
| alaski | fawadkhaliq: you would add something like https://github.com/openstack/nova/blob/master/nova/cells/filters/image_properties.py but just pull the tenant info from request_spec and then filter out the cells it shouldn't land in | 22:26 |
| alaski | you'll have to hardcode the mapping or add a lookup to some datasource | 22:27 |
| *** erlon has quit IRC | 22:27 | |
| alaski | I've seen it done with a config file that's loaded in | 22:27 |
| alaski | and you'll need to add the filter to the config at https://github.com/openstack/nova/blob/master/nova/conf/cells.py#L512 | 22:28 |
| fawadkhaliq | alaski: perfect, that looks very cool. The filter can be seeded by a config file, which maps tenants to cells. | 22:28 |
| fawadkhaliq | alaski: seems like as soon as I add and register a filter, it will be picked by Nova and I should be good to go. | 22:29 |
| alaski | yep | 22:29 |
| fawadkhaliq | alaski: thanks, this is very helpful. I will hack a bit. | 22:29 |
| alaski | np | 22:29 |
| alaski | melwitt: are you good with the patch after dansmith fixed it up. if not I can clean that method up further | 22:31 |
| *** dimtruck is now known as zz_dimtruck | 22:31 | |
| cburgess | mriedem Wow actually this is worse then I thought. THis patch is wrong and this is really a cinder problem. | 22:32 |
| openstackgerrit | Andrew Laski proposed openstack/nova: Populate instance_mappings during boot https://review.openstack.org/263925 | 22:33 |
| *** drankis has quit IRC | 22:34 | |
| openstackgerrit | Andrew Laski proposed openstack/nova: Populate instance_mappings during boot https://review.openstack.org/263925 | 22:34 |
| *** pwnall has joined #openstack-nova | 22:34 | |
| dansmith | alaski: sorry I orphaned your other patches | 22:34 |
| melwitt | alaski: yes, going back to it now | 22:34 |
| alaski | dansmith: no worries, thanks for fixing it. I fixed the assertNone on the next patch as well but I'll hold off on further patches until these have more review | 22:36 |
| *** pushkaru has joined #openstack-nova | 22:39 | |
| *** penick has quit IRC | 22:44 | |
| openstackgerrit | Anusha Unnam proposed openstack/nova: config options: Centralize xenapi options https://review.openstack.org/297301 | 22:48 |
| *** pwnall has quit IRC | 22:51 | |
| cburgess | mriedem Oh and he breaks iscsi with this patch... | 22:51 |
| tpeoples | bauzas: thanks for working on that. i tested out live migration, working fine... force_hosts is not wiped from the DB. | 22:53 |
| *** vishwanathj has quit IRC | 22:55 | |
| *** gongysh has quit IRC | 22:56 | |
| *** cfarquhar has quit IRC | 23:00 | |
| *** akshai has quit IRC | 23:00 | |
| *** cfarquhar has joined #openstack-nova | 23:00 | |
| *** cfarquhar has quit IRC | 23:00 | |
| *** cfarquhar has joined #openstack-nova | 23:00 | |
| *** thorst has joined #openstack-nova | 23:02 | |
| *** rk4n has quit IRC | 23:03 | |
| *** thorst has quit IRC | 23:05 | |
| *** thorst has joined #openstack-nova | 23:05 | |
| *** ljxiash has joined #openstack-nova | 23:06 | |
| *** fawadkhaliq has quit IRC | 23:06 | |
| *** fawadkhaliq has joined #openstack-nova | 23:07 | |
| *** davidlenwell has quit IRC | 23:07 | |
| dansmith | alaski: functional tests are happy on top of that set now: https://review.openstack.org/#/c/294758/15 | 23:07 |
| *** zenoway has joined #openstack-nova | 23:08 | |
| aunnam_ | Hi., trying to run test coverage report using tox -e cover, but it is not generating covhtml folder. Any suggestions? | 23:08 |
| *** thorst has quit IRC | 23:10 | |
| *** ljxiash has quit IRC | 23:10 | |
| *** ducttape_ has joined #openstack-nova | 23:11 | |
| *** zenoway has quit IRC | 23:12 | |
| *** aNupoisc has joined #openstack-nova | 23:13 | |
| aNupoisc | Hi guys. Is there a way to see what files affect n-api or n-sch service? I want to make changes to few files but don't know what service they affect. Can anybody help me | 23:14 |
| *** thorst has joined #openstack-nova | 23:14 | |
| *** browne has joined #openstack-nova | 23:14 | |
| *** davidlenwell has joined #openstack-nova | 23:14 | |
| *** thorst_ has joined #openstack-nova | 23:17 | |
| *** thorst has quit IRC | 23:19 | |
| *** thorst has joined #openstack-nova | 23:21 | |
| *** thorst_ has quit IRC | 23:22 | |
| *** ducttape_ has quit IRC | 23:25 | |
| *** ducttape_ has joined #openstack-nova | 23:26 | |
| *** vilobhmm11 has quit IRC | 23:32 | |
| *** ducttape_ has quit IRC | 23:33 | |
| *** ducttape_ has joined #openstack-nova | 23:34 | |
| *** ducttape_ has quit IRC | 23:34 | |
| mriedem | cburgess: nice | 23:35 |
| mriedem | aNupoisc: the services are started from the nova/cmd/* files | 23:36 |
| mriedem | aNupoisc: if you have an example it would be more helpful | 23:37 |
| cburgess | mriedem I provided feedback on the review. Durgen is dead on about what we need to do though. | 23:37 |
| *** gongysh has joined #openstack-nova | 23:39 | |
| mriedem | alaski: saw your comments in tbachman's vmware driver patch | 23:40 |
| *** _ducttape_ has joined #openstack-nova | 23:40 | |
| mriedem | alaski: welcome to the wonderful world of unique vif keys per each neutron plugin driver | 23:40 |
| mriedem | it's awesome | 23:40 |
| *** vilobhmm11 has joined #openstack-nova | 23:42 | |
| aNupoisc | like I want to make changes to nova/api/openstack/compute/servers.py but don't know whether that will affect n-api service or n-sch service | 23:43 |
| aNupoisc | mriedem: like I want to make changes to nova/api/openstack/compute/servers.py but don't know whether that will affect n-api service or n-sch service | 23:43 |
| mriedem | aNupoisc: n-api | 23:44 |
| aNupoisc | mriedem: thnx for letting me know about nova/cmd | 23:44 |
| mriedem | the rest api stuff is under nova/api/ | 23:44 |
| mriedem | scheduler stuff is under nova/scheduler | 23:44 |
| aNupoisc | Okay great. Is this mentioned in some script or its just like that? I mean is it done during start_nova_api that gets called from stack.sh on devstackfunction | 23:48 |
| *** claudiub|2 has quit IRC | 23:49 | |
| aNupoisc | mriedem: Okay great. Is this mentioned in some script or its just like that? I mean is it done during start_nova_api that gets called from stack.sh on devstack | 23:49 |
| *** zehua has joined #openstack-nova | 23:50 | |
| *** vilobhmm111 has joined #openstack-nova | 23:51 | |
| *** vilobhmm11 has quit IRC | 23:54 | |
| *** sbezverk has quit IRC | 23:54 | |
| mriedem | aNupoisc: the n-api service is started from start_nova_api in devstack yes | 23:57 |
| mriedem | http://docs.openstack.org/developer/nova/architecture.html might help | 23:57 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!