| *** liverpooler has quit IRC | 00:01 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: libvirt: pass log_path to _create_pty_device for non-kvm/qemu https://review.openstack.org/442209 | 00:04 |
|---|---|---|
| mriedem | tonygunk: ^ | 00:04 |
| mriedem | dansmith: melwitt: ^ simple fix for a breaking bug once you upgrade to ocata if you fall into that deployment scenario | 00:06 |
| *** mriedem has quit IRC | 00:06 | |
| *** owalsh is now known as owalsh-afk | 00:07 | |
| *** gaoyuan has joined #openstack-nova | 00:11 | |
| *** tbachman has joined #openstack-nova | 00:13 | |
| openstackgerrit | Dan Smith proposed openstack/nova master: Avoid logging entire CellMapping objects https://review.openstack.org/442211 | 00:13 |
| *** delatte has quit IRC | 00:13 | |
| *** felipemonteiro_ has quit IRC | 00:13 | |
| *** kylek3h has joined #openstack-nova | 00:13 | |
| *** ZZelle has quit IRC | 00:14 | |
| *** yassine has joined #openstack-nova | 00:15 | |
| *** gfhellma has quit IRC | 00:17 | |
| *** gcb has quit IRC | 00:26 | |
| *** claudiub has quit IRC | 00:27 | |
| openstackgerrit | Logan V proposed openstack/nova master: Add LVM thin pool support https://review.openstack.org/442126 | 00:27 |
| *** dtp has quit IRC | 00:31 | |
| *** dimtruck is now known as zz_dimtruck | 00:32 | |
| *** hongbin has quit IRC | 00:34 | |
| *** zz_dimtruck is now known as dimtruck | 00:35 | |
| *** amotoki has joined #openstack-nova | 00:38 | |
| *** zsli has joined #openstack-nova | 00:42 | |
| *** zsli has quit IRC | 00:42 | |
| *** Shunli has joined #openstack-nova | 00:42 | |
| *** amotoki has quit IRC | 00:43 | |
| *** baoli has joined #openstack-nova | 00:47 | |
| *** mriedem has joined #openstack-nova | 00:49 | |
| *** baoli has quit IRC | 00:53 | |
| *** salv-orlando has joined #openstack-nova | 00:53 | |
| *** ducttape_ has joined #openstack-nova | 00:54 | |
| *** ducttape_ has quit IRC | 00:56 | |
| *** ducttape_ has joined #openstack-nova | 00:57 | |
| *** salv-orlando has quit IRC | 00:58 | |
| *** sdague has quit IRC | 01:01 | |
| *** zsli_ has joined #openstack-nova | 01:01 | |
| *** iceyao has joined #openstack-nova | 01:03 | |
| *** namnh has joined #openstack-nova | 01:03 | |
| *** gaoyuan has quit IRC | 01:04 | |
| *** Shunli has quit IRC | 01:04 | |
| *** huanxie has joined #openstack-nova | 01:04 | |
| *** zhurong has joined #openstack-nova | 01:06 | |
| *** iceyao has quit IRC | 01:08 | |
| *** ducttape_ has quit IRC | 01:08 | |
| *** gongysh has joined #openstack-nova | 01:09 | |
| *** dimtruck is now known as zz_dimtruck | 01:11 | |
| *** zz_dimtruck is now known as dimtruck | 01:13 | |
| *** liangy has quit IRC | 01:15 | |
| *** thorst has joined #openstack-nova | 01:17 | |
| *** thorst has quit IRC | 01:18 | |
| *** gongysh has quit IRC | 01:23 | |
| *** yuntongjin has joined #openstack-nova | 01:26 | |
| *** gongysh has joined #openstack-nova | 01:26 | |
| *** diga has quit IRC | 01:26 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove unused placement_database config options https://review.openstack.org/442035 | 01:26 |
| *** Apoorva has quit IRC | 01:26 | |
| *** thorst has joined #openstack-nova | 01:27 | |
| *** lucasxu has joined #openstack-nova | 01:28 | |
| *** ducttape_ has joined #openstack-nova | 01:32 | |
| *** trinaths has joined #openstack-nova | 01:33 | |
| *** trinaths has quit IRC | 01:34 | |
| *** unicell has quit IRC | 01:34 | |
| *** gongysh has quit IRC | 01:34 | |
| *** trinaths has joined #openstack-nova | 01:34 | |
| *** gongysh has joined #openstack-nova | 01:35 | |
| *** trinaths1 has joined #openstack-nova | 01:35 | |
| *** kevinz has joined #openstack-nova | 01:38 | |
| *** trinaths has quit IRC | 01:39 | |
| *** iceyao has joined #openstack-nova | 01:43 | |
| *** ducttape_ has quit IRC | 01:47 | |
| *** armax has quit IRC | 01:54 | |
| *** salv-orlando has joined #openstack-nova | 01:55 | |
| *** salv-orlando has quit IRC | 01:59 | |
| *** edmondsw has joined #openstack-nova | 02:00 | |
| *** edmondsw has quit IRC | 02:00 | |
| *** edmondsw has joined #openstack-nova | 02:00 | |
| *** Apoorva has joined #openstack-nova | 02:00 | |
| *** zsli__ has joined #openstack-nova | 02:01 | |
| *** dimtruck is now known as zz_dimtruck | 02:02 | |
| *** zsli_ has quit IRC | 02:03 | |
| *** gaoyuan has joined #openstack-nova | 02:04 | |
| *** awaugama has quit IRC | 02:05 | |
| *** gaoyuan has quit IRC | 02:08 | |
| *** takashin has joined #openstack-nova | 02:09 | |
| *** mtanino_ has quit IRC | 02:09 | |
| *** crusio has joined #openstack-nova | 02:12 | |
| *** erlon has quit IRC | 02:15 | |
| openstackgerrit | Dan Smith proposed openstack/nova master: Make CellMapping use SensitiveStringField https://review.openstack.org/442235 | 02:15 |
| *** markvoelker has quit IRC | 02:16 | |
| *** ducttape_ has joined #openstack-nova | 02:16 | |
| *** zz_dimtruck is now known as dimtruck | 02:17 | |
| *** ducttape_ has quit IRC | 02:18 | |
| *** nic has quit IRC | 02:20 | |
| *** yuntongjin has quit IRC | 02:23 | |
| openstackgerrit | Alex Xu proposed openstack/nova master: placement: Add Trait and TraitList objects https://review.openstack.org/376199 | 02:28 |
| openstackgerrit | Alex Xu proposed openstack/nova master: placement: Add simple cache object for traits https://review.openstack.org/441829 | 02:28 |
| *** huanxie has quit IRC | 02:32 | |
| *** thorst has quit IRC | 02:33 | |
| *** thorst has joined #openstack-nova | 02:33 | |
| *** tbachman has quit IRC | 02:37 | |
| *** thorst has quit IRC | 02:38 | |
| *** amotoki has joined #openstack-nova | 02:39 | |
| *** yuntongjin has joined #openstack-nova | 02:40 | |
| *** dimtruck is now known as zz_dimtruck | 02:41 | |
| *** yuntongjin has quit IRC | 02:42 | |
| *** zhs__ has joined #openstack-nova | 02:43 | |
| *** amotoki has quit IRC | 02:44 | |
| *** zhs_ has quit IRC | 02:46 | |
| *** oomichi has quit IRC | 02:50 | |
| *** Apoorva has quit IRC | 02:50 | |
| *** oomichi has joined #openstack-nova | 02:50 | |
| *** diga has joined #openstack-nova | 02:54 | |
| *** edmondsw has quit IRC | 02:55 | |
| *** edmondsw has joined #openstack-nova | 02:56 | |
| *** hongbin has joined #openstack-nova | 02:56 | |
| *** ducttape_ has joined #openstack-nova | 03:00 | |
| *** edmondsw has quit IRC | 03:01 | |
| *** bkopilov has quit IRC | 03:11 | |
| *** trinaths1 has quit IRC | 03:11 | |
| *** phuongnh has joined #openstack-nova | 03:12 | |
| mriedem | dansmith: she fell asleep | 03:13 |
| dansmith | mriedem: laura? | 03:13 |
| *** tlian has quit IRC | 03:14 | |
| *** ducttape_ has quit IRC | 03:14 | |
| *** zhs_ has joined #openstack-nova | 03:14 | |
| *** lucasxu has quit IRC | 03:15 | |
| mriedem | yeah | 03:15 |
| mriedem | dansmith: also, i was wrong in https://review.openstack.org/#/c/441543/1/nova/scheduler/client/report.py@692 | 03:15 |
| *** sudipto has joined #openstack-nova | 03:15 | |
| *** sudipto_ has joined #openstack-nova | 03:15 | |
| mriedem | it took the diff between 1..2 to see it | 03:15 |
| mriedem | the lack of a test in ps1 showed that the exception we'd hit there wasn't even defined :( | 03:16 |
| *** zhs__ has quit IRC | 03:16 | |
| dansmith | mriedem: how does that mean that we report the good ones and not the bad ones? | 03:16 |
| dansmith | we just don't report anything right? | 03:16 |
| mriedem | we won't report anything | 03:17 |
| mriedem | it'll be an error | 03:17 |
| dansmith | yeah | 03:17 |
| dansmith | just seems like it'd be better for us to sanitize each thing and report | 03:17 |
| dansmith | in the case of traits we have to be graceful about being newer than placement | 03:17 |
| dansmith | might not be as much of a deal here, but still.. | 03:18 |
| *** ijw has quit IRC | 03:20 | |
| dansmith | mriedem: are you going to +W the latest? | 03:20 |
| mriedem | i believe so, | 03:20 |
| mriedem | but i'm going to mention the lack of actual functional testing of the report client and all of the mock-based tests concerns me | 03:21 |
| mriedem | because there is quite a bit of per-method mocking going on for error conditions and that could miss some things later, functional tests would be best | 03:21 |
| *** yuntongjin has joined #openstack-nova | 03:22 | |
| dansmith | did you seem my comments from the previous patch? | 03:22 |
| dansmith | I wish he hadn't abandoned that and re-used it for one of these, but oh well | 03:22 |
| dansmith | either way, I said the same thing.. I feel like the coverage is fine, but it seems like these are very unit-y tests | 03:23 |
| dansmith | however, placement kinda has to be considered external to nova so not having solid functional tests seems as resonable as not having them for cinder | 03:23 |
| *** lucasxu has joined #openstack-nova | 03:24 | |
| *** lucasxu has quit IRC | 03:24 | |
| *** fragatin_ has joined #openstack-nova | 03:24 | |
| mriedem | we do have functional tests for a lot of this report client stack though | 03:25 |
| *** bswartz has quit IRC | 03:25 | |
| mriedem | i think for the negative scenario we need a fake virt driver that returns garbage, which is easy enough | 03:26 |
| mriedem | i +Wed it just to keep it moving | 03:26 |
| mriedem | i think we can hammer out a functional test as a follow on | 03:26 |
| *** eliqiao_ is now known as eliqiao | 03:28 | |
| *** fragatina has quit IRC | 03:28 | |
| *** fragatin_ has quit IRC | 03:29 | |
| dansmith | mriedem: for the report stuff? like, it runs a placement? | 03:29 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: [POC] Searchlight intergration https://review.openstack.org/442248 | 03:30 |
| mriedem | i was thinking of self.url | 03:30 |
| mriedem | damn | 03:31 |
| mriedem | https://review.openstack.org/#/c/441543/2/nova/tests/functional/api/openstack/placement/test_report_client.py | 03:31 |
| *** armax has joined #openstack-nova | 03:31 | |
| mriedem | but that's not running a placement service | 03:31 |
| mriedem | well, actually i think it is | 03:31 |
| mriedem | it deploys a wsgi app | 03:31 |
| mriedem | i'm not sure how the wsgi_intercept stuff works | 03:31 |
| dansmith | is that different from this patch though? it uses wsgi_intercept too | 03:32 |
| *** Apoorva has joined #openstack-nova | 03:32 | |
| diga | mriedem: https://bugs.launchpad.net/nova/+bug/1670262 Getting this issues when setup fresh devstack & run server list command from cli | 03:32 |
| openstack | Launchpad bug 1670262 in OpenStack Compute (nova) "DBNonExistenceDatabase: Unknown database "nova_cell0"" [Undecided,Incomplete] | 03:32 |
| mriedem | diga: then create nova_cell0 | 03:32 |
| *** zz_dimtruck is now known as dimtruck | 03:32 | |
| *** tlian has joined #openstack-nova | 03:33 | |
| *** yuntongjin has quit IRC | 03:34 | |
| *** thorst has joined #openstack-nova | 03:34 | |
| mriedem | diga: https://github.com/openstack-dev/devstack/blob/8fe48af1625cd2deca496de81dd72573e78b3ef2/lib/nova#L709 | 03:34 |
| *** armax has quit IRC | 03:34 | |
| diga | mriedem: okay, will go through it | 03:35 |
| *** links has joined #openstack-nova | 03:35 | |
| diga | mriedem: may be my devstack source code is old | 03:35 |
| mriedem | dansmith: i'm actually not sure why he couldn't have just modified https://review.openstack.org/#/c/441543/2/nova/tests/functional/api/openstack/placement/test_report_client.py to call set_inventory_for_provider with a garbage resource class | 03:35 |
| diga | mriedem: need to pull latest | 03:35 |
| diga | mriedem: thank you | 03:35 |
| mriedem | and assert it raises the expected exception | 03:35 |
| mriedem | i can try that quick | 03:36 |
| *** yuntongjin has joined #openstack-nova | 03:38 | |
| *** thorst has quit IRC | 03:38 | |
| *** yamahata_ has joined #openstack-nova | 03:40 | |
| *** browne has quit IRC | 03:40 | |
| *** yamahata has quit IRC | 03:41 | |
| *** guchihiro has joined #openstack-nova | 03:42 | |
| *** alineb has quit IRC | 03:43 | |
| mriedem | dansmith: yeah this would have been much easier https://gist.github.com/mriedem/1fb38bd4bb62cff0c5bcd90abb049c88 | 03:45 |
| *** yamahata_ has quit IRC | 03:46 | |
| dansmith | oh well | 03:48 |
| *** fragatina has joined #openstack-nova | 03:49 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add functional test for bad res class in set_inventory_for_provider https://review.openstack.org/442254 | 03:49 |
| mriedem | dansmith: ^ oh nothing | 03:49 |
| dansmith | that's what we need, more patches | 03:50 |
| mriedem | "more good patches" | 03:50 |
| mriedem | was the BCN phrase of the week | 03:50 |
| *** dimtruck is now known as zz_dimtruck | 03:50 | |
| dansmith | heh | 03:51 |
| *** fragatina has quit IRC | 03:53 | |
| *** mdrabe has quit IRC | 03:54 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: autospec the virt driver mock in test_resource_tracker https://review.openstack.org/442258 | 03:54 |
| *** gouthamr has quit IRC | 03:55 | |
| *** yuntongjin1 has joined #openstack-nova | 03:55 | |
| *** salv-orlando has joined #openstack-nova | 03:55 | |
| *** thorst has joined #openstack-nova | 03:56 | |
| *** thorst has quit IRC | 03:56 | |
| *** yuntongjin has quit IRC | 03:56 | |
| mriedem | with that i'll call it a day | 03:56 |
| *** mriedem has quit IRC | 03:56 | |
| *** diga has quit IRC | 03:59 | |
| *** salv-orlando has quit IRC | 04:00 | |
| *** gaoyuan has joined #openstack-nova | 04:00 | |
| *** Apoorva_ has joined #openstack-nova | 04:01 | |
| *** Apoorva has quit IRC | 04:01 | |
| *** gaoyuan has quit IRC | 04:05 | |
| openstackgerrit | Sivasathurappan Radhakrishnan proposed openstack/nova master: Port binding based on events during live migration https://review.openstack.org/434870 | 04:14 |
| *** ducttape_ has joined #openstack-nova | 04:15 | |
| *** markvoelker has joined #openstack-nova | 04:16 | |
| *** yonglihe has quit IRC | 04:19 | |
| *** ducttape_ has quit IRC | 04:20 | |
| *** yamahata has joined #openstack-nova | 04:20 | |
| *** markvoelker has quit IRC | 04:21 | |
| *** psachin has joined #openstack-nova | 04:25 | |
| *** yuntongjin1 has quit IRC | 04:31 | |
| *** sudipto has quit IRC | 04:32 | |
| *** sudipto_ has quit IRC | 04:32 | |
| *** sudipto has joined #openstack-nova | 04:33 | |
| *** sudipto_ has joined #openstack-nova | 04:33 | |
| *** bkopilov has joined #openstack-nova | 04:34 | |
| *** Apoorva_ has quit IRC | 04:35 | |
| *** bswartz has joined #openstack-nova | 04:36 | |
| *** nicolasbock has quit IRC | 04:38 | |
| *** armax has joined #openstack-nova | 04:39 | |
| *** amotoki has joined #openstack-nova | 04:40 | |
| *** unicell has joined #openstack-nova | 04:40 | |
| *** unicell has quit IRC | 04:40 | |
| *** unicell has joined #openstack-nova | 04:40 | |
| *** karimb has joined #openstack-nova | 04:42 | |
| *** unicell has quit IRC | 04:43 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(1/2) https://review.openstack.org/408955 | 04:43 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2) https://review.openstack.org/408964 | 04:43 |
| *** sudipto_ has quit IRC | 04:43 | |
| *** sudipto has quit IRC | 04:43 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Add parameters in cold migrate action https://review.openstack.org/410042 | 04:44 |
| *** amotoki has quit IRC | 04:44 | |
| *** karimb has quit IRC | 04:44 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Add functional tests for cold migration to same host https://review.openstack.org/414926 | 04:44 |
| *** karimb has joined #openstack-nova | 04:45 | |
| *** hongbin has quit IRC | 04:45 | |
| *** udesale has joined #openstack-nova | 04:46 | |
| *** unicell has joined #openstack-nova | 04:47 | |
| *** mdnadeem has joined #openstack-nova | 04:55 | |
| *** Apoorva has joined #openstack-nova | 04:56 | |
| *** thorst has joined #openstack-nova | 04:57 | |
| *** trinaths has joined #openstack-nova | 04:57 | |
| *** trinaths1 has joined #openstack-nova | 05:00 | |
| *** ayogi has joined #openstack-nova | 05:00 | |
| *** trinaths1 has quit IRC | 05:01 | |
| *** trinaths has quit IRC | 05:02 | |
| *** thorst has quit IRC | 05:02 | |
| *** trinaths1 has joined #openstack-nova | 05:04 | |
| openstackgerrit | Alex Xu proposed openstack/nova master: placement: Add Trait and TraitList objects https://review.openstack.org/376199 | 05:05 |
| *** trinaths has joined #openstack-nova | 05:06 | |
| *** trinaths1 has quit IRC | 05:06 | |
| *** trinaths has quit IRC | 05:08 | |
| *** trinaths has joined #openstack-nova | 05:08 | |
| *** trinaths1 has joined #openstack-nova | 05:09 | |
| *** trinaths has quit IRC | 05:12 | |
| *** trinaths has joined #openstack-nova | 05:13 | |
| *** trinaths1 has quit IRC | 05:13 | |
| *** trinaths has quit IRC | 05:14 | |
| *** fragatina has joined #openstack-nova | 05:15 | |
| *** ducttape_ has joined #openstack-nova | 05:16 | |
| *** zhs_ has quit IRC | 05:17 | |
| *** fragatina has quit IRC | 05:19 | |
| *** ducttape_ has quit IRC | 05:21 | |
| *** fragatina has joined #openstack-nova | 05:28 | |
| *** claudiub has joined #openstack-nova | 05:29 | |
| *** fragatina has quit IRC | 05:29 | |
| *** fragatina has joined #openstack-nova | 05:29 | |
| *** fragatina has quit IRC | 05:32 | |
| *** fragatina has joined #openstack-nova | 05:32 | |
| *** fragatina has quit IRC | 05:32 | |
| *** fragatina has joined #openstack-nova | 05:33 | |
| *** salv-orlando has joined #openstack-nova | 05:39 | |
| *** sudipto has joined #openstack-nova | 05:39 | |
| *** sudipto_ has joined #openstack-nova | 05:39 | |
| *** rcernin has joined #openstack-nova | 05:39 | |
| *** prateek has joined #openstack-nova | 05:39 | |
| *** yuntongjin has joined #openstack-nova | 05:43 | |
| *** salv-orlando has quit IRC | 05:43 | |
| *** Jack_I has joined #openstack-nova | 05:44 | |
| *** ijw has joined #openstack-nova | 05:46 | |
| *** markvoelker has joined #openstack-nova | 05:46 | |
| *** tlian has quit IRC | 05:47 | |
| *** gaoyuan has joined #openstack-nova | 05:49 | |
| *** markvoelker has quit IRC | 05:50 | |
| *** Jeffrey4l has quit IRC | 05:52 | |
| *** wllabs has joined #openstack-nova | 05:53 | |
| *** gaoyuan has quit IRC | 05:53 | |
| wllabs | version 2.14 | 05:53 |
| wllabs | how to define version? | 05:54 |
| *** youzp has joined #openstack-nova | 05:54 | |
| wllabs | how to know my code's version | 05:54 |
| *** Jeffrey4l has joined #openstack-nova | 05:56 | |
| *** zsli__ has quit IRC | 05:58 | |
| *** thorst has joined #openstack-nova | 05:58 | |
| *** zsli__ has joined #openstack-nova | 05:59 | |
| *** CristinaPauna has quit IRC | 06:00 | |
| *** CristinaPauna has joined #openstack-nova | 06:01 | |
| wllabs | qinatov | 06:02 |
| wllabs | aignatov | 06:02 |
| *** thorst has quit IRC | 06:03 | |
| *** slaweq has quit IRC | 06:07 | |
| *** tovin07 is now known as tovin07_at_work | 06:08 | |
| *** sridharg has joined #openstack-nova | 06:11 | |
| *** Apoorva has quit IRC | 06:12 | |
| *** rcernin has quit IRC | 06:13 | |
| *** salv-orlando has joined #openstack-nova | 06:16 | |
| *** ducttape_ has joined #openstack-nova | 06:17 | |
| *** adisky_ has joined #openstack-nova | 06:19 | |
| *** ducttape_ has quit IRC | 06:22 | |
| *** gcb has joined #openstack-nova | 06:28 | |
| *** jvarlamova__ has joined #openstack-nova | 06:30 | |
| *** markvoelker has joined #openstack-nova | 06:36 | |
| *** guchihiro has quit IRC | 06:37 | |
| *** nkrinner_afk is now known as nkrinner | 06:40 | |
| *** amotoki has joined #openstack-nova | 06:41 | |
| jvarlamova__ | Hi everyone! Can someone help me? I get error "Failed to allocate the network(s), not rescheduling", when trying to boot multiple instances in parallel. What could be the reason? | 06:42 |
| *** ekuris has joined #openstack-nova | 06:43 | |
| *** moshele has joined #openstack-nova | 06:43 | |
| *** unicell1 has joined #openstack-nova | 06:43 | |
| *** unicell has quit IRC | 06:44 | |
| *** amotoki has quit IRC | 06:46 | |
| *** satyar has joined #openstack-nova | 06:47 | |
| *** ijw has quit IRC | 06:47 | |
| *** moshele has quit IRC | 06:47 | |
| *** moshele has joined #openstack-nova | 06:48 | |
| *** karimb has quit IRC | 06:48 | |
| *** namnh has quit IRC | 06:48 | |
| *** markvoelker has quit IRC | 06:49 | |
| *** tuanluong has joined #openstack-nova | 06:51 | |
| *** jaosorior has joined #openstack-nova | 06:54 | |
| *** jaosorior has quit IRC | 06:54 | |
| *** prateek has quit IRC | 06:55 | |
| *** tojuvone has quit IRC | 06:55 | |
| *** zsli__ has quit IRC | 06:56 | |
| *** zsli__ has joined #openstack-nova | 06:57 | |
| *** lpetrut has joined #openstack-nova | 06:58 | |
| *** prateek_ has joined #openstack-nova | 06:58 | |
| openstackgerrit | Akira KAMIO proposed openstack/nova master: Replace root volume during rebuild https://review.openstack.org/442295 | 06:58 |
| *** thorst has joined #openstack-nova | 06:59 | |
| *** markus_z has joined #openstack-nova | 07:00 | |
| *** thorst has quit IRC | 07:04 | |
| *** Guest27337 has joined #openstack-nova | 07:04 | |
| *** zsli__ has quit IRC | 07:08 | |
| *** andreas_s has joined #openstack-nova | 07:17 | |
| *** ducttape_ has joined #openstack-nova | 07:18 | |
| *** salv-orl_ has joined #openstack-nova | 07:23 | |
| *** ducttape_ has quit IRC | 07:23 | |
| *** NightKhaos has quit IRC | 07:23 | |
| *** NightKhaos has joined #openstack-nova | 07:24 | |
| *** CristinaPauna has quit IRC | 07:24 | |
| *** CristinaPauna has joined #openstack-nova | 07:24 | |
| *** salv-orlando has quit IRC | 07:26 | |
| *** tojuvone has joined #openstack-nova | 07:26 | |
| *** namnh has joined #openstack-nova | 07:35 | |
| *** salv-orl_ has quit IRC | 07:35 | |
| *** armax has quit IRC | 07:36 | |
| *** gaoyuan has joined #openstack-nova | 07:37 | |
| *** jaosorior has joined #openstack-nova | 07:40 | |
| *** slaweq has joined #openstack-nova | 07:41 | |
| *** amotoki has joined #openstack-nova | 07:42 | |
| *** gaoyuan has quit IRC | 07:42 | |
| *** lpetrut has quit IRC | 07:43 | |
| *** gcb has quit IRC | 07:45 | |
| *** amotoki has quit IRC | 07:46 | |
| openstackgerrit | Danil Akhmetov proposed openstack/nova master: Use proper user and tenant in the owner section of libvirt.xml. https://review.openstack.org/399679 | 07:46 |
| *** tesseract has joined #openstack-nova | 07:47 | |
| *** unicell1 has quit IRC | 07:48 | |
| *** pcaruana has joined #openstack-nova | 07:50 | |
| *** william has joined #openstack-nova | 07:51 | |
| *** william is now known as Guest34860 | 07:51 | |
| openstackgerrit | Akira KAMIO proposed openstack/nova master: Replace root volume during rebuild https://review.openstack.org/442295 | 07:51 |
| *** salv-orlando has joined #openstack-nova | 07:53 | |
| *** claudiub|2 has joined #openstack-nova | 07:54 | |
| *** rcernin has joined #openstack-nova | 07:55 | |
| *** claudiub has quit IRC | 07:57 | |
| *** karthiks has quit IRC | 07:58 | |
| *** dosaboy has quit IRC | 07:58 | |
| bauzas | good morning Nova | 07:58 |
| *** gcb has joined #openstack-nova | 07:58 | |
| *** dosaboy has joined #openstack-nova | 07:59 | |
| *** thorst has joined #openstack-nova | 08:00 | |
| *** karthiks has joined #openstack-nova | 08:01 | |
| *** ralonsoh has joined #openstack-nova | 08:01 | |
| *** iceyao has quit IRC | 08:02 | |
| slaweq | hello | 08:03 |
| slaweq | I have a question about rebuild of instance | 08:03 |
| *** thorst has quit IRC | 08:04 | |
| slaweq | (I'm using Juno release still), so sometimes when I rebuild active instance it's rebuild fine but in libvirt logs I can see something like "Cannot open log file: '/var/log/libvirt/qemu/instance-002d6c52.log'" and instance is then shutdown by libvirt | 08:05 |
| *** ltomasbo|away is now known as ltomasbo | 08:05 | |
| slaweq | maybe someone of You got such issue already and know how to fix it? | 08:05 |
| openstackgerrit | jichenjc proposed openstack/nova master: Trivial Fix https://review.openstack.org/442310 | 08:06 |
| openstackgerrit | Akira KAMIO proposed openstack/nova master: Replace root volume during rebuild https://review.openstack.org/442295 | 08:08 |
| *** maciejjozefczyk has quit IRC | 08:08 | |
| *** iceyao has joined #openstack-nova | 08:10 | |
| *** lpetrut has joined #openstack-nova | 08:12 | |
| openstackgerrit | jichenjc proposed openstack/nova master: Remove extension in API layer https://review.openstack.org/442312 | 08:15 |
| *** amotoki has joined #openstack-nova | 08:16 | |
| *** Jeffrey4l_ has joined #openstack-nova | 08:30 | |
| *** Jeffrey4l has quit IRC | 08:34 | |
| *** Jeffrey4l_ has quit IRC | 08:37 | |
| *** amoralej|off is now known as amoralej | 08:38 | |
| *** Jeffrey4l_ has joined #openstack-nova | 08:38 | |
| *** jpena|off is now known as jpena | 08:43 | |
| *** sam_nowitzki has joined #openstack-nova | 08:49 | |
| *** yuntongjin has quit IRC | 08:50 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: [POC] Searchlight intergration https://review.openstack.org/442248 | 08:50 |
| *** jpena is now known as jpena|off | 08:57 | |
| *** karimb has joined #openstack-nova | 08:57 | |
| openstackgerrit | Akira KAMIO proposed openstack/nova master: Replace root volume during rebuild https://review.openstack.org/442295 | 08:59 |
| *** zzzeek has quit IRC | 09:00 | |
| *** thorst has joined #openstack-nova | 09:01 | |
| *** zzzeek has joined #openstack-nova | 09:01 | |
| *** derekh has joined #openstack-nova | 09:02 | |
| *** jpena|off is now known as jpena | 09:04 | |
| *** thorst has quit IRC | 09:05 | |
| *** fragatina has quit IRC | 09:10 | |
| *** lpetrut has quit IRC | 09:15 | |
| *** tanee is now known as tanee_away | 09:15 | |
| *** tanee_away is now known as tanee | 09:15 | |
| *** lpetrut has joined #openstack-nova | 09:17 | |
| *** yamahata has quit IRC | 09:19 | |
| *** ducttape_ has joined #openstack-nova | 09:19 | |
| *** derekh has quit IRC | 09:21 | |
| *** mvk has quit IRC | 09:21 | |
| *** faizy has joined #openstack-nova | 09:22 | |
| *** jcook has quit IRC | 09:23 | |
| *** sneti has quit IRC | 09:24 | |
| *** belmoreira has joined #openstack-nova | 09:24 | |
| *** ducttape_ has quit IRC | 09:25 | |
| *** gaoyuan has joined #openstack-nova | 09:25 | |
| *** vgadiraj has quit IRC | 09:25 | |
| *** bkopilov has quit IRC | 09:26 | |
| openstackgerrit | Akira KAMIO proposed openstack/nova master: Replace root volume during rebuild https://review.openstack.org/442295 | 09:28 |
| *** gongysh has quit IRC | 09:28 | |
| *** gongysh has joined #openstack-nova | 09:28 | |
| *** bkopilov has joined #openstack-nova | 09:29 | |
| *** gaoyuan has quit IRC | 09:30 | |
| *** gongysh has quit IRC | 09:30 | |
| *** ralonsoh_ has joined #openstack-nova | 09:31 | |
| *** jcook has joined #openstack-nova | 09:32 | |
| *** ralonsoh has quit IRC | 09:34 | |
| *** sneti has joined #openstack-nova | 09:35 | |
| *** vgadiraj has joined #openstack-nova | 09:36 | |
| *** derekh has joined #openstack-nova | 09:37 | |
| *** derekh has quit IRC | 09:41 | |
| *** derekh has joined #openstack-nova | 09:41 | |
| *** jaosorior is now known as jaosorior_brb | 09:43 | |
| openstackgerrit | Akira KAMIO proposed openstack/nova master: Replace root volume during rebuild https://review.openstack.org/442295 | 09:44 |
| *** tovin07_at_work has quit IRC | 09:46 | |
| *** kevinz has quit IRC | 09:46 | |
| *** aarefiev_afk is now known as aarefiev | 09:47 | |
| *** mvk has joined #openstack-nova | 09:52 | |
| *** ducnc has quit IRC | 09:55 | |
| *** udesale has quit IRC | 09:56 | |
| *** bkopilov has quit IRC | 09:56 | |
| *** nmathew has joined #openstack-nova | 10:00 | |
| *** thorst has joined #openstack-nova | 10:02 | |
| *** ralonsoh_ is now known as ralonsoh | 10:02 | |
| *** links has quit IRC | 10:05 | |
| *** thorst has quit IRC | 10:06 | |
| *** jed56_ has joined #openstack-nova | 10:07 | |
| mdbooth | lyarwood: What do you know of lvm thin pools? | 10:08 |
| *** bkopilov has joined #openstack-nova | 10:09 | |
| *** markvoelker has joined #openstack-nova | 10:10 | |
| *** ociuhandu has joined #openstack-nova | 10:10 | |
| *** namnh_ has joined #openstack-nova | 10:10 | |
| openstackgerrit | Akira KAMIO proposed openstack/nova master: Replace root volume during rebuild https://review.openstack.org/442295 | 10:13 |
| *** namnh has quit IRC | 10:13 | |
| lyarwood | mdbooth: not much, I've never really had the chance to use them tbh | 10:14 |
| mdbooth | lyarwood: k | 10:14 |
| lyarwood | mdbooth: we don't use them in the lvm imagebackend do we? | 10:14 |
| mdbooth | lyarwood: I'm just reviewing this patch: https://review.openstack.org/#/c/442126/2/nova/virt/libvirt/storage/lvm.py | 10:15 |
| mdbooth | I'm curious, because I thought we already did use them if we specified sparse | 10:15 |
| mdbooth | And this patch is adding something new | 10:15 |
| *** namnh_ has quit IRC | 10:16 | |
| mdbooth | I'm just reading some man pages to try to work out what's different from what we already do | 10:16 |
| *** cdent has joined #openstack-nova | 10:18 | |
| *** links has joined #openstack-nova | 10:19 | |
| markus_z | lyarwood: Anything I can do for the stable/newton backport https://review.openstack.org/#/c/406898/ ? I answered all questions to that so far. | 10:19 |
| *** owalsh-afk is now known as owalsh | 10:20 | |
| *** youzp has quit IRC | 10:20 | |
| sfinucan | johnthetubaguy: Could you take a look at https://review.openstack.org/#/c/435575? It's pretty noddy but it keeps getting into merge conflicts | 10:20 |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova-specs master: Enable SR-IOV NIC offload feature discovery https://review.openstack.org/435954 | 10:20 |
| lyarwood | markus_z: I was going to take another look at it today before matt gets online so we can hopefully close it out | 10:20 |
| lyarwood | markus_z: I think you've answered his questions but just wanted to check first | 10:20 |
| johnthetubaguy | sfinucan: oh, I can see what why would happen, yeah, looking | 10:21 |
| sfinucan | johnthetubaguy: Cheers :) | 10:21 |
| *** ducttape_ has joined #openstack-nova | 10:21 | |
| lyarwood | mdbooth: ah right, before we would just create a sparse LV and manually grow that LV (I *think*) | 10:21 |
| markus_z | lyarwood: OK, thanks. JFYI, I might leave early today ~1600UTC | 10:21 |
| lyarwood | at least, that's what RHEV would do back in the day with sparse LVs | 10:22 |
| mdbooth | johnthetubaguy: Following our discussion last week, I suspect that even with Cinder V3 we're going to end up doing this: https://review.openstack.org/#/c/383859/ | 10:22 |
| mdbooth | i.e. nova-side management of shared connections | 10:22 |
| *** satyar has quit IRC | 10:23 | |
| *** salv-orl_ has joined #openstack-nova | 10:23 | |
| mdbooth | johnthetubaguy: Any chance we could get that over the line? Fixes an issue today, I suspect the code can eventually be lifted elsewhere. | 10:24 |
| johnthetubaguy | mdbooth: given my queue of work today, I can't see me getting the time to look at that I am afraid | 10:24 |
| *** wllabs has quit IRC | 10:24 | |
| mdbooth | johnthetubaguy: Well it's been sitting for months, so I'm not going to stress about today :) | 10:25 |
| *** iceyao has quit IRC | 10:25 | |
| mdbooth | If you have a 'sometime soon' list, though, that would be awesome :) | 10:25 |
| johnthetubaguy | mdbooth: I understand it better now, having chatted through it, which is all good, yeah, its totally on that list | 10:25 |
| *** sambetts|afk is now known as sambetts | 10:25 | |
| mdbooth | Cool, thanks. | 10:25 |
| johnthetubaguy | mdbooth: if I haven't been to review that early next week, totally bug me again | 10:26 |
| *** ducttape_ has quit IRC | 10:26 | |
| mdbooth | johnthetubaguy: You may regret that. | 10:26 |
| mdbooth | Sure. | 10:26 |
| *** slaweq has quit IRC | 10:26 | |
| johnthetubaguy | mdbooth: probably, but its for the best :) | 10:26 |
| *** gcb has quit IRC | 10:26 | |
| *** salv-orlando has quit IRC | 10:27 | |
| *** salv-orl_ has quit IRC | 10:27 | |
| *** tanee is now known as tanee_away | 10:27 | |
| *** slaweq has joined #openstack-nova | 10:27 | |
| *** mvk has quit IRC | 10:28 | |
| openstackgerrit | Akira KAMIO proposed openstack/nova master: Replace root volume during rebuild https://review.openstack.org/442295 | 10:30 |
| gibi | johnthetubaguy: hi! a frindly reminder about the Show persisted scheduler hints in server details spec https://review.openstack.org/#/c/440580/ It gathered even more +1s yesterday :) | 10:30 |
| *** openstackgerrit has quit IRC | 10:33 | |
| *** iceyao has joined #openstack-nova | 10:34 | |
| * cdent puts that in his queue to add more +1s to | 10:34 | |
| *** openstackgerrit has joined #openstack-nova | 10:34 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Move 'floating_ips' opts into 'network' https://review.openstack.org/431039 | 10:34 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Deprecate 'default_floating_pool' https://review.openstack.org/431038 | 10:34 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Add neutron.floating_pool https://review.openstack.org/431037 | 10:34 |
| jvarlamova__ | Hi, can someone help me? I get error "VirtualInterfaceCreateException: Virtual Interface creation failed", when trying to boot multiple instances in parallel. What's the reason? | 10:35 |
| *** gszasz has joined #openstack-nova | 10:37 | |
| *** diga has joined #openstack-nova | 10:38 | |
| *** iceyao has quit IRC | 10:38 | |
| *** links has quit IRC | 10:39 | |
| rpodolyaka | jvarlamova__: could you please provide a complete stack trace from the nova-compute log? | 10:39 |
| *** sam_nowitzki has quit IRC | 10:39 | |
| *** cdent has quit IRC | 10:40 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: trivial: Standardize naming of variables https://review.openstack.org/430272 | 10:40 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: trivial: Standardize indentation of test_vif https://review.openstack.org/430271 | 10:40 |
| *** mvk has joined #openstack-nova | 10:41 | |
| *** Guest6667 is now known as Guest6666 | 10:43 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: libvirt: Make 'get_domain' private https://review.openstack.org/417378 | 10:45 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: console: Move proxies to 'console/proxy' https://review.openstack.org/408192 | 10:47 |
| *** jed56_ has quit IRC | 10:49 | |
| *** jed56_ has joined #openstack-nova | 10:49 | |
| *** claudiub|2 has quit IRC | 10:51 | |
| openstackgerrit | Akira KAMIO proposed openstack/nova master: Replace root volume during rebuild https://review.openstack.org/442295 | 10:53 |
| *** links has joined #openstack-nova | 10:55 | |
| *** thorst has joined #openstack-nova | 11:02 | |
| openstackgerrit | Evgeny Antyshev proposed openstack/nova master: get_model method missing for Ploop image https://review.openstack.org/441989 | 11:07 |
| *** thorst has quit IRC | 11:07 | |
| *** phuongnh has quit IRC | 11:09 | |
| *** ltomasbo is now known as ltomasbo|away | 11:09 | |
| jvarlamova__ | rpodolyaka: http://pastebin.com/0jY6Ax1C | 11:12 |
| *** gaoyuan has joined #openstack-nova | 11:13 | |
| *** jaosorior_brb is now known as jaosorior | 11:16 | |
| *** cdent has joined #openstack-nova | 11:17 | |
| *** gaoyuan has quit IRC | 11:18 | |
| *** edmondsw has joined #openstack-nova | 11:19 | |
| rpodolyaka | jvarlamova__: this exception is raised when nova-compute times out waiting for vif-plugged event - https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L5052-L5061 | 11:20 |
| *** diga has quit IRC | 11:20 | |
| *** nicolasbock has joined #openstack-nova | 11:21 | |
| rpodolyaka | jvarlamova__: you should check nova-api / neutron-server / neutron l2 agent logs for this device id (== instance uuid) to see if such event was delivered to nova at all | 11:21 |
| *** edmondsw has quit IRC | 11:23 | |
| *** sdague has joined #openstack-nova | 11:27 | |
| mdbooth | alex_xu: Hey, just thinking about your response on https://review.openstack.org/#/c/441829/4 about the cache.clear(). This is all running with the placement service, right? Are we guaranteed to only have 1 of those globally? | 11:27 |
| mdbooth | cdent: ^^^ ? | 11:27 |
| mdbooth | alex_xu: I'm just thinking that if a Trait is created through 1 service, then accessed through another, the cache on the second won't have been refreshed. | 11:28 |
| * cdent catches up | 11:28 | |
| mdbooth | However, I don't know how this is intended to be deployed, so this may not make sense. | 11:28 |
| cdent | mdbooth: there's only one placement database, but could be multiple placement api servers | 11:29 |
| mdbooth | cdent: That's what I would have expected. Thanks. | 11:29 |
| * mdbooth is firing from the hip here, and trying not to be a pita :) | 11:30 | |
| cdent | I'm a bit nonplussed about the traits and resource class caches | 11:30 |
| mdbooth | cdent: Explain | 11:30 |
| cdent | they are trying to optimize sql queries when some of the data is not in an rdbms | 11:30 |
| cdent | and thus they are...weird | 11:31 |
| cdent | I would have prefered the optimizations to come later | 11:31 |
| cdent | mdbooth: bonus points for "Explain" | 11:33 |
| mdbooth | :) | 11:33 |
| *** nmathew has quit IRC | 11:33 | |
| mdbooth | cdent: They look like they're getting all their data from the db | 11:34 |
| mdbooth | What did I miss? | 11:34 |
| mdbooth | For context: I never looked at this before yesterday | 11:34 |
| cdent | which "they" did you mean? the traits cache may be different (and I haven't looked closely at it yet) | 11:34 |
| mdbooth | 'they' == resource class and traits caches | 11:35 |
| mdbooth | cdent: Ah... Yes, some resource classes are hard-coded | 11:35 |
| mdbooth | I've slept since I read that | 11:35 |
| *** gaoyuan has joined #openstack-nova | 11:36 | |
| cdent | you sleep? | 11:36 |
| mdbooth | Don't tell anyone | 11:36 |
| *** sudipto has quit IRC | 11:36 | |
| *** sudipto_ has quit IRC | 11:36 | |
| jvarlamova__ | rpodolyaka: As I see from logs, neutron sent 'network-vif-plugged' event for that instance to nova, and there's message "Creating event network-vif -plugged ...for instance ..." in nova-api logs. Looks like nova received event. | 11:37 |
| *** kevinz has joined #openstack-nova | 11:39 | |
| *** iceyao has joined #openstack-nova | 11:39 | |
| alex_xu | mdbooth: ah...i got you, good point | 11:41 |
| alex_xu | mdbooth: cdent probably resource-class won't have such problem. the cache for resource-class is just for query single resource-class. | 11:45 |
| mdbooth | alex_xu: The negative lookup is going to be tough for the startswith match, I think. | 11:47 |
| cdent | alex_xu: is it possible to avoid having the cache through the initial implementation, and only add the cache at the end? | 11:48 |
| mdbooth | alex_xu: Whereas the other 2 are definite lookups: it exists or it doesn't, and you normally expect that it exists | 11:48 |
| cdent | building in cache from the outset almost always leads to harder to maintain code | 11:48 |
| mdbooth | Returning no values or some values doesn't necessarily mean that you're not out of date | 11:48 |
| * mdbooth doesn't have an immediate suggestion there. | 11:48 | |
| mdbooth | cdent: How often will we hit this thing? Would an index optimised for a startswith query be appropriate? | 11:50 |
| * cdent doesn't know yet | 11:50 | |
| *** markvoelker has quit IRC | 11:50 | |
| cdent | (thus my concerns about early optimizations and speculations before measuring, etc) | 11:50 |
| mdbooth | From reading the context, the expectation seems to be a lot. Whose was that expectation? | 11:50 |
| alex_xu | cdent: in the poc, actually I query the db directly. but I try to get the API "GET /traits?name=in:TRAIT1,TRAIT2" to use the cache, since that is the API used to validate whether the traits is valid or not | 11:51 |
| cdent | mdbooth: it will be alot, eventually | 11:51 |
| cdent | but that's based on abstractions, not measurement | 11:51 |
| mdbooth | Are we carefully avoiding the db throughout this whole code path? | 11:52 |
| alex_xu | wait, no, that is for "GET /traits?prefix=xxx", that can be from the db directly. "GET /traits?name=in:TRAIT1,TRAIT2" can get benefit from the single trait lookup | 11:52 |
| * cdent really needs to look at the code more closely | 11:53 | |
| cdent | but I haven't had a chance yet | 11:53 |
| mdbooth | alex_xu: Yeah, I think the concrete lookup is ok to cache. Either you get a hit (but wait... what if it was deleted?), or you don't, in which case you can refresh. | 11:53 |
| cdent | (been concentrating on the spec) | 11:53 |
| alex_xu | mdbooth: yea, if user query a non-existed trait, that will trigger a db call | 11:54 |
| mdbooth | alex_xu: Deleting a cached trait/resource class is also an interesting case. | 11:54 |
| mdbooth | Does the user have an expectation that after a delete call completes, a subsequent lookup will fail? | 11:55 |
| mdbooth | What if it doesn't? | 11:55 |
| mdbooth | I suspect that probably doesn't matter as much as the other case. | 11:55 |
| * cdent sighs | 11:56 | |
| alex_xu | yes, I remember I point out that problem from the resource class patch, but I forget what answer I got now | 11:56 |
| cdent | you guys are proving my point | 11:57 |
| cdent | first make it right | 11:57 |
| cdent | then add the cache | 11:57 |
| mdbooth | alex_xu: Would cdent's suggestion fly? | 11:57 |
| jvarlamova__ | rpodolyaka: by the way, some instances were created successfully while other got an exception. Is it a known issue of parallel instances creation? | 11:57 |
| alex_xu | cdent: all the cache? even for single trait lookup? | 11:58 |
| cdent | theoretically, yes: that's just good practice: make code that does the right thing | 11:59 |
| cdent | only once it is doign the right thing consider adding cache | 11:59 |
| cdent | the mix between custom traits and the os-traits library makes things a bit weird in all this | 11:59 |
| cdent | but since the number of traits is, let's say, less than 10,000 python can happily work with lists of strings of that size | 11:59 |
| cdent | but again: I haven't looked at the codet yet! :) | 12:00 |
| cdent | I just hear a lot of early optimizating, and feel like that's worth flagging before we get too caught up in the details of that when we don't even have a fully working api yet | 12:00 |
| *** rfolco has joined #openstack-nova | 12:00 | |
| alex_xu | cdent: yea, that sounds make sense. hope we can catch jay also. I feel he will say something when I remove the cache :) | 12:00 |
| cdent | I'm sure he will, jay is a notorious early optimizer :) | 12:01 |
| alex_xu | cdent: heh :) | 12:01 |
| mdbooth | alex_xu: Anyway, I'm going to remind you again that I don't have nearly enough context here. I can only point out potential problems and ask if they're real problems. | 12:01 |
| mdbooth | alex_xu: Don't mean to be a pita. | 12:02 |
| cdent | if what we're really concerned about is performance of the api then we should, _later_, consider a namespaced memcached layer on top of the placement api that allows marking collections as dirty | 12:02 |
| alex_xu | mdbooth: appreciate your review same, it is really helpful :) | 12:02 |
| cdent | alex_xu, mdbooth: I think we should have lots more of this type of conversation, so I think this is good | 12:02 |
| alex_xu | mdbooth: btw, the spec is at https://review.openstack.org/345138, in case you are interesting :) | 12:03 |
| *** yuntongjin has joined #openstack-nova | 12:03 | |
| mdbooth | alex_xu: Thanks, I should probably have read that first. | 12:03 |
| *** thorst has joined #openstack-nova | 12:03 | |
| *** yuntongjin has quit IRC | 12:03 | |
| cdent | I think jay's concerns are at the database layer, in which case we can address that by simply having all the traits in the same table, and isn't that being done when we sync from os-traits? | 12:04 |
| *** yuntongjin has joined #openstack-nova | 12:04 | |
| *** aysyd has joined #openstack-nova | 12:04 | |
| alex_xu | cdent: yes, all the traits in one table | 12:04 |
| cdent | then why have a cache at all? | 12:05 |
| alex_xu | by 'placement-manage os-traits sync' | 12:05 |
| alex_xu | cdent: I guess jay think the trait/resource class won't update frequently, then to same some db call... | 12:06 |
| cdent | -1 | 12:06 |
| cdent | :) | 12:06 |
| alex_xu | :) | 12:06 |
| cdent | with the resource class stuff the cache is because the classes are _not_ all int he same table | 12:06 |
| cdent | with traits, if they are, then... | 12:07 |
| *** udesale has joined #openstack-nova | 12:07 | |
| alex_xu | ah... | 12:07 |
| alex_xu | yea a abstraction layer | 12:07 |
| *** thorst has quit IRC | 12:08 | |
| *** yuntongjin has quit IRC | 12:08 | |
| *** yuntongjin has joined #openstack-nova | 12:08 | |
| *** salv-orlando has joined #openstack-nova | 12:09 | |
| alex_xu | that is why I put the traits sync in the cache before, now we pull that sync out.. | 12:09 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: List/show all server migration types https://review.openstack.org/430608 | 12:09 |
| cdent | mdbooth: in a round about way you have done a good deed. if your quota is one a day, you're done, good job, time for a beer. | 12:10 |
| *** tuanluong has quit IRC | 12:11 | |
| *** faizy has quit IRC | 12:12 | |
| *** bkopilov has quit IRC | 12:13 | |
| *** yassine has quit IRC | 12:15 | |
| *** yassine has joined #openstack-nova | 12:18 | |
| *** GK___1wm____SU has joined #openstack-nova | 12:22 | |
| *** dgonzalez has quit IRC | 12:22 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova-specs master: List/show all server migration types https://review.openstack.org/407237 | 12:22 |
| *** GK___1wm____SU has quit IRC | 12:23 | |
| *** markvoelker has joined #openstack-nova | 12:23 | |
| openstackgerrit | Danil Akhmetov proposed openstack/nova master: Use proper user and tenant in the owner section of libvirt.xml. https://review.openstack.org/399679 | 12:24 |
| *** catintheroof has joined #openstack-nova | 12:24 | |
| *** hferenc has quit IRC | 12:26 | |
| gibi | bauzas: Hi! thanks for looking at the scheduler hint API spec. I suggested a way to handle missing hints in the review. | 12:27 |
| rfolco | cdent, review_begging++ https://review.openstack.org/#/c/415885/ | 12:28 |
| *** salv-orlando has quit IRC | 12:29 | |
| cdent | rfolco: yup that's on my list for today | 12:31 |
| rfolco | cdent, thx :) | 12:31 |
| *** smatzek has joined #openstack-nova | 12:31 | |
| *** swebster has joined #openstack-nova | 12:32 | |
| lyarwood | mdbooth: have you played around with lvm2 and sparse files yet? | 12:32 |
| lyarwood | mdbooth: we are already creating thinp volumes AFAICT | 12:33 |
| *** smatzek_ has joined #openstack-nova | 12:33 | |
| lyarwood | mdbooth: http://paste.openstack.org/show/601745/ | 12:34 |
| openstackgerrit | Steven Webster proposed openstack/nova master: Update nova network info when doing rebuild for evacuate operation https://review.openstack.org/382853 | 12:34 |
| *** thorst has joined #openstack-nova | 12:34 | |
| *** smatzek has quit IRC | 12:37 | |
| *** owalsh is now known as owalsh-afk | 12:38 | |
| *** yuntongjin has quit IRC | 12:40 | |
| *** yuntongjin has joined #openstack-nova | 12:40 | |
| *** claudiub|2 has joined #openstack-nova | 12:42 | |
| *** prateek_ has quit IRC | 12:43 | |
| *** erlon has joined #openstack-nova | 12:43 | |
| *** prateek has joined #openstack-nova | 12:43 | |
| *** zhurong has quit IRC | 12:45 | |
| *** yuntongjin has quit IRC | 12:45 | |
| *** yuntongjin has joined #openstack-nova | 12:46 | |
| *** crusio has quit IRC | 12:47 | |
| *** crusio has joined #openstack-nova | 12:47 | |
| *** ayogi has quit IRC | 12:50 | |
| *** jed56_ has quit IRC | 12:50 | |
| *** lucasagomes is now known as lucas-hungry | 12:53 | |
| *** takashin has left #openstack-nova | 12:55 | |
| *** dave-mccowan has joined #openstack-nova | 12:56 | |
| *** jaypipes has joined #openstack-nova | 12:59 | |
| *** edmondsw has joined #openstack-nova | 13:00 | |
| *** faizy has joined #openstack-nova | 13:01 | |
| *** tblakes has joined #openstack-nova | 13:01 | |
| *** kevinz has quit IRC | 13:02 | |
| *** kevinz has joined #openstack-nova | 13:03 | |
| *** ijw has joined #openstack-nova | 13:05 | |
| *** edmondsw has quit IRC | 13:06 | |
| *** ijw has quit IRC | 13:06 | |
| *** diegows has quit IRC | 13:07 | |
| *** edmondsw has joined #openstack-nova | 13:08 | |
| *** sudipto has joined #openstack-nova | 13:09 | |
| *** sudipto_ has joined #openstack-nova | 13:09 | |
| *** ltomasbo|away is now known as ltomasbo | 13:10 | |
| *** ijw_ has joined #openstack-nova | 13:12 | |
| *** eharney has quit IRC | 13:12 | |
| *** edmondsw has quit IRC | 13:12 | |
| *** jsheeren has joined #openstack-nova | 13:14 | |
| *** edmondsw has joined #openstack-nova | 13:14 | |
| *** edmondsw has quit IRC | 13:14 | |
| *** edmondsw has joined #openstack-nova | 13:15 | |
| jsheeren | hi, i'm having issues with nova-spiceproxy and spice-html5 using Windows instances. Everything is upside down, which makes it a little difficult to use the horizon console. Has anyone seen this as well? | 13:15 |
| jsheeren | with using spice-html5 from git, i'm seeing the same upside down console | 13:16 |
| *** ijw_ has quit IRC | 13:16 | |
| *** gaoyuan has quit IRC | 13:16 | |
| jsheeren | with linux instances there is no problem at all, spice console works as expected | 13:16 |
| *** kevinz has quit IRC | 13:18 | |
| *** kevinz has joined #openstack-nova | 13:19 | |
| *** faizy has quit IRC | 13:20 | |
| *** Drankis has joined #openstack-nova | 13:21 | |
| mnaser | hi nova team, is there plans of cutting a stable/newton release soon? | 13:22 |
| mnaser | we had a lot of problems with upgrades re this: https://review.openstack.org/#/c/408725/ -- we manually worked around it but it would be nice to not have to | 13:22 |
| *** liverpooler has joined #openstack-nova | 13:23 | |
| *** ducttape_ has joined #openstack-nova | 13:23 | |
| *** bkopilov has joined #openstack-nova | 13:24 | |
| *** salv-orlando has joined #openstack-nova | 13:28 | |
| *** ducttape_ has quit IRC | 13:28 | |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova master: Use Cinder API v3 as default https://review.openstack.org/420201 | 13:28 |
| *** bkopilov has quit IRC | 13:29 | |
| *** hoangcx has joined #openstack-nova | 13:32 | |
| *** zhurong has joined #openstack-nova | 13:35 | |
| *** salv-orl_ has joined #openstack-nova | 13:36 | |
| *** yuntongjin has quit IRC | 13:36 | |
| *** yuntongjin has joined #openstack-nova | 13:37 | |
| *** jsheeren has quit IRC | 13:38 | |
| *** claudiub|2 is now known as claudiub | 13:38 | |
| *** salv-orlando has quit IRC | 13:39 | |
| *** bkopilov has joined #openstack-nova | 13:41 | |
| *** gouthamr has joined #openstack-nova | 13:43 | |
| *** cleong has joined #openstack-nova | 13:46 | |
| *** salv-orl_ has quit IRC | 13:46 | |
| *** youzp has joined #openstack-nova | 13:48 | |
| *** ducttape_ has joined #openstack-nova | 13:49 | |
| *** lpetrut has quit IRC | 13:49 | |
| *** gaoyuan has joined #openstack-nova | 13:51 | |
| *** jpena is now known as jpena|lunch | 13:51 | |
| *** links has quit IRC | 13:51 | |
| *** esberglu has joined #openstack-nova | 13:51 | |
| Dinesh_Bhor | Vek: Hi, Could you please take a look at it? https://review.openstack.org/#/c/441750/ | 13:52 |
| *** youzp has quit IRC | 13:53 | |
| *** yuntongjin has quit IRC | 13:53 | |
| *** sudipto_ has quit IRC | 13:54 | |
| *** sudipto has quit IRC | 13:54 | |
| *** lucas-hungry is now known as lucasagomes | 13:54 | |
| *** amoralej is now known as amoralej|lunch | 13:54 | |
| openstackgerrit | Ed Leafe proposed openstack/nova master: Add test for inventory with invalid ResourceClass https://review.openstack.org/442498 | 13:55 |
| *** crushil has joined #openstack-nova | 13:55 | |
| *** gszasz has quit IRC | 13:56 | |
| *** prateek has quit IRC | 13:57 | |
| tdurakov | paul-carlton2 eliqiao shaohe_feng pkoniszewski yuntongjin mdbooth kashyap kmartin claudiub tobasco tdurakov abhishekk mrhillsman raj_singh wznoinsk siva_krishnan live migration subteam meeting in 2 minutes on #openstack-meeting-3 | 13:59 |
| *** CristinaPauna has quit IRC | 13:59 | |
| *** gouthamr has quit IRC | 13:59 | |
| *** owalsh-afk is now known as owalsh | 13:59 | |
| *** gszasz has joined #openstack-nova | 14:01 | |
| *** gouthamr has joined #openstack-nova | 14:02 | |
| *** yingjun has joined #openstack-nova | 14:03 | |
| *** mdnadeem has quit IRC | 14:06 | |
| cdent | edleafe gets the pedant of the day award | 14:09 |
| * bauzas has a paperwork day :( | 14:09 | |
| edleafe | cdent: After all, I used to head the Department of Redundancy Department | 14:09 |
| *** lpetrut has joined #openstack-nova | 14:09 | |
| *** mdrabe has joined #openstack-nova | 14:10 | |
| cdent | You sure you weren't the head chairperson? | 14:10 |
| *** gaoyuan has quit IRC | 14:10 | |
| sfinucan | sdague: Could I ask you to take a look at this patch and dependencies? There are some merge conflicts in later patches and I don't want to push a 15 patch series rebase again https://review.openstack.org/#/c/431022 | 14:10 |
| sfinucan | it's focused on deprecating remaining nova-net options (of which there are many) | 14:11 |
| *** CristinaPauna has joined #openstack-nova | 14:12 | |
| sdague | sfinucan: ok, I guess I wonder why it's in 8 patches? | 14:12 |
| *** ducttape_ has quit IRC | 14:12 | |
| sfinucan | sdague: I figured that it would be easier to review in chunks. I can combine if not | 14:12 |
| *** yuntongjin has joined #openstack-nova | 14:13 | |
| markus_z | Anyone here has a vagrant multinode setup with Ubuntu1604 and likes to share it? | 14:14 |
| sdague | honestly, I think that all the deprecations in nova/conf/network.py as one patch might make it simpler | 14:15 |
| *** xyang1 has joined #openstack-nova | 14:15 | |
| sfinucan | sdague: Sure thing. Gimme two | 14:15 |
| sdague | I also threw a quick comment in here - https://review.openstack.org/#/c/430914/3 | 14:15 |
| sdague | just took me a second to realize how that could be not used any more | 14:15 |
| *** zhurong has quit IRC | 14:16 | |
| *** youzp has joined #openstack-nova | 14:17 | |
| *** mlavalle has joined #openstack-nova | 14:17 | |
| *** ociuhandu has quit IRC | 14:18 | |
| *** hamzy has quit IRC | 14:21 | |
| sfinucan | sdague: Ah, I remember why I broke it up. The options were used by different components of nova-network and I gave that reason in the commit message (to avoid forcing people to go root that out themselves) | 14:21 |
| sdague | sfinucan: ok, well it could just be a long commit message | 14:21 |
| sdague | I'm fine with long commit messages | 14:21 |
| sfinucan | Aye, that's what I'll do now :) | 14:21 |
| *** mriedem has joined #openstack-nova | 14:24 | |
| *** jaosorior has quit IRC | 14:24 | |
| *** ociuhandu has joined #openstack-nova | 14:29 | |
| *** satyar has joined #openstack-nova | 14:30 | |
| *** smatzek_ has quit IRC | 14:32 | |
| *** lucasagomes has quit IRC | 14:32 | |
| johnthetubaguy | lyarwood mdbooth: thanks for you great comments on this one, would be awesome if you could take another peak at that cinder API spec: https://review.openstack.org/#/c/373203/ | 14:32 |
| mdbooth | johnthetubaguy: Will do | 14:32 |
| *** yuntongjin has quit IRC | 14:32 | |
| *** yuntongjin has joined #openstack-nova | 14:33 | |
| lyarwood | johnthetubaguy: ack will try to later today | 14:33 |
| *** lucasagomes has joined #openstack-nova | 14:33 | |
| *** gabor_antal_ is now known as gabor_antal | 14:34 | |
| mriedem | edleafe: on https://review.openstack.org/#/c/442254/ i'm not using nova/tests/functional/api/openstack/placement | 14:35 |
| johnthetubaguy | lyarwood: mdbooth: awesome, thank you | 14:35 |
| mriedem | i'm testing the scheduler report client | 14:35 |
| mriedem | well the gabbits anyway | 14:35 |
| mriedem | edleafe: your test is testing the placement API directly, which is good, but the functional test i added is testing the report client, which is needed for the logic in there, as noted here https://review.openstack.org/#/c/441543/2/nova/tests/unit/scheduler/client/test_report.py@1242 | 14:37 |
| cdent | mriedem, edleafe yeah seems we need both | 14:37 |
| cdent | s/need/should have/ | 14:37 |
| *** liangy has joined #openstack-nova | 14:38 | |
| *** liangy has quit IRC | 14:38 | |
| *** liangy has joined #openstack-nova | 14:39 | |
| *** nmathew has joined #openstack-nova | 14:41 | |
| *** nmathew has quit IRC | 14:42 | |
| openstackgerrit | Rafael Folco proposed openstack/nova-specs master: DELETE all inventories for a resource provider https://review.openstack.org/415885 | 14:42 |
| mriedem | sfinucan: we need to get this backported to ocata - breaks people using libvirt+xen that upgrade to ocata if they have new enough libvirt/qemu for virtlogd https://review.openstack.org/#/c/442209/ | 14:44 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Fix indentation in conf/netconf https://review.openstack.org/430717 | 14:44 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: netutils: Ignore 'use_ipv6' for network templates https://review.openstack.org/430910 | 14:44 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Deprecate 'use_ipv6' https://review.openstack.org/430911 | 14:44 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: libvirt: Ignore 'use_ipv6' for port filters https://review.openstack.org/430909 | 14:44 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Deprecate 'allow_same_net_traffic' https://review.openstack.org/430913 | 14:44 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Final cleanups in conf/network https://review.openstack.org/431023 | 14:44 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: libvirt: Ignore 'allow_same_net_traffic' for port filters https://review.openstack.org/430912 | 14:44 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Deprecate most 'network' option https://review.openstack.org/430914 | 14:44 |
| mdbooth | kaboom! | 14:44 |
| mriedem | i'd like to get that backported to ocata and released before a lot of people start upgrading | 14:44 |
| sfinucan | sdague: ^^^ | 14:44 |
| sfinucan | mriedem: Sure. I'll take a look now | 14:44 |
| edleafe | mriedem: not sure I follow. The test you added is in ova/tests/functional/api/openstack/placement/test_report_client.py | 14:46 |
| mriedem | edleafe: yeah which is a functional test on the scheduler report client | 14:46 |
| mriedem | which is what we need | 14:46 |
| mriedem | edleafe: as noted in the review, the functional test is testing the entire report client stack for that negative scenario, rather than just piecemeal unit tests - which are removed in that same change | 14:47 |
| mriedem | because the unit tests can lie | 14:47 |
| *** jpena|lunch is now known as jpena | 14:47 | |
| * edleafe is on a call and slow to respond | 14:47 | |
| mriedem | bauzas: i plan on backport https://review.openstack.org/#/c/442035/ to ocata, do you have any issue with that? | 14:49 |
| mriedem | *backporting | 14:49 |
| *** mtanino has joined #openstack-nova | 14:49 | |
| *** kevinz has quit IRC | 14:49 | |
| ildikov | smcginnis: FYI, I added a few extra characters and whitespaces to that reno and now the test seems happy... :) | 14:49 |
| smcginnis | ildikov: Oh good. I should take a look at how you did that. I wasn't able to get it to work. | 14:50 |
| smcginnis | (and was too lazy to really figure it out) :) | 14:50 |
| mriedem | smcginnis: you need the | for rst formatting | 14:50 |
| sfinucan | mriedem: Yeah, that fix makes sense. Done | 14:51 |
| *** jaosorior has joined #openstack-nova | 14:51 | |
| mriedem | sfinucan: thanks | 14:51 |
| mriedem | smcginnis: https://docs.openstack.org/developer/reno/usage.html#note-file-syntax | 14:51 |
| mdbooth | mriedem: Speaking of backports, you had a W-1 on this Newton backport until the Ocata backport merged, which it now has: https://review.openstack.org/#/c/431540/ | 14:51 |
| ildikov | mriedem: smcginnis: that was my guess too, but wasn't 100% sure | 14:51 |
| smcginnis | ildikov, mriedem: So use a | and guve a little extra indent. | 14:51 |
| sfinucan | mriedem: Someone should probably submit a patch to reno to use '|' in the default templates | 14:51 |
| *** gszasz has quit IRC | 14:51 | |
| mriedem | sfinucan: go for it | 14:51 |
| mriedem | i always use | | 14:51 |
| mriedem | regardless of whether i need it or not | 14:51 |
| ildikov | smcginnis: I think the extra indent wasn't necessary | 14:52 |
| ildikov | mriedem: +1 | 14:52 |
| *** arne_r has joined #openstack-nova | 14:52 | |
| smcginnis | ildikov: That link helps. I know I've read it before, but must have missed that detail. | 14:52 |
| *** yuntongjin has quit IRC | 14:52 | |
| ildikov | smcginnis: it's a pretty easy one to miss like how I did yesterday | 14:52 |
| smcginnis | ildikov: Definitely. | 14:53 |
| ildikov | smcginnis: but the tests look good for that patch overall, so I'm optimistic about v3 :) | 14:53 |
| sfinucan | mriedem: Ah, someone already has. Good to know | 14:53 |
| smcginnis | ildikov: It looked pretty good other than the reno failure last ime. | 14:54 |
| ildikov | smcginnis: yep, so the base v3 should be there any minute and then we can figure out the microversion part | 14:54 |
| mdbooth | sfinucan: Thanks for the re-review of the nfs patch. It's now on top of this one, btw: https://review.openstack.org/#/c/438970/ | 14:55 |
| smcginnis | ildikov: Great first step to build off of. | 14:55 |
| ildikov | smcginnis: and will need a new cinderclient for that too IIRC | 14:55 |
| *** amoralej|lunch is now known as amoralej | 14:55 | |
| *** ducttape_ has joined #openstack-nova | 14:55 | |
| *** smatzek_ has joined #openstack-nova | 14:55 | |
| *** youzp has quit IRC | 14:55 | |
| mdbooth | This had a bit of discussion last week in the context of the Cinder v3 stuff. | 14:55 |
| smcginnis | ildikov: I'm still waiting on one final fix before a new release. I'll try to push on that one. | 14:55 |
| sfinucan | mdbooth: Ah - missed that. I'll take a look at that too shortly | 14:55 |
| mdbooth | sfinucan: Thanks | 14:55 |
| ildikov | smcginnis: cool, tnx | 14:55 |
| *** udesale has quit IRC | 14:57 | |
| *** zz_dimtruck is now known as dimtruck | 14:58 | |
| *** sandanar has joined #openstack-nova | 14:59 | |
| *** ducttape_ has quit IRC | 14:59 | |
| mriedem | mdbooth: lyarwood: on https://review.openstack.org/#/c/431540/ | 15:00 |
| mriedem | so you've got this nice tempest test https://review.openstack.org/#/c/427808/ | 15:00 |
| mriedem | but the problem is that doesn't run on stable/newton | 15:00 |
| *** owalsh has quit IRC | 15:00 | |
| mriedem | because devstack on newton doesn't enable swap volume testing | 15:00 |
| mriedem | i think that's because we needed that other pivot patch | 15:01 |
| mriedem | but i think that's merged | 15:01 |
| *** hoangcx has quit IRC | 15:02 | |
| lyarwood | mriedem: true, so I need to backport the devstack change to stable/newton before this lands right? | 15:02 |
| mriedem | this one https://review.openstack.org/#/c/427290/ | 15:02 |
| mriedem | ^ is what we needed to fix swap volume right? | 15:02 |
| lyarwood | mriedem: yeah | 15:02 |
| *** nicolasbock has quit IRC | 15:03 | |
| *** hurricanerix has joined #openstack-nova | 15:03 | |
| *** gszasz has joined #openstack-nova | 15:04 | |
| mriedem | ok hold up there were actually 2 changes in devstack | 15:04 |
| *** hurricanerix has quit IRC | 15:04 | |
| *** cdent has quit IRC | 15:05 | |
| mdbooth | mriedem: You're an encyclopaedia(sp?) of CI knowledge | 15:05 |
| mriedem | i will take that as a compliment | 15:05 |
| lyarwood | mriedem: are you cp'ing them back to newton? | 15:06 |
| lyarwood | mriedem: happy to if you don't have time | 15:06 |
| mriedem | lyarwood: https://review.openstack.org/#/c/442548/ | 15:06 |
| *** hurricanerix has joined #openstack-nova | 15:06 | |
| *** youzp has joined #openstack-nova | 15:07 | |
| lyarwood | mriedem: thanks | 15:07 |
| *** hongbin has joined #openstack-nova | 15:07 | |
| *** hamzy has joined #openstack-nova | 15:09 | |
| *** pcaruana has quit IRC | 15:09 | |
| *** youzp has quit IRC | 15:11 | |
| *** yingjun has quit IRC | 15:11 | |
| *** lucasxu has joined #openstack-nova | 15:11 | |
| *** baoli has joined #openstack-nova | 15:12 | |
| *** baoli_ has joined #openstack-nova | 15:13 | |
| mriedem | lyarwood: btw i think i'll plan a stable/ocata release for early next week | 15:13 |
| *** arne_r has quit IRC | 15:14 | |
| lyarwood | mriedem: cool, I'll try to go over the open queue a few times this week to clear it out | 15:15 |
| mriedem | lyarwood: btw, https://review.openstack.org/#/c/441457/ | 15:15 |
| mriedem | the ocata backport of that isn't merged yet :) | 15:15 |
| mriedem | maybe i should have -WIP'ed | 15:16 |
| lyarwood | really? | 15:16 |
| lyarwood | urgh | 15:16 |
| mriedem | https://review.openstack.org/#/c/441456/ | 15:16 |
| lyarwood | time to uninstall gertty | 15:16 |
| mriedem | despite being the fuel for intergalactic travel, no one uses spice | 15:16 |
| lyarwood | haha, true | 15:16 |
| lyarwood | I'll +W the ocata one now, I was sure I had already | 15:17 |
| *** owalsh has joined #openstack-nova | 15:17 | |
| mriedem | cool | 15:17 |
| *** baoli has quit IRC | 15:17 | |
| *** hurricanerix has quit IRC | 15:17 | |
| *** hurricanerix has joined #openstack-nova | 15:18 | |
| openstackgerrit | Roman Podoliaka proposed openstack/nova-specs master: Ironic: add support for instance RAID configuration https://review.openstack.org/408151 | 15:18 |
| *** andreas_s has quit IRC | 15:21 | |
| *** ducttape_ has joined #openstack-nova | 15:22 | |
| *** pcaruana has joined #openstack-nova | 15:23 | |
| *** armax has joined #openstack-nova | 15:23 | |
| *** ijw has joined #openstack-nova | 15:24 | |
| *** dgonzalez has joined #openstack-nova | 15:24 | |
| *** salv-orlando has joined #openstack-nova | 15:24 | |
| *** ekuris has quit IRC | 15:25 | |
| *** alexpilotti has joined #openstack-nova | 15:28 | |
| *** annegentle has joined #openstack-nova | 15:33 | |
| *** sandanar has quit IRC | 15:33 | |
| *** ducttape_ has quit IRC | 15:39 | |
| *** gaoyuan has joined #openstack-nova | 15:40 | |
| *** ducttape_ has joined #openstack-nova | 15:40 | |
| *** claudiub has quit IRC | 15:40 | |
| *** GK___1wm____SU has joined #openstack-nova | 15:41 | |
| *** tbachman has joined #openstack-nova | 15:41 | |
| *** ducttape_ has quit IRC | 15:42 | |
| *** ducttape_ has joined #openstack-nova | 15:42 | |
| *** psachin has quit IRC | 15:43 | |
| *** jed56_ has joined #openstack-nova | 15:44 | |
| *** jed56_ is now known as jed56 | 15:44 | |
| *** gaoyuan has quit IRC | 15:45 | |
| *** awaugama has joined #openstack-nova | 15:45 | |
| *** jed56 has quit IRC | 15:45 | |
| *** iceyao has quit IRC | 15:45 | |
| *** jed56 has joined #openstack-nova | 15:45 | |
| *** eharney has joined #openstack-nova | 15:45 | |
| *** GK___1wm____SU has quit IRC | 15:46 | |
| *** adrian_otto has joined #openstack-nova | 15:46 | |
| *** cdent has joined #openstack-nova | 15:49 | |
| *** awaugama has quit IRC | 15:52 | |
| *** nkrinner is now known as nkrinner_afk | 15:53 | |
| jaypipes | vdrok: good evening. so the first patch in the Ironic inventory series has merged: https://review.openstack.org/#/c/441543/ | 15:55 |
| vdrok | jaypipes: yup, saw that, thank you! | 15:55 |
| jaypipes | vdrok: the next in the series is here: https://review.openstack.org/#/c/441544/3 | 15:55 |
| jaypipes | vdrok: working on that now. | 15:55 |
| vdrok | I'm tacking those :) | 15:55 |
| jaypipes | vdrok: would be great to take the final patch in the series and start running your integration tests. | 15:56 |
| jaypipes | vdrok: final patch is a new functional test mriedem added: https://review.openstack.org/#/c/442254/1 | 15:56 |
| *** baoli has joined #openstack-nova | 15:56 | |
| mriedem | that's not the final patch is it? | 15:56 |
| mriedem | just a side thing | 15:57 |
| mriedem | little something something on the side | 15:57 |
| jaypipes | mriedem: oh, it's not dependent? | 15:57 |
| jaypipes | gotcha | 15:57 |
| jaypipes | vdrok: k, feel free to ignore that last patch ;) | 15:57 |
| jaypipes | vdrok: just test on 441544 :) | 15:57 |
| vdrok | gotcha | 15:58 |
| mriedem | jaypipes: right i just thought, while reviewing the base patch, that we could do an easy functional test to the invalid resource class case to avoid the mocked unit tests failing us later, | 15:58 |
| mriedem | it was easier to add than i thought so i pushed it up last night | 15:59 |
| *** baoli_ has quit IRC | 15:59 | |
| jaypipes | mriedem: cool, thanks very much for that :) | 15:59 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: virt: implement get_inventory() for Ironic https://review.openstack.org/441544 | 15:59 |
| jaypipes | mriedem: fixed pep8^ | 15:59 |
| *** hurricanerix has quit IRC | 15:59 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add test for inventory with invalid ResourceClass https://review.openstack.org/442498 | 15:59 |
| *** hurricanerix has joined #openstack-nova | 16:00 | |
| *** hieulq_ has joined #openstack-nova | 16:01 | |
| *** lpetrut has quit IRC | 16:03 | |
| *** Oku_OS is now known as Oku_OS-away | 16:05 | |
| *** awaugama has joined #openstack-nova | 16:05 | |
| mriedem | easy docs bug if someone wants to take it - will help out operators https://bugs.launchpad.net/nova/+bug/1670738 | 16:06 |
| openstack | Launchpad bug 1670738 in OpenStack Compute (nova) "man page needed for "nova-manage db online_data_migrations"" [Medium,Triaged] | 16:06 |
| mriedem | hmm, dansmith didn't we used to have a hook that ran online_data_migrations in the CI jobs and dump the results? | 16:07 |
| mriedem | i'm not seeing the hook here http://logs.openstack.org/52/440652/2/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/6883d38/logs/ | 16:07 |
| mriedem | or http://logs.openstack.org/52/440652/2/check/gate-grenade-dsvm-neutron-ubuntu-xenial/74bea3f/logs/ | 16:07 |
| *** ducttape_ has quit IRC | 16:07 | |
| dansmith | mriedem: in the grenade jobs yeah | 16:08 |
| mriedem | ok something regressed, i'll check it out | 16:08 |
| dansmith | it's in upgrade.sh, I just grepped | 16:08 |
| dansmith | nova/upgrade.sh | 16:08 |
| mriedem | that's not what i'm thinking of | 16:09 |
| mriedem | oh i'm thinking of this https://github.com/openstack/nova/blob/master/gate/post_test_hook.sh | 16:09 |
| *** ducttape_ has joined #openstack-nova | 16:09 | |
| *** ijw has quit IRC | 16:10 | |
| *** dave-mccowan has quit IRC | 16:10 | |
| mriedem | hmm, unrelated but http://logs.openstack.org/52/440652/2/check/gate-grenade-dsvm-neutron-ubuntu-xenial/74bea3f/logs/grenade.sh.txt.gz#_2017-03-02_20_38_16_615 | 16:11 |
| mriedem | nova-manage cell_v2 map_cell0: error: argument --database_connection: expected one argument | 16:11 |
| *** mlakat_ is now known as mlakat | 16:11 | |
| *** dave-mccowan has joined #openstack-nova | 16:11 | |
| mriedem | database_connection_url: command not found | 16:11 |
| mriedem | i see the issue | 16:13 |
| mriedem | https://review.openstack.org/442599 | 16:16 |
| mriedem | i'm not sure why we have the || true in there | 16:16 |
| mriedem | probably for the old bug where map_cell0 would return 1 | 16:16 |
| *** annegent_ has joined #openstack-nova | 16:16 | |
| *** markus_z has quit IRC | 16:16 | |
| openstackgerrit | Dan Smith proposed openstack/nova master: WIP: Get instance availability_zone without hitting the api db https://review.openstack.org/439754 | 16:17 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Enlighten server tags API about cells https://review.openstack.org/440856 | 16:17 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Teach HostAPI about cells https://review.openstack.org/442162 | 16:17 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Make os-availability-zones know about cells https://review.openstack.org/442163 | 16:17 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Target cell in super conductor operations https://review.openstack.org/438022 | 16:17 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Change MQ targeting to honor only what is in the context https://review.openstack.org/438023 | 16:17 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Make scheduler target cells to get compute node instance info https://review.openstack.org/439891 | 16:17 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Make consoleauth target the proper cell https://review.openstack.org/441461 | 16:17 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Add cell field to Destination object https://review.openstack.org/438024 | 16:17 |
| openstackgerrit | Dan Smith proposed openstack/nova master: WIP: Make conductor ask scheduler to limit migrates to same cell https://review.openstack.org/438025 | 16:17 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Teach simple_tenant_usage about cells https://review.openstack.org/441388 | 16:17 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Teach os-aggregates about cells https://review.openstack.org/442200 | 16:17 |
| *** GK___1wm____SU has joined #openstack-nova | 16:19 | |
| *** GK___1wm____SU has left #openstack-nova | 16:19 | |
| *** GK___1wm____SU has quit IRC | 16:19 | |
| *** annegentle has quit IRC | 16:19 | |
| *** kristian__ has joined #openstack-nova | 16:21 | |
| *** belmoreira has quit IRC | 16:22 | |
| *** tbachman has quit IRC | 16:22 | |
| *** crushil has quit IRC | 16:22 | |
| *** hieulq_ has quit IRC | 16:26 | |
| *** pcaruana has quit IRC | 16:26 | |
| *** tbachman has joined #openstack-nova | 16:29 | |
| *** rcernin has quit IRC | 16:30 | |
| mriedem | dansmith: the archive_deleted_rows post-test-hook only runs on the nova-next job i guess http://logs.openstack.org/52/440652/2/check/gate-tempest-dsvm-neutron-nova-next-full-ubuntu-xenial-nv/51f2ea4/logs/devstack-gate-post_test_hook.txt.gz | 16:30 |
| mriedem | just fyi | 16:30 |
| dansmith | hmm why is that? | 16:31 |
| *** owalsh has quit IRC | 16:32 | |
| *** Drankis has quit IRC | 16:32 | |
| mriedem | https://github.com/openstack-infra/project-config/commit/df865a4e9d6830abb553f077ce885d8493210efa | 16:33 |
| mriedem | i'm guessing we just put it there since it's a nova-specific job really, | 16:33 |
| mriedem | and was a fancy snowflake at the time, which is where we put things like that | 16:33 |
| dansmith | ah I see, because it evolved from the placement job/ | 16:34 |
| mriedem | https://review.openstack.org/#/c/376567/ says we could make it fatal | 16:34 |
| mriedem | could/should | 16:34 |
| mriedem | we could think about that now | 16:34 |
| mriedem | so if archive starts breaking then we fail the job | 16:34 |
| dansmith | yeah | 16:34 |
| *** ijw has joined #openstack-nova | 16:35 | |
| *** owalsh has joined #openstack-nova | 16:35 | |
| *** eharney_ has joined #openstack-nova | 16:36 | |
| *** gfhellma has joined #openstack-nova | 16:36 | |
| *** GK___1wm____SU has joined #openstack-nova | 16:36 | |
| dansmith | mriedem: why is there no single-create test in this? https://github.com/openstack/tempest/blob/master/tempest/api/compute/admin/test_auto_allocate_network.py | 16:37 |
| mriedem | there is | 16:37 |
| mriedem | https://github.com/openstack/tempest/blob/master/tempest/api/compute/admin/test_auto_allocate_network.py#L150 | 16:37 |
| *** markvoelker has quit IRC | 16:37 | |
| dansmith | mriedem: that tests no allocate right? | 16:37 |
| mriedem | correct | 16:37 |
| dansmith | but the multi one tests that there _is_ allocation | 16:38 |
| *** lucasxu has quit IRC | 16:38 | |
| mriedem | but we still create a server | 16:38 |
| *** tesseract has quit IRC | 16:38 | |
| mriedem | why would it matter for multi cell? | 16:38 |
| *** eharney has quit IRC | 16:38 | |
| dansmith | I don't know why it's failing, and was looking for a non-multi-create test to see why that works and multi doesn't | 16:38 |
| *** eharney_ is now known as eharney | 16:39 | |
| mriedem | log link me? | 16:39 |
| dansmith | changing it to one server doesn't make it pass, so it must not be related to multi-create, I was just curious | 16:39 |
| *** JoseMello has joined #openstack-nova | 16:39 | |
| dansmith | mriedem: http://logs.openstack.org/94/436094/12/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/051cffa/logs/testr_results.html.gz | 16:39 |
| *** lucasxu has joined #openstack-nova | 16:40 | |
| *** sridharg has quit IRC | 16:40 | |
| *** jamesdenton has joined #openstack-nova | 16:42 | |
| openstackgerrit | Ed Leafe proposed openstack/nova master: Add test for inventory with invalid ResourceClass https://review.openstack.org/442498 | 16:42 |
| dansmith | I'm not even sure what to look at | 16:43 |
| *** baoli has quit IRC | 16:46 | |
| mriedem | just look into my eyes and don't be afraid | 16:46 |
| *** baoli has joined #openstack-nova | 16:46 | |
| *** GK___1wm____SU has quit IRC | 16:46 | |
| mriedem | dansmith: so we're failing on https://github.com/openstack/tempest/blob/master/tempest/api/compute/admin/test_auto_allocate_network.py#L188 | 16:47 |
| mriedem | are we going to the proper cell to get the info_cache? | 16:47 |
| mriedem | that test creates 3 servers and waits for them all to be ACTIVE, | 16:47 |
| mriedem | then iterates over them and lists their addresses to make sure there was a nic assigned | 16:47 |
| dansmith | should be.. you think that's the only test that looks at stuff from info cache? | 16:48 |
| mriedem | so either the nic isn't getting assigned, or we're not pulling the info_cache properly | 16:48 |
| *** crushil has joined #openstack-nova | 16:48 | |
| mriedem | there should be other tests that do that for attaching interfaces | 16:48 |
| dansmith | yeah, and those appear to work | 16:48 |
| mriedem | i'm not sure what they assert though | 16:49 |
| mriedem | checking the n-cpu logs now | 16:49 |
| dansmith | mriedem: and if I boot a server manually I see an address | 16:49 |
| *** felipemonteiro_ has joined #openstack-nova | 16:49 | |
| dansmith | let me try with auto networks | 16:49 |
| mriedem | dansmith: with devstack? | 16:49 |
| dansmith | yeah | 16:49 |
| mriedem | dansmith: that's not the same, | 16:49 |
| mriedem | because your tenant in devstack has access to the public network already | 16:49 |
| mriedem | so we don't hit the auto-allocate paths | 16:49 |
| mriedem | in n-cou | 16:49 |
| dansmith | right, I'm saying we're pulling those nic details | 16:49 |
| mriedem | *n-cpu | 16:49 |
| dansmith | in the api | 16:49 |
| mriedem | yeah, but this specific test has a tenant which has no access to any existing networks | 16:50 |
| openstackgerrit | Dan Smith proposed openstack/nova master: WIP: Get instance availability_zone without hitting the api db https://review.openstack.org/439754 | 16:50 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Make os-availability-zones know about cells https://review.openstack.org/442163 | 16:50 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Change MQ targeting to honor only what is in the context https://review.openstack.org/438023 | 16:50 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Make scheduler target cells to get compute node instance info https://review.openstack.org/439891 | 16:50 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Add cell field to Destination object https://review.openstack.org/438024 | 16:50 |
| openstackgerrit | Dan Smith proposed openstack/nova master: WIP: Make conductor ask scheduler to limit migrates to same cell https://review.openstack.org/438025 | 16:50 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Teach os-aggregates about cells https://review.openstack.org/442200 | 16:50 |
| dansmith | right, I'm just saying I think api is pulling info cache from the right cell db because I see nic details normally | 16:50 |
| *** nicolasbock has joined #openstack-nova | 16:52 | |
| *** mdrabe has quit IRC | 16:55 | |
| mriedem | hmm http://logs.openstack.org/94/436094/12/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/051cffa/logs/screen-n-cpu.txt.gz#_2017-03-07_00_14_21_656 | 16:56 |
| mriedem | No network configured allocate_for_instance | 16:56 |
| mriedem | Instance network_info: |[]| | 16:56 |
| dansmith | is that before it goes to allocate or after? | 16:56 |
| *** yamahata has joined #openstack-nova | 16:56 | |
| mriedem | during | 16:57 |
| dansmith | once we're on the compute node, there's no cell switching involved, so I'm not sure why multicell has anything to do with this | 16:57 |
| dansmith | is there something we put into info cache from the api side on instance create or something? | 16:58 |
| dansmith | on my test system, I have nothing in cell0 for info caches, so it's not like we're creating them there by accident or something | 16:58 |
| mriedem | i don't see "Automatically allocating a network for project" | 16:58 |
| *** moshele has quit IRC | 16:58 | |
| dansmith | should that be logged by n-cpu? | 16:59 |
| mriedem | yeah | 16:59 |
| mriedem | https://github.com/openstack/nova/blob/master/nova/network/neutronv2/api.py#L1495 | 16:59 |
| dansmith | yeah | 17:00 |
| *** aarefiev is now known as aarefiev_afk | 17:00 | |
| mriedem | i'm also not seeing this https://github.com/openstack/nova/blob/master/nova/network/neutronv2/api.py#L1542 | 17:00 |
| mriedem | ^ is called via the api | 17:01 |
| mriedem | oh well i should look for that in the n-api logs | 17:01 |
| *** mdrabe has joined #openstack-nova | 17:02 | |
| mriedem | i don't see that in n-api logs either | 17:02 |
| mriedem | i wonder if somehow the auto_allocate flag isn't getting set on the NetworkRequest object? | 17:02 |
| dansmith | not sure why that would be.. we don't even persist that right? | 17:03 |
| *** GK___1wm____SU has joined #openstack-nova | 17:05 | |
| mriedem | we don't persist it | 17:06 |
| mriedem | are we somehow mangling what we pass between n-api and conductor? | 17:06 |
| mriedem | i would doubt it because otherwise we wouldn't have any tests that create a server on a specific nic passing | 17:07 |
| mriedem | which i believe is most of tempest | 17:07 |
| *** karimb has quit IRC | 17:07 | |
| mriedem | they all have tenant-isolated networks | 17:07 |
| *** GK___1wm____SU has quit IRC | 17:08 | |
| *** yamahata has quit IRC | 17:08 | |
| dansmith | mriedem: no mangling that I know of, and this all passes on the same stack of patches with normal devstack, AFAICT | 17:09 |
| mriedem | ok. notifications meeting atm and then i'll keep digging. | 17:11 |
| mriedem | we might need a debug logging patch | 17:11 |
| *** markvoelker has joined #openstack-nova | 17:12 | |
| *** ansiwen_ is now known as ansiwen | 17:12 | |
| dansmith | okay thanks for helping | 17:13 |
| *** jamesdenton has quit IRC | 17:14 | |
| mriedem | https://www.youtube.com/watch?v=RWc5gsU-eeY | 17:14 |
| mriedem | mtreinish: ^ | 17:15 |
| *** lucasxu has quit IRC | 17:16 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: spawn/destroy #3: TaskFlow https://review.openstack.org/438729 | 17:16 |
| dansmith | heh | 17:16 |
| openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: spawn/destroy #4: full flavor https://review.openstack.org/391288 | 17:16 |
| *** GK___1wm____SU has joined #openstack-nova | 17:17 | |
| efried | thorst adreznec esberglu Need re-+1 on https://review.openstack.org/#/c/438729/ (TaskFlow) please. | 17:17 |
| *** gfhellma_ has joined #openstack-nova | 17:18 | |
| *** gfhellma has quit IRC | 17:18 | |
| *** nic has joined #openstack-nova | 17:20 | |
| *** baoli has quit IRC | 17:21 | |
| *** gfhellma_ has quit IRC | 17:23 | |
| *** ltomasbo is now known as ltomasbo|away | 17:24 | |
| mdbooth | lyarwood: Before I looked, I thought there was a swap_volume flow where we simply replaced 1 volume with another, contents and all. However, looking now it appears we'll always attempt to rebase the contents of the old volume on to the new volume. Is that right, or am I just blind today? | 17:24 |
| lyarwood | mdbooth: we always rebase, even with the cinder retype flow | 17:25 |
| *** Jack_V has joined #openstack-nova | 17:25 | |
| mdbooth | Well the cinder retype flow is the one I knew did it | 17:26 |
| *** Apoorva has joined #openstack-nova | 17:26 | |
| mdbooth | I thought the crackpot flow didn't do it though, for some reason | 17:26 |
| mriedem | efried: you guys are using taskflow huh | 17:26 |
| lyarwood | mdbooth: yeah volume-update does it as well | 17:26 |
| efried | mriedem Yes. | 17:26 |
| mdbooth | lyarwood: Thanks | 17:26 |
| efried | That's... good, right mriedem ? | 17:26 |
| *** GK___1wm____SU has quit IRC | 17:26 | |
| mriedem | efried: we've never used it in nova | 17:26 |
| mriedem | i know cinder uses it, sort of, for some tihngs | 17:27 |
| mriedem | but never liked it enough to use it for all operations | 17:27 |
| efried | Oh. I was under the impression that e.g. libvirt used it. | 17:27 |
| *** jamesdenton has joined #openstack-nova | 17:27 | |
| mriedem | maybe it's fine on a small scale confied to a single virt driver | 17:27 |
| efried | mriedem We find it super useful for rollbacks. | 17:28 |
| *** gfhellma_ has joined #openstack-nova | 17:28 | |
| *** Jack_V has quit IRC | 17:28 | |
| *** Jack_I has quit IRC | 17:29 | |
| *** Jack_I has joined #openstack-nova | 17:29 | |
| mriedem | like spawn fails, detach interfaces and block devices? | 17:30 |
| *** jaosorior has quit IRC | 17:30 | |
| efried | yup | 17:30 |
| mriedem | dude you can totally just have a 300LOC mega method with lots of try/excepts :) | 17:30 |
| *** lucasxu has joined #openstack-nova | 17:31 | |
| efried | Hey, y'all are the ones who wanted smaller change sets. | 17:31 |
| efried | We'll bloat it later for ya. | 17:31 |
| *** GK___1wm____SU has joined #openstack-nova | 17:32 | |
| *** GK___1wm____SU has quit IRC | 17:32 | |
| * jaypipes dons headphones, prepares for an afternoon of reviews and coding... | 17:33 | |
| * dansmith turns up the music and prepares for all the people to solicit reviews from jaypipes | 17:33 | |
| mdbooth | jaypipes: Did somebody bust you out of some meetings? | 17:33 |
| jaypipes | dansmith: and yes, I'm listening to "Business Time" | 17:33 |
| dansmith | good man | 17:34 |
| cburgess | sdague: OMG thats awesome. I offered that up as a joke of the problems such a model could create. I love that its officially a rejected model for the reason that it makes me sad. | 17:35 |
| jaypipes | mdbooth: :) | 17:35 |
| *** lucasagomes is now known as lucas-afk | 17:36 | |
| *** gfhellma_ has quit IRC | 17:38 | |
| mriedem | jaypipes: https://www.youtube.com/watch?v=w5Fgp-KihIA | 17:39 |
| mriedem | ^ is what you're looking for | 17:39 |
| sfinucan | mdbooth: Done https://review.openstack.org/#/c/438970. Thanks for breaking that out | 17:39 |
| mdbooth | Thanks for looking :) | 17:39 |
| dansmith | mriedem: right before we call from conductor to the compute to do the build, requested_networks is None for that test | 17:40 |
| mriedem | gotta go, people looking over my shoulder waiting for lunch | 17:40 |
| *** dtp has joined #openstack-nova | 17:40 | |
| mriedem | will check when i'm back | 17:40 |
| *** erhudy has joined #openstack-nova | 17:42 | |
| *** fragatina has joined #openstack-nova | 17:43 | |
| *** swebster has quit IRC | 17:45 | |
| *** swebster has joined #openstack-nova | 17:45 | |
| jaypipes | mriedem: lol | 17:46 |
| *** kristian__ has quit IRC | 17:46 | |
| *** yamahata has joined #openstack-nova | 17:47 | |
| *** sudipto has joined #openstack-nova | 17:48 | |
| *** sudipto_ has joined #openstack-nova | 17:48 | |
| *** david-lyle_ has joined #openstack-nova | 17:52 | |
| *** gszasz has quit IRC | 17:53 | |
| *** david-lyle has quit IRC | 17:54 | |
| *** david-lyle_ is now known as david-lyle | 17:55 | |
| *** abalutoiu has joined #openstack-nova | 17:55 | |
| *** derekh has quit IRC | 17:55 | |
| *** ayogi has joined #openstack-nova | 17:56 | |
| *** READ10 has joined #openstack-nova | 17:57 | |
| *** ayogi has quit IRC | 17:58 | |
| *** ducttape_ has quit IRC | 17:59 | |
| *** sudipto has quit IRC | 18:00 | |
| *** sudipto_ has quit IRC | 18:00 | |
| *** ralonsoh has quit IRC | 18:01 | |
| *** links has joined #openstack-nova | 18:01 | |
| macsz | bugs team meeting in #openstack-meeting-4 | 18:02 |
| *** owalsh is now known as owalsh-afk | 18:02 | |
| *** david-lyle has quit IRC | 18:04 | |
| *** ducttape_ has joined #openstack-nova | 18:04 | |
| *** david-lyle has joined #openstack-nova | 18:05 | |
| *** sambetts is now known as sambetts|afk | 18:08 | |
| *** baoli has joined #openstack-nova | 18:10 | |
| *** gfhellma has joined #openstack-nova | 18:10 | |
| *** jed56 has quit IRC | 18:10 | |
| *** gfhellma has quit IRC | 18:10 | |
| *** gfhellma has joined #openstack-nova | 18:11 | |
| dansmith | jaypipes: would you prefer I double the math in the tests, keep the single-cell mock, or completely change the way that test works here? https://review.openstack.org/#/c/441388/3 | 18:13 |
| *** mnestratov has joined #openstack-nova | 18:14 | |
| dansmith | just doubling the math does verify that we're iterating over the two default cells, so maybe that's the better way | 18:15 |
| *** ayogi has joined #openstack-nova | 18:15 | |
| *** gfhellma has quit IRC | 18:15 | |
| jaypipes | dansmith: got it. a short comment above would suffice I think | 18:16 |
| jaypipes | dansmith: just to assuage the reader of the test at a later time ;) | 18:16 |
| openstackgerrit | Dan Peschman proposed openstack/nova master: No API cell up-call to delete consoleauth tokens https://review.openstack.org/415922 | 18:16 |
| dansmith | jaypipes: lemme do the double math and then document that bit to explain the multiple cells thing.. that'll validate what we're doing, be less code, and more obvious I think | 18:16 |
| dansmith | since it's all contained in a helper checker | 18:16 |
| jaypipes | cools with me | 18:17 |
| *** fragatina has quit IRC | 18:22 | |
| *** david-lyle has quit IRC | 18:23 | |
| jaypipes | mdbooth: around? | 18:23 |
| jaypipes | mdbooth: would you mind reading my review comment on https://review.openstack.org/#/c/399679/ and commenting on that? Basically, the patch introduces the potential of the libvirt.xml file storing in the owner metadata block a user *ID* that doesn't match the user *name* -- same for project ID being different than the project name. Because the patch sets the userid to the instance.user_id (the owner user) and username to context.user_name (which | 18:25 |
| jaypipes | could be the admin). | 18:25 |
| *** david-lyle has joined #openstack-nova | 18:25 | |
| *** claudiub has joined #openstack-nova | 18:27 | |
| *** david-lyle has quit IRC | 18:29 | |
| *** john51 has quit IRC | 18:29 | |
| *** john51 has joined #openstack-nova | 18:34 | |
| *** liangy has quit IRC | 18:36 | |
| *** david-lyle has joined #openstack-nova | 18:36 | |
| *** liangy has joined #openstack-nova | 18:36 | |
| *** mvk has quit IRC | 18:36 | |
| *** salv-orlando has quit IRC | 18:39 | |
| *** ijw has quit IRC | 18:39 | |
| *** abalutoiu has quit IRC | 18:41 | |
| *** eharney has quit IRC | 18:41 | |
| *** salv-orlando has joined #openstack-nova | 18:44 | |
| jaypipes | dansmith: Snoop Dan? :) | 18:44 |
| *** eharney has joined #openstack-nova | 18:45 | |
| dansmith | jaypipes: yeah, I'm o-g | 18:45 |
| jaypipes | dansmith: the original grokster. | 18:46 |
| mriedem | jaypipes: on the searchlight connectivity check, i replied in the spec, but i think we just handle it like we did for the placement safe_connect thing in the scheduler report client, but we don't retry | 18:48 |
| mriedem | if it's there use it, else we log a warning, set a flag and don't retry until restart | 18:49 |
| jaypipes | mriedem: cools with me. | 18:49 |
| mriedem | figuring out if searchlight even supports versioned notifications from nova is priority 1 for me atm, because i don't think they do | 18:49 |
| mriedem | which is a pretty big blocker for us | 18:49 |
| cdent | mriedem: you mean being willing to store different versions, differently? | 18:50 |
| mriedem | cdent: no, like actually listening for that type of notification | 18:50 |
| jaypipes | mriedem: they listen on the same queue, no? | 18:50 |
| mriedem | yeah i think so https://github.com/openstack/searchlight/blob/2.0.0/searchlight/elasticsearch/plugins/nova/notification_handler.py#L73 | 18:51 |
| *** jpena is now known as jpena|away | 18:52 | |
| mriedem | i'm not entirely sure that this maps the same https://github.com/openstack/searchlight/blob/2.0.0/searchlight/elasticsearch/plugins/nova/notification_handler.py#L82 | 18:52 |
| *** ducttape_ has quit IRC | 18:56 | |
| mriedem | cdent: jaypipes: for example, searchlight has https://github.com/openstack/searchlight/blob/2.0.0/searchlight/elasticsearch/plugins/nova/notification_handler.py#L92 | 18:57 |
| mriedem | compute.instance.power_off.end | 18:57 |
| mriedem | our versioned notification sample has https://github.com/openstack/nova/blob/master/doc/notification_samples/instance-power_off-end.json#L2 | 18:57 |
| mriedem | instance.power_off.end | 18:57 |
| mriedem | i'm not sure where the compute. prefix comes from | 18:58 |
| *** baoli has quit IRC | 18:58 | |
| cdent | mriedem: it looks like it may be something they are doing: https://github.com/openstack/searchlight/blob/2.0.0/searchlight/elasticsearch/plugins/ironic/notification_handlers.py | 18:58 |
| cdent | (sorry, line 37) | 18:58 |
| *** adisky_ has quit IRC | 18:59 | |
| mriedem | cdent: i'm not so sure https://github.com/openstack/nova/blob/master/nova/rpc.py#L292 | 18:59 |
| cdent | indeed | 19:00 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Get instance availability_zone without hitting the api db https://review.openstack.org/439754 | 19:00 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Enlighten server tags API about cells https://review.openstack.org/440856 | 19:00 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Teach HostAPI about cells https://review.openstack.org/442162 | 19:00 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Make os-availability-zones know about cells https://review.openstack.org/442163 | 19:00 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Target cell in super conductor operations https://review.openstack.org/438022 | 19:00 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Change MQ targeting to honor only what is in the context https://review.openstack.org/438023 | 19:00 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Make scheduler target cells to get compute node instance info https://review.openstack.org/439891 | 19:00 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Make consoleauth target the proper cell https://review.openstack.org/441461 | 19:00 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Add cell field to Destination object https://review.openstack.org/438024 | 19:00 |
| openstackgerrit | Dan Smith proposed openstack/nova master: WIP: Make conductor ask scheduler to limit migrates to same cell https://review.openstack.org/438025 | 19:00 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Teach simple_tenant_usage about cells https://review.openstack.org/441388 | 19:00 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Teach os-aggregates about cells https://review.openstack.org/442200 | 19:00 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Teach os-migrations about cells https://review.openstack.org/442685 | 19:00 |
| *** mnestratov has quit IRC | 19:00 | |
| cdent | even if it is problem, based on the conversations we've had with searchlight in the past they've strenuously said: we will make this work... | 19:00 |
| *** sdague has quit IRC | 19:02 | |
| *** swebster has quit IRC | 19:03 | |
| *** swebster has joined #openstack-nova | 19:03 | |
| *** jamesdenton has quit IRC | 19:03 | |
| mriedem | gibi: do you know where the compute. prefix shows up in the unversioned legacy notifications? | 19:04 |
| *** david-lyle has quit IRC | 19:04 | |
| mriedem | cdent: yeah also https://blueprints.launchpad.net/searchlight/+spec/nova-versioned-notifications | 19:04 |
| *** dtp has quit IRC | 19:04 | |
| cdent | ack | 19:05 |
| *** david-lyle has joined #openstack-nova | 19:06 | |
| *** jamesdenton has joined #openstack-nova | 19:06 | |
| *** mvk has joined #openstack-nova | 19:07 | |
| *** crushil has quit IRC | 19:08 | |
| *** baoli has joined #openstack-nova | 19:09 | |
| cdent | i have continuouspartialattentionitis | 19:11 |
| mriedem | i have a headache staring at this notification code | 19:11 |
| mriedem | i think we've also changed compute.instance.create.start to conductor.instance.create.start | 19:12 |
| mriedem | maybe | 19:12 |
| *** ducttape_ has joined #openstack-nova | 19:13 | |
| *** _ducttape_ has joined #openstack-nova | 19:14 | |
| *** satyar has quit IRC | 19:15 | |
| openstackgerrit | Jay Pipes proposed openstack/nova master: placement: standard memory page resource classes https://review.openstack.org/442718 | 19:15 |
| *** fragatina has joined #openstack-nova | 19:16 | |
| mriedem | cdent: i think this is where we get the compute. prefix on the unversioned notification https://github.com/openstack/nova/blob/master/nova/compute/utils.py#L324 | 19:17 |
| mriedem | versioned is https://github.com/openstack/nova/blob/master/nova/compute/utils.py#L356 | 19:17 |
| mriedem | which doesn't have the prefix | 19:17 |
| *** ducttape_ has quit IRC | 19:18 | |
| *** crushil has joined #openstack-nova | 19:18 | |
| cdent | that seems rather unfortunate | 19:18 |
| cdent | he said blandly | 19:18 |
| *** _ducttape_ has quit IRC | 19:18 | |
| *** READ10 has quit IRC | 19:19 | |
| mriedem | i assume that was intentional | 19:21 |
| *** salv-orl_ has joined #openstack-nova | 19:22 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Update docstrings for legacy notification methods https://review.openstack.org/442723 | 19:22 |
| cdent | I know that there was intention to make it so you could have both of them going at the same time, but from the consumer side, having different names (instead of different topics (is that right, I'm not sure of the nomenclature)) makes switching harder | 19:23 |
| *** ayogi has quit IRC | 19:23 | |
| mriedem | by default nova is configured to send both | 19:23 |
| mriedem | http://logs.openstack.org/59/382959/17/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/f2fe0de/logs/screen-n-cpu.txt.gz#_2017-03-06_14_39_55_787 | 19:24 |
| mriedem | notifications.notification_format = both | 19:24 |
| * cdent nods | 19:24 | |
| mriedem | http://logs.openstack.org/59/382959/17/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/f2fe0de/logs/screen-n-cpu.txt.gz#_2017-03-06_14_39_55_756 | 19:24 |
| mriedem | i only see the one topic | 19:24 |
| mriedem | oslo_messaging_notifications.topics = ['notifications'] | 19:24 |
| * cdent nods | 19:24 | |
| *** salv-orlando has quit IRC | 19:25 | |
| *** GK___1wm____SU has joined #openstack-nova | 19:25 | |
| mriedem | https://specs.openstack.org/openstack/nova-specs/specs/newton/implemented/versioned-notification-transformation-newton.html#notifications-impact | 19:27 |
| mriedem | "As implemented in the versioned-notification-api bp the versioned notifications are always emitted to a different amqp topic called ‘versioned_notifications’ so the consumer can differentiate between the legacy and the new format by the topic." | 19:27 |
| mriedem | i'm not sure that's true | 19:27 |
| *** GK___1wm____SU has quit IRC | 19:27 | |
| mriedem | what i think i'd like to see, is some docs in https://docs.openstack.org/developer/nova/notifications.html about how to actually configure / set things up to use this | 19:28 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Add workaround to disable group policy check upcall https://review.openstack.org/442736 | 19:28 |
| mriedem | because i'm putting this together, shittily, from code and old specs | 19:28 |
| cdent | everyone else has continuouspartialattentionitis too | 19:30 |
| mriedem | oh i see it https://github.com/openstack/nova/blob/master/nova/rpc.py#L92 | 19:31 |
| mriedem | the default topic is 'notifications' | 19:31 |
| mriedem | which is why the legacy notifier doesn't define a topics kwarg | 19:31 |
| mriedem | so we're sending to separate topics | 19:31 |
| mriedem | with different event types | 19:31 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Get instance availability_zone without hitting the api db https://review.openstack.org/439754 | 19:32 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Teach HostAPI about cells https://review.openstack.org/442162 | 19:32 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Make os-availability-zones know about cells https://review.openstack.org/442163 | 19:32 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Target cell in super conductor operations https://review.openstack.org/438022 | 19:32 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Change MQ targeting to honor only what is in the context https://review.openstack.org/438023 | 19:32 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Teach os-migrations about cells https://review.openstack.org/442685 | 19:32 |
| dansmith | jaypipes: guh, sorry, pep8 fail on the _second_ friggin patch | 19:32 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Make scheduler target cells to get compute node instance info https://review.openstack.org/439891 | 19:32 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Make consoleauth target the proper cell https://review.openstack.org/441461 | 19:32 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Add cell field to Destination object https://review.openstack.org/438024 | 19:32 |
| openstackgerrit | Dan Smith proposed openstack/nova master: WIP: Make conductor ask scheduler to limit migrates to same cell https://review.openstack.org/438025 | 19:32 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Teach simple_tenant_usage about cells https://review.openstack.org/441388 | 19:32 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Teach os-aggregates about cells https://review.openstack.org/442200 | 19:32 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Add workaround to disable group policy check upcall https://review.openstack.org/442736 | 19:32 |
| jaypipes | dansmith: damn you, Smith! | 19:35 |
| dansmith | I know, I suck | 19:40 |
| *** amoralej is now known as amoralej|off | 19:43 | |
| *** tbachman has quit IRC | 19:45 | |
| mriedem | dansmith: found your issue | 19:54 |
| mriedem | http://logs.openstack.org/94/436094/12/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/051cffa/logs/screen-n-api.txt.gz#_2017-03-07_00_14_09_285 | 19:54 |
| mriedem | something jacked with the compute service version | 19:55 |
| mriedem | https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/servers.py#L448 | 19:55 |
| *** lucasxu has quit IRC | 19:57 | |
| *** gouthamr has quit IRC | 20:01 | |
| *** salv-orl_ has quit IRC | 20:03 | |
| *** Hazelesque_ is now known as Hazelesque | 20:03 | |
| *** lucasxu has joined #openstack-nova | 20:04 | |
| *** gfhellma has joined #openstack-nova | 20:06 | |
| *** ducttape_ has joined #openstack-nova | 20:07 | |
| openstackgerrit | Sarafraj Singh proposed openstack/nova-specs master: WIP: Add live-migration-force-after-timeout spec https://review.openstack.org/438467 | 20:08 |
| *** ducttape_ has quit IRC | 20:08 | |
| *** ducttape_ has joined #openstack-nova | 20:08 | |
| *** vladikr has quit IRC | 20:08 | |
| *** links has quit IRC | 20:09 | |
| openstackgerrit | Sarafraj Singh proposed openstack/nova-specs master: Add live-migration-force-after-timeout spec https://review.openstack.org/438467 | 20:09 |
| dansmith | mriedem: oh I bet it's just that we need to check all cells | 20:09 |
| dansmith | mriedem: for the service version | 20:09 |
| dansmith | mriedem: thanks for that | 20:09 |
| mriedem | \o/ | 20:10 |
| dansmith | yeah, that's an easy fix | 20:12 |
| dansmith | alternately we can probably remove that as it's for mitaka computes right? | 20:12 |
| efried | Did flake8 get an upgrade recently? Suddenly my imports are wrong. | 20:12 |
| *** simondodsley has joined #openstack-nova | 20:13 | |
| mriedem | dansmith: maybe? | 20:14 |
| mriedem | dansmith: we have this talk every few weeks about supporting n-2 vs handling service version checking | 20:15 |
| mriedem | we don't support n-2 officially, we don't test that way, | 20:15 |
| *** gfhellma has quit IRC | 20:15 | |
| mriedem | but we don't have anything that says what the minimum required compute is either | 20:15 |
| dansmith | mriedem: we've never supported anything beyond n-1, | 20:15 |
| *** gfhellma has joined #openstack-nova | 20:15 | |
| dansmith | mriedem: we do because n-2 computes will refuse to start | 20:15 |
| mriedem | we could use nova-status upgrade check mofo ftw | 20:15 |
| dansmith | mriedem: you literally can't start n-2 computes, I think that's pretty clear messaging :) | 20:16 |
| mriedem | why? | 20:16 |
| dansmith | why is that clear or why can you not? | 20:16 |
| mriedem | why can you not? | 20:16 |
| dansmith | the service checkin will refuse to roll the minimum version back | 20:16 |
| mriedem | but what if i never restarted the compute and i've just got a mitaka compute still running | 20:17 |
| dansmith | mriedem: you likely wouldn't have been able to complete online migrations, and thus never have been able to apply ocata schema migrations, etc | 20:18 |
| dansmith | mriedem: we've had those kinds of deps before, not sure if we really have had them between mitaka and ocata, but.. | 20:18 |
| mriedem | to be clear, you're talking about this? https://github.com/openstack/nova/blob/master/nova/objects/service.py#L304 | 20:18 |
| dansmith | it's always been a blanket policy, I'm not sure why we're suddenly concerned about it | 20:18 |
| dansmith | yeah | 20:18 |
| mriedem | "This could happen if we were started with older code by accident, either due to a rollback or an old and un-updated node suddenly coming back onto the network." | 20:18 |
| mriedem | so i get that, like with an evacuated node | 20:19 |
| mriedem | i'm not super concerned about this, i'd just like something a bit more blocking during upgrade that says, oh you have computes < x and x is the minimum to get to pike, so stop upgrade now | 20:20 |
| dansmith | well, nova-status is the right place for that I guess | 20:20 |
| dansmith | and if you ignore it, then shame on you | 20:20 |
| mriedem | hmm, service version 12 wasn't mitaka either | 20:21 |
| mriedem | mitaka max service version was 9 | 20:21 |
| dansmith | mriedem: right 12 was newton, | 20:22 |
| mriedem | newton max was 15 | 20:22 |
| dansmith | it checks for <12 right? | 20:22 |
| migi | dansmith: hey, question if I want to check "nova-manage db archive_deleted_rows" do I need to have full dump of nova db (maybe even different one??) or just some tables? | 20:22 |
| mriedem | dansmith: the get me a network checks for < 12 yeah | 20:22 |
| migi | dansmith: I was reading the https://docs.openstack.org/ops-guide/ops-backup-recovery.html and am unsure if the nova db is sufficient | 20:22 |
| dansmith | mriedem: right, so it's checking for mitaka, or n1 or whatever | 20:22 |
| mriedem | dansmith: we could add a min compute service version check to nova-status that requires compute minimum of 16 | 20:22 |
| dansmith | migi: you have to have all the tables, but don't have to have data | 20:23 |
| *** dtp has joined #openstack-nova | 20:23 | |
| dansmith | migi: but it won't be much of a test without data | 20:23 |
| dansmith | mriedem: right, that's the only place I think it makes sense | 20:23 |
| mriedem | so we land that, then remove the straddle code in the api, maybe doesn't have to be in order | 20:23 |
| melwitt | jaypipes: I'm currently hacking up the count resources for quota patch if you want to hold off on that one (also, thanks for reviewing the quotas object one) | 20:23 |
| mriedem | just before we release | 20:23 |
| migi | dansmith: thanks | 20:23 |
| dansmith | mriedem: it's advisory anyway, but I can do it as part of removing this check yeah | 20:24 |
| mriedem | dansmith: that would be lovely | 20:24 |
| mriedem | i'm going to start reviewing https://review.openstack.org/#/c/440856/ and up, do i need any pep talk before i start? | 20:25 |
| dansmith | mriedem: you can do it, you big code reviewing stud muffin you | 20:25 |
| mriedem | heh | 20:25 |
| * mriedem gets oiled up | 20:25 | |
| melwitt | you might need a montage | 20:25 |
| mriedem | thanks miyagi https://www.youtube.com/watch?v=iBktYJsJq-E | 20:26 |
| dansmith | mriedem: actually the pep talk is: consider all the infra puppies you kill by asking for nitty changes at the bottom, knowing that I'll follow up with legit stuff :P | 20:26 |
| mriedem | psh | 20:27 |
| mriedem | just for saying that, i'm going to be extra critical | 20:27 |
| dansmith | I'm shocked I haven't gotten a citation already today with all the resources this set is consuming | 20:27 |
| mriedem | sdague removed all of the scenario tests so no one cares anymore | 20:27 |
| dansmith | heh | 20:27 |
| *** gouthamr has joined #openstack-nova | 20:29 | |
| dansmith | I'd like to add up my "bill" for all the resources I use in nodepool for a month | 20:31 |
| dansmith | just for shits and giggles | 20:31 |
| *** JoseMello has quit IRC | 20:35 | |
| migi | dansmith: ok one more question for today. Let's say I have this dump and will run nova-manage. Do I need to know for which nova version the sql was created or can I use nova-manage from Liberty against db created by master? | 20:38 |
| dansmith | migi: they have to match | 20:39 |
| mriedem | dansmith: blam https://review.openstack.org/#/c/441388/5 | 20:39 |
| migi | dansmith: and this info is somewhere in sql or do I need to store it in metadata somewhere outisde | 20:39 |
| migi | dansmith: that was too much of questions as I said last one one before | 20:39 |
| dansmith | migi: you can get the version number out of the database somewhere, yeah | 20:40 |
| migi | dansmith: ok will find it then, thanks | 20:40 |
| *** ducttape_ has quit IRC | 20:40 | |
| dansmith | migi: I don't recall where, but it's like a single-row table that sqla-migrate uses | 20:40 |
| mriedem | it's the 'nova.migrations' table i believe | 20:40 |
| migi | mriedem: thanks, will find it :) | 20:41 |
| *** gaoyuan has joined #openstack-nova | 20:43 | |
| dansmith | mriedem: ack | 20:44 |
| mriedem | https://www.youtube.com/watch?v=X9UM4N5sGBs | 20:44 |
| dansmith | well, that was qick | 20:44 |
| mriedem | ha | 20:44 |
| dansmith | here and I was being nice and writing your nova-status thing | 20:44 |
| mriedem | i'm not sure i can put links to judas priest songs in reviews | 20:44 |
| jaypipes | melwitt: sure, no prob. I was just reviewing the migration one. you want me to hold off? | 20:45 |
| melwitt | jaypipes: hold off on the top patch, the third one | 20:45 |
| jaypipes | gotcha. | 20:45 |
| jaypipes | will do. | 20:45 |
| melwitt | thanks :) | 20:45 |
| *** awaugama has quit IRC | 20:47 | |
| *** gaoyuan has quit IRC | 20:47 | |
| *** claudiub|2 has joined #openstack-nova | 20:50 | |
| *** tbachman has joined #openstack-nova | 20:51 | |
| *** claudiub has quit IRC | 20:53 | |
| *** baoli has quit IRC | 20:57 | |
| *** baoli has joined #openstack-nova | 20:58 | |
| *** Jack_I has quit IRC | 21:02 | |
| *** sdague has joined #openstack-nova | 21:03 | |
| *** lpetrut has joined #openstack-nova | 21:07 | |
| openstackgerrit | Dan Smith proposed openstack/nova master: Add nova-status check for minimum service version https://review.openstack.org/442787 | 21:07 |
| dansmith | mriedem: ^ | 21:07 |
| *** dimtruck is now known as zz_dimtruck | 21:08 | |
| *** armax has quit IRC | 21:09 | |
| * mriedem puts on rubber gloves | 21:11 | |
| *** aysyd has quit IRC | 21:12 | |
| *** eharney has quit IRC | 21:13 | |
| * sdague wonders if I should jump back off irc.... | 21:14 | |
| mriedem | just get in line | 21:14 |
| *** owalsh has joined #openstack-nova | 21:16 | |
| *** rfolco has quit IRC | 21:17 | |
| dansmith | mriedem: guh, that breaks my math doubling even for the other things | 21:19 |
| dansmith | mriedem: and if I have only one cell, then we don't poke this | 21:19 |
| *** owalsh-afk has quit IRC | 21:19 | |
| *** ijw has joined #openstack-nova | 21:20 | |
| *** gfhellma has quit IRC | 21:22 | |
| *** gfhellma has joined #openstack-nova | 21:25 | |
| *** swebster has quit IRC | 21:25 | |
| *** swebster_ has joined #openstack-nova | 21:25 | |
| *** smatzek_ has quit IRC | 21:28 | |
| *** eharney has joined #openstack-nova | 21:29 | |
| openstackgerrit | Dan Smith proposed openstack/nova master: Get instance availability_zone without hitting the api db https://review.openstack.org/439754 | 21:30 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Teach HostAPI about cells https://review.openstack.org/442162 | 21:30 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Make os-availability-zones know about cells https://review.openstack.org/442163 | 21:30 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Target cell in super conductor operations https://review.openstack.org/438022 | 21:30 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Change MQ targeting to honor only what is in the context https://review.openstack.org/438023 | 21:30 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Teach os-migrations about cells https://review.openstack.org/442685 | 21:30 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Make scheduler target cells to get compute node instance info https://review.openstack.org/439891 | 21:30 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Make consoleauth target the proper cell https://review.openstack.org/441461 | 21:30 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Add cell field to Destination object https://review.openstack.org/438024 | 21:30 |
| openstackgerrit | Dan Smith proposed openstack/nova master: WIP: Make conductor ask scheduler to limit migrates to same cell https://review.openstack.org/438025 | 21:30 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Teach simple_tenant_usage about cells https://review.openstack.org/441388 | 21:30 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Teach os-aggregates about cells https://review.openstack.org/442200 | 21:30 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Add workaround to disable group policy check upcall https://review.openstack.org/442736 | 21:30 |
| * jaypipes shakes fists at dansmith | 21:30 | |
| dansmith | mriedem: look at all that puppy blood you spilled | 21:30 |
| dansmith | jaypipes: shake it at mriedem ! | 21:30 |
| mriedem | dude | 21:30 |
| jaypipes | :) | 21:30 |
| mriedem | clearly | 21:30 |
| dansmith | mriedem: when I get back I'll remove that check behind the nova-status change | 21:31 |
| mriedem | maybe if you guys didn't break the api contract | 21:31 |
| mriedem | ... | 21:31 |
| dansmith | nobody uses simple_tenant_usage | 21:31 |
| mriedem | dansmith: comments are in the nova-status change also | 21:31 |
| * dansmith runs | 21:31 | |
| dansmith | yep, I saw | 21:31 |
| *** owalsh has quit IRC | 21:32 | |
| *** owalsh_ has joined #openstack-nova | 21:32 | |
| *** owalsh_ is now known as owalsh-afk | 21:33 | |
| *** zz_dimtruck is now known as dimtruck | 21:33 | |
| openstackgerrit | Sarafraj Singh proposed openstack/nova-specs master: Add live-migration-force-after-timeout spec https://review.openstack.org/438467 | 21:34 |
| *** dtp has quit IRC | 21:34 | |
| *** Jeffrey4l_ has quit IRC | 21:34 | |
| *** gouthamr has quit IRC | 21:37 | |
| *** Jeffrey4l_ has joined #openstack-nova | 21:37 | |
| *** salv-orlando has joined #openstack-nova | 21:38 | |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: Plumbing for tagged nic attachment https://review.openstack.org/415622 | 21:38 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: Delete nic metadata when detaching interface https://review.openstack.org/435387 | 21:38 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: Libvirt support for tagged volume attachment https://review.openstack.org/418100 | 21:38 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: Delete disk metadata when detaching volume https://review.openstack.org/435082 | 21:38 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: Plumbing for tagged volume attachment https://review.openstack.org/391941 | 21:38 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: API support for tagged device attachment https://review.openstack.org/376021 | 21:38 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: Libvirt support for tagged nic attachment https://review.openstack.org/415520 | 21:38 |
| *** rfolco has joined #openstack-nova | 21:39 | |
| *** rfolco has quit IRC | 21:39 | |
| *** ducttape_ has joined #openstack-nova | 21:40 | |
| *** shaner has quit IRC | 21:41 | |
| *** shaner has joined #openstack-nova | 21:43 | |
| *** ducttape_ has quit IRC | 21:46 | |
| mriedem | dansmith: questions about sort order and cell0 in here https://review.openstack.org/#/c/441388/ | 21:47 |
| mriedem | i think cell0 is fine but wanted to ask it out loud | 21:47 |
| *** abalutoiu has joined #openstack-nova | 21:51 | |
| openstackgerrit | melanie witt proposed openstack/nova master: Count resources to check quota for cells https://review.openstack.org/416521 | 21:52 |
| openstackgerrit | melanie witt proposed openstack/nova master: Make Quotas object favor the API database https://review.openstack.org/410945 | 21:52 |
| openstackgerrit | melanie witt proposed openstack/nova master: Add online migration to move quotas to API database https://review.openstack.org/410946 | 21:52 |
| *** dave-mcc_ has joined #openstack-nova | 21:52 | |
| *** Guest27337 has quit IRC | 21:53 | |
| *** dave-mccowan has quit IRC | 21:54 | |
| *** cleong has quit IRC | 21:55 | |
| *** youzp has joined #openstack-nova | 21:55 | |
| mtreinish | mriedem: good waffles do stick together :) | 21:58 |
| *** gouthamr has joined #openstack-nova | 21:59 | |
| *** youzp has quit IRC | 22:00 | |
| *** crushil has quit IRC | 22:00 | |
| *** lucasxu has quit IRC | 22:02 | |
| *** lucasxu has joined #openstack-nova | 22:03 | |
| *** dave-mcc_ has quit IRC | 22:03 | |
| *** swebster_ has quit IRC | 22:04 | |
| *** swebster_ has joined #openstack-nova | 22:05 | |
| openstackgerrit | Sujitha proposed openstack/nova master: [WIP] Reduce calls to load_tags() to 0 https://review.openstack.org/435146 | 22:07 |
| openstackgerrit | Sujitha proposed openstack/nova master: Reduce calls to load_tags() to 0 https://review.openstack.org/435146 | 22:08 |
| *** baoli has quit IRC | 22:08 | |
| *** inc0 has joined #openstack-nova | 22:12 | |
| inc0 | hey guys, so we've run into serious issue with new libvirt 2.5 | 22:12 |
| *** armax has joined #openstack-nova | 22:12 | |
| *** thorst has quit IRC | 22:12 | |
| inc0 | issue being nova, when creating files for vms (like console.log), uses creates directory with permissions 744 | 22:12 |
| inc0 | so if you run nova-compute with different user than qemu, what we do in Kolla | 22:13 |
| inc0 | qemu can't access console.log which makes spawning vm impossible | 22:13 |
| inc0 | and with libvirt 2.5 for reasons unknown to me, dynamic permissiosn seems to be failing to chown these files | 22:14 |
| inc0 | anyone seen that happening? can we specify permissions for vm files (i.e. change them to 774)? | 22:14 |
| *** ducttape_ has joined #openstack-nova | 22:16 | |
| *** baoli has joined #openstack-nova | 22:21 | |
| *** smatzek_ has joined #openstack-nova | 22:25 | |
| *** lpetrut has quit IRC | 22:26 | |
| *** gaoyuan has joined #openstack-nova | 22:31 | |
| openstackgerrit | Sarafraj Singh proposed openstack/nova-specs master: Spec to add per instance timeout for LM https://review.openstack.org/442809 | 22:32 |
| mriedem | inc0: you'll probably need to talk to kashyap and/or sgordon on that one, maybe mdbooth - some of the red hat people, | 22:33 |
| mriedem | given the time of day kashyap and mdbooth are probably no longer around, not sure about sgordon | 22:33 |
| *** tblakes has quit IRC | 22:33 | |
| mriedem | we're nowhere near testing with libvirt 2.5 in the gate | 22:33 |
| mriedem | we test with 1.3.1 which is what's in xenial | 22:34 |
| inc0 | yes...but not in cloud archive | 22:34 |
| openstackgerrit | Sarafraj Singh proposed openstack/nova-specs master: Add live-migration-force-after-timeout spec https://review.openstack.org/438467 | 22:34 |
| inc0 | we will probably fall back to 1.3 ourselves | 22:34 |
| openstackgerrit | Sarafraj Singh proposed openstack/nova-specs master: Spec to add per instance timeout for LM https://review.openstack.org/442809 | 22:35 |
| mriedem | inc0: we == kolla? | 22:35 |
| inc0 | mriedem: correct | 22:35 |
| *** gaoyuan has quit IRC | 22:35 | |
| inc0 | I'm not sure when canonical bumped libvirt version this high..but here we are | 22:36 |
| mriedem | ok. i'm not sure how you guys pick what your use or support, but definitely using anything beyond what we test with in the integrated gate using xenial opens you up to risk for stuff like this | 22:36 |
| *** smatzek_ has quit IRC | 22:36 | |
| *** tbachman has quit IRC | 22:36 | |
| mriedem | inc0: might as well report a bug regardless | 22:36 |
| *** lucasxu has quit IRC | 22:36 | |
| inc0 | well yeah, but cloud archive is where you're supposed to pick it up if you install openstack from apt | 22:37 |
| inc0 | and there is libvirt out there too | 22:37 |
| inc0 | not sure if it's libvirt issue or nova issue | 22:37 |
| mriedem | kashyap is good at figuring out what changed in upstream libvirt and why they made such changes, in case they are not backward compatible | 22:37 |
| inc0 | seems more like libvirt tbh..dynamic ownership used to work wel | 22:37 |
| inc0 | l | 22:37 |
| mriedem | i'm assuming it was a conscious decision | 22:37 |
| mriedem | dynamic ownership sounds like a security exposure that they'd close, but i don't know much about it | 22:38 |
| inc0 | kashyap lives in which timezone? | 22:38 |
| *** baoli has quit IRC | 22:38 | |
| mriedem | i think he's in bulgaria? | 22:38 |
| mriedem | dansmith: ^? | 22:38 |
| dansmith | netherlands | 22:38 |
| mriedem | or that | 22:38 |
| mriedem | idk, somewhere in europe :) | 22:38 |
| mriedem | it's all the same right?! | 22:38 |
| inc0 | well, Europe, thanks, I'll try to poke him tomorrow morning | 22:38 |
| mriedem | yw | 22:38 |
| inc0 | no, not the same:) | 22:38 |
| inc0 | but close enough:) | 22:38 |
| mriedem | i asked a russian at the ptg if they vacation in crimea, which apparently was a faux pas...so par for the course | 22:39 |
| inc0 | don't worry, it's getting confusing, in Poland for example time difference to UK changes from 1hr to 2hrs every 6 months | 22:40 |
| mriedem | that's the same in the US | 22:40 |
| mriedem | except for arizona | 22:40 |
| inc0 | daylight savings \o/ | 22:40 |
| *** lucasxu has joined #openstack-nova | 22:40 | |
| inc0 | saves no daylight since 1800s | 22:40 |
| mriedem | yeah which is here in another week or so | 22:41 |
| mriedem | it saves me from being able to sleep properly | 22:41 |
| *** catintheroof has quit IRC | 22:41 | |
| sdague | that reminds me, I really need to find another venue for my "when will then be now" talk about time in software :) | 22:42 |
| *** catintheroof has joined #openstack-nova | 22:42 | |
| openstackgerrit | Eric Brown proposed openstack/python-novaclient master: Remove py34 tox env and pypi classifier https://review.openstack.org/442847 | 22:43 |
| mriedem | sdague: as in not AZ? | 22:43 |
| *** penick has joined #openstack-nova | 22:44 | |
| sdague | it included all kinds of fun about dates, timezones, dst, the leapsecond | 22:44 |
| *** tbachman has joined #openstack-nova | 22:44 | |
| inc0 | duh running qemu with nova user fixes it | 22:45 |
| inc0 | but it's no bueno security | 22:46 |
| *** catintheroof has quit IRC | 22:47 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova-specs master: List instances using Searchlight (spec) https://review.openstack.org/441692 | 22:47 |
| inc0 | anyway guys, please check out libvirt 2.5.0, it potentially breaks lots of stuff | 22:47 |
| inc0 | I mean it's not horrible if only we could make vm files being group-writable | 22:48 |
| *** esberglu has quit IRC | 22:49 | |
| *** esberglu has joined #openstack-nova | 22:49 | |
| *** mdrabe has quit IRC | 22:50 | |
| inc0 | I think when nova creates dirs for instance it uses default umask which would be 722, if we could make it 772, that would fix it | 22:50 |
| *** esberglu_ has joined #openstack-nova | 22:53 | |
| *** esberglu has quit IRC | 22:54 | |
| *** swebster_ has quit IRC | 22:56 | |
| *** sdague has quit IRC | 22:57 | |
| *** swebster_ has joined #openstack-nova | 22:57 | |
| *** esberglu_ has quit IRC | 22:57 | |
| *** baoli has joined #openstack-nova | 22:58 | |
| *** baoli has quit IRC | 22:59 | |
| *** jamesdenton has quit IRC | 23:02 | |
| *** iceyao has joined #openstack-nova | 23:03 | |
| *** baoli has joined #openstack-nova | 23:04 | |
| *** ducttape_ has quit IRC | 23:04 | |
| *** penick has quit IRC | 23:05 | |
| *** erlon has quit IRC | 23:05 | |
| *** ducttape_ has joined #openstack-nova | 23:07 | |
| *** iceyao has quit IRC | 23:07 | |
| *** snikitin has quit IRC | 23:07 | |
| *** felipemonteiro_ has quit IRC | 23:10 | |
| *** tbachman has quit IRC | 23:10 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova-specs master: Fix a couple of spec names https://review.openstack.org/432469 | 23:11 |
| openstackgerrit | Matt Riedemann proposed openstack/nova-specs master: Fix paths when moving specs https://review.openstack.org/432470 | 23:11 |
| openstackgerrit | Matt Riedemann proposed openstack/nova-specs master: Move implemented specs for Ocata https://review.openstack.org/432471 | 23:11 |
| *** lucasxu has quit IRC | 23:12 | |
| *** dimtruck is now known as zz_dimtruck | 23:13 | |
| *** mlakat has quit IRC | 23:15 | |
| *** edmondsw has quit IRC | 23:16 | |
| *** eharney has quit IRC | 23:16 | |
| dansmith | mriedem: did you see my question about the sort order thing? | 23:17 |
| *** ociuhandu has quit IRC | 23:17 | |
| dansmith | mriedem: I think the sort order is just there to ensure a stable marker search within a single query, not for the overall sort of the results | 23:18 |
| *** grumpycatt has quit IRC | 23:18 | |
| *** leseb_ has quit IRC | 23:18 | |
| *** grumpycatt has joined #openstack-nova | 23:18 | |
| *** annegent_ has quit IRC | 23:20 | |
| mriedem | looking | 23:20 |
| *** leseb_ has joined #openstack-nova | 23:20 | |
| dansmith | ah yeah diana just confirmed | 23:22 |
| mriedem | me too | 23:23 |
| mriedem | the marker is the uuid | 23:23 |
| mriedem | and i sort of remember talking about this in the spec, | 23:23 |
| mriedem | that uuid being a marker is silly because you can create a new instance and it's random uuid could mess up your marker/sort order | 23:24 |
| *** tbachman has joined #openstack-nova | 23:24 | |
| dansmith | meh, pagination is lossy anyway | 23:24 |
| dansmith | that doesn't bother me | 23:24 |
| dansmith | so I'm good to push right? | 23:24 |
| mriedem | umm | 23:25 |
| mriedem | so isn't she saying we should be sorting? | 23:25 |
| mriedem | at least by project_id? | 23:25 |
| *** grumpycatt has quit IRC | 23:25 | |
| dansmith | no? | 23:25 |
| *** grumpycatt has joined #openstack-nova | 23:25 | |
| mriedem | i'm going to read https://specs.openstack.org/openstack/nova-specs/specs/ocata/approved/paginate-simple-tenant-usage.html quick | 23:25 |
| mriedem | https://specs.openstack.org/openstack/nova-specs/specs/ocata/approved/paginate-simple-tenant-usage.html#data-model-impact | 23:25 |
| dansmith | the only reason we'd need to sort the results is if you care about the sort order as a user right? | 23:26 |
| mriedem | then why was it added to https://specs.openstack.org/openstack/nova-specs/specs/ocata/approved/paginate-simple-tenant-usage.html#data-model-impact? | 23:26 |
| mriedem | https://review.openstack.org/#/c/386771/ | 23:27 |
| *** xyang1 has quit IRC | 23:27 | |
| dansmith | what I have doesn't affect the stability of the sequence across cells | 23:27 |
| dansmith | all that matters is that the last uuid of the end of your limit query lands you in the same place when you go back digging through the cells for the next page | 23:27 |
| dansmith | oh you think she means sorted results to the user _was_ a requirement | 23:29 |
| mriedem | yes | 23:29 |
| dansmith | I read that as "fell out" like, it wasn't in the spec and the code was never updated | 23:29 |
| dansmith | but I guess that's not what she means | 23:29 |
| mriedem | there is a bunch of discussion between alex_xu and diana and jaypipes and johnthetubaguy in https://review.openstack.org/#/c/386771/ about sorting | 23:29 |
| dansmith | so this is even larger of a merge sort than list instances right? | 23:30 |
| dansmith | I mean, this is potentially an order of magnitude or more larger than just a list of live instances | 23:30 |
| mriedem | depends on the window i suppose | 23:31 |
| *** catintheroof has joined #openstack-nova | 23:32 | |
| mriedem | i'm wondering if we only need to sort if detailed=True | 23:32 |
| dansmith | okay then I'm going to move this to later in the set so we can progress on some of the other ones while we argue about this forever | 23:32 |
| mriedem | but, it might also be dependent on the limit/marker | 23:32 |
| *** cdent has quit IRC | 23:33 | |
| mriedem | if you get a tenant_id then all it does is sort on the uuids | 23:33 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Enlighten server tags API about cells https://review.openstack.org/440856 | 23:33 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Target cell in super conductor operations https://review.openstack.org/438022 | 23:33 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Change MQ targeting to honor only what is in the context https://review.openstack.org/438023 | 23:33 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Teach os-migrations about cells https://review.openstack.org/442685 | 23:33 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Make consoleauth target the proper cell https://review.openstack.org/441461 | 23:33 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Add cell field to Destination object https://review.openstack.org/438024 | 23:33 |
| openstackgerrit | Dan Smith proposed openstack/nova master: WIP: Make conductor ask scheduler to limit migrates to same cell https://review.openstack.org/438025 | 23:33 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Add workaround to disable group policy check upcall https://review.openstack.org/442736 | 23:33 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Get instance availability_zone without hitting the api db https://review.openstack.org/439754 | 23:33 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Teach HostAPI about cells https://review.openstack.org/442162 | 23:33 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Make os-availability-zones know about cells https://review.openstack.org/442163 | 23:33 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Make scheduler target cells to get compute node instance info https://review.openstack.org/439891 | 23:33 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Teach simple_tenant_usage about cells https://review.openstack.org/441388 | 23:33 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Teach os-aggregates about cells https://review.openstack.org/442200 | 23:33 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Remove Mitaka-era service version check https://review.openstack.org/442861 | 23:33 |
| *** claudiub|2 has quit IRC | 23:33 | |
| dansmith | I really don't understand the sorting discussion on that spec | 23:34 |
| mriedem | need diana and alex_xu to explain | 23:34 |
| mriedem | i can't read through all of that and "sort" it out | 23:34 |
| mriedem | get it | 23:34 |
| dansmith | yeah | 23:34 |
| mriedem | i'm literally laughing here to myself and my own joke | 23:35 |
| * dansmith groans | 23:35 | |
| mriedem | i think it's time to head home | 23:35 |
| *** mriedem has quit IRC | 23:36 | |
| *** ociuhandu has joined #openstack-nova | 23:40 | |
| *** thorst has joined #openstack-nova | 23:45 | |
| *** thorst has quit IRC | 23:45 | |
| *** baoli has quit IRC | 23:47 | |
| *** gfhellma has quit IRC | 23:47 | |
| *** owalsh has joined #openstack-nova | 23:50 | |
| openstackgerrit | Jin Li proposed openstack/nova master: Transform instance.soft_delete notifications https://review.openstack.org/383970 | 23:51 |
| *** adrian_otto has quit IRC | 23:52 | |
| *** edmondsw has joined #openstack-nova | 23:52 | |
| *** ducttape_ has quit IRC | 23:55 | |
| *** thorst has joined #openstack-nova | 23:55 | |
| *** ducttape_ has joined #openstack-nova | 23:56 | |
| *** edmondsw has quit IRC | 23:56 | |
| *** esberglu has joined #openstack-nova | 23:58 | |
| *** yingjun has joined #openstack-nova | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!