| *** sacharya has quit IRC | 00:00 | |
| *** sacharya has joined #openstack-nova | 00:00 | |
| mriedem | i farqed up the discover_hosts devstack patch, update coming | 00:04 |
|---|---|---|
| mikal | mriedem: what does it mean to have an instance mapping in a cell with no hosts? | 00:05 |
| mriedem | mikal: mayhem is what | 00:05 |
| mriedem | mikal: context? | 00:05 |
| mikal | mriedem: see comment on https://review.openstack.org/#/c/420451 | 00:05 |
| mikal | mriedem: you check if there are any hosts, and only if there are none do you check if there are any instances | 00:06 |
| mikal | mriedem: which seems... redundant to me | 00:06 |
| mikal | mriedem: but also not the end of the world | 00:06 |
| mriedem | good question | 00:06 |
| dansmith | they're different things, and if we had a host_map delete command, | 00:06 |
| dansmith | then you'd still need to check that there aren't any instances | 00:06 |
| *** dikonoor has joined #openstack-nova | 00:07 | |
| dansmith | this delete command is to let you delete cells you created and didn't setup | 00:07 |
| dansmith | since you can't really have multiple real cells yet, we don't likely need more of the delete commands just yet | 00:07 |
| mikal | dansmith: but doesn't no hosts imply no instances? | 00:07 |
| dansmith | no | 00:07 |
| dansmith | I mean, I know what you're saying, | 00:07 |
| mikal | dansmith: right, explain that bit in small words for me | 00:07 |
| dansmith | but if you had deleted a host you would still have some instance mappings | 00:07 |
| mikal | So deleting the host which has the instances on it doesn't cascade the delete to the instances? | 00:08 |
| dansmith | there is no foreign key constraint on that stuff, | 00:08 |
| dansmith | so you could have nuked some things | 00:08 |
| dansmith | you can't delete a host yet, so .. no :) | 00:08 |
| mikal | Heh | 00:09 |
| mriedem | like today you can delete a compute node in the db by deleting the nova-compute service for that host | 00:09 |
| mriedem | and i'm pretty sure that doesn't mean you can't still have instnaces in the db pointing at that node | 00:09 |
| dansmith | right | 00:09 |
| mikal | That's a fair point | 00:09 |
| dansmith | just sanity checking | 00:09 |
| mikal | I don't feel we need to hold the new thing to a higher standard than what we have now | 00:10 |
| *** catintheroof has joined #openstack-nova | 00:11 | |
| *** rfolco has joined #openstack-nova | 00:11 | |
| mriedem | mikal: i'm failing to pick up on sarcasm there | 00:14 |
| *** zz_dimtruck is now known as dimtruck | 00:15 | |
| *** jdurgin has quit IRC | 00:17 | |
| *** jerrygb has quit IRC | 00:18 | |
| mikal | mriedem: Oh, sorry | 00:19 |
| *** david-lyle has joined #openstack-nova | 00:19 | |
| *** hfu has joined #openstack-nova | 00:19 | |
| mikal | mriedem: I am glad the new code is consistent in its level of terrible with the old | 00:19 |
| mikal | mgood enough? | 00:19 |
| Kevin_Zheng | Hi cores, any particular reason that we have limited the length of instance tag to be 60 but created a db model with max length of 80? | 00:19 |
| *** thorst has joined #openstack-nova | 00:19 | |
| *** thorst has quit IRC | 00:21 | |
| Kevin_Zheng | http://git.openstack.org/cgit/openstack/nova/tree/nova/db/sqlalchemy/models.py#n1456 db model | 00:22 |
| *** sdague has quit IRC | 00:22 | |
| mriedem | Kevin_Zheng: hmm, probably a question for jaypipes | 00:22 |
| Kevin_Zheng | mriedm: hmm ok | 00:23 |
| mriedem | my guess is the 60 number in the API comes from the spec https://specs.openstack.org/openstack/nova-specs/specs/liberty/approved/tag-instances.html#data-model-impact | 00:24 |
| Kevin_Zheng | http://git.openstack.org/cgit/openstack/nova/tree/nova/objects/tag.py#n18 FYI | 00:24 |
| Kevin_Zheng | yeah, but why 80 in db model :) | 00:24 |
| jaypipes | Kevin_Zheng: no idea... probably just an oversight. | 00:25 |
| mriedem | snikitin: ^ | 00:25 |
| *** baoli has quit IRC | 00:26 | |
| mriedem | the db model change was https://review.openstack.org/#/c/97168/ | 00:27 |
| mriedem | which was a long long slog | 00:27 |
| mriedem | so my guess would be, the db model stuff merged, | 00:27 |
| mriedem | and then a release or more later the REST API change merged | 00:27 |
| Kevin_Zheng | maybe we can expand limit to 80? as we already created 80 in db | 00:27 |
| mriedem | at which point we'd totally forgotten what we were doing | 00:27 |
| Kevin_Zheng | haha | 00:27 |
| *** mtan_____ has joined #openstack-nova | 00:28 | |
| *** mtanino has quit IRC | 00:29 | |
| mriedem | yeah even looking at patch set 1 of https://review.openstack.org/#/c/128940/127/nova/objects/tag.py | 00:30 |
| Kevin_Zheng | looks like it start with 80 in the initial patch and nobody asked about it | 00:30 |
| Kevin_Zheng | yeah | 00:30 |
| mriedem | https://review.openstack.org/#/c/128940/1/nova/api/openstack/compute/schemas/v2/instance_tags.py | 00:31 |
| mriedem | it was 60 | 00:31 |
| mriedem | ^ had 127 patch sets | 00:31 |
| mriedem | i don't know if the 60 vs 80 has anything to do with unicode? | 00:31 |
| Kevin_Zheng | ah maybe, he said something about converting in spec | 00:32 |
| *** jdurgin has joined #openstack-nova | 00:32 | |
| mriedem | Kevin_Zheng: i assume you're asking because you need to know the schema for booting an instance with tags right? | 00:32 |
| *** hongbin has quit IRC | 00:32 | |
| mriedem | snikitin would be the person to ask if he remembers a good reason for it | 00:32 |
| Kevin_Zheng | ah, good, thanks, will try to catch him | 00:34 |
| *** jerrygb has joined #openstack-nova | 00:34 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Adds CPUFeaturesField to ImageMetaProps https://review.openstack.org/420960 | 00:35 |
| *** efried has quit IRC | 00:35 | |
| jaypipes | mriedem: no, 60 vs. 80 has nothing to do with unicode. | 00:35 |
| *** jaypipes has quit IRC | 00:37 | |
| Kevin_Zheng | jaypipes: so is it just an oversight or do we have any particular reason? | 00:39 |
| *** jose-phillips has quit IRC | 00:40 | |
| *** hoangcx has joined #openstack-nova | 00:41 | |
| *** david-lyle has quit IRC | 00:43 | |
| *** efried has joined #openstack-nova | 00:47 | |
| mriedem | i think it's probably just an oversight | 00:48 |
| Kevin_Zheng | lets sync them :) | 00:49 |
| mriedem | well, | 00:50 |
| mriedem | we could | 00:50 |
| mriedem | Kevin_Zheng: is there a technical reason you need tags in the REST API to allow up to 80? | 00:51 |
| *** Sukhdev has joined #openstack-nova | 00:52 | |
| mriedem | Kevin_Zheng: normally changing something like this in the REST API schema would be a microversion change, but this isn't making things more restrictive so it might be ok to change it without a microversion | 00:52 |
| Kevin_Zheng | yeah | 00:52 |
| mriedem | Kevin_Zheng: i think we should start with a post to the ML thread about it so snikitin has a chance to respond (and alex_xu) | 00:52 |
| mriedem | Kevin_Zheng: can you start that thread? | 00:52 |
| Kevin_Zheng | yeah sure | 00:52 |
| mriedem | thanks | 00:52 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Expand tag length limit to 80 https://review.openstack.org/421004 | 00:54 |
| Kevin_Zheng | ^ was already doing this, but we can block it until have a clean idea from ML | 00:55 |
| *** paw_ has joined #openstack-nova | 00:55 | |
| *** zayro has joined #openstack-nova | 00:55 | |
| *** paw_ is now known as Guest98732 | 00:55 | |
| *** paw has quit IRC | 00:58 | |
| mriedem | something still failing in discover_hosts on the subnode http://logs.openstack.org/76/420976/4/check/gate-tempest-dsvm-neutron-multinode-full-ubuntu-xenial-nv/1e4a55e/logs/subnode-2/devstacklog.txt.gz#_2017-01-17_00_35_47_248 | 01:02 |
| *** phuongnh has joined #openstack-nova | 01:03 | |
| *** takashin has joined #openstack-nova | 01:05 | |
| *** catintheroof has quit IRC | 01:09 | |
| *** jerrygb has quit IRC | 01:13 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/420844 | 01:13 |
| *** amotoki has quit IRC | 01:19 | |
| *** amotoki has joined #openstack-nova | 01:19 | |
| *** rfolco has quit IRC | 01:20 | |
| *** thorst has joined #openstack-nova | 01:22 | |
| *** trananhkma has quit IRC | 01:24 | |
| *** rfolco has joined #openstack-nova | 01:24 | |
| *** rfolco has quit IRC | 01:24 | |
| *** salv-orlando has joined #openstack-nova | 01:25 | |
| *** mlavalle has quit IRC | 01:26 | |
| *** thorst has quit IRC | 01:27 | |
| *** Sukhdev has quit IRC | 01:29 | |
| *** kevinz has joined #openstack-nova | 01:29 | |
| *** ljxiash has quit IRC | 01:39 | |
| *** unicell has joined #openstack-nova | 01:40 | |
| *** unicell has quit IRC | 01:41 | |
| *** thorst has joined #openstack-nova | 01:42 | |
| *** thorst has quit IRC | 01:43 | |
| openstackgerrit | Tuan Luong-Anh proposed openstack/nova: Support Ironic interface attach/detach in nova virt https://review.openstack.org/364413 | 01:43 |
| *** mdnadeem has joined #openstack-nova | 01:44 | |
| openstackgerrit | Sivasathurappan Radhakrishnan proposed openstack/nova: Initiate network changes from source during live migration https://review.openstack.org/404607 | 01:45 |
| *** yuntongjin has joined #openstack-nova | 01:45 | |
| *** mdnadeem has quit IRC | 01:46 | |
| *** salv-orlando has quit IRC | 01:46 | |
| *** mingyu has joined #openstack-nova | 01:50 | |
| openstackgerrit | Anh Tran proposed openstack/nova: Remove mox stubs in api/openstack/compute/test_server_reset_state.py https://review.openstack.org/320826 | 01:57 |
| *** mtan_____ has quit IRC | 01:57 | |
| *** tbachman has joined #openstack-nova | 01:58 | |
| *** sacharya has quit IRC | 01:59 | |
| *** sacharya has joined #openstack-nova | 02:00 | |
| *** dikonoor has quit IRC | 02:01 | |
| *** yamahata has quit IRC | 02:01 | |
| *** tiendc has joined #openstack-nova | 02:02 | |
| *** ljxiash has joined #openstack-nova | 02:02 | |
| *** ljxiash has quit IRC | 02:03 | |
| *** ljxiash has joined #openstack-nova | 02:03 | |
| *** nicolasbock has quit IRC | 02:03 | |
| *** jamesdenton has joined #openstack-nova | 02:03 | |
| openstackgerrit | Sivasathurappan Radhakrishnan proposed openstack/nova: Initiate network changes from source during live migration https://review.openstack.org/404607 | 02:06 |
| *** yamahata has joined #openstack-nova | 02:06 | |
| openstackgerrit | Sivasathurappan Radhakrishnan proposed openstack/nova: Refactor _post_live_migratiion https://review.openstack.org/413021 | 02:08 |
| *** jamesden_ has joined #openstack-nova | 02:09 | |
| *** jamesdenton has quit IRC | 02:09 | |
| *** salv-orlando has joined #openstack-nova | 02:13 | |
| *** jerrygb has joined #openstack-nova | 02:14 | |
| openstackgerrit | Anh Tran proposed openstack/nova: Remove mox stubs in api/openstack/compute/test_virtual_interfaces.py https://review.openstack.org/320325 | 02:18 |
| *** jamesden_ has quit IRC | 02:19 | |
| openstackgerrit | Dan Smith proposed openstack/nova: WIP Move instance creation to conductor https://review.openstack.org/319379 | 02:20 |
| *** jamesdenton has joined #openstack-nova | 02:22 | |
| *** jamielennox is now known as jamielennox|away | 02:25 | |
| *** Apoorva has quit IRC | 02:28 | |
| *** yuntongjin has quit IRC | 02:29 | |
| *** salv-orlando has quit IRC | 02:32 | |
| *** gcb has joined #openstack-nova | 02:36 | |
| *** jamesdenton has quit IRC | 02:38 | |
| *** fandi has joined #openstack-nova | 02:38 | |
| *** fandi has quit IRC | 02:39 | |
| openstackgerrit | Spencer Yu proposed openstack/nova: Transform instance.live_migration_rollback_dest notification https://review.openstack.org/420629 | 02:39 |
| *** mingyu has quit IRC | 02:40 | |
| *** CristinaPauna has quit IRC | 02:42 | |
| *** takashin has left #openstack-nova | 02:42 | |
| *** kevinz has quit IRC | 02:43 | |
| *** kevinz has joined #openstack-nova | 02:43 | |
| *** thorst has joined #openstack-nova | 02:44 | |
| *** mingyu has joined #openstack-nova | 02:46 | |
| *** mingyu_ has joined #openstack-nova | 02:47 | |
| *** mingyu has quit IRC | 02:47 | |
| *** eliqiao has joined #openstack-nova | 02:48 | |
| *** thorst has quit IRC | 02:48 | |
| *** sudipto_ has joined #openstack-nova | 02:52 | |
| *** sudipto has joined #openstack-nova | 02:52 | |
| *** hongbin has joined #openstack-nova | 02:53 | |
| *** CristinaPauna has joined #openstack-nova | 02:53 | |
| *** diga has joined #openstack-nova | 02:54 | |
| *** yuntongjin has joined #openstack-nova | 02:55 | |
| *** woodster_ has quit IRC | 02:55 | |
| *** amotoki has quit IRC | 02:55 | |
| *** salv-orlando has joined #openstack-nova | 03:00 | |
| *** tojuvone has quit IRC | 03:00 | |
| *** tojuvone has joined #openstack-nova | 03:00 | |
| *** mingyu_ has quit IRC | 03:02 | |
| *** jamielennox|away is now known as jamielennox | 03:03 | |
| openstackgerrit | Dan Peschman proposed openstack/nova: nova-manage cell_v2 map_cell0 exit 0 https://review.openstack.org/420132 | 03:05 |
| *** salv-orlando has quit IRC | 03:05 | |
| *** bkopilov has quit IRC | 03:05 | |
| *** bkopilov_ has quit IRC | 03:06 | |
| *** gouthamr has quit IRC | 03:12 | |
| *** jose-phillips has joined #openstack-nova | 03:13 | |
| *** amotoki has joined #openstack-nova | 03:13 | |
| *** jose-phillips has quit IRC | 03:14 | |
| *** timello has quit IRC | 03:14 | |
| *** jose-phillips has joined #openstack-nova | 03:15 | |
| *** jose-phillips has quit IRC | 03:17 | |
| *** jose-phillips has joined #openstack-nova | 03:17 | |
| *** jose-phillips has quit IRC | 03:19 | |
| *** mingyu has joined #openstack-nova | 03:19 | |
| *** jose-phillips has joined #openstack-nova | 03:19 | |
| *** jose-phillips has quit IRC | 03:21 | |
| *** jose-phillips has joined #openstack-nova | 03:22 | |
| *** jose-phillips has quit IRC | 03:27 | |
| *** edmondsw has joined #openstack-nova | 03:31 | |
| *** links has joined #openstack-nova | 03:32 | |
| *** Apoorva has joined #openstack-nova | 03:34 | |
| *** edmondsw has quit IRC | 03:36 | |
| *** timello has joined #openstack-nova | 03:36 | |
| openstackgerrit | xianming.mao proposed openstack/nova: :Add wsgi response for attach method https://review.openstack.org/411773 | 03:41 |
| *** yuntongjin has quit IRC | 03:41 | |
| *** pbandark has joined #openstack-nova | 03:43 | |
| openstackgerrit | Anusha Unnam proposed openstack/nova: Add keypairs field to InstancePayload https://review.openstack.org/419730 | 03:44 |
| *** mingyu has quit IRC | 03:48 | |
| *** amotoki has quit IRC | 03:48 | |
| openstackgerrit | xianming.mao proposed openstack/nova: Add wsgi response for attach method https://review.openstack.org/411773 | 03:48 |
| *** amotoki has joined #openstack-nova | 03:51 | |
| *** mingyu has joined #openstack-nova | 03:54 | |
| *** mdnadeem has joined #openstack-nova | 03:56 | |
| *** xinliang has quit IRC | 03:58 | |
| *** ljxiash has quit IRC | 04:04 | |
| *** amotoki has quit IRC | 04:05 | |
| *** sudipto has quit IRC | 04:09 | |
| *** sudipto has joined #openstack-nova | 04:09 | |
| *** psachin has joined #openstack-nova | 04:09 | |
| *** sudipto_ has quit IRC | 04:10 | |
| *** xinliang has joined #openstack-nova | 04:11 | |
| *** sudipto_ has joined #openstack-nova | 04:11 | |
| *** diga has quit IRC | 04:12 | |
| *** sheel has joined #openstack-nova | 04:13 | |
| *** hongbin has quit IRC | 04:16 | |
| *** hongbin has joined #openstack-nova | 04:17 | |
| *** bkopilov has joined #openstack-nova | 04:27 | |
| *** mingyu has quit IRC | 04:27 | |
| *** mingyu has joined #openstack-nova | 04:27 | |
| *** mingyu has quit IRC | 04:27 | |
| *** sudipto has quit IRC | 04:28 | |
| *** sudipto_ has quit IRC | 04:28 | |
| *** ljxiash has joined #openstack-nova | 04:31 | |
| *** ljxiash has quit IRC | 04:31 | |
| *** ljxiash has joined #openstack-nova | 04:31 | |
| *** bkopilov_ has joined #openstack-nova | 04:34 | |
| *** udesale has joined #openstack-nova | 04:35 | |
| *** Apoorva has quit IRC | 04:38 | |
| *** thorst has joined #openstack-nova | 04:45 | |
| *** Sukhdev has joined #openstack-nova | 04:45 | |
| *** Apoorva has joined #openstack-nova | 04:47 | |
| *** amotoki has joined #openstack-nova | 04:47 | |
| *** Apoorva has quit IRC | 04:49 | |
| *** thorst has quit IRC | 04:50 | |
| *** tlian has quit IRC | 04:50 | |
| *** salv-orlando has joined #openstack-nova | 05:01 | |
| *** ayogi has joined #openstack-nova | 05:03 | |
| *** prateek has joined #openstack-nova | 05:04 | |
| openstackgerrit | Yoshihiko Atsumi proposed openstack/nova: Add confirm resized server functional negative tests https://review.openstack.org/421074 | 05:05 |
| *** prateek has joined #openstack-nova | 05:05 | |
| *** ljxiash has quit IRC | 05:06 | |
| *** salv-orlando has quit IRC | 05:06 | |
| *** jerrygb has quit IRC | 05:08 | |
| *** unicell has joined #openstack-nova | 05:14 | |
| *** hongbin has quit IRC | 05:17 | |
| *** houming has joined #openstack-nova | 05:17 | |
| *** ratailor has joined #openstack-nova | 05:19 | |
| *** houming has quit IRC | 05:20 | |
| *** houming has joined #openstack-nova | 05:21 | |
| *** vladikr has quit IRC | 05:22 | |
| *** satyar has joined #openstack-nova | 05:22 | |
| *** sudipto_ has joined #openstack-nova | 05:26 | |
| *** sudipto has joined #openstack-nova | 05:26 | |
| *** dikonoor has joined #openstack-nova | 05:26 | |
| *** dimtruck is now known as zz_dimtruck | 05:26 | |
| *** ekuris has joined #openstack-nova | 05:28 | |
| *** dikonoor has quit IRC | 05:31 | |
| *** adisky_ has joined #openstack-nova | 05:31 | |
| *** david-lyle has joined #openstack-nova | 05:33 | |
| *** yuntongjin has joined #openstack-nova | 05:34 | |
| *** Apoorva has joined #openstack-nova | 05:37 | |
| *** ayogi has quit IRC | 05:41 | |
| *** david-lyle has quit IRC | 05:44 | |
| *** kylek3h has quit IRC | 05:46 | |
| *** kylek3h has joined #openstack-nova | 05:46 | |
| *** timello has quit IRC | 05:50 | |
| *** ratailor has quit IRC | 05:50 | |
| *** kylek3h has quit IRC | 05:51 | |
| *** david-lyle has joined #openstack-nova | 05:53 | |
| *** timello has joined #openstack-nova | 05:53 | |
| *** ratailor has joined #openstack-nova | 06:02 | |
| *** salv-orlando has joined #openstack-nova | 06:02 | |
| gmann | mriedem: Kevin_Zheng i think we should just do DB to 60, because it need microversion to make it 80 on API side for interoperability | 06:06 |
| *** salv-orlando has quit IRC | 06:07 | |
| *** nkrinner_afk is now known as nkrinner | 06:14 | |
| *** rama_y has quit IRC | 06:15 | |
| *** amotoki has quit IRC | 06:16 | |
| *** Apoorva has quit IRC | 06:22 | |
| *** david-lyle has quit IRC | 06:24 | |
| *** Apoorva has joined #openstack-nova | 06:26 | |
| *** markmcclain has quit IRC | 06:26 | |
| *** Apoorva has quit IRC | 06:28 | |
| *** sacharya has quit IRC | 06:28 | |
| *** sacharya has joined #openstack-nova | 06:29 | |
| *** Jeffrey4l_ has joined #openstack-nova | 06:30 | |
| *** Sukhdev has quit IRC | 06:31 | |
| *** sudipto_ has quit IRC | 06:33 | |
| *** sudipto has quit IRC | 06:33 | |
| *** sudipto has joined #openstack-nova | 06:33 | |
| *** sudipto has quit IRC | 06:33 | |
| *** sudipto has joined #openstack-nova | 06:34 | |
| *** sudipto_ has joined #openstack-nova | 06:34 | |
| *** Jeffrey4l has quit IRC | 06:34 | |
| *** Jeffrey4l_ is now known as Jeffrey4l | 06:34 | |
| *** mjura has joined #openstack-nova | 06:38 | |
| *** amotoki has joined #openstack-nova | 06:43 | |
| *** Sukhdev has joined #openstack-nova | 06:43 | |
| *** thorst has joined #openstack-nova | 06:46 | |
| *** kylek3h has joined #openstack-nova | 06:47 | |
| *** eandersson_ has joined #openstack-nova | 06:48 | |
| *** amotoki has quit IRC | 06:49 | |
| *** sridharg has joined #openstack-nova | 06:50 | |
| *** thorst has quit IRC | 06:50 | |
| *** eandersson has quit IRC | 06:51 | |
| *** kylek3h has quit IRC | 06:51 | |
| *** houming has quit IRC | 06:53 | |
| *** lpetrut has joined #openstack-nova | 06:58 | |
| *** moshele has joined #openstack-nova | 06:59 | |
| *** tbachman has quit IRC | 06:59 | |
| *** houming has joined #openstack-nova | 07:00 | |
| *** hachi has left #openstack-nova | 07:05 | |
| *** aojea has quit IRC | 07:05 | |
| *** houming has quit IRC | 07:06 | |
| *** houming has joined #openstack-nova | 07:07 | |
| *** tuanluong has joined #openstack-nova | 07:09 | |
| *** sree has joined #openstack-nova | 07:10 | |
| *** markus_z has joined #openstack-nova | 07:12 | |
| *** Oku_OS-away is now known as Oku_OS | 07:15 | |
| *** tesseract has joined #openstack-nova | 07:16 | |
| *** aojea has joined #openstack-nova | 07:17 | |
| *** voelzmo has joined #openstack-nova | 07:25 | |
| *** voelzmo has quit IRC | 07:25 | |
| *** voelzmo has joined #openstack-nova | 07:27 | |
| *** andreas_s has joined #openstack-nova | 07:33 | |
| openstackgerrit | Wang Qing wu proposed openstack/nova: Introduce base PowerVM driver https://review.openstack.org/391288 | 07:38 |
| Dinesh_Bhor | Hi all, can anyone tell me is there any document which explains what all libvirt related events generated by nova in what cases? | 07:38 |
| Kevin_Zheng | gmann: Hmm, users would love to have longer tags, but you have the point. | 07:39 |
| *** slaweq has quit IRC | 07:40 | |
| *** salv-orlando has joined #openstack-nova | 07:40 | |
| *** mriedem has quit IRC | 07:41 | |
| *** slaweq has joined #openstack-nova | 07:45 | |
| *** lpetrut has quit IRC | 07:46 | |
| gmann | Kevin_Zheng: yea we could have done initially but now it is difficult | 07:46 |
| Kevin_Zheng | will there be any problem if we shorten DB from 60 to 80? | 07:47 |
| Kevin_Zheng | when upgrading | 07:47 |
| *** lpetrut has joined #openstack-nova | 07:53 | |
| *** Sukhdev has quit IRC | 07:53 | |
| *** sree has quit IRC | 07:54 | |
| *** yuntongjin has quit IRC | 07:59 | |
| *** dmellado has quit IRC | 08:02 | |
| gmann | yea it might and we need migration script also? | 08:03 |
| *** dmellado has joined #openstack-nova | 08:04 | |
| *** takedakn has joined #openstack-nova | 08:07 | |
| *** salv-orlando has quit IRC | 08:09 | |
| *** ansiwen has quit IRC | 08:11 | |
| *** ansiwen has joined #openstack-nova | 08:13 | |
| *** amoralej|off is now known as amoralej | 08:15 | |
| *** yuntongjin has joined #openstack-nova | 08:24 | |
| *** ralonsoh has joined #openstack-nova | 08:30 | |
| *** amotoki has joined #openstack-nova | 08:34 | |
| *** sahid has joined #openstack-nova | 08:39 | |
| *** jpena|off is now known as jpena | 08:43 | |
| *** thorst has joined #openstack-nova | 08:47 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add sort_key white list for server list/detail https://review.openstack.org/415142 | 08:50 |
| *** yassine has joined #openstack-nova | 08:51 | |
| *** yassine is now known as Guest60784 | 08:51 | |
| *** thorst has quit IRC | 08:52 | |
| Kevin_Zheng | Sergey Nikitin thoughts it will might be much painful in DB migration than a new specless microversion in ML | 08:53 |
| *** tovin07 has quit IRC | 08:55 | |
| *** jpena is now known as jpena|off | 08:56 | |
| *** rmart04 has joined #openstack-nova | 08:56 | |
| *** ekuris has quit IRC | 08:57 | |
| *** ekuris has joined #openstack-nova | 08:58 | |
| *** zzzeek has quit IRC | 09:00 | |
| *** zzzeek has joined #openstack-nova | 09:00 | |
| *** rha has quit IRC | 09:03 | |
| *** jpena|off is now known as jpena | 09:03 | |
| *** tuanluong has quit IRC | 09:03 | |
| *** tuanluong has joined #openstack-nova | 09:04 | |
| *** markus_z has quit IRC | 09:04 | |
| openstackgerrit | huangtianhua proposed openstack/python-novaclient: Make _console() public https://review.openstack.org/414364 | 09:11 |
| *** rama_y has joined #openstack-nova | 09:12 | |
| *** Lehtu has quit IRC | 09:17 | |
| *** Matias has joined #openstack-nova | 09:18 | |
| *** markus_z has joined #openstack-nova | 09:19 | |
| *** tiendc has quit IRC | 09:23 | |
| *** tiendc has joined #openstack-nova | 09:23 | |
| *** mvk has quit IRC | 09:30 | |
| *** derekh has joined #openstack-nova | 09:38 | |
| *** swamireddy has quit IRC | 09:43 | |
| *** yuntongjin has quit IRC | 09:44 | |
| *** lucas-afk is now known as lucasagomes | 09:45 | |
| *** rama_y has quit IRC | 09:49 | |
| *** takedakn has quit IRC | 09:52 | |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: libbvirt: set vlan tag for macvtap on SRIOV VFs https://review.openstack.org/421186 | 09:58 |
| *** owalsh has quit IRC | 09:59 | |
| *** owalsh_ has joined #openstack-nova | 09:59 | |
| *** swamireddy has joined #openstack-nova | 10:01 | |
| *** hfu has quit IRC | 10:05 | |
| *** mvk has joined #openstack-nova | 10:06 | |
| *** hoangcx has quit IRC | 10:10 | |
| *** salv-orlando has joined #openstack-nova | 10:10 | |
| *** zayro has quit IRC | 10:11 | |
| *** phuongnh has quit IRC | 10:11 | |
| *** kevinz has quit IRC | 10:12 | |
| *** salv-orlando has quit IRC | 10:14 | |
| *** jose-phillips has joined #openstack-nova | 10:15 | |
| *** ekuris has quit IRC | 10:18 | |
| *** haplo37_ has quit IRC | 10:19 | |
| *** haplo37_ has joined #openstack-nova | 10:19 | |
| *** owalsh_ is now known as owalsh | 10:20 | |
| *** satyar has quit IRC | 10:24 | |
| openstackgerrit | Spencer Yu proposed openstack/nova: Transform instance.live_migration_rollback_dest notification https://review.openstack.org/420629 | 10:25 |
| *** udesale has quit IRC | 10:26 | |
| *** amotoki has quit IRC | 10:27 | |
| *** udesale has joined #openstack-nova | 10:27 | |
| openstackgerrit | Jeremy Liu proposed openstack/nova: Transform instance.live_migration_rollback_dest notification https://review.openstack.org/420629 | 10:30 |
| *** sahid has quit IRC | 10:30 | |
| *** mvk has quit IRC | 10:32 | |
| *** sahid has joined #openstack-nova | 10:33 | |
| *** ekuris has joined #openstack-nova | 10:34 | |
| *** tommylikehu has quit IRC | 10:40 | |
| *** mvk has joined #openstack-nova | 10:47 | |
| openstackgerrit | Pavel Gluschak proposed openstack/nova: Added live migration test hook for VNC console https://review.openstack.org/421207 | 10:47 |
| *** thorst has joined #openstack-nova | 10:48 | |
| *** udesale has quit IRC | 10:48 | |
| *** udesale has joined #openstack-nova | 10:49 | |
| *** tommylikehu has joined #openstack-nova | 10:49 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: [WIP] ignore me - trying experimental jobs https://review.openstack.org/421209 | 10:52 |
| *** thorst has quit IRC | 10:52 | |
| *** mingyu has joined #openstack-nova | 10:53 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: [WIP] ignore me - trying experimental jobs https://review.openstack.org/421209 | 10:54 |
| *** udesale__ has joined #openstack-nova | 10:54 | |
| *** udesale has quit IRC | 10:57 | |
| *** mriedem has joined #openstack-nova | 11:00 | |
| *** sudipto_ has quit IRC | 11:02 | |
| *** sudipto has quit IRC | 11:02 | |
| *** amotoki has joined #openstack-nova | 11:03 | |
| *** abalutoiu has quit IRC | 11:06 | |
| *** lpetrut has quit IRC | 11:06 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add sort_key white list for server list/detail https://review.openstack.org/415142 | 11:07 |
| *** cdent has joined #openstack-nova | 11:07 | |
| *** amotoki has quit IRC | 11:09 | |
| *** markus_z has quit IRC | 11:11 | |
| *** tbachman has joined #openstack-nova | 11:12 | |
| mriedem | https://bugs.launchpad.net/nova/+bug/1657087 if anyone wants to investigate that | 11:12 |
| openstack | Launchpad bug 1657087 in OpenStack Compute (nova) "Functional versioned notification test test_create_delete_server_with_instance_update randomly fails" [Medium,Confirmed] | 11:12 |
| openstackgerrit | Wang Qing wu proposed openstack/nova: Introduce base PowerVM driver https://review.openstack.org/391288 | 11:13 |
| *** sree has joined #openstack-nova | 11:14 | |
| *** mingyu has quit IRC | 11:15 | |
| *** tiendc_ has joined #openstack-nova | 11:15 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add new policy for server list/detail with all_tenants https://review.openstack.org/415330 | 11:15 |
| *** udesale__ has quit IRC | 11:17 | |
| openstackgerrit | Rikimaru Honjo proposed openstack/nova: Consolidate unit tests for shelve API https://review.openstack.org/421220 | 11:18 |
| *** tiendc has quit IRC | 11:18 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Add more details when test_create_delete_server_with_instance_update fails https://review.openstack.org/421221 | 11:19 |
| *** aloga has quit IRC | 11:20 | |
| *** aloga has joined #openstack-nova | 11:20 | |
| sahid | hello mriedem, anychance you look at this serie when you have a moment https://review.openstack.org/#/c/409260/ | 11:22 |
| *** ociuhandu has quit IRC | 11:23 | |
| mriedem | i see you and pkoniszewski are having a debate on the top patch in the series | 11:24 |
| *** rtheis has joined #openstack-nova | 11:24 | |
| *** rfolco has joined #openstack-nova | 11:25 | |
| sahid | yes the first one has already +2 and the last one need a third person | 11:25 |
| *** tbachman has quit IRC | 11:27 | |
| *** rha has joined #openstack-nova | 11:27 | |
| *** mvk has quit IRC | 11:28 | |
| *** houming has quit IRC | 11:28 | |
| openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Fix races with nfs volume mount/umount https://review.openstack.org/383859 | 11:29 |
| *** houming has joined #openstack-nova | 11:35 | |
| *** amotoki has joined #openstack-nova | 11:37 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Strict pattern match query parameters https://review.openstack.org/420494 | 11:38 |
| *** jose-phillips has quit IRC | 11:39 | |
| *** tbachman has joined #openstack-nova | 11:40 | |
| *** rtheis has quit IRC | 11:41 | |
| *** mvk has joined #openstack-nova | 11:41 | |
| *** sridharg has quit IRC | 11:41 | |
| *** jose-phillips has joined #openstack-nova | 11:41 | |
| *** sridharg has joined #openstack-nova | 11:41 | |
| *** JoseMello has joined #openstack-nova | 11:43 | |
| *** tbachman has quit IRC | 11:44 | |
| *** moshele has quit IRC | 11:47 | |
| *** sree has quit IRC | 11:48 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Fanout the db sync to all cells if --local_cell is False https://review.openstack.org/420973 | 11:50 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Add --verbose option to discover_hosts command https://review.openstack.org/421228 | 11:50 |
| *** bvanhav_ has joined #openstack-nova | 11:51 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Add --verbose option to discover_hosts command https://review.openstack.org/421228 | 11:53 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Fanout the db sync to all cells if --local_cell is False https://review.openstack.org/420973 | 11:53 |
| *** cdent has quit IRC | 11:58 | |
| *** tuanluong has quit IRC | 11:59 | |
| *** cdelatte has joined #openstack-nova | 11:59 | |
| *** tiendc_ has quit IRC | 11:59 | |
| *** nicolasbock has joined #openstack-nova | 12:01 | |
| openstackgerrit | Pavel Gluschak proposed openstack/nova: live-migration test hook for serial console https://review.openstack.org/347471 | 12:07 |
| openstackgerrit | Pavel Gluschak proposed openstack/nova: Added live migration test hook for VNC console https://review.openstack.org/421207 | 12:10 |
| *** lpetrut has joined #openstack-nova | 12:10 | |
| *** sree has joined #openstack-nova | 12:10 | |
| *** diga has joined #openstack-nova | 12:10 | |
| openstackgerrit | Pavel Gluschak proposed openstack/nova: don't store VNC/SPICE listen addrs in migration data if disabled https://review.openstack.org/396218 | 12:10 |
| *** lucasagomes is now known as lucas-hungry | 12:11 | |
| *** links has quit IRC | 12:12 | |
| *** kevinz has joined #openstack-nova | 12:14 | |
| *** jose-phillips has quit IRC | 12:14 | |
| *** gszasz has joined #openstack-nova | 12:14 | |
| *** markus_z has joined #openstack-nova | 12:16 | |
| *** bkopilov_ has quit IRC | 12:22 | |
| *** bkopilov has quit IRC | 12:23 | |
| *** tesseract has quit IRC | 12:23 | |
| *** mkoderer has quit IRC | 12:26 | |
| *** diga has quit IRC | 12:29 | |
| *** cdent has joined #openstack-nova | 12:29 | |
| *** catintheroof has joined #openstack-nova | 12:32 | |
| *** ekuris has quit IRC | 12:36 | |
| *** smatzek has joined #openstack-nova | 12:36 | |
| mriedem | claudiub|2: need to add hw_cpu_features to ImageMetaProps in a change before this https://review.openstack.org/#/c/398509/ | 12:40 |
| *** kashyap has quit IRC | 12:41 | |
| *** sdague has joined #openstack-nova | 12:43 | |
| dims | "Pass 1265 Failure 60 Skip 85" on the first run of gate-tempest-dsvm-nova-py35-ubuntu-xenial (DSVM/Tempest/py35), not too shabby | 12:44 |
| dims | Anyone interested in helping with this? ^ | 12:44 |
| mriedem | Kevin_Zheng: feel free to get a start on the pike spec for the tags length microversion, it should be a simple spec | 12:44 |
| mriedem | and the nova-specs repo is available for posting pike specs now | 12:44 |
| *** sree_ has joined #openstack-nova | 12:44 | |
| Kevin_Zheng | mriedem: thanks | 12:45 |
| *** sree_ is now known as Guest90801 | 12:45 | |
| Kevin_Zheng | will do it if got time | 12:45 |
| *** thorst has joined #openstack-nova | 12:46 | |
| *** kashyap has joined #openstack-nova | 12:47 | |
| *** sree has quit IRC | 12:47 | |
| claudiub|2 | mriedem: yep. that is why I've put that patch on top of this: https://review.openstack.org/#/c/420960/1 | 12:48 |
| *** mriedem has quit IRC | 12:48 | |
| *** houming has quit IRC | 12:48 | |
| *** gouthamr has joined #openstack-nova | 12:49 | |
| *** esberglu has quit IRC | 12:49 | |
| *** ratailor has quit IRC | 12:50 | |
| *** ekuris has joined #openstack-nova | 12:53 | |
| *** hoangcx has joined #openstack-nova | 12:54 | |
| *** hoangcx has quit IRC | 12:56 | |
| *** tbachman has joined #openstack-nova | 12:56 | |
| *** gcb has quit IRC | 12:58 | |
| *** links has joined #openstack-nova | 12:58 | |
| *** jwcroppe has joined #openstack-nova | 12:59 | |
| *** hoangcx has joined #openstack-nova | 13:00 | |
| *** links has quit IRC | 13:01 | |
| *** dave-mccowan has joined #openstack-nova | 13:02 | |
| *** edmondsw has joined #openstack-nova | 13:03 | |
| *** takashin has joined #openstack-nova | 13:05 | |
| *** tbachman has quit IRC | 13:06 | |
| *** tbachman has joined #openstack-nova | 13:07 | |
| *** tblakes has joined #openstack-nova | 13:07 | |
| *** amoralej is now known as amoralej|lunch | 13:09 | |
| *** jheroux has joined #openstack-nova | 13:10 | |
| *** sahid has quit IRC | 13:10 | |
| *** eharney has joined #openstack-nova | 13:10 | |
| openstackgerrit | Lee Yarwood proposed openstack/nova: Compute: Reset bdm connection_info during LM rollback https://review.openstack.org/338929 | 13:20 |
| openstackgerrit | Lee Yarwood proposed openstack/nova: Live migration of instances with encrypted volumes https://review.openstack.org/389608 | 13:20 |
| *** lucas-hungry is now known as lucasagomes | 13:21 | |
| *** jaypipes has joined #openstack-nova | 13:22 | |
| jaypipes | *yawn* | 13:24 |
| *** cristicalin has joined #openstack-nova | 13:25 | |
| * cdent passes jaypipes the coffee pot | 13:26 | |
| jaypipes | cdent: thx :) | 13:26 |
| *** kevinz has quit IRC | 13:27 | |
| openstackgerrit | Chris Dent proposed openstack/nova: A demo script for updating resource inventory https://review.openstack.org/382613 | 13:27 |
| *** kevinz has joined #openstack-nova | 13:27 | |
| *** acabot has joined #openstack-nova | 13:28 | |
| *** tbachman has quit IRC | 13:28 | |
| *** tbachman has joined #openstack-nova | 13:32 | |
| *** ralonsoh_ has joined #openstack-nova | 13:33 | |
| *** ralonsoh has quit IRC | 13:33 | |
| *** kevinz has quit IRC | 13:37 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: [WIP] ignore me - trying experimental jobs https://review.openstack.org/421209 | 13:37 |
| *** kevinz has joined #openstack-nova | 13:38 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Enable cold migration with target host(1/2) https://review.openstack.org/408955 | 13:38 |
| *** esberglu has joined #openstack-nova | 13:40 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Enable cold migration with target host(2/2) https://review.openstack.org/408964 | 13:40 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: api-ref: Add parameters in cold migrate action https://review.openstack.org/410042 | 13:40 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Add functional tests for cold migration to same host https://review.openstack.org/414926 | 13:41 |
| *** vladikr has joined #openstack-nova | 13:41 | |
| *** jpena is now known as jpena|lunch | 13:42 | |
| *** huanxie2 has joined #openstack-nova | 13:43 | |
| *** sudipto has joined #openstack-nova | 13:45 | |
| *** sudipto_ has joined #openstack-nova | 13:45 | |
| *** markvoelker has joined #openstack-nova | 13:45 | |
| *** mingyu has joined #openstack-nova | 13:45 | |
| openstackgerrit | garyk proposed openstack/nova: VMware: retry network lookup if network not ready https://review.openstack.org/391778 | 13:47 |
| * mdbooth would really, really like to get this bugfix in: https://review.openstack.org/#/c/383859/ | 13:48 | |
| mdbooth | I think it's fallen into the 'nobody really understands this code' hole, though. | 13:48 |
| mdbooth | All the CI we have thinks it's good. | 13:49 |
| kashyap | mdbooth: In such cases, shouldn't one just merge it, and not leave it in limbo? As the author, you understand it the deepest, and you're more than willing to explain any question to anyone. | 13:52 |
| kashyap | I think it should simply be merged if there are no regressions, or obvious repurcussions -- which there are none at the moment. | 13:53 |
| kashyap | Let's see what MattR has to say when he's here. He's usually quite clear with his stance. | 13:54 |
| * kashyap needs to step out for 40 min; bbiab | 13:55 | |
| *** sahid has joined #openstack-nova | 13:56 | |
| tdurakov | PaulMurray paul-carlton2 eliqiao shaohe_feng pkoniszewski yuntongjin mdbooth kashyap kmartin claudiub tobasco tdurakov andreas_s abhishekk mrhillsman raj_singh wznoinsk siva_krishnan hi folks, live-migration subteam meeting in 5 minutes on #openstack-meeting-3 | 13:56 |
| huanxie2 | Hi all, I made a very small patch to correct the comment message about input param's data type https://review.openstack.org/#/c/419308/could you help have a review, thanks a lot | 13:57 |
| *** baoli has joined #openstack-nova | 13:59 | |
| *** kylek3h has joined #openstack-nova | 13:59 | |
| *** krtaylor has joined #openstack-nova | 13:59 | |
| *** baoli has quit IRC | 14:00 | |
| *** baoli has joined #openstack-nova | 14:01 | |
| *** mdnadeem has quit IRC | 14:01 | |
| jaypipes | huanxie2: approved it. thx for the fix. | 14:02 |
| huanxie2 | Thanks jaypipes :) | 14:02 |
| jaypipes | np | 14:02 |
| huanxie2 | I also made another patch to fix the waiting vif-plug-event timeout problem when rescuing a VM https://review.openstack.org/#/c/413469/, could you also have a review if possible? | 14:04 |
| *** yamahata has quit IRC | 14:05 | |
| jaypipes | huanxie2: yes, will do shortlyu. | 14:06 |
| huanxie2 | Many thanks jaypipes | 14:08 |
| *** cristicalin has quit IRC | 14:08 | |
| *** mriedem has joined #openstack-nova | 14:10 | |
| *** mdrabe has joined #openstack-nova | 14:10 | |
| *** salv-orlando has joined #openstack-nova | 14:10 | |
| mriedem | jaypipes: i took a look at your aggregate map patch, comments inline | 14:10 |
| jaypipes | mriedem: already responded... | 14:11 |
| *** dikonoor has joined #openstack-nova | 14:11 | |
| mriedem | ok | 14:11 |
| jaypipes | mriedem: thx, great review comments. | 14:11 |
| mriedem | school is closed up here today so i'm going to be on and off | 14:11 |
| mdbooth | mriedem: Snow day? | 14:11 |
| jaypipes | mriedem: thx for letting me know. trying to perform a bunch of my day-job duties today :( (management stuff) | 14:11 |
| mriedem | ice dsay | 14:11 |
| mriedem | *day | 14:11 |
| *** liverpooler has quit IRC | 14:13 | |
| mdbooth | mriedem: I'm still banging on https://review.openstack.org/#/c/383859/ btw | 14:13 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add query parameters white list for server list/detail https://review.openstack.org/408571 | 14:13 |
| *** amoralej|lunch is now known as amoralej | 14:13 | |
| *** jerrygb has joined #openstack-nova | 14:14 | |
| mdbooth | I just updated it. sfinucan had asked for some docs, and I noticed an additional potential race while I was writing them. I've fixed the race and added docstrings. | 14:14 |
| mdbooth | Jenkins thinks it's good, and I've run the updated patch against devstack nfs, which still passes. | 14:15 |
| *** zz_dimtruck is now known as dimtruck | 14:15 | |
| *** ociuhandu has joined #openstack-nova | 14:16 | |
| lyarwood | mdbooth: quick nit on that review around your use of os.path.ismounted | 14:17 |
| mdbooth | The race was: if we lose and regain our libvirt connection, when we re-initialise the mount state we couldn't be certain that there were no ongoing operations from before we lost the connection. | 14:17 |
| *** sahid has quit IRC | 14:17 | |
| lyarwood | mdbooth: still looking at the rest, started there given the other change I'm working on | 14:17 |
| mdbooth | lyarwood: Thanks for looking! | 14:18 |
| *** sahid has joined #openstack-nova | 14:19 | |
| tdurakov | johnthetubaguy: hi, are you around? | 14:19 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add sort_key white list for server list/detail https://review.openstack.org/415142 | 14:20 |
| *** mlavalle has joined #openstack-nova | 14:22 | |
| mdbooth | lyarwood: That raises a follow-on question about is_mounted, btw. | 14:23 |
| *** tlian has joined #openstack-nova | 14:23 | |
| *** cleong has joined #openstack-nova | 14:23 | |
| mdbooth | I'm going to tread carefully around that rabbit hole :) I don't want to go down it. | 14:23 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add new policy for server list/detail with all_tenants https://review.openstack.org/415330 | 14:24 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Strict pattern match query parameters https://review.openstack.org/420494 | 14:25 |
| mriedem | mdbooth: ack. are you ok with the latest version of https://review.openstack.org/#/c/312488/ now? | 14:25 |
| lyarwood | mdbooth: I assume your going to ask if it did that check before right? | 14:26 |
| lyarwood | you're* | 14:26 |
| *** ducttape_ has quit IRC | 14:26 | |
| *** jamesdenton has joined #openstack-nova | 14:26 | |
| mdbooth | mriedem: Just checking again. I was nearly ok with with it last time iirc. | 14:26 |
| mdbooth | lyarwood: I responded to your comment. | 14:27 |
| lyarwood | mdbooth: yup good point | 14:28 |
| *** pbandark has quit IRC | 14:29 | |
| diana_clarke | mriedem: Let me know if there is anything you want help with, given ice day and all. In particular, is there anything I can help with wrt what tripleo needs from nova? | 14:30 |
| diana_clarke | man pages etc | 14:30 |
| *** abalutoiu has joined #openstack-nova | 14:31 | |
| mriedem | ok, hm | 14:31 |
| openstackgerrit | Gábor Antal proposed openstack/nova: Transform instance.reboot.error notification https://review.openstack.org/411791 | 14:31 |
| mriedem | it would be nice to have man page entries for the cells v2 commands | 14:31 |
| mriedem | in here http://docs.openstack.org/developer/nova/man/nova-manage.html | 14:32 |
| diana_clarke | mriedem: yeah, I saw the ask in the nova-cells-v2 wiki, which is why I asked. I no one is doing that, I can take that on. | 14:32 |
| *** bkopilov_ has joined #openstack-nova | 14:32 | |
| diana_clarke | typo: *if* no one is doing that... | 14:33 |
| mriedem | i don't think anyone is working on that so it's all yours | 14:33 |
| mriedem | you might want to do them one at a time in a series just to make reviews easier | 14:33 |
| *** bkopilov has joined #openstack-nova | 14:33 | |
| diana_clarke | mriedem: okie, dokie. and I'll update the wiki, claiming that task | 14:34 |
| mriedem | we might want to get https://review.openstack.org/#/c/420442/ merged to avoid conflicts too | 14:34 |
| mriedem | sdague: super simple change https://review.openstack.org/#/c/420442/ | 14:34 |
| *** mingyu has quit IRC | 14:34 | |
| sdague | mriedem: bam | 14:35 |
| *** gcb has joined #openstack-nova | 14:36 | |
| *** psachin has quit IRC | 14:37 | |
| mriedem | thanks emeril | 14:37 |
| mriedem | sean-k-mooney: do you know if we can re-use _get_vif_name in here https://review.openstack.org/#/c/410737/ ? i.e. is the device field set on that type of vif for vhostuser? | 14:38 |
| *** takashin_ has joined #openstack-nova | 14:38 | |
| *** takashin has quit IRC | 14:39 | |
| mriedem | i guess the vif does have a devname, "devname":"tap96429488-f7" | 14:39 |
| mriedem | but it's not what's needed here i guess | 14:39 |
| *** cristicalin has joined #openstack-nova | 14:41 | |
| *** huanxie2 has quit IRC | 14:43 | |
| markus_z | tdurakov: sorry, I missed the meeting | 14:43 |
| tdurakov | markus_z: ping me once update the hook | 14:44 |
| *** satyar has joined #openstack-nova | 14:44 | |
| *** hoangcx has quit IRC | 14:44 | |
| markus_z | tdurakov: I pushed the teardown pkoniszewski asked for in ps42. https://review.openstack.org/#/c/347471/ | 14:45 |
| *** ralonsoh_ is now known as ralonsoh | 14:45 | |
| tdurakov | acked, let me check it again | 14:45 |
| markus_z | thanks! | 14:45 |
| *** cristicalin has quit IRC | 14:47 | |
| *** hoangcx has joined #openstack-nova | 14:47 | |
| *** kevinz has quit IRC | 14:49 | |
| sean-k-mooney | mriedem just looking now | 14:49 |
| *** mingyu has joined #openstack-nova | 14:49 | |
| *** salv-orlando has quit IRC | 14:50 | |
| *** dimtruck is now known as zz_dimtruck | 14:50 | |
| *** Guest98732 is now known as paw | 14:50 | |
| sean-k-mooney | mriedem: looking at the code the vhost-user socket name must = the name of the openvswitch port | 14:50 |
| *** paw has quit IRC | 14:50 | |
| *** paw has joined #openstack-nova | 14:51 | |
| sean-k-mooney | mriedem: if we called _get_vif_name() we would have to modify neutron to generate the socket path differently | 14:51 |
| *** READ10 has joined #openstack-nova | 14:51 | |
| sean-k-mooney | my intention was to modify neutron to pass the name of the device in the port binding | 14:52 |
| sean-k-mooney | and then nave nova use that if present or fall back to generating it if not set | 14:52 |
| *** dikonoor has quit IRC | 14:53 | |
| mriedem | ok so in other words this is fine | 14:54 |
| sean-k-mooney | mriedem: yes what is there is currently fine but i would like to decouple it in the future so that we dont have to ensure that neutron and nova generate the name with the same algoritim | 14:55 |
| *** Guest90801 has quit IRC | 14:56 | |
| tdurakov | markus_z: done, thank you | 14:56 |
| markus_z | tdurakov: yeah, thanks! | 14:56 |
| tdurakov | mriedem, sdague hi, could you please take a look https://review.openstack.org/#/c/347471/43 | 14:56 |
| *** jpena|lunch is now known as jpena | 14:56 | |
| jaypipes | sdague: well said, on the PWG thing. I won't say that publicly, of course, since I'd get my head chewed off internally. | 14:57 |
| cdent | jaypipes: you must have very flexible teeth or jaw | 14:58 |
| *** xinliang has quit IRC | 14:58 | |
| *** udesale has joined #openstack-nova | 14:59 | |
| *** ducttape_ has joined #openstack-nova | 14:59 | |
| *** tovin07 has joined #openstack-nova | 15:00 | |
| *** sree has joined #openstack-nova | 15:01 | |
| *** swamireddy has quit IRC | 15:02 | |
| mriedem | sean-k-mooney: ok i've also gone through https://review.openstack.org/#/c/385061/ | 15:03 |
| *** hoangcx has quit IRC | 15:03 | |
| *** rama_y has joined #openstack-nova | 15:04 | |
| *** JoseMello has quit IRC | 15:05 | |
| *** owalsh has quit IRC | 15:06 | |
| *** sree has quit IRC | 15:06 | |
| *** jaosorior has joined #openstack-nova | 15:06 | |
| *** owalsh has joined #openstack-nova | 15:06 | |
| *** mingyu has quit IRC | 15:08 | |
| *** mingyu has joined #openstack-nova | 15:09 | |
| mriedem | alex_xu: your intel buddies might want to look at this https://review.openstack.org/#/c/420960/1/nova/objects/fields.py | 15:10 |
| mriedem | pkoniszewski: ^ | 15:10 |
| *** xinliang has joined #openstack-nova | 15:12 | |
| *** swamireddy has joined #openstack-nova | 15:12 | |
| cdent | _gryf: I think that endpoint stuff is simply that I hadn't revisited that update_inventory code in a long time (perhaps 4 months?) | 15:12 |
| *** sree has joined #openstack-nova | 15:12 | |
| *** takashin_ has left #openstack-nova | 15:12 | |
| _gryf | cdent, ok, cool | 15:13 |
| *** yamahata has joined #openstack-nova | 15:14 | |
| *** pcaruana has quit IRC | 15:15 | |
| *** nkrinner is now known as nkrinner_afk | 15:17 | |
| *** tbachman has quit IRC | 15:17 | |
| *** zz_dimtruck is now known as dimtruck | 15:17 | |
| *** sree has quit IRC | 15:17 | |
| *** burgerk has joined #openstack-nova | 15:19 | |
| *** udesale has quit IRC | 15:21 | |
| *** ekuris has quit IRC | 15:21 | |
| alex_xu | mriedem: that sounds like that overlap with traits. in the future we can request specific trait in the image metadata, the requested trait can be cpu feature. there is the spec about request traits https://review.openstack.org/#/c/351063/3/specs/ocata/approved/request-standardized-capabilities.rst@66 | 15:22 |
| *** hongbin has joined #openstack-nova | 15:24 | |
| *** david-lyle has joined #openstack-nova | 15:24 | |
| *** gcb has quit IRC | 15:26 | |
| pkoniszewski | thanks mriedem, i will take a look | 15:27 |
| *** david-lyle has quit IRC | 15:29 | |
| *** david-lyle has joined #openstack-nova | 15:29 | |
| cdent | edleafe: I think you're on PTO, but if not I fixed that bug in gabbi and made a new one: https://pypi.python.org/pypi/gabbi | 15:31 |
| openstackgerrit | Diana Clarke proposed openstack/nova: nova-manage docs: add cells commands prep https://review.openstack.org/421320 | 15:34 |
| *** david-lyle has quit IRC | 15:34 | |
| *** prateek has quit IRC | 15:42 | |
| *** david-lyle has joined #openstack-nova | 15:42 | |
| * cdent goes for a docs walk | 15:43 | |
| *** cdent has quit IRC | 15:43 | |
| *** marst has joined #openstack-nova | 15:45 | |
| *** tbachman has joined #openstack-nova | 15:45 | |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Add ComputeNodeList.get_all_by_uuids method https://review.openstack.org/418134 | 15:48 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: WIP: Scheduler calling the Placement API https://review.openstack.org/417961 | 15:48 |
| bauzas | mriedem: jaypipes: folks I need to bail out for a few, but you can bite ^ | 15:48 |
| openstackgerrit | Vladyslav Drok proposed openstack/nova: Generate necessary network metadata for ironic port groups https://review.openstack.org/388756 | 15:49 |
| *** mdrabe has quit IRC | 15:49 | |
| *** mvk has quit IRC | 15:51 | |
| *** timello has quit IRC | 15:51 | |
| *** tbachman has quit IRC | 15:52 | |
| *** tanee has quit IRC | 15:53 | |
| *** timello has joined #openstack-nova | 15:53 | |
| openstackgerrit | Diana Clarke proposed openstack/nova: nova-manage docs: cell_v2 simple_cell_setup https://review.openstack.org/421333 | 15:53 |
| *** tbachman has joined #openstack-nova | 15:54 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add query parameters white list for server list/detail https://review.openstack.org/408571 | 15:54 |
| *** burgerk has quit IRC | 15:54 | |
| *** david-lyle has quit IRC | 15:55 | |
| *** kfarr has joined #openstack-nova | 15:56 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add sort_key white list for server list/detail https://review.openstack.org/415142 | 15:56 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add new policy for server list/detail with all_tenants https://review.openstack.org/415330 | 15:57 |
| *** erhudy has joined #openstack-nova | 15:58 | |
| *** _ducttape_ has joined #openstack-nova | 15:59 | |
| *** mdrabe has joined #openstack-nova | 16:00 | |
| *** cdent has joined #openstack-nova | 16:02 | |
| *** ducttape_ has quit IRC | 16:03 | |
| *** mingyu has quit IRC | 16:03 | |
| *** ociuhandu has quit IRC | 16:05 | |
| *** tovin07 has left #openstack-nova | 16:06 | |
| *** tovin07 has joined #openstack-nova | 16:07 | |
| *** sheel has quit IRC | 16:07 | |
| *** david-lyle has joined #openstack-nova | 16:07 | |
| mriedem | tdurakov: markus_z: for https://review.openstack.org/#/c/347471/ i see it depends on a tempest patch to enable the test, do we have a patch up anywhere that depends on the nova change to see it all working and running the serial console tests in the live migration job? | 16:08 |
| mriedem | or is it self-testing? | 16:08 |
| markus_z | mriedem: it's self testing | 16:09 |
| mriedem | oh really http://logs.openstack.org/71/347471/43/check/gate-grenade-dsvm-neutron-multinode-live-migration-nv/c427a53/console.html#_2017-01-17_13_10_19_246017 | 16:09 |
| markus_z | http://logs.openstack.org/71/347471/43/check/gate-grenade-dsvm-neutron-multinode-live-migration-nv/c427a53/console.html#_2017-01-17_13_10_22_840459 | 16:09 |
| markus_z | We have to skip in in grenade. Let me search for the Ocata-Ocata case. | 16:10 |
| mriedem | oh right | 16:11 |
| mriedem | this is the o-o one http://logs.openstack.org/71/347471/43/check/gate-tempest-dsvm-multinode-live-migration-ubuntu-xenial/0b45362/console.html#_2017-01-17_12_34_13_290332 | 16:11 |
| mriedem | still skipped | 16:11 |
| mriedem | melwitt: i'm struggling trying to get discover_hosts to work on the subnode in a multinode job http://logs.openstack.org/81/420981/5/check/gate-tempest-dsvm-neutron-multinode-full-ubuntu-xenial-nv/6767a3a/logs/subnode-2/devstacklog.txt.gz#_2017-01-17_14_49_15_138 | 16:11 |
| mriedem | melwitt: please tell me there is something you know about this that i don't | 16:11 |
| mriedem | the subnode nova.conf has the api_database connection and we're getting the cell mappings from the api db ok, | 16:12 |
| mriedem | it's the target_cell that's failing on the non-cell0 cell | 16:12 |
| mriedem | for some reason it's trying to connect to 127.0.0.1 | 16:12 |
| markus_z | mriedem: That was the first run before the hook enables the serial console. After the hook we have http://logs.openstack.org/71/347471/43/check/gate-tempest-dsvm-multinode-live-migration-ubuntu-xenial/0b45362/console.html#_2017-01-17_12_35_19_716558 | 16:13 |
| markus_z | mriedem: The teardown of the hook sets everything back, so we have http://logs.openstack.org/71/347471/43/check/gate-tempest-dsvm-multinode-live-migration-ubuntu-xenial/0b45362/console.html#_2017-01-17_12_36_38_329078 | 16:13 |
| *** bswartz has quit IRC | 16:13 | |
| *** hieulq_ has joined #openstack-nova | 16:16 | |
| shaner | hi all - any idea why live-migrations won't show up when issuing 'migrations-list'? | 16:20 |
| shaner | (in kilo) | 16:20 |
| shaner | only those that were migrated using 'migrate' show up .. vs 'live-migration' | 16:21 |
| *** mjura has quit IRC | 16:23 | |
| jaypipes | shaner: pls see /topic. best to ask on ask.openstack.org or mailing list for usage questions. | 16:25 |
| shaner | jaypipes, thx, will do | 16:25 |
| *** jaosorior has quit IRC | 16:26 | |
| *** burgerk has joined #openstack-nova | 16:26 | |
| mriedem | migrate is cold migration | 16:26 |
| openstackgerrit | int32bit proposed openstack/python-novaclient: Allow multiple tag add/delete from cli https://review.openstack.org/414635 | 16:27 |
| shaner | mriedem, shouldn't 'migration-list' list all types of migrations though? | 16:28 |
| mriedem | i don't think until 2.23 http://docs.openstack.org/developer/nova/api_microversion_history.html#id21 | 16:28 |
| mriedem | fwiw this is the api code in kilo https://github.com/openstack/nova/blob/kilo-eol/nova/api/openstack/compute/contrib/migrations.py#L53 | 16:29 |
| openstackgerrit | int32bit proposed openstack/nova: Choose VMware datastore by the least provisioned space https://review.openstack.org/402632 | 16:30 |
| *** carthaca_ has joined #openstack-nova | 16:30 | |
| *** baoli has quit IRC | 16:30 | |
| *** bswartz has joined #openstack-nova | 16:32 | |
| mriedem | shaner: was the migration in progress or complete? if the migration was complete i wonder if the migration record was deleted | 16:36 |
| mriedem | at this point kilo is pretty far eol so it would take some investigating | 16:36 |
| *** ijw_ has joined #openstack-nova | 16:37 | |
| shaner | mriedem, it seems either way, in-progress/complete, it doesn't show up | 16:37 |
| shaner | I've tested it several times to no avail | 16:37 |
| shaner | appreciate the info! | 16:38 |
| mriedem | have you looked at the migrations table in the nova db? | 16:38 |
| mriedem | are there any records where deleted = 0? | 16:39 |
| shaner | mriedem, let me check | 16:39 |
| *** ijw has quit IRC | 16:40 | |
| *** cleong has quit IRC | 16:40 | |
| *** moshele has joined #openstack-nova | 16:41 | |
| mriedem | melwitt: hmm this is interesting | 16:43 |
| mriedem | the db connection for the cell0 mapping in this failed run has localhost in it http://logs.openstack.org/81/420981/5/check/gate-tempest-dsvm-neutron-multinode-full-ubuntu-xenial-nv/6767a3a/logs/devstacklog.txt.gz#_2017-01-17_14_43_53_943 | 16:43 |
| mriedem | nova-manage cell_v2 map_cell0 --database_connection 'mysql+pymysql://root:secretmysql@127.0.0.1/nova_cell0?charset=utf8' | 16:43 |
| mriedem | when we call simple_cell_setup the transport_url has a valid ip in it nova-manage cell_v2 simple_cell_setup --transport-url rabbit://stackrabbit:secretrabbit@10.20.124.3:5672/ | 16:45 |
| andreykurilin | mriedem: hi! I remember you spent a bit of time investigation an issue with novaclient's gate failures (which was fixed by blocking openstacksdk 0.9.12). If you have any details please share them here - https://review.openstack.org/#/c/418663 | 16:45 |
| *** tanee has joined #openstack-nova | 16:46 | |
| mriedem | andreykurilin: done | 16:47 |
| andreykurilin | mriedem: thanks | 16:47 |
| *** smatzek has quit IRC | 16:48 | |
| *** andreas_s has quit IRC | 16:49 | |
| mriedem | hmm wtf http://logs.openstack.org/81/420981/5/check/gate-tempest-dsvm-neutron-multinode-full-ubuntu-xenial-nv/6767a3a/logs/devstacklog.txt.gz#_2017-01-17_14_27_19_600 | 16:50 |
| mriedem | BASE_SQL_CONN=mysql+pymysql://root:secretmysql@127.0.0.1 | 16:50 |
| *** smatzek has joined #openstack-nova | 16:50 | |
| *** salv-orlando has joined #openstack-nova | 16:51 | |
| alaski | mriedem: yesterday was a holiday for me so I missed your message. Thanks for not letting a national figure distract from the important birthday | 16:53 |
| mriedem | alaski: you're MY national figure | 16:54 |
| shaner | mriedem, all items in migrations db are cold migrations and indeed have 'deleted = 0' | 16:54 |
| shaner | I don't see any from the past live-migrations I've done, so it seems, a live-migration isn't updating this table? | 16:54 |
| alaski | mriedem: :) | 16:54 |
| mriedem | DATABASE_HOST=MYSQL_HOST=SERVICE_LOCAL_HOST=127.0.0.1 blarg! | 16:54 |
| *** salv-orlando has quit IRC | 16:54 | |
| alaski | mriedem: I see there was a question of naming, did that get resolved? | 16:54 |
| *** salv-orlando has joined #openstack-nova | 16:55 | |
| mriedem | shaner: would have to check if live migrations were tracked with migrations records in kilo | 16:55 |
| mriedem | alaski: sort of https://review.openstack.org/#/c/420439/ | 16:55 |
| mriedem | there was much gnashing of teeth | 16:55 |
| mriedem | alaski: https://review.openstack.org/#/c/420439/5/nova/cmd/manage.py@1122 | 16:56 |
| mriedem | basically we weren't sure about the original intent of using the api_database.connection for the cell0 netloc and name | 16:56 |
| *** marst has quit IRC | 16:56 | |
| mriedem | but all agreed that nova_api_cell0 was a confusing db name for a db that is not using the api db schema | 16:56 |
| alaski | yep | 16:57 |
| mriedem | but also agreed that cell0 might not be on the same host as the api db, or main db, | 16:57 |
| mriedem | and finally agreed that us trying to guess is a bad idea and people should just provide the value explicitly | 16:57 |
| alaski | right, in a large deployment I expect all dbs to be separate | 16:57 |
| *** stvnoyes has joined #openstack-nova | 16:57 | |
| mriedem | we probably need to make a note in our docs for ^ | 16:57 |
| mriedem | i.e. .. note:: we'll guess, but might be wrong, and then cell0 won't work, don't say we didn't tell you | 16:58 |
| mriedem | sdague: so in devstack, if you're not on the subnode DATABASE_HOST isn't set, | 16:59 |
| mriedem | sdague: and on the api (primary) node, it gets defaulted to localhost: DATABASE_HOST=MYSQL_HOST=SERVICE_LOCAL_HOST=127.0.0.1 | 16:59 |
| *** _ducttape_ has quit IRC | 16:59 | |
| alaski | yeah. the simple_cell_setup command seems to sit in the same place as something like devstack, i.e. it'll get you setup and show you what needs doing but probably isn't what you want for production | 16:59 |
| mriedem | sdague: but the devstack docs say that DATABASE_HOST=$SERVICE_HOST by default if not set, and that's also what's used for the MQ host URL | 16:59 |
| mriedem | sdague: so anything you know of that would be bad for me trying to fix that? | 16:59 |
| *** ducttape_ has joined #openstack-nova | 16:59 | |
| *** markus_z has quit IRC | 17:00 | |
| alaski | mriedem: so the original reason for colocating cell0 with the api db vs the main db is that there is only ever expected to be one api db and one cell0 db, but there could be multiple "main" dbs. So thinking of where cell0 is gets confusing when there's multiple | 17:00 |
| *** nic has joined #openstack-nova | 17:00 | |
| openstackgerrit | Sivasathurappan Radhakrishnan proposed openstack/nova: Rollback on live migration failure https://review.openstack.org/379032 | 17:00 |
| sdague | mriedem: trying to establish context | 17:01 |
| mriedem | well i guess https://github.com/openstack-dev/devstack/blob/master/doc/source/guides/multinode-lab.rst#configure-compute-nodes says MYSQL_HOST=$SERVICE_HOST | 17:01 |
| *** _ducttape_ has joined #openstack-nova | 17:01 | |
| mriedem | sdague: in devstack today we create cell mappings on the primary node | 17:01 |
| mriedem | sdague: and i'm seeing that the db connection in those has 127.0.0.1 as the host | 17:01 |
| mriedem | because MYSQL_HOST=${MYSQL_HOST:-$SERVICE_LOCAL_HOST} | 17:01 |
| mriedem | sdague: and i've got a patch up to discover the subnode compute once n-cpu is running, | 17:02 |
| mriedem | but that's failing because the subnode can't connect to the cell db on 127.0.0.1 b/c the db doesn't exist on the subnode | 17:02 |
| mriedem | i'm actually confused as to how the subnode nova.conf has the proper database connection url | 17:03 |
| sdague | mriedem: which services are on the subnode? | 17:03 |
| mriedem | n-cpu | 17:03 |
| *** oomichi has joined #openstack-nova | 17:03 | |
| mriedem | oh i know why the subnode nova.conf is correct | 17:03 |
| sdague | I'm confused, n-cpu doesn't have a db connection url | 17:03 |
| mriedem | because d-g passes in the DATABASE_HOST in localrc | 17:03 |
| mriedem | it does in nova.conf | 17:03 |
| *** sahid has quit IRC | 17:03 | |
| openstackgerrit | Diana Clarke proposed openstack/nova: nova-manage docs: cell_v2 map_cell0 https://review.openstack.org/421384 | 17:04 |
| sdague | mriedem: ok, but n-cpu doesn't use it, right? | 17:04 |
| mriedem | http://logs.openstack.org/81/420981/5/check/gate-tempest-dsvm-neutron-multinode-full-ubuntu-xenial-nv/6767a3a/logs/subnode-2/etc/nova/nova.conf.txt.gz | 17:04 |
| mriedem | nope | 17:04 |
| sdague | because that's was the whole point of conductor :) | 17:04 |
| mriedem | yeah | 17:04 |
| mriedem | well, my change doesn't it need it for n-cpu | 17:04 |
| *** ducttape_ has quit IRC | 17:04 | |
| mriedem | it needs it for the discover_hosts CLI to run and map the subnode compute host to the parent cell | 17:04 |
| mriedem | but that parent cell is created when setting up the api node, which thinks it's db host ip is 127.0.0.1 | 17:05 |
| mriedem | anywho, i think it's a simple fix, i'll just push that up and we can see | 17:05 |
| mriedem | it's a really weird mix of d-g doesn't do something for the primary node, but does for the subnode, and you have to keep that all in your head at the same time to know what's what | 17:05 |
| mriedem | d-g should probably just always set DATABASE_HOST in localrc for multinode jobs | 17:06 |
| *** Apoorva has joined #openstack-nova | 17:06 | |
| mriedem | then devstack doesn't need to guess | 17:06 |
| owalsh | mriedem, bauzas: hola, I've a question about the cell_v2 api db sync... | 17:06 |
| mriedem | melwitt: you can continue doing all that you droppe doing from earlier pings, i've figured out the problem :) | 17:07 |
| mriedem | owalsh: shoot | 17:07 |
| mriedem | owalsh: first, that's not a thing | 17:07 |
| *** jaosorior has joined #openstack-nova | 17:07 | |
| mriedem | but ok | 17:07 |
| sdague | mriedem: I think the DATABASE_HOST was jammed in for cinder | 17:08 |
| openstackgerrit | int32bit proposed openstack/python-novaclient: Add CLI to show instance usage audit logs https://review.openstack.org/409413 | 17:08 |
| owalsh | mriedem: if a user installs the N tripleo undercloud but upgrades to O before adding any ironic nodes the sync will fail | 17:08 |
| owalsh | mriedem: here https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api_migrations/migrate_repo/versions/030_require_cell_setup.py#L51 | 17:08 |
| mriedem | N = newton? | 17:08 |
| owalsh | ^ yes | 17:08 |
| sdague | and because it never broken anything else, it was never looked at | 17:08 |
| sdague | mriedem: https://review.openstack.org/#/c/106043/ is where the subnode setting of that comes from | 17:08 |
| mriedem | sdague: yeah, i could probably have a patch in both projects | 17:09 |
| mriedem | sdague: yeah i know | 17:09 |
| mriedem | i was in there yesterday | 17:09 |
| mriedem | sdague: specifically this https://review.openstack.org/#/c/106043/23/devstack-vm-gate.sh@312 | 17:09 |
| mriedem | i'm saying we should just always set DATABASE_HOST in the localrc regardless of node | 17:09 |
| *** voelzmo has quit IRC | 17:10 | |
| *** cdent has quit IRC | 17:10 | |
| owalsh | mriedem: question being is that check absolutely necessary? | 17:10 |
| owalsh | mriedem: if a user has not run simple setup then it will fail a previous check. If map_cell0/create_cell etc... is being used then can we assume they know what they are doing | 17:11 |
| mriedem | owalsh: that check assumed that if you were doing a fresh install there would be no flavors yet and it would opt out | 17:11 |
| mriedem | *noop i mean | 17:11 |
| mriedem | but if there were flavors, it's an upgrade and there would be computes | 17:11 |
| owalsh | mriedem: ah, forgot to mention that installing the undercloud creates default flavors | 17:11 |
| mriedem | gdi | 17:11 |
| mriedem | can someone please reset the 'days since tripleo disaster' counter? | 17:11 |
| owalsh | mriedem: :-) | 17:12 |
| mriedem | so what are your steps again? | 17:12 |
| mriedem | install newton, sync api db, sync nova db, create flavors,... | 17:13 |
| sdague | mriedem: https://github.com/openstack-dev/devstack/blob/3c8ccecb1f0dff855e8043ba681d1749645de738/lib/nova#L468-L469 - probably that should be conditional behind n-api / n-cond / n-sched enabled | 17:13 |
| *** dtp has joined #openstack-nova | 17:14 | |
| *** raunak has joined #openstack-nova | 17:15 | |
| *** sridharg has quit IRC | 17:15 | |
| owalsh | mriedem: install newton, create flavors (implicitly by installing TripleO undercloud), upgrade to ocata, create cell0 db, map cell0, create cell1, sync nova db, discover_hosts (none yet), sync api db ... deploy computes, discover_hosts | 17:17 |
| *** hieulq_ has quit IRC | 17:17 | |
| owalsh | mriedem: I err think... | 17:17 |
| openstackgerrit | Dan Peschman proposed openstack/nova: nova-manage cell_v2 map_cell0 exit 0 https://review.openstack.org/420132 | 17:17 |
| *** mvk has joined #openstack-nova | 17:17 | |
| mriedem | owalsh: so between "discover_hosts (none yet), sync api db" you blow up | 17:18 |
| owalsh | mriedem: yes | 17:18 |
| mriedem | fudg | 17:19 |
| mriedem | *e | 17:19 |
| *** ijw has joined #openstack-nova | 17:19 | |
| *** rmart04 has quit IRC | 17:19 | |
| mriedem | see how nova-status handles this exact same thing https://github.com/openstack/nova/blob/master/nova/cmd/status.py#L153-L171 | 17:20 |
| mriedem | it doesn't consider it a failure | 17:20 |
| *** dtp is now known as dtp-afk | 17:21 | |
| owalsh | mriedem: yea, I'm thinking the db migraion could be relaxed. It will fail on the first check if the user hasn't run simple_cell_setup | 17:22 |
| *** yamahata has quit IRC | 17:22 | |
| *** rama_y has quit IRC | 17:22 | |
| *** tovin07 has quit IRC | 17:22 | |
| *** ijw_ has quit IRC | 17:22 | |
| owalsh | mriedem: if it gets as far as the host mapping check it implies that map_cell0 and create_cell were used, or simple_cell_setup failed... | 17:23 |
| *** karimb has joined #openstack-nova | 17:23 | |
| *** sudipto has quit IRC | 17:24 | |
| *** sudipto_ has quit IRC | 17:24 | |
| *** david-lyle_ has joined #openstack-nova | 17:26 | |
| *** unicell has quit IRC | 17:28 | |
| mriedem | i need to reset my brain on how we can even add cell mappings before running the api db sync | 17:28 |
| mriedem | in devstack we create the nova_api db and then sync it here https://github.com/openstack-dev/devstack/blob/3c8ccecb1f0dff855e8043ba681d1749645de738/lib/nova#L694 | 17:29 |
| mriedem | then later we run simple_cell_setup here https://github.com/openstack-dev/devstack/blob/3c8ccecb1f0dff855e8043ba681d1749645de738/lib/nova#L950 | 17:30 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: [WIP] ignore me - trying experimental jobs https://review.openstack.org/421209 | 17:30 |
| mriedem | oh devstack works because when we sync the api_db we don't have any flavors yet | 17:30 |
| mriedem | so it noops | 17:30 |
| mriedem | ok let me wrap up this thing i'm fixing first, then i can come back to this | 17:31 |
| *** sacharya has quit IRC | 17:33 | |
| *** sacharya has joined #openstack-nova | 17:33 | |
| gibi | mriedem: hi! I saw you patch chasing the race in the notification sample test https://review.openstack.org/#/c/421221 | 17:34 |
| *** david-lyle_ has quit IRC | 17:34 | |
| mriedem | yeah | 17:34 |
| gibi | mriedem: do we want to merge that to see more logs or just use to locally reproduce the problem? | 17:34 |
| mriedem | gibi: more logs | 17:34 |
| *** lpetrut has quit IRC | 17:34 | |
| mriedem | and so i can fingerprint the failure in elastic-recheck | 17:34 |
| gibi | mriedem: OK, then I'm totally +1 on that | 17:35 |
| *** moshele has quit IRC | 17:36 | |
| *** ociuhandu has joined #openstack-nova | 17:37 | |
| *** baoli has joined #openstack-nova | 17:37 | |
| openstackgerrit | Diana Clarke proposed openstack/nova: nova-manage docs: cell_v2 map_instances https://review.openstack.org/421404 | 17:40 |
| *** browne has joined #openstack-nova | 17:41 | |
| *** marst has joined #openstack-nova | 17:43 | |
| *** Kevin_Zheng has quit IRC | 17:47 | |
| *** derekh has quit IRC | 17:49 | |
| *** ociuhandu has quit IRC | 17:49 | |
| melwitt | mriedem: sweet! glad I could help by not being around | 17:50 |
| mriedem | melwitt: i think this is the culmination of 2 days of frustration https://review.openstack.org/#/c/421399/1 | 17:51 |
| mriedem | and about 4 hours of sleep | 17:51 |
| dims | mriedem : wow one liner! | 17:52 |
| *** dtp-afk_ has joined #openstack-nova | 17:52 | |
| melwitt | oof | 17:52 |
| *** lucasagomes is now known as lucas-afk | 17:53 | |
| mriedem | so in devstack runs we create flavors basically last | 17:53 |
| mriedem | we create the dbs and sync them, | 17:53 |
| mriedem | we run simple_cell_setup | 17:53 |
| mriedem | then we create the flavors | 17:54 |
| mriedem | and that's how devstack avoids failing the api db migration for there being no host mappings (well we'd have host mappings after running simple_cell_setup) | 17:54 |
| *** dtp-afk has quit IRC | 17:55 | |
| owalsh | ... and the api db doesn't know about hosts etc... so the flavor is the trigger yea | 17:55 |
| mriedem | in any other fresh install, you can't create flavors in the api db until you sync the api db | 17:55 |
| *** rama_y has joined #openstack-nova | 17:55 | |
| *** dtp-afk_ is now known as dtp-afk | 17:55 | |
| owalsh | sequence above is an upgrade, so we have the tables from newton I expect | 17:55 |
| mriedem | so, we have a particularly terrible scenario where you're actively going out of your way to create the flavors in the api db in newton, then upgrade to ocata and sync the api db before having any hosts | 17:56 |
| mriedem | owalsh: wtf does tripleo do this? | 17:56 |
| owalsh | mriedem: it doesn't, but it's a possible sequence of event | 17:56 |
| owalsh | mriedem: user would need to do this | 17:56 |
| mriedem | i thought the point of tripleo was to keep the user from hurting themselves? | 17:57 |
| mriedem | s/user/operator/ | 17:57 |
| mriedem | i mean i guess it's valid to create the newton control plane + flavors, and then decide, i might as well upgrade this to ocata even though i don't have any computes yet | 17:58 |
| mriedem | it just seems really weird | 17:58 |
| owalsh | it wouldn't be typical, but if an operator did this they wouldn't be able to recover easily I expect | 17:59 |
| *** gszasz has quit IRC | 17:59 | |
| *** david-lyle_ has joined #openstack-nova | 18:00 | |
| mriedem | well, they'd have to delete the flavors | 18:01 |
| *** yamahata has joined #openstack-nova | 18:01 | |
| mriedem | there would be 2 outs: | 18:01 |
| mriedem | 1. delete the flavors, sync the api db, do the cell setup stuff, then add the flavors | 18:02 |
| mriedem | 2. create the computes, run discover_hosts or simple_cell_setup, then sync the api db | 18:02 |
| *** lpetrut has joined #openstack-nova | 18:02 | |
| *** ralonsoh has quit IRC | 18:02 | |
| owalsh | should clarify, I'm talking about the all-in-one cloud that deploys the real cloud | 18:04 |
| owalsh | operator could install it, maybe create custom flavors etc... upgrade to ocata, then add hosts to deploy to | 18:06 |
| owalsh | far less likely to have a production cloud with no hosts | 18:06 |
| owalsh | but with flavors | 18:06 |
| *** ociuhandu has joined #openstack-nova | 18:11 | |
| *** ociuhandu has quit IRC | 18:15 | |
| openstackgerrit | Diana Clarke proposed openstack/nova: nova-manage docs: cell_v2 map_cell_and_hosts https://review.openstack.org/421420 | 18:16 |
| *** smatzek has quit IRC | 18:16 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Fix and add some notes to the cells v2 first time setup doc https://review.openstack.org/421421 | 18:17 |
| mriedem | melwitt: found a typo in the cells v2 setup doc ^ | 18:17 |
| owalsh | mriedem: so for fresh installs it would be: install ocata, create nova/api/cell0 dbs, sync api db, map_cell0, create_cell cell1, sync nova db ... deploy computes, discover_hosts | 18:20 |
| *** jpena is now known as jpena|off | 18:23 | |
| mriedem | owalsh: well, create_cell cell1 implies you have a cell1 db | 18:23 |
| mriedem | you don't have to create a cell1, | 18:25 |
| mriedem | simple_cell_setup and map_cell_and_hosts will default to use the nova db as cell1 | 18:25 |
| owalsh | can't use simple_cell_setup as we have no hosts yet | 18:26 |
| owalsh | create_cell cell1 defaults to using the nova db too AFAICT | 18:26 |
| *** unicell has joined #openstack-nova | 18:26 | |
| mriedem | yeah which fails w/o https://review.openstack.org/#/c/420079/ | 18:26 |
| mriedem | owalsh: yeah you're right, | 18:27 |
| mriedem | create_cell will use CONF.database.connection which is by default the main (non-api) db | 18:27 |
| *** ijw has quit IRC | 18:28 | |
| mriedem | owalsh: so yeah i think that's all correct for first time initial ocata setup | 18:28 |
| mriedem | also in http://docs.openstack.org/developer/nova/cells.html#setup-of-cells-v2 | 18:28 |
| mriedem | i think we plan on eventually getting rid of simple_cell_setup as it's not that simple for fresh install | 18:29 |
| *** adisky_ has quit IRC | 18:29 | |
| *** karimb_ has joined #openstack-nova | 18:30 | |
| comstud | mriedem: what's this cells thing you speak of? | 18:31 |
| comstud | never heard of it | 18:31 |
| comstud | i only know zones | 18:31 |
| *** karimb has quit IRC | 18:33 | |
| *** krtaylor has quit IRC | 18:36 | |
| mriedem | comstud: cells v2.0 | 18:36 |
| mriedem | cells: the reckoning | 18:37 |
| comstud | mriedem: Cells 2: Electric Boogaloo | 18:37 |
| comstud | i've been meaning to look @ what laski et al came up with, but no time | 18:38 |
| openstackgerrit | Diana Clarke proposed openstack/nova: nova-manage docs: cell_v2 verify_instance https://review.openstack.org/421436 | 18:39 |
| mriedem | cells IV: the revenge of behrens | 18:39 |
| comstud | lol | 18:39 |
| mriedem | that's for when you come back | 18:40 |
| openstackgerrit | Diana Clarke proposed openstack/nova: nova-manage docs: cell_v2 verify_instance https://review.openstack.org/421436 | 18:41 |
| owalsh | mriedem: I think the difficulty with all of this is the sync api db | 18:42 |
| *** Swami has joined #openstack-nova | 18:42 | |
| owalsh | ideall upgrades would be similar: install ocata, create nova/api/cell0 dbs, sync api db, map_cell0, create_cell cell1, sync nova db, discover_hosts | 18:42 |
| owalsh | s/ideall/ideally/ | 18:42 |
| *** karimb_ has quit IRC | 18:44 | |
| openstackgerrit | Diana Clarke proposed openstack/nova: nova-manage docs: cell_v2 discover_hosts https://review.openstack.org/421442 | 18:46 |
| mriedem | sdague: melwitt: got past that db host issue on the subnode \o/ | 18:46 |
| mriedem | owalsh: is there an upgrade step in there? | 18:47 |
| mriedem | wait, upgrade step in where? the doc? | 18:47 |
| *** david-lyle has quit IRC | 18:47 | |
| owalsh | mriedem: install ocata, from newton | 18:48 |
| mriedem | ok. | 18:48 |
| melwitt | mriedem: sweet, nice work | 18:48 |
| mriedem | when coming from newton you'd have the nova and nova_api dbs | 18:49 |
| mriedem | you still have to create cell0 | 18:49 |
| owalsh | ah, yes, typo | 18:49 |
| mriedem | and sync it and map it | 18:49 |
| *** markvoelker has quit IRC | 18:49 | |
| owalsh | yea, but after sync api db | 18:49 |
| owalsh | if it did have that migration | 18:49 |
| owalsh | s/did/didn't/ | 18:50 |
| *** dharinic is now known as dharinic|lunch | 18:50 | |
| *** ijw has joined #openstack-nova | 18:50 | |
| *** david-lyle_ has quit IRC | 18:50 | |
| mriedem | well, | 18:50 |
| *** karimb has joined #openstack-nova | 18:51 | |
| mriedem | the upgrade scenario we had in mind was install ocata, create cell0 db, run simple_cell_setup which creates the cell0 mapping in the api db, runs the db schema migration on cell0, and then does map_cell_and_hosts for the existing nova database | 18:51 |
| mriedem | ^ is what grenade does | 18:52 |
| mriedem | https://github.com/openstack-dev/grenade/blob/master/projects/60_nova/from-newton/upgrade-nova | 18:52 |
| mriedem | the problem is upgrading from newton to ocata with no compute hosts | 18:52 |
| mriedem | but having flavors | 18:53 |
| *** cdent has joined #openstack-nova | 18:53 | |
| mriedem | which is not something we anticipated | 18:53 |
| *** ijw has quit IRC | 18:54 | |
| openstackgerrit | Diana Clarke proposed openstack/nova: nova-manage docs: cell_v2 create_cell https://review.openstack.org/421444 | 18:55 |
| owalsh | where/when is the api db sync? | 18:55 |
| mriedem | after ^ | 18:59 |
| mriedem | here https://github.com/openstack-dev/devstack/blob/master/lib/nova#L681 | 19:00 |
| *** cdent has quit IRC | 19:00 | |
| melwitt | this is reminding me what I said in the ML thread about the setup docs, we need to include step-by-step instructions for each common scenario | 19:00 |
| openstackgerrit | Diana Clarke proposed openstack/nova: nova-manage docs: cell_v2 map_instances https://review.openstack.org/421404 | 19:01 |
| openstackgerrit | Diana Clarke proposed openstack/nova: nova-manage docs: cell_v2 map_cell_and_hosts https://review.openstack.org/421420 | 19:01 |
| openstackgerrit | Diana Clarke proposed openstack/nova: nova-manage docs: cell_v2 verify_instance https://review.openstack.org/421436 | 19:01 |
| openstackgerrit | Diana Clarke proposed openstack/nova: nova-manage docs: cell_v2 discover_hosts https://review.openstack.org/421442 | 19:01 |
| openstackgerrit | Diana Clarke proposed openstack/nova: nova-manage docs: cell_v2 create_cell https://review.openstack.org/421444 | 19:01 |
| *** amoralej is now known as amoralej|off | 19:01 | |
| mriedem | before you ask me why we recreate the nova_api db after upgrading from ocata, that's a good question | 19:02 |
| mriedem | melwitt: i think you just signed up for that | 19:03 |
| melwitt | yeah, I should do that :) | 19:04 |
| mriedem | i'm not entirely sure what goes on with the dbs in grenade | 19:06 |
| mriedem | sdague: is https://github.com/openstack-dev/devstack/blob/master/lib/nova#L676 not called during grenade? | 19:08 |
| mriedem | oh i guess not | 19:11 |
| mriedem | which is why we don't stomp on the databases from the n-1 run | 19:11 |
| mriedem | ok i'm going to be away for awhile | 19:12 |
| owalsh | mriedem: me too, thanks! | 19:12 |
| *** karimb has quit IRC | 19:14 | |
| *** jose-phillips has joined #openstack-nova | 19:14 | |
| *** satyar has quit IRC | 19:14 | |
| *** Oku_OS is now known as Oku_OS-away | 19:15 | |
| openstackgerrit | Eric Fried proposed openstack/nova: Introduce base PowerVM driver https://review.openstack.org/391288 | 19:17 |
| *** owalsh has quit IRC | 19:17 | |
| *** owalsh_ has joined #openstack-nova | 19:17 | |
| openstackgerrit | Eric Fried proposed openstack/nova: Introduce base PowerVM driver https://review.openstack.org/391288 | 19:17 |
| *** owalsh_ is now known as owalsh | 19:17 | |
| openstackgerrit | Diana Clarke proposed openstack/nova: nova-manage docs: cell_v2 map_instances https://review.openstack.org/421404 | 19:18 |
| openstackgerrit | Diana Clarke proposed openstack/nova: nova-manage docs: cell_v2 map_cell_and_hosts https://review.openstack.org/421420 | 19:18 |
| openstackgerrit | Diana Clarke proposed openstack/nova: nova-manage docs: cell_v2 verify_instance https://review.openstack.org/421436 | 19:18 |
| openstackgerrit | Diana Clarke proposed openstack/nova: nova-manage docs: cell_v2 simple_cell_setup https://review.openstack.org/421333 | 19:18 |
| openstackgerrit | Diana Clarke proposed openstack/nova: nova-manage docs: cell_v2 map_cell0 https://review.openstack.org/421384 | 19:18 |
| openstackgerrit | Diana Clarke proposed openstack/nova: nova-manage docs: cell_v2 discover_hosts https://review.openstack.org/421442 | 19:18 |
| openstackgerrit | Diana Clarke proposed openstack/nova: nova-manage docs: cell_v2 create_cell https://review.openstack.org/421444 | 19:18 |
| openstackgerrit | Diana Clarke proposed openstack/nova: nova-manage docs: cell_v2 list_cells https://review.openstack.org/421458 | 19:18 |
| openstackgerrit | Diana Clarke proposed openstack/nova: nova-manage docs: cell_v2 delete_cell https://review.openstack.org/421459 | 19:18 |
| *** unicell has quit IRC | 19:24 | |
| openstackgerrit | Diana Clarke proposed openstack/nova: nova-manage docs: cell_v2 delete_cell https://review.openstack.org/421459 | 19:31 |
| openstackgerrit | Diana Clarke proposed openstack/nova: nova-manage docs: cell_v2 list_cells https://review.openstack.org/421458 | 19:31 |
| *** dharinic|lunch is now known as dharinic | 19:32 | |
| *** tommylikehu has quit IRC | 19:40 | |
| *** cdent has joined #openstack-nova | 19:44 | |
| *** dtp-afk has quit IRC | 19:48 | |
| *** dtp-afk has joined #openstack-nova | 19:49 | |
| *** unicell has joined #openstack-nova | 19:49 | |
| *** dtp-afk is now known as dtp | 19:49 | |
| *** david-lyle has joined #openstack-nova | 19:50 | |
| *** beagles is now known as beagles_brb | 19:52 | |
| *** rfolco has quit IRC | 19:54 | |
| *** unicell has quit IRC | 19:59 | |
| *** marst_ has joined #openstack-nova | 20:00 | |
| *** marst has quit IRC | 20:03 | |
| *** rama_y has quit IRC | 20:03 | |
| openstackgerrit | Sujitha proposed openstack/nova: Adding auto_disk_config field to InstancePayload https://review.openstack.org/419185 | 20:04 |
| *** Sukhdev_ has joined #openstack-nova | 20:05 | |
| openstackgerrit | Sujitha proposed openstack/nova: Adding auto_disk_config field to InstancePayload https://review.openstack.org/419185 | 20:06 |
| openstackgerrit | Joshua Harlow proposed openstack/nova: Handle oslo.serialization type error and binascii error https://review.openstack.org/421482 | 20:08 |
| *** jidar has joined #openstack-nova | 20:11 | |
| *** salv-orlando has quit IRC | 20:11 | |
| openstackgerrit | Eric Fried proposed openstack/nova: Introduce base PowerVM driver https://review.openstack.org/391288 | 20:21 |
| *** unicell has joined #openstack-nova | 20:23 | |
| *** marst_ has quit IRC | 20:25 | |
| *** krtaylor has joined #openstack-nova | 20:26 | |
| *** mtanino has joined #openstack-nova | 20:28 | |
| openstackgerrit | Jiri Suchomel proposed openstack/nova: Retry few times when creating the initial connection to libvirtd. https://review.openstack.org/416928 | 20:29 |
| *** gszasz has joined #openstack-nova | 20:29 | |
| *** woodster_ has joined #openstack-nova | 20:32 | |
| *** bvanhav_ has quit IRC | 20:33 | |
| *** READ10 has quit IRC | 20:37 | |
| openstackgerrit | Sivasathurappan Radhakrishnan proposed openstack/nova: Extract _post_live_migratiion_network_setup https://review.openstack.org/413021 | 20:43 |
| *** lpetrut has quit IRC | 20:47 | |
| *** lpetrut has joined #openstack-nova | 20:49 | |
| *** beagles_brb is now known as beagles | 20:50 | |
| *** sacharya_ has joined #openstack-nova | 20:57 | |
| *** sacharya has quit IRC | 20:59 | |
| claudiub|2 | mriedem: hi. around? | 21:00 |
| *** ducttape_ has joined #openstack-nova | 21:01 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: [WIP] ignore me - trying experimental jobs https://review.openstack.org/421209 | 21:02 |
| *** jdurgin has quit IRC | 21:03 | |
| *** _ducttape_ has quit IRC | 21:04 | |
| *** Guest60784 has quit IRC | 21:05 | |
| *** ijw_ has joined #openstack-nova | 21:07 | |
| *** marst_ has joined #openstack-nova | 21:08 | |
| *** dave-mccowan has quit IRC | 21:09 | |
| harlowja | mriedem if u get a chance, when this is ok, https://review.openstack.org/#/c/421482/ | 21:09 |
| harlowja | need for a oslo.serialization release that will unify the exceptions (until then cinder uses one type, nova uses the other, so its a PITA, lol) | 21:10 |
| smcginnis | Hah - "can proceed with breaking nova" | 21:11 |
| efried | I was amused | 21:11 |
| smcginnis | efried: ;) | 21:11 |
| *** ijw_ has quit IRC | 21:12 | |
| openstackgerrit | Chris Dent proposed openstack/nova: Remove invalid URL in gabbi tests https://review.openstack.org/421529 | 21:13 |
| *** lpetrut has quit IRC | 21:13 | |
| cdent | dims ^ | 21:13 |
| cdent | I'm not sure if I tidied up the py3 blacklist file properly or not, but if not, let me know and I'll fix it tomorrow, I'm done with today | 21:14 |
| *** cdent has quit IRC | 21:14 | |
| nicolasbock | Hi! I am trying to understand how nova-metadata works. I got to the point where I am reading nova/api/metadata/base.py:InstanceMetadata.get_ec2_metadata() | 21:16 |
| nicolasbock | I suppose that this method returns the ssh public key for instance | 21:16 |
| *** ijw has joined #openstack-nova | 21:17 | |
| nicolasbock | how does this work though when I enable ssl in nova? | 21:17 |
| *** ducttape_ has quit IRC | 21:17 | |
| *** cdelatte has quit IRC | 21:17 | |
| *** ducttape_ has joined #openstack-nova | 21:17 | |
| nicolasbock | cloud-init in a server will request http://169.254.169.254/latest/.... and nova-metadata will expect an ssl connection | 21:18 |
| nicolasbock | wouldn't that clash? | 21:18 |
| *** jdurgin has joined #openstack-nova | 21:22 | |
| harlowja | smcginnis oops, haha | 21:27 |
| smcginnis | harlowja: ;) | 21:27 |
| openstackgerrit | Joshua Harlow proposed openstack/nova: Handle oslo.serialization type error and binascii error https://review.openstack.org/421482 | 21:27 |
| *** marst_ has quit IRC | 21:29 | |
| *** Jeffrey4l has quit IRC | 21:35 | |
| openstackgerrit | Ed Leafe proposed openstack/nova: Remove the Allocation.create() method. https://review.openstack.org/416751 | 21:42 |
| openstackgerrit | Ed Leafe proposed openstack/nova: Add check for invalid allocation amounts https://review.openstack.org/416752 | 21:42 |
| edleafe | bauzas: found a bug in your recent merge: https://review.openstack.org/#/c/392569/27/nova/tests/functional/api/openstack/placement/fixtures.py@106 (and line 112) | 21:42 |
| edleafe | bauzas: there is a step_size of 10, but an allocation of 512, which should fail, but doesn't | 21:43 |
| edleafe | bauzas: the change to AllocationList.create_all() caught it | 21:43 |
| *** Jeffrey4l has joined #openstack-nova | 21:48 | |
| openstackgerrit | Jiri Suchomel proposed openstack/nova: Retry few times when creating the initial connection to libvirtd. https://review.openstack.org/416928 | 21:50 |
| bauzas | edleafe: not sure it's due to my bug but okay:) | 21:54 |
| edleafe | bauzas: when you added the step_size, min_unit, and max_unit, the subsequent allocation should have failed, since 512 isn't a multiple of 10 | 21:57 |
| edleafe | The old code happily created the allocation | 21:58 |
| edleafe | The new code won't | 21:58 |
| siva_krishnan | tdurakov ? | 21:59 |
| *** thorst has quit IRC | 21:59 | |
| *** krtaylor has quit IRC | 21:59 | |
| tdurakov | siva_krishnan: hi, what's up? | 22:00 |
| *** gouthamr has quit IRC | 22:01 | |
| siva_krishnan | this is regarding bug https://bugs.launchpad.net/nova/+bug/1599057 | 22:01 |
| openstack | Launchpad bug 1647451 in OpenStack Compute (nova) "duplicate for #1599057 Post live migration step could fail due to auth errors" [Medium,Fix released] - Assigned to Timofey Durakov (tdurakov) | 22:01 |
| *** harlowja has quit IRC | 22:01 | |
| bauzas | edleafe: but I haven't provided the allocation API, right? | 22:01 |
| tdurakov | Yes | 22:02 |
| bauzas | unless I misunderstand | 22:02 |
| siva_krishnan | tdurakov: my intent of putting up patch for that bug is to do rollback on exception which occurs prior to live migration | 22:02 |
| slaweq | oomichi: hello | 22:03 |
| slaweq | oomichi: can You maybe look at https://review.openstack.org/#/c/420176/ ? | 22:03 |
| slaweq | oomichi: I answered to Your comment there | 22:04 |
| siva_krishnan | as you mentioned the bug description says the timeout happened during post live migration | 22:04 |
| slaweq | oomichi: and I want to know what You think a about it | 22:04 |
| tdurakov | siva_krishnan: yes, I understand that, the thing is that trying to call rollback here could be very tricky | 22:05 |
| edleafe | bauzas: nothing to do with the API | 22:05 |
| tdurakov | E.g. Exception is raised during roll-back itself | 22:05 |
| edleafe | bauzas: In the change I referenced you added constraints to the inventory in the fixture that should have made the subsequent allocations invalid | 22:06 |
| *** sacharya_ has quit IRC | 22:06 | |
| bauzas | ah right | 22:06 |
| edleafe | When I rebased on master these tests started failing, since my code removes Allocation.create() | 22:06 |
| *** sacharya has joined #openstack-nova | 22:06 | |
| bauzas | sorry then | 22:06 |
| edleafe | bauzas: Oh, don't be | 22:06 |
| bauzas | so only the gabbits are failing | 22:07 |
| * bauzas whispers | 22:07 | |
| edleafe | bauzas: I was just saying that the placement code is getting more accurate | 22:07 |
| *** rama_y has joined #openstack-nova | 22:07 | |
| *** rama_y_ has joined #openstack-nova | 22:07 | |
| siva_krishnan | tdurakov: ah ok you are right but I do see that pre_live_migration failure does the rollback as well | 22:07 |
| bauzas | edleafe: unfortunately, that modification was done by jay, but I didn't reviewed it | 22:08 |
| tdurakov | I'll leave a comment on the patch trying to explain details, pre-step is way more specific, that's why it's safe | 22:08 |
| bauzas | edleafe: thanks for that | 22:08 |
| *** burgerk has quit IRC | 22:09 | |
| *** jheroux has quit IRC | 22:10 | |
| siva_krishnan | tdurakov: that helps :). | 22:10 |
| *** pradk has quit IRC | 22:14 | |
| *** dimtruck is now known as zz_dimtruck | 22:16 | |
| *** catintheroof has quit IRC | 22:20 | |
| *** slaweq has quit IRC | 22:22 | |
| *** thorst has joined #openstack-nova | 22:24 | |
| *** thorst has quit IRC | 22:24 | |
| *** vladikr has quit IRC | 22:24 | |
| *** salv-orlando has joined #openstack-nova | 22:28 | |
| *** tblakes has quit IRC | 22:30 | |
| *** baoli has quit IRC | 22:32 | |
| *** zz_dimtruck is now known as dimtruck | 22:33 | |
| *** tlian has quit IRC | 22:34 | |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Add ComputeNodeList.get_all_by_uuids method https://review.openstack.org/418134 | 22:37 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: WIP: Scheduler calling the Placement API https://review.openstack.org/417961 | 22:37 |
| *** kfarr has quit IRC | 22:38 | |
| *** marst_ has joined #openstack-nova | 22:41 | |
| *** edmondsw has quit IRC | 22:42 | |
| *** thorst has joined #openstack-nova | 22:46 | |
| *** david-lyle has quit IRC | 22:51 | |
| *** thorst has quit IRC | 22:53 | |
| *** tlian has joined #openstack-nova | 22:58 | |
| *** jamesdenton has quit IRC | 23:00 | |
| *** mdrabe has quit IRC | 23:00 | |
| *** Swami has quit IRC | 23:02 | |
| *** david-lyle has joined #openstack-nova | 23:03 | |
| *** ducttape_ has quit IRC | 23:07 | |
| openstackgerrit | Anusha Unnam proposed openstack/nova: Add keypairs field to InstancePayload https://review.openstack.org/419730 | 23:09 |
| *** david-lyle has quit IRC | 23:10 | |
| *** gszasz has quit IRC | 23:14 | |
| *** salv-orlando has quit IRC | 23:21 | |
| *** edmondsw has joined #openstack-nova | 23:25 | |
| *** gouthamr has joined #openstack-nova | 23:27 | |
| *** sdague has quit IRC | 23:28 | |
| *** edmondsw has quit IRC | 23:30 | |
| *** marst_ has quit IRC | 23:36 | |
| *** esberglu has quit IRC | 23:44 | |
| *** esberglu has joined #openstack-nova | 23:45 | |
| *** esberglu has quit IRC | 23:49 | |
| *** dave-mccowan has joined #openstack-nova | 23:51 | |
| *** Kevin_Zheng has joined #openstack-nova | 23:54 | |
| *** diga has joined #openstack-nova | 23:57 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!