| *** muawiakhan has joined #openstack-nova | 00:01 | |
| mikal | Heh | 00:02 |
|---|---|---|
| mikal | But I have the lurgy! | 00:02 |
| *** rfolco_ has joined #openstack-nova | 00:02 | |
| *** rfolco_ has quit IRC | 00:02 | |
| *** rfolco_ has joined #openstack-nova | 00:03 | |
| dansmith | mikal: I will be your best distant friend.. beyond the reach of the pox | 00:03 |
| *** shaohe_feng has quit IRC | 00:03 | |
| *** rfolco_ has quit IRC | 00:03 | |
| *** amotoki has joined #openstack-nova | 00:03 | |
| mikal | Heh | 00:03 |
| *** shaohe_feng has joined #openstack-nova | 00:03 | |
| *** rfolco_ has joined #openstack-nova | 00:03 | |
| mikal | I would review if gerrit was working | 00:03 |
| mikal | But it might be my internets | 00:03 |
| openstackgerrit | Dan Smith proposed openstack/nova: Device tagging API support https://review.openstack.org/304510 | 00:04 |
| *** rfolco_ has quit IRC | 00:04 | |
| dansmith | it's not just you | 00:04 |
| dansmith | gerrit seems to be constipated from here as well | 00:04 |
| artom | Oh, is it Gerrit FF failure time? | 00:05 |
| *** thorst has joined #openstack-nova | 00:05 | |
| dansmith | just about | 00:06 |
| *** muawiakhan has quit IRC | 00:06 | |
| dansmith | mikal: I just saw you pushed up another patch for my and other's comments on the vendordata thing | 00:06 |
| *** amotoki has quit IRC | 00:08 | |
| *** rama_y has quit IRC | 00:08 | |
| *** thorst has quit IRC | 00:12 | |
| *** shaohe_feng has quit IRC | 00:13 | |
| *** shaohe_feng has joined #openstack-nova | 00:14 | |
| *** gongysh has joined #openstack-nova | 00:15 | |
| *** thorst has joined #openstack-nova | 00:16 | |
| *** stewie925 has quit IRC | 00:16 | |
| *** thorst has quit IRC | 00:17 | |
| *** rfolco_ has joined #openstack-nova | 00:23 | |
| *** rfolco_ has quit IRC | 00:23 | |
| *** rfolco__ has joined #openstack-nova | 00:23 | |
| *** rfolco__ has quit IRC | 00:23 | |
| *** shaohe_feng has quit IRC | 00:23 | |
| *** shaohe_feng has joined #openstack-nova | 00:24 | |
| artom | Boom, Tempest test! | 00:26 |
| *** Apoorva has joined #openstack-nova | 00:26 | |
| artom | dansmith, and it passes locally on your code! | 00:26 |
| dansmith | woot | 00:26 |
| * artom gets... supper? Yeah, why not, let's call it supper. | 00:27 | |
| dansmith | heh | 00:28 |
| *** Sukhdev has quit IRC | 00:30 | |
| *** Apoorva has quit IRC | 00:31 | |
| mriedem | artom: dansmith: i have a query | 00:31 |
| mriedem | should the rest api schema for tags be the same as we have for the server tags api? | 00:32 |
| *** MikeG451 has quit IRC | 00:32 | |
| * mikal goes to look at that review | 00:32 | |
| *** shaohe_feng has quit IRC | 00:34 | |
| *** shaohe_feng has joined #openstack-nova | 00:34 | |
| *** Jeffrey4l has joined #openstack-nova | 00:36 | |
| dansmith | mriedem: I think being consistent with those is probably a good idea | 00:36 |
| *** baoli has joined #openstack-nova | 00:38 | |
| *** baoli has quit IRC | 00:39 | |
| *** yamahata has quit IRC | 00:41 | |
| *** ducttape_ has joined #openstack-nova | 00:42 | |
| *** shaohe_feng has quit IRC | 00:44 | |
| *** shaohe_feng has joined #openstack-nova | 00:44 | |
| *** kevinzh has joined #openstack-nova | 00:46 | |
| *** Sukhdev has joined #openstack-nova | 00:47 | |
| mriedem | artom: dansmith: on the api change, -1 is for the schema thing https://review.openstack.org/#/c/304510/ | 00:48 |
| mriedem | i have some nits in the unit tests about maybe mocking some things that don't need to be mocked, but those aren't a big deal | 00:49 |
| mriedem | but that one is really close, should be mergable tonight | 00:49 |
| *** shaohe_feng has quit IRC | 00:54 | |
| *** shaohe_feng has joined #openstack-nova | 00:55 | |
| *** baoli has joined #openstack-nova | 00:56 | |
| *** baoli has quit IRC | 00:57 | |
| *** links has joined #openstack-nova | 00:57 | |
| *** woodster_ has quit IRC | 00:59 | |
| *** shihanzhang has joined #openstack-nova | 00:59 | |
| *** gongysh has quit IRC | 00:59 | |
| *** phuongnh has joined #openstack-nova | 01:03 | |
| *** shaohe_feng has quit IRC | 01:04 | |
| *** shaohe_feng has joined #openstack-nova | 01:05 | |
| *** shaohe_feng has quit IRC | 01:15 | |
| *** shaohe_feng has joined #openstack-nova | 01:15 | |
| *** greghaynes has quit IRC | 01:16 | |
| *** thorst has joined #openstack-nova | 01:17 | |
| *** rfolco__ has joined #openstack-nova | 01:17 | |
| *** rfolco__ has quit IRC | 01:18 | |
| *** ducttape_ has quit IRC | 01:19 | |
| artom | mriedem, the mocks are because I couldn't get create to finish without them | 01:19 |
| artom | I might be missing something, but I needed all those mocked otherwise they threw errors | 01:19 |
| eliqiao | hi mriedem danpb, evening, thanks for your reviewing on "libvirt perf" patches :) | 01:22 |
| *** code-R has quit IRC | 01:22 | |
| mriedem | eliqiao: np | 01:23 |
| *** ducttape_ has joined #openstack-nova | 01:24 | |
| openstackgerrit | Artom Lifshitz proposed openstack/nova: Device tagging metadata API support https://review.openstack.org/304511 | 01:25 |
| *** shaohe_feng has quit IRC | 01:25 | |
| artom | mriedem, don't panic that's ^^ just a rebase | 01:25 |
| *** amotoki has joined #openstack-nova | 01:25 | |
| artom | Because *someone* didn't push the entire series | 01:25 |
| *** thorst has quit IRC | 01:26 | |
| *** shaohe_feng has joined #openstack-nova | 01:26 | |
| *** rrecio_ has joined #openstack-nova | 01:26 | |
| eliqiao | mriedem: BTW, we will have Bug smash day next week in China, I wonder if there is any hot bug list that you can pointing to? I would like to help to do something. | 01:27 |
| *** cheneydc has joined #openstack-nova | 01:27 | |
| mriedem | eliqiao: you should ask markus_z | 01:27 |
| mriedem | he runs the bug meeting | 01:28 |
| eliqiao | mriedem: ya, thanks, good to konw that, will try to catch him my evening time. | 01:29 |
| *** amotoki has quit IRC | 01:30 | |
| mriedem | artom: just for that, -1 on the metadata api change | 01:30 |
| artom | oh noes now it's been made personal | 01:30 |
| *** baoli has joined #openstack-nova | 01:30 | |
| *** greghaynes has joined #openstack-nova | 01:30 | |
| *** baoli has quit IRC | 01:31 | |
| artom | Oh crap, there's an actual for serious -1 though :D | 01:31 |
| mriedem | heh yeah | 01:32 |
| mriedem | i'm reviewing more now | 01:32 |
| *** thorst has joined #openstack-nova | 01:33 | |
| *** thorst has quit IRC | 01:33 | |
| *** thorst has joined #openstack-nova | 01:33 | |
| *** wangqun has joined #openstack-nova | 01:34 | |
| *** rrecio_ has quit IRC | 01:34 | |
| *** ducttape_ has quit IRC | 01:34 | |
| *** Apoorva has joined #openstack-nova | 01:34 | |
| *** shaohe_feng has quit IRC | 01:35 | |
| *** Apoorva has quit IRC | 01:36 | |
| *** shaohe_feng has joined #openstack-nova | 01:36 | |
| *** thorst has quit IRC | 01:38 | |
| *** jichen has joined #openstack-nova | 01:38 | |
| *** browne has quit IRC | 01:39 | |
| *** yuikotakadamori has joined #openstack-nova | 01:41 | |
| *** EinstCrazy has joined #openstack-nova | 01:41 | |
| *** ssurana has quit IRC | 01:42 | |
| *** gongysh has joined #openstack-nova | 01:44 | |
| *** shaohe_feng has quit IRC | 01:45 | |
| *** shaohe_feng has joined #openstack-nova | 01:46 | |
| mriedem | artom: ok, done reviewing the metadata api change | 01:46 |
| *** Sukhdev has quit IRC | 01:50 | |
| openstackgerrit | Hieu LE proposed openstack/nova: Remove mox from unit/virt/libvirt/test_imagebackend.py (3) https://review.openstack.org/335765 | 01:51 |
| *** gjayavelu has joined #openstack-nova | 01:52 | |
| *** TravT has quit IRC | 01:54 | |
| *** baoli has joined #openstack-nova | 01:54 | |
| *** sdake has joined #openstack-nova | 01:54 | |
| *** yamahata has joined #openstack-nova | 01:54 | |
| *** baoli has quit IRC | 01:55 | |
| *** shaohe_feng has quit IRC | 01:56 | |
| *** thorst has joined #openstack-nova | 01:56 | |
| *** thorst has quit IRC | 01:56 | |
| *** shaohe_feng has joined #openstack-nova | 01:57 | |
| *** thorst has joined #openstack-nova | 01:57 | |
| mriedem | gus: for all intents and purposes, should we just mark this complete? https://blueprints.launchpad.net/nova/+spec/privsep | 01:58 |
| *** code-R has joined #openstack-nova | 01:59 | |
| *** code-R has quit IRC | 01:59 | |
| *** sacharya has joined #openstack-nova | 01:59 | |
| *** houming has joined #openstack-nova | 01:59 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add pagination and Changes-since filter support for os-migrations. https://review.openstack.org/330406 | 02:01 |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Use oslo_config new type PortOpt for port options https://review.openstack.org/238347 | 02:03 |
| artom | mriedem, https://review.openstack.org/#/c/304511/67/nova/api/metadata/base.py@371 am I understanding correctly that that's more of an improvement for future work? | 02:04 |
| openstackgerrit | Li Yingjun proposed openstack/nova: Microversion 2.32 adds pagination support for hypervisors https://review.openstack.org/326940 | 02:04 |
| *** sdake has quit IRC | 02:04 | |
| mriedem | artom: yeah | 02:04 |
| gus | mriedem: We haven't actually shipped anything using privsep, and even if we had, then there's still cleanup work on the obsolete rootwrap bits... | 02:04 |
| mriedem | gus: yeah, but the cleanup is going to be multiple releases | 02:05 |
| mriedem | which hasn't started | 02:05 |
| mriedem | once the grenade issue is sorted out then we'll be using os-brick with privsep, and os-vif with privsep | 02:05 |
| *** thorst has quit IRC | 02:05 | |
| mriedem | but those are going to be handled via the bug i think | 02:05 |
| gus | mriedem: I agree that I don't think the spec is helping anymore, so we could certainly close it if you just want to clean house. | 02:05 |
| mriedem | gus: yup, i'm just cleaning up blueprints for newton | 02:05 |
| mriedem | thanks | 02:05 |
| *** markvoelker has quit IRC | 02:06 | |
| *** shaohe_feng has quit IRC | 02:06 | |
| *** shaohe_feng has joined #openstack-nova | 02:07 | |
| *** ducttape_ has joined #openstack-nova | 02:07 | |
| *** dingboopt has quit IRC | 02:08 | |
| *** cheneydc has quit IRC | 02:08 | |
| *** Apoorva has joined #openstack-nova | 02:08 | |
| mriedem | melwitt: is https://blueprints.launchpad.net/nova/+spec/cells-mq-connection-switching done ? | 02:09 |
| *** _ducttape_ has joined #openstack-nova | 02:09 | |
| *** gjayavelu has quit IRC | 02:11 | |
| *** ducttape_ has quit IRC | 02:11 | |
| *** sacharya_ has joined #openstack-nova | 02:12 | |
| *** TravT has joined #openstack-nova | 02:12 | |
| *** rama_y has joined #openstack-nova | 02:14 | |
| *** sacharya has quit IRC | 02:15 | |
| *** shaohe_feng has quit IRC | 02:16 | |
| openstackgerrit | bailin.zhang proposed openstack/nova: Delete reduplicate code in test_compute_mgr.py https://review.openstack.org/331108 | 02:16 |
| *** shaohe_feng has joined #openstack-nova | 02:17 | |
| *** kevin_zh has joined #openstack-nova | 02:18 | |
| *** kevinzh has quit IRC | 02:18 | |
| openstackgerrit | Artom Lifshitz proposed openstack/nova: Device tagging metadata API support https://review.openstack.org/304511 | 02:19 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova: Device tagging API support https://review.openstack.org/304510 | 02:19 |
| *** tjones has joined #openstack-nova | 02:21 | |
| *** gongysh has quit IRC | 02:25 | |
| *** shaohe_feng has quit IRC | 02:26 | |
| *** shaohe_feng has joined #openstack-nova | 02:27 | |
| *** _ducttape_ has quit IRC | 02:32 | |
| *** ljxiash has joined #openstack-nova | 02:32 | |
| *** ljxiash has quit IRC | 02:33 | |
| *** jbernard has quit IRC | 02:33 | |
| *** ljxiash has joined #openstack-nova | 02:33 | |
| *** ljxiash has quit IRC | 02:34 | |
| *** jraju has joined #openstack-nova | 02:34 | |
| *** ljxiash has joined #openstack-nova | 02:34 | |
| *** ljxiash has quit IRC | 02:34 | |
| *** ljxiash has joined #openstack-nova | 02:34 | |
| *** links has quit IRC | 02:36 | |
| *** rfolco__ has joined #openstack-nova | 02:36 | |
| *** rfolco__ has quit IRC | 02:36 | |
| mriedem | mikal: a couple of easy +Ws here on the ones with the +2s https://review.openstack.org/#/q/topic:bp/rm-object-dict-compat-newton+status:open,n,z | 02:36 |
| mriedem | then i'm going to mark that bp complete for newton | 02:36 |
| mriedem | it's just a cleanup series anyway | 02:36 |
| *** ducttape_ has joined #openstack-nova | 02:37 | |
| *** shaohe_feng has quit IRC | 02:37 | |
| *** shaohe_feng has joined #openstack-nova | 02:37 | |
| *** ljxiash has quit IRC | 02:39 | |
| mriedem | tjones: is the vmware subteam good with this? https://review.openstack.org/#/c/297574/ | 02:39 |
| *** ljxiash has joined #openstack-nova | 02:40 | |
| *** baoli has joined #openstack-nova | 02:41 | |
| *** keivn has joined #openstack-nova | 02:43 | |
| *** kevin_zh has quit IRC | 02:43 | |
| *** keivn is now known as kevin_zh | 02:43 | |
| *** kevinzh has joined #openstack-nova | 02:43 | |
| mriedem | tjones: nevermind, -1 | 02:44 |
| openstackgerrit | Eli Qiao proposed openstack/nova: Mapping power_state from integer to string https://review.openstack.org/335776 | 02:44 |
| *** baoli has quit IRC | 02:45 | |
| *** gjayavelu has joined #openstack-nova | 02:46 | |
| mriedem | alaski: no tests needed for this? https://review.openstack.org/#/c/332216/ | 02:46 |
| *** Daisy has joined #openstack-nova | 02:47 | |
| *** shaohe_feng has quit IRC | 02:47 | |
| *** amotoki has joined #openstack-nova | 02:47 | |
| *** shaohe_feng has joined #openstack-nova | 02:47 | |
| tjones | mriedem: lemme look just in case | 02:47 |
| *** browne has joined #openstack-nova | 02:48 | |
| mriedem | BobBall: ^ | 02:48 |
| tjones | mriedem: ok i'll ping rado on it | 02:49 |
| mriedem | tjones: tomorrow is non-priority feature freeze | 02:51 |
| *** amotoki has quit IRC | 02:51 | |
| mriedem | tjones: fyi | 02:51 |
| tjones | i know :-) he gets up soon (bulgaria) | 02:52 |
| openstackgerrit | Dao Cong Tien proposed openstack/nova: Ironic Driver: override get_serial_console() https://review.openstack.org/328157 | 02:55 |
| *** shaohe_feng has quit IRC | 02:57 | |
| *** shaohe_feng has joined #openstack-nova | 02:58 | |
| *** ducttape_ has quit IRC | 02:59 | |
| mriedem | eliqiao: is there a dev list thread i should put in the nova meeting agenda for the bug smash? | 02:59 |
| mriedem | or some other wiki page or something? | 02:59 |
| *** harlowja_at_home has joined #openstack-nova | 03:00 | |
| mriedem | tristanC: kashyap: is anyone going to update this? https://review.openstack.org/#/c/327624/ | 03:00 |
| *** thorst has joined #openstack-nova | 03:03 | |
| *** jraju has quit IRC | 03:03 | |
| *** markvoelker has joined #openstack-nova | 03:06 | |
| *** shaohe_feng has quit IRC | 03:07 | |
| *** shaohe_feng has joined #openstack-nova | 03:08 | |
| *** Daisy has quit IRC | 03:09 | |
| *** Daisy has joined #openstack-nova | 03:09 | |
| *** thorst has quit IRC | 03:10 | |
| *** ljxiash has quit IRC | 03:10 | |
| *** ljxiash has joined #openstack-nova | 03:11 | |
| *** Daisy has quit IRC | 03:11 | |
| *** Daisy has joined #openstack-nova | 03:11 | |
| *** markvoelker has quit IRC | 03:11 | |
| *** Daisy has quit IRC | 03:16 | |
| mriedem | artom: 11:17pm for you? | 03:16 |
| artom | mriedem, yeah | 03:17 |
| mriedem | artom: ok, real quick, on https://review.openstack.org/#/c/304510/ the -1 is for the release note having the metadata api thing in it - that should go in the following change | 03:17 |
| mriedem | that adds the meta api support | 03:18 |
| artom | I'll handle that feedback from you tomorrow, I think I'll post the lyrics to Rick Roll if I attempt to do anything else tonight | 03:18 |
| *** shaohe_feng has quit IRC | 03:18 | |
| mriedem | ha | 03:18 |
| mriedem | fair enough | 03:18 |
| dansmith | artom: go to bed | 03:18 |
| artom | dansmith, finishing up the season finale of GoT | 03:18 |
| artom | :P | 03:18 |
| *** shaohe_feng has joined #openstack-nova | 03:18 | |
| mriedem | i could probably just update this myself real quick | 03:18 |
| mriedem | but mroning is fine | 03:18 |
| dansmith | mriedem: I can do it | 03:18 |
| mriedem | *morning | 03:18 |
| mriedem | dansmith: ok | 03:19 |
| *** sree has joined #openstack-nova | 03:20 | |
| *** harlowja_at_home has quit IRC | 03:20 | |
| mriedem | dansmith: also on the following meta api change https://review.openstack.org/#/c/304511/ | 03:21 |
| mriedem | easy -1 to address | 03:21 |
| mriedem | then i'm +2 on that one | 03:21 |
| dansmith | okay | 03:21 |
| mriedem | gonna watch the daily show quick | 03:22 |
| dansmith | oh, don't miss that important piece of "news" | 03:22 |
| *** harlowja_at_home has joined #openstack-nova | 03:22 | |
| * artom sleeps | 03:22 | |
| *** Apoorva has quit IRC | 03:23 | |
| *** Daisy has joined #openstack-nova | 03:24 | |
| *** ducttape_ has joined #openstack-nova | 03:28 | |
| *** shaohe_feng has quit IRC | 03:28 | |
| dansmith | mriedem: here it comes | 03:28 |
| openstackgerrit | Dan Smith proposed openstack/nova: Device tagging metadata API support https://review.openstack.org/304511 | 03:28 |
| *** sheel has joined #openstack-nova | 03:28 | |
| openstackgerrit | Dan Smith proposed openstack/nova: Device tagging API support https://review.openstack.org/304510 | 03:28 |
| dansmith | I haven't touched these two other than these tweaks just now, so if you're still comfortable with me +Wing after you +2 I'll do that | 03:29 |
| *** shaohe_feng has joined #openstack-nova | 03:29 | |
| openstackgerrit | Rong Han proposed openstack/nova: libvirt: Optimize _get_instance_disk_info https://review.openstack.org/297598 | 03:30 |
| *** jwcroppe has quit IRC | 03:30 | |
| *** amotoki has joined #openstack-nova | 03:36 | |
| *** ljxiash has quit IRC | 03:37 | |
| *** ljxiash has joined #openstack-nova | 03:37 | |
| *** shaohe_feng has quit IRC | 03:38 | |
| *** shaohe_feng has joined #openstack-nova | 03:39 | |
| mriedem | dansmith: not with that daily show sarcasm | 03:40 |
| dansmith | fair enough | 03:40 |
| *** ljxiash has quit IRC | 03:42 | |
| *** zz_dimtruck is now known as dimtruck | 03:42 | |
| *** zerda2 has joined #openstack-nova | 03:46 | |
| mriedem | gmann_: alex_xu: oomichi: can you take a look at this? https://review.openstack.org/#/c/304510/63/nova/api/openstack/compute/schemas/block_device_mapping.py@77 i'm not sure if this is going to work given how the server tags extension does it's tag validation | 03:46 |
| mriedem | dansmith: ^ i'm not sure about that one so gonna leave it until one of the other guys can comment | 03:47 |
| dansmith | mriedem: okay cool | 03:47 |
| *** shaohe_feng has quit IRC | 03:48 | |
| mriedem | something slipped into the meta api change too that should have been in the rest api change | 03:49 |
| mriedem | so i'm gonna call it a night | 03:49 |
| *** mriedem has quit IRC | 03:50 | |
| dansmith | oh crap yeah | 03:50 |
| *** amotoki has quit IRC | 03:50 | |
| *** Apoorva has joined #openstack-nova | 03:51 | |
| *** zhangjn has quit IRC | 03:51 | |
| *** shaohe_feng has joined #openstack-nova | 03:51 | |
| *** Daisy has quit IRC | 03:52 | |
| *** Daisy has joined #openstack-nova | 03:52 | |
| openstackgerrit | Dan Smith proposed openstack/nova: Device tagging metadata API support https://review.openstack.org/304511 | 03:53 |
| openstackgerrit | Dan Smith proposed openstack/nova: Device tagging API support https://review.openstack.org/304510 | 03:53 |
| *** Daisy_ has joined #openstack-nova | 03:55 | |
| *** Daisy has quit IRC | 03:55 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/333181 | 03:55 |
| *** zerda2 has quit IRC | 03:56 | |
| *** Daisy has joined #openstack-nova | 03:57 | |
| *** amotoki has joined #openstack-nova | 03:58 | |
| *** shaohe_feng has quit IRC | 03:59 | |
| *** code-R has joined #openstack-nova | 03:59 | |
| *** shaohe_feng has joined #openstack-nova | 03:59 | |
| *** zhangjn has joined #openstack-nova | 03:59 | |
| *** Daisy has quit IRC | 04:01 | |
| *** Daisy_ has quit IRC | 04:01 | |
| *** Daisy has joined #openstack-nova | 04:01 | |
| *** rama_y has quit IRC | 04:04 | |
| *** zhangjn has quit IRC | 04:05 | |
| *** tbachman has quit IRC | 04:05 | |
| *** tbachman_ has joined #openstack-nova | 04:05 | |
| *** pwnall has quit IRC | 04:05 | |
| *** markvoelker has joined #openstack-nova | 04:07 | |
| *** thorst has joined #openstack-nova | 04:08 | |
| *** shaohe_feng has quit IRC | 04:09 | |
| *** shaohe_feng has joined #openstack-nova | 04:09 | |
| *** tiendc_ has joined #openstack-nova | 04:09 | |
| *** gjayavel_ has joined #openstack-nova | 04:10 | |
| *** zhangjn has joined #openstack-nova | 04:11 | |
| *** markvoelker has quit IRC | 04:12 | |
| tiendc_ | dansmith: for the bp ironic serial console support --> https://blueprints.launchpad.net/nova/+spec/ironic-serial-console-support | 04:12 |
| tiendc_ | dansmith: this bp is depended on another RFE in Ironic | 04:13 |
| tiendc_ | and the patch-set take a long time for review https://review.openstack.org/#/c/328157/ | 04:13 |
| tiendc_ | dansmith: we are currently working on both side nova and ironic | 04:13 |
| tiendc_ | dansmith: but today is the deadline of non-priority feature freeze, can you take a look and think about move this bp to exception list | 04:14 |
| *** gjayavelu has quit IRC | 04:14 | |
| *** thorst has quit IRC | 04:15 | |
| *** ducttape_ has quit IRC | 04:16 | |
| *** amotoki has quit IRC | 04:18 | |
| *** shaohe_feng has quit IRC | 04:19 | |
| *** shaohe_feng has joined #openstack-nova | 04:22 | |
| *** Sukhdev has joined #openstack-nova | 04:25 | |
| *** Apoorva has quit IRC | 04:26 | |
| *** amotoki has joined #openstack-nova | 04:26 | |
| *** jwcroppe has joined #openstack-nova | 04:28 | |
| *** ayoung has quit IRC | 04:29 | |
| *** ljxiash has joined #openstack-nova | 04:29 | |
| *** shaohe_feng has quit IRC | 04:29 | |
| *** shaohe_feng has joined #openstack-nova | 04:30 | |
| *** jwcroppe has quit IRC | 04:30 | |
| *** diga has joined #openstack-nova | 04:30 | |
| *** harlowja_at_home has quit IRC | 04:30 | |
| *** jwcroppe has joined #openstack-nova | 04:33 | |
| *** ljxiash has quit IRC | 04:36 | |
| *** shaohe_feng has quit IRC | 04:40 | |
| *** shaohe_feng has joined #openstack-nova | 04:40 | |
| *** jwcroppe has quit IRC | 04:44 | |
| *** sdake has joined #openstack-nova | 04:49 | |
| *** irenab has quit IRC | 04:50 | |
| *** shaohe_feng has quit IRC | 04:50 | |
| *** shaohe_feng has joined #openstack-nova | 04:50 | |
| *** sdake_ has joined #openstack-nova | 04:51 | |
| *** sdake_ has quit IRC | 04:53 | |
| *** sdake has quit IRC | 04:54 | |
| *** abalutoiu has quit IRC | 04:54 | |
| *** dimtruck is now known as zz_dimtruck | 04:54 | |
| *** xenogear has quit IRC | 04:55 | |
| *** zhs_ss has joined #openstack-nova | 04:57 | |
| openstackgerrit | melanie witt proposed openstack/nova: WIP Get cell service version for compute RPC calls https://review.openstack.org/326906 | 04:57 |
| *** zhs_s has quit IRC | 04:57 | |
| *** salv-orlando has joined #openstack-nova | 05:00 | |
| *** shaohe_feng has quit IRC | 05:00 | |
| *** shaohe_feng has joined #openstack-nova | 05:00 | |
| *** jwcroppe has joined #openstack-nova | 05:01 | |
| *** salv-orl_ has quit IRC | 05:03 | |
| openstackgerrit | Aihua Edward Li proposed openstack/nova: Always clean up network resource when VM creation failed close-bug: #1597596 https://review.openstack.org/335788 | 05:03 |
| openstack | bug 1597596 in OpenStack Compute (nova) "network not always cleaned up when spawning VMs" [Undecided,New] https://launchpad.net/bugs/1597596 - Assigned to Aihua Edward Li (aihuaedwardli) | 05:03 |
| *** sudipto has joined #openstack-nova | 05:04 | |
| *** s1061123 has joined #openstack-nova | 05:04 | |
| *** jwcroppe has quit IRC | 05:05 | |
| *** ratailor has joined #openstack-nova | 05:06 | |
| *** GB21 has joined #openstack-nova | 05:06 | |
| *** moshele has joined #openstack-nova | 05:06 | |
| *** ljxiash has joined #openstack-nova | 05:10 | |
| *** shaohe_feng has quit IRC | 05:10 | |
| *** klkumar has joined #openstack-nova | 05:11 | |
| *** shaohe_feng has joined #openstack-nova | 05:11 | |
| *** gjayavel_ has quit IRC | 05:12 | |
| *** prateek has joined #openstack-nova | 05:12 | |
| *** thorst has joined #openstack-nova | 05:13 | |
| *** abalutoiu has joined #openstack-nova | 05:13 | |
| *** zerda2 has joined #openstack-nova | 05:14 | |
| *** prateek has quit IRC | 05:17 | |
| *** lpetrut has joined #openstack-nova | 05:18 | |
| *** edand has joined #openstack-nova | 05:18 | |
| *** thorst has quit IRC | 05:21 | |
| *** irenab has joined #openstack-nova | 05:21 | |
| *** shaohe_feng has quit IRC | 05:21 | |
| *** shaohe_feng has joined #openstack-nova | 05:21 | |
| *** rama_y has joined #openstack-nova | 05:22 | |
| openstackgerrit | Dao Cong Tien proposed openstack/nova: Ironic Driver: override get_serial_console() https://review.openstack.org/328157 | 05:22 |
| *** sdake has joined #openstack-nova | 05:24 | |
| *** rcernin has joined #openstack-nova | 05:26 | |
| *** rama_y has quit IRC | 05:26 | |
| *** rama_y has joined #openstack-nova | 05:27 | |
| *** sacharya_ has quit IRC | 05:27 | |
| *** sacharya has joined #openstack-nova | 05:27 | |
| *** vtech has joined #openstack-nova | 05:30 | |
| *** jwcroppe has joined #openstack-nova | 05:30 | |
| *** shaohe_feng has quit IRC | 05:31 | |
| *** shaohe_feng has joined #openstack-nova | 05:32 | |
| *** code-R has quit IRC | 05:32 | |
| *** code-R has joined #openstack-nova | 05:32 | |
| *** liverpooler has joined #openstack-nova | 05:33 | |
| *** rcernin has quit IRC | 05:34 | |
| *** abalutoiu has quit IRC | 05:34 | |
| *** jichen has quit IRC | 05:35 | |
| *** prateek has joined #openstack-nova | 05:35 | |
| *** jichen has joined #openstack-nova | 05:35 | |
| *** sdake_ has joined #openstack-nova | 05:36 | |
| *** sdake has quit IRC | 05:37 | |
| *** code-R_ has joined #openstack-nova | 05:39 | |
| *** Daisy has quit IRC | 05:41 | |
| *** Daisy has joined #openstack-nova | 05:41 | |
| *** shaohe_feng has quit IRC | 05:41 | |
| *** shaohe_feng has joined #openstack-nova | 05:42 | |
| *** code-R has quit IRC | 05:42 | |
| *** fragatina has joined #openstack-nova | 05:46 | |
| *** ducttape_ has joined #openstack-nova | 05:46 | |
| *** hoangcx has joined #openstack-nova | 05:47 | |
| *** Sukhdev has quit IRC | 05:47 | |
| *** bswartz has quit IRC | 05:47 | |
| *** s1061123 has quit IRC | 05:47 | |
| *** chandankumar has joined #openstack-nova | 05:48 | |
| *** abalutoiu has joined #openstack-nova | 05:49 | |
| *** nkrinner_afk is now known as nkrinner | 05:50 | |
| *** Daisy has quit IRC | 05:51 | |
| *** Daisy has joined #openstack-nova | 05:51 | |
| *** shaohe_feng has quit IRC | 05:51 | |
| *** ducttape_ has quit IRC | 05:52 | |
| *** shaohe_feng has joined #openstack-nova | 05:52 | |
| *** Daisy has quit IRC | 05:53 | |
| *** code-R has joined #openstack-nova | 05:53 | |
| *** Daisy has joined #openstack-nova | 05:53 | |
| *** code-R_ has quit IRC | 05:54 | |
| *** code-R_ has joined #openstack-nova | 05:56 | |
| *** fragatina has quit IRC | 05:57 | |
| *** fragatina has joined #openstack-nova | 05:57 | |
| *** Daisy has quit IRC | 05:58 | |
| *** vtech has quit IRC | 05:58 | |
| *** code-R has quit IRC | 05:59 | |
| *** pbandark has joined #openstack-nova | 06:01 | |
| *** Daisy has joined #openstack-nova | 06:02 | |
| *** shaohe_feng has quit IRC | 06:02 | |
| *** rama_y has quit IRC | 06:02 | |
| *** shaohe_feng has joined #openstack-nova | 06:03 | |
| *** pradk has quit IRC | 06:03 | |
| *** mfedosin has joined #openstack-nova | 06:04 | |
| *** muawiakhan has joined #openstack-nova | 06:04 | |
| *** diga has quit IRC | 06:05 | |
| gmann_ | johnthetubaguy: alex_xu alaski oomichi dansmith : can you check this (serial console support for Ironic driver) - https://review.openstack.org/#/c/328157/6 | 06:06 |
| gmann_ | johnthetubaguy: alex_xu alaski oomichi dansmith : i feel that should depends on Ironic patch which implement that on ironic side so that we do not publish the feature before ironic implementation gets merged | 06:07 |
| gmann_ | need your opinion | 06:08 |
| *** eantyshev has left #openstack-nova | 06:08 | |
| *** harlowja_at_home has joined #openstack-nova | 06:08 | |
| *** markvoelker has joined #openstack-nova | 06:09 | |
| *** rcernin has joined #openstack-nova | 06:09 | |
| *** paul-carlton2 has joined #openstack-nova | 06:09 | |
| *** shaohe_feng has quit IRC | 06:12 | |
| *** paul-carlton2 has quit IRC | 06:12 | |
| *** paul-carlton2 has joined #openstack-nova | 06:13 | |
| *** markvoelker has quit IRC | 06:13 | |
| *** shaohe_feng has joined #openstack-nova | 06:15 | |
| *** gjayavelu has joined #openstack-nova | 06:15 | |
| *** rook- has quit IRC | 06:17 | |
| *** chhavi has joined #openstack-nova | 06:19 | |
| *** vtech has joined #openstack-nova | 06:20 | |
| *** andreas_s has joined #openstack-nova | 06:22 | |
| *** shaohe_feng has quit IRC | 06:22 | |
| *** shaohe_feng has joined #openstack-nova | 06:23 | |
| *** liverpooler has quit IRC | 06:24 | |
| *** sridharg has joined #openstack-nova | 06:24 | |
| *** liverpooler has joined #openstack-nova | 06:24 | |
| openstackgerrit | octopuszhang proposed openstack/nova: Allow irrelevant,self-defined specs in ComputeCapacityFilter https://review.openstack.org/317306 | 06:26 |
| *** sdake_ has quit IRC | 06:30 | |
| *** chhavi has quit IRC | 06:31 | |
| *** shaohe_feng has quit IRC | 06:32 | |
| *** shaohe_feng has joined #openstack-nova | 06:33 | |
| *** pcaruana has joined #openstack-nova | 06:37 | |
| *** gjayavelu has quit IRC | 06:37 | |
| *** harlowja_at_home has quit IRC | 06:39 | |
| *** belmoreira has joined #openstack-nova | 06:40 | |
| *** shaohe_feng has quit IRC | 06:43 | |
| *** shaohe_feng has joined #openstack-nova | 06:43 | |
| *** jichen has quit IRC | 06:44 | |
| *** ducttape_ has joined #openstack-nova | 06:48 | |
| *** zenoway has joined #openstack-nova | 06:48 | |
| *** luis5tb has joined #openstack-nova | 06:50 | |
| *** sacharya has quit IRC | 06:51 | |
| *** shaohe_feng has quit IRC | 06:53 | |
| *** ducttape_ has quit IRC | 06:53 | |
| *** shaohe_feng has joined #openstack-nova | 06:54 | |
| *** mpavone has joined #openstack-nova | 06:56 | |
| *** Daisy has quit IRC | 06:57 | |
| *** Daisy has joined #openstack-nova | 06:57 | |
| *** jwcroppe has quit IRC | 06:59 | |
| *** Daisy has quit IRC | 07:00 | |
| *** Daisy has joined #openstack-nova | 07:00 | |
| openstackgerrit | Hieu LE proposed openstack/nova: Remove mox from unit/virt/libvirt/test_imagebackend.py (3) https://review.openstack.org/335765 | 07:01 |
| *** paul-carlton2 has quit IRC | 07:01 | |
| openstackgerrit | Hieu LE proposed openstack/nova: Remove mox from unit/virt/libvirt/test_imagebackend.py (2) https://review.openstack.org/335411 | 07:01 |
| openstackgerrit | Hieu LE proposed openstack/nova: Remove mox from unit/virt/libvirt/test_imagebackend.py (1) https://review.openstack.org/334409 | 07:01 |
| openstackgerrit | Hieu LE proposed openstack/nova: Remove mox from unit/virt/libvirt/test_imagebackend.py (4) https://review.openstack.org/335820 | 07:01 |
| *** sahid has joined #openstack-nova | 07:01 | |
| *** paul-carlton2 has joined #openstack-nova | 07:01 | |
| *** shaohe_feng has quit IRC | 07:03 | |
| *** diga has joined #openstack-nova | 07:03 | |
| *** shaohe_feng has joined #openstack-nova | 07:04 | |
| *** aloga has quit IRC | 07:06 | |
| *** aloga has joined #openstack-nova | 07:06 | |
| *** markvoelker has joined #openstack-nova | 07:10 | |
| *** mfeoktistov has joined #openstack-nova | 07:11 | |
| *** thorst has joined #openstack-nova | 07:11 | |
| *** shaohe_feng has quit IRC | 07:13 | |
| *** markvoelker has quit IRC | 07:14 | |
| *** shaohe_feng has joined #openstack-nova | 07:14 | |
| *** tesseract- has joined #openstack-nova | 07:15 | |
| *** hoangcx has quit IRC | 07:15 | |
| *** roeyc has joined #openstack-nova | 07:15 | |
| *** pilgrimstack has joined #openstack-nova | 07:17 | |
| gmann_ | alex_xu: dansmith : regarding this - https://review.openstack.org/#/c/304510/64 | 07:19 |
| gmann_ | alex_xu: dansmith we are not doing tag on volume-attach and interface-attach API ? | 07:19 |
| gmann_ | those were mentioned in spec | 07:20 |
| *** thorst has quit IRC | 07:20 | |
| gmann_ | in this - https://specs.openstack.org/openstack/nova-specs/specs/newton/approved/virt-device-role-tagging.html | 07:20 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: WIP: Allow multi-attach in compute api https://review.openstack.org/271047 | 07:22 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: libvirt: Allow multiple volume attachments https://review.openstack.org/267587 | 07:22 |
| *** gszasz has joined #openstack-nova | 07:22 | |
| *** paul-carlton2 has quit IRC | 07:23 | |
| *** shaohe_feng has quit IRC | 07:24 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: Pre-add instance actions to avoid merge conflicts https://review.openstack.org/330513 | 07:25 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: Transform instance.delete notifications https://review.openstack.org/313654 | 07:25 |
| *** shaohe_feng has joined #openstack-nova | 07:25 | |
| *** GB21 has quit IRC | 07:25 | |
| *** jichen has joined #openstack-nova | 07:28 | |
| *** jlanoux has joined #openstack-nova | 07:30 | |
| *** jlanoux has quit IRC | 07:30 | |
| *** jlanoux has joined #openstack-nova | 07:31 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: Pre-add instance actions to avoid merge conflicts https://review.openstack.org/330513 | 07:33 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: Transform instance.delete notifications https://review.openstack.org/313654 | 07:33 |
| *** shaohe_feng has quit IRC | 07:34 | |
| *** shaohe_feng has joined #openstack-nova | 07:34 | |
| *** fragatina has quit IRC | 07:36 | |
| *** paul-carlton2 has joined #openstack-nova | 07:37 | |
| *** zz_dimtruck is now known as dimtruck | 07:37 | |
| *** adisky has joined #openstack-nova | 07:38 | |
| *** dedery has joined #openstack-nova | 07:41 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Imported Translations from Zanata https://review.openstack.org/331996 | 07:41 |
| *** dedery has joined #openstack-nova | 07:41 | |
| *** itamarl has joined #openstack-nova | 07:41 | |
| *** vtech has quit IRC | 07:41 | |
| *** liverpooler has quit IRC | 07:43 | |
| *** liverpooler has joined #openstack-nova | 07:43 | |
| *** penick_ has joined #openstack-nova | 07:44 | |
| *** penick has quit IRC | 07:44 | |
| *** penick_ is now known as penick | 07:44 | |
| *** mfedosin has quit IRC | 07:44 | |
| *** shaohe_feng has quit IRC | 07:44 | |
| *** shaohe_feng has joined #openstack-nova | 07:45 | |
| *** GB21 has joined #openstack-nova | 07:46 | |
| *** jwcroppe has joined #openstack-nova | 07:48 | |
| *** ducttape_ has joined #openstack-nova | 07:50 | |
| *** fragatina has joined #openstack-nova | 07:50 | |
| *** code-R has joined #openstack-nova | 07:52 | |
| *** code-R_ has quit IRC | 07:52 | |
| *** ducttape_ has quit IRC | 07:54 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: Pre-add instance actions to avoid merge conflicts https://review.openstack.org/330513 | 07:54 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: Transform instance.delete notifications https://review.openstack.org/313654 | 07:54 |
| *** shaohe_feng has quit IRC | 07:54 | |
| *** shaohe_feng has joined #openstack-nova | 07:55 | |
| openstackgerrit | Evgeny Antyshev proposed openstack/nova: Release note for vzstorage volume driver https://review.openstack.org/335439 | 07:56 |
| *** browne has quit IRC | 07:58 | |
| *** zzzeek has quit IRC | 08:00 | |
| bauzas | good morning Nova | 08:00 |
| *** pilgrimstack1 has joined #openstack-nova | 08:00 | |
| *** pilgrimstack has quit IRC | 08:00 | |
| *** zzzeek has joined #openstack-nova | 08:00 | |
| *** jichen has quit IRC | 08:01 | |
| *** jichen has joined #openstack-nova | 08:02 | |
| openstackgerrit | Zhihai Song proposed openstack/nova: Transform instance.restore notifications https://review.openstack.org/331972 | 08:02 |
| openstackgerrit | Zhihai Song proposed openstack/nova: Transform instance.power_on notifications https://review.openstack.org/329746 | 08:03 |
| *** derekh has joined #openstack-nova | 08:04 | |
| *** shaohe_feng has quit IRC | 08:05 | |
| *** jlanoux_ has joined #openstack-nova | 08:05 | |
| *** shaohe_feng has joined #openstack-nova | 08:05 | |
| openstackgerrit | jichenjc proposed openstack/nova: refactory query user_id code for quota-sets https://review.openstack.org/280165 | 08:06 |
| *** pilgrimstack1 has quit IRC | 08:06 | |
| *** pilgrimstack has joined #openstack-nova | 08:06 | |
| *** jlanoux has quit IRC | 08:08 | |
| *** gmann_ has quit IRC | 08:08 | |
| *** gmann has joined #openstack-nova | 08:09 | |
| *** mjura has quit IRC | 08:10 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: [WIP] Testing latest u-c https://review.openstack.org/303345 | 08:10 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: [WIP] Testing latest u-c https://review.openstack.org/303345 | 08:10 |
| *** markus_z has joined #openstack-nova | 08:10 | |
| *** mjura has joined #openstack-nova | 08:10 | |
| *** markvoelker has joined #openstack-nova | 08:11 | |
| *** takashin has joined #openstack-nova | 08:11 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Add swap volume notifications (start, end) https://review.openstack.org/250283 | 08:12 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Add swap volume notifications (error) https://review.openstack.org/328055 | 08:13 |
| openstackgerrit | John Hua proposed openstack/nova: POC: WIP: XenAPI: device tagging https://review.openstack.org/333781 | 08:13 |
| *** shaohe_feng has quit IRC | 08:15 | |
| *** markvoelker has quit IRC | 08:15 | |
| *** zhs_ss has quit IRC | 08:15 | |
| *** zhs_ss has joined #openstack-nova | 08:16 | |
| *** jamielennox is now known as jamielennox|away | 08:16 | |
| *** danpb has joined #openstack-nova | 08:17 | |
| *** shaohe_feng has joined #openstack-nova | 08:17 | |
| *** thorst has joined #openstack-nova | 08:18 | |
| *** shz has joined #openstack-nova | 08:18 | |
| *** shihanzhang has quit IRC | 08:19 | |
| openstackgerrit | Gene Kuo proposed openstack/nova: Fix nova delete with EFI booted VMs https://review.openstack.org/335512 | 08:20 |
| openstackgerrit | Lee Yarwood proposed openstack/nova: libvirt: Implement stable disk device instance rescues https://review.openstack.org/270289 | 08:22 |
| *** shaohe_feng has quit IRC | 08:25 | |
| *** thorst has quit IRC | 08:25 | |
| *** shaohe_feng has joined #openstack-nova | 08:26 | |
| *** zhangyufei has joined #openstack-nova | 08:26 | |
| *** Daisy has quit IRC | 08:27 | |
| *** Daisy has joined #openstack-nova | 08:28 | |
| *** tbachman_ has quit IRC | 08:28 | |
| *** lucas-afk is now known as lucasagomes | 08:30 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: Pre-add instance actions to avoid merge conflicts https://review.openstack.org/330513 | 08:31 |
| *** tbachman has joined #openstack-nova | 08:31 | |
| openstackgerrit | Daniel Berrange proposed openstack/nova: DO NOT MERGE - ROOAAAAAARRRR ! Testing live migration bug. https://review.openstack.org/335549 | 08:31 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: Transform instance.delete notifications https://review.openstack.org/313654 | 08:31 |
| openstackgerrit | Dao Cong Tien proposed openstack/nova: Ironic Driver: override get_serial_console() https://review.openstack.org/328157 | 08:32 |
| *** jwcroppe has quit IRC | 08:32 | |
| *** Daisy has quit IRC | 08:32 | |
| openstackgerrit | jichenjc proposed openstack/nova: Verify os-aggregates.inc on sample files https://review.openstack.org/333114 | 08:34 |
| *** shaohe_feng has quit IRC | 08:35 | |
| *** shaohe_feng has joined #openstack-nova | 08:36 | |
| *** pradk has joined #openstack-nova | 08:37 | |
| *** muawiakhan has quit IRC | 08:37 | |
| *** muawiakhan has joined #openstack-nova | 08:37 | |
| openstackgerrit | John Garbutt proposed openstack/nova: Always call port_update in allocate_for_instance https://review.openstack.org/326970 | 08:38 |
| openstackgerrit | John Hua proposed openstack/nova: POC: WIP: XenAPI: device tagging https://review.openstack.org/333781 | 08:39 |
| *** danpb has quit IRC | 08:41 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Add swap volume notifications (start, end) https://review.openstack.org/250283 | 08:42 |
| *** danpb has joined #openstack-nova | 08:42 | |
| alex_xu | gmann: good point | 08:43 |
| markus_z | kaisers: Hi, I've seen your bug report https://bugs.launchpad.net/nova/+bug/1597644 | 08:43 |
| openstack | Launchpad bug 1597644 in OpenStack Compute (nova) "Permissioned denied on console.log during instance startup" [Undecided,New] | 08:43 |
| markus_z | kaisers: The link you provided doesn't open for me. | 08:44 |
| markus_z | kaisers: I assume you mean this one http://78.46.57.153:8081/refs-changes-30-260930-40/ ? | 08:44 |
| *** sheel has quit IRC | 08:45 | |
| *** shaohe_feng has quit IRC | 08:46 | |
| *** shaohe_feng has joined #openstack-nova | 08:46 | |
| markus_z | kaisers: I was wondering why the domain XML has a "serial type=file" and a "console type=file" at the same time. | 08:47 |
| *** ihrachys has joined #openstack-nova | 08:47 | |
| *** yassine has joined #openstack-nova | 08:49 | |
| *** yassine is now known as Guest85383 | 08:49 | |
| *** ducttape_ has joined #openstack-nova | 08:50 | |
| *** s1061123 has joined #openstack-nova | 08:53 | |
| *** roeyc has quit IRC | 08:55 | |
| *** ducttape_ has quit IRC | 08:55 | |
| *** zenoway has quit IRC | 08:56 | |
| *** shaohe_feng has quit IRC | 08:56 | |
| *** jwcroppe has joined #openstack-nova | 08:56 | |
| *** shaohe_feng has joined #openstack-nova | 08:56 | |
| *** imcsk8_ has joined #openstack-nova | 08:57 | |
| *** bapalm has quit IRC | 08:57 | |
| *** zhs_s has joined #openstack-nova | 08:57 | |
| *** mnestratov|2 has joined #openstack-nova | 08:59 | |
| *** PaulMurray has joined #openstack-nova | 08:59 | |
| *** ildikov_ has joined #openstack-nova | 08:59 | |
| *** bapalm has joined #openstack-nova | 09:00 | |
| *** kfox1111 has joined #openstack-nova | 09:00 | |
| *** fragatina has quit IRC | 09:00 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Add swap volume notifications (start, end) https://review.openstack.org/250283 | 09:00 |
| *** jwcroppe has quit IRC | 09:01 | |
| openstackgerrit | Paul Carlton proposed openstack/nova: Permit Live Migration of Rescued Instances https://review.openstack.org/308198 | 09:01 |
| openstackgerrit | Paul Carlton proposed openstack/nova: API changes for live migration of rescued instance https://review.openstack.org/328280 | 09:01 |
| *** mrsoul` has joined #openstack-nova | 09:02 | |
| *** migi_ has joined #openstack-nova | 09:02 | |
| *** bradm_ has joined #openstack-nova | 09:03 | |
| *** zehua- has joined #openstack-nova | 09:03 | |
| *** zenoway has joined #openstack-nova | 09:03 | |
| *** aloga has quit IRC | 09:04 | |
| *** aloga has joined #openstack-nova | 09:04 | |
| *** Dave_ has joined #openstack-nova | 09:04 | |
| *** auggy_ has joined #openstack-nova | 09:05 | |
| *** alex_xu_ has joined #openstack-nova | 09:06 | |
| *** markmc` has joined #openstack-nova | 09:06 | |
| *** shaohe_feng has quit IRC | 09:06 | |
| *** shaohe_feng has joined #openstack-nova | 09:06 | |
| *** mirrorbo1 has joined #openstack-nova | 09:07 | |
| *** sfinucan_ has joined #openstack-nova | 09:07 | |
| *** afazekas_ has joined #openstack-nova | 09:07 | |
| *** zhs_ss has quit IRC | 09:08 | |
| *** sudipto has quit IRC | 09:08 | |
| *** diogogmt has quit IRC | 09:08 | |
| *** mirrorbox has quit IRC | 09:08 | |
| *** imcsk8 has quit IRC | 09:08 | |
| *** ericksonsantos has quit IRC | 09:08 | |
| *** alex_xu has quit IRC | 09:08 | |
| *** mnestratov has quit IRC | 09:08 | |
| *** chenfei has quit IRC | 09:08 | |
| *** jinquan has quit IRC | 09:08 | |
| *** sfinucan has quit IRC | 09:08 | |
| *** FL1SK has quit IRC | 09:08 | |
| *** afazekas has quit IRC | 09:08 | |
| *** kfox1111_ has quit IRC | 09:08 | |
| *** j_king has quit IRC | 09:08 | |
| *** Dave has quit IRC | 09:08 | |
| *** bradm has quit IRC | 09:08 | |
| *** mordred has quit IRC | 09:08 | |
| *** mrsoul has quit IRC | 09:08 | |
| *** zehua has quit IRC | 09:08 | |
| *** auggy has quit IRC | 09:08 | |
| *** migi has quit IRC | 09:08 | |
| *** ildikov has quit IRC | 09:08 | |
| *** markmc has quit IRC | 09:08 | |
| *** rmk has quit IRC | 09:08 | |
| *** konetzed has quit IRC | 09:08 | |
| *** zehua- is now known as zehua | 09:08 | |
| *** tiendc_ has quit IRC | 09:09 | |
| *** rmk has joined #openstack-nova | 09:09 | |
| *** j_king has joined #openstack-nova | 09:09 | |
| *** sudipto has joined #openstack-nova | 09:09 | |
| *** ildikov_ is now known as ildikov | 09:09 | |
| *** edand has quit IRC | 09:09 | |
| *** auggy_ is now known as auggy | 09:10 | |
| *** chenfei has joined #openstack-nova | 09:10 | |
| *** mordred has joined #openstack-nova | 09:12 | |
| *** ericksonsantos has joined #openstack-nova | 09:14 | |
| *** jinquan has joined #openstack-nova | 09:15 | |
| *** Daisy has joined #openstack-nova | 09:16 | |
| *** FL1SK has joined #openstack-nova | 09:16 | |
| *** shaohe_feng has quit IRC | 09:16 | |
| *** shaohe_feng has joined #openstack-nova | 09:17 | |
| *** Daisy has quit IRC | 09:18 | |
| *** Daisy has joined #openstack-nova | 09:18 | |
| *** mrsoul` is now known as mrsoul | 09:19 | |
| *** matrohon has joined #openstack-nova | 09:20 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Add swap volume notifications (error) https://review.openstack.org/328055 | 09:20 |
| *** baoli has joined #openstack-nova | 09:22 | |
| *** thorst has joined #openstack-nova | 09:24 | |
| *** mfedosin has joined #openstack-nova | 09:25 | |
| *** paul-carlton has joined #openstack-nova | 09:25 | |
| *** paul-carlton2 has quit IRC | 09:26 | |
| *** konetzed has joined #openstack-nova | 09:26 | |
| *** paul-carlton has quit IRC | 09:26 | |
| *** baoli has quit IRC | 09:26 | |
| *** paul-carlton2 has joined #openstack-nova | 09:26 | |
| *** Daisy has quit IRC | 09:27 | |
| *** shaohe_feng has quit IRC | 09:27 | |
| *** Daisy has joined #openstack-nova | 09:27 | |
| *** shaohe_feng has joined #openstack-nova | 09:27 | |
| *** Dave_ is now known as Dave | 09:28 | |
| *** GB21 has quit IRC | 09:29 | |
| *** jinquan has quit IRC | 09:29 | |
| *** sameo has joined #openstack-nova | 09:29 | |
| *** thorst has quit IRC | 09:30 | |
| markus_z | kaisers: I left a possible workaround|hack in the bug report. | 09:30 |
| *** ljxiash has quit IRC | 09:32 | |
| openstackgerrit | Lee Yarwood proposed openstack/nova: WIP - libvirt: Use the recreated config disk during a rescue https://review.openstack.org/335884 | 09:34 |
| *** hoangcx has joined #openstack-nova | 09:34 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Add swap volume notifications samples https://review.openstack.org/328866 | 09:34 |
| *** shaohe_feng has quit IRC | 09:37 | |
| *** konetzed has quit IRC | 09:37 | |
| *** shaohe_feng has joined #openstack-nova | 09:39 | |
| openstackgerrit | Paul Carlton proposed openstack/nova: Permit Live Migration of Rescued Instances https://review.openstack.org/308198 | 09:40 |
| openstackgerrit | Paul Carlton proposed openstack/nova: API changes for live migration of rescued instance https://review.openstack.org/328280 | 09:41 |
| bauzas | gibi_: I had a question in https://review.openstack.org/#/c/313654/41 | 09:45 |
| bauzas | gibi_: oh snap, nvm | 09:45 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: DO NOT MERGE - ROOAAAAAARRRR ! Testing live migration bug. https://review.openstack.org/335549 | 09:45 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: DO NOT MERGE - ROOAAAAAARRRR ! Testing live migration bug. https://review.openstack.org/335901 | 09:45 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: DO NOT MERGE - ROOAAAAAARRRR ! Testing live migration bug. https://review.openstack.org/335902 | 09:45 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: DO NOT MERGE - ROOAAAAAARRRR ! Testing live migration bug. https://review.openstack.org/335903 | 09:45 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: DO NOT MERGE - ROOAAAAAARRRR ! Testing live migration bug. https://review.openstack.org/335904 | 09:45 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: DO NOT MERGE - ROOAAAAAARRRR ! Testing live migration bug. https://review.openstack.org/335905 | 09:45 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: DO NOT MERGE - ROOAAAAAARRRR ! Testing live migration bug. https://review.openstack.org/335906 | 09:45 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: DO NOT MERGE - ROOAAAAAARRRR ! Testing live migration bug. https://review.openstack.org/335907 | 09:45 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: DO NOT MERGE - ROOAAAAAARRRR ! Testing live migration bug. https://review.openstack.org/335908 | 09:45 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: DO NOT MERGE - ROOAAAAAARRRR ! Testing live migration bug. https://review.openstack.org/335909 | 09:45 |
| * danpb hopes that will speed up reproducing the non-deterministic live migration bug :-) | 09:46 | |
| *** shaohe_feng has quit IRC | 09:47 | |
| *** shaohe_feng has joined #openstack-nova | 09:47 | |
| andrearosa | danpb: +1 for the ascii art | 09:49 |
| PaulMurray | danpb, that's got to be the best commit message I have seen | 09:49 |
| danpb | PaulMurray: which one, they're all different :-) | 09:49 |
| *** sree has quit IRC | 09:50 | |
| PaulMurray | danpb, I like the tyrannosaurus - probably says something about my character | 09:51 |
| *** ljxiash has joined #openstack-nova | 09:51 | |
| artom | gmann, alex_xu_, answer to your query in https://review.openstack.org/#/c/304510/ - I hope it it satisfactory :) | 09:52 |
| *** breton has quit IRC | 09:52 | |
| kashyap | Ah, dan submitted multiple invocations in one go. | 09:53 |
| alex_xu_ | artom: it is an agreement with the team? | 09:53 |
| artom | alex_xu_, hrmm, I don't think it's been explicitly described as such | 09:54 |
| *** sameo has quit IRC | 09:56 | |
| artom | alex_xu_, but for example here: https://review.openstack.org/#/c/303335/ | 09:56 |
| artom | We're talking about the changes that need to to happen to the config drive, and I understand it as dansmith and alaski being onboard | 09:57 |
| artom | But best to wait until they are here :) | 09:57 |
| *** shaohe_feng has quit IRC | 09:57 | |
| gmann | artom: yea, actually spec still have those, that's why got confused | 09:57 |
| artom | gmann, for sure | 09:58 |
| *** shaohe_feng has joined #openstack-nova | 09:58 | |
| gmann | artom: Thanks :) | 09:58 |
| *** GB21 has joined #openstack-nova | 09:58 | |
| artom | I totally get the confusion. There's been specs that have been implemented over multiple cycles before, no? | 09:58 |
| gmann | artom: yea but here we are doing microversion on device tag and its always good to do all related changes in one version bump instead of multiple | 10:00 |
| artom | gmann, ah, yeah, good point. Definitely cleaner to not bump twice for similar things | 10:01 |
| gmann | yea | 10:01 |
| PaulMurray | tdurakov, what's happening with https://review.openstack.org/#/c/314932/ ? | 10:03 |
| openstackgerrit | John Garbutt proposed openstack/nova: Always call port_update in allocate_for_instance https://review.openstack.org/326970 | 10:04 |
| artom | Well, it's you guys's call :) On our side we'll grovel for an extension if the API thing is blocked and add tagged interface/volume if we get it. | 10:04 |
| *** shhjwang has joined #openstack-nova | 10:05 | |
| *** wangqun has quit IRC | 10:06 | |
| openstackgerrit | John Garbutt proposed openstack/nova: Always call port_update in allocate_for_instance https://review.openstack.org/326970 | 10:08 |
| *** shaohe_feng has quit IRC | 10:08 | |
| *** david-lyle has quit IRC | 10:09 | |
| *** shaohe_feng has joined #openstack-nova | 10:09 | |
| *** david-lyle_ has joined #openstack-nova | 10:09 | |
| *** fragatina has joined #openstack-nova | 10:10 | |
| *** pilgrimstack has quit IRC | 10:12 | |
| openstackgerrit | John Garbutt proposed openstack/nova: Create _validate_requested_port_ids https://review.openstack.org/329600 | 10:16 |
| openstackgerrit | John Garbutt proposed openstack/nova: Extract _validate_requested_network_ids https://review.openstack.org/329601 | 10:16 |
| openstackgerrit | John Garbutt proposed openstack/nova: Extract _filter_hypervisor_macs https://review.openstack.org/329569 | 10:16 |
| gibi_ | bauzas: thanks for the review | 10:17 |
| *** fragatina has quit IRC | 10:17 | |
| *** roeyc has joined #openstack-nova | 10:17 | |
| *** lchen__ has joined #openstack-nova | 10:17 | |
| *** shaohe_feng has quit IRC | 10:18 | |
| *** lchen_ has quit IRC | 10:18 | |
| *** shaohe_feng has joined #openstack-nova | 10:19 | |
| openstackgerrit | John Garbutt proposed openstack/nova: Reduce complexity in _stub_allocate_for_instance https://review.openstack.org/332086 | 10:22 |
| openstackgerrit | jichenjc proposed openstack/nova: Verify os-aggregates.inc on sample files https://review.openstack.org/333114 | 10:22 |
| openstackgerrit | John Garbutt proposed openstack/nova: Reorder allocate_for_instance preamble https://review.openstack.org/329850 | 10:22 |
| openstackgerrit | John Garbutt proposed openstack/nova: Make _validate_requested_network_ids return a dict https://review.openstack.org/329849 | 10:22 |
| *** thorst has joined #openstack-nova | 10:28 | |
| *** shaohe_feng has quit IRC | 10:28 | |
| *** fragatina has joined #openstack-nova | 10:28 | |
| *** shaohe_feng has joined #openstack-nova | 10:29 | |
| *** sridharg has quit IRC | 10:31 | |
| *** hshiina has joined #openstack-nova | 10:32 | |
| *** fragatina has quit IRC | 10:32 | |
| *** fragatina has joined #openstack-nova | 10:33 | |
| *** mdavidson has quit IRC | 10:33 | |
| *** thorst has quit IRC | 10:35 | |
| *** smatzek has joined #openstack-nova | 10:36 | |
| openstackgerrit | Alexis Lee proposed openstack/nova: Make LM downtime params mutable https://review.openstack.org/319254 | 10:36 |
| *** jichen has quit IRC | 10:37 | |
| *** hoangcx has quit IRC | 10:38 | |
| *** shaohe_feng has quit IRC | 10:38 | |
| *** jichen has joined #openstack-nova | 10:39 | |
| openstackgerrit | Hieu LE proposed openstack/nova: Remove mox from unit/virt/libvirt/test_imagebackend.py (5) https://review.openstack.org/335921 | 10:40 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: Transform instance.update notification https://review.openstack.org/325923 | 10:40 |
| openstackgerrit | John Garbutt proposed openstack/nova: Extract port create from allocate_for_instance https://review.openstack.org/328966 | 10:40 |
| *** shaohe_feng has joined #openstack-nova | 10:41 | |
| gibi_ | johnthetubaguy, danpb: I had to rebase https://review.openstack.org/#/c/325923/ and fix the test due to a change on master. Could you re-check it please? | 10:42 |
| *** frickler has quit IRC | 10:43 | |
| *** Guest79590 has quit IRC | 10:43 | |
| *** Daisy has quit IRC | 10:43 | |
| *** _fortis has quit IRC | 10:43 | |
| *** Daisy has joined #openstack-nova | 10:43 | |
| *** toabctl has quit IRC | 10:44 | |
| *** jichen_ has joined #openstack-nova | 10:44 | |
| danpb | gibi_: so iiuc the change was instance_type_id being renamed to flavor_id ? | 10:45 |
| *** jichen has quit IRC | 10:45 | |
| *** frickler has joined #openstack-nova | 10:45 | |
| *** toabctl has joined #openstack-nova | 10:46 | |
| gibi_ | danpb: that was a fix in the instance.delete patch propagated to the instance.update. But besides that some neutron v2 interaction changed on master that caused that the sample test needed adjustem in the NeutronFixture | 10:46 |
| *** toabctl has quit IRC | 10:47 | |
| * danpb wishes gerrit could show you a proper diff-of-diffs between two versions | 10:47 | |
| *** jinquan has joined #openstack-nova | 10:47 | |
| openstackgerrit | Hieu LE proposed openstack/nova: Remove mox from unit/virt/libvirt/test_imagebackend.py (end) https://review.openstack.org/335925 | 10:48 |
| BobBall | danpb: +100 | 10:48 |
| gibi_ | danpb: unfortunately if a rebase has merge conflict then we cannot really separate the code coming in due to the rebase from the code needed to solve the conflict | 10:48 |
| *** toabctl has joined #openstack-nova | 10:48 | |
| *** Daisy has quit IRC | 10:48 | |
| *** shaohe_feng has quit IRC | 10:49 | |
| openstackgerrit | John Garbutt proposed openstack/nova: Extract port create from allocate_for_instance https://review.openstack.org/328966 | 10:49 |
| *** shaohe_feng has joined #openstack-nova | 10:49 | |
| *** jichen_ has quit IRC | 10:49 | |
| BobBall | But being able to diff(diff(base1,patch9,diff(base2,patch10)) would be very useful even if the two diffs aren't directly comparible because of a conflict... | 10:49 |
| *** _fortis has joined #openstack-nova | 10:50 | |
| *** vtech has joined #openstack-nova | 10:50 | |
| sahid | ahahah danpb you looks to have fun time with that serie https://review.openstack.org/#/q/topic:mig-debug | 10:53 |
| *** sridharg has joined #openstack-nova | 10:54 | |
| openstackgerrit | Paul Carlton proposed openstack/nova: Permit Live Migration of Rescued Instances https://review.openstack.org/308198 | 10:55 |
| openstackgerrit | Paul Carlton proposed openstack/nova: API changes for live migration of rescued instance https://review.openstack.org/328280 | 10:55 |
| kashyap | sahid: Epic commit messages, eh :-) | 10:55 |
| openstackgerrit | Alexis Lee proposed openstack/nova: Make LM downtime params mutable https://review.openstack.org/319254 | 10:55 |
| paul-carlton2 | johnthetubaguy, would you be able to review the above patches again pls | 10:56 |
| *** vtech has quit IRC | 10:56 | |
| openstackgerrit | John Garbutt proposed openstack/nova: Extract _update_ports_for_instance https://review.openstack.org/329851 | 10:56 |
| johnthetubaguy | paul-carlton2: I will try get to those again today, probably after lunch | 10:56 |
| openstackgerrit | John Garbutt proposed openstack/nova: Extract port create from allocate_for_instance https://review.openstack.org/328966 | 10:56 |
| paul-carlton2 | ok, thanks | 10:56 |
| *** shaohe_feng has quit IRC | 10:59 | |
| *** shaohe_feng has joined #openstack-nova | 10:59 | |
| *** sameo has joined #openstack-nova | 10:59 | |
| *** salv-orl_ has joined #openstack-nova | 11:00 | |
| *** phuongnh has quit IRC | 11:01 | |
| openstackgerrit | John Garbutt proposed openstack/nova: WIP: Make allocate_for_instance skip create option https://review.openstack.org/332712 | 11:02 |
| openstackgerrit | John Garbutt proposed openstack/nova: WIP: Make allocate_for_instance skip create option https://review.openstack.org/332712 | 11:03 |
| openstackgerrit | John Garbutt proposed openstack/nova: Create _validate_requested_port_ids https://review.openstack.org/329600 | 11:03 |
| openstackgerrit | John Garbutt proposed openstack/nova: Reduce complexity in _stub_allocate_for_instance https://review.openstack.org/332086 | 11:03 |
| openstackgerrit | John Garbutt proposed openstack/nova: Extract _validate_requested_network_ids https://review.openstack.org/329601 | 11:03 |
| openstackgerrit | John Garbutt proposed openstack/nova: Reorder allocate_for_instance preamble https://review.openstack.org/329850 | 11:03 |
| openstackgerrit | John Garbutt proposed openstack/nova: Always call port_update in allocate_for_instance https://review.openstack.org/326970 | 11:03 |
| openstackgerrit | John Garbutt proposed openstack/nova: Extract _update_ports_for_instance https://review.openstack.org/329851 | 11:03 |
| openstackgerrit | John Garbutt proposed openstack/nova: Extract port create from allocate_for_instance https://review.openstack.org/328966 | 11:03 |
| openstackgerrit | John Garbutt proposed openstack/nova: Make _validate_requested_network_ids return a dict https://review.openstack.org/329849 | 11:03 |
| openstackgerrit | John Garbutt proposed openstack/nova: Extract _filter_hypervisor_macs https://review.openstack.org/329569 | 11:03 |
| *** salv-orlando has quit IRC | 11:03 | |
| *** salv-orl_ has quit IRC | 11:05 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova: Fixes compute API unit tests for python3 https://review.openstack.org/311483 | 11:05 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Add swap volume notifications (start, end) https://review.openstack.org/250283 | 11:06 |
| *** rook- has joined #openstack-nova | 11:06 | |
| *** sree has joined #openstack-nova | 11:08 | |
| *** shaohe_feng has quit IRC | 11:09 | |
| *** shaohe_feng has joined #openstack-nova | 11:10 | |
| *** sree has quit IRC | 11:12 | |
| *** EinstCrazy has quit IRC | 11:13 | |
| openstackgerrit | Alex Xu proposed openstack/nova: Allow irrelevant,self-defined specs in ComputeCapacityFilter https://review.openstack.org/317306 | 11:14 |
| *** claudiub has joined #openstack-nova | 11:15 | |
| *** rtheis has joined #openstack-nova | 11:16 | |
| *** mirrorbo1 is now known as mirrorbox | 11:19 | |
| *** mirrorbox has joined #openstack-nova | 11:19 | |
| *** shaohe_feng has quit IRC | 11:19 | |
| *** shaohe_feng has joined #openstack-nova | 11:20 | |
| *** bvanhav has joined #openstack-nova | 11:22 | |
| *** irenab has quit IRC | 11:26 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Add swap volume notifications (error) https://review.openstack.org/328055 | 11:28 |
| *** ducttape_ has joined #openstack-nova | 11:29 | |
| *** mvk has quit IRC | 11:29 | |
| *** shaohe_feng has quit IRC | 11:30 | |
| *** smcginnis has quit IRC | 11:30 | |
| *** smcginnis has joined #openstack-nova | 11:30 | |
| *** shaohe_feng has joined #openstack-nova | 11:30 | |
| *** ericksonsantos has quit IRC | 11:38 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Add swap volume notifications samples https://review.openstack.org/328866 | 11:39 |
| *** pilgrimstack has joined #openstack-nova | 11:39 | |
| *** shaohe_feng has quit IRC | 11:40 | |
| *** jinquan has quit IRC | 11:40 | |
| *** jinquan has joined #openstack-nova | 11:42 | |
| openstackgerrit | Matthew Booth proposed openstack/nova: Fix fake data returned by LibvirtDriverTestCase._disk_info https://review.openstack.org/328366 | 11:42 |
| openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Pass path to Image base class https://review.openstack.org/333271 | 11:42 |
| openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Improve mocking of Image objects in test_driver https://review.openstack.org/333242 | 11:42 |
| openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: rewrite test_rescue and test_rescue_config_drive https://review.openstack.org/335966 | 11:42 |
| mdbooth | lyarwood: ^^^ | 11:42 |
| mdbooth | lyarwood: Just going to grab lunch. A quick look at the top of that stack should tell you if it's what you want. | 11:42 |
| *** shaohe_feng has joined #openstack-nova | 11:43 | |
| *** jed56 has quit IRC | 11:45 | |
| *** sahid has quit IRC | 11:46 | |
| *** rfolco has joined #openstack-nova | 11:47 | |
| *** GB21 has quit IRC | 11:47 | |
| *** links has joined #openstack-nova | 11:48 | |
| *** links has quit IRC | 11:48 | |
| *** shaohe_feng has quit IRC | 11:50 | |
| *** sahid has joined #openstack-nova | 11:51 | |
| *** shaohe_feng has joined #openstack-nova | 11:51 | |
| *** svenkat has joined #openstack-nova | 11:51 | |
| *** thorst has joined #openstack-nova | 11:52 | |
| *** thorst has quit IRC | 11:53 | |
| *** thorst_ has joined #openstack-nova | 11:53 | |
| *** GB21 has joined #openstack-nova | 11:54 | |
| *** diga has quit IRC | 11:55 | |
| *** sdake has joined #openstack-nova | 11:55 | |
| *** ducttape_ has quit IRC | 11:57 | |
| *** sdake_ has joined #openstack-nova | 11:57 | |
| *** mvk has joined #openstack-nova | 11:57 | |
| *** eharney has quit IRC | 11:58 | |
| *** shaohe_feng has quit IRC | 12:00 | |
| *** sdake has quit IRC | 12:01 | |
| *** mgoddard_ has joined #openstack-nova | 12:01 | |
| *** fragatina has quit IRC | 12:01 | |
| *** shaohe_feng has joined #openstack-nova | 12:01 | |
| *** lucasagomes is now known as lucas-hungry | 12:03 | |
| *** jwcroppe has joined #openstack-nova | 12:03 | |
| *** mgoddard has quit IRC | 12:04 | |
| markus_z | danpb: https://review.openstack.org/#/c/334480/ has finally all the CI +1 it needs | 12:04 |
| *** chenfei has quit IRC | 12:05 | |
| *** pilgrimstack has quit IRC | 12:05 | |
| danpb | markus_z: lol, i hate bugs like that where you stare at it for hours missing the obvious | 12:05 |
| markus_z | danpb: Yeah, I was a little embarrassed. :/ | 12:06 |
| markus_z | Funny thing is, I looked at it for a second and thought for myself "yeah, no issue, python uses the last variable as return value automatically" ... | 12:06 |
| *** salv-orlando has joined #openstack-nova | 12:06 | |
| markus_z | bad thing is, it doesn't. Scala does... | 12:06 |
| markus_z | Sometimes it's bad to know multiple languages | 12:07 |
| *** Fdaisuke has quit IRC | 12:07 | |
| *** hshiina has quit IRC | 12:07 | |
| *** hieulq_ has joined #openstack-nova | 12:07 | |
| *** hshiina has joined #openstack-nova | 12:07 | |
| *** Fdaisuke has joined #openstack-nova | 12:07 | |
| danpb | markus_z: perl does too :-) | 12:07 |
| *** Qiming has quit IRC | 12:07 | |
| *** qadaemon has joined #openstack-nova | 12:08 | |
| *** nicolasbock_ has quit IRC | 12:08 | |
| mdbooth | And Ruby | 12:08 |
| *** nicolasbock has joined #openstack-nova | 12:09 | |
| *** qadaemon has quit IRC | 12:09 | |
| markus_z | Ah, I wasn't sure anymore with Ruby. But as Scala "borrowed" a lot from Ruby, it makes sense. | 12:09 |
| * mdbooth has never liked that language feature, though. | 12:09 | |
| *** hieulq has quit IRC | 12:09 | |
| * mdbooth likes things to be explicit | 12:09 | |
| *** chenfei has joined #openstack-nova | 12:09 | |
| danpb | and ruby shamelessly copied perl :-) | 12:09 |
| mdbooth | And perl copied everything simultaneously, including mutually exclusive design concepts :) | 12:10 |
| *** Qiming has joined #openstack-nova | 12:10 | |
| markus_z | Ha! So perl is guilty one! | 12:10 |
| mdbooth | perl: Yeah, I guess you could do it that way, too. | 12:11 |
| *** shaohe_feng has quit IRC | 12:11 | |
| markus_z | :D | 12:11 |
| * mdbooth is fond of perl | 12:11 | |
| *** shaohe_feng has joined #openstack-nova | 12:11 | |
| markus_z | yeah? I've looked at perl code samples and I was a little revolted by it. | 12:12 |
| mdbooth | It's very easy to write bad perl, but that doesn't mean you have to :) | 12:12 |
| mdbooth | Also, you have to love regular expressions. Pretty sure that's a rule. | 12:13 |
| *** pilgrimstack has joined #openstack-nova | 12:13 | |
| *** raildo-afk is now known as raildo | 12:13 | |
| *** markvoelker has joined #openstack-nova | 12:13 | |
| *** daemontool has joined #openstack-nova | 12:14 | |
| *** svenkat has quit IRC | 12:14 | |
| markus_z | hm, maybe I put it on my "languages to learn" list | 12:14 |
| *** salv-orlando has quit IRC | 12:14 | |
| *** svenkat has joined #openstack-nova | 12:14 | |
| *** apanaitiu has joined #openstack-nova | 12:14 | |
| *** markvoelker has quit IRC | 12:14 | |
| *** markvoelker has joined #openstack-nova | 12:15 | |
| mdbooth | Its time is gone, imho. | 12:15 |
| *** eharney has joined #openstack-nova | 12:15 | |
| *** zhangyufei has quit IRC | 12:15 | |
| markus_z | "go" is the cool kid now, isn't it? | 12:17 |
| *** vishwanathj has joined #openstack-nova | 12:17 | |
| * mdbooth only knows about the buzz. Haven't really looked at the language. Rust is very interesting, imho, but not for openstack. | 12:18 | |
| alaski | artom: gmann alex_xu_ just catching up on review comments, but I'm okay with splitting the API exposure to expose tags on boot now and then tags on attach later. That feels like a natural split to me, as compared to saying lets do nics now and bdms later. | 12:18 |
| *** svenkat has quit IRC | 12:18 | |
| alaski | as in it feels complete to me to say you can now tag things on boot | 12:19 |
| * markus_z crossed out "perl" from the list and added "rust" | 12:19 | |
| danpb | mdbooth: i'll come back in 10 years and see which of the "cool" new languages are still in use :-) | 12:19 |
| alex_xu_ | alaski: cool, i'm also think that is ok | 12:19 |
| mdbooth | danpb: Hehe, pretty much. | 12:19 |
| *** ociuhandu has joined #openstack-nova | 12:20 | |
| mdbooth | danpb: Rust will either take off, or inspire some other language to have all its most useful features. | 12:20 |
| mdbooth | It really does have some great ideas. | 12:20 |
| *** zerda2 has quit IRC | 12:21 | |
| *** shaohe_feng has quit IRC | 12:21 | |
| markus_z | mdbooth: What's your favorite idea? | 12:21 |
| *** shaohe_feng has joined #openstack-nova | 12:21 | |
| mdbooth | markus_z: The big one is its explicit object ownership model. | 12:21 |
| mdbooth | The language itself doesn't have GC (although it has a library which does GC), but it can still automatically manage lifecycles in many/most cases | 12:22 |
| mdbooth | Because lifespans are explicit in the language | 12:22 |
| artom | alaski, awesome :) | 12:22 |
| *** ericksonsantos has joined #openstack-nova | 12:23 | |
| markus_z | mdbooth: Ah, so the scope defines the lifetime, if I read that correctly. | 12:24 |
| *** hshiina has quit IRC | 12:24 | |
| *** Fdaisuke has quit IRC | 12:24 | |
| *** baoli_ has joined #openstack-nova | 12:24 | |
| *** Fdaisuke has joined #openstack-nova | 12:24 | |
| *** hshiina has joined #openstack-nova | 12:24 | |
| lyarwood | mdbooth: just back from lunch and looking at the rescue test rewrite now, thanks again | 12:25 |
| *** edmondsw has joined #openstack-nova | 12:25 | |
| mdbooth | markus_z: I haven't written any in over a year, so it's starting to get fuzzy in my head, but it's way more involved than that. You can pass pointers around and the compilers still knows statically when the object is out of scope. | 12:25 |
| mdbooth | Also, when it is accessible from another thread. | 12:25 |
| mdbooth | In Rust, the compiler knows all things. | 12:25 |
| *** ericksonsantos has quit IRC | 12:26 | |
| *** bswartz has joined #openstack-nova | 12:26 | |
| *** hieulq has joined #openstack-nova | 12:26 | |
| *** hieulq_ has quit IRC | 12:26 | |
| *** jamie_h has joined #openstack-nova | 12:27 | |
| *** paul-carlton2 has quit IRC | 12:27 | |
| markus_z | I see. That's definitely something for cold winter evenings, I'm going to have a look :) | 12:27 |
| *** rlrossit has joined #openstack-nova | 12:30 | |
| *** adisky has quit IRC | 12:30 | |
| *** paul-carlton2 has joined #openstack-nova | 12:31 | |
| openstackgerrit | Maciej Szankin proposed openstack/nova: config options: Improve help for SPICE https://review.openstack.org/317046 | 12:31 |
| *** shaohe_feng has quit IRC | 12:31 | |
| *** mdrabe has joined #openstack-nova | 12:32 | |
| *** shaohe_feng has joined #openstack-nova | 12:32 | |
| *** paul-carlton has joined #openstack-nova | 12:33 | |
| *** fragatina has joined #openstack-nova | 12:34 | |
| *** paul-carlton has quit IRC | 12:35 | |
| *** aloga has quit IRC | 12:36 | |
| *** aloga has joined #openstack-nova | 12:36 | |
| *** ericksonsantos has joined #openstack-nova | 12:37 | |
| *** mgoddard_ has quit IRC | 12:38 | |
| *** mgoddard has joined #openstack-nova | 12:38 | |
| mdbooth | artom vladikr: Another regression in I931421ea688641e2ceb212c6dc099639c53433f2, btw. For uml, it now tries to chown the root disk before it has been created. | 12:39 |
| mdbooth | I'm assuming that would actually prevent uml from working. Does anybody use that? | 12:39 |
| *** pilgrimstack has quit IRC | 12:41 | |
| *** shaohe_feng has quit IRC | 12:41 | |
| markus_z | danpb: During the virtlogd tests I've seen that the boot message won't get printed into the "console.log" file *unless* I'm already connected to the terminal (executing a 'reboot' for example). | 12:42 |
| markus_z | danpb: Folks from my Linux department looked into it and they guess that this could be an issue in qemu itself. | 12:42 |
| *** shaohe_feng has joined #openstack-nova | 12:42 | |
| *** ratailor has quit IRC | 12:42 | |
| artom | mdbooth, thanks for catching that | 12:43 |
| mdbooth | artom: Just writing a bug for it | 12:43 |
| *** pilgrimstack has joined #openstack-nova | 12:43 | |
| *** pradk has quit IRC | 12:43 | |
| markus_z | danpb: They said it looks like the qemu logd code gets only executed *after* a terminal connection, which prevents it to log the boot messages. | 12:43 |
| mdbooth | Caught it rebasing my patch series on yours :) | 12:43 |
| artom | To be honest I'm not sure what uml is, but it's definitely something we'll look into | 12:43 |
| *** salv-orlando has joined #openstack-nova | 12:43 | |
| mdbooth | uml == user mode linux | 12:43 |
| artom | mdbooth, btw, did you see vladikr's fix for the first bug? | 12:44 |
| artom | https://review.openstack.org/#/c/335676/ | 12:44 |
| markus_z | danpb: Maybe you can have a look at it? I tried to understand your change http://git.qemu.org/?p=qemu.git;a=blobdiff;f=qemu-char.c;h=11caa5648de99c9e0ee158f280fbc02ab05915d3;hp=d7be1851e5e9d268aa924a05958da292b048839c;hb=d0d7708ba29cbcc343364a46bff981e0ff88366f;hpb=f1c17521e79df863a5771d96974fab0d07f02be0 but I have no experience in qemu itself. | 12:44 |
| *** GB21 has quit IRC | 12:44 | |
| mdbooth | artom: Will look, thanks | 12:44 |
| *** dtantsur has joined #openstack-nova | 12:44 | |
| dtantsur | hi everyone! does this ring any bells? http://logs.openstack.org/58/272658/58/check/gate-grenade-dsvm-ironic/3436213/logs/new/screen-n-cpu.txt.gz?level=ERROR | 12:44 |
| dtantsur | ironic grenade jobs started failing due to it several hours ago | 12:45 |
| artom | mdbooth, tag me on the bug or something if you're filing it now please :) | 12:45 |
| mdbooth | artom: Ok | 12:45 |
| mdbooth | I actually have a patch which fixes it, but as it's patch 19 in my series, I don't mind if you fix it first :) | 12:45 |
| artom | mdbooth, haha - as long as it gets fixed by someone somewhere ;) | 12:46 |
| *** chandankumar has quit IRC | 12:46 | |
| *** xyang1 has joined #openstack-nova | 12:47 | |
| markus_z | dtantsur: Looks like the exception is raised here: https://github.com/openstack/nova/blob/0871f4953d99bc6e5b35dbe8ccf8644a282167d3/nova/db/sqlalchemy/api.py#L1524-L1524 | 12:48 |
| danpb | markus_z: hmm, its possible - will have a look | 12:48 |
| *** apanaitiu has quit IRC | 12:48 | |
| danpb | data will only be written to the log file when it is also written to the primary channel | 12:48 |
| markus_z | dtantsur: Without having the DB error it's hard to say what's wrong. Can you add a log and repeat it? | 12:48 |
| *** mdavidson has joined #openstack-nova | 12:49 | |
| mdbooth | artom: Consider yourself spammed :) | 12:49 |
| danpb | but i though that when no client is connected, the data would just be written to null effectively, so the logs would still be output | 12:49 |
| artom | mdbooth, as long as its now enlargement pills | 12:49 |
| artom | Err, not | 12:49 |
| * artom gulps more coffee | 12:49 | |
| dtantsur | markus_z, hmm, so it requires patching nova... what about NotImplementedError above? | 12:50 |
| *** tbachman has quit IRC | 12:50 | |
| alaski | dtantsur: artom might be related to https://review.openstack.org/#/c/264017/ | 12:50 |
| artom | alaski, it is? | 12:51 |
| artom | I'm seeing the virt driver raise a NotImplemented error on detach_interface, maybe I'm not reading deep enough | 12:52 |
| markus_z | danpb: In my setup it didn't. I did it on a system z machine, if that's important. I wasn't able to get a setup running on x86 with the needed libvirt and qemu versions. | 12:52 |
| *** shaohe_feng has quit IRC | 12:52 | |
| alaski | artom: yeah, not sure on that yet. But I'm seeing http://logs.openstack.org/58/272658/58/check/gate-grenade-dsvm-ironic/3436213/logs/new/screen-n-cpu.txt.gz?level=ERROR#_2016-06-30_12_37_11_285 as wekk | 12:52 |
| alaski | well | 12:52 |
| *** shaohe_feng has joined #openstack-nova | 12:52 | |
| openstackgerrit | Alexis Lee proposed openstack/nova: Notifications: single source schema https://review.openstack.org/312010 | 12:52 |
| artom | alaski, ok, that last one we need to know what's happening at the db level | 12:53 |
| artom | alaski, althought, the db changes for our stuff merged ages ago, comparatively | 12:53 |
| artom | If this was a schema thing we'd have caught it earlier | 12:53 |
| dtantsur | I wonder why it only happens in grenade | 12:54 |
| *** tbachman has joined #openstack-nova | 12:54 | |
| markus_z | dtantsur: That's with the ironic driver, isn't it? | 12:55 |
| dtantsur | yep | 12:55 |
| markus_z | the ironic driver doesn't override this https://github.com/openstack/nova/blob/b466c382eaecb95566357b473e599f749baeaf6f/nova/virt/driver.py#L528-L528 | 12:55 |
| alaski | if we throw up a logging patch in Nova and then a patch in Ironic that depends on it I think we can get logs | 12:55 |
| dtantsur | markus_z, should we? it was not required for quite some time | 12:56 |
| markus_z | note: I'm no expert with ironic, I'm just reading code here | 12:56 |
| dtantsur | alaski, ++ | 12:56 |
| *** mdrabe_ has joined #openstack-nova | 12:56 | |
| *** mdrabe has quit IRC | 12:56 | |
| *** mfeoktistov has quit IRC | 12:57 | |
| dtantsur | I wonder if we hit some hidden upgrade problem | 12:57 |
| openstackgerrit | Andrea Rosa proposed openstack/nova: Enabling detach root volume for shelved_offloaded https://review.openstack.org/326899 | 12:58 |
| *** ducttape_ has joined #openstack-nova | 12:58 | |
| openstackgerrit | Andrea Rosa proposed openstack/nova: Attach/detach root volume for shelved_offloaded https://review.openstack.org/317446 | 12:58 |
| openstackgerrit | Andrea Rosa proposed openstack/nova: Adding device_present field to bdm table https://review.openstack.org/268053 | 12:58 |
| *** rodrigods has quit IRC | 12:58 | |
| *** rodrigods has joined #openstack-nova | 12:58 | |
| openstackgerrit | Ryan Moore proposed openstack/nova: Add better help to rdp options https://review.openstack.org/253556 | 12:58 |
| openstackgerrit | Andrew Laski proposed openstack/nova: DNM Log extra stuff for debugging https://review.openstack.org/335995 | 12:59 |
| alaski | dtantsur: ^ | 12:59 |
| *** jed56 has joined #openstack-nova | 12:59 | |
| alaski | can you throw up the ironic patch? | 12:59 |
| dtantsur | thnx, sure | 12:59 |
| *** tbachman has quit IRC | 13:00 | |
| *** andymaier has joined #openstack-nova | 13:00 | |
| dtantsur | alaski, https://review.openstack.org/335998 | 13:01 |
| *** amotoki has quit IRC | 13:01 | |
| alaski | dtantsur: thansk | 13:01 |
| dtantsur | alaski, also FWIW I think this logging should be there | 13:01 |
| alaski | *thanks | 13:01 |
| alaski | yeah, I agree | 13:02 |
| *** shaohe_feng has quit IRC | 13:02 | |
| *** ducttape_ has quit IRC | 13:02 | |
| *** dedery has quit IRC | 13:02 | |
| *** ljxiash has quit IRC | 13:03 | |
| *** shaohe_feng has joined #openstack-nova | 13:03 | |
| markus_z | dtantsur: IIUC, change https://review.openstack.org/#/c/328168/ was the first one which logged the NotImplementedError. | 13:03 |
| gibi_ | alaski: Hi! I fixed your comment in https://review.openstack.org/#/c/313654/ | 13:04 |
| markus_z | according to logstash | 13:04 |
| *** tblakeslee has joined #openstack-nova | 13:04 | |
| dtantsur | markus_z, good catch. so this NotImplementedError is probably not the cause | 13:04 |
| *** lucas-hungry is now known as lucasagomes | 13:06 | |
| markus_z | dtantsur: nah. I guess, that the NotImplementedError gets raised because it tries to rollback the failed VIF creation. | 13:06 |
| *** code-R_ has joined #openstack-nova | 13:06 | |
| johnthetubaguy | alaski: are you not tempted to merge this one? https://review.openstack.org/#/c/335995/1/nova/db/sqlalchemy/api.py | 13:07 |
| *** amotoki has joined #openstack-nova | 13:07 | |
| *** muawiakhan has quit IRC | 13:07 | |
| markus_z | which means: "unknown DB error" -> raise VirtualInterfaceCreateException -> rollback VIF creation -> detach_interface -> raise NotImplementedError | 13:08 |
| alaski | I suppose we could, I just didn't put much thought into the message there | 13:08 |
| *** zenoway has quit IRC | 13:08 | |
| *** code-R_ has quit IRC | 13:09 | |
| *** code-R_ has joined #openstack-nova | 13:09 | |
| *** code-R has quit IRC | 13:09 | |
| dtantsur | yeah, probably | 13:09 |
| dtantsur | so seems like if we fix the DB error, we can get back to being green | 13:09 |
| *** muawiakhan has joined #openstack-nova | 13:09 | |
| markus_z | I assume that, yes | 13:09 |
| *** sdake has joined #openstack-nova | 13:10 | |
| markus_z | dtantsur: rethinking the rollback behavior for the ironic driver could be another task. | 13:10 |
| dtantsur | yep | 13:11 |
| *** houming has quit IRC | 13:11 | |
| *** muawiakhan has quit IRC | 13:11 | |
| openstackgerrit | Alexis Lee proposed openstack/nova: Catch errors during VIF plug/unplug https://review.openstack.org/264146 | 13:11 |
| openstackgerrit | John Garbutt proposed openstack/nova: Improve help text for resource tracker options https://review.openstack.org/320147 | 13:11 |
| *** pilgrimstack has quit IRC | 13:11 | |
| *** shaohe_feng has quit IRC | 13:12 | |
| openstackgerrit | Andrew Laski proposed openstack/nova: Log DB exception if VIF creation fails https://review.openstack.org/335995 | 13:12 |
| alaski | johnthetubaguy: should be mergeable now ^ | 13:12 |
| *** sdake_ has quit IRC | 13:13 | |
| alaski | that set us back about 11 minutes on getting logs | 13:13 |
| *** shaohe_feng has joined #openstack-nova | 13:13 | |
| *** cleong has joined #openstack-nova | 13:14 | |
| *** rlrossit has quit IRC | 13:14 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova: WIP: policy: clean-up https://review.openstack.org/333925 | 13:17 |
| *** takashin has left #openstack-nova | 13:17 | |
| *** prateek has quit IRC | 13:17 | |
| *** erlon has joined #openstack-nova | 13:18 | |
| *** woodster_ has joined #openstack-nova | 13:19 | |
| *** pilgrimstack has joined #openstack-nova | 13:20 | |
| *** tbachman has joined #openstack-nova | 13:20 | |
| bauzas | p!spy | 13:20 |
| *** mriedem has joined #openstack-nova | 13:21 | |
| *** ametts has joined #openstack-nova | 13:21 | |
| edleafe | OK, so I have a reno formatting failure for a patch. How can I test that locally to avoid the wrath of Gerrit? | 13:22 |
| *** Administrator_ has joined #openstack-nova | 13:22 | |
| *** shaohe_feng has quit IRC | 13:22 | |
| mriedem | edleafe: tox -e releasenotes | 13:23 |
| *** shaohe_feng has joined #openstack-nova | 13:23 | |
| edleafe | mriedem: thx | 13:23 |
| markus_z | edleafe: IIRC the reno file needs to be committed locally to check it with tox. | 13:23 |
| mriedem | yeah that too | 13:24 |
| mriedem | reno only works on commits | 13:24 |
| mriedem | not staged files | 13:24 |
| edleafe | I'm not afraid of commitment | 13:24 |
| markus_z | ;) | 13:24 |
| *** pwnall has joined #openstack-nova | 13:24 | |
| mriedem | markus_z: i learned about walpurgis night last night | 13:25 |
| mriedem | fun stuff | 13:25 |
| markus_z | yeah, we're known for that :) | 13:26 |
| *** takashin has joined #openstack-nova | 13:26 | |
| mriedem | do the witches get together with satan, or get burned in a bonfire, or both? | 13:26 |
| kashyap | mriedem: Noticed your ping from yesterday about that stable/liberty CVE fix, just uploaded this change, does this look reasonable? (Comment on the review): https://review.openstack.org/#/c/327624/ | 13:26 |
| mriedem | or depends on country? | 13:26 |
| markus_z | since a few years we don't use people anymore | 13:26 |
| mriedem | ha, ok | 13:26 |
| markus_z | AFAIK you could do fingerpointing and yell "witch" and the rest will be resolved in fire... | 13:27 |
| markus_z | yeah, good old times... | 13:27 |
| kashyap | markus_z: I _now_ learnt Walpurgis Night! Crazy... | 13:28 |
| markus_z | I'm not super into history, so don't rely solely on my information :D | 13:29 |
| dansmith | alex_xu_: yeah, tags in attach commands will have to come later | 13:29 |
| dansmith | alex_xu_: we have to rework some configdrive code in order to support it | 13:29 |
| *** kevinzs has joined #openstack-nova | 13:29 | |
| *** jbernard has joined #openstack-nova | 13:30 | |
| *** Administrator_ has quit IRC | 13:31 | |
| *** jamesdenton has joined #openstack-nova | 13:31 | |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Raise an exception in v2.client for direct_use https://review.openstack.org/332829 | 13:31 |
| *** rlrossit has joined #openstack-nova | 13:32 | |
| *** kevinzh has quit IRC | 13:32 | |
| *** kevin_zh has quit IRC | 13:33 | |
| *** shaohe_feng has quit IRC | 13:33 | |
| mriedem | dansmith: would you agree here? https://review.openstack.org/#/c/313654/41/nova/notifications/objects/instance.py@129 | 13:33 |
| *** shaohe_feng has joined #openstack-nova | 13:33 | |
| *** READ10 has joined #openstack-nova | 13:33 | |
| *** kevin_zh has joined #openstack-nova | 13:33 | |
| dansmith | mriedem: yups, commented | 13:34 |
| *** jaypipes has joined #openstack-nova | 13:34 | |
| mriedem | did the tags thing get sorted out? | 13:34 |
| *** sacharya has joined #openstack-nova | 13:34 | |
| jaypipes | morning supernovas. | 13:35 |
| mriedem | ah, jsonschema can only validate the tag in the body | 13:35 |
| *** s1061123 has quit IRC | 13:35 | |
| mriedem | not the path | 13:35 |
| mriedem | cool | 13:35 |
| *** tiendc has joined #openstack-nova | 13:35 | |
| mriedem | jaypipes: o/ | 13:35 |
| *** karimb has joined #openstack-nova | 13:35 | |
| mriedem | jaypipes: not to be confused with https://github.com/major/supernova | 13:36 |
| jaypipes | mriedem: heh, true nuf. | 13:36 |
| *** moshele has left #openstack-nova | 13:37 | |
| *** moshele has quit IRC | 13:37 | |
| openstackgerrit | Andrew Laski proposed openstack/nova: Bump service version for BuildRequest deletion https://review.openstack.org/332478 | 13:37 |
| openstackgerrit | Andrew Laski proposed openstack/nova: Stop instance build if BuildRequest deleted https://review.openstack.org/332479 | 13:37 |
| openstackgerrit | Andrew Laski proposed openstack/nova: Refactor block_device_mapping handling during boot https://review.openstack.org/334094 | 13:37 |
| openstackgerrit | Andrew Laski proposed openstack/nova: Add block_device_mappings to BuildRequest https://review.openstack.org/334095 | 13:37 |
| openstackgerrit | Andrew Laski proposed openstack/nova: Cells: Handle delete with BuildRequest https://review.openstack.org/325985 | 13:37 |
| dtantsur | alaski, created a bug to track the issue: https://bugs.launchpad.net/nova/+bug/1597770 | 13:38 |
| openstack | Launchpad bug 1597770 in Ironic "Ironic grenade gate fauls due to VirtualInterfaceCreateException in nova-compute" [Critical,Confirmed] | 13:38 |
| *** xenogear has joined #openstack-nova | 13:39 | |
| *** sacharya has quit IRC | 13:39 | |
| openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova: libvirt: remove dead code path in "_get_console_output_pty" https://review.openstack.org/336021 | 13:39 |
| mriedem | dansmith: do it to it https://review.openstack.org/#/c/304510/ | 13:39 |
| dansmith | woot | 13:40 |
| dtantsur | alaski, btw we've also found http://logs.openstack.org/58/272658/58/check/gate-grenade-dsvm-ironic/3436213/logs/new/screen-q-dhcp.txt.gz?level=ERROR I wonder if it's related as well | 13:40 |
| openstackgerrit | Paul Carlton proposed openstack/python-novaclient: [microversion] Bump to 2.32 https://review.openstack.org/316026 | 13:40 |
| openstackgerrit | Paul Carlton proposed openstack/python-novaclient: Add support for microversion 2.31 https://review.openstack.org/334390 | 13:40 |
| mriedem | dtantsur: http://logs.openstack.org/58/272658/58/check/gate-grenade-dsvm-ironic/3436213/logs/new/screen-n-cpu.txt.gz?level=TRACE | 13:41 |
| mriedem | actually that shouldn't break you... | 13:41 |
| dtantsur | interesting... | 13:41 |
| dtantsur | ETOOMANYSTACKTRACES | 13:41 |
| *** code-R has joined #openstack-nova | 13:42 | |
| mriedem | it fails here | 13:42 |
| mriedem | http://logs.openstack.org/58/272658/58/check/gate-grenade-dsvm-ironic/3436213/logs/new/screen-n-cpu.txt.gz?level=TRACE#_2016-06-30_12_37_14_856 | 13:42 |
| *** andymaier has quit IRC | 13:43 | |
| mriedem | dtantsur: this is the change yesterday https://github.com/openstack/nova/commit/e2eb6659a426bae05a54b93ebf31ff3247125258 | 13:43 |
| *** karimb has quit IRC | 13:43 | |
| mriedem | i don't know why that would break ironic | 13:43 |
| mriedem | i guess ironic is using it's own conductor? | 13:43 |
| *** shaohe_feng has quit IRC | 13:43 | |
| *** shaohe_feng has joined #openstack-nova | 13:43 | |
| dtantsur | wdym by "own conductor"? I don't think we mess with nova-conductor | 13:43 |
| *** sheel has joined #openstack-nova | 13:44 | |
| *** edmondsw has quit IRC | 13:44 | |
| mriedem | nvm | 13:45 |
| mriedem | i don't know why this would fail though http://logs.openstack.org/58/272658/58/check/gate-grenade-dsvm-ironic/3436213/logs/new/screen-n-cpu.txt.gz#_2016-06-30_12_37_14_856 | 13:45 |
| *** code-R_ has quit IRC | 13:45 | |
| mriedem | probably something with the mac address on the vif? | 13:45 |
| *** tbachman has quit IRC | 13:46 | |
| mriedem | ironic is the only virt driver that provides macs to allocate_for_instance | 13:46 |
| alaski | mriedem: we've got https://review.openstack.org/#/c/335995/ up to try to find out | 13:46 |
| johnthetubaguy | dansmith: I refresh the my refactor stuff after the tagging merge, if you get chance today | 13:46 |
| *** bvanhav has quit IRC | 13:46 | |
| *** tbachman has joined #openstack-nova | 13:46 | |
| alaski | mriedem: and https://review.openstack.org/#/c/335998/ | 13:46 |
| *** bvanhav has joined #openstack-nova | 13:46 | |
| dansmith | johnthetubaguy: cool, one of your patches screwed me up yesterday, so glad we returned the favor :P | 13:47 |
| johnthetubaguy | dansmith: :p | 13:47 |
| mriedem | alaski: we should log something here https://review.openstack.org/#/c/264017/75/nova/network/neutronv2/api.py@701 | 13:47 |
| dansmith | johnthetubaguy: one of those rare times where jenkins (or whatever) merged without a failure, but the code was subtly wrong.. | 13:47 |
| mriedem | alaski: but the db exception tracing is good too | 13:47 |
| johnthetubaguy | dansmith: oh, nice | 13:47 |
| *** ayoung has joined #openstack-nova | 13:48 | |
| mriedem | alaski: dtantsur: my money is on duplicate macs | 13:48 |
| mriedem | schema.UniqueConstraint("address", "deleted", | 13:48 |
| mriedem | name="uniq_virtual_interfaces0address0deleted"), | 13:48 |
| dtantsur | this is likely, except for why didn't it fail previously? | 13:49 |
| mriedem | dtantsur: we never created virtualinterfaces in nova's db when using neutron before yesterday | 13:49 |
| dtantsur | hmmm | 13:50 |
| dansmith | mriedem: just to be clear, johnthetubaguy's refactor is covered under at least one priority realm, right? | 13:50 |
| dtantsur | then it's possible. we definitely do not change MACs between old and new runs | 13:50 |
| *** tbachman_ has joined #openstack-nova | 13:50 | |
| *** rlrossit has quit IRC | 13:50 | |
| mriedem | dansmith: i wasn't tracking that as a priority | 13:50 |
| mriedem | it was a dependency for neutron routed networks | 13:50 |
| mriedem | which is also dependent on generic resource pools | 13:51 |
| dansmith | mriedem: hmm | 13:51 |
| *** chandankumar has joined #openstack-nova | 13:51 | |
| dansmith | mriedem: it's kinda critical for cells and general sanity | 13:51 |
| johnthetubaguy | although routed networks kinda required to drop cells v1 | 13:51 |
| dansmith | ooh yeah | 13:51 |
| dtantsur | mriedem, so if it's really duplicated MACs, how do we fix it? | 13:51 |
| *** tbachman has quit IRC | 13:51 | |
| *** tbachman_ is now known as tbachman | 13:51 | |
| *** pradk has joined #openstack-nova | 13:52 | |
| mriedem | dtantsur: well let's figure out first if that's what it is | 13:52 |
| dtantsur | okie | 13:52 |
| mriedem | https://specs.openstack.org/openstack/nova-specs/priorities/newton-priorities.html | 13:52 |
| *** krtaylor has joined #openstack-nova | 13:52 | |
| mriedem | we already have a lot of priority stuff still to work on | 13:52 |
| mriedem | and while that doc doesn't mention api proxy deprecations as priority, i'm guessing sdague will want it to be so we can get that out as early as possible | 13:53 |
| mriedem | same with api extension removal | 13:53 |
| dansmith | it's also debt cleanup until a point | 13:53 |
| mriedem | i'm not even sure if libvirt storage pools has started | 13:53 |
| *** shaohe_feng has quit IRC | 13:53 | |
| dansmith | mriedem: what? of course it has :) | 13:53 |
| *** shaohe_feng has joined #openstack-nova | 13:53 | |
| mriedem | the imagebackend stuff was a separate bp | 13:53 |
| mriedem | storage pools built o nthat | 13:54 |
| *** diogogmt has joined #openstack-nova | 13:54 | |
| mriedem | but i thought there were still specs up for storage pools | 13:54 |
| dansmith | mriedem: thought there are code patches too | 13:54 |
| mriedem | PaulMurray: what's the status on the libvirt storage pools bp? | 13:54 |
| johnthetubaguy | anyways, patches are up that I think would unblock the first phase of routed networks (the non-scheduler bits) | 13:54 |
| mriedem | is there code? | 13:54 |
| *** irenab has joined #openstack-nova | 13:54 | |
| dansmith | https://review.openstack.org/#/c/314134/ | 13:55 |
| mriedem | PaulMurray: and i mean after mdbooth and diana_clarke's imagebackend storage series | 13:55 |
| PaulMurray | mriedem, its dependant on the stuff mdbooth and diana_clarke are doing | 13:55 |
| PaulMurray | what do you need to know ? | 13:55 |
| mriedem | PaulMurray: is it just that one patch dansmith linked above? | 13:55 |
| *** karimb has joined #openstack-nova | 13:55 | |
| mriedem | dansmith: the spec isn't approved yet either https://review.openstack.org/#/c/310505/ | 13:55 |
| johnthetubaguy | hmm, fun, its not | 13:56 |
| danpb | mriedem: about https://blueprints.launchpad.net/nova/+spec/websocket-proxy-to-host-security | 13:56 |
| danpb | mriedem: "Doesn't look like anything was pushed up for code for this so I'm deferring for Newton. -- mriedem 20160629" | 13:56 |
| danpb | mriedem: i've pushed 10 new versions of the patches during this cycle :-( | 13:56 |
| PaulMurray | mriedem, it just didn't get approved yet..... | 13:56 |
| *** rlrossit has joined #openstack-nova | 13:56 | |
| mriedem | danpb: https://review.openstack.org/#/q/topic:bp/websocket-proxy-to-host-security,n,z | 13:56 |
| PaulMurray | mriedem, I actually forgot that - is there a deadline for the specs for priority ? | 13:57 |
| *** akshai has joined #openstack-nova | 13:57 | |
| mriedem | danpb: what were those changes tracked under? | 13:57 |
| danpb | mriedem: they're listed right there on the blueprint whiteboard | 13:57 |
| danpb | they've been on review since aug 2014 now | 13:58 |
| mriedem | danpb: i only see specs in the whiteboard | 13:58 |
| mriedem | and abandoned changes from solly | 13:58 |
| *** kylek3h has joined #openstack-nova | 13:58 | |
| mriedem | danpb: please link me to the series in gerrit | 13:58 |
| mriedem | nova meeting starting in 1 minute | 13:59 |
| *** ducttape_ has joined #openstack-nova | 13:59 | |
| danpb | mriedem: those changes by solly aren't abandoned - they're the ones i'm frequently updating | 13:59 |
| *** ctrath has joined #openstack-nova | 13:59 | |
| mriedem | ah i see | 13:59 |
| mdbooth | danpb: ? | 13:59 |
| mriedem | so it is https://review.openstack.org/#/q/topic:bp/websocket-proxy-to-host-security | 13:59 |
| mriedem | i saw the merge conflicts | 14:00 |
| mdbooth | Ah, websocket. | 14:00 |
| mriedem | danpb: sorry, i'll update the bp | 14:00 |
| mdbooth | For a moment I thought you were talking about libvirt storage pools | 14:00 |
| danpb | mriedem: its the god damn nova/exception.py file whicih is merge conflict hell | 14:00 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: Introduce VNC Security Proxy Framework https://review.openstack.org/115483 | 14:00 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: Add VeNCrypt (TLS/x509) Security Proxy Driver https://review.openstack.org/115484 | 14:00 |
| * danpb should just add new exceptions randomly in the middle of the file instead of add the end to avoid conflicts | 14:00 | |
| *** kevin_zh has quit IRC | 14:00 | |
| mdbooth | danpb: I was just about to advise that :) | 14:00 |
| PaulMurray | mriedem, dansmith I was going to give an update on storage pools in nova meeting | 14:01 |
| lxsli | danpb: https://review.openstack.org/#/c/319203/ if you'd be so kind, Alex +2d | 14:01 |
| *** kevin_zh has joined #openstack-nova | 14:01 | |
| lxsli | I have also removed a lot of complexity from https://review.openstack.org/319254 | 14:01 |
| andreykurilin | sdague mriedem: hi! Could you look at small backport to novaclient's stable/mitaka? https://review.openstack.org/#/c/332038/ | 14:01 |
| *** gjayavelu has joined #openstack-nova | 14:02 | |
| mriedem | nova meeting | 14:02 |
| markus_z | danpb: yeah, I use that strategy too | 14:02 |
| mdbooth | sahid: https://review.openstack.org/#/c/333271/ That -1 is kinda harsh. I think mine is more readable, so there isn't really anywhere to go from there. It's subjective. | 14:02 |
| *** mpavone has quit IRC | 14:03 | |
| openstackgerrit | John L. Villalovos proposed openstack/nova: WIP: Revert "Compute manager device tagging support" https://review.openstack.org/336041 | 14:03 |
| *** ducttape_ has quit IRC | 14:03 | |
| *** zenoway has joined #openstack-nova | 14:03 | |
| *** shaohe_feng has quit IRC | 14:03 | |
| danpb | lxsli: ok i'll look | 14:04 |
| mdbooth | sahid: I appreciate the note, but -1 means: the code base is better off without this. | 14:04 |
| *** diga has joined #openstack-nova | 14:04 | |
| *** shaohe_feng has joined #openstack-nova | 14:04 | |
| *** s1061123 has joined #openstack-nova | 14:07 | |
| openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova: libvirt: virtlogd: use virtlogd for char devices https://review.openstack.org/323765 | 14:07 |
| openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova: libvirt: simplify "get_console_output" interface https://review.openstack.org/334480 | 14:07 |
| openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova: libvirt: remove dead code path in "_get_console_output_pty" https://review.openstack.org/336021 | 14:07 |
| *** roeyc has quit IRC | 14:07 | |
| *** sdake_ has joined #openstack-nova | 14:07 | |
| markus_z | dang it... | 14:08 |
| sahid | mdbooth: well i gave to this change several comments so I don't understand which one you are talking about | 14:08 |
| *** smatzek has quit IRC | 14:08 | |
| mdbooth | https://review.openstack.org/#/c/333271/ | 14:08 |
| mdbooth | All comments are about the same thing, which is a minor stylistic thing | 14:09 |
| *** EinstCrazy has joined #openstack-nova | 14:09 | |
| sahid | mdbooth: not really | 14:09 |
| mdbooth | And I prefer mine, because it requires less immediate context to read | 14:09 |
| mdbooth | I don't think yours is bad, just mine is better | 14:09 |
| mdbooth | And I already wrote mine, and don't want to change it :) | 14:10 |
| mdbooth | And it's directly relevant to the patch it's in. | 14:10 |
| sahid | mdbooth: feel free to ignore my comments | 14:10 |
| danpb | markus_z: given the issue with console.log not being written unless connected to the serial port, i'm wondering whether we should hold off on the console patch, or have it temporarily disabled by default ? | 14:10 |
| mdbooth | Well I would, but they're accompanied by an ugly -1. | 14:10 |
| mdbooth | -1 isn't really for that. | 14:10 |
| *** syjulian has joined #openstack-nova | 14:10 | |
| *** gongysh_ has quit IRC | 14:11 | |
| *** sdake has quit IRC | 14:11 | |
| *** aloga_ has joined #openstack-nova | 14:11 | |
| sahid | it's not like it's a minus 2 so i guess if your code is good and my comments silly, you will get the 2 necessaries +2 | 14:11 |
| *** paul-carlton2 has quit IRC | 14:11 | |
| *** paul-carlton2 has joined #openstack-nova | 14:12 | |
| mdbooth | sahid: -1 means "This patch needs further work before it can be merged" | 14:12 |
| mdbooth | That's literally what it means, we discussed it a while back and changed the wording. | 14:13 |
| mdbooth | It means that the codebase is better off without the change. | 14:13 |
| *** amotoki has quit IRC | 14:13 | |
| mdbooth | You're asserting with your -1 that the codebase is worse off with the change than without it. | 14:13 |
| *** gagehugo has joined #openstack-nova | 14:13 | |
| *** GB21 has joined #openstack-nova | 14:13 | |
| mdbooth | Because of a minor stylistic difference. | 14:13 |
| markus_z | danpb: yeah, I can add a "AND False" conditional with a TODO note, would that be ok? Easy to remove after we figured out a solution. | 14:14 |
| mdbooth | It isn't what it's for. | 14:14 |
| *** shaohe_feng has quit IRC | 14:14 | |
| *** diogogmt has quit IRC | 14:14 | |
| *** shaohe_feng has joined #openstack-nova | 14:14 | |
| danpb | markus_z: or could add a config option to the workaround group to disable it and default to disabled | 14:14 |
| mdbooth | I know that it gets used that way a fair bit anyway, but it really shouldn't be. | 14:14 |
| danpb | so people can at least try it out | 14:14 |
| *** rcernin has quit IRC | 14:14 | |
| markus_z | danpb: yeah, the workaround section sounds good, I'll push it in the next hour | 14:15 |
| * mdbooth is quite careful to mark nits as nits, and I only -1 if there's at least 1 thing which isn't a nit. | 14:15 | |
| *** tbachman has quit IRC | 14:17 | |
| rlrossit | dansmith: I have an objects question for you after the nova meeting is over (so don't go running away when it's done) | 14:18 |
| *** sdake_ is now known as sdake | 14:20 | |
| *** kevin_zh has quit IRC | 14:23 | |
| jaypipes | mriedem, dansmith: here is the line that introduces the issue: https://github.com/openstack/nova/commit/e2eb6659a426bae05a54b93ebf31ff3247125258#diff-da7e16a0727fa850ace01ae175cc2363R675 | 14:23 |
| *** mjura has quit IRC | 14:23 | |
| lxsli | thanks Dan | 14:23 |
| *** nkrinner has quit IRC | 14:23 | |
| mriedem | jaypipes: that's nova-net | 14:23 |
| jaypipes | mriedem, dansmith, artom: when a request for devices has >1 tag, it attempts to insert one record in virtual_interfaces for each tag. | 14:23 |
| *** kevin_zh has joined #openstack-nova | 14:23 | |
| mriedem | ironic is blowing up with neutron | 14:24 |
| *** itamarl has quit IRC | 14:24 | |
| *** shaohe_feng has quit IRC | 14:24 | |
| artom | jaypipes, you mean one device with multiple tags? | 14:24 |
| dansmith | yeah and that line is just a lookup table for us | 14:24 |
| artom | That... shouldn't be possible? | 14:24 |
| dansmith | artom: right | 14:24 |
| dansmith | artom: and the bug is not in n-net anyway, so not that I think :) | 14:24 |
| *** tbachman has joined #openstack-nova | 14:25 | |
| *** shaohe_feng has joined #openstack-nova | 14:25 | |
| alaski | this telnet interface to zuul(or whatever I'm actually connected to) is pretty nifty | 14:26 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: Pre-add instance actions to avoid merge conflicts https://review.openstack.org/330513 | 14:26 |
| *** edmondsw has joined #openstack-nova | 14:26 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: Transform instance.delete notifications https://review.openstack.org/313654 | 14:26 |
| gibi_ | mriedem, dansmith: fixed your comments in https://review.openstack.org/313654 ^^ | 14:27 |
| *** aloga_ has quit IRC | 14:28 | |
| *** belmoreira has quit IRC | 14:28 | |
| openstackgerrit | Lee Yarwood proposed openstack/nova: WIP - libvirt: Use the recreated config disk during a rescue https://review.openstack.org/335884 | 14:29 |
| *** kevin_zh has quit IRC | 14:29 | |
| *** rcernin has joined #openstack-nova | 14:29 | |
| *** kevinzs has quit IRC | 14:30 | |
| *** ctrath has quit IRC | 14:31 | |
| *** jistr is now known as jistr|mtg | 14:31 | |
| edleafe | mdbooth: I looked at that patch, and agree that it isn't just "style". That line can be using the wrong value. | 14:32 |
| mdbooth | edleafe: You mean if the super class alters the path it was given to store? | 14:33 |
| mdbooth | Ok, I'll update the contract to assert that it doesn't :) That would be confusing. | 14:33 |
| *** ctrath has joined #openstack-nova | 14:33 | |
| edleafe | mdbooth: sure - just like you altered the path that was passed | 14:34 |
| *** krtaylor has quit IRC | 14:34 | |
| *** shaohe_feng has quit IRC | 14:34 | |
| mdbooth | edleafe: My patch doesn't alter the path... | 14:34 |
| mdbooth | That would be bug | 14:34 |
| edleafe | mdbooth: if the superclass is modified to handle some other path-related thing, this class would never see it (at least in the __init__) | 14:34 |
| mdbooth | I would be happy to address that. | 14:34 |
| jaypipes | mriedem, dansmith, artom: yeah, sorry, I was wrong :( looks like now that in the neutronv2 code we're creating an objects.VirtualInterface object for each requested network we're running into that MAC address unique constraint. | 14:34 |
| dansmith | jaypipes: yeah, I know what the problem is, no question about that | 14:35 |
| mdbooth | edleafe: I'll update the doctext if you like to state that it doesn't do that. | 14:35 |
| dansmith | jaypipes: just not sure why we're getting dupes from ironic | 14:35 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Add swap volume notifications (start, end) https://review.openstack.org/250283 | 14:35 |
| *** shaohe_feng has joined #openstack-nova | 14:35 | |
| edleafe | mdbooth: sure it does: on line 446 | 14:35 |
| *** sahid has quit IRC | 14:35 | |
| dansmith | jaypipes: but I suspect it may be because we're not deleting the VIFs in the neutron case so we fail when we've used one a second time | 14:35 |
| mdbooth | edleafe: That's *before* passing it to the super class | 14:35 |
| edleafe | mdbooth: right | 14:36 |
| *** vsaienko1 has joined #openstack-nova | 14:36 | |
| mdbooth | Every base class now follows the same pattern: it creates a path, which it gives to the super class | 14:36 |
| edleafe | path gets passed to this class. This class potentially changes it, and then passes it to the superclass | 14:36 |
| jaypipes | dansmith: even so, I don't believe Neutron has a similar constraint about MAC uniqueness does it? | 14:36 |
| edleafe | mdbooth: there's no contract that says that the superclass might manipulate it further | 14:36 |
| mdbooth | Actually, if the super class did something with it, we would still be correct to use the unmodified one | 14:36 |
| dansmith | jaypipes: it does not, no | 14:36 |
| mdbooth | Because that's the context of the user | 14:36 |
| *** ducttape_ has joined #openstack-nova | 14:38 | |
| mdbooth | edleafe: Look at it this way, disk_info_path is explicitly created from the local path we created locally | 14:38 |
| mdbooth | If the super class altered the path variable to do something else | 14:38 |
| vsaienko1 | hello nova team, does anybody seen VirtualInterfaceCreateException_Remote: Virtual Interface creation failed Error? | 14:38 |
| mdbooth | Well, that would be subversive and silly | 14:38 |
| *** gcb has quit IRC | 14:38 | |
| mdbooth | However, we would *still* want to use the local path we created locally | 14:39 |
| *** MikeG451 has joined #openstack-nova | 14:39 | |
| *** ducttape_ has quit IRC | 14:39 | |
| mdbooth | So, in fact, using the super class path instance variable is more fragile | 14:39 |
| mdbooth | But it's completely irrelevant in either case | 14:39 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: Transform instance.update notification https://review.openstack.org/325923 | 14:39 |
| mdbooth | Because I wrote it this way, and it's not worth a -1, even if you disagree with me | 14:39 |
| *** smatzek has joined #openstack-nova | 14:39 | |
| vsaienko1 | full trace is http://paste.openstack.org/show/524274/ | 14:39 |
| paul-carlton2 | johnthetubaguy, did you get chance to look at https://review.openstack.org/#/q/topic:bp/live-migrate-rescued-instances | 14:40 |
| *** phuongnh has joined #openstack-nova | 14:40 | |
| edleafe | mdbooth: I am not as familiar with the entire libvirt codebase. But if you have an instance variable, it should be respected throughout the class hierarchy | 14:40 |
| *** sacharya has joined #openstack-nova | 14:40 | |
| edleafe | mdbooth: otherwise, it should be local | 14:40 |
| *** Guest85383 is now known as yass | 14:40 | |
| mdbooth | It is local | 14:40 |
| mdbooth | The patch made it local | 14:40 |
| edleafe | mdbooth: are you saying that you no longer need 'self.path' anywhere else in the codebase? | 14:40 |
| * mdbooth goes back to his 30 substantive patches :) | 14:41 | |
| mdbooth | This is the worst kind of bikeshedding | 14:41 |
| alaski | vsaienko1: that bug is being tracked at https://bugs.launchpad.net/nova/+bug/1597770, it was brought up this morning | 14:41 |
| openstack | Launchpad bug 1597770 in OpenStack Compute (nova) "Ironic grenade gate fails: Failed to allocate the network(s)" [Critical,Confirmed] | 14:41 |
| *** gcb has joined #openstack-nova | 14:41 | |
| *** aunnam_ has quit IRC | 14:41 | |
| *** dharinic__ has joined #openstack-nova | 14:41 | |
| edleafe | mdbooth: no, it's getting a separate POV | 14:41 |
| edleafe | mdbooth: again, I'm not a libvirt expert, so you may be perfectly safe | 14:42 |
| dtantsur | alaski, mriedem, the logs: http://logs.openstack.org/98/335998/1/check/gate-grenade-dsvm-ironic/e8cbaac/logs/new/screen-n-cpu.txt.gz?level=TRACE I'm having hard time finding the relevant bit though | 14:42 |
| dtantsur | the initial failure caused too many failures itself... | 14:42 |
| edleafe | mdbooth: my issue is strictly python class interactions | 14:42 |
| johnthetubaguy | paul-carlton2: not yet, will do | 14:43 |
| *** aunnam_ has joined #openstack-nova | 14:43 | |
| paul-carlton2 | thanks | 14:43 |
| alaski | dtantsur: it's on conductor because computes defer db queries there http://logs.openstack.org/98/335998/1/check/gate-grenade-dsvm-ironic/e8cbaac/logs/new/screen-n-cond.txt.gz#_2016-06-30_14_32_25_360 | 14:43 |
| *** cdelatte has joined #openstack-nova | 14:43 | |
| alaski | artom: dansmith ^ | 14:43 |
| artom | dtantsur, why not just modify that review 335998 to not catch anything in the db API and just barf up the raw sqlalchemy error? | 14:43 |
| dtantsur | artom, it's not a question for me, rather for nova folks :) | 14:44 |
| dansmith | I have a hack to propose | 14:44 |
| dansmith | give me a few | 14:44 |
| dtantsur | alaski, so mriedem was right, that's the NIC duplicate | 14:44 |
| dtantsur | s/NIC/MAC/ | 14:44 |
| *** shaohe_feng has quit IRC | 14:44 | |
| danpb | PaulMurray: success, in so much as we got a failure in the live-migraiton CI job ! | 14:44 |
| *** sacharya has quit IRC | 14:45 | |
| *** ljxiash has joined #openstack-nova | 14:45 | |
| danpb | PaulMurray: the non-deterministic failure is caused by QEMU getting an I/O error when flushing data on the source qemu :-( | 14:45 |
| artom | dtantsur, it's apparently already been done by someone else anyways | 14:45 |
| *** tonytan4ever has joined #openstack-nova | 14:45 | |
| PaulMurray | good job danpb | 14:45 |
| *** rcernin has quit IRC | 14:46 | |
| PaulMurray | danpb, I seem to remember that there was an IO error - but didn't know why | 14:46 |
| PaulMurray | can you tell ? | 14:46 |
| *** sudipto has quit IRC | 14:46 | |
| *** pcaruana has quit IRC | 14:46 | |
| *** sudipto has joined #openstack-nova | 14:47 | |
| *** pushkaru has joined #openstack-nova | 14:47 | |
| *** shaohe_feng has joined #openstack-nova | 14:48 | |
| paul-carlton2 | what is the best room for tempest dev questions? | 14:48 |
| markus_z | mriedem: I don't anymore if I RSVP'ed to the midcycle, but I won't make it this time. Company's money is tight again. | 14:48 |
| *** jlanoux_ has quit IRC | 14:48 | |
| dansmith | dtantsur: is this blocking ironic code? meaning, is this a voting job on ironic? | 14:48 |
| dtantsur | dansmith, yes | 14:49 |
| dansmith | okay | 14:49 |
| *** dimtruck is now known as zz_dimtruck | 14:49 | |
| PaulMurray | mdbooth, are you going to go to mid cycle? | 14:51 |
| PaulMurray | or diana_clarke ? | 14:51 |
| mdbooth | PaulMurray: No. Is that a problem? | 14:51 |
| mriedem | markus_z: you didn't rsvp, but yeah, thanks | 14:51 |
| *** david-lyle_ is now known as david-lyle | 14:51 | |
| *** takashin has quit IRC | 14:51 | |
| PaulMurray | mdbooth, probably not | 14:51 |
| danpb | PaulMurray: yeah, syslog is showing a kernel iscsi error at the same time as qemu gets EIO | 14:51 |
| mdbooth | PaulMurray: Cool. | 14:51 |
| diana_clarke | PaulMurray: I think we're only sending cores to mid cycle. | 14:52 |
| PaulMurray | mdbooth, paul-carlton2 will be there with me | 14:52 |
| mriedem | paul-carlton2: #openstack-qa | 14:52 |
| openstackgerrit | Bob Ball proposed openstack/nova: XenAPI: Add UT for independent compute option https://review.openstack.org/336065 | 14:52 |
| mdbooth | diana_clarke: You're not going, are you? | 14:52 |
| diana_clarke | mdbooth: nope | 14:52 |
| *** jaugustine has joined #openstack-nova | 14:52 | |
| mdbooth | Didn't think so. | 14:52 |
| danpb | mriedem: do you have any idea how cinder deals with iscsi during migration | 14:53 |
| mriedem | BobBall: thanks | 14:53 |
| *** diga has quit IRC | 14:53 | |
| edleafe | diana_clarke: you're not using that "baby" excuse again, are you? :) | 14:53 |
| danpb | mriedem: in particular it seems that we must have multiple qemu instances connected to the same cinder iscsi lun at the same time | 14:53 |
| mriedem | danpb: i'd have to look | 14:53 |
| mriedem | danpb: hemnavacation would know, but he's on vacation this week | 14:53 |
| *** diga has joined #openstack-nova | 14:53 | |
| danpb | i'm wondering if connecting the 2nd qemu could cause cinder to kick off the first qemu ? | 14:53 |
| PaulMurray | danpb, both ends are connected before the migration | 14:53 |
| diana_clarke | edleafe: Baby didn't have a passport in time for PyCon, but she has one now. I might head to PyGotham next week with her. | 14:53 |
| PaulMurray | danpb, but we rely on destinatino not doing anything | 14:54 |
| *** muawiakhan has joined #openstack-nova | 14:54 | |
| *** raj_singh has quit IRC | 14:54 | |
| johnthetubaguy | oh, good question, normally you do have two things pointing at the same lun, but only ever have one "active" at once, in theory | 14:54 |
| *** shaohe_feng has quit IRC | 14:55 | |
| alex_xu_ | diga: if you are interesting to help on the api-ref, there is mail from sdague explain how we fix the doc http://lists.openstack.org/pipermail/openstack-dev/2016-April/092936.html, and there is wiki recorded some rules for api-ref https://wiki.openstack.org/wiki/NovaAPIRef | 14:55 |
| danpb | and that's true here - source qemu is trying to flush out all outstanding writes before the target qemu starts running vCPUs | 14:55 |
| johnthetubaguy | but not sure if the domain is active in the live-migrate destination node, before the old domain stops, well active in the send of ISCSI being connected | 14:55 |
| *** hshiina has quit IRC | 14:55 | |
| danpb | so we should only have one writer here | 14:55 |
| * bauzas f***cked, missed the nova meeting | 14:55 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: Pre-add instance actions to avoid merge conflicts https://review.openstack.org/330513 | 14:55 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: Transform instance.update notification https://review.openstack.org/325923 | 14:55 |
| danpb | but it seems like the source qemu iscsi connection is getting interrupted | 14:55 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: Transform instance.delete notifications https://review.openstack.org/313654 | 14:55 |
| paul-carlton2 | mriedem, ta | 14:55 |
| diga | alex_xu_: okay | 14:55 |
| *** shaohe_feng has joined #openstack-nova | 14:56 | |
| *** sdake_ has joined #openstack-nova | 14:56 | |
| rlrossit | dansmith: I have been discussing the HostManager with doffm, and we're wondering: does the ComputeNodeList.get_all() call at https://github.com/openstack/nova/blob/master/nova/scheduler/host_manager.py#L574 go through conductor, or is there magic in place that allows conductor to go directly to the database to get the hosts? | 14:56 |
| mriedem | bauzas: don't worry we cancelled since you weren't there | 14:56 |
| alex_xu_ | diga: and there are lot of patch for reference https://review.openstack.org/#/q/status:merged+topic:bp/api-ref-in-rst, you can start with some review first, then you will know how to start quickly | 14:56 |
| openstackgerrit | Dan Smith proposed openstack/nova: HACK: Test making VIF.address unique with port id for neutron https://review.openstack.org/336069 | 14:56 |
| bauzas | mriedem: meh, untrue :p | 14:56 |
| PaulMurray | danpb, I really don't know enough about iscsi, but I thought it could handle multiple writers | 14:56 |
| dansmith | dtantsur: can you get a run on top of https://review.openstack.org/#/c/336069/1 ? | 14:56 |
| alex_xu_ | diga: also free to ping me if you need help :) | 14:56 |
| diga | sure alex_xu_ | 14:56 |
| dansmith | rlrossit: depends on who is callingit | 14:56 |
| dtantsur | dansmith, sure | 14:56 |
| sudipto | mriedem, are there going to be any extensions for non prio features via extension applications beyond today? | 14:57 |
| PaulMurray | danpb, it should just mess up the data on disk if you don't coordinate right | 14:57 |
| openstackgerrit | Ryan Moore proposed openstack/nova: Add better help to rdp options https://review.openstack.org/253556 | 14:57 |
| rlrossit | dansmith: in this case, it would be the host manager within the scheduler | 14:57 |
| diga | alex_xu_: sure | 14:57 |
| dansmith | mriedem: the above is a pretty silly hack, but it's easy to hide and it avoids just changing the constraint which could have some unintended n-net impacts | 14:57 |
| lxsli | alex_xu_: thanks for your +2 earlier, this one is similar: https://review.openstack.org/#/c/319254/ | 14:57 |
| danpb | PaulMurray: i think we'll need to get some cinder people involved here | 14:57 |
| dansmith | mriedem: also happy to just change the constraint in some way if you'd prefer | 14:57 |
| PaulMurray | danpb, yes | 14:57 |
| dtantsur | dansmith, https://review.openstack.org/#/c/335998/ | 14:57 |
| edleafe | Hmmm... must be doing something wrong. Pulled a fresh master and ran `tox -r -e releasenotes` on it, and it failed. | 14:58 |
| *** raj_singh has joined #openstack-nova | 14:58 | |
| dansmith | rlrossit: then no, not through conductor | 14:58 |
| edleafe | What's the next step for debugging this? | 14:58 |
| markus_z | edleafe: What does the error say? | 14:58 |
| alex_xu_ | lxsli: np, let me check | 14:58 |
| rlrossit | dansmith: where is this magic that lets it pass through? | 14:58 |
| PaulMurray | scottda, you around ? | 14:58 |
| danpb | PaulMurray: at this point it is looking like kernel / iscsi client / iscsi server problem, not qemu | 14:58 |
| dansmith | dtantsur: thanks, poke me when it finishes if you would | 14:58 |
| openstackgerrit | Nicolas Hicher proposed openstack/nova: Purge soft-deleted instances cmd https://review.openstack.org/203751 | 14:58 |
| scottda | PaulMurray: Hi | 14:59 |
| danpb | and some kind of race condition in the mix | 14:59 |
| dansmith | rlrossit: it's always direct unless your service declares indirection_api, which compute does, but scheduler, api (and of course conductor) don't | 14:59 |
| *** sdake has quit IRC | 14:59 | |
| edleafe | markus_z: http://paste.openstack.org/show/524279/ | 14:59 |
| *** sacharya has joined #openstack-nova | 14:59 | |
| *** rcernin has joined #openstack-nova | 14:59 | |
| PaulMurray | Hi scottda who would be best to talk to about iscsi issues ? | 14:59 |
| dtantsur | jlvillal, I'm leaving soon, could you please watch https://review.openstack.org/#/c/335998/ and poke dansmith when it finishes? | 14:59 |
| edleafe | markus_z: do you want the full log? | 14:59 |
| mriedem | dansmith: ah tricky, hacking the uniqueness scoped to the port uuid | 14:59 |
| openstackgerrit | Sujitha proposed openstack/nova: Transform instance.pause notifications https://review.openstack.org/329141 | 14:59 |
| rlrossit | dansmith: so where is the indirection_api defined for the scheduler? (or for a service in general) | 14:59 |
| *** akshai has quit IRC | 14:59 | |
| johnthetubaguy | edleafe: have you checked out mitaka stable? | 14:59 |
| PaulMurray | scottda, danpb has tracked a live migration bug down to an iscsi kernel error | 14:59 |
| markus_z | edleafe: yeah, maybe the content of /tmp/sphinx-err-yYpjnS.log | 14:59 |
| scottda | PaulMurray: hemanthm would be better, but he's on vacation I think. I can help. What's the issue? | 14:59 |
| dansmith | mriedem: there's plenty of space in the address field for it, and our facade lets us hide it really easily.. it's so easy I'm tempted to say this is a good workaround for a bit, but it's definitely a hack | 14:59 |
| edleafe | johnthetubaguy: nope, just a fresh master | 15:00 |
| scottda | PaulMurray: Bug? | 15:00 |
| mriedem | dansmith: sure | 15:00 |
| danpb | scottda: https://bugs.launchpad.net/nova/+bug/1524898 | 15:00 |
| scottda | PaulMurray: As in Launchpad bug #? | 15:00 |
| openstack | Launchpad bug 1524898 in OpenStack Compute (nova) "Volume based live migration aborted unexpectedly" [High,Confirmed] | 15:00 |
| dansmith | rlrossit: it's not that's why it's direct, but look in cmd/compute for an example | 15:00 |
| mriedem | dansmith: some small comments for if this is the actual thing | 15:00 |
| johnthetubaguy | edleafe: so it seems to say this is failing: ['git', 'log', '--simplify-by-decoration', '--pretty="%d"', 'origin/stable/mitaka'] | 15:00 |
| danpb | scottda: lemme recap though, its easier than reading the bug history | 15:00 |
| *** efried has joined #openstack-nova | 15:00 | |
| dansmith | mriedem: sure, just a bluesky hail mary at this point :) | 15:00 |
| *** kevin_zh has joined #openstack-nova | 15:00 | |
| scottda | danpb: OK | 15:00 |
| *** kevinzs has joined #openstack-nova | 15:00 | |
| johnthetubaguy | edleafe: see if that works on your command line I guess | 15:00 |
| *** akshai has joined #openstack-nova | 15:00 | |
| bauzas | rlrossit: there is no indirection api for the scheduler | 15:00 |
| danpb | scottda: we're basically seeing failures in the nova live migration CI jobs | 15:00 |
| jlvillal | dtantsur, I will watch it | 15:00 |
| bauzas | rlrossit: it goes read directly the DB | 15:00 |
| johnthetubaguy | edleafe: maybe you don't use "origin" as the name of your upstream remote? | 15:01 |
| rlrossit | dansmith: ah: https://github.com/openstack/nova/blob/master/nova/cmd/compute.py#L65-L66 | 15:01 |
| danpb | in this case we have 2 QEMU running on different hosts, both connected to cinder volumes via iscsi | 15:01 |
| alaski | dansmith: should be vifobj not vif | 15:01 |
| dtantsur | jlvillal, thanks! | 15:01 |
| rlrossit | doffm: ^^ | 15:01 |
| danpb | and are migrating between them, so the source QEMU is still writing to the iSCSI volume, and target QEMU just has it open readonly | 15:01 |
| bauzas | rlrossit: compared to https://github.com/openstack/nova/blob/master/nova/cmd/compute.py#L63-L66 | 15:01 |
| edleafe | johnthetubaguy: hmmm... didn | 15:01 |
| bauzas | argh, jinxed | 15:01 |
| edleafe | didn't notice the 'mitaka' reference | 15:01 |
| dansmith | alaski: eff, thanks | 15:01 |
| danpb | scottda: randomly though, we're seeing the source hosts's connection to the iscsi server break | 15:01 |
| *** jaypipes has quit IRC | 15:02 | |
| danpb | scottda: whicih causes qemu to fail its migration | 15:02 |
| edleafe | johnthetubaguy: but yes, 'origin' is the upstream remote name | 15:02 |
| openstackgerrit | Dan Smith proposed openstack/nova: HACK: Test making VIF.address unique with port id for neutron https://review.openstack.org/336069 | 15:02 |
| danpb | scottda: you can see an example syslog iscsi error here http://logs.openstack.org/03/335903/1/check/gate-tempest-dsvm-multinode-live-migration/5bae103/logs/subnode-2/syslog.txt.gz#_Jun_30_14_28_12 | 15:02 |
| mriedem | gsilvis: are you going to be at the nova midcycle? | 15:02 |
| *** krtaylor has joined #openstack-nova | 15:02 | |
| mriedem | mnestratov|2: ^ same question | 15:02 |
| danpb | scottda: wondering if there's anything in cinder which could be getting upset by us having 2 qemus connected to the same cinder volume ? | 15:03 |
| mriedem | pkoniszewski: ^ | 15:03 |
| *** mat128 is now known as mat128|afk | 15:03 | |
| pkoniszewski | mriedem: yes, i will be there | 15:03 |
| *** BjoernT has joined #openstack-nova | 15:03 | |
| danpb | eg, could connecting the 2nd qemu cause a temporary interruption in the iscsi connection of the original QEMU for some reason | 15:03 |
| rlrossit | dansmith: bauzas: so out of curiosity, how would I do a rolling upgrade with multiple schedulers if it isn't using the indirection of conductor? | 15:03 |
| mriedem | pkoniszewski: ack | 15:03 |
| edleafe | johnthetubaguy: that 'git log' command didn't work. | 15:03 |
| dansmith | rlrossit: you don't | 15:03 |
| johnthetubaguy | edleafe: you mean git log --simplify-by-decoration --pretty="%d" origin/stable/mitaka | 15:03 |
| scottda | danpb: Not that I can think of offhand. Cinder just provides an iscsi target and you can do whatever you want with it... | 15:04 |
| bauzas | rlrossit: what dansmith said | 15:04 |
| dansmith | rlrossit: you just (a) don't run two schedulers and (b) have to reset scheduler as part of the api and conductor group | 15:04 |
| edleafe | johnthetubaguy: Do I have to explicitly check out stable mitaka locally? | 15:04 |
| johnthetubaguy | edleafe: have you done "git remote origin update" ? | 15:04 |
| markus_z | edleafe: maybe your git version doesn't know yet the flag "--simplify-by-decoration"? | 15:04 |
| bauzas | it's not yet part of the contract we honor | 15:04 |
| scottda | danpb: i.e. you can attach 2 volumes to the target, or 10. | 15:04 |
| *** kevinzh has joined #openstack-nova | 15:04 | |
| johnthetubaguy | markus_z: oh, good point | 15:04 |
| edleafe | markus_z: no, I get "fatal: ambiguous argument 'origin/stable/mitaka': unknown revision or path not in the working tree." | 15:04 |
| rlrossit | dansmith: so if I do have multiple schedulers, I need to take them all down when doing an upgrade | 15:04 |
| rlrossit | ? | 15:04 |
| johnthetubaguy | edleafe: yeah, try that remote origin update | 15:05 |
| *** moorryan has joined #openstack-nova | 15:05 | |
| *** ducttape_ has joined #openstack-nova | 15:05 | |
| scottda | danpb: But there is plenty I don't know, so I might be missing something. I'll point people in #cinder to this, in case there's other insight. | 15:05 |
| dansmith | rlrossit: they're just like API nodes.. they have no protection... it may work on one release depending on what has changed, but otherwise, yeah | 15:05 |
| *** shaohe_feng has quit IRC | 15:05 | |
| *** tbachman_ has joined #openstack-nova | 15:05 | |
| dansmith | rlrossit: does't matter that you have one or two, they're both the same | 15:05 |
| *** tbachman has quit IRC | 15:05 | |
| *** tbachman_ is now known as tbachman | 15:05 | |
| rlrossit | dansmith: I'm not following, why doesn't it matter if I have one or two? | 15:05 |
| *** shaohe_feng has joined #openstack-nova | 15:06 | |
| edleafe | johnthetubaguy: doesn't seem to be a legal command | 15:06 |
| edleafe | johnthetubaguy: "error: Unknown subcommand: origin" | 15:06 |
| johnthetubaguy | edleafe: sorry, git remote update origin | 15:06 |
| edleafe | johnthetubaguy: doh! | 15:06 |
| *** paul-carlton2 has quit IRC | 15:06 | |
| dansmith | rlrossit: it's just no different to have N vs. 1 scheduler.. neither calls through conductor nor has any sort of upgrade protection | 15:06 |
| gsilvis | mriedem: I am, as is knikolla | 15:06 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Add swap volume notifications (start, end) https://review.openstack.org/250283 | 15:07 |
| *** paul-carlton2 has joined #openstack-nova | 15:07 | |
| mriedem | gsilvis: ack | 15:07 |
| rlrossit | dansmith: oh I thought you were saying there is literally nothing gained by horizontally scaling the scheduler | 15:07 |
| rlrossit | that's the part that confused me | 15:07 |
| edleafe | johnthetubaguy: ok, that pulled stable/liberty and stable/mitaka | 15:07 |
| *** kevinzs has quit IRC | 15:07 | |
| *** kevin_zh has quit IRC | 15:07 | |
| dansmith | no | 15:07 |
| openstackgerrit | Charles Volzka proposed openstack/nova: Add get_host_memory_stats virt driver API https://review.openstack.org/288838 | 15:07 |
| openstackgerrit | Charles Volzka proposed openstack/nova: Add memory metric monitor https://review.openstack.org/288848 | 15:07 |
| *** kevin_zh has joined #openstack-nova | 15:07 | |
| bauzas | sudipto: well, not sure we should import some 3rd-party library this way https://review.openstack.org/#/c/203163/ | 15:07 |
| *** code-R has quit IRC | 15:07 | |
| *** armax_ has joined #openstack-nova | 15:08 | |
| *** dtantsur is now known as dtantsur|afk | 15:08 | |
| johnthetubaguy | edleafe: wow, I guess you don't run that one much, I usually do that to update my origin/master, although its a little wasteful | 15:08 |
| openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova: libvirt: virtlogd: use virtlogd for char devices https://review.openstack.org/323765 | 15:08 |
| *** paul-carlton2 has quit IRC | 15:08 | |
| openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova: libvirt: simplify "get_console_output" interface https://review.openstack.org/334480 | 15:08 |
| johnthetubaguy | edleafe: the gate does a fresh clone I guess, which would do the same thing | 15:08 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: [functional] Fix wrong message in server description test https://review.openstack.org/336079 | 15:08 |
| markus_z | danpb: ^ uses the workaround config | 15:08 |
| *** paul-carlton2 has joined #openstack-nova | 15:08 | |
| *** armax has quit IRC | 15:09 | |
| *** armax_ is now known as armax | 15:09 | |
| *** mgould has joined #openstack-nova | 15:09 | |
| *** ducttape_ has quit IRC | 15:09 | |
| edleafe | johnthetubaguy: ok, I'm just trying to figure out why it didn't like the reno in https://review.openstack.org/335648 | 15:09 |
| mriedem | sudipto: bauzas: libvirt-python is optional and managed through g-r | 15:09 |
| BobBall | mriedem: No probs. You're right, could have added in the first version. | 15:09 |
| mriedem | as is oslo.vmware | 15:09 |
| rlrossit | dansmith: so if I have multiple APIs, it's kinda the same thing, you better figure it out yourself? | 15:10 |
| mriedem | BobBall: i even saw you said 'obviously need to add tests but just poc' | 15:10 |
| mriedem | :) | 15:10 |
| markus_z | edleafe: could be the # char | 15:10 |
| andreykurilin | Folks, does anyone know about changes in error messages for `nova update` call? | 15:10 |
| *** sdake has joined #openstack-nova | 15:10 | |
| rlrossit | is there documentation anywhere telling me how I should do an upgrade like that? | 15:10 |
| *** jaypipes has joined #openstack-nova | 15:10 | |
| dansmith | rlrossit: you just have to reset them at the same time.. total outage window like 10ms | 15:10 |
| bauzas | mriedem: yeah, that's my point | 15:10 |
| edleafe | markus_z: that's what I was thinking, but I wanted to test locally first | 15:10 |
| mriedem | sudipto: i guess this is the lib? https://pypi.python.org/pypi/pcp/0.1 | 15:10 |
| rlrossit | dansmith: you're making the assumption they all come back up properly ;) | 15:10 |
| bauzas | mriedem: just importing a pypi libary without explaining why at least is bad | 15:10 |
| dansmith | rlrossit: is this line of questioning related to FF today? | 15:11 |
| rlrossit | dansmith: it is not | 15:11 |
| *** coreywright has quit IRC | 15:11 | |
| BobBall | mriedem: oops :) | 15:11 |
| mriedem | rlrossit: we have several fires right now | 15:11 |
| dansmith | rlrossit: I will totally discuss this with you at length after tomorrow, if that's okay :) | 15:11 |
| rlrossit | dansmith: fine with me | 15:11 |
| *** ducttape_ has joined #openstack-nova | 15:11 | |
| dansmith | rlrossit: sorry, but thanks | 15:12 |
| mriedem | rlrossit: there are upgrade docs in the ops guide i believe | 15:12 |
| sudipto | mriedem, yeah that's right... | 15:12 |
| mriedem | rlrossit: or probably ask around in #openstack-operators | 15:12 |
| *** coreywright has joined #openstack-nova | 15:12 | |
| mriedem | sudipto: pre-alpha 0.1 :( | 15:12 |
| bauzas | rlrossit: TBH, that just means to me that we can't live upgrade the schedulers, that's it | 15:12 |
| *** diogogmt has joined #openstack-nova | 15:12 | |
| sudipto | mriedem, asked in the pcp channel...just give me a few mins... | 15:13 |
| *** sdake_ has quit IRC | 15:13 | |
| sudipto | typically, i don't install it via pip - | 15:13 |
| rlrossit | bauzas: which, for most people is fine, but now I need to start thinking in mega scale, where things like that start breaking down :) | 15:13 |
| bauzas | rlrossit: but there are other ways for circling around upgrading schedulers :) | 15:13 |
| *** isq_ has quit IRC | 15:13 | |
| sudipto | mriedem, like i thought, this is the response " they just get distributed alongside pcp core, built at the same time as the c code" | 15:13 |
| bauzas | sudipto: well, not sure I love some library being installed by other way than pip honestly | 15:13 |
| sudipto | bauzas, ok. | 15:14 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Add swap volume notifications (error) https://review.openstack.org/328055 | 15:14 |
| mriedem | i guess most distros will probably also package their python/perl/ruby/java/c clients along with the main service package | 15:14 |
| *** muawiakhan has quit IRC | 15:15 | |
| *** tbachman has quit IRC | 15:15 | |
| danpb | PaulMurray: cinder is running on the target host. so another possibility could be that when qemu is started on the target, and neutron configures its nics, perhaps something is causing a temporary network interruption which breaks the iscsi connection to the source host | 15:15 |
| openstackgerrit | Victor Stinner proposed openstack/nova: Port test_fakelibvirt to Python 3 https://review.openstack.org/333251 | 15:15 |
| openstackgerrit | Victor Stinner proposed openstack/nova: Port ironic test_driver to Python 3 https://review.openstack.org/333250 | 15:15 |
| openstackgerrit | Victor Stinner proposed openstack/nova: Port libvirt test_driver to Python 3 https://review.openstack.org/333249 | 15:15 |
| openstackgerrit | Victor Stinner proposed openstack/nova: Port test_virt_drivers to Python 3 https://review.openstack.org/333248 | 15:15 |
| openstackgerrit | Victor Stinner proposed openstack/nova: Port virt test_hardware to Python 3 https://review.openstack.org/333247 | 15:15 |
| openstackgerrit | Victor Stinner proposed openstack/nova: Port pci unit tests to Python 3 https://review.openstack.org/332791 | 15:15 |
| openstackgerrit | Victor Stinner proposed openstack/nova: Port test_metadata to Python 3 https://review.openstack.org/332738 | 15:15 |
| sudipto | mriedem, i guess that's the case here... | 15:15 |
| openstackgerrit | Victor Stinner proposed openstack/nova: Port test_hacking to Python 3 https://review.openstack.org/332737 | 15:15 |
| bauzas | mriedem: but that's another story, not the upstream one :) | 15:15 |
| *** shaohe_feng has quit IRC | 15:15 | |
| mriedem | bauzas: sure, you still have to get the pcpd from the distro | 15:15 |
| mriedem | well, not have to i guess | 15:15 |
| mriedem | you could build from source yourself | 15:15 |
| bauzas | mriedem: yup, I mean I'm fine with distros providing their own versions | 15:16 |
| mriedem | what do we do with osinfo? | 15:16 |
| bauzas | mriedem: but that's something we at least need to import from pypi upstream IMO | 15:16 |
| *** shaohe_feng has joined #openstack-nova | 15:16 | |
| *** rrecio has joined #openstack-nova | 15:16 | |
| mriedem | libosinfo = importutils.import_module( | 15:16 |
| mriedem | 'gi.repository.Libosinfo') | 15:16 |
| *** jlanoux has joined #openstack-nova | 15:17 | |
| *** Daisy has joined #openstack-nova | 15:17 | |
| bauzas | huh | 15:17 |
| mriedem | bauzas: there is a py-osinfo in pypi | 15:17 |
| bauzas | I thought we added it to g-r | 15:17 |
| mriedem | nope | 15:18 |
| bauzas | so, say I want to use a big 3rd-party lib, I could just import the python API in Nova and not add it as a dep ? | 15:18 |
| openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova: libvirt: virtlogd: use virtlogd for char devices https://review.openstack.org/323765 | 15:19 |
| danpb | mriedem: bauzas libosinfo doesn't have any python bindings at all | 15:19 |
| *** rcernin has quit IRC | 15:19 | |
| danpb | the magic of gobject introspection means the python interface just magically appears as a FFI wrapper around the c library | 15:20 |
| mriedem | https://www.google.com/url?sa=t&rct=j&q=&esrc=s&source=web&cd=1&ved=0ahUKEwiriuXohdDNAhXJOSYKHdjlD6MQFggcMAA&url=https%3A%2F%2Fpypi.python.org%2Fpypi%2Fpy-osinfo&usg=AFQjCNFcQwhU9mzzXPJTJguh_gfB3Kaufg&sig2=PRXpHASMqrtHL_bY9xgIsA&bvm=bv.125801520,d.eWE&cad=rja | 15:20 |
| mriedem | oops | 15:20 |
| mriedem | # TODO(vladikr) The current implementation will serve only as a temporary | 15:20 |
| mriedem | # solution, due to it's dependency on the libosinfo gobject library. | 15:20 |
| mriedem | # In the future it will be replaced by a pure python library or by a direct | 15:20 |
| mriedem | # parsing of the libosinfo XML files. However, it will be possible only when | 15:20 |
| mriedem | # libosinfo project will declare the XML structure to be a stable ABI. | 15:20 |
| mriedem | yeah | 15:20 |
| bauzas | mriedem: google troller | 15:20 |
| bauzas | danpb: ack | 15:20 |
| mriedem | danpb: that's how libvirt-python works right? | 15:20 |
| PaulMurray | danpb, does all this happen before the migration is considered complete ? | 15:20 |
| mriedem | whch always confuses me | 15:20 |
| danpb | no, libvirt python is a hand written python C binding | 15:20 |
| openstackgerrit | Victor Stinner proposed openstack/nova: Port test_fakelibvirt to Python 3 https://review.openstack.org/333251 | 15:21 |
| openstackgerrit | Victor Stinner proposed openstack/nova: Port ironic test_driver to Python 3 https://review.openstack.org/333250 | 15:21 |
| openstackgerrit | Victor Stinner proposed openstack/nova: Port libvirt test_driver to Python 3 https://review.openstack.org/333249 | 15:21 |
| openstackgerrit | Victor Stinner proposed openstack/nova: Port test_virt_drivers to Python 3 https://review.openstack.org/333248 | 15:21 |
| openstackgerrit | Victor Stinner proposed openstack/nova: Port virt test_hardware to Python 3 https://review.openstack.org/333247 | 15:21 |
| PaulMurray | danpb, the networking gets configured after the migration completes | 15:21 |
| bauzas | the thing is, I'm fine with us commenting in the relnotes that "this" feature now uses a 3rd party lib | 15:21 |
| danpb | mriedem: nb py-osinfo is nothing at all todo with libosinfo - just a name collision | 15:21 |
| PaulMurray | danpb, when the ports are bound on the destination | 15:21 |
| *** Daisy has quit IRC | 15:21 | |
| danpb | PaulMurray: yeah, we get the fail before we switch to running on the target | 15:22 |
| *** armax has quit IRC | 15:22 | |
| sudipto | bauzas, i think i mentioned that in the release notes... | 15:22 |
| danpb | PaulMurray: it just wonder if there's some background task in neutron - as we've created the ports by then, just not started using them | 15:22 |
| bauzas | sudipto: orly? in that change? | 15:22 |
| PaulMurray | danpb, that could be right | 15:22 |
| PaulMurray | danpb, I'm guessing we don't use DVR in CI | 15:23 |
| sudipto | https://review.openstack.org/#/c/329785/11/releasenotes/notes/bp-memory-bw-c423266d4e4ae663.yaml bauzas - it's got a minor space issue, but you can comment if i need to add more info. | 15:23 |
| danpb | and i know neutron does alot of async work after api requests finish | 15:23 |
| *** burt has joined #openstack-nova | 15:23 | |
| bauzas | sudipto: oh that's in a follow-up change | 15:23 |
| *** jistr|mtg is now known as jistr | 15:23 | |
| sudipto | bauzas, yeah...on the final one that enables the monitor. | 15:23 |
| bauzas | sudipto: well, I was about to say that relnotes are good, but we loose track on what we add when we're on the next cycle | 15:24 |
| danpb | mriedem: sudipto bauzas looking at this pcp change, i can't help thinking this functionality should be exposed directly by libvirt | 15:24 |
| mgould | hi everyone | 15:24 |
| danpb | rather than having it go out to this pcp api | 15:24 |
| openstackgerrit | Sarafraj Singh proposed openstack/nova: Improve help context of ironic options https://review.openstack.org/332433 | 15:24 |
| *** gb21_ has joined #openstack-nova | 15:24 | |
| mgould | over in #openstack-ironic, we're trying to understand the extra_specs syntax | 15:24 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Add swap volume notifications samples https://review.openstack.org/328866 | 15:24 |
| danpb | we're already doing similar stuff in libvirt with the code we just added to libvirt + nova to query performance counters for cpu cache related stuff | 15:24 |
| sudipto | danpb, yeah, when i started working on this about 1 year back - there was no libvirt exposure to perf data...i see that there are a few patches submitted now. | 15:24 |
| bauzas | danpb: oh good to know, is libvirt exposing that ? | 15:25 |
| mgould | is the value being matched a string, or can it be something else? | 15:25 |
| *** GB21 has quit IRC | 15:25 | |
| danpb | so it feels like it would be better to do memory bandwidth in this same way | 15:25 |
| *** gb21_ is now known as GB21 | 15:25 | |
| dansmith | danpb: if available, that sounds like a much better plan to me | 15:25 |
| *** browne has joined #openstack-nova | 15:25 | |
| *** gagehugo has quit IRC | 15:25 | |
| mriedem | one knock against that will be, | 15:25 |
| sudipto | danpb, while libvirt is definitely a good route from dependency management per say - i guess pcp gives a lot of insight into perf data, that libvirt has to still build into? | 15:25 |
| danpb | dansmith: nb the API structure is available - we'd have to actually make libvirt be able to report the memory bandwidth stuff before we could use it | 15:25 |
| mriedem | that's libvirt specific | 15:25 |
| *** shaohe_feng has quit IRC | 15:25 | |
| danpb | so you'd obviously end up waiting another release cycle | 15:25 |
| mriedem | but having said that, we only track cpu metrics for libvirt i think | 15:26 |
| *** shaohe_feng has joined #openstack-nova | 15:26 | |
| bauzas | danpb: that would be a far better option actually | 15:26 |
| danpb | mriedem: this new proposed code is libvirt specific already iiuc | 15:26 |
| mriedem | well maybe not, but at least for this bw monitor thing | 15:26 |
| dansmith | mriedem: this is dependent on the libvirt driver anyway | 15:26 |
| mriedem | yeah | 15:26 |
| mriedem | the cpu monitor just uses host stats right? | 15:26 |
| bauzas | mriedem: yup, what dansmith said | 15:26 |
| mriedem | which comes from all virt drivers | 15:26 |
| dansmith | mriedem: you can still have the monitor use a generic blob like cpu, | 15:26 |
| dansmith | you just should get that data from libvirt in the driver | 15:27 |
| dansmith | same structure as the cpu one | 15:27 |
| danpb | commit 71f24dfeeeb6713f530b260a0e2b4bb23c00effa | 15:27 |
| danpb | Author: Eli Qiao <liyong.qiao@intel.com> | 15:27 |
| danpb | Date: Wed May 25 14:48:08 2016 +0800 | 15:27 |
| danpb | libvirt: add perf event support when create instance | 15:27 |
| danpb | that's the change i'm refering to in nova | 15:27 |
| *** sacharya_ has joined #openstack-nova | 15:27 | |
| *** tesseract- has quit IRC | 15:27 | |
| mriedem | that's per instance though right? | 15:27 |
| mriedem | sudipto's is per host | 15:27 |
| openstackgerrit | Sarafraj Singh proposed openstack/nova: Improve help context of ironic options https://review.openstack.org/332433 | 15:28 |
| sudipto | mriedem, yeah that's per instance and only x86 specific at the moment. | 15:28 |
| bauzas | mriedem: nope, monitors are per host | 15:28 |
| danpb | mriedem: oh, hmm, yes, good point | 15:28 |
| bauzas | unless I'm heavily wrong | 15:28 |
| mriedem | bauzas: so s/nope/yup/ | 15:28 |
| danpb | i still wonder if we should add a api to libvirt to report these per-host perf data | 15:28 |
| mriedem | danpb: that's not yet in upstream libvirt? | 15:29 |
| *** kaisers has quit IRC | 15:29 | |
| bauzas | mriedem: I strongly don't disagree with people not expressing their disagreements with me | 15:29 |
| mriedem | bauzas: your double negative mastery amazes me | 15:29 |
| danpb | mriedem: indeed not | 15:29 |
| mriedem | danpb: ok, so that would push actually testing this out or being in a distro quite a bit | 15:30 |
| danpb | yeah, as i say, it'd certainly delay things for 1 nova cycle | 15:30 |
| *** sacharya has quit IRC | 15:30 | |
| sudipto | danpb, i read through the libvirt patches that were submitted for the L3-cache and memory bandwidth (which i think will be done next term), however personally i don't like the way perf is being handled by libvirt with the current design. | 15:30 |
| * mriedem steps away for a bit | 15:30 | |
| markus_z | edleafe: It's a mixture of the # plus : | 15:30 |
| sudipto | however, that's a different forum discussion maybe... | 15:30 |
| bauzas | mriedem: but monitors are entrypoints, right? | 15:30 |
| markus_z | edleafe: I don't know why, but I could solve it locally by removing those two chars | 15:31 |
| bauzas | so, people can ship their own pcp monitors out-of-tree if they wish | 15:31 |
| mriedem | bauzas: i was told those shouldn't be pluggable | 15:31 |
| mriedem | but i think they are yeah | 15:31 |
| mriedem | and sudipto's series changed one of the ABC methods in the base class which will break any plugins out of tree | 15:31 |
| mriedem | which was a comment i brought up a few times | 15:31 |
| *** code-R_ has joined #openstack-nova | 15:31 | |
| bauzas | mriedem: yup, but that's a good move | 15:31 |
| mriedem | breaking stuff? | 15:32 |
| tjones | mriedem: there are just 4 vmware patches ready for core. rado replied to your -1 last night. they are https://review.openstack.org/#/c/281134 https://review.openstack.org/#/c/196171/ https://review.openstack.org/#/c/204219/ https://review.openstack.org/#/c/297574/ thanks if you guys can give them a look | 15:32 |
| bauzas | mriedem: if the breakage leads to a better interface | 15:32 |
| bauzas | mriedem: that's what we did with filters using the RequestSpec for example :) | 15:32 |
| sudipto | the reason i said i didn't like the libvirt implementation of perf is because it hardcodes the value of the platform it supports into libvirt itself...while pcp depends a external library - to load it for various processor types. | 15:32 |
| sudipto | with that said, the memory bandwidth implementation is currently only there for x86. | 15:33 |
| mgould | bauzas: I'm trying to understand the semantics of extra_specs, can you help? | 15:33 |
| mriedem | mdbooth: can you see https://bugs.launchpad.net/nova/+bug/1524274/comments/68 ? | 15:33 |
| openstack | Launchpad bug 1524274 in OpenStack Security Advisory "[OSSA 2016-001] Unprivileged api user can access host data using instance snapshot (CVE-2015-7548)" [Critical,Fix released] - Assigned to Tristan Cacqueray (tristan-cacqueray) | 15:33 |
| mdbooth | mriedem: Looking | 15:33 |
| bauzas | mgould: well, okay... will try :) | 15:33 |
| *** thumpba has joined #openstack-nova | 15:33 | |
| bauzas | flavor extra specs are a beast | 15:33 |
| mgould | in particular, does <all-in> apply to strings or to collections? | 15:33 |
| mriedem | sudipto: which is important for you because you care about ppc | 15:33 |
| mgould | bauzas: thanks! | 15:33 |
| PaulMurray | jaypipes, I added you to https://review.openstack.org/#/c/253556/ because you reviewed https://review.openstack.org/#/c/318974 | 15:34 |
| PaulMurray | jaypipes, I hope you don't mind - they were both doing the same thing at the same time | 15:34 |
| jaypipes | PaulMurray: will try to get to that. still waiting on gertty to sync up 980+ patchsets... | 15:34 |
| mgould | background: we need something similar in Ironic, and are trying to work out if the semantics are a good fit for us | 15:34 |
| sudipto | mriedem, i just care about perf to be honest :) and want it to be in a way that different archs can use it... | 15:34 |
| PaulMurray | jaypipes, np - just letting you know | 15:34 |
| openstackgerrit | Anusha Unnam proposed openstack/nova: Transform instance.resize notifications https://review.openstack.org/329255 | 15:34 |
| bauzas | mgould: you mean the filter semantics ? | 15:35 |
| mgould | yes | 15:35 |
| *** shaohe_feng has quit IRC | 15:36 | |
| mgould | sorry, my grasp of nova is still pretty weak | 15:36 |
| mdbooth | shhjwang: Looking at https://bugs.launchpad.net/nova/+bug/1524274/comments/68 | 15:36 |
| openstack | Launchpad bug 1524274 in OpenStack Security Advisory "[OSSA 2016-001] Unprivileged api user can access host data using instance snapshot (CVE-2015-7548)" [Critical,Fix released] - Assigned to Tristan Cacqueray (tristan-cacqueray) | 15:36 |
| mdbooth | Firstly, are you sure you have a vulnerable version of Nova? | 15:37 |
| *** shaohe_feng has joined #openstack-nova | 15:37 | |
| mdbooth | This was fixed quite a while ago | 15:37 |
| mgould | bauzas: further context is we're looking at https://review.openstack.org/#/c/308398 and trying to decide what tests it needs | 15:37 |
| bauzas | mgould: well, the UTs are good examples, wsec | 15:37 |
| *** fche has joined #openstack-nova | 15:37 | |
| shhjwang | mdbooth: Yeah, my platform has the unpatched nova. | 15:37 |
| bauzas | mgould: https://github.com/openstack/nova/blob/master/nova/tests/unit/scheduler/filters/test_extra_specs_ops.py#L202-L235 | 15:38 |
| mdbooth | shhjwang: Why is the passwd file a binary file? | 15:39 |
| sudipto | mriedem, danpb i am not sure if that's an convincing answer, but can you tell me what i should do next? | 15:39 |
| *** lberk has joined #openstack-nova | 15:39 | |
| *** pushkaru has quit IRC | 15:39 | |
| mgould | bauzas: right, so it's matching against a list | 15:40 |
| mgould | thanks! | 15:40 |
| *** sridharg has quit IRC | 15:40 | |
| shhjwang | mdbooth: You mean it should not be a binary file? I use " glance image-download xxx --file passwd" as mentioned. So the generated passwd is binary. | 15:41 |
| *** jaugustine has quit IRC | 15:41 | |
| bauzas | mgould: indeed | 15:41 |
| bauzas | because we .split() | 15:41 |
| mdbooth | shhjwang: Run file against it. What does it say? | 15:41 |
| mdbooth | shhjwang: $ file passwd | 15:41 |
| mdbooth | What does it say? | 15:41 |
| bauzas | mgould: https://github.com/openstack/nova/blob/master/nova/scheduler/filters/extra_specs_ops.py#L39 | 15:42 |
| *** rlrossit has quit IRC | 15:42 | |
| bauzas | mgould: so we contract on a string, but we analyse it as a list | 15:42 |
| openstackgerrit | Chris Friesen proposed openstack/nova: Allow logging detailed info only on sched failure https://review.openstack.org/331306 | 15:44 |
| *** kaisers has joined #openstack-nova | 15:44 | |
| mgould | bauzas: I see the tests match str(values) | 15:44 |
| shhjwang | mdbooth: It shows "passwd: Qemu Image, Format: Qcow , Version: 2" | 15:44 |
| mgould | which means specs_matcher.match(str(['foo', 'bar']), '<all-in> f') returns True | 15:44 |
| mgould | is that expected behaviour? | 15:44 |
| bauzas | mgould: see my above link | 15:45 |
| mdbooth | shhjwang: Ok, run $ qemu-img info passwd | 15:45 |
| mdbooth | What backing file does it have? | 15:45 |
| mgould | bauzas: the link to the tests? | 15:45 |
| mgould | yes, that's what I'm looking at | 15:45 |
| *** shaohe_feng has quit IRC | 15:46 | |
| *** rlrossit has joined #openstack-nova | 15:46 | |
| gibi_ | bauzas, johnthetubaguy: before you call it a day could you check (hopefully last time) the notification transformation patches? | 15:46 |
| *** shaohe_feng has joined #openstack-nova | 15:47 | |
| mgould | bauzas: let me push up a patch so you can see what I'm talking about, one moment... | 15:47 |
| bauzas | mgould: no, the fact that the match() method takes a string as a req, but splits it | 15:47 |
| shhjwang | mdbooth: No backing file was shown. | 15:47 |
| mgould | bauzas: you mean it splits the matcher spec? | 15:47 |
| mgould | I'm talking about the value-being-matched | 15:48 |
| shhjwang | mdbooth: So this is the behavior after the patches? | 15:48 |
| mdbooth | shhjwang: The behaviour after the patches should be that you get the contents of bad.qcow2 in passwd | 15:48 |
| mdbooth | Before the patches you get the contents of /etc/passwd | 15:48 |
| *** harlowja_at_home has joined #openstack-nova | 15:48 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova: WIP: policy: clean-up https://review.openstack.org/333925 | 15:49 |
| *** harlowja_at_home has quit IRC | 15:49 | |
| mriedem | jlvillal: is there an ironic patch running with dansmith's workaround? | 15:49 |
| jlvillal | mriedem, Yes | 15:49 |
| bauzas | mgould: it splits each value of the dict that is instance_type.extra_specs | 15:50 |
| mriedem | ah yeah https://review.openstack.org/#/c/335998/ | 15:50 |
| jlvillal | mriedem, https://review.openstack.org/#/c/335998/ | 15:50 |
| jlvillal | mriedem, Darn, you are faster than me :) | 15:50 |
| *** salv-orlando has quit IRC | 15:50 | |
| openstackgerrit | STEW TY proposed openstack/nova: Complete verification for os-floating-ips-bulk https://review.openstack.org/333075 | 15:51 |
| jlvillal | mriedem, I updated the bug to say a test with reverting that one patch did fix the issue. | 15:51 |
| mriedem | jlvillal: yeah but we're not going to revert it | 15:51 |
| mriedem | jlvillal: it's part of the rest api now | 15:51 |
| mriedem | *required for the rest api now | 15:51 |
| jlvillal | Okay | 15:51 |
| mriedem | dam | 15:51 |
| shhjwang | mdbooth: But why the backing file is not there. I have set use_cow_images to false | 15:51 |
| mriedem | *dan's change should workaround it | 15:51 |
| dansmith | artom: question in here: https://review.openstack.org/#/c/304511/70 | 15:51 |
| mriedem | as it will make the mac unique per port uuid | 15:52 |
| shhjwang | mdbooth: And qemu-img info bad.qcow2 shows the backing file. | 15:52 |
| mriedem | vif.address i mean | 15:52 |
| dansmith | mriedem: yeah, been watching it | 15:52 |
| mriedem | the telnet thing doesn't work for me | 15:52 |
| mdbooth | shhjwang: What's images_type in nova.conf? | 15:52 |
| edleafe | markus_z: I don't think it was the # or the : - I fixed it by adding a | and newline | 15:53 |
| edleafe | markus_z: like this: http://paste.openstack.org/show/524296/ | 15:53 |
| *** ctrath has quit IRC | 15:53 | |
| shhjwang | mdbooth: It's default | 15:53 |
| mgould | bauzas: so if instance_type.extra_specs is {k: 'foo bar'} then '<all-in> f a' should not match it, but '<all-in> foo bar' should? | 15:53 |
| shhjwang | images_type=default | 15:53 |
| *** gagehugo has joined #openstack-nova | 15:53 | |
| mriedem | edleafe: have you looked at http://docs.openstack.org/developer/reno/usage.html ? | 15:54 |
| mriedem | http://docs.openstack.org/developer/reno/usage.html#formatting | 15:54 |
| edleafe | mriedem: yes, but never got that far down :) | 15:54 |
| mriedem | - | is required for ReST | 15:54 |
| *** fragatina has quit IRC | 15:54 | |
| *** EinstCrazy has quit IRC | 15:55 | |
| mdbooth | shhjwang: What do you get if you do: $ qemu-img convert -f qcow2 -O raw passwd passwd.raw? | 15:55 |
| *** fragatina has joined #openstack-nova | 15:55 | |
| *** pbandark has quit IRC | 15:55 | |
| mriedem | oh nvm the telnet thing does work, nice | 15:55 |
| mdbooth | shhjwang: I wonder if it's the snapshot image conversion converting the file to raw | 15:55 |
| mdbooth | s/raw/qcow2/ | 15:55 |
| *** ctrath has joined #openstack-nova | 15:55 | |
| bauzas | mgould: yup, like what the test says :) | 15:55 |
| * mdbooth vaguely recalls something about that | 15:55 | |
| markus_z | edleafe: Ha, interesting. | 15:55 |
| *** andreas_s has quit IRC | 15:56 | |
| *** shaohe_feng has quit IRC | 15:56 | |
| edleafe | markus_z: mriedem: Funny thing is that the template you get when creating a reno doesn't follow the correct formatting | 15:56 |
| shhjwang | mdbooth: I get the file passwd.raw without any console output. | 15:57 |
| edleafe | I just followed the template's format | 15:57 |
| mdbooth | shhjwang: What does passwd.raw contain? | 15:57 |
| *** shaohe_feng has joined #openstack-nova | 15:57 | |
| mgould | bauzas: OK, I think there's a bug in the tests then | 15:57 |
| openstackgerrit | Miles Gould proposed openstack/nova: DO NOT MERGE! Test all-in semantics https://review.openstack.org/336094 | 15:58 |
| mgould | it should say `value=values` rather than `value=str(values)` | 15:58 |
| mriedem | dansmith: artom: i had commented on the same version thing last night | 15:58 |
| mriedem | in the meta api | 15:58 |
| dansmith | mriedem: oh sorry I missed that | 15:58 |
| mriedem | it might have been rebased out, not sure | 15:58 |
| markus_z | edleafe: I've seen "|" sometimes in other reno files, but was never sure why it was there. Maybe an update on https://github.com/openstack/reno/blob/master/reno/create.py is appropriate? | 15:59 |
| shhjwang | mdbooth: OK. Now "qemu-img info" shows backing file /etc/passwd | 15:59 |
| mriedem | anyway, easy fix | 15:59 |
| edleafe | markus_z: I was just looking at that :) | 15:59 |
| *** mgoddard has quit IRC | 15:59 | |
| mdbooth | shhjwang: Ok, sounds like you got a qcow2 whose contents is your bad.qcow2 | 15:59 |
| mdbooth | Which means you haven't hit the vulnerability | 16:00 |
| dansmith | jlvillal: are the other failures on that patch expected? | 16:00 |
| mdbooth | I can't promise that's because you're not vulnerable, though | 16:00 |
| *** dims has quit IRC | 16:00 | |
| shhjwang | mdbooth: so is there any limitation on the type of glance image or any other things? | 16:01 |
| jlvillal | dansmith, Let me look | 16:01 |
| shhjwang | mdbooth: I have a glance images with raw disk format. Is that right? | 16:02 |
| mdbooth | shhjwang: I'm not going to be able to help debug this this evening, unfortunately. Can you ping me tomorrow and I'll try to help? | 16:03 |
| mdbooth | In in the UK TZ (GMT + 1) | 16:03 |
| dansmith | johnthetubaguy: amazing job breaking these changes into move, refactor, move, refactor | 16:04 |
| *** DaiDV has joined #openstack-nova | 16:04 | |
| shhjwang | mdbooth: Yeah. Thank you very much for you help. | 16:04 |
| *** DaiDV has quit IRC | 16:05 | |
| *** gjayavelu has quit IRC | 16:05 | |
| *** matrohon has quit IRC | 16:05 | |
| jlvillal | dansmith, Regarding https://review.openstack.org/#/c/335998/ ? I don't see it finished running yet. I didn't check zuul though | 16:05 |
| dansmith | jlvillal: yeah, in zuul | 16:06 |
| *** tonytan4ever has quit IRC | 16:06 | |
| mriedem | running smoke tests | 16:06 |
| *** dims has joined #openstack-nova | 16:06 | |
| dansmith | jlvillal: a bunch of tests I would have expected to pass failed | 16:06 |
| danpb | anyone tell me which openstack git repo has the code to determine which logfiles to capture for CI runs ? | 16:06 |
| danpb | i need to edit it to capture log files from the iscsi server too | 16:06 |
| jlvillal | dansmith, Let me check zuul | 16:06 |
| mriedem | danpb: devstack-gate | 16:06 |
| dansmith | http://logs.openstack.org/98/335998/2/check/gate-tempest-dsvm-ironic-agent_ssh/f34ee43/ | 16:06 |
| dansmith | jlvillal: ^ | 16:06 |
| danpb | thanks | 16:06 |
| *** shaohe_feng has quit IRC | 16:06 | |
| mriedem | dansmith: that's a non-voting job | 16:07 |
| dansmith | jlvillal: ah crap it is my patch | 16:07 |
| mriedem | which always times out | 16:07 |
| *** shaohe_feng has joined #openstack-nova | 16:07 | |
| dansmith | the vif object does one thing strangely and screwed me yp | 16:07 |
| mriedem | ah | 16:07 |
| mnestratov|2 | mriedem: yeah I am | 16:07 |
| mriedem | mnestratov|2: ack | 16:08 |
| mriedem | did that rescue test run in the vz ci yet? | 16:08 |
| jlvillal | dansmith, Okay and :( | 16:08 |
| dansmith | crap and now a rebase | 16:08 |
| edleafe | markus_z: https://review.openstack.org/#/c/336098/ | 16:08 |
| *** Jeffrey4l has quit IRC | 16:08 | |
| mnestratov|2 | mriedem: running devstack right now | 16:09 |
| mriedem | NameError: global name \'self\' is not defined | 16:09 |
| mriedem | yikes | 16:09 |
| *** mnestratov|2 is now known as mnestratov | 16:09 | |
| mriedem | ah db_vif['address'].split | 16:10 |
| dansmith | johnthetubaguy's code makes it easier | 16:10 |
| mnestratov | yesterday we ran it with correct regexp but forgot to change tempest.conf accordinly, thus all rescue tests were skipped | 16:11 |
| openstackgerrit | Dan Smith proposed openstack/nova: HACK: Test making VIF.address unique with port id for neutron https://review.openstack.org/336069 | 16:12 |
| mriedem | danpb: log collection happens somewhere in here https://github.com/openstack-infra/devstack-gate/blob/master/functions.sh#L740 | 16:12 |
| dansmith | jlvillal: recheck ^ | 16:12 |
| *** baoli_ has quit IRC | 16:12 | |
| jlvillal | dansmith, will do | 16:12 |
| danpb | mriedem: yep, thanks | 16:12 |
| *** baoli has joined #openstack-nova | 16:13 | |
| dansmith | jlvillal: I'll work on local tests to catch any other issues sooner in case something else needs changing | 16:13 |
| *** ljxiash has quit IRC | 16:13 | |
| *** thorst_ has quit IRC | 16:13 | |
| jlvillal | dansmith, Thanks. And recheck started | 16:13 |
| *** thorst_ has joined #openstack-nova | 16:14 | |
| *** fragatina has quit IRC | 16:15 | |
| *** thorst__ has joined #openstack-nova | 16:15 | |
| *** gongysh has joined #openstack-nova | 16:15 | |
| *** kevinzs has joined #openstack-nova | 16:15 | |
| *** sdake_ has joined #openstack-nova | 16:15 | |
| *** cdelatte has quit IRC | 16:16 | |
| *** rama_y has joined #openstack-nova | 16:16 | |
| *** shaohe_feng has quit IRC | 16:17 | |
| *** migi_ is now known as migi | 16:17 | |
| *** shaohe_feng has joined #openstack-nova | 16:17 | |
| *** browne has quit IRC | 16:17 | |
| *** thorst_ has quit IRC | 16:18 | |
| *** sdake has quit IRC | 16:18 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova: trivial: remove unnecessary mock from servers API test https://review.openstack.org/336105 | 16:19 |
| dansmith | jlvillal: tests say it's good | 16:19 |
| *** kevinzs has quit IRC | 16:20 | |
| *** thorst__ has quit IRC | 16:20 | |
| *** kevin_zh has quit IRC | 16:20 | |
| *** kevinzh has quit IRC | 16:20 | |
| *** zhangjn has quit IRC | 16:20 | |
| jlvillal | dansmith, Great :) Hopefully Grenade too | 16:20 |
| *** aloga_ has joined #openstack-nova | 16:20 | |
| mriedem | tjones: nack on https://review.openstack.org/#/c/297574/ | 16:21 |
| mriedem | the spec was approved in mitaka and newton, and if there was a dependency in glanceclient, that shouldn't have been a surprise by now | 16:22 |
| *** pwnall has quit IRC | 16:22 | |
| *** pwnall has joined #openstack-nova | 16:22 | |
| *** sdake_ is now known as sdake | 16:23 | |
| *** zhangjn has joined #openstack-nova | 16:25 | |
| gibi_ | mriedem, dansmith: I'm off for about two hours but I can still fix issues with the notification transformation patches if you have any | 16:27 |
| openstackgerrit | Matthew Booth proposed openstack/nova: Remove max_size argument to images.fetch and fetch_to_raw https://review.openstack.org/326947 | 16:27 |
| openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Rename the import of nova.virt.disk.api in driver https://review.openstack.org/333521 | 16:27 |
| openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Add DiskFromImage and DiskFromFunc https://review.openstack.org/333522 | 16:27 |
| openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Pass is_block_dev explicitly from create_from_func https://review.openstack.org/333263 | 16:27 |
| openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Move swap disk fetch into a separate function https://review.openstack.org/333984 | 16:27 |
| openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Add _create_external_boot_disks https://review.openstack.org/333980 | 16:27 |
| openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Move local root disk fetch into a separate function https://review.openstack.org/333982 | 16:27 |
| openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Move enumeration of ephemerals into a separate function https://review.openstack.org/333983 | 16:27 |
| openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Replace some uses of invalid uuids in test_driver https://review.openstack.org/333978 | 16:27 |
| openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Don't re-resize disks in finish_migration() https://review.openstack.org/331118 | 16:27 |
| openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Remove some unnecessary mocking in test_driver https://review.openstack.org/333979 | 16:27 |
| openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Add create_from_image and create_from_func to Backend https://review.openstack.org/333244 | 16:27 |
| openstackgerrit | Matthew Booth proposed openstack/nova: Resize an existing disk https://review.openstack.org/332416 | 16:27 |
| *** shaohe_feng has quit IRC | 16:27 | |
| openstackgerrit | Matthew Booth proposed openstack/nova: Introduce ImageCacheLocalDir https://review.openstack.org/279669 | 16:27 |
| openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Rename Backend snapshot and image https://review.openstack.org/333272 | 16:27 |
| openstackgerrit | Matthew Booth proposed openstack/nova: Add check_backing_from_func & check_backing_from_image https://review.openstack.org/322285 | 16:27 |
| openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Use valid uuids and fnames in test_create_image_initrd https://review.openstack.org/333241 | 16:27 |
| openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Pass instance object to _create_images_and_backing in test https://review.openstack.org/333243 | 16:27 |
| mriedem | gibi_: i'm pretty sure mdbooth just killed gerrit | 16:27 |
| mriedem | but ok | 16:27 |
| mriedem | gerrit/zuul | 16:27 |
| * mdbooth hates those things anyway | 16:28 | |
| *** shaohe_feng has joined #openstack-nova | 16:28 | |
| gibi_ | mriedem: ohh, good move on ff day :) | 16:28 |
| *** armax has joined #openstack-nova | 16:28 | |
| andreykurilin | mriedem: could you look at +1;-1 change to unblock novaclient's gates?) | 16:28 |
| mdbooth | lyarwood: If you look around here: https://review.openstack.org/#/c/333984/ | 16:28 |
| mriedem | andreykurilin: which thing? | 16:29 |
| mdbooth | lyarwood: _create_image is now a really short function | 16:29 |
| andreykurilin | mriedem: https://review.openstack.org/#/c/336079/ | 16:29 |
| mdbooth | Well, comparatively short | 16:29 |
| openstackgerrit | Aihua Edward Li proposed openstack/nova: Always clean up network resource when VM creation failed close-bug: #1597596 https://review.openstack.org/335788 | 16:29 |
| openstack | bug 1597596 in OpenStack Compute (nova) "network not always cleaned up when spawning VMs" [Undecided,Fix committed] https://launchpad.net/bugs/1597596 - Assigned to Aihua Edward Li (aihuaedwardli) | 16:29 |
| mdbooth | lyarwood: So, rescue and spawn() both want to do: external boot, config disk, root disk, injection | 16:30 |
| mdbooth | That's all rescue wants to do | 16:30 |
| mdbooth | spawn want to additionally do ephemeral and swap | 16:30 |
| * mdbooth was going to create a function from the above bag of bits to do the common bits | 16:31 | |
| danpb | mdbooth: rescue should do ephemeral and swap too | 16:31 |
| lyarwood | mdbooth: that's all it wants to do for now, stable rescues would just tag on an additional boot disk at the end | 16:31 |
| mdbooth | danpb: This is for *creation* | 16:31 |
| lyarwood | mdbooth: ah right | 16:32 |
| mriedem | andreykurilin: when did that start failing? | 16:32 |
| mdbooth | danpb: Those are the only things it wants to create, but it should add everything for stable rescue | 16:32 |
| mdbooth | lyarwood: However, given the current behaviour of cache(), calling it on an existing ephemeral/swap disk is only a wart | 16:33 |
| mdbooth | Because it won't actually do anything | 16:33 |
| andreykurilin | mriedem: I don't know. I found that tempest release is changed in gates, but quick look of tempest changes did not show me the root of issue | 16:33 |
| lyarwood | that's a useful wart for now | 16:33 |
| mdbooth | Right | 16:33 |
| mdbooth | You can safely rely on that, in fact. Other code does. | 16:33 |
| mdbooth | This series is about removing that assumption, so I'm happy to clean up stable rescue. | 16:34 |
| mdbooth | I was planning to do it anyway | 16:34 |
| mdbooth | Well, unstable rescue | 16:34 |
| *** baoli has quit IRC | 16:35 | |
| lyarwood | yeah but without this _create_image rewrite I'd have to go in and hack around the eph creation logic at present | 16:35 |
| openstackgerrit | Paul Carlton proposed openstack/nova: Permit Live Migration of Rescued Instances https://review.openstack.org/308198 | 16:35 |
| mdbooth | lyarwood: Ahh... you said it's actually creating disk.eph0.rescue? | 16:35 |
| mdbooth | You can hack round that | 16:36 |
| *** ctrath has quit IRC | 16:36 | |
| mdbooth | See _create_image() has 2 functions up the top, image() and raw() | 16:36 |
| mdbooth | which are used to create image objects | 16:36 |
| lyarwood | mdbooth: disk.local.rescue sorry no eph | 16:36 |
| lyarwood | not* | 16:36 |
| mdbooth | Right, same thing | 16:36 |
| lyarwood | true | 16:36 |
| mdbooth | Update _create_image to not use image() for ephemeral and swap | 16:37 |
| mdbooth | It's not required for rescue anyway | 16:37 |
| *** vtech has joined #openstack-nova | 16:37 | |
| *** shaohe_feng has quit IRC | 16:37 | |
| mdbooth | And it's a much smaller fix than trying to take my series | 16:37 |
| *** kaisers has quit IRC | 16:37 | |
| mdbooth | Anyway, I must dash | 16:37 |
| lyarwood | mdbooth: np, thanks again | 16:38 |
| *** shaohe_feng has joined #openstack-nova | 16:38 | |
| *** baoli has joined #openstack-nova | 16:38 | |
| tiendc | dansmith: Hi, could you take a look at the gmann's comment in https://review.openstack.org/#/c/328157 | 16:39 |
| tiendc | dansmith: today is the deadline for this patch, I would appreciate if you can give your opinion about that | 16:40 |
| *** ctrath has joined #openstack-nova | 16:40 | |
| tiendc | dansmith: I'm trying to convince the core team for adding the patch to exception list | 16:40 |
| dansmith | tiendc: what? deadline today? I had no idea! | 16:40 |
| tiendc | dansmith: I means non-priority feature freeze | 16:41 |
| *** abalutoiu has quit IRC | 16:41 | |
| dansmith | tiendc: I know, I'm being sarcastic.. :) | 16:41 |
| markus_z | tiendc: you get used to it someday ;) | 16:41 |
| *** cdelatte has joined #openstack-nova | 16:41 | |
| dansmith | tiendc: I haven't looked at that code before so there are soooo many things pending today I'm not sure I'll get to it | 16:42 |
| dansmith | tiendc: it would be good to at least get an ack on it from someone like sahid who is familiar with that code | 16:42 |
| *** baoli has quit IRC | 16:43 | |
| *** ssurana has joined #openstack-nova | 16:43 | |
| mriedem | andreykurilin: https://review.openstack.org/#/c/333795/ | 16:43 |
| mriedem | ? | 16:43 |
| tiendc | dansmith: thanks, I will try to ask him | 16:44 |
| andreykurilin | mriedem: trivial fix. it didn't change nothing for py27 env | 16:44 |
| *** aloga_ has quit IRC | 16:45 | |
| *** paul-carlton has joined #openstack-nova | 16:46 | |
| andreykurilin | mriedem: python2.7 -c "import string;print(string.letters == string.ascii_letters)" => True | 16:46 |
| *** danpb has quit IRC | 16:47 | |
| *** paul-carlton2 has quit IRC | 16:47 | |
| *** apanaitiu has joined #openstack-nova | 16:47 | |
| *** shaohe_feng has quit IRC | 16:47 | |
| *** shaohe_feng has joined #openstack-nova | 16:48 | |
| *** zz_dimtruck is now known as dimtruck | 16:49 | |
| *** mdrabe_ has quit IRC | 16:50 | |
| *** diga has quit IRC | 16:50 | |
| *** lucasagomes is now known as lucas-afk | 16:50 | |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: Device tagging metadata API support https://review.openstack.org/304511 | 16:51 |
| openstackgerrit | Ed Leafe proposed openstack/nova: Return HTTP 400 on list for invalid status https://review.openstack.org/315964 | 16:51 |
| openstackgerrit | Ed Leafe proposed openstack/nova: Return HTTP 200 on list for invalid status https://review.openstack.org/335648 | 16:51 |
| *** markus_z has quit IRC | 16:51 | |
| *** salv-orlando has joined #openstack-nova | 16:51 | |
| *** abalutoiu has joined #openstack-nova | 16:51 | |
| mnestratov | mriedem: dansmith: our CI finally did what we begged it to do in https://review.openstack.org/#/c/217679 :) | 16:52 |
| * mriedem looks for "if change xyz pass rescue tests" :) | 16:52 | |
| dansmith | haha | 16:52 |
| *** abalutoiu_ has joined #openstack-nova | 16:53 | |
| artom | dansmith, hey, back now, went for a run, needed it badly - looks like vladikr already addressed the _os_version stuff in https://review.openstack.org/#/c/304511/, right? | 16:53 |
| *** baoli has joined #openstack-nova | 16:54 | |
| dansmith | artom: ah, hadn't noticed, but yeah | 16:54 |
| dansmith | artom: mdbooth was DoSing gerrit anyway | 16:54 |
| artom | dansmith, can you wag at him like you did at me last time? | 16:55 |
| dansmith | artom: mdbooth is on permanent wag status | 16:56 |
| *** abalutoiu has quit IRC | 16:56 | |
| *** RuiChen has quit IRC | 16:56 | |
| *** salv-orl_ has joined #openstack-nova | 16:57 | |
| mriedem | thar she blar 15:22:27 + [[ 217679,21 =~ 217679 ]] | 16:57 |
| mgould | bauzas: https://review.openstack.org/#/c/336094/ passed CI | 16:57 |
| *** diga has joined #openstack-nova | 16:57 | |
| mgould | which I think means the extra_specs tests aren't testing what you think | 16:58 |
| *** shaohe_feng has quit IRC | 16:58 | |
| *** apanaitiu has quit IRC | 16:58 | |
| *** salv-orlando has quit IRC | 16:58 | |
| *** _ducttape_ has joined #openstack-nova | 16:58 | |
| *** shaohe_feng has joined #openstack-nova | 16:58 | |
| *** dimtruck is now known as zz_dimtruck | 16:59 | |
| mriedem | dansmith: +2 on https://review.openstack.org/#/c/217679/ now | 17:00 |
| openstackgerrit | Ludovic Beliveau proposed openstack/nova: Fix revert on migration with SR-IOV https://review.openstack.org/326174 | 17:00 |
| openstackgerrit | Ludovic Beliveau proposed openstack/nova: Raise exception if PCI device spec is not found https://review.openstack.org/331830 | 17:00 |
| openstackgerrit | Ludovic Beliveau proposed openstack/nova: Allocate PCI devices on migration https://review.openstack.org/328983 | 17:00 |
| openstackgerrit | Ludovic Beliveau proposed openstack/nova: Update binding:profile for SR-IOV ports https://review.openstack.org/242573 | 17:00 |
| *** markmc` is now known as markmc | 17:00 | |
| *** daemontool_ has joined #openstack-nova | 17:01 | |
| *** sree has joined #openstack-nova | 17:01 | |
| dansmith | mriedem: cool | 17:01 |
| dansmith | mriedem: I'm just going through the tests of that metadata api patch and then I'm good on it too | 17:01 |
| *** abalutoiu_ has quit IRC | 17:02 | |
| *** salv-orl_ has quit IRC | 17:02 | |
| *** derekh has quit IRC | 17:02 | |
| *** ducttape_ has quit IRC | 17:03 | |
| mriedem | the newton version check should come after the liberty version check, but that's a nit | 17:03 |
| *** browne has joined #openstack-nova | 17:03 | |
| *** _ducttape_ has quit IRC | 17:03 | |
| *** vtech has quit IRC | 17:04 | |
| dansmith | mriedem: I thought they were going for alphabetical order | 17:04 |
| mnestratov | dansmith: mriedem: thanks! | 17:04 |
| *** daemontool has quit IRC | 17:04 | |
| dansmith | I guess not | 17:04 |
| dansmith | heh, r comes after p | 17:05 |
| *** thorst_ has joined #openstack-nova | 17:05 | |
| mriedem | small nit in the reno too | 17:05 |
| mriedem | this isn't modifying the userdata | 17:05 |
| mriedem | i think i introduced that confusion with a previous comment | 17:05 |
| *** sree has quit IRC | 17:05 | |
| mriedem | other than that i'm good | 17:06 |
| dansmith | okay | 17:06 |
| *** sudipto has quit IRC | 17:06 | |
| *** mvk has quit IRC | 17:06 | |
| *** chandankumar has quit IRC | 17:06 | |
| dansmith | jlvillal: the normal runs are passing now, which is good :) | 17:06 |
| *** thorst_ has quit IRC | 17:07 | |
| *** svenkat has joined #openstack-nova | 17:07 | |
| *** thorst_ has joined #openstack-nova | 17:07 | |
| jlvillal | dansmith, Yep only grenade left. But grenade was the only thing that was failing. | 17:07 |
| mriedem | vladikr: artom: want to address the nits quick? https://review.openstack.org/#/c/304511/ | 17:08 |
| dansmith | jlvillal: right, I just mean, the extra problem in my patch is fixed at least | 17:08 |
| jlvillal | So the patch isn't breaking anything it seems :) | 17:08 |
| mriedem | i'm going to grab lunch | 17:08 |
| *** shaohe_feng has quit IRC | 17:08 | |
| *** shaohe_feng has joined #openstack-nova | 17:08 | |
| openstackgerrit | Paul Carlton proposed openstack/nova: API changes for live migration of rescued instance https://review.openstack.org/328280 | 17:09 |
| *** jlanoux has quit IRC | 17:09 | |
| *** PaulMurray has quit IRC | 17:10 | |
| artom | mriedem, I'm on it, bon appetit! | 17:10 |
| dansmith | man, I need another monitor | 17:10 |
| artom | dansmith, or a tiling window manager :) | 17:11 |
| dansmith | artom: already tiling | 17:11 |
| artom | dansmith, you need less, err, work then :P | 17:11 |
| *** ducttape_ has joined #openstack-nova | 17:13 | |
| cfriesen_ | I think a curved 40+ inch 4K monitor would be nice... | 17:13 |
| *** ducttape_ has quit IRC | 17:13 | |
| artom | Or just develop in VR | 17:13 |
| dansmith | I have three 27s and a 24 already.. not sure a single 4k would be an upgrade | 17:13 |
| cfriesen_ | I'm jealous...I've only got two | 17:14 |
| alaski | a VR headset is the answer | 17:14 |
| *** ducttape_ has joined #openstack-nova | 17:15 | |
| alaski | ahh, artom already said that | 17:15 |
| *** gjayavelu has joined #openstack-nova | 17:15 | |
| *** gjayavelu has quit IRC | 17:15 | |
| *** kaisers has joined #openstack-nova | 17:17 | |
| dansmith | jlvillal: it's upgrading now | 17:17 |
| dansmith | jlvillal: it failed before in the post-upgrade tempest run I assume? | 17:18 |
| *** ihrachys has quit IRC | 17:18 | |
| artom | alaski, jinx! | 17:18 |
| *** shaohe_feng has quit IRC | 17:18 | |
| *** shaohe_feng has joined #openstack-nova | 17:19 | |
| jlvillal | dansmith, Are you watching the telnet stream like me? :) | 17:19 |
| dansmith | jlvillal: sorry :) | 17:19 |
| jlvillal | I need to look at the previous runs | 17:19 |
| *** lucas__ has joined #openstack-nova | 17:19 | |
| *** lucas__ has quit IRC | 17:20 | |
| jlvillal | dansmith, Looking at a previous failure it died farther down the line | 17:21 |
| dansmith | okay | 17:21 |
| dansmith | figured | 17:21 |
| *** zenoway has quit IRC | 17:22 | |
| jaypipes | johnthetubaguy: my hat is off to you for your patience untangling this unholy mess of unit test feces in neutronv2.py | 17:22 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: WIP: policy: clean-up https://review.openstack.org/333925 | 17:22 |
| *** zenoway has joined #openstack-nova | 17:23 | |
| *** tblakeslee has quit IRC | 17:23 | |
| jaypipes | johnthetubaguy: I know the pain, having just gone through a lot of that with the existing resource tracker unit tests. | 17:23 |
| *** tbachman has joined #openstack-nova | 17:24 | |
| *** _ducttape_ has joined #openstack-nova | 17:24 | |
| *** yamahata has quit IRC | 17:24 | |
| *** ducttape_ has quit IRC | 17:25 | |
| *** zzelle_ has joined #openstack-nova | 17:25 | |
| *** mgould is now known as mgould|afk | 17:27 | |
| *** kaisers has quit IRC | 17:27 | |
| *** chandankumar has joined #openstack-nova | 17:27 | |
| *** zenoway has quit IRC | 17:27 | |
| *** cdelatte has quit IRC | 17:27 | |
| dansmith | jaypipes: feces doesn't really cut it for me | 17:28 |
| dansmith | it's shit. | 17:28 |
| mriedem | tjones: on https://review.openstack.org/#/c/196171/ | 17:28 |
| jaypipes | dansmith: heh | 17:28 |
| mriedem | tjones: that's only supported for attaching an interface to an existing server? we can't apply those limits when booting the server? | 17:28 |
| *** shaohe_feng has quit IRC | 17:28 | |
| jaypipes | dansmith: I believe I call it poo-pile in the review. | 17:29 |
| *** shaohe_feng has joined #openstack-nova | 17:29 | |
| *** salv-orlando has joined #openstack-nova | 17:29 | |
| *** imcsk8_ is now known as imcsk8|doctor | 17:30 | |
| dansmith | jaypipes: you said feces above | 17:30 |
| jaypipes | dansmith: oh, I know... wordsmithing, you know, makes me happy. | 17:30 |
| *** abalutoiu has joined #openstack-nova | 17:30 | |
| dansmith | heh | 17:30 |
| *** tonytan4ever has joined #openstack-nova | 17:31 | |
| *** klkumar has quit IRC | 17:32 | |
| openstackgerrit | Artom Lifshitz proposed openstack/nova: Device tagging metadata API support https://review.openstack.org/304511 | 17:32 |
| dansmith | artom: you don't need to remove the pluralness in the test | 17:32 |
| dansmith | I was just making a comment | 17:32 |
| dansmith | makes me feel important | 17:32 |
| artom | dansmith, well, I was respining anyways, might as well | 17:33 |
| artom | I agree it's weird to have fakes not correspond to reality | 17:33 |
| *** gjayavelu has joined #openstack-nova | 17:33 | |
| artom | mriedem, ^^^ when you're done pheeding | 17:33 |
| openstackgerrit | Lee Yarwood proposed openstack/nova: libvirt: Use the recreated config disk during a rescue https://review.openstack.org/335884 | 17:33 |
| openstackgerrit | Moshe Levi proposed openstack/nova: Update available resources before confirm stage https://review.openstack.org/327356 | 17:33 |
| *** salv-orlando has quit IRC | 17:34 | |
| mriedem | tjones: also on https://review.openstack.org/#/c/196171/ the nsx ci results from 6/7 are gone | 17:35 |
| mriedem | should be able to keep ci logs around for longer than 3 weeks | 17:35 |
| *** penick_ has joined #openstack-nova | 17:35 | |
| dansmith | jlvillal: got a bunch of server tests that passed | 17:36 |
| *** penick has quit IRC | 17:36 | |
| *** penick_ is now known as penick | 17:36 | |
| *** TravT has quit IRC | 17:39 | |
| jlvillal | dansmith, Yep. I'm still watching. I gotta have high hopes | 17:39 |
| *** shaohe_feng has quit IRC | 17:39 | |
| dansmith | jlvillal: mriedem: yeah passed one we failed before this fix | 17:39 |
| dansmith | 2016-06-30 17:32:57.617360 | {0} tempest.api.compute.servers.test_server_addresses.ServerAddressesTestJSON.test_list_server_addresses [0.085498s] ... ok | 17:39 |
| *** shaohe_feng has joined #openstack-nova | 17:39 | |
| *** sudipto has joined #openstack-nova | 17:40 | |
| dansmith | I have the cleaned up patch ready to push when this finishes | 17:40 |
| lbeliveau | mriedem: looks like there is no tempest test for migration with sriov wrt network connectivity (looking at https://github.com/openstack/tempest/blob/master/tempest/scenario/test_network_advanced_server_ops.py) | 17:40 |
| lbeliveau | mriedem: I'm surprised, am I missing something ? | 17:40 |
| *** tonytan4ever has quit IRC | 17:41 | |
| gjayavelu | mriedem: tjones: the results were probably gone due to the change in jenkins server. let me check if we can pull those results back | 17:41 |
| mriedem | lbeliveau: hmm, i thought something happened in tempest with configuring the port | 17:41 |
| mriedem | for like direct or macvtap | 17:41 |
| *** tonytan4ever has joined #openstack-nova | 17:41 | |
| mriedem | lbeliveau: yeah port_vnic_type | 17:42 |
| jlvillal | dansmith, I assume you saw the py27 failure in the nova side | 17:42 |
| mriedem | lbeliveau: but port_vnic_type isn't used in test_network_advanced_server_ops.py | 17:42 |
| dansmith | jlvillal: yeah, hadn't done any test work on that patch when I submitted that | 17:42 |
| mriedem | gjayavelu: can you answer the question of why we don't apply vif limits when creating the server? | 17:42 |
| dansmith | jlvillal: got that all fixed locally, just can't push or it will reset your test | 17:42 |
| jlvillal | dansmith, Thanks | 17:42 |
| *** kaisers has joined #openstack-nova | 17:43 | |
| lbeliveau | mriedem: I see tests for resize, reboot, etc, but nothing for migration there, I guess we'll need to add it | 17:43 |
| mriedem | gjayavelu: i rechecked the change otherwise | 17:43 |
| mriedem | lbeliveau: there are no scenario tests in tempest for migration with 2 nodes | 17:43 |
| mriedem | i guess resize would be fine if you had 2 nodes | 17:43 |
| lbeliveau | mriedem: thanks for confirming | 17:44 |
| *** paul-carlton has quit IRC | 17:44 | |
| mriedem | lbeliveau: but i think that port_vnic_type is the thing that tempest uses for sriov testing | 17:44 |
| *** sdake has quit IRC | 17:45 | |
| gjayavelu | mriedem: we do apply the limits when creating the vifs https://github.com/openstack/nova/blob/master/nova/virt/vmwareapi/vm_util.py#L248 | 17:45 |
| gjayavelu | mriedem: the current patch is when adding new interfaces | 17:46 |
| mriedem | lbeliveau: actually the base class's create_server method uses vnic_type | 17:46 |
| *** tbachman has quit IRC | 17:46 | |
| lbeliveau | mriedem: yeah https://github.com/openstack/tempest/blob/master/tempest/scenario/manager.py#L180 | 17:47 |
| mriedem | gjayavelu: ok so that made it into mitaka | 17:47 |
| gjayavelu | mriedem: that's right | 17:47 |
| mriedem | lbeliveau: so do we have any ci jobs that run with vnic type direct or macvtap that runs the resize test in test_network_advanced_server_ops.py ? | 17:48 |
| lbeliveau | mriedem: yes the mellanox CI is executing test_server_connectivity_resize() | 17:48 |
| mriedem | lbeliveau: so it does http://13.69.151.247/73/242573/36/check-nova/Nova-ML2-Sriov/67c190c/testr_results.html.gz | 17:49 |
| *** shaohe_feng has quit IRC | 17:49 | |
| mriedem | http://13.69.151.247/73/242573/36/check-nova/Nova-ML2-Sriov/67c190c/tempest.conf.gz | 17:49 |
| *** karimb has quit IRC | 17:49 | |
| mriedem | port_vnic_type = direct | 17:49 |
| *** zz_dimtruck is now known as dimtruck | 17:49 | |
| *** tbachman has joined #openstack-nova | 17:49 | |
| lbeliveau | mridem: I just submitted a new patchset (it wasn't updating neutron ports if the host hasn't changed), I tested it in my env, we'll know soon from CI | 17:50 |
| mriedem | and macvtap here http://13.69.151.247/73/242573/34/check-nova/Nova-MACVTAP-ML2-Sriov/3b695d8/testr_results.html.gz | 17:50 |
| *** shaohe_feng has joined #openstack-nova | 17:50 | |
| *** yamahata has joined #openstack-nova | 17:53 | |
| dansmith | mriedem: jlvillal Zuul] Job complete, result: SUCCESS | 17:54 |
| openstackgerrit | Dan Smith proposed openstack/nova: Make VIF.address unique with port id for neutron https://review.openstack.org/336069 | 17:55 |
| *** paul-carlton has joined #openstack-nova | 17:55 | |
| *** tblakeslee has joined #openstack-nova | 17:55 | |
| jlvillal | dansmith, Woo hoo! :) | 17:56 |
| dansmith | yeah | 17:57 |
| *** paul-carlton has quit IRC | 17:57 | |
| *** paul-carlton2 has joined #openstack-nova | 17:57 | |
| *** salv-orlando has joined #openstack-nova | 17:57 | |
| dansmith | having an ironic person in town adds a level of in-person violence potential that helps to motivate :) | 17:58 |
| mriedem | artom: vladikr: dansmith: congrats https://review.openstack.org/#/c/304511/ | 17:58 |
| dansmith | mriedem: thanks a lot for all your work on it | 17:59 |
| *** dimtruck is now known as zz_dimtruck | 17:59 | |
| artom | mriedem, indeed, many thanks | 17:59 |
| *** shaohe_feng has quit IRC | 17:59 | |
| vladikr | dansmith, mriedem, Thanks :) | 17:59 |
| *** shaohe_feng has joined #openstack-nova | 18:00 | |
| mriedem | are there other virt drivers for device tagging that we need to get on quick? | 18:01 |
| artom | mriedem, claudio has his stuff here: https://review.openstack.org/#/c/331889/ | 18:01 |
| dansmith | mriedem: yeah, the hyper-v one had pending review comments last I checked | 18:01 |
| artom | mriedem, it looks fine to me, but if vladikr and dansmith took a look I'd be more confident | 18:01 |
| artom | The xenapi is here: https://review.openstack.org/#/c/333781/ but I don't think that's mergeable | 18:02 |
| artom | Also, can we make our Tempest test run against a hyper-v virt driver? | 18:02 |
| mriedem | hyperv doesn't support tagging for block devices? | 18:02 |
| artom | Or are we too pressed for time for this | 18:02 |
| mriedem | artom: the hyper-v ci would have to do that | 18:02 |
| jlvillal | dansmith, Heh. | 18:03 |
| *** zz_dimtruck is now known as dimtruck | 18:03 | |
| mriedem | oh nvm | 18:03 |
| *** sameo has quit IRC | 18:03 | |
| dansmith | mriedem: you mean vifs? | 18:03 |
| mriedem | i found my problem | 18:04 |
| *** salv-orlando has quit IRC | 18:05 | |
| *** salv-orlando has joined #openstack-nova | 18:05 | |
| *** tiendc has quit IRC | 18:05 | |
| sudipto | mriedem, wondering why pypi is preferred over a distro package? Just for my own understanding... | 18:06 |
| mriedem | sudipto: so that people don't have to rely on a distro and can pip install the code | 18:06 |
| mriedem | but you'll still need the pcpd package | 18:06 |
| sudipto | mriedem, yeah pcp package would still have to be distro specific... | 18:07 |
| mriedem | there might be other reasons | 18:07 |
| dansmith | artom: I guess next step is getting the tempest test in tree | 18:07 |
| artom | dansmith, yep, and the client | 18:07 |
| dansmith | yeah | 18:08 |
| artom | Wait, the FF doesn't also apply to Tempest, does it? | 18:08 |
| fche | mriedem, fwiw relying on a distro is a good thing if the distro is tracking and pypi is abandoned | 18:08 |
| dansmith | artom: nor the client, AFAIK | 18:08 |
| mriedem | client freeze is like 9/2 | 18:08 |
| *** chandankumar has quit IRC | 18:08 | |
| mriedem | https://wiki.openstack.org/wiki/Nova/Newton_Release_Schedule | 18:09 |
| mriedem | final client release | 18:09 |
| artom | dansmith, good, otherwise my assumptions and time management would be seriously upended. | 18:09 |
| *** mvk has joined #openstack-nova | 18:09 | |
| *** shaohe_feng has quit IRC | 18:09 | |
| *** shaohe_feng has joined #openstack-nova | 18:10 | |
| *** lpetrut has quit IRC | 18:12 | |
| *** harshad has joined #openstack-nova | 18:12 | |
| *** Sukhdev has joined #openstack-nova | 18:15 | |
| *** thumpba has quit IRC | 18:18 | |
| *** harshad has quit IRC | 18:18 | |
| *** shaohe_feng has quit IRC | 18:20 | |
| *** shaohe_feng has joined #openstack-nova | 18:20 | |
| *** mdrabe has joined #openstack-nova | 18:21 | |
| mriedem | claudiub: the hyper-v ci runs against nova-network right? | 18:21 |
| dansmith | does it/ | 18:22 |
| dansmith | I thought they required neutron | 18:22 |
| *** mfedosin has quit IRC | 18:24 | |
| mriedem | looks like it is neutron http://64.119.130.115/nova/331889/6/devstack_logs/ | 18:25 |
| mriedem | yeah that would make sense given their hyperv neutron agent | 18:25 |
| mriedem | :) | 18:25 |
| dansmith | yeah | 18:25 |
| mriedem | alright, so on https://review.openstack.org/#/c/331889/ i think i want to hold that up until artom's tempest change lands and the hyper-v ci can test with it | 18:25 |
| mriedem | unless they can pull that tempest test in as a dependency to their ci for testing this patch | 18:26 |
| mriedem | like the vz guys did | 18:26 |
| dansmith | mriedem: but okay with it landing after FF? | 18:26 |
| mriedem | maybe | 18:26 |
| dansmith | I really wanted to see that they could implement it, which they did, so I'm good, but would be nice to show up with more than one driver actually supporting it | 18:27 |
| mriedem | yeah i agree | 18:27 |
| *** dharinic__ has quit IRC | 18:27 | |
| mriedem | maybe we make that decision at the midcycle | 18:27 |
| mriedem | if it's ready by then | 18:27 |
| mriedem | claudiub: will be there | 18:28 |
| *** Sukhdev has quit IRC | 18:28 | |
| artom | mriedem, who's in charge of the hyper-v CI? claudiub, maybe you know? | 18:28 |
| artom | I can always poke them with my diplomacy stick | 18:28 |
| dansmith | artom: getting the tempest test in shape ASAP makes it easier | 18:29 |
| ociuhandu | artom: hello, we're in charge of the hyper-v ci | 18:29 |
| *** thumpba has joined #openstack-nova | 18:29 | |
| artom | ociuhandu, hello! We have the device tagging feature for hyper-v here: https://review.openstack.org/#/c/331889/ | 18:30 |
| openstackgerrit | Sujitha proposed openstack/nova: Transform instance.pause notifications https://review.openstack.org/329141 | 18:30 |
| *** shaohe_feng has quit IRC | 18:30 | |
| artom | ociuhandu, and the Tempest test for it here: https://review.openstack.org/#/c/305120/ | 18:30 |
| artom | ociuhandu, the problem is the Tempest test hasn't merged yet | 18:30 |
| artom | ociuhandu, is it at all feasible to run it against that change? | 18:30 |
| claudiub | mriedem: hi, hyper-v ci runs via neutron-hyperv-agent. | 18:31 |
| *** shaohe_feng has joined #openstack-nova | 18:31 | |
| claudiub | artom: I suppose the test will also require all the other patches in the stack, right? | 18:32 |
| ociuhandu | artom: the easiest thing we could to is launch a "debug run" on the patch and fetch the expected tempest and test against that. The only thing is that this will be done "manually" and the logs will be placed in a debug location, but I don't think it should be an issue | 18:32 |
| artom | claudiub, they've all merged at this point | 18:32 |
| dansmith | ociuhandu: I think that would be okay, if it's not too much trouble | 18:33 |
| *** sridharg has joined #openstack-nova | 18:33 | |
| *** GB21 has quit IRC | 18:33 | |
| artom | claudiub, I think the last one is going through the gate, but assuming that doens't explode it's all in-tree | 18:33 |
| mriedem | ociuhandu: claudiub: the other thing to note is the ci job that's testing this right now is gate-tempest-dsvm-neutron-full-ssh | 18:33 |
| mriedem | which is a non-voting job | 18:33 |
| ociuhandu | we can launch debug runs that stop just before running tempest so we can fetch at that time the correct tempest | 18:33 |
| mriedem | i'm not sure what configuration differences you might have, but it's got ssh validation enabled in tempest.conf | 18:33 |
| *** thumpba has quit IRC | 18:34 | |
| artom | (ssh validation which is absolutely required for this test) | 18:34 |
| mriedem | yeah, the skip checks are here https://review.openstack.org/#/c/305120/11/tempest/api/compute/servers/test_device_tagging.py@39 | 18:34 |
| dansmith | artom: last one is still in check, but headed to the gate in a few minutes | 18:34 |
| mriedem | headed for a heartbreak | 18:35 |
| *** thumpba has joined #openstack-nova | 18:35 | |
| *** thumpba has quit IRC | 18:35 | |
| *** thumpba has joined #openstack-nova | 18:35 | |
| mriedem | looking at http://64.119.130.115/nova/331889/6/devstack_config/tempest.conf.gz | 18:36 |
| claudiub | mriedem: yeah, that's not a problem. we use a cirros image anyways, so ssh won't be a problem. | 18:36 |
| mriedem | run_validation=False | 18:37 |
| mriedem | ok | 18:37 |
| mriedem | that would have to be updated | 18:37 |
| mriedem | it doesn't disable config drive or the metadata service, and those default to True in tempest.conf, so assuming they should work in the hyperv ci | 18:37 |
| mriedem | enabled_apis = osapi_compute,metadata | 18:37 |
| mriedem | ooo yeah | 18:37 |
| artom | And I think worst case checking only one of those two is better than nothing, and still gives us some confidence | 18:37 |
| mriedem | so change run_validation=True and you should be good to go | 18:38 |
| artom | Makes me think I should split those into two different tests maybe? | 18:38 |
| mriedem | artom: the test won't run if they aren't both enabled | 18:38 |
| mriedem | artom: it would be nice to test at least one of them | 18:38 |
| mriedem | since some jobs in the gate don't run the meta service | 18:38 |
| artom | mriedem, so maybe I can post a WIP of the Tempest test that splits it into two | 18:38 |
| artom | One for config drive, one for API | 18:38 |
| mriedem | one thought is you could run the validation per each that's enabled, and if neither are enabled in skip_checks then you skip | 18:38 |
| mriedem | but that way you wouldn't have to duplicate the setup | 18:39 |
| artom | Ah, yeah, smarter | 18:39 |
| openstackgerrit | Ryan Rossiter proposed openstack/nova: WIP: Shared state host manager https://review.openstack.org/332463 | 18:39 |
| mriedem | mtreinish: how do you feel about this https://review.openstack.org/#/c/305120/11/tempest/api/compute/servers/test_device_tagging.py@46 ? | 18:40 |
| *** shaohe_feng has quit IRC | 18:40 | |
| *** shaohe_feng has joined #openstack-nova | 18:40 | |
| *** paul-carlton2 has quit IRC | 18:42 | |
| *** tbachman has quit IRC | 18:43 | |
| *** Sukhdev has joined #openstack-nova | 18:47 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/333181 | 18:47 |
| *** ociuhandu has quit IRC | 18:48 | |
| mriedem | dansmith: unit test for the vif object change in https://review.openstack.org/#/c/336069/ ? for the actual conversion from the db record | 18:49 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-novaclient: Updated from global requirements https://review.openstack.org/336235 | 18:49 |
| dansmith | mriedem: ah yeah sorry | 18:49 |
| *** shaohe_feng has quit IRC | 18:50 | |
| *** Sukhdev has quit IRC | 18:51 | |
| *** mdrabe has quit IRC | 18:51 | |
| *** mdrabe_ has joined #openstack-nova | 18:51 | |
| *** shaohe_feng has joined #openstack-nova | 18:51 | |
| *** TravT has joined #openstack-nova | 18:52 | |
| oomichi | gmann: about https://review.openstack.org/#/c/328157 yeah, I have the same opinion which requires Depends-On to ironic side patch | 18:52 |
| *** mdrabe_ has quit IRC | 18:53 | |
| *** mdrabe_ has joined #openstack-nova | 18:54 | |
| *** TravT has quit IRC | 18:54 | |
| *** mdrabe_ has quit IRC | 18:54 | |
| *** mdrabe_ has joined #openstack-nova | 18:55 | |
| *** TravT has joined #openstack-nova | 18:55 | |
| openstackgerrit | Dan Smith proposed openstack/nova: Make VIF.address unique with port id for neutron https://review.openstack.org/336069 | 18:56 |
| dansmith | mriedem: ^ | 18:56 |
| mriedem | styyyyle not professional | 18:56 |
| dansmith | come on, after that commit message I get this one. | 18:57 |
| mriedem | namespaced mac addresses, you want'em | 18:57 |
| *** mdrabe_ has quit IRC | 18:57 | |
| openstackgerrit | Giridhar Jayavelu proposed openstack/nova: VMware: configure limits during resize operation https://review.openstack.org/204219 | 18:58 |
| openstackgerrit | Giridhar Jayavelu proposed openstack/nova: VMware: create vif with resource limitations https://review.openstack.org/196171 | 18:58 |
| *** mdrabe_ has joined #openstack-nova | 18:58 | |
| oomichi | mriedem: about https://review.openstack.org/#/c/304510 That is already merged, and I cannot find any problems about the schema validation. | 18:59 |
| oomichi | mriedem: btw, the schema is hard to read for humen, due to copy.deepcopy and changing original schema. | 18:59 |
| mriedem | dansmith: comment in that change | 18:59 |
| *** mdrabe_ has quit IRC | 19:00 | |
| oomichi | mriedem: this thing happens on tempest side and gmann will expand these schemas for the readability, how about doing that on nova side? | 19:00 |
| *** mdrabe__ has joined #openstack-nova | 19:00 | |
| *** shaohe_feng has quit IRC | 19:01 | |
| mriedem | oomichi: i'd probably need to see an example | 19:01 |
| mriedem | but i agree the copying gets weird | 19:01 |
| mriedem | the nice thing about the copy is we make sure we don't lose anything from the prevoius microversion | 19:01 |
| *** shaohe_feng has joined #openstack-nova | 19:01 | |
| oomichi | mriedem: ok, maybe tempest changes will be good examples before doing that on nova side | 19:02 |
| oomichi | mriedem: yeah, I can see the point also | 19:02 |
| openstackgerrit | Dan Smith proposed openstack/nova: Make VIF.address unique with port id for neutron https://review.openstack.org/336069 | 19:04 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/333181 | 19:07 |
| openstackgerrit | Ryan Rossiter proposed openstack/nova: Use ovo in test_obj_make_compatible() https://review.openstack.org/272299 | 19:09 |
| *** dimtruck is now known as zz_dimtruck | 19:11 | |
| *** shaohe_feng has quit IRC | 19:11 | |
| *** shaohe_feng has joined #openstack-nova | 19:12 | |
| *** sridharg has quit IRC | 19:13 | |
| mriedem | alaski: got a lonely +2 party of one going on here https://review.openstack.org/#/c/336069/ | 19:14 |
| *** amotoki has joined #openstack-nova | 19:14 | |
| *** tbachman has joined #openstack-nova | 19:15 | |
| *** jed56 has quit IRC | 19:15 | |
| * alaski joins the party | 19:15 | |
| mriedem | melwitt: johnthetubaguy: replied to https://review.openstack.org/#/c/331236/ | 19:15 |
| mriedem | melwitt: johnthetubaguy: they are different error conditions | 19:15 |
| *** lpetrut has joined #openstack-nova | 19:16 | |
| pumaranikar | hi, can anyone confirm if we are still using os-volumes_boot API as a part of V2.1 | 19:17 |
| mriedem | pumaranikar: we are | 19:17 |
| melwitt | mriedem: I meant why not make each string a message for the exception object, so it gets propagated up that way. or you're saying there isn't any logging of the exception message later on? | 19:17 |
| mriedem | melwitt: oh that's what you're saying... | 19:18 |
| mriedem | i'd have to check on if anything logs it later | 19:18 |
| mriedem | pumaranikar: it's an extension on the servers API | 19:18 |
| mriedem | so it's a bit weird | 19:18 |
| mriedem | pumaranikar: are you asking about the bug for no functional api sample test for that api? | 19:19 |
| pumaranikar | mriedem, thanks :). yes | 19:19 |
| mriedem | pumaranikar: are you new? | 19:19 |
| mriedem | like, new to nova | 19:20 |
| pumaranikar | mriedem, yes. I can not find that extension in API extensions doc | 19:20 |
| mriedem | pumaranikar: i would recommend avoiding that one | 19:20 |
| mriedem | sdague or alex_xu_ or gmann might be able to help with it, but it's probably something terrible | 19:20 |
| *** shaohe_feng has quit IRC | 19:21 | |
| pumaranikar | mriedem, ok. | 19:21 |
| *** shaohe_feng has joined #openstack-nova | 19:22 | |
| sudipto | is it necessary that whatever is specified in g-r has to be pip installed? I am reading through this : https://github.com/openstack/requirements and i am not sure if it's a necessity? | 19:22 |
| sudipto | mriedem, ^ | 19:22 |
| mriedem | sudipto: is there a specific section you're looking at | 19:24 |
| mriedem | ? | 19:24 |
| sudipto | mriedem, "For new requirements" that one | 19:24 |
| *** mdrabe__ has quit IRC | 19:25 | |
| *** sacharya_ has quit IRC | 19:26 | |
| *** sacharya has joined #openstack-nova | 19:26 | |
| mriedem | sudipto: i'm not makinga connection between that and your question about being pip installed | 19:29 |
| *** dharinic_ has joined #openstack-nova | 19:29 | |
| fche | I believe the question is whether being listed in the requirements.txt file means that each python package must be installed via pip | 19:29 |
| fche | as opposed to being found on the distro | 19:29 |
| sudipto | mriedem, basically pcp satisfies the "new requirements" criteria - however, i wanted to know if the g-r means it has to be pip installed. | 19:29 |
| artom | mriedem, https://review.openstack.org/305120 hail mary respin, I think my local devstack borked, running the test can't reach the booted instance | 19:31 |
| *** shaohe_feng has quit IRC | 19:31 | |
| mriedem | sudipto: if it were in nova's requirements.txt then yes it would have to be in g-r and pip installable, but since it's an optional dependency it won't be in requirements.txt. now it could be in test-requirements.txt, like oslo.vmware | 19:32 |
| *** mdrabe has joined #openstack-nova | 19:32 | |
| mriedem | so we could actually unit test with the python library | 19:32 |
| *** shaohe_feng has joined #openstack-nova | 19:32 | |
| mriedem | given the 0.1 and pre-alpha status of https://pypi.python.org/pypi/pcp/0.1 i'm guessing that would be a hard sell to get it into g-r | 19:32 |
| sudipto | mriedem, yeah i am not looking to sell the pypi package at all... | 19:33 |
| mriedem | heh i know :) | 19:33 |
| *** bvanhav has quit IRC | 19:33 | |
| mriedem | hmm https://github.com/performancecopilot/pcp/blob/master/src/python/setup.py#L41 | 19:35 |
| *** sheel has quit IRC | 19:35 | |
| mriedem | the python bindings in pcp itself are marked as stable | 19:35 |
| mtreinish | mriedem: that seems kinda weird. Do all the tests in the class require both config drive and the metadata service? | 19:35 |
| mriedem | mtreinish: no | 19:36 |
| mriedem | mtreinish: there are 2 validation calls, one for metadata and one for config drive | 19:36 |
| mriedem | i was suggesitng we run those conditionally | 19:36 |
| mriedem | mtreinish: but skip early if neither of those are enabled | 19:36 |
| mtreinish | then yeah, that's wrong | 19:36 |
| mriedem | i thought i'd seen other scenario tests like that | 19:36 |
| mtreinish | why not just use skip decorators with the conf value | 19:37 |
| mriedem | mtreinish: or that | 19:37 |
| mriedem | sudipto: any idea if this supports python 3? | 19:37 |
| sudipto | mriedem, that shows it's stable no? | 19:37 |
| *** tonytan4ever has quit IRC | 19:37 | |
| sudipto | mriedem, yeah it does. fche ? | 19:37 |
| mriedem | ah now i know who fche is | 19:38 |
| fche | just some guy :) | 19:38 |
| mriedem | #3 contributor is pretty good | 19:39 |
| fche | anyway it's 'stable' and changing (growing) periodically too. designed to be used. | 19:39 |
| mriedem | fche: ok, i guess pypi doesn't represent that | 19:39 |
| mriedem | but it didn't for libvirt-python for a long time either | 19:39 |
| *** aloga_ has joined #openstack-nova | 19:39 | |
| fche | yeah. the pypi copy was posted long long ago, and that author is NORDO as they say | 19:39 |
| openstackgerrit | Giridhar Jayavelu proposed openstack/nova: VMware: configure limits during resize operation https://review.openstack.org/204219 | 19:40 |
| openstackgerrit | Giridhar Jayavelu proposed openstack/nova: VMware: create vif with resource limitations https://review.openstack.org/196171 | 19:40 |
| *** hoonetorg has quit IRC | 19:40 | |
| mriedem | sudipto: ok so i'm going to drop the pypi / g-r thing | 19:41 |
| openstackgerrit | Sivasathurappan Radhakrishnan proposed openstack/nova: Improve help text for vmwareapi_opts https://review.openstack.org/332503 | 19:41 |
| sudipto | mriedem, ok thanks! However i think i have already missed the bus for this release... | 19:41 |
| *** shaohe_feng has quit IRC | 19:42 | |
| *** shaohe_feng has joined #openstack-nova | 19:42 | |
| mriedem | i'd like to get danpb's thoughts on libvirt providing this info for the host, but that's definitely something way out and not available for awhile, so depending on that isn't great | 19:43 |
| mriedem | i'm not sure i want to rush this in today either | 19:43 |
| fche | libvirt info is great, but there is a big set of other stats pcp can offer, re. containers, rest-of-the-system, etc. two separate sweet spots, two separate tools | 19:45 |
| dansmith | fche: the bar for depending on another external thing is pretty high for me | 19:45 |
| dansmith | fche: like, the data we get has to be really compelling and even what is proposed now does not seem to meet that bar, IMHO | 19:46 |
| fche | yeah, understood. | 19:46 |
| fche | I suspect sudipto et al. are going to look around for a more interesting sampling of data from pcpland | 19:46 |
| sudipto | dansmith, agreed...mem bw is probably not compelling enough atm | 19:46 |
| *** daemontool_ has quit IRC | 19:46 | |
| *** jaugustine has joined #openstack-nova | 19:47 | |
| fche | at least sudipto's pcp_util.py infrastructure is common in the sense that any of the thousands of other metrics can be processed through that bit of code. | 19:47 |
| sudipto | however, this is one of the vehicles i chose to get into more interesting metrics in the future... | 19:47 |
| *** aloga_ has quit IRC | 19:48 | |
| fche | dansmith, mriedem, I assume you haven't had a chance to look at the full smorgasbord - but are there system perf areas you think openstack needs more numbers about? | 19:48 |
| fche | generally speaking? | 19:49 |
| mriedem | idk, jaypipes might have an opinion - this is really specific for scheduling | 19:49 |
| mriedem | i honestly don't know how many deployments of nova are relying on the metrics weigher | 19:49 |
| mriedem | to get that fine-grained level of scheduling | 19:50 |
| *** tonytan4ever has joined #openstack-nova | 19:50 | |
| *** abc has joined #openstack-nova | 19:50 | |
| mriedem | this is also just another thing that will be special and we won't test | 19:50 |
| *** hoonetorg has joined #openstack-nova | 19:51 | |
| fche | ceilometer doesn't need a boost either? | 19:52 |
| *** shaohe_feng has quit IRC | 19:52 | |
| *** shaohe_feng has joined #openstack-nova | 19:53 | |
| sudipto | i would imagine it's beyond just scheduling... however i am not sure if that means it's not of importance to nova that much... | 19:53 |
| mriedem | do these metrics get into anything that ceilometer consumes? | 19:54 |
| dansmith | fche: no, I don't really | 19:54 |
| mriedem | we don't have these going into events or notifications that i'm aware of | 19:55 |
| mriedem | do we emit anything for compute node stats? | 19:56 |
| mriedem | jaypipes: ^? | 19:56 |
| dansmith | not that I know of | 19:56 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: policy: clean-up https://review.openstack.org/333925 | 19:56 |
| mriedem | dansmith: ok i didn't think so | 19:56 |
| dansmith | ceilometer could, of course, scrape them from pcp or whatever itself | 19:56 |
| dansmith | which would make much more sense, | 19:56 |
| dansmith | than having nova proxy all that gorp for it | 19:56 |
| mriedem | ceilometer already goes directly to libvirt for some stuff | 19:57 |
| mriedem | anyway, that's sort of tangential | 19:57 |
| fche | it could work to start small & very optional, then see | 19:57 |
| gibi_ | mriedem: I don't think we have any notification about these stats either | 19:57 |
| *** yass has quit IRC | 19:57 | |
| dansmith | fche: we hate optional stuff | 19:58 |
| mriedem | heh :) | 19:58 |
| fche | ok mandate it then :-) | 19:58 |
| dansmith | mriedem: well, it's tre | 19:58 |
| mriedem | i know | 19:58 |
| mriedem | fche: we're doing that | 19:58 |
| mriedem | http://lists.openstack.org/pipermail/openstack-dev/2016-June/098556.html | 19:59 |
| mriedem | http://specs.openstack.org/openstack/nova-specs/specs/newton/approved/api-no-more-extensions.html | 19:59 |
| mriedem | unicorns are slowly being weeded out | 19:59 |
| *** dansmith is now known as unicorn_slayer | 19:59 | |
| fche | understood; it must be a difficult balance identifying unicorns vs. promising seedlings | 19:59 |
| unicorn_slayer | not really, it's pretty easy actually :) | 20:00 |
| fche | haha | 20:00 |
| mriedem | it usually comes down to at a minimum what we test in our CI | 20:00 |
| mriedem | and having more than one consumer | 20:01 |
| *** vtech has joined #openstack-nova | 20:01 | |
| *** shaohe_feng has quit IRC | 20:02 | |
| *** shaohe_feng has joined #openstack-nova | 20:03 | |
| *** fragatina has joined #openstack-nova | 20:03 | |
| sudipto | hmm and then you could get system wide utilization data of memory/cpu/disk etc which are not VM specific... | 20:04 |
| *** smatzek has quit IRC | 20:04 | |
| sudipto | even network information... | 20:05 |
| *** tonytan4ever has quit IRC | 20:05 | |
| *** tonytan4ever has joined #openstack-nova | 20:06 | |
| *** ihrachys has joined #openstack-nova | 20:07 | |
| *** _ducttape_ has quit IRC | 20:07 | |
| *** ducttape_ has joined #openstack-nova | 20:09 | |
| openstackgerrit | Clenimar Filemon proposed openstack/nova: Support Identity v3 when connecting to Ironic https://review.openstack.org/300154 | 20:10 |
| *** tonytan4ever has quit IRC | 20:12 | |
| *** jaugustine has quit IRC | 20:12 | |
| *** shaohe_feng has quit IRC | 20:12 | |
| *** Daisy has joined #openstack-nova | 20:13 | |
| *** tonytan4ever has joined #openstack-nova | 20:13 | |
| *** shaohe_feng has joined #openstack-nova | 20:13 | |
| *** unicorn_slayer is now known as dansmith | 20:15 | |
| *** Daisy has quit IRC | 20:17 | |
| *** tonytan4ever has quit IRC | 20:17 | |
| *** tonytan4ever has joined #openstack-nova | 20:18 | |
| *** shaohe_feng has quit IRC | 20:23 | |
| *** shaohe_feng has joined #openstack-nova | 20:24 | |
| *** gyee has joined #openstack-nova | 20:26 | |
| *** stewie925 has joined #openstack-nova | 20:28 | |
| *** Daisy has joined #openstack-nova | 20:29 | |
| *** cdelatte has joined #openstack-nova | 20:30 | |
| mriedem | least scary ascii croc ever https://review.openstack.org/#/c/335906/1 | 20:30 |
| openstackgerrit | Sivasathurappan Radhakrishnan proposed openstack/nova: Improve help text for cloudpipe https://review.openstack.org/331285 | 20:32 |
| * auggy is speechless | 20:33 | |
| *** shaohe_feng has quit IRC | 20:33 | |
| mriedem | flying cat? https://review.openstack.org/#/c/335907/1 | 20:33 |
| *** shaohe_feng has joined #openstack-nova | 20:33 | |
| mriedem | and idk wtf this is https://review.openstack.org/#/c/335903/1 | 20:33 |
| *** Daisy has quit IRC | 20:34 | |
| melwitt | omg that's amazing | 20:34 |
| dansmith | mriedem: faceplant | 20:34 |
| dansmith | feet in the air | 20:34 |
| dansmith | I think | 20:34 |
| *** cdelatte has quit IRC | 20:34 | |
| mriedem | heh maybe | 20:35 |
| mriedem | this is like fine art | 20:35 |
| openstackgerrit | Andy Hsiang proposed openstack/nova: Transform instance.power_off notifications https://review.openstack.org/330761 | 20:35 |
| mriedem | everyone interprets it differently | 20:35 |
| *** jaugustine has joined #openstack-nova | 20:35 | |
| *** sudipto has quit IRC | 20:36 | |
| *** aloga_ has joined #openstack-nova | 20:36 | |
| stewie925 | hello, I am working on the api-ref testing some nova commands - I was able to create a DNS domain by executing 'PUT /os-floating-ip-dns/{domains}' successfully | 20:36 |
| stewie925 | but when I tried to list the dns domain, it is returning empty domain_entries [ ] | 20:36 |
| melwitt | danpb, you're a genius | 20:37 |
| openstackgerrit | Chris Friesen proposed openstack/nova: Allow logging detailed info only on sched failure https://review.openstack.org/331306 | 20:37 |
| *** imcsk8|doctor is now known as imcsk8 | 20:39 | |
| dansmith | melwitt: your bar is pretty low | 20:39 |
| *** cdelatte has joined #openstack-nova | 20:39 | |
| melwitt | hmph! | 20:39 |
| *** thumpba has quit IRC | 20:40 | |
| dansmith | I can copy and paste too :) | 20:40 |
| cfriesen_ | someone should try one of those picture-to-ascii-art conversion sites | 20:41 |
| *** rcernin has joined #openstack-nova | 20:41 | |
| *** shaohe_feng has quit IRC | 20:43 | |
| dansmith | gimp used to have ascii-art export built in | 20:43 |
| *** shaohe_feng has joined #openstack-nova | 20:44 | |
| openstackgerrit | Peter Hamilton proposed openstack/nova: Add certificate signature verification https://review.openstack.org/331823 | 20:46 |
| *** TravT has quit IRC | 20:47 | |
| *** dharinic_ has quit IRC | 20:48 | |
| *** tblakeslee has quit IRC | 20:48 | |
| *** sree has joined #openstack-nova | 20:50 | |
| openstackgerrit | Julian proposed openstack/nova: Security issue refactor for wrap_exception notification https://review.openstack.org/336284 | 20:51 |
| *** tblakeslee has joined #openstack-nova | 20:51 | |
| *** shaohe_feng has quit IRC | 20:53 | |
| *** jaugustine has quit IRC | 20:54 | |
| *** shaohe_feng has joined #openstack-nova | 20:54 | |
| *** sree has quit IRC | 20:55 | |
| mriedem | melwitt: replied again in https://review.openstack.org/#/c/331236/ - basically i think the existing exception message is fine to go back up the stack | 20:55 |
| mriedem | w/o the network dict - which is really what i wanted to log | 20:56 |
| *** stewie925 has quit IRC | 20:56 | |
| mriedem | melwitt: but you got me looking at that error and we don't handle it as a BuildAbortException in the compute manager, and we probably should | 20:56 |
| mriedem | at least for neutron | 20:56 |
| mriedem | because rescheduling isn't going ot fix it | 20:56 |
| mriedem | *to | 20:56 |
| melwitt | mriedem: gotcha | 20:57 |
| *** syjulian has quit IRC | 20:58 | |
| *** ssurana has quit IRC | 21:00 | |
| *** rfolco has quit IRC | 21:01 | |
| *** eharney has quit IRC | 21:01 | |
| *** ssurana has joined #openstack-nova | 21:01 | |
| *** lpetrut has quit IRC | 21:02 | |
| *** raildo is now known as raildo-afk | 21:02 | |
| mriedem | dansmith: so with the libvirt and api changes done, i'd like to mark https://blueprints.launchpad.net/nova/+spec/virt-device-role-tagging as complete - any issues with that? the scheduler filter is a later optimization, and the hyper-v thing would be feature parity which may be ok to get a FFE for | 21:03 |
| *** shaohe_feng has quit IRC | 21:04 | |
| *** shaohe_feng has joined #openstack-nova | 21:04 | |
| mriedem | tdurakov: alaski: i'm guessing https://review.openstack.org/#/q/topic:bp/remove-compute-compute-communication+status:open isn't going to make it today right? | 21:08 |
| *** cdelatte has quit IRC | 21:09 | |
| openstackgerrit | Andrew Laski proposed openstack/nova: Add nova-manage commands for policy helpers https://review.openstack.org/335667 | 21:09 |
| mriedem | alaski: is ^ the same as https://blueprints.launchpad.net/nova/+spec/discoverable-policy-cli ? | 21:09 |
| *** gagehugo has quit IRC | 21:10 | |
| alaski | mriedem: tdurakov yeah, that's not going to make it today | 21:10 |
| claudiub | mriedem: ye... still working on that.. | 21:10 |
| alaski | mriedem: no, it's more along the lines of https://blueprints.launchpad.net/nova/+spec/policy-in-code | 21:10 |
| mriedem | alaski: ok | 21:10 |
| *** tblakeslee has quit IRC | 21:10 | |
| claudiub | mriedem: I've been working on the policy-in-code, and was planning to finish the cli stuff after finishing the policy-in-code stuff. | 21:11 |
| mriedem | ok | 21:11 |
| alaski | mriedem: that last patch is just a nice to have, not essential to the main point of the spec | 21:11 |
| *** jaugustine has joined #openstack-nova | 21:11 | |
| claudiub | speaking of which, this is amoung the last patches for policy-in-code: https://review.openstack.org/#/c/333925/ | 21:11 |
| *** tblakeslee has joined #openstack-nova | 21:12 | |
| mriedem | claudiub: ok, that gets a pass for FF | 21:12 |
| *** jaugustine has quit IRC | 21:13 | |
| *** ametts has quit IRC | 21:13 | |
| *** TravT has joined #openstack-nova | 21:14 | |
| *** shaohe_feng has quit IRC | 21:14 | |
| *** TravT has quit IRC | 21:14 | |
| *** shaohe_feng has joined #openstack-nova | 21:15 | |
| *** TravT has joined #openstack-nova | 21:15 | |
| diana_clarke | mdbooth: I'm sure you're gone, but here are my notes about ephemeral disks not resizing (before our patch series): https://notehub.org/o3mjz | 21:15 |
| diana_clarke | mdbooth: Had you already entered a bug for this? | 21:15 |
| *** akshai has quit IRC | 21:16 | |
| *** TravT has quit IRC | 21:16 | |
| *** TravT has joined #openstack-nova | 21:17 | |
| diana_clarke | mdbooth: Anyhoo, now that I'm done fighting with devstack, I'm finally going to start on that tempest test (but I'm off tomorrow). | 21:18 |
| openstackgerrit | Kris Lindgren proposed openstack/nova: create network metadata information as static even with dhcp https://review.openstack.org/336293 | 21:21 |
| *** tonytan4ever has quit IRC | 21:21 | |
| *** vishwanathj has quit IRC | 21:22 | |
| *** kylek3h has quit IRC | 21:24 | |
| *** edmondsw has quit IRC | 21:24 | |
| *** shaohe_feng has quit IRC | 21:24 | |
| *** shaohe_feng has joined #openstack-nova | 21:25 | |
| *** svenkat has quit IRC | 21:26 | |
| *** lucas-afk has quit IRC | 21:28 | |
| *** pkoniszewski_ has joined #openstack-nova | 21:28 | |
| *** lucasagomes has joined #openstack-nova | 21:28 | |
| *** rtheis has quit IRC | 21:29 | |
| *** tblakeslee has quit IRC | 21:29 | |
| *** pkoniszewski has quit IRC | 21:29 | |
| gjayavelu | mriedem: updated the unit tests for https://review.openstack.org/#/c/196171. please let me know if this is ok. Also, I heard the issue with posting the right link for vmware ci is fixed. waiting for the results | 21:30 |
| mriedem | gjayavelu: ok | 21:30 |
| mriedem | gjayavelu: looks ok, just waiting on those ci results | 21:32 |
| gjayavelu | mriedem: sure. thanks. | 21:33 |
| *** ctrath has quit IRC | 21:33 | |
| *** Shalmazar has joined #openstack-nova | 21:33 | |
| *** ayoung has quit IRC | 21:34 | |
| *** yamahata has quit IRC | 21:34 | |
| *** Madasi has quit IRC | 21:34 | |
| *** Shalmazar is now known as Madasi | 21:34 | |
| *** shaohe_feng has quit IRC | 21:34 | |
| *** shaohe_feng has joined #openstack-nova | 21:35 | |
| *** diogogmt has quit IRC | 21:36 | |
| *** jamesdenton has quit IRC | 21:37 | |
| *** rlrossit has quit IRC | 21:39 | |
| *** HenryG_ has joined #openstack-nova | 21:41 | |
| *** thorst_ has quit IRC | 21:42 | |
| *** jroll|dupe has joined #openstack-nova | 21:44 | |
| *** jroll|dupe has quit IRC | 21:44 | |
| *** jroll|dupe has joined #openstack-nova | 21:44 | |
| *** shaohe_feng has quit IRC | 21:45 | |
| *** hoonetorg has quit IRC | 21:45 | |
| *** BjoernT has quit IRC | 21:45 | |
| *** cleong has quit IRC | 21:45 | |
| *** hieulq has quit IRC | 21:45 | |
| *** jinquan has quit IRC | 21:45 | |
| *** mordred has quit IRC | 21:45 | |
| *** mirrorbox has quit IRC | 21:45 | |
| *** bradm_ has quit IRC | 21:45 | |
| *** zhs_s has quit IRC | 21:45 | |
| *** HenryG has quit IRC | 21:45 | |
| *** dulek has quit IRC | 21:45 | |
| *** mkoderer__ has quit IRC | 21:45 | |
| *** john51 has quit IRC | 21:45 | |
| *** Wenzhi has quit IRC | 21:45 | |
| *** _gryf has quit IRC | 21:45 | |
| *** jroll has quit IRC | 21:45 | |
| *** ianw has quit IRC | 21:45 | |
| *** hogepodge has quit IRC | 21:45 | |
| *** hamzy has quit IRC | 21:45 | |
| *** tpatzig has quit IRC | 21:45 | |
| *** mkoderer has quit IRC | 21:45 | |
| *** gothicmindfood has quit IRC | 21:45 | |
| *** jroll|dupe is now known as jroll | 21:45 | |
| *** shaohe_feng has joined #openstack-nova | 21:45 | |
| *** yamahata has joined #openstack-nova | 21:47 | |
| *** jamie_h has quit IRC | 21:47 | |
| *** fragatina has quit IRC | 21:47 | |
| *** mdrabe has quit IRC | 21:48 | |
| *** cdelatte has joined #openstack-nova | 21:51 | |
| openstackgerrit | Clenimar Filemon proposed openstack/nova: Support Identity v3 when connecting to Ironic https://review.openstack.org/300154 | 21:52 |
| *** HenryG_ is now known as HenryG | 21:52 | |
| *** sdake_ has joined #openstack-nova | 21:53 | |
| *** sdake_ has quit IRC | 21:53 | |
| *** sdake_ has joined #openstack-nova | 21:53 | |
| *** stewie925 has joined #openstack-nova | 21:54 | |
| *** shaohe_feng has quit IRC | 21:55 | |
| *** irenab has quit IRC | 21:55 | |
| *** shaohe_feng has joined #openstack-nova | 21:55 | |
| mriedem | gibi_: +2 on https://review.openstack.org/#/c/313654/ | 22:02 |
| mriedem | i don't see a change for instance.create events | 22:02 |
| mriedem | i see the one for instance.update is higher up the stack with some others in between | 22:02 |
| mriedem | like pause and power off | 22:02 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/333181 | 22:02 |
| mriedem | maybe those would be easy to get through though since the bulk of those changes are just the samples | 22:03 |
| *** catintheroof has joined #openstack-nova | 22:03 | |
| mriedem | bauzas: want to take a look at this and the one below it again? https://review.openstack.org/#/c/330513/ | 22:03 |
| mriedem | bauzas: there are 2 +2s on ^ | 22:03 |
| mriedem | and a +2 on the base patch | 22:03 |
| mriedem | it's only like 10pm for you | 22:04 |
| bauzas | mriedem: heh, you lucky becauce it's midnight now :p | 22:04 |
| *** gszasz has quit IRC | 22:04 | |
| bauzas | mriedem: but I was looking at an Euro2016 soccer match | 22:04 |
| mriedem | i saw UK lost to iceland | 22:05 |
| mriedem | a big deal | 22:05 |
| *** shaohe_feng has quit IRC | 22:05 | |
| mriedem | anyway, i have to head home | 22:05 |
| mriedem | back online later | 22:05 |
| *** shaohe_feng has joined #openstack-nova | 22:06 | |
| bauzas | mriedem: yeah, now Iceland vs. France on Sunday ! | 22:06 |
| mriedem | better watch out | 22:06 |
| bauzas | anyway, is the above an non-prio one ? | 22:06 |
| mriedem | bauzas: yeah, but would be good to get the bottom one in | 22:07 |
| mriedem | might have an FFE for the update one | 22:07 |
| bauzas | k | 22:07 |
| *** dharinic_ has joined #openstack-nova | 22:07 | |
| bauzas | yeah, the first one is nice | 22:07 |
| *** mriedem has quit IRC | 22:07 | |
| *** cdelatte has quit IRC | 22:09 | |
| *** TravT has quit IRC | 22:12 | |
| *** irenab has joined #openstack-nova | 22:14 | |
| *** TravT has joined #openstack-nova | 22:14 | |
| *** shaohe_feng has quit IRC | 22:15 | |
| *** thorst_ has joined #openstack-nova | 22:16 | |
| *** shaohe_feng has joined #openstack-nova | 22:16 | |
| openstackgerrit | Rawlin Peters proposed openstack/nova: [DO NOT REVIEW - FOR TESTING PURPOSES ONLY] https://review.openstack.org/332474 | 22:17 |
| *** burt has quit IRC | 22:20 | |
| *** thorst_ has quit IRC | 22:20 | |
| *** thorst_ has joined #openstack-nova | 22:21 | |
| *** derekh has joined #openstack-nova | 22:22 | |
| *** mordred has joined #openstack-nova | 22:22 | |
| *** shaohe_feng has quit IRC | 22:26 | |
| *** shaohe_feng has joined #openstack-nova | 22:26 | |
| openstackgerrit | Kris Lindgren proposed openstack/nova: Option to always create network metadata information as static https://review.openstack.org/336293 | 22:27 |
| *** cdelatte has joined #openstack-nova | 22:27 | |
| *** thorst_ has quit IRC | 22:29 | |
| *** sdake_ has quit IRC | 22:33 | |
| *** pradk has quit IRC | 22:34 | |
| *** salv-orl_ has joined #openstack-nova | 22:34 | |
| *** salv-orlando has quit IRC | 22:35 | |
| *** shaohe_feng has quit IRC | 22:36 | |
| *** jamielennox|away is now known as jamielennox | 22:37 | |
| *** shaohe_feng has joined #openstack-nova | 22:37 | |
| *** shaohe_feng has quit IRC | 22:46 | |
| *** shaohe_feng has joined #openstack-nova | 22:47 | |
| *** aloga_ has quit IRC | 22:49 | |
| *** ducttape_ has quit IRC | 22:49 | |
| *** cdelatte has quit IRC | 22:52 | |
| openstackgerrit | Ed Leafe proposed openstack/nova: Return HTTP 400 on list for invalid status https://review.openstack.org/315964 | 22:53 |
| *** shaohe_feng has quit IRC | 22:56 | |
| *** shaohe_feng has joined #openstack-nova | 22:57 | |
| *** geekinutah has joined #openstack-nova | 22:58 | |
| *** shaohe_feng has quit IRC | 23:07 | |
| *** shaohe_feng has joined #openstack-nova | 23:07 | |
| *** thorst_ has joined #openstack-nova | 23:08 | |
| *** amotoki has quit IRC | 23:08 | |
| *** thorst_ has quit IRC | 23:08 | |
| *** zzelle_ has quit IRC | 23:09 | |
| *** thorst_ has joined #openstack-nova | 23:09 | |
| *** ayoung has joined #openstack-nova | 23:09 | |
| *** abc has quit IRC | 23:09 | |
| *** amotoki has joined #openstack-nova | 23:10 | |
| *** sdake has joined #openstack-nova | 23:11 | |
| *** sdake has quit IRC | 23:11 | |
| *** sacharya has quit IRC | 23:11 | |
| *** shaohe_feng has quit IRC | 23:17 | |
| *** shaohe_feng has joined #openstack-nova | 23:17 | |
| *** thorst_ has quit IRC | 23:18 | |
| *** catintheroof has quit IRC | 23:18 | |
| *** mriedem has joined #openstack-nova | 23:18 | |
| *** Prophidys has quit IRC | 23:20 | |
| *** READ10 has quit IRC | 23:21 | |
| *** markvoelker has quit IRC | 23:21 | |
| *** vtech has quit IRC | 23:27 | |
| *** shaohe_feng has quit IRC | 23:27 | |
| openstackgerrit | Giridhar Jayavelu proposed openstack/nova: VMware: configure limits during resize operation https://review.openstack.org/204219 | 23:28 |
| *** Prophidys has joined #openstack-nova | 23:28 | |
| openstackgerrit | Giridhar Jayavelu proposed openstack/nova: VMware: create vif with resource limitations https://review.openstack.org/196171 | 23:28 |
| *** EinstCrazy has joined #openstack-nova | 23:28 | |
| *** ihrachys has quit IRC | 23:29 | |
| openstackgerrit | Tracy Jones proposed openstack/nova: remove mox from vmwareapi/test_driver_api.py https://review.openstack.org/270303 | 23:31 |
| openstackgerrit | Tracy Jones proposed openstack/nova: Stop using mox stubs in test_configdrive https://review.openstack.org/265476 | 23:31 |
| openstackgerrit | Tracy Jones proposed openstack/nova: remove mox from vmwareapi/test_vif.py https://review.openstack.org/271069 | 23:31 |
| openstackgerrit | Tracy Jones proposed openstack/nova: VMware - Favor shared datastores https://review.openstack.org/289631 | 23:32 |
| *** EinstCrazy has quit IRC | 23:33 | |
| *** MikeG451 has quit IRC | 23:34 | |
| dansmith | mriedem: looks like ceph job failures have crept up a bit, but I _think_ it's related to cinder.. I sent a mail | 23:36 |
| *** yamahata has quit IRC | 23:36 | |
| mriedem | dansmith: the no valid host one? | 23:36 |
| mriedem | i noticed that a couple weeks ago | 23:36 |
| dansmith | mriedem: yeah | 23:36 |
| mriedem | something with the weight | 23:36 |
| dansmith | okay, the fail percentages weren't as striking until recently, so I didn't go looking | 23:36 |
| *** dharinic_ has quit IRC | 23:37 | |
| *** shaohe_feng has joined #openstack-nova | 23:38 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Add swap volume notifications (start, end) https://review.openstack.org/250283 | 23:41 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Add swap volume notifications (error) https://review.openstack.org/328055 | 23:42 |
| *** baoli has quit IRC | 23:42 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Add swap volume notifications samples https://review.openstack.org/328866 | 23:42 |
| *** cdelatte has joined #openstack-nova | 23:44 | |
| *** thorst_ has joined #openstack-nova | 23:44 | |
| *** BjoernT has joined #openstack-nova | 23:44 | |
| *** amotoki has quit IRC | 23:44 | |
| *** READ10 has joined #openstack-nova | 23:45 | |
| *** pwnall has quit IRC | 23:45 | |
| *** erlon has quit IRC | 23:46 | |
| *** derekh has quit IRC | 23:47 | |
| *** shaohe_feng has quit IRC | 23:48 | |
| *** shaohe_feng has joined #openstack-nova | 23:48 | |
| *** BjoernT has quit IRC | 23:50 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova: hyper-v: device tagging https://review.openstack.org/331889 | 23:50 |
| *** thorst_ has quit IRC | 23:54 | |
| *** thorst_ has joined #openstack-nova | 23:55 | |
| *** phuongnh has quit IRC | 23:57 | |
| *** shaohe_feng has quit IRC | 23:58 | |
| *** shaohe_feng has joined #openstack-nova | 23:59 | |
| *** hogepodge has joined #openstack-nova | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!