*** yangyape_ has quit IRC | 00:00 | |
openstackgerrit | Matthew Treinish proposed openstack/nova master: Remove cloudpipe APIs https://review.openstack.org/459470 | 00:00 |
---|---|---|
*** jamesdenton has quit IRC | 00:00 | |
*** yamahata has joined #openstack-nova | 00:02 | |
*** ssurana has joined #openstack-nova | 00:04 | |
*** eric_young has quit IRC | 00:04 | |
*** armax has quit IRC | 00:05 | |
*** eric_young has joined #openstack-nova | 00:06 | |
*** amotoki has quit IRC | 00:07 | |
*** yingjun has joined #openstack-nova | 00:08 | |
*** jamesdenton has joined #openstack-nova | 00:17 | |
*** xgerman has joined #openstack-nova | 00:20 | |
*** test222 has joined #openstack-nova | 00:22 | |
mriedem | dansmith: yeah gawd | 00:26 |
*** jamesdenton has quit IRC | 00:28 | |
*** gyee has quit IRC | 00:29 | |
*** zhurong has joined #openstack-nova | 00:31 | |
*** ijw has quit IRC | 00:34 | |
*** coreywright has joined #openstack-nova | 00:35 | |
mriedem | mtreinish: we shall see | 00:43 |
*** winston-d_ has joined #openstack-nova | 00:46 | |
*** tovin07_ has joined #openstack-nova | 00:46 | |
*** artom has quit IRC | 00:48 | |
*** artom has joined #openstack-nova | 00:49 | |
*** artom has quit IRC | 00:50 | |
*** artom has joined #openstack-nova | 00:50 | |
*** Jack_Iv has quit IRC | 00:51 | |
mriedem | mtreinish: hells yes | 00:53 |
mriedem | +2 | 00:53 |
mriedem | alex_xu_: oomichi: https://review.openstack.org/#/c/459470/6 | 00:53 |
*** cfriesen has joined #openstack-nova | 00:58 | |
*** phuongnh has joined #openstack-nova | 00:58 | |
*** abhishek has joined #openstack-nova | 01:00 | |
*** tuanluong has joined #openstack-nova | 01:02 | |
*** dave-mccowan has joined #openstack-nova | 01:03 | |
*** coreywright has quit IRC | 01:04 | |
*** abhishek has quit IRC | 01:07 | |
mtreinish | mriedem: awesome, thanks | 01:09 |
*** yangyapepeng has joined #openstack-nova | 01:11 | |
*** yangyapepeng has quit IRC | 01:11 | |
mtreinish | mriedem: I think once that merges it'll be my first nova bp implemented since the coverage extension back in grizzly | 01:11 |
mriedem | mtreinish: oh it's not done yet, | 01:11 |
mriedem | novaclient changes buddy | 01:11 |
* mtreinish rereads the spec | 01:12 | |
*** gjayavelu has quit IRC | 01:12 | |
mriedem | i don't think it mentions it | 01:12 |
mtreinish | it doesn't | 01:12 |
mriedem | certs was already deprecated in novaclient https://github.com/openstack/python-novaclient/commit/b6aea669630dc88296bdded963051a35cc9f995e | 01:13 |
mriedem | but cloudpipe wasn't https://github.com/openstack/python-novaclient/blob/master/novaclient/v2/cloudpipe.py | 01:13 |
mriedem | i can probably do that if you want, i've been doing a lot of that in novaclient lately | 01:13 |
mtreinish | mriedem: if you want, otherwise I'd push those up tomorrow | 01:13 |
*** yolkfull__ has joined #openstack-nova | 01:13 | |
mriedem | mtreinish: you should probably do it since it'll just enable me to procrastinate on summit prep | 01:14 |
mtreinish | mriedem: heh, what do you think it is for me? | 01:14 |
mriedem | don't use the certs one as an example, use the one that deprecated os-hosts in novaclient | 01:14 |
mriedem | use this as the example https://github.com/openstack/python-novaclient/commit/ea3b9f7fef28a7f06b51a3b176c85a1fc33cecaa | 01:15 |
mtreinish | ok | 01:15 |
*** nic has quit IRC | 01:15 | |
*** thorst has joined #openstack-nova | 01:15 | |
mriedem | mtreinish: there is a nova onboarding session at the summit you can attend too | 01:15 |
mriedem | since you're a new contributor again | 01:15 |
*** david-lyle has joined #openstack-nova | 01:16 | |
*** zhurong has quit IRC | 01:16 | |
mtreinish | mriedem: heh, I'll consider it | 01:17 |
mtreinish | I just looked my last nova commit before the cert removal was in feb. 2016 | 01:17 |
*** coreywright has joined #openstack-nova | 01:17 | |
mriedem | probably when you were doing the tenant isolation stuff | 01:18 |
mriedem | no, that was older | 01:18 |
mtreinish | mriedem: https://review.openstack.org/#/c/279799/ | 01:18 |
mriedem | ah i remember that | 01:19 |
*** cNilesh has joined #openstack-nova | 01:21 | |
*** Apoorva__ has quit IRC | 01:21 | |
mtreinish | mriedem: at some point this cycle I'm also going to work on adding docs on how to install nova (or any project) from a tarball, because there were no instructions I could find for that (something from my dirty clouds talk) | 01:22 |
*** Shunli has joined #openstack-nova | 01:23 | |
*** gcb has joined #openstack-nova | 01:23 | |
mriedem | ok | 01:23 |
*** Yangyape_ has joined #openstack-nova | 01:23 | |
mriedem | i noticed your talk was accepted but i think i have a conflict... | 01:23 |
mriedem | which sucks | 01:23 |
*** kaisers_ has joined #openstack-nova | 01:25 | |
openstackgerrit | Vladik Romanovsky proposed openstack/nova master: neutron: retrieve physical network name from a multi-provider network https://review.openstack.org/430026 | 01:26 |
*** zhurong has joined #openstack-nova | 01:26 | |
mtreinish | mriedem: oh well, there will always be the video | 01:28 |
mtreinish | mriedem: I linked you the spoiler picture of what the cloud looked like right? | 01:28 |
*** kaisers has quit IRC | 01:29 | |
vladikr | jaypipes, dansmith, thanks for the reviews. I've update the multi-segments patch (comments mostly), could you please take a look? https://review.openstack.org/430026 | 01:29 |
vladikr | s/update/updated | 01:29 |
mriedem | mtreinish: i think so yeah | 01:30 |
jaypipes | vladikr: done! | 01:30 |
vladikr | jaypipes, thanks! | 01:31 |
jaypipes | np :) | 01:31 |
*** yamamoto has joined #openstack-nova | 01:31 | |
*** cfriesen has quit IRC | 01:31 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Use plain routes list for server-password endpoint instead of stevedore https://review.openstack.org/462049 | 01:32 |
*** dixiaoli has joined #openstack-nova | 01:33 | |
*** david-lyle has quit IRC | 01:34 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Use plain routes list for server-password endpoint instead of stevedore https://review.openstack.org/462049 | 01:35 |
*** yamamoto has quit IRC | 01:35 | |
*** litao has joined #openstack-nova | 01:41 | |
mriedem | Kevin_Zheng: do you want to rebase this? https://review.openstack.org/#/c/459593/ | 01:41 |
*** mingyu has joined #openstack-nova | 01:41 | |
mriedem | i suppose i can in gerrit | 01:42 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Support tag instances when boot(3/4) https://review.openstack.org/459593 | 01:42 |
Kevin_Zheng | mriedem sure, but why? | 01:42 |
mriedem | just because it messes up the patch set order when going through the changes in gerrit if it's not rebased | 01:42 |
mriedem | nothing major | 01:43 |
Kevin_Zheng | AH, OK, I was going to ask you to review this one if you have time | 01:43 |
mriedem | i can run through it quick | 01:44 |
*** mingyu has quit IRC | 01:46 | |
Kevin_Zheng | One question | 01:46 |
Kevin_Zheng | if we are using placement, which is mandatory in O, it seems that we are unable to use AggregateCoreFilter properly | 01:48 |
*** gjayavelu has joined #openstack-nova | 01:48 | |
*** ijw has joined #openstack-nova | 01:49 | |
*** ijw has quit IRC | 01:54 | |
*** masber has quit IRC | 01:56 | |
mriedem | Kevin_Zheng: i'm not familiar with that filter; comments in your tags 3/4 patch btw | 01:57 |
mriedem | edleafe: bauzas: jaypipes: ^ re: placement + AggregateCoreFilter | 01:57 |
jaypipes | Kevin_Zheng: that filter wasn't touched by the placement service. | 01:58 |
mriedem | i don't think that's the question | 01:58 |
jaypipes | mriedem: I don't see a question... | 01:58 |
mriedem | this is a host aggregate with cpu_allocation_ratio meta set on it? | 01:58 |
mriedem | is that the idea? | 01:59 |
mriedem | jaypipes: so with placement in the filter scheduler we basically ignore the corefilter because we get compute node RPs via the VCPU resource class filtering right? | 01:59 |
openstackgerrit | fengzhr proposed openstack/nova master: Modify the description of flat_injected in nova.conf https://review.openstack.org/458789 | 01:59 |
mriedem | we don't ignore it, but it's redundant | 01:59 |
mriedem | and i think we said you could/should disable the CoreFilter, RAMFilter and DiskFilter once you're using placement | 01:59 |
jaypipes | yeah... | 02:00 |
jaypipes | but aggregate metadata should still be processed, IIRC. | 02:00 |
mriedem | yeah i'm not sure why this would be a problem, | 02:01 |
mriedem | it's just extra filtering once we get the list of compute node resource providers filtered on VCPU to start | 02:01 |
*** marst has joined #openstack-nova | 02:01 | |
mriedem | Kevin_Zheng: it's probably best to just report a bug with details | 02:01 |
mriedem | especially because i have a date with my couch and tv | 02:01 |
Kevin_Zheng | ha, i will | 02:02 |
mriedem | good night all | 02:03 |
mriedem | and day | 02:03 |
*** mriedem has quit IRC | 02:03 | |
*** salv-orlando has joined #openstack-nova | 02:05 | |
Kevin_Zheng | As described in aggregatecorefilter it will try to get ratio from aggregate first and if not possible then fall back to per host ratio | 02:05 |
*** cfriesen has joined #openstack-nova | 02:07 | |
Kevin_Zheng | If we use placement, then it directly selected hosts with per host ratio, if we have different configurations in aggregate meta, then the host selection might not be what we expected | 02:08 |
*** salv-orlando has quit IRC | 02:09 | |
Kevin_Zheng | Say, if people only use aggregatecorefilter as it will fall back to per host ratio, it won't work with placement | 02:10 |
*** dave-mccowan has quit IRC | 02:10 | |
*** gjayavelu has quit IRC | 02:12 | |
*** yolkfull has joined #openstack-nova | 02:13 | |
*** yolkfull_ has joined #openstack-nova | 02:15 | |
*** yolkfull__ has quit IRC | 02:16 | |
*** yolkfull__ has joined #openstack-nova | 02:17 | |
*** yolkfull has quit IRC | 02:18 | |
*** yolkfull_ has quit IRC | 02:19 | |
*** dave-mccowan has joined #openstack-nova | 02:20 | |
openstackgerrit | shaofeng cheng proposed openstack/nova master: Update nova version newton to ocata in upgrade.rst https://review.openstack.org/457197 | 02:22 |
*** yolkfull__ has quit IRC | 02:23 | |
*** yolkfull__ has joined #openstack-nova | 02:23 | |
*** zhurong has quit IRC | 02:34 | |
*** yamamoto has joined #openstack-nova | 02:36 | |
*** zhurong has joined #openstack-nova | 02:36 | |
*** markvoelker has quit IRC | 02:40 | |
*** ssurana has quit IRC | 02:40 | |
*** thorst has quit IRC | 02:41 | |
*** yamamoto has quit IRC | 02:46 | |
*** fragatina has joined #openstack-nova | 02:51 | |
*** dave-mccowan has quit IRC | 02:51 | |
*** Jack_Iv has joined #openstack-nova | 02:51 | |
*** masber has joined #openstack-nova | 02:52 | |
*** fragatina has quit IRC | 02:53 | |
*** david-lyle has joined #openstack-nova | 02:53 | |
*** hongbin has joined #openstack-nova | 02:53 | |
*** fragatina has joined #openstack-nova | 02:53 | |
*** fragatin_ has quit IRC | 02:54 | |
*** gjayavelu has joined #openstack-nova | 02:55 | |
*** Jack_Iv has quit IRC | 02:56 | |
*** lucasxu has joined #openstack-nova | 02:58 | |
*** nicolasbock has quit IRC | 02:58 | |
*** vladikr has quit IRC | 02:59 | |
*** lucasxu has quit IRC | 03:00 | |
*** fragatin_ has joined #openstack-nova | 03:01 | |
*** john51 has joined #openstack-nova | 03:03 | |
*** sbezverk_ has joined #openstack-nova | 03:04 | |
*** manjeets__ has joined #openstack-nova | 03:04 | |
*** fragatina has quit IRC | 03:04 | |
*** dimtruck is now known as zz_dimtruck | 03:05 | |
*** iceyao_ has joined #openstack-nova | 03:05 | |
*** crushil_ has joined #openstack-nova | 03:05 | |
*** fragatin_ has quit IRC | 03:05 | |
*** salv-orlando has joined #openstack-nova | 03:06 | |
*** raorn has joined #openstack-nova | 03:07 | |
*** slagle__ has joined #openstack-nova | 03:07 | |
*** belliott_ has joined #openstack-nova | 03:07 | |
*** jgrassler has joined #openstack-nova | 03:07 | |
*** gaurangt_ has joined #openstack-nova | 03:10 | |
*** zsli_ has joined #openstack-nova | 03:10 | |
*** irenab_ has joined #openstack-nova | 03:10 | |
*** salv-orlando has quit IRC | 03:11 | |
*** rha_ has joined #openstack-nova | 03:11 | |
*** test222 has quit IRC | 03:11 | |
*** iceyao has quit IRC | 03:11 | |
*** slagle has quit IRC | 03:11 | |
*** rodrigod` has quit IRC | 03:11 | |
*** irenab has quit IRC | 03:11 | |
*** mariojv_ has quit IRC | 03:11 | |
*** Guest73837 has quit IRC | 03:11 | |
*** jgrassle1 has quit IRC | 03:11 | |
*** raorn_ has quit IRC | 03:11 | |
*** crushil has quit IRC | 03:11 | |
*** jaypipes has quit IRC | 03:11 | |
*** john51_ has quit IRC | 03:11 | |
*** gaurangt has quit IRC | 03:11 | |
*** ekuris has quit IRC | 03:11 | |
*** dillaman has quit IRC | 03:11 | |
*** sbezverk has quit IRC | 03:11 | |
*** hamzy has quit IRC | 03:11 | |
*** Kevin_Zheng has quit IRC | 03:11 | |
*** manjeets has quit IRC | 03:11 | |
*** belliott has quit IRC | 03:11 | |
*** mdavidson has quit IRC | 03:11 | |
*** rha has quit IRC | 03:11 | |
*** irenab_ is now known as irenab | 03:11 | |
*** rodrigods has joined #openstack-nova | 03:11 | |
*** Shunli has quit IRC | 03:12 | |
*** tovin07_ has quit IRC | 03:16 | |
*** tovin07_ has joined #openstack-nova | 03:17 | |
*** trungnv has quit IRC | 03:18 | |
*** Guest73837 has joined #openstack-nova | 03:18 | |
*** mdavidson has joined #openstack-nova | 03:18 | |
*** trungnv has joined #openstack-nova | 03:18 | |
*** ekuris has joined #openstack-nova | 03:18 | |
openstackgerrit | Jackie Truong proposed openstack/nova master: Add missing rootwrap filter for cryptsetup https://review.openstack.org/462348 | 03:18 |
*** mariojv_ has joined #openstack-nova | 03:18 | |
*** hamzy has joined #openstack-nova | 03:19 | |
*** dillaman has joined #openstack-nova | 03:19 | |
*** jaypipes has joined #openstack-nova | 03:19 | |
*** jaypipes has quit IRC | 03:19 | |
*** Kevin_Zheng has joined #openstack-nova | 03:24 | |
*** masber has quit IRC | 03:24 | |
*** dixiaoli has quit IRC | 03:30 | |
*** armax has joined #openstack-nova | 03:31 | |
*** mingyu has joined #openstack-nova | 03:32 | |
*** gongysh has joined #openstack-nova | 03:32 | |
openstackgerrit | chenaidong1 proposed openstack/nova master: Fix a typo https://review.openstack.org/462349 | 03:33 |
*** mingyu has quit IRC | 03:38 | |
openstackgerrit | chenaidong1 proposed openstack/python-novaclient master: Fix a typo https://review.openstack.org/462351 | 03:38 |
*** thorst has joined #openstack-nova | 03:41 | |
*** markvoelker has joined #openstack-nova | 03:44 | |
*** thorst has quit IRC | 03:45 | |
*** gouthamr has quit IRC | 03:58 | |
*** iceyao_ has quit IRC | 03:59 | |
*** armax has quit IRC | 03:59 | |
openstackgerrit | LiChunlin proposed openstack/nova master: update the description of hypervisor statistics response https://review.openstack.org/462355 | 04:03 |
*** abhishek has joined #openstack-nova | 04:05 | |
*** salv-orlando has joined #openstack-nova | 04:07 | |
*** sridharg has joined #openstack-nova | 04:08 | |
*** masber has joined #openstack-nova | 04:11 | |
*** salv-orlando has quit IRC | 04:11 | |
*** zhurong has quit IRC | 04:12 | |
*** thorst has joined #openstack-nova | 04:12 | |
*** ssurana has joined #openstack-nova | 04:12 | |
*** adisky_ has joined #openstack-nova | 04:13 | |
*** trinaths has joined #openstack-nova | 04:17 | |
*** xinliang has quit IRC | 04:20 | |
*** abhishek has quit IRC | 04:23 | |
*** thorst has quit IRC | 04:30 | |
*** xinliang has joined #openstack-nova | 04:33 | |
*** zhurong has joined #openstack-nova | 04:35 | |
*** yamahata has quit IRC | 04:35 | |
*** mingyu has joined #openstack-nova | 04:36 | |
*** Sukhdev has joined #openstack-nova | 04:36 | |
*** fragatina has joined #openstack-nova | 04:39 | |
*** fragatina has quit IRC | 04:40 | |
*** fragatina has joined #openstack-nova | 04:40 | |
*** mingyu has quit IRC | 04:41 | |
*** slaweq has quit IRC | 04:46 | |
*** ayogi has joined #openstack-nova | 04:47 | |
*** salv-orlando has joined #openstack-nova | 04:52 | |
openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova master: Remove oslo.config deprecated parameter enforce_type https://review.openstack.org/462364 | 04:52 |
*** hongbin has quit IRC | 04:56 | |
*** iceyao has joined #openstack-nova | 04:56 | |
*** ratailor has joined #openstack-nova | 04:57 | |
*** cfriesen has quit IRC | 04:59 | |
*** tjones has quit IRC | 04:59 | |
*** iceyao has quit IRC | 05:03 | |
*** Sukhdev has quit IRC | 05:05 | |
*** arunman has joined #openstack-nova | 05:08 | |
*** vks1 has joined #openstack-nova | 05:08 | |
*** marst has quit IRC | 05:11 | |
*** tjones has joined #openstack-nova | 05:11 | |
*** baoli has quit IRC | 05:12 | |
*** test222 has joined #openstack-nova | 05:14 | |
*** prateek has joined #openstack-nova | 05:16 | |
*** yamamoto has joined #openstack-nova | 05:18 | |
*** links has joined #openstack-nova | 05:18 | |
*** yamamoto has quit IRC | 05:21 | |
*** diga has joined #openstack-nova | 05:22 | |
*** trungnv has quit IRC | 05:38 | |
*** Dinesh_Bhor has quit IRC | 05:41 | |
*** abhi89 has joined #openstack-nova | 05:41 | |
*** Dinesh_Bhor has joined #openstack-nova | 05:41 | |
*** Dinesh_Bhor has quit IRC | 05:43 | |
*** irenab has left #openstack-nova | 05:44 | |
*** Dinesh_Bhor has joined #openstack-nova | 05:48 | |
*** trungnv has joined #openstack-nova | 05:48 | |
*** sree has joined #openstack-nova | 05:51 | |
*** trungnv has quit IRC | 05:51 | |
*** Jack_Iv has joined #openstack-nova | 05:52 | |
*** Jack_Iv has quit IRC | 05:56 | |
*** trungnv has joined #openstack-nova | 06:01 | |
*** kiwi_rot has joined #openstack-nova | 06:02 | |
*** salv-orlando has quit IRC | 06:04 | |
*** amotoki has joined #openstack-nova | 06:05 | |
*** mdnadeem has joined #openstack-nova | 06:07 | |
*** rajathagasthya has joined #openstack-nova | 06:09 | |
*** iceyao has joined #openstack-nova | 06:10 | |
*** mingyu has joined #openstack-nova | 06:11 | |
*** coreywright has quit IRC | 06:14 | |
*** avolkov has joined #openstack-nova | 06:14 | |
*** rajathagasthya has quit IRC | 06:15 | |
*** rajathagasthya has joined #openstack-nova | 06:15 | |
*** mingyu has quit IRC | 06:16 | |
*** trinaths has quit IRC | 06:18 | |
*** brault has quit IRC | 06:19 | |
*** rajathagasthya has quit IRC | 06:21 | |
*** yamamoto has joined #openstack-nova | 06:21 | |
*** trinaths has joined #openstack-nova | 06:23 | |
*** voelzmo has joined #openstack-nova | 06:23 | |
*** sbezverk_ has quit IRC | 06:23 | |
*** udesale has joined #openstack-nova | 06:24 | |
*** mingyu has joined #openstack-nova | 06:25 | |
*** yamamoto has quit IRC | 06:26 | |
*** yamamoto has joined #openstack-nova | 06:26 | |
*** thorst has joined #openstack-nova | 06:27 | |
*** yamamoto has quit IRC | 06:28 | |
*** andreas_s has joined #openstack-nova | 06:28 | |
*** Oku_OS-away is now known as Oku_OS | 06:28 | |
*** kiwi_rot has quit IRC | 06:30 | |
*** kiwi_rot has joined #openstack-nova | 06:30 | |
*** coreywright has joined #openstack-nova | 06:32 | |
*** thorst has quit IRC | 06:32 | |
*** ltomasbo|away is now known as ltomasbo | 06:37 | |
*** mingyu has quit IRC | 06:41 | |
*** mingyu has joined #openstack-nova | 06:42 | |
*** rmart04_ has joined #openstack-nova | 06:42 | |
*** pcaruana has joined #openstack-nova | 06:45 | |
*** mnestratov has joined #openstack-nova | 06:45 | |
*** slaweq has joined #openstack-nova | 06:47 | |
*** rha_ is now known as rha | 06:51 | |
*** rha has joined #openstack-nova | 06:51 | |
*** slaweq has quit IRC | 06:52 | |
*** arunman has quit IRC | 06:52 | |
*** mingyu has quit IRC | 06:55 | |
*** mingyu has joined #openstack-nova | 06:56 | |
*** yamamoto has joined #openstack-nova | 06:59 | |
*** Jack_Iv has joined #openstack-nova | 07:00 | |
*** Jack_Iv has quit IRC | 07:00 | |
*** Jack_Iv has joined #openstack-nova | 07:00 | |
*** damien_r has quit IRC | 07:01 | |
*** arunman has joined #openstack-nova | 07:02 | |
*** damien_r has joined #openstack-nova | 07:04 | |
*** Jack_Iv has quit IRC | 07:04 | |
*** moshele has joined #openstack-nova | 07:04 | |
*** jamielennox is now known as jamielennox|away | 07:06 | |
*** yamamoto has quit IRC | 07:09 | |
*** iceyao has quit IRC | 07:11 | |
*** damien_r1 has joined #openstack-nova | 07:13 | |
*** damien_r has quit IRC | 07:13 | |
*** iceyao has joined #openstack-nova | 07:15 | |
*** ralonsoh has joined #openstack-nova | 07:17 | |
*** ijw has joined #openstack-nova | 07:19 | |
*** iceyao has quit IRC | 07:20 | |
*** iceyao has joined #openstack-nova | 07:24 | |
*** aarefiev_afk is now known as aarefiev | 07:24 | |
*** ijw has quit IRC | 07:24 | |
*** salv-orlando has joined #openstack-nova | 07:24 | |
openstackgerrit | jichenjc proposed openstack/nova master: Use plain routes list for os-instance-usage-audit-log endpoint instead of stevedore https://review.openstack.org/461647 | 07:25 |
*** Jack_Iv has joined #openstack-nova | 07:26 | |
openstackgerrit | jichenjc proposed openstack/nova master: use plain routes list for os-simple-tenant-usage https://review.openstack.org/460821 | 07:27 |
*** thorst has joined #openstack-nova | 07:28 | |
*** tesseract has joined #openstack-nova | 07:29 | |
*** thorst has quit IRC | 07:33 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Use plain routes list for os-server-groups endpoint instead of stevedore https://review.openstack.org/462169 | 07:33 |
openstackgerrit | Andreas Scheuring proposed openstack/nova master: DO NOT MERGE https://review.openstack.org/462421 | 07:38 |
*** jpena|off is now known as jpena | 07:40 | |
*** Yangyape_ has quit IRC | 07:45 | |
*** Yangyape_ has joined #openstack-nova | 07:46 | |
openstackgerrit | jichenjc proposed openstack/nova master: Use plain routes list for os-quota-classes endpoint instead of stevedore https://review.openstack.org/461641 | 07:48 |
*** ssurana has quit IRC | 07:48 | |
openstackgerrit | jichenjc proposed openstack/nova master: refactory some code on get_password https://review.openstack.org/449458 | 07:49 |
*** karimb has joined #openstack-nova | 07:49 | |
*** rmart04_ has quit IRC | 07:49 | |
*** lpetrut has joined #openstack-nova | 07:51 | |
openstackgerrit | Alex Xu proposed openstack/nova master: Remove cloudpipe APIs https://review.openstack.org/459470 | 07:52 |
openstackgerrit | Alex Xu proposed openstack/nova master: Remove cloudpipe APIs https://review.openstack.org/459470 | 07:53 |
*** markus_z has joined #openstack-nova | 07:59 | |
*** zzzeek has quit IRC | 08:00 | |
*** zzzeek has joined #openstack-nova | 08:00 | |
*** zsli_ has quit IRC | 08:06 | |
*** zsli_ has joined #openstack-nova | 08:07 | |
*** udesale has quit IRC | 08:11 | |
*** gcb has quit IRC | 08:12 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Use plain routes list for server-password endpoint instead of stevedore https://review.openstack.org/462049 | 08:13 |
*** lucas-afk is now known as lucasagomes | 08:13 | |
*** udesale has joined #openstack-nova | 08:14 | |
*** rajathagasthya has joined #openstack-nova | 08:14 | |
*** jaosorior_away is now known as jaosorior | 08:14 | |
*** rmart04 has joined #openstack-nova | 08:15 | |
*** rajathagasthya has quit IRC | 08:15 | |
sfinucan | mdbooth: Could you take a look at https://review.openstack.org/#/q/topic:disk_info_processing today? It's been around a long time and it seems like something you should look at first | 08:19 |
*** brault has joined #openstack-nova | 08:24 | |
*** trinaths has quit IRC | 08:25 | |
openstackgerrit | shaofeng cheng proposed openstack/nova master: Replace newton to release_name in upgrade.rst https://review.openstack.org/457197 | 08:26 |
*** karimb has quit IRC | 08:28 | |
*** thorst has joined #openstack-nova | 08:29 | |
*** gjayavelu has quit IRC | 08:31 | |
*** derekh has joined #openstack-nova | 08:36 | |
*** adreznec has quit IRC | 08:45 | |
*** adreznec has joined #openstack-nova | 08:47 | |
openstackgerrit | jichenjc proposed openstack/nova master: [placement] Add POST for resource provider classes https://review.openstack.org/462441 | 08:48 |
*** slaweq has joined #openstack-nova | 08:48 | |
*** thorst has quit IRC | 08:49 | |
sfinucan | johnthetubaguy: Are we allowed to delete configuration without a deprecation cycle, even if they do nothing? https://review.openstack.org/#/c/459470/ | 08:49 |
*** gcb has joined #openstack-nova | 08:50 | |
johnthetubaguy | sfinucan: so cloudpipe only works with nova-network, and nova-network has been deprecated (on and off) for a long time | 08:50 |
johnthetubaguy | sfinucan: I would give that one a pass | 08:50 |
*** karimb has joined #openstack-nova | 08:50 | |
sfinucan | johnthetubaguy: Cool. Sounds fair | 08:51 |
*** ratailor_ has joined #openstack-nova | 08:52 | |
*** slaweq has quit IRC | 08:53 | |
openstackgerrit | jichenjc proposed openstack/nova master: [placement] Add POST for resource provider classes https://review.openstack.org/462441 | 08:54 |
*** ratailor has quit IRC | 08:55 | |
*** salv-orl_ has joined #openstack-nova | 08:56 | |
johnthetubaguy | sfinucan: +1 deleting that already deprecated option that does nothing now | 08:56 |
abhi89 | johnthetubaguy: can you please review https://review.openstack.org/#/c/460305/ | 08:56 |
johnthetubaguy | abhi89: ah, I should take a look at that now | 08:57 |
*** salv-orlando has quit IRC | 08:59 | |
*** karimb has quit IRC | 09:00 | |
*** sridharg has quit IRC | 09:00 | |
*** sree_ has joined #openstack-nova | 09:00 | |
*** sree_ is now known as Guest97550 | 09:01 | |
*** zsli_ has quit IRC | 09:01 | |
johnthetubaguy | abhi89: I really like that regression test now, really nice proof that your change works | 09:02 |
*** karimb has joined #openstack-nova | 09:02 | |
*** Shunli has joined #openstack-nova | 09:03 | |
*** sree has quit IRC | 09:03 | |
*** Shunli has quit IRC | 09:03 | |
abhi89 | johnthetubaguy: yes.. i have well tested the changes also | 09:04 |
johnthetubaguy | abhi89: I just love seeing the automated tests like that, really stops us breaking your fix by accident later | 09:04 |
*** Guest7024 has joined #openstack-nova | 09:05 | |
abhi89 | johnthetubaguy: yes.. Thanks John | 09:05 |
johnthetubaguy | abhi89: thank you for sticking with it, great to get that fixed | 09:06 |
*** zhurong has quit IRC | 09:06 | |
sfinucan | johnthetubaguy: Could you take a punt at https://review.openstack.org/#/c/431037/ and successors today? | 09:10 |
johnthetubaguy | sfinucan: yeah, I want to sit down with the quota code again at some point though | 09:11 |
openstackgerrit | falseuser proposed openstack/nova master: [BugFix] Change the parameter of the exception error message https://review.openstack.org/461985 | 09:11 |
*** djohnsto has joined #openstack-nova | 09:11 | |
sfinucan | johnthetubaguy: ah, at some point then :) | 09:14 |
*** zhurong has joined #openstack-nova | 09:15 | |
*** yamamoto has joined #openstack-nova | 09:15 | |
*** Guest7024 is now known as Frecy2 | 09:17 | |
*** Shunli has joined #openstack-nova | 09:18 | |
*** moshele has quit IRC | 09:18 | |
*** tovin07 has quit IRC | 09:18 | |
*** yamamoto has quit IRC | 09:19 | |
sfinucan | johnthetubaguy: Ta :) | 09:24 |
*** markvoelker has quit IRC | 09:25 | |
*** cdent has joined #openstack-nova | 09:29 | |
*** rmart04 has quit IRC | 09:29 | |
*** purplerbot has joined #openstack-nova | 09:29 | |
*** yingjun has quit IRC | 09:31 | |
*** iceyao has quit IRC | 09:36 | |
*** aweeks has quit IRC | 09:38 | |
*** aweeks has joined #openstack-nova | 09:38 | |
*** test222 has quit IRC | 09:39 | |
*** test222 has joined #openstack-nova | 09:41 | |
*** lpetrut_ has joined #openstack-nova | 09:41 | |
*** lpetrut has quit IRC | 09:41 | |
*** lpetrut_ has quit IRC | 09:42 | |
*** lpetrut_ has joined #openstack-nova | 09:42 | |
*** links has quit IRC | 09:45 | |
*** karimb has quit IRC | 09:47 | |
*** karimb has joined #openstack-nova | 09:50 | |
*** cNilesh has quit IRC | 09:57 | |
openstackgerrit | Stephen Finucane proposed openstack/nova master: Fix default_availability_zone docs https://review.openstack.org/438887 | 09:58 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Move availability zones opts to a group https://review.openstack.org/462469 | 09:58 |
*** moshele has joined #openstack-nova | 09:59 | |
*** iceyao has joined #openstack-nova | 10:02 | |
*** mvk has quit IRC | 10:03 | |
*** Jack_Iv has quit IRC | 10:04 | |
*** tovin07_ has quit IRC | 10:04 | |
*** Jack_Iv has joined #openstack-nova | 10:05 | |
*** iceyao has quit IRC | 10:06 | |
*** zhurong has quit IRC | 10:07 | |
*** nicolasbock has joined #openstack-nova | 10:08 | |
*** iceyao has joined #openstack-nova | 10:10 | |
*** sdague has joined #openstack-nova | 10:11 | |
openstackgerrit | Stephen Finucane proposed openstack/nova master: Add PCIWeigher https://review.openstack.org/379524 | 10:13 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: Prefer non-PCI host nodes for non-PCI instances https://review.openstack.org/379625 | 10:13 |
*** mingyu has quit IRC | 10:13 | |
openstackgerrit | Stephen Finucane proposed openstack/nova master: tools/xenserver: Remove 'cleanup_sm_locks' https://review.openstack.org/416520 | 10:14 |
sfinucan | bauzas: Fancy taking a look at two availability zone changes? https://review.openstack.org/438887 https://review.openstack.org/462469 | 10:15 |
*** mingyu has joined #openstack-nova | 10:15 | |
*** adisky_ has quit IRC | 10:19 | |
*** adisky_ has joined #openstack-nova | 10:20 | |
*** zhurong has joined #openstack-nova | 10:24 | |
*** markvoelker has joined #openstack-nova | 10:26 | |
*** tuanluong has quit IRC | 10:28 | |
*** gongysh has quit IRC | 10:30 | |
*** mvk has joined #openstack-nova | 10:30 | |
*** arunman has quit IRC | 10:31 | |
*** markvoelker has quit IRC | 10:33 | |
*** arunman has joined #openstack-nova | 10:33 | |
*** phuongnh has quit IRC | 10:36 | |
*** mingyu has quit IRC | 10:36 | |
*** phuongnh has joined #openstack-nova | 10:38 | |
*** tbachman has quit IRC | 10:43 | |
*** stian__ has quit IRC | 10:44 | |
*** ratailor_ has quit IRC | 10:46 | |
*** thorst has joined #openstack-nova | 10:46 | |
*** wxy has quit IRC | 10:48 | |
*** brault_ has joined #openstack-nova | 10:48 | |
*** wxy has joined #openstack-nova | 10:48 | |
*** mrhillsman has quit IRC | 10:48 | |
*** phuongnh has quit IRC | 10:48 | |
*** smatzek has joined #openstack-nova | 10:48 | |
*** brault has quit IRC | 10:49 | |
*** mrhillsman has joined #openstack-nova | 10:49 | |
*** slaweq has joined #openstack-nova | 10:49 | |
*** mingyu has joined #openstack-nova | 10:49 | |
*** thorst has quit IRC | 10:51 | |
*** slaweq has quit IRC | 10:53 | |
*** salv-orl_ has quit IRC | 10:55 | |
*** ijw has joined #openstack-nova | 10:56 | |
*** yolkfull__ has quit IRC | 10:58 | |
*** ijw has quit IRC | 11:01 | |
*** Frecy2 has left #openstack-nova | 11:04 | |
*** jpena is now known as jpena|lunch | 11:04 | |
*** dave-mccowan has joined #openstack-nova | 11:08 | |
*** nicolasbock has quit IRC | 11:12 | |
*** ociuhandu has joined #openstack-nova | 11:12 | |
*** abhi89 has quit IRC | 11:14 | |
*** Jack_Iv has quit IRC | 11:15 | |
*** edmondsw has quit IRC | 11:16 | |
openstackgerrit | Matthew Booth proposed openstack/nova master: Fix scope of instance error context manager in resize_instance https://review.openstack.org/462499 | 11:16 |
mdbooth | Hopefully pretty simple bugfix ^^^ | 11:16 |
sean-k-mooney | hi | 11:20 |
sean-k-mooney | is there a way to make devstack not use systemd still | 11:21 |
mdbooth | sean-k-mooney: That got pushed yesterday, I think. | 11:21 |
mdbooth | https://review.openstack.org/#/c/462163/3 | 11:21 |
sean-k-mooney | swapping to systemd or keeping the legacy way | 11:21 |
mdbooth | Keeps systemd option, but defaults to legacy | 11:22 |
*** diga has quit IRC | 11:22 | |
sean-k-mooney | i pulled devstack today and it defaulted to systemd instead of screen | 11:22 |
sean-k-mooney | but ok ill look into it. i hate systemd... this might actully be the thing that makes me stop using devstack for my development | 11:23 |
*** dpawlik has joined #openstack-nova | 11:24 | |
sean-k-mooney | ah i have to set USE_SCREEN=True to get the old behavior | 11:24 |
dpawlik | Hello. Can someone of you tell me if I should add DocImpact to the commit message in this PS: https://review.openstack.org/#/c/459753/ ? | 11:25 |
sean-k-mooney | dpawlik: you will need to update the docs for the new image metadata item but i belive that would be in glance's docs not novas | 11:27 |
dpawlik | sean-k-mooney: hey. So better when I add here DocImpact and ask glance team to add it in its docs | 11:28 |
*** karimb has quit IRC | 11:29 | |
sean-k-mooney | dpawlik: https://wiki.openstack.org/wiki/Documentation/DocImpact | 11:30 |
dpawlik | sean-k-mooney: thx | 11:30 |
*** thorst has joined #openstack-nova | 11:33 | |
*** vks1 has quit IRC | 11:33 | |
*** Shunli has quit IRC | 11:34 | |
*** nicolasbock has joined #openstack-nova | 11:34 | |
*** gszasz has joined #openstack-nova | 11:36 | |
*** sbezverk has joined #openstack-nova | 11:40 | |
openstackgerrit | Daniel Pawlik proposed openstack/nova master: Added kvm_hidden feature https://review.openstack.org/459753 | 11:44 |
sean-k-mooney | sdague: hi, 2 quick quetstions regarding the new systemd workflow for devstack will the screen workflow be tested and maintined going forword? also will devstack now work across reboots again? | 11:45 |
*** mingyu has quit IRC | 11:45 | |
*** mingyu has joined #openstack-nova | 11:45 | |
*** catintheroof has joined #openstack-nova | 11:47 | |
sdague | sean-k-mooney: screen goes away in Queens | 11:49 |
sdague | http://lists.openstack.org/pipermail/openstack-dev/2017-May/116301.html | 11:50 |
sdague | I'm not sure what works across reboots means | 11:50 |
* mdbooth wonders if the library which writes to the systemd journal drops messages if it would block | 11:50 | |
sdague | mdbooth: that's entirely possible... | 11:50 |
mdbooth | I don't think the journal itself drops messages | 11:50 |
sdague | mdbooth: we did disable it now, so we're out of that woods for the moment | 11:51 |
mdbooth | Yeah, makes sense | 11:51 |
mdbooth | One to investigate though, for sure | 11:51 |
*** catintheroof has quit IRC | 11:51 | |
sean-k-mooney | sdague: ok i might need to make changes to our devstack plugins/CIs. | 11:53 |
*** Yangyape_ has quit IRC | 11:53 | |
sean-k-mooney | i think they will work but they may not | 11:54 |
sdague | sean-k-mooney: yeh, that thread has some common issues people hit | 11:54 |
*** salv-orlando has joined #openstack-nova | 11:54 | |
sean-k-mooney | sdague: i guess il have to give it a fair shot but honestly the fact that the process ran in screen sessions was one of the things i liked about devstack as it made it easy to wrap your head around when you were new to openstack | 11:56 |
kashyap | sean-k-mooney: It isn't any more? I haven't run a DevStack run in a couple of weeks | 11:57 |
sdague | sean-k-mooney: ok, an equally large number of people have stated that screen was the most confusing thing ever | 11:57 |
*** jpena|lunch is now known as jpena | 11:57 | |
*** kiwi_rot has quit IRC | 11:57 | |
sdague | but, regardless, there are other reasons we need this change | 11:57 |
kashyap | I've seen the systemd / 'screen' related change whoosh by. /me looks closer... | 11:57 |
kashyap | Hmm, it's this patch: "make USE_SCREEN=False imply USE_SYSTEMD=True" | 11:57 |
*** Guest97550 has quit IRC | 11:57 | |
*** mingyu has quit IRC | 11:58 | |
*** tbachman has joined #openstack-nova | 11:58 | |
*** sree has joined #openstack-nova | 11:58 | |
sean-k-mooney | yes i swapped the flag back and it still failed to stack so my local failure wasnt related to the systemd change so im going to start with a clean environment again | 11:58 |
sean-k-mooney | kashyap: ^ | 11:59 |
kashyap | sean-k-mooney: What is the implication of that patch? Still reading this - https://review.openstack.org/#/c/460062/ | 11:59 |
sean-k-mooney | sdague: screen not being maintianed for one | 11:59 |
sean-k-mooney | kashyap: the openstack service deployed by devstack are now executed via systemd instead of in screen with there logs sent to the journal | 12:00 |
kashyap | sean-k-mooney: Hmm. I like systemd journal logs (for its nice 'query-ability'), but I miss firing up a quick `screen -r` to flip through all the services | 12:01 |
sean-k-mooney | kashyap: that and quickly running a service in a debugger or viewing the live logs though journalctl supports that too. | 12:02 |
kashyap | Hmm. What is the rationale for the "feature"? | 12:02 |
kashyap | I'd agree with mtreinish there, where he says: | 12:03 |
*** sree has quit IRC | 12:03 | |
kashyap | "I think expanding this comment a bit to say the intent here would have been nice, since I got confused by this at first." | 12:03 |
sean-k-mooney | kashyap: make test closer to production | 12:03 |
* kashyap nods; that sounds reasonable | 12:04 | |
sdague | kashyap: all sorts of stuff on the mailing list over the last month, I'd rather not just reiterate all that here | 12:04 |
kashyap | Then we can _really_ claim: "If it works on DevStack, it's all kosher" :P | 12:04 |
*** yamahata has joined #openstack-nova | 12:04 | |
sean-k-mooney | kashyap: haha sure... | 12:04 |
kashyap | sdague: Yeah, sorry, I'd go catch up the thread. I did see the mail, but spent all of 2 seconds on it. | 12:04 |
sean-k-mooney | sdague: if devstack works across reboots again this would be cool sideffect but it might cause some people to swap to other tools for dev. | 12:06 |
openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova master: Use check_string_length from oslo_utils https://review.openstack.org/286390 | 12:07 |
kashyap | sean-k-mooney: But _does_ it work cope with reboots with this patch? If so, that'd be nice | 12:07 |
sean-k-mooney | sdague: anyway thanks for trying to keep it working for plugins. at least 1 of your plugins seeam to be working fine with the change ill check the otheres when i get time or delegate to the rest of my team :) | 12:07 |
*** udesale has quit IRC | 12:07 | |
sean-k-mooney | kashyap: dont know havent tried it | 12:07 |
sean-k-mooney | i miss rejoin stack for that abiltiy it was really useful for demoing stuff but it also had lots of issues if you didnt know how it worked | 12:08 |
*** mingyu has joined #openstack-nova | 12:09 | |
openstackgerrit | Matthew Booth proposed openstack/nova master: WIP: Automatically revert resize which fails on destination https://review.openstack.org/462521 | 12:10 |
*** abhi89 has joined #openstack-nova | 12:11 | |
*** mingyu has quit IRC | 12:14 | |
*** lucasagomes is now known as lucas-hungry | 12:14 | |
sdague | sean-k-mooney: you are going to need to explain what your reboot workflow is, because I only reboot my devstack environments to get rid of cruft. :) So I honestly am not sure what "works with reboots" means | 12:15 |
*** trinaths has joined #openstack-nova | 12:17 | |
sean-k-mooney | sdague: before rejoin_stack.sh was removed you just had to start ovs,mysql,apache and rabbit then run rejoin_stack.sh and your devstack enviorment came back up | 12:17 |
*** edmondsw has joined #openstack-nova | 12:17 | |
*** smatzek has quit IRC | 12:17 | |
sdague | sudo systemctl start devstack@* post boot might do that | 12:17 |
sean-k-mooney | sdague: this was useful for demos as you could prepera an enviorment and snapshot it in a vm and give it to someone to try | 12:17 |
sean-k-mooney | sdague: yep you are enabling the service by default i think so it might just work aslong as you dont run unstack.sh | 12:18 |
sean-k-mooney | sdague: or clean.sh i guess | 12:18 |
sdague | yep | 12:18 |
*** pchavva has joined #openstack-nova | 12:18 | |
sean-k-mooney | anyway it might just be a nice sideffect to get that fucntionality back | 12:18 |
sean-k-mooney | i used to jsut run in offline mode instad but that ment you still had to recreated the vms/netwoks/flavour ... | 12:19 |
sean-k-mooney | thats not hard jsut extra steps | 12:19 |
*** Jack_Iv has joined #openstack-nova | 12:20 | |
*** mingyu has joined #openstack-nova | 12:20 | |
alex_xu_ | johnthetubaguy: sdague sfinucan appreciate if you can help this two easy api-ref changes https://review.openstack.org/461998 https://review.openstack.org/458738 | 12:21 |
sdague | sean-k-mooney: yeh, let me know how it works. If we need some tweaks to make that use can solid for you, that would be cool | 12:22 |
*** liverpooler has joined #openstack-nova | 12:22 | |
sdague | there was so much hacktastic around getting screen to be reliable, I look forward to that code being deleted with fire | 12:22 |
*** sshnaidm has joined #openstack-nova | 12:25 | |
*** kiwi_rot has joined #openstack-nova | 12:25 | |
sshnaidm | has anybody seen nova errors like: AttributeError: 'Instance' object has no attribute '_save_tags' ? | 12:26 |
*** zhurong has quit IRC | 12:30 | |
*** mingyu has quit IRC | 12:33 | |
johnthetubaguy | sshnaidm: I think I saw a bug and patch up for that somewhere | 12:33 |
johnthetubaguy | sshnaidm: I think gibi has this patch up: https://review.openstack.org/#/c/450265/ | 12:34 |
johnthetubaguy | dansmith: I wasn't sure about the object change in this one actually https://review.openstack.org/#/c/450265/ | 12:34 |
*** lucas-hungry is now known as lucasagomes | 12:34 | |
*** mingyu has joined #openstack-nova | 12:35 | |
*** ijw has joined #openstack-nova | 12:36 | |
sshnaidm | johnthetubaguy, thanks, so as I understand it's not critical, right? | 12:36 |
*** ratailor has joined #openstack-nova | 12:37 | |
johnthetubaguy | sshnaidm: I believe so | 12:37 |
*** abalutoiu has joined #openstack-nova | 12:38 | |
*** eharney has joined #openstack-nova | 12:38 | |
*** artom has quit IRC | 12:39 | |
*** slagle__ is now known as slagle | 12:40 | |
*** ijw has quit IRC | 12:41 | |
*** abalutoiu has quit IRC | 12:41 | |
*** ayogi has quit IRC | 12:44 | |
*** sou has joined #openstack-nova | 12:46 | |
sou | Hey, I am stuck with an issue for past 2 days | 12:47 |
sou | I am trying a nova rebuild | 12:47 |
sou | and it fails all the time | 12:47 |
sou | with error: ERROR oslo_messaging.rpc.dispatcher [req-b5730d47-3c34-4949-9836-01139238f829 8a205b2aa31c4c1185a417c6e99e852c fcb05e141ede4164ab6db9ad0724a08e - - -] Exception during message handling: id | 12:47 |
sou | nothing else in logs | 12:48 |
sou | the exception is generated from /usr/lib/python2.7/site-packages/glanceclient/openstack/common/apiclient/base.py | 12:48 |
sou | line 494, in __getattr__ | 12:48 |
sou | any one else has faced similar issue? | 12:49 |
sou | I am using Kilo version | 12:49 |
sou | compute nodes uses KVM and images are lvm | 12:49 |
*** lyan has joined #openstack-nova | 12:50 | |
*** slaweq has joined #openstack-nova | 12:50 | |
*** cfriesen has joined #openstack-nova | 12:52 | |
johnthetubaguy | sou: please try #openstack, see channel topic | 12:52 |
*** ratailor_ has joined #openstack-nova | 12:52 | |
sou | Thanks @johnthetubadguy | 12:53 |
*** Yangyapeng has joined #openstack-nova | 12:53 | |
*** kiwi_rot has quit IRC | 12:54 | |
*** yamamoto has joined #openstack-nova | 12:54 | |
*** slaweq has quit IRC | 12:54 | |
*** ratailor has quit IRC | 12:56 | |
*** brault has joined #openstack-nova | 12:56 | |
*** brault_ has quit IRC | 12:57 | |
*** thorst is now known as thorst_afk | 12:59 | |
*** vladikr has joined #openstack-nova | 13:12 | |
*** toure|gone is now known as toure | 13:13 | |
*** jamesdenton has joined #openstack-nova | 13:13 | |
*** karimb has joined #openstack-nova | 13:13 | |
*** mingyu has quit IRC | 13:14 | |
*** iceyao_ has joined #openstack-nova | 13:15 | |
*** iceyao has quit IRC | 13:15 | |
*** mingyu has joined #openstack-nova | 13:16 | |
*** rfolco has joined #openstack-nova | 13:20 | |
*** ratailor_ has quit IRC | 13:21 | |
*** yamamoto has quit IRC | 13:22 | |
*** swebster_ has quit IRC | 13:23 | |
*** markvoelker has joined #openstack-nova | 13:23 | |
*** yamamoto has joined #openstack-nova | 13:24 | |
*** mriedem has joined #openstack-nova | 13:24 | |
*** mriedem has left #openstack-nova | 13:25 | |
*** mriedem has joined #openstack-nova | 13:25 | |
openstackgerrit | Alex Xu proposed openstack/nova master: Use plain routes list for server-tags instead of stevedore https://review.openstack.org/461645 | 13:26 |
openstackgerrit | Alex Xu proposed openstack/nova master: Use plain routes list for os-interface endpoint instead of stevedore https://review.openstack.org/459979 | 13:26 |
openstackgerrit | Alex Xu proposed openstack/nova master: Use plain routes list for server-migrations endpoint instead of stevedore https://review.openstack.org/461646 | 13:26 |
abhi89 | johnthetubaguy: can we back port this fix https://review.openstack.org/#/c/460305/ to stable/ocata.. we were looking to have this change in this release in our product.. | 13:30 |
johnthetubaguy | abhi89: I think that should be OK | 13:30 |
dansmith | johnthetubaguy: yes, as you said, and as convention in the code, the empty save handler is the right approach | 13:31 |
abhi89 | johnthetubaguy: grt.. i will cherry pick the fix to ocata.. | 13:33 |
gibi | mriedem: hi! I could not attend on the nova meeting today, I left small notes on the agenda | 13:33 |
mriedem | gibi: ok thanks | 13:33 |
*** crushil_ has quit IRC | 13:36 | |
*** gouthamr has joined #openstack-nova | 13:38 | |
*** kiwi_rot has joined #openstack-nova | 13:41 | |
*** zhurong has joined #openstack-nova | 13:41 | |
*** Jack_Iv has quit IRC | 13:43 | |
*** Jack_Iv has joined #openstack-nova | 13:43 | |
*** esberglu has joined #openstack-nova | 13:44 | |
mriedem | gibi: do we need to call out https://review.openstack.org/#/c/401992/ when it's approved? | 13:45 |
mriedem | melwitt: sdague: dansmith: looks like the use_journal=True thing was causing the cells job failures http://status.openstack.org/elastic-recheck/#1580728 | 13:46 |
mriedem | we should still get https://review.openstack.org/#/c/461912 in though | 13:47 |
*** mingyu has quit IRC | 13:47 | |
*** mingyu has joined #openstack-nova | 13:48 | |
dansmith | mriedem: okay, I've yet to see my devstack patch fail with journald remoed | 13:48 |
dansmith | well, fail for a related reason anyway | 13:48 |
abhi89 | johnthetubaguy: please review cherry picked fix https://review.openstack.org/#/c/462561/ for stable/ocata | 13:49 |
*** xyang1 has joined #openstack-nova | 13:49 | |
*** zhurong_ has joined #openstack-nova | 13:50 | |
*** zhurong has quit IRC | 13:51 | |
*** yamahata has quit IRC | 13:51 | |
*** lucasxu has joined #openstack-nova | 13:52 | |
*** mlavalle has joined #openstack-nova | 13:53 | |
*** baoli has joined #openstack-nova | 13:55 | |
*** sou has quit IRC | 13:55 | |
*** awaugama has joined #openstack-nova | 13:55 | |
mriedem | nova meeting in 4 minutes in #openstack-meeting | 13:56 |
*** bkopilov has quit IRC | 13:56 | |
*** takashin has joined #openstack-nova | 13:56 | |
*** vks1 has joined #openstack-nova | 13:56 | |
*** jamielennox|away is now known as jamielennox | 13:58 | |
*** crushil_ has joined #openstack-nova | 13:59 | |
*** mariojv_ is now known as mariojv | 14:00 | |
*** jaypipes has joined #openstack-nova | 14:00 | |
*** cleong has joined #openstack-nova | 14:03 | |
*** sam___ has joined #openstack-nova | 14:06 | |
*** brault has quit IRC | 14:06 | |
*** sam___ has left #openstack-nova | 14:09 | |
*** ekuris has quit IRC | 14:09 | |
*** prateek has quit IRC | 14:09 | |
*** zhurong_ has quit IRC | 14:12 | |
*** armax has joined #openstack-nova | 14:12 | |
*** moshele has quit IRC | 14:14 | |
*** liuyulong has joined #openstack-nova | 14:14 | |
*** coreywright has quit IRC | 14:16 | |
*** zz_dimtruck is now known as dimtruck | 14:18 | |
*** catintheroof has joined #openstack-nova | 14:20 | |
*** catinthe_ has joined #openstack-nova | 14:23 | |
*** catintheroof has quit IRC | 14:24 | |
*** coreywright has joined #openstack-nova | 14:28 | |
*** markvoelker has quit IRC | 14:31 | |
*** iceyao_ has quit IRC | 14:32 | |
*** peter-hamilton has joined #openstack-nova | 14:32 | |
mtreinish | kashyap: fwiw, I was talking about the grenade line there. It's not 100% obvious that it's there for making sure systemd isn't used on grenade | 14:32 |
mtreinish | I had to think it through out loud to understand the logic in that if | 14:33 |
kashyap | mtreinish: Noted. :-) | 14:33 |
openstackgerrit | Gábor Antal proposed openstack/nova master: Transform aggregate.updateprop notification https://review.openstack.org/462576 | 14:33 |
*** kylek3h has joined #openstack-nova | 14:33 | |
*** nammo has joined #openstack-nova | 14:34 | |
nammo | Hi How can I remove a RetryFilter? | 14:34 |
nammo | I removed it from nova.conf but it still exists | 14:35 |
bauzas | manjeets__: pull it out from enabled_filters | 14:37 |
bauzas | oops | 14:38 |
bauzas | nammo: ^ | 14:38 |
* bauzas isn't drunk | 14:38 | |
*** takashin has left #openstack-nova | 14:38 | |
*** stvnoyes has quit IRC | 14:38 | |
*** armax has quit IRC | 14:38 | |
openstackgerrit | Gábor Antal proposed openstack/nova master: [WIP] Transform aggregate.update_metadata notification https://review.openstack.org/460625 | 14:38 |
nammo | bauzas: Thanks for your response | 14:38 |
nammo | I did it but it still show up in logs | 14:38 |
*** tbachman_ has joined #openstack-nova | 14:38 | |
openstackgerrit | Jay Pipes proposed openstack/nova master: placement: tests for non-shared with shared https://review.openstack.org/461899 | 14:39 |
openstackgerrit | Jay Pipes proposed openstack/nova master: placement: test for agg association not sharing https://review.openstack.org/461901 | 14:39 |
openstackgerrit | Jay Pipes proposed openstack/nova master: placement: test non-shared out of inventory https://review.openstack.org/461900 | 14:39 |
openstackgerrit | Jay Pipes proposed openstack/nova master: placement: shared resources when finding providers https://review.openstack.org/461574 | 14:39 |
openstackgerrit | Jay Pipes proposed openstack/nova master: placement: get providers sharing capacity https://review.openstack.org/460798 | 14:39 |
jaypipes | dansmith: fixed that silly copy/paste error in the functional test ^^ | 14:39 |
jaypipes | dansmith: thx for your reviews! | 14:39 |
dansmith | jaypipes: did you see the rest? I think I had a -1 on all of them | 14:40 |
*** stvnoyes has joined #openstack-nova | 14:40 | |
nammo | bauzas: I restarted n-sch, n-cond | 14:40 |
nammo | Is that sufficient? | 14:40 |
bauzas | nammo: only n-sch | 14:40 |
*** tbachman has quit IRC | 14:40 | |
*** tbachman_ is now known as tbachman | 14:40 | |
nammo | bauzas: No it is not working for me :( | 14:41 |
*** voelzmo has quit IRC | 14:42 | |
*** aloga has quit IRC | 14:42 | |
*** aloga has joined #openstack-nova | 14:43 | |
*** gongysh has joined #openstack-nova | 14:44 | |
*** yamamoto has quit IRC | 14:44 | |
*** annegentle has joined #openstack-nova | 14:44 | |
*** lpetrut_ has quit IRC | 14:44 | |
*** voelzmo_ has joined #openstack-nova | 14:44 | |
nammo | bauzas: http://paste.openstack.org/show/608874/ | 14:44 |
*** crushil_ has quit IRC | 14:44 | |
jaypipes | dansmith: I've fixed some issues. with regards to the long code comment, where do you want me to put that stuff? | 14:46 |
dansmith | jaypipes: I dunno, it was just an idea.. I'd like to avoid moving the actual db query code elsewhere | 14:47 |
bauzas | jaypipes: FWIW, I was reviewing your bottom change, but I was thinking about using Workbench | 14:47 |
jaypipes | dansmith: right... I moved that code into the module-level scope per your request. | 14:47 |
bauzas | jaypipes: because $sql | 14:48 |
dansmith | jaypipes: yep, I saw, and I appreciate that. I'm referring to edleafe and cdent's desire to see it moved farther away | 14:48 |
jaypipes | dansmith: oh. sorry, do you agree with that? | 14:48 |
*** claudiub has quit IRC | 14:49 | |
dansmith | jaypipes: agree with what? I just said I didn't want to see that code move out, so if we need to make things smaller, maybe we should move the comments out | 14:49 |
*** tbachman has quit IRC | 14:49 | |
*** adisky_ has quit IRC | 14:49 | |
*** gongysh has quit IRC | 14:49 | |
openstackgerrit | Gábor Antal proposed openstack/nova master: Transform aggregate.update_metadata notification https://review.openstack.org/460625 | 14:50 |
cdent | I'm not wed to the idea of moving it out, is fine in the same file, but I find huge python files difficult to think about | 14:51 |
*** slaweq has joined #openstack-nova | 14:51 | |
jaypipes | dansmith: right, and I was asking where you'd like to move the SQL comments. | 14:51 |
openstackgerrit | Jay Pipes proposed openstack/nova master: placement: tests for non-shared with shared https://review.openstack.org/461899 | 14:51 |
openstackgerrit | Jay Pipes proposed openstack/nova master: placement: test for agg association not sharing https://review.openstack.org/461901 | 14:51 |
openstackgerrit | Jay Pipes proposed openstack/nova master: placement: test non-shared out of inventory https://review.openstack.org/461900 | 14:51 |
openstackgerrit | Jay Pipes proposed openstack/nova master: placement: shared resources when finding providers https://review.openstack.org/461574 | 14:51 |
dansmith | jaypipes: and I said "I dunno" :P | 14:52 |
dansmith | jaypipes: my preference would be to not move things out of that file | 14:52 |
*** tongli has joined #openstack-nova | 14:52 | |
jaypipes | dansmith: ah, sorry, I missed that. :) | 14:52 |
bauzas | cdent: jaypipes: dansmithwhat needs to move ? | 14:52 |
bauzas | I'm fine with leaving ORM code as it is | 14:52 |
bauzas | if that's the point | 14:53 |
*** smatzek has joined #openstack-nova | 14:53 | |
*** markvoelker has joined #openstack-nova | 14:53 | |
*** slaweq has quit IRC | 14:55 | |
*** salv-orl_ has joined #openstack-nova | 14:56 | |
*** armax has joined #openstack-nova | 14:56 | |
jaypipes | bauzas: the discussion is not around the ORM code but rather there is a discussion about how long the code comment with SQL is on that function. | 14:58 |
bauzas | well | 14:58 |
bauzas | okay | 14:58 |
bauzas | not sure it's a problme | 14:58 |
bauzas | when I wrote the long docstring, I also fixed the comment when I changed the calls | 14:58 |
bauzas | that's why I'd like to at least pdb your change for verifying the SQL query | 14:59 |
*** salv-orlando has quit IRC | 14:59 | |
mriedem | won't we want the big ass sql comment for the complicated orm code? | 14:59 |
mriedem | as dansmith said earlier in the week, this is all fine but i won't be debugging it :) | 14:59 |
bauzas | it's just part of a reviewer duty /me shrugs | 14:59 |
dansmith | there is no question about ORM stuff AT ALL | 15:00 |
dansmith | it's about where to put these comments about the sql | 15:00 |
*** hongbin has joined #openstack-nova | 15:00 | |
bauzas | tbc, I'm fine with the current situation | 15:00 |
bauzas | having large docstrings is good wit hme | 15:00 |
*** yingjun has joined #openstack-nova | 15:00 | |
bauzas | anyway, /me needs to take my daughter \o | 15:00 |
*** dillaman has quit IRC | 15:01 | |
nammo | jaypipes: dansmith Hi guys! Can you please help me with removing a filter? | 15:05 |
peter-hamilton | mriedem: just wanted to put this on your radar - https://review.openstack.org/#/c/357202/ | 15:05 |
nammo | I removed it from enabled_filters, restarted n-sch but still it exists | 15:06 |
jaypipes | nammo: give us 15-25 minutes. we're currently on a call. | 15:06 |
nammo | Oh ok sorry. | 15:06 |
jaypipes | no worries :) | 15:06 |
peter-hamilton | mriedem: danefichter and i want more nova eyes on it before we go any further | 15:06 |
*** jerrygb has joined #openstack-nova | 15:07 | |
*** claudiub has joined #openstack-nova | 15:07 | |
*** crushil_ has joined #openstack-nova | 15:08 | |
mtreinish | johnthetubaguy: what were you looking for on: https://review.openstack.org/#/c/459470 ? The comments are a bit vague on what's missing (its more of a discussion) | 15:10 |
johnthetubaguy | mtreinish: good question, I have a feeling I got distracted and moved to a different review | 15:10 |
*** Oku_OS is now known as Oku_OS-away | 15:11 | |
*** kiwi_rot has quit IRC | 15:12 | |
*** kaisers has joined #openstack-nova | 15:13 | |
johnthetubaguy | mtreinish: you know, the only nit was that config that I think we can just remove, but I have a feeling mriedem wasn't 100% sure about the remove, lets worry about that in a follow up patch | 15:14 |
*** dillaman has joined #openstack-nova | 15:15 | |
*** hshiina has joined #openstack-nova | 15:15 | |
*** karimb has quit IRC | 15:17 | |
mtreinish | johnthetubaguy: yeah, that's why I added the additional information to the help instead of deleting it | 15:18 |
*** kaisers has quit IRC | 15:18 | |
*** iceyao has joined #openstack-nova | 15:18 | |
*** brault has joined #openstack-nova | 15:19 | |
*** karimb has joined #openstack-nova | 15:20 | |
*** tbachman has joined #openstack-nova | 15:20 | |
*** stvnoyes has quit IRC | 15:22 | |
*** iceyao has quit IRC | 15:22 | |
*** claudiub|2 has joined #openstack-nova | 15:23 | |
*** stvnoyes has joined #openstack-nova | 15:23 | |
*** ijw has joined #openstack-nova | 15:23 | |
*** brault has quit IRC | 15:24 | |
*** abhi89 has quit IRC | 15:24 | |
*** kfarr has joined #openstack-nova | 15:24 | |
*** armax has quit IRC | 15:25 | |
*** tbachman has quit IRC | 15:25 | |
*** aarefiev is now known as aarefiev_afk | 15:27 | |
*** claudiub has quit IRC | 15:27 | |
*** tbachman has joined #openstack-nova | 15:27 | |
openstackgerrit | Matthew Treinish proposed openstack/python-novaclient master: Remove the deprecated certs interfaces https://review.openstack.org/462598 | 15:28 |
mtreinish | mriedem: ^^^ | 15:28 |
mtreinish | I'll work on the cloudpipe client deprecation later | 15:28 |
*** moshele has joined #openstack-nova | 15:30 | |
*** marst has joined #openstack-nova | 15:31 | |
*** vks1 has quit IRC | 15:33 | |
*** mdnadeem has quit IRC | 15:33 | |
*** dpawlik has quit IRC | 15:34 | |
*** chyka has joined #openstack-nova | 15:35 | |
jaypipes | nammo: so, which filter did you try removing? | 15:35 |
*** liuyulong has quit IRC | 15:36 | |
*** moshele has quit IRC | 15:37 | |
*** iceyao has joined #openstack-nova | 15:39 | |
openstackgerrit | Roman Podoliaka proposed openstack/nova master: service: use restart_method='mutate' for all services https://review.openstack.org/462600 | 15:41 |
*** andreas_s has quit IRC | 15:42 | |
*** yamamoto has joined #openstack-nova | 15:45 | |
*** shaohe_feng has quit IRC | 15:46 | |
*** voelzmo_ has quit IRC | 15:46 | |
*** alex_xu_ has quit IRC | 15:46 | |
*** heyongli has quit IRC | 15:46 | |
*** voelzmo has joined #openstack-nova | 15:46 | |
*** manjeets__ is now known as manjeets | 15:48 | |
openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova master: Read Neutron port 'binding_profile' during boot https://review.openstack.org/449257 | 15:49 |
openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova master: Add Neutron port capabilities to devspec in request https://review.openstack.org/451777 | 15:49 |
*** Jack_Iv has quit IRC | 15:49 | |
*** gyee has joined #openstack-nova | 15:50 | |
*** claudiub|2 has quit IRC | 15:51 | |
*** damien_r1 has quit IRC | 15:51 | |
mriedem | peter-hamilton: it's going to be hard today, lots of meetings | 15:52 |
dansmith | edleafe: think you could address some of the feedback that has been sitting on these patches for a few days? https://review.openstack.org/#/c/460177 | 15:52 |
mriedem | mtreinish: you can't remove the nova cert CLI stuff until queens | 15:52 |
mriedem | mtreinish: we need to deprecate the cloudpipe API bindings and CLIs in novaclient like i did for the os-hosts stuff though | 15:53 |
*** Jack_Iv has joined #openstack-nova | 15:53 | |
*** lucasxu has quit IRC | 15:53 | |
mriedem | johnthetubaguy: oh the vpn_ip? | 15:53 |
dansmith | edleafe: I haven't seen any replies from bauzas so I assume he's busy with other stuff | 15:53 |
mriedem | yeah i wasn't sure | 15:53 |
*** jaosorior has quit IRC | 15:54 | |
johnthetubaguy | mriedem: yeah, I figured mtreinish updated the description, we can always rip it out later if that turns out to be the right call. | 15:54 |
*** Apoorva has joined #openstack-nova | 15:54 | |
edleafe | dansmith: I know he was on holiday earlier this week | 15:55 |
dansmith | edleafe: yeah | 15:55 |
edleafe | bauzas: still around? | 15:55 |
*** lucasxu has joined #openstack-nova | 15:56 | |
mriedem | he's picking up his daugher from school i think | 15:56 |
edleafe | dansmith: I mean, I'd be happy to work on this, but didn't want to step on bauzas's toes | 15:56 |
edleafe | mriedem: what, are they walking? :) | 15:56 |
*** lucasxu has quit IRC | 15:56 | |
dansmith | edleafe: yeah, it's been days though so I figured he's just busy, but cool to check | 15:56 |
mriedem | edleafe: probably an old fashioned bike would be my guess | 15:56 |
*** lucasxu has joined #openstack-nova | 15:56 | |
mriedem | with cigs and croissants | 15:57 |
*** yamamoto has quit IRC | 15:57 | |
dansmith | but if you're willing I would think some of it could be shared at least | 15:57 |
edleafe | dansmith: that's fine with me | 15:57 |
*** cfriesen has quit IRC | 15:57 | |
*** cfriesen has joined #openstack-nova | 15:57 | |
edleafe | dansmith: I'll start on it after the api-wg meeting, unless bauzas speaks up first | 15:58 |
nammo | jaypipes: RetryFilter | 15:58 |
dansmith | edleafe: cool, thanks | 15:58 |
jaypipes | nammo: that cannot be removed, AFAIK. | 15:59 |
*** voelzmo has quit IRC | 15:59 | |
edleafe | nammo: why would you want to remove that? | 15:59 |
*** lucasxu has quit IRC | 15:59 | |
melwitt | it used to be able to be removed | 15:59 |
nammo | jaypipes: edleafe Any consecutive request to boot are failing due to RetryFilter | 15:59 |
*** slaweq has joined #openstack-nova | 16:00 | |
edleafe | nammo: RetryFilter only comes into play when a request to build on a host has already failed | 16:00 |
jaypipes | melwitt: really? TIL... | 16:01 |
nammo | edleafe: I deleted that VM and tried to boot again | 16:01 |
melwitt | we had removed it because of the integration with other systems we had done, vast majority of failures were deterministic and retrying a different compute wasn't going to help and would waste time, so we didn't have RetryFilter in our list and set scheduler_max_attempts = 1 | 16:01 |
*** lucasxu has joined #openstack-nova | 16:02 | |
nammo | melwitt: Is it intended for me? | 16:02 |
melwitt | nammo: no. just trivia on one reason someone would remove it | 16:03 |
dansmith | yay for more reasons to not do scheduler retries | 16:04 |
nammo | melwitt: scheduler_max_attempts This should be under default section? | 16:04 |
mtreinish | mriedem: oh I thought the deprecation timer started long enough ago. Well I'll just wip it to block the merge until after the release | 16:05 |
*** iceyao has quit IRC | 16:06 | |
melwitt | nammo: these days it's [scheduler]/max_attempts. but the fact that you see "Previously tried hosts" in your log still means the RetryFilter is still there, which doesn't make sense if you removed it from enabled_filters and restarted n-sch | 16:06 |
nammo | melwitt: So is there any other solution to launch VM on the same host? | 16:07 |
nammo | melwitt: host is up and available | 16:07 |
melwitt | nammo: what are you trying to do? the RetryFilter just retries on a different host if the selected host fails. you're trying to get multiple VMs to land on the same host? | 16:08 |
*** nic has joined #openstack-nova | 16:08 | |
nammo | melwitt: I just have one node that satifies my VM spec | 16:09 |
nammo | melwitt: I will see the sched logs to see why RetryFilter failed | 16:10 |
melwitt | that's usually what you want with a vanilla nova deployment, is to have retries. because if the host the scheduler selected fails to spawn the instance, it's usually best to try a different host next | 16:10 |
mriedem | mtreinish: it's going to be sitting quite awhile if you do that, butok | 16:10 |
nammo | melwitt: Agree | 16:10 |
nammo | But for me it just one host that my VM can launch on | 16:10 |
melwitt | nammo: I see. you should check the sched logs to determine why the first attempt failed. usually means not enough resources or extra_specs didn't match or something like that. if you turn on debug log level for n-sch you will see all the detail of which filters it tried and which one failed | 16:11 |
*** mvk has quit IRC | 16:11 | |
*** lucasxu has quit IRC | 16:15 | |
*** iceyao has joined #openstack-nova | 16:18 | |
nammo | melwitt: http://paste.openstack.org/show/608889/ | 16:21 |
nammo | melwitt: I see no error here | 16:25 |
nammo | melwitt: Can you please help? | 16:25 |
openstackgerrit | Dan Smith proposed openstack/nova master: WIP: Debug race in delete-while-booting test https://review.openstack.org/461513 | 16:26 |
*** lucasagomes is now known as lucas-afk | 16:26 | |
*** yamamoto has joined #openstack-nova | 16:26 | |
*** salv-orl_ has quit IRC | 16:26 | |
dansmith | nammo: please see topic, this channel is for development | 16:26 |
openstackgerrit | Dan Smith proposed openstack/nova master: WIP: De-complicate some of the instance delete path https://review.openstack.org/458636 | 16:27 |
openstackgerrit | Dan Smith proposed openstack/nova master: Move to proper target_cell calling convention https://review.openstack.org/458634 | 16:27 |
openstackgerrit | Dan Smith proposed openstack/nova master: Make target_cell() yield a new context https://review.openstack.org/458635 | 16:27 |
openstackgerrit | Dan Smith proposed openstack/nova master: DNM: Canary test against multi-cell devstack https://review.openstack.org/458537 | 16:27 |
nammo | dansmith: I see. Thanks | 16:28 |
dansmith | jaypipes: is the reason that first test worked because of the trait? | 16:28 |
melwitt | nammo: to close the loop on this, agree there's no problem in n-sch and you need to check your n-cpu logs for why it failed to spawn the instance on the selected host | 16:29 |
mriedem | dansmith: your access has been...granted | 16:29 |
dansmith | mriedem: thanks | 16:29 |
*** ijw has quit IRC | 16:30 | |
nammo | melwitt: Thanks :) | 16:30 |
*** ijw has joined #openstack-nova | 16:31 | |
*** iceyao has quit IRC | 16:32 | |
*** iceyao has joined #openstack-nova | 16:32 | |
*** iceyao has quit IRC | 16:33 | |
*** mrhillsman has quit IRC | 16:35 | |
*** wxy has quit IRC | 16:35 | |
*** aweeks has quit IRC | 16:35 | |
*** test222 has quit IRC | 16:35 | |
*** ijw has quit IRC | 16:35 | |
*** yamamoto has quit IRC | 16:37 | |
*** pcaruana has quit IRC | 16:40 | |
*** aweeks has joined #openstack-nova | 16:41 | |
*** ijw has joined #openstack-nova | 16:41 | |
mriedem | bauzas: jaypipes: edleafe: you guys will likely want to see this https://review.openstack.org/#/c/394857/ since it has to do with overhead calculation and overrides in the RT and virt drivers | 16:42 |
*** test222 has joined #openstack-nova | 16:43 | |
edleafe | mriedem: just reading the commit message on that seems like it's a misunderstanding | 16:43 |
mriedem | i'm -1ing now | 16:44 |
*** stvnoyes has quit IRC | 16:45 | |
mriedem | edleafe: done - you can proofread it | 16:45 |
mriedem | :) | 16:45 |
*** wxy has joined #openstack-nova | 16:46 | |
*** mrhillsman has joined #openstack-nova | 16:46 | |
*** armax has joined #openstack-nova | 16:47 | |
mriedem | bauzas: replied in https://review.openstack.org/#/c/454899/ - that's how the online data migrations were done for adding uuid to aggregates and compute_nodes tables, | 16:48 |
mriedem | not sure that's worth a -1 | 16:48 |
mriedem | but maybe others have strong opinions | 16:48 |
*** erlon has joined #openstack-nova | 16:49 | |
dansmith | mriedem: I kinda like them in the object file, but I don't think you should change it just for that | 16:49 |
*** lucasxu has joined #openstack-nova | 16:49 | |
*** imacdonn has quit IRC | 16:50 | |
edleafe | mriedem: added my own -1 and explanation | 16:50 |
*** imacdonn has joined #openstack-nova | 16:50 | |
*** moshele has joined #openstack-nova | 16:52 | |
jaypipes | dansmith: yes. first test passed because of the trait. | 16:52 |
dansmith | jaypipes: yeah, ack | 16:52 |
*** fragatina has quit IRC | 16:54 | |
*** moshele has quit IRC | 16:55 | |
openstackgerrit | Jackie Truong proposed openstack/nova master: Encode encryption key passed to dmcrypt https://review.openstack.org/462674 | 16:58 |
*** psachin has joined #openstack-nova | 16:58 | |
*** baoli has quit IRC | 16:59 | |
*** stvnoyes has joined #openstack-nova | 16:59 | |
openstackgerrit | Jay Pipes proposed openstack/nova master: placement: tests for non-shared with shared https://review.openstack.org/461899 | 17:00 |
openstackgerrit | Jay Pipes proposed openstack/nova master: placement: test for agg association not sharing https://review.openstack.org/461901 | 17:00 |
openstackgerrit | Jay Pipes proposed openstack/nova master: placement: test non-shared out of inventory https://review.openstack.org/461900 | 17:00 |
openstackgerrit | Jay Pipes proposed openstack/nova master: placement: shared resources when finding providers https://review.openstack.org/461574 | 17:00 |
openstackgerrit | Jay Pipes proposed openstack/nova master: placement: get providers sharing capacity https://review.openstack.org/460798 | 17:00 |
jaypipes | dansmith, edleafe, cdent: ok, I believe I've responded or fixed all your review comments on above series. | 17:01 |
*** baoli has joined #openstack-nova | 17:01 | |
*** kfarr has quit IRC | 17:01 | |
jaypipes | other than the moving of the long SQL code comment somewhere... | 17:01 |
dansmith | jaypipes: gdi jay, you pushed too fast, I wanted to claim that review credit for my commit message nit :P | 17:01 |
jaypipes | we can bikeshed about where to move that later, ok? | 17:01 |
*** derekh has quit IRC | 17:01 | |
jaypipes | heh, sorry dansmith | 17:02 |
*** ralonsoh has quit IRC | 17:02 | |
dansmith | *shakes fist* | 17:02 |
dansmith | mriedem is calling me out BY NAME in public so I have to up my game | 17:02 |
edleafe | jaypipes: for the record, I like the comment within the method. I just don't like a module-level method in the middle of the object class definitions | 17:02 |
*** karimb has quit IRC | 17:02 | |
*** arunman has quit IRC | 17:03 | |
jaypipes | edleafe: noted. :) | 17:03 |
cdent | blue | 17:03 |
dansmith | heh | 17:03 |
*** hieulq_ has joined #openstack-nova | 17:05 | |
*** hshiina has quit IRC | 17:07 | |
* cdent goes to the polls to do his civic duty | 17:07 | |
*** litao has quit IRC | 17:09 | |
*** crushil_ has quit IRC | 17:10 | |
*** jpena is now known as jpena|off | 17:12 | |
*** markus_z has quit IRC | 17:12 | |
dansmith | jaypipes: in SQLandia, if I to a SELECT * WHERE longstring="uuidthing", | 17:13 |
dansmith | is that much more efficient than doing an IN check for a list of one? | 17:13 |
*** fragatina has joined #openstack-nova | 17:13 | |
openstackgerrit | Ed Leafe proposed openstack/nova master: WIP: Don't delete allocation if instance being scheduled https://review.openstack.org/460177 | 17:13 |
openstackgerrit | Ed Leafe proposed openstack/nova master: Add a convenience attribute for reportclient https://review.openstack.org/462679 | 17:13 |
edleafe | dansmith: ^^ | 17:13 |
dansmith | meaning SELECT * WHERE longstring IN ("uuidthing") or whatever the syntax is | 17:13 |
dansmith | edleafe: woot | 17:13 |
edleafe | dansmith: just addressed the comments. Will work on the rest now | 17:13 |
mriedem | on an indexed column | 17:13 |
dansmith | mriedem: ah, yeah, good point the index | 17:13 |
*** ltomasbo is now known as ltomasbo|away | 17:14 | |
mriedem | dansmith: well, after https://review.openstack.org/#/c/454887/ that is | 17:14 |
jaypipes | dansmith: all optimizers I know of will rewrite the latter to the former. | 17:14 |
dansmith | jaypipes: ah, okay | 17:14 |
dansmith | actually, I think I remember seeing warnings about doing that | 17:14 |
*** gjayavelu has joined #openstack-nova | 17:17 | |
mriedem | dansmith: if you see the patch after it, i could have ComputeNode.get_by_uuid call ComputeNodeList.get_all_by_uuids and internally handle the [uuid] and [0], but i just wondered about the IN performance | 17:18 |
*** baoli has quit IRC | 17:18 | |
dansmith | mriedem: yeah | 17:19 |
*** baoli has joined #openstack-nova | 17:19 | |
mriedem | but maybe this is a small enough difference that it doesn't really matter and we could just squash those two changes, and just hide it all in the object code | 17:20 |
dansmith | yeah, seems like it might be | 17:20 |
*** psachin has quit IRC | 17:20 | |
mriedem | http://stackoverflow.com/questions/9926433/mysql-performance-in-clause-vs-equals-for-a-single-value | 17:21 |
mriedem | "There is no difference between the MySQL statements, and the MySQL optimiser will transform the IN to the = when IN is just one element. Don't bother." | 17:21 |
mriedem | so what jaypipes said | 17:21 |
openstackgerrit | Dan Smith proposed openstack/nova master: WIP: De-complicate some of the instance delete path https://review.openstack.org/458636 | 17:22 |
openstackgerrit | Dan Smith proposed openstack/nova master: Move to proper target_cell calling convention https://review.openstack.org/458634 | 17:22 |
openstackgerrit | Dan Smith proposed openstack/nova master: Make target_cell() yield a new context https://review.openstack.org/458635 | 17:22 |
openstackgerrit | Dan Smith proposed openstack/nova master: DNM: Canary test against multi-cell devstack https://review.openstack.org/458537 | 17:22 |
*** jaypipes has left #openstack-nova | 17:24 | |
*** jaypipes has joined #openstack-nova | 17:24 | |
*** lpetrut has joined #openstack-nova | 17:24 | |
peter-hamilton | mriedem: sorry, i was out - not a problem, i just wanted to put it in the queue :) | 17:25 |
*** trinaths has left #openstack-nova | 17:26 | |
*** ijw has quit IRC | 17:27 | |
*** igordcar1 has quit IRC | 17:27 | |
*** igordcard has joined #openstack-nova | 17:27 | |
*** tesseract has quit IRC | 17:30 | |
*** hieulq_ has quit IRC | 17:37 | |
*** hieulq_ has joined #openstack-nova | 17:39 | |
*** crushil_ has joined #openstack-nova | 17:41 | |
*** mingyu has quit IRC | 17:41 | |
*** gyee has quit IRC | 17:45 | |
*** baoli_ has joined #openstack-nova | 17:47 | |
*** yingjun has quit IRC | 17:48 | |
*** karthiks has joined #openstack-nova | 17:48 | |
*** baoli has quit IRC | 17:49 | |
*** smatzek has quit IRC | 17:51 | |
*** fragatina has quit IRC | 17:54 | |
*** fragatina has joined #openstack-nova | 17:54 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: db api: add service_get_by_uuid https://review.openstack.org/461428 | 17:55 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Handle uuid in HostAPI._find_service https://review.openstack.org/461519 | 17:55 |
*** moshele has joined #openstack-nova | 17:55 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: objects: add Service.get_by_uuid method https://review.openstack.org/461429 | 17:55 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: objects: add ComputeNode.get_by_uuid method https://review.openstack.org/461448 | 17:55 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Handle uuid in HostAPI.compute_node_get https://review.openstack.org/461532 | 17:55 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Add online data migration for populating services.uuid https://review.openstack.org/454899 | 17:55 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Add uuid to service.update notification payload https://review.openstack.org/461473 | 17:55 |
*** Apoorva has quit IRC | 17:56 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Handle uuid in HostAPI._find_service https://review.openstack.org/461519 | 17:57 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: objects: add ComputeNode.get_by_uuid method https://review.openstack.org/461448 | 17:57 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Handle uuid in HostAPI.compute_node_get https://review.openstack.org/461532 | 17:57 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Add uuid to service.update notification payload https://review.openstack.org/461473 | 17:57 |
dansmith | mriedem: pushing the same set three times in a row is fooling nobody | 17:58 |
mriedem | 2 times in a row | 17:58 |
dansmith | N times in a row | 17:58 |
*** Sukhdev has joined #openstack-nova | 18:03 | |
*** lpetrut has quit IRC | 18:03 | |
mriedem | gah, plus i screwed up the topic branch | 18:04 |
efried | mriedem mordred jaypipes [service catalog bp] I'd like to settle this issue of determining interface if not otherwise specified. | 18:05 |
mriedem | pistols at dawn it is | 18:05 |
efried | At some point in the early (at least in my involvement) discussions, there was talk of trying the known ones in order. | 18:05 |
*** moshele has quit IRC | 18:05 | |
efried | This is a dict lookup, so that's cheap. | 18:06 |
mriedem | maybe the idea with starting using internal first is to avoid defaulting to public in ksa? | 18:06 |
efried | But on the other side, there's "use the default from ksa" | 18:06 |
efried | ...which is "public", for whatever that's worth. | 18:06 |
efried | Can any of you remember the rationale behind the "loop through 'em" suggestion? | 18:06 |
mriedem | efried: that was mordred or jaypipes' idea i thought, but presumably to hit the internal/admin urls before public | 18:07 |
efried | mriedem Yeah, starting with "internal" seemed like a more... uh... "secure" idea. | 18:07 |
mriedem | since today ops are probably using internal/admin | 18:07 |
efried | Okay, right now I've got internal->public->admin; that's an easy swap. But question remains: do we want to do the loop thing at all? | 18:08 |
mriedem | i don't really feel strongly either way, looping over a hard-coded list which might not match what's actually in the service catalog seems weird to me | 18:09 |
*** yonglihe has quit IRC | 18:09 | |
efried | mriedem Yeah, that makes me a tad uncomfortable too. Maybe mordred can point us to a way to get that list from ksa. | 18:09 |
*** abhi89 has joined #openstack-nova | 18:10 | |
mriedem | https://github.com/openstack/keystoneauth/blob/master/keystoneauth1/identity/base.py#L173 | 18:10 |
mriedem | the docs say one thing but i'm pretty sure interface is open ended | 18:11 |
mriedem | although at one point i thought i found json schema in the keystone api for v3 that restricted the interfaces | 18:11 |
mriedem | to a known set | 18:11 |
efried | One approach is to say, we try these three, and if it ain't one of those, they better specify it. | 18:13 |
efried | They can always specify it. | 18:13 |
mriedem | efried: https://github.com/openstack/keystone/blob/master/keystone/catalog/schema.py#L94 | 18:14 |
efried | noyce. | 18:14 |
mriedem | so the interface types are validated via jsonschema in the keystone api, at least for the v3 api i think | 18:14 |
mriedem | so iterating through admin/internal/public probably seems fine | 18:15 |
*** dtp has joined #openstack-nova | 18:15 | |
*** zerda3 has quit IRC | 18:15 | |
efried | What order would you like? | 18:15 |
*** zerda3 has joined #openstack-nova | 18:15 | |
mriedem | don't ask me :) | 18:16 |
mriedem | ask operators | 18:16 |
mriedem | #openstack-operators | 18:16 |
abhi89 | dansmith, jaypipes: please review https://review.openstack.org/#/c/462561/.. its for stable/ocata merge.. | 18:17 |
*** slaweq has quit IRC | 18:17 | |
*** slaweq has joined #openstack-nova | 18:18 | |
*** ociuhandu has quit IRC | 18:18 | |
*** harlowja has quit IRC | 18:19 | |
*** Sukhdev has quit IRC | 18:20 | |
*** nammo has quit IRC | 18:21 | |
*** smatzek has joined #openstack-nova | 18:22 | |
*** slaweq has quit IRC | 18:23 | |
*** smatzek has quit IRC | 18:30 | |
*** cdent has quit IRC | 18:35 | |
*** baoli_ has quit IRC | 18:37 | |
*** baoli has joined #openstack-nova | 18:38 | |
*** gszasz has quit IRC | 18:38 | |
*** mingyu has joined #openstack-nova | 18:41 | |
mriedem | melwitt: looks like we have a KeyError when the tests run now because of ^ https://review.openstack.org/#/c/460305/6/nova/tests/unit/compute/test_compute.py | 18:43 |
mriedem | https://bugs.launchpad.net/nova/+bug/1688366 | 18:47 |
openstack | Launchpad bug 1688366 in OpenStack Compute (nova) "KeyError: u'overs' in unit test runs" [Low,Triaged] | 18:47 |
melwitt | mriedem: sigh ... I miss that config option that used to make format errors fatal for tests | 18:47 |
mriedem | i don't know why that was removed | 18:47 |
*** cdent has joined #openstack-nova | 18:47 | |
mriedem | maybe it was too useful | 18:47 |
melwitt | yeah, really. it was removed as part of the centralize-config-options work, and I tried to recreate the behavior with wonky monkey patching and wasn't successful https://review.openstack.org/#/c/411615 | 18:48 |
*** Apoorva has joined #openstack-nova | 18:50 | |
dansmith | 184489 | 18:52 |
dansmith | heh sorry | 18:52 |
melwitt | there's been something in the unit tests spewing garbled text into the output for months now, I assume that's another KeyError some place in a test exception | 18:53 |
*** peter-hamilton has quit IRC | 18:54 | |
*** amotoki has quit IRC | 18:55 | |
openstackgerrit | Eric Fried proposed openstack/nova-specs master: Updates for [glance]api_servers and ksa https://review.openstack.org/461481 | 18:56 |
efried | mriedem ^^ | 18:56 |
openstackgerrit | Steve Noyes proposed openstack/nova master: Add Cinder v3 detach to shutdown_instance https://review.openstack.org/456877 | 19:05 |
dansmith | melwitt: I wonder if I'm failing this test because of legit lack of cells support in consoleauth? http://logs.openstack.org/37/458537/9/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/fe108a8/logs/testr_results.html.gz | 19:08 |
* dansmith goes hunting | 19:08 | |
melwitt | dansmith: I would guess so. looking to see if we do a target cell for getting the vnc console | 19:12 |
melwitt | I can't remember what the state of the compute rpcapi mq switching is | 19:13 |
dansmith | yeah, I think it might actually be in the proxy thing | 19:13 |
*** gouthamr has quit IRC | 19:14 | |
melwitt | in nova/api/openstack/compute/remote_consoles.py it gets an instance and passes it down to get_vnc_console, so if the context has already been targeted by the get_instance, it should work | 19:16 |
*** ociuhandu has joined #openstack-nova | 19:17 | |
dansmith | melwitt: right, but that's to get the url, right? then we actually connect through the novncproxy to get to the rfb data, | 19:17 |
dansmith | and it has to know where to proxy to, which is what is failing I think | 19:18 |
dansmith | it makes a consoleauth rpc call, which may be going to the wrong place, I'm guessing | 19:18 |
melwitt | oh, yeah. must be | 19:18 |
*** abhi89 has quit IRC | 19:18 | |
dansmith | although I don't see anything barfing about an rpc timeout or anything | 19:18 |
melwitt | yeah, dtp's change made the consoleauth calls stay at the API and that call shouldn't try to mq switch | 19:19 |
melwitt | I can never remember how the consoles work. get the url first, now looking for where it does the consoleauth part | 19:20 |
dansmith | yeah I have to re-learn each time | 19:21 |
melwitt | okay, the consoleauth is in compute/api, in get_vnc_console, so that should be fine I think | 19:22 |
dansmith | that's the thing that authorizes, not what checks right? | 19:23 |
dansmith | yeah | 19:23 |
dansmith | so we go there, create a token, return a url | 19:23 |
*** kaisers_ has quit IRC | 19:23 | |
dansmith | then we connect to the proxy, do a websocket and expect rfb data | 19:23 |
*** kaisers has joined #openstack-nova | 19:23 | |
dansmith | but we have to check the token in there again, and we do a rpc call to do it | 19:23 |
dansmith | https://github.com/openstack/nova/blob/master/nova/console/websocketproxy.py#L105-L106 | 19:24 |
*** cfriesen has quit IRC | 19:24 | |
*** gouthamr has joined #openstack-nova | 19:24 | |
melwitt | okay, proxy is running at the API, so is consoleauth, seems like it should be okay | 19:25 |
dansmith | but the tokens are in the cell db, no? | 19:25 |
* dansmith forgets | 19:25 | |
melwitt | tokens are in memcached | 19:25 |
melwitt | why are there no errors in the logs anywhere :| | 19:26 |
dansmith | oh, heh, and so the consoleauth service is just there to ask memcache for us for checking tokens? | 19:27 |
dansmith | yeah, that's the weird thing | 19:27 |
melwitt | that's my understanding, yeah | 19:27 |
*** hieulq_ has quit IRC | 19:28 | |
dansmith | yeah, looks like it | 19:28 |
dansmith | wtf | 19:28 |
dansmith | oh no, | 19:28 |
dansmith | it also calls to compute | 19:28 |
*** pchavva has quit IRC | 19:28 | |
dansmith | to validate a port | 19:28 |
dansmith | but it does the mapping lookup and the target | 19:29 |
dansmith | so yeah seems like maybe no logs because it's mostly working | 19:29 |
melwitt | I dunno if this is relevant but I don't see a log in consoleauth for the test you're failing http://logs.openstack.org/37/458537/9/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/fe108a8/logs/screen-n-cauth.txt.gz?#_May_04_18_01_45_091782 | 19:29 |
*** dave-mccowan has quit IRC | 19:30 | |
dansmith | I do see an authorize console report though | 19:30 |
dansmith | not for that test though yeah | 19:31 |
melwitt | yeah, based on the project/user names from the context | 19:31 |
*** smatzek has joined #openstack-nova | 19:31 | |
dansmith | so, I see the corresponding things in n-api.log for that server actions test, | 19:33 |
dansmith | but nothing for the vnc test | 19:33 |
dansmith | dub tee eff | 19:33 |
*** kaisers_ has joined #openstack-nova | 19:33 | |
melwitt | I can't even find that server uuid in n-api log | 19:34 |
dansmith | Body: {"console": {"url": "http://198.72.124.114:6080/vnc_auto.html?token=8bfe6807-a8be-44de-bca7-d9087fb2da4a", "type": "novnc"}} | 19:34 |
dansmith | sure looks like it was called by tempest though | 19:34 |
dansmith | hmm, yeah that's super bizarre | 19:34 |
melwitt | yeah, like, wtf | 19:34 |
dansmith | that server uuid is nowhere in the logs | 19:34 |
dansmith | this finished at noon, so it has to be without the journal usage | 19:35 |
melwitt | I found it a few times in n-cpu, but nothing about it spawning | 19:35 |
*** kaisers has quit IRC | 19:35 | |
melwitt | just accounting stuff | 19:36 |
dansmith | o.O | 19:36 |
dansmith | I also noticed what looked like a few restarts of the api service at the top of the log | 19:36 |
dansmith | like in the tripleo logs, but that's because of puppet I think | 19:37 |
dansmith | so I wonder if we have an api worker that isn't actually logging | 19:37 |
*** baoli has quit IRC | 19:37 | |
bauzas | dansmith: edleafe: jaypipes: FWIW, I was writing some UTs for the scheduler-claims | 19:37 |
*** baoli has joined #openstack-nova | 19:38 | |
dansmith | mriedem: maybe those are the two api worker threads starting actually | 19:38 |
jaypipes | bauzas: cool. will review promptly when you push. | 19:38 |
bauzas | edleafe: could you pease upload your last point for https://review.openstack.org/#/c/460177/ for leaving me providing UTs ? | 19:38 |
dansmith | sdague: can you think of any reason why we might be missing logs from one of the api workers or something like that? | 19:39 |
melwitt | dansmith: I dunno, on the surface it looks like we have to be missing some logs | 19:39 |
dansmith | melwitt: yeah, seems like impossible that we're not missing logs | 19:39 |
mriedem | dansmith: haven't been following - did you pull the journal down and check there? | 19:40 |
dansmith | mriedem: was just about to | 19:40 |
*** brault has joined #openstack-nova | 19:40 | |
*** avolkov has quit IRC | 19:41 | |
*** hieulq_ has joined #openstack-nova | 19:41 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Add BDM to InstancePayload https://review.openstack.org/448779 | 19:41 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Add keypairs field to InstancePayload https://review.openstack.org/419730 | 19:41 |
dansmith | I only see it in nova-compute logs | 19:42 |
*** kfarr has joined #openstack-nova | 19:43 | |
dansmith | in the journal I mean | 19:43 |
*** dave-mccowan has joined #openstack-nova | 19:43 | |
dansmith | I see it in scheduler too | 19:44 |
dansmith | but no api or anywhere else | 19:44 |
melwitt | dansmith: since you're depending on devstack change, did you rebase that to get the use_journal change. it looks like you did already | 19:44 |
dansmith | and I see it in neutron-server | 19:45 |
*** brault_ has joined #openstack-nova | 19:45 | |
*** dave-mcc_ has joined #openstack-nova | 19:45 | |
dansmith | melwitt: yep | 19:45 |
sdague | dansmith: what's going on? | 19:45 |
dansmith | sdague: we think we're missing nova-api logs | 19:45 |
dansmith | well, and nova-compute logs actually | 19:45 |
dansmith | we see instances that were booted and used by tempest, | 19:45 |
dansmith | but zero evidence of that uuid in nova api logs, very little in compute (not even the boot part) | 19:46 |
dansmith | in scheduler, and neutron | 19:46 |
dansmith | so it clearly exists | 19:46 |
dansmith | but it's like shockingly absent from the other services | 19:46 |
*** crushil_ has quit IRC | 19:46 | |
dansmith | sdague: that's the case in the text logs and in the raw journal I pulled down | 19:47 |
dansmith | (unified journal ftmfw by the way) | 19:47 |
sdague | right, they'll be the same | 19:47 |
sdague | ok, give me the example in question | 19:47 |
dansmith | well, I know they should be, but.. checking :) | 19:47 |
dansmith | http://logs.openstack.org/37/458537/9/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/fe108a8/logs/ | 19:47 |
dansmith | instance db767e42-2dbe-48d7-868d-b766d9879eb6 | 19:47 |
*** harlowja has joined #openstack-nova | 19:48 | |
*** dave-mccowan has quit IRC | 19:48 | |
*** brault has quit IRC | 19:48 | |
dansmith | and if you look for the NoVNCConsoleTestJSON tenant/user, | 19:48 |
dansmith | you get one hit in scheduler and nothing else | 19:48 |
sdague | so, it is entirely possible that the default rate limitters for journald are getting tripped | 19:49 |
sdague | I wonder if we can find out about that | 19:49 |
dansmith | oh interesting | 19:49 |
sdague | dansmith: you have the journals all down already? | 19:49 |
dansmith | sdague: just one journal, right? | 19:49 |
sdague | search for -k systemd | 19:49 |
dansmith | devstack.journal.gz? | 19:49 |
sdague | well, there are usual multiples of them because they roll over | 19:50 |
sdague | but, yeh, only looks like one in there | 19:50 |
dansmith | -k is dmesg, which you can't do on a file | 19:50 |
dansmith | unless you mean something else | 19:50 |
dansmith | -u? | 19:50 |
dansmith | journalctl | grep systemd yields not much of relevance | 19:51 |
sdague | -t | 19:51 |
sdague | sorry | 19:51 |
dansmith | same as the grep | 19:51 |
edleafe | bauzas: should be ready to upload soon. I've already created a unit test. | 19:52 |
*** salv-orlando has joined #openstack-nova | 19:52 | |
sdague | ok, so default seems to be 1000 messages per 30s | 19:52 |
sdague | I am sure we are exceeding that | 19:52 |
dansmith | surely at startup | 19:52 |
sdague | let me put in a fix for that | 19:54 |
dansmith | cool thanks | 19:54 |
dansmith | link me and I'll depend on it and recheck | 19:54 |
*** cdent has quit IRC | 19:54 | |
*** hieulq_ has quit IRC | 19:55 | |
melwitt | dansmith: so the websocketproxy goes direct to the instance? it doesn't need to know the compute host MQ right | 19:56 |
dansmith | it goes direct to the host | 19:56 |
dansmith | consoleauth validate_token does talk to the compute mq, but it does the looku | 19:56 |
melwitt | okay. we'll just have to see the logs then, nothing is jumping out at me | 19:57 |
dansmith | I think the token in memcache must have the ip/port that the proxy needs to connect to | 19:57 |
dansmith | yeah | 19:57 |
dansmith | hopefully something dumb | 19:57 |
dansmith | simple and dumb | 19:57 |
dansmith | like me. | 19:57 |
sdague | dansmith: I think this is it - https://review.openstack.org/462725 | 19:57 |
melwitt | lol :P | 19:57 |
*** cdent has joined #openstack-nova | 19:58 | |
openstackgerrit | Dan Smith proposed openstack/nova master: DNM: Canary test against multi-cell devstack https://review.openstack.org/458537 | 19:58 |
dansmith | sdague: ack thanks | 19:58 |
dansmith | sdague: if that's really happening we probably want to merge that like.. yesterday :P | 19:58 |
mriedem | gibi: i rebased and commented in https://review.openstack.org/#/c/419730/ for adding keypairs to InstancePayload - summary is i don't think we should lazy load instance.keypairs for every instance action, for the same reason I didn't want to do that for instance.tags | 19:59 |
sdague | dansmith: yep agreed, I want to make sure it looks sane | 20:00 |
sdague | hey, it's all about learning new things :) | 20:00 |
*** lbragstad_alt has joined #openstack-nova | 20:02 | |
dansmith | sdague: I'm not complaining :) | 20:02 |
sdague | also, what's up with the linuxbridge job? | 20:03 |
sdague | that seem things to be hating me today | 20:03 |
*** Jack_Iv has quit IRC | 20:04 | |
dansmith | sdague: that was an impressively incomprehensible exclamation | 20:04 |
*** Jack_Iv has joined #openstack-nova | 20:04 | |
openstackgerrit | Ed Leafe proposed openstack/nova master: Don't delete allocation if instance being scheduled https://review.openstack.org/460177 | 20:04 |
edleafe | bauzas: Here ya go ^^ | 20:04 |
*** Apoorva_ has joined #openstack-nova | 20:05 | |
dansmith | what is this nonsense? http://logs.openstack.org/79/462679/1/check/gate-nova-tox-functional-ubuntu-xenial/c760042/testr_results.html.gz | 20:05 |
bauzas | edleafe: okay, I'll update the next top changes | 20:05 |
melwitt | dansmith: socket_options, duh | 20:07 |
* dansmith glares at melwitt | 20:07 | |
melwitt | :D | 20:07 |
*** iceyao has joined #openstack-nova | 20:08 | |
*** eharney has quit IRC | 20:09 | |
*** Apoorva has quit IRC | 20:09 | |
*** liverpooler has quit IRC | 20:09 | |
cdent | dansmith: might be mismatched versions of urllib3 and wsgi-intercept | 20:10 |
cdent | I saw that a while back but thought it was better. | 20:11 |
dansmith | cdent: it's broken is what it is | 20:11 |
dansmith | looks rather fundamental, so wondering if it's worth a recheck | 20:11 |
cdent | yessir, but I think it got better. are your deps maybe stale? | 20:11 |
dansmith | that patch is new from today | 20:11 |
cdent | hmmm | 20:11 |
dansmith | I guess edleafe already rechecked it | 20:12 |
cdent | i'll mess with it a bit | 20:12 |
cdent | (locally) | 20:12 |
*** iceyao has quit IRC | 20:12 | |
mriedem | can't provide new key_name on rebuild? | 20:12 |
melwitt | mriedem: I don't think you can specify anything other than the image. there have been requests to allow new userdata and such, but the answer has been "just boot a new instance" if you're looking at re-imaging it anyway | 20:17 |
*** hongbin has quit IRC | 20:17 | |
mriedem | do we rebuild the config drive during rebulid? | 20:18 |
melwitt | I don't think so | 20:18 |
melwitt | but not sure, not a config drive expert | 20:18 |
mriedem | i'm looking, just this keypair notification patch has me thinking about weird stuf | 20:19 |
mriedem | https://review.openstack.org/#/c/419730/ | 20:19 |
mriedem | b/c i don't want to lazy-load instance.keypairs for every gd instance action notification | 20:19 |
melwitt | we definitely don't change anything on it. I don't know if it is "recreated" at all though | 20:19 |
mriedem | jaypipes: dansmith: ^ same predicament as with the instance.tags patch | 20:20 |
mriedem | for notifications that is | 20:20 |
mriedem | which would include every vm/task state change if you enabled that option... | 20:21 |
mriedem | CONF.notifications.notify_on_state_change i mean | 20:21 |
melwitt | preload all the things | 20:22 |
*** slaweq has joined #openstack-nova | 20:22 | |
melwitt | jk | 20:22 |
mriedem | well someone proposed that for instance.tags, | 20:22 |
mriedem | preload by default | 20:22 |
jroll | just keep the whole db in RAM, wcpgw | 20:22 |
mriedem | we nacked that | 20:22 |
melwitt | see, jroll knows what I'm talkin about | 20:23 |
openstackgerrit | Alex Szarka proposed openstack/nova master: Transform instance.exists notification https://review.openstack.org/403660 | 20:23 |
mriedem | i worry that some of the perf issues people are reporting in newton/ocata are due to all of the versioned notifications doing lazy-loads like crazy | 20:23 |
*** damien_r has joined #openstack-nova | 20:23 | |
jroll | melwitt: :D | 20:24 |
jroll | feels like state change notifications shouldn't need tags, keypairs, etc | 20:24 |
*** Jack_Iv has quit IRC | 20:24 | |
mriedem | that's what i've said | 20:25 |
mriedem | and pushed back on | 20:25 |
*** Jack_Iv has joined #openstack-nova | 20:25 | |
mriedem | nor bdms... | 20:25 |
mriedem | we're basically throwing the kitchen sink at the notification consumer | 20:25 |
mriedem | because (1) we can and (2) we don't know what notification consumers even want/need | 20:26 |
jaypipes | mriedem: currently reviewing dansmith's target_cell patches. will get to those a little later (all the notification ones) | 20:26 |
mriedem | jaypipes: you don't really need to probably, just thinking out loud | 20:26 |
mriedem | i've already -1ed | 20:26 |
jroll | mriedem: and I just realized (3) notifications are considered lossy, so we can't assume consumers have state n-1 to diff properly | 20:26 |
melwitt | :\ a lot of people use notifications, I thought. so one would think we would have some idea what notification consumers want | 20:26 |
jroll | is the most sensible reason to me | 20:26 |
mriedem | melwitt: rebuild calls virt driver destroy and then spawn, so i'd think we're rebuilding the config drive... | 20:27 |
*** jose-phillips has joined #openstack-nova | 20:27 | |
mriedem | but i guess you just get the same key as before | 20:27 |
*** ociuhandu has quit IRC | 20:28 | |
melwitt | mriedem: yeah, I don't know what the term "rebuild" means in the context of config drive. I know we don't change it | 20:28 |
mriedem | melwitt: the only consumers i know of really are ceilometer, mistral and searchlight | 20:28 |
*** kfarr has quit IRC | 20:28 | |
mriedem | not sure how much ceilometer is used, and since they didn't get the stuff they needed in the notifications years ago, they just hit the rest api | 20:28 |
mriedem | not sure about mistral or searchlight | 20:28 |
melwitt | notifications were supposed to be the answer to instance actions records. I have used those before in a previous life. and notifications, but just to know the history of what happened to an instance when the customer comes calling | 20:29 |
*** Jack_Iv has quit IRC | 20:29 | |
mriedem | melwitt: what i meant by rebuild was really recreate, since destroy_disks=True when driver.destroy is called during rebuild | 20:33 |
mriedem | so i think that will delete the config drive for the instance from the host, | 20:33 |
mriedem | and then spawn would recreate it | 20:33 |
*** dave-mcc_ has quit IRC | 20:34 | |
cdent | edleafe, dansmith: I narrowed down the sitch on https://review.openstack.org/#/c/462679/ and left a comment there | 20:34 |
cdent | I can push the fix if desired | 20:34 |
melwitt | mriedem: I see what you mean. that makes sense with the requests too since it's being recreated anyhow and people think, why not allow metadata changes | 20:35 |
*** liangy has joined #openstack-nova | 20:35 | |
mriedem | right so i for whatever reason thought you could provide a new key during a rebuild, | 20:36 |
mriedem | and since i was -1ing that review saying scope the full keypair object to just a couple of notifications, like instance.create, | 20:36 |
mriedem | i was thinking instance.update too, but the key doesn't actually change, so we shouldn't need the full keypair in there | 20:36 |
mriedem | so i'm thinking we just put the full keypair in the instance.create notification and call it good | 20:37 |
melwitt | I see | 20:37 |
mriedem | and put instance.key_name in everything else | 20:37 |
mriedem | since that's just stored on the instance record and not lazy-loaded | 20:37 |
melwitt | IMHO it seems like notifications are a lot more fine-grained than I would have expected or thought needed | 20:37 |
*** lyan has quit IRC | 20:37 | |
mriedem | fine grained as in providing more detail than you'd expect? | 20:38 |
mriedem | or fine-grained as in different payloads for different notifications? | 20:38 |
melwitt | yeah, I guess on create you already have the key_name from the request, what else do you need | 20:38 |
mriedem | we also set instance.keypairs in the api during create | 20:38 |
mriedem | so we don't need to lazy-load it | 20:38 |
jaypipes | dansmith: if you have time: https://review.openstack.org/#/c/457782/ | 20:39 |
melwitt | mriedem: I meant fine-grained like rich subobjects for everything | 20:39 |
*** lyan_ has joined #openstack-nova | 20:40 | |
melwitt | I didn't even know keypair was a field on instance | 20:40 |
mriedem | yeah that's the other argument is that the instance action payloads already have the full flavor and instance info_cache | 20:40 |
melwitt | I thought there was only the key name if anything | 20:40 |
mriedem | we started storing instance.keypairs when we moved keypairs to the instance_extras table | 20:41 |
mriedem | and by we i mean dan | 20:41 |
mriedem | well, | 20:41 |
melwitt | ah, right. I forgot about that even though it was mentioned to me recently | 20:41 |
mriedem | or maybe that was the move to the api db | 20:41 |
mriedem | i get confused | 20:41 |
*** ociuhandu has joined #openstack-nova | 20:41 | |
melwitt | yeah I can't remember why. probably to avoid having to do a lookup in compute or something | 20:42 |
*** hamzy has quit IRC | 20:42 | |
*** liangy has quit IRC | 20:43 | |
*** cleong has quit IRC | 20:44 | |
*** cdent has quit IRC | 20:47 | |
*** hongbin has joined #openstack-nova | 20:50 | |
*** moshele has joined #openstack-nova | 20:55 | |
*** gouthamr has quit IRC | 20:56 | |
*** salv-orl_ has joined #openstack-nova | 20:56 | |
edleafe | cdent: dansmith: I'll push the fix. | 20:57 |
dansmith | ah | 20:57 |
dansmith | that was an odd failure | 20:57 |
*** salv-orlando has quit IRC | 20:59 | |
*** thorst_afk has quit IRC | 20:59 | |
mriedem | hemna: so looking at https://review.openstack.org/#/c/382097/47/etc/cinder/rootwrap.d/volume.filters and https://review.openstack.org/#/c/443951/10/etc/nova/rootwrap.d/compute.filters i thought we were supposed to push that common root helper stuff into os-brick? | 21:00 |
mriedem | doesn't doing a rootwrap filter in cinder and nova kind of defeating the purpose? | 21:00 |
*** lucasxu has quit IRC | 21:01 | |
mriedem | heh it's not even used in the volume driver code in nova | 21:02 |
melwitt | mriedem: I found there is a bug in oslo.log too regarding the unicode thing https://review.openstack.org/#/c/462735 | 21:02 |
mriedem | melwitt: nice catch | 21:03 |
*** thorst_afk has joined #openstack-nova | 21:03 | |
*** brault_ has quit IRC | 21:03 | |
melwitt | :) | 21:04 |
johnsom | Hi nova folks. Is there a known issue where cirros instances come up ACTIVE in nova show, but there is no console.log content? | 21:04 |
*** Apoorva_ has quit IRC | 21:04 | |
mtreinish | johnsom: when I hit that locally on my cloud it was because the glance image was empty... | 21:05 |
*** Apoorva has joined #openstack-nova | 21:05 | |
* mtreinish was just writing a slide about that exact issue for his talk next week | 21:05 | |
johnsom | Yeah, two of us just restacked today and cirros and our image both won't come up | 21:06 |
mtreinish | johnsom: I would check the image size in glance. If it's zero bytes that's probably it | 21:07 |
johnsom | Well, glance is showing content for the cirros image... hmmm | 21:07 |
mtreinish | hmm, ok maybe it's something else :) | 21:07 |
johnsom | The console logs aren't in this new fanlged journalctl logs are they? | 21:07 |
edleafe | crap, looks like cdent has left | 21:07 |
mtreinish | johnsom: that's independent. THat's just service logs, because the services are running under systemd | 21:07 |
*** thorst_afk has quit IRC | 21:08 | |
mtreinish | console logs are grabbed from the virt driver | 21:08 |
edleafe | I don't see where he is referring to in his comment. dansmith? | 21:08 |
johnsom | Yeah, I wouldn't have expected it. Especially because there is a zero byte console.log file sitting there | 21:08 |
*** damien_r1 has joined #openstack-nova | 21:08 | |
mtreinish | johnsom: I would check the n-cpu log | 21:08 |
*** damien_r2 has joined #openstack-nova | 21:08 | |
edleafe | oh, nvm | 21:09 |
edleafe | I had the unit test test_resource_tracker.py open, not the functional | 21:09 |
openstackgerrit | Alex Szarka proposed openstack/nova master: Refactor instance-exists comment and test https://review.openstack.org/462738 | 21:09 |
*** bmace has quit IRC | 21:09 | |
*** bmace has joined #openstack-nova | 21:09 | |
*** awaugama has quit IRC | 21:09 | |
dansmith | edleafe: I didn't look close, I just read what he said and thought "mocks, okay, complicated" | 21:10 |
edleafe | heh | 21:11 |
*** eric_young has quit IRC | 21:11 | |
openstackgerrit | Alex Szarka proposed openstack/nova master: Transform instance.exists notification https://review.openstack.org/403660 | 21:11 |
openstackgerrit | Alex Szarka proposed openstack/nova master: Refactor instance-exists comment and test https://review.openstack.org/462738 | 21:11 |
johnsom | Yeah, I started there. No ERROR records and it shows 2 vcpus allocated. It looks normal to me. Ok, I will dig in. I just didn't want to burn a bunch of time if this was a known issue. | 21:11 |
dansmith | edleafe: assume he means this: https://pastebin.com/x6MJut7P | 21:12 |
openstackgerrit | Ed Leafe proposed openstack/nova master: Don't delete allocation if instance being scheduled https://review.openstack.org/460177 | 21:12 |
openstackgerrit | Ed Leafe proposed openstack/nova master: Add a convenience attribute for reportclient https://review.openstack.org/462679 | 21:12 |
*** annegentle has quit IRC | 21:12 | |
edleafe | dansmith: yep ^^ | 21:12 |
*** damien_r1 has quit IRC | 21:12 | |
*** damien_r has quit IRC | 21:13 | |
*** kiwi_rot has joined #openstack-nova | 21:13 | |
*** kiwi_rot has quit IRC | 21:18 | |
johnsom | mtreinish Ok, I am suspecting the new qemu packages from ubuntu. Those processes are sitting at 120% in top | 21:18 |
*** dimtruck is now known as zz_dimtruck | 21:20 | |
*** damien_r2 has quit IRC | 21:22 | |
*** marst has quit IRC | 21:23 | |
*** gouthamr has joined #openstack-nova | 21:26 | |
*** smatzek has quit IRC | 21:27 | |
*** Guest73837 has quit IRC | 21:28 | |
*** marst has joined #openstack-nova | 21:30 | |
*** moshele has quit IRC | 21:31 | |
*** Guest73837 has joined #openstack-nova | 21:33 | |
*** jerrygb has quit IRC | 21:35 | |
mtreinish | johnsom: that could be it | 21:37 |
johnsom | Yeah, digging, will try a rollback | 21:37 |
*** karthiks has quit IRC | 21:38 | |
*** annegentle has joined #openstack-nova | 21:42 | |
*** mingyu has quit IRC | 21:43 | |
*** baoli has quit IRC | 21:44 | |
*** lbragstad has quit IRC | 21:45 | |
*** mingyu has joined #openstack-nova | 21:46 | |
openstackgerrit | melanie witt proposed openstack/nova master: DNM: Revert "Use six.text_type() when logging Instance object" https://review.openstack.org/462744 | 21:47 |
openstackgerrit | Ed Leafe proposed openstack/nova master: WIP: Delete existing allocations for move operations or retries https://review.openstack.org/460433 | 21:48 |
openstackgerrit | Ed Leafe proposed openstack/nova master: WIP: Allocate instance by the conductor https://review.openstack.org/460455 | 21:48 |
mriedem | melwitt: that's not going to work | 21:49 |
mriedem | unless we have a ci job that uses LIBS_FROM_GIT=oslo.log | 21:49 |
mriedem | the depends-on won't work | 21:50 |
melwitt | mriedem: it won't make a new oslo.log? oh. I wasn't sure which things Depends-On works with | 21:50 |
mriedem | efried had the same question earlier this week | 21:50 |
mriedem | basically, | 21:50 |
mriedem | can't depends-on to project-config changes or libraries/clients | 21:50 |
mriedem | the latter requires LIBS_FROM_GIT | 21:50 |
*** salv-orl_ has quit IRC | 21:51 | |
*** mvk has joined #openstack-nova | 21:51 | |
efried | Good news is that it seems fairly straightforward to get a new release cut and through requirements. | 21:52 |
melwitt | mriedem: does Depends-On work on changes in other projects like cinder for example? | 21:52 |
mriedem | wtf are we not running the releasenotes job anymore? | 21:52 |
mriedem | melwitt: yes | 21:52 |
melwitt | okay. oh, right because devstack always pulls those from git | 21:53 |
melwitt | I was just curious what the log would look like with the Instance object if we didn't do anything to convert it, with the oslo.log change | 21:54 |
mriedem | melwitt: you could do a devstack change that sets LIBS_FROM_GIT=oslo.log, | 21:55 |
mriedem | and have that depends on the nova and oslo.log changes | 21:55 |
mriedem | i think | 21:55 |
melwitt | because the ovo repr encodes to ascii, which has the chars with the \x escapes in it. I guess it would just look like in my unit test, with \x in it | 21:55 |
johnsom | mtreinish Yep, downgrade packages work. | 21:56 |
johnsom | apt-get install seabios=1.8.2-1ubuntu1 qemu-system-sparc=1:2.5+dfsg-5ubuntu10.11 qemu-system-ppc=1:2.5+dfsg-5ubuntu10.11 qemu-system-arm=1:2.5+dfsg-5ubuntu10.11 qemu-kvm=1:2.5+dfsg-5ubuntu10.11 qemu-system-x86=1:2.5+dfsg-5ubuntu10.11 qemu-system-mips=1:2.5+dfsg-5ubuntu10.11 qemu-system-misc=1:2.5+dfsg-5ubuntu10.11 | 21:56 |
johnsom | restart nova and life is better | 21:56 |
*** hongbin has quit IRC | 22:03 | |
*** annegentle has quit IRC | 22:03 | |
*** annegentle has joined #openstack-nova | 22:04 | |
*** annegentle has quit IRC | 22:04 | |
*** Apoorva_ has joined #openstack-nova | 22:05 | |
*** hongbin has joined #openstack-nova | 22:05 | |
*** artom has joined #openstack-nova | 22:06 | |
mriedem | sdague: in https://review.openstack.org/#/c/435432/11/releasenotes/notes/project_id_validation-568d31c13c3ef735.yaml what does "If the user has enough permissions in user" mean? | 22:06 |
mriedem | the 'in user' part | 22:06 |
*** jamesdenton has quit IRC | 22:08 | |
*** Apoorva has quit IRC | 22:09 | |
*** mingyu has quit IRC | 22:09 | |
*** mingyu has joined #openstack-nova | 22:09 | |
*** edmondsw has quit IRC | 22:12 | |
*** mingyu has quit IRC | 22:13 | |
mriedem | nvm i'll clean it up | 22:14 |
*** edmondsw has joined #openstack-nova | 22:15 | |
*** edmondsw has quit IRC | 22:20 | |
*** slaweq has quit IRC | 22:22 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Verify project id for flavor access calls https://review.openstack.org/435432 | 22:28 |
mriedem | oomichi: ^ i cleaned that one up | 22:30 |
*** xyang1 has quit IRC | 22:31 | |
*** hongbin_ has joined #openstack-nova | 22:33 | |
*** hongbin has quit IRC | 22:33 | |
oomichi | mriedem: thanks, lets close the bug | 22:34 |
dansmith | sdague: melwitt: shockingly, I now have logs for that instance | 22:36 |
*** jerrygb has joined #openstack-nova | 22:37 | |
melwitt | you mean with sdague's change you do? | 22:37 |
dansmith | melwitt: checkit, like I thought: http://logs.openstack.org/37/458537/10/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/0601a5e/logs/screen-n-cauth.txt.gz#_May_04_20_47_48_620783 | 22:38 |
dansmith | melwitt: yeah | 22:38 |
melwitt | validate_console_port | 22:39 |
dansmith | dinnuh I say? | 22:40 |
melwitt | yeah | 22:40 |
dansmith | I shall flog that tomorrow | 22:40 |
melwitt | cool. somehow instance must not be targeted to the right MQ even though there's a target_cell call in there | 22:41 |
*** jerrygb has quit IRC | 22:41 | |
dansmith | something. | 22:44 |
*** mlavalle has quit IRC | 22:45 | |
melwitt | strange. not seeing what's wrong there | 22:47 |
mnaser | i've been flying around the openstack api but this might be a little crazy. is it possible to remove an existing ephermal drive and replace it by cinder volume... on an existing instance? | 22:48 |
melwitt | the context gets wiped out after the context manager exits, but all of the calls are under it | 22:48 |
mnaser | so converting to boot from cinder for an existing VM (bdm stuff relevant at all here?) | 22:48 |
*** tbachman has quit IRC | 22:49 | |
*** salv-orlando has joined #openstack-nova | 22:51 | |
*** winston-d_ has quit IRC | 22:53 | |
*** gouthamr has quit IRC | 22:55 | |
*** Apoorva has joined #openstack-nova | 22:56 | |
*** Apoorva_ has quit IRC | 22:56 | |
*** salv-orlando has quit IRC | 22:58 | |
*** artom has quit IRC | 23:02 | |
*** brault has joined #openstack-nova | 23:04 | |
*** catinthe_ has quit IRC | 23:08 | |
*** tbachman has joined #openstack-nova | 23:08 | |
*** brault has quit IRC | 23:09 | |
*** lyan_ has quit IRC | 23:14 | |
jaypipes | mriedem, dansmith, edleafe, alex_xu: your prompt attention to https://review.openstack.org/#/q/topic:cleanups+project:openstack/os-traits would be appreciated. I'd like to cut a 0.3 os-traits release ASAP because I need to use it in some of the new placement work... | 23:16 |
dansmith | jaypipes: I -1'ed the fsck out of that last patch | 23:18 |
dansmith | utter garbage! | 23:18 |
jaypipes | dansmith: fixed. :) | 23:20 |
*** sdague has quit IRC | 23:20 | |
dansmith | jaypipes: that was almost a very embarrassing event | 23:23 |
jaypipes | :) | 23:23 |
*** baoli has joined #openstack-nova | 23:24 | |
*** baoli has quit IRC | 23:24 | |
*** hamzy has joined #openstack-nova | 23:24 | |
*** erlon has quit IRC | 23:24 | |
*** baoli has joined #openstack-nova | 23:25 | |
*** edmondsw has joined #openstack-nova | 23:25 | |
mriedem | ha | 23:29 |
mriedem | got you both | 23:29 |
*** edmondsw has quit IRC | 23:30 | |
dansmith | mriedem: luckily I don't care about my ratio on os-traits, so impede progress all you want buddy :) | 23:30 |
mriedem | i'll always remember | 23:32 |
mriedem | like your love of pizza and cookies and complete sentences | 23:32 |
dansmith | heh | 23:32 |
mriedem | jaypipes: on the last one, | 23:33 |
mriedem | that regex isn't enforced in the api schema from what i can tell | 23:33 |
mriedem | https://github.com/openstack/nova/blob/master/nova/api/openstack/placement/handlers/trait.py#L29 | 23:33 |
mriedem | or does that just mean for custom traits? | 23:33 |
jaypipes | yeah, that's just custom traits. | 23:34 |
mriedem | well, https://review.openstack.org/#/c/462759/2/os_traits/hw/cpu/x86.py is a standard trait no? | 23:34 |
jaypipes | no | 23:35 |
jaypipes | that's a module. | 23:35 |
jaypipes | hw.cpu.x86.SSE42 is a standard trait *symbol*, but the trait string is HW_CPU_X86_SSE42 | 23:35 |
mriedem | but based on the api schema, | 23:36 |
mriedem | i could create a trait that's named HW_CPU_X86_AES-NI right? | 23:36 |
*** chyka has quit IRC | 23:37 | |
mriedem | or are you only allowed to create custom traits via the API? https://github.com/openstack/nova/blob/master/nova/api/openstack/placement/handlers/trait.py#L99 | 23:38 |
mriedem | and standard traits come from the lib? | 23:38 |
*** chyka has joined #openstack-nova | 23:38 | |
*** chyka has quit IRC | 23:43 | |
jaypipes | mriedem: correct. | 23:44 |
jaypipes | mriedem: meh, fixed that README typo. | 23:47 |
*** artom has joined #openstack-nova | 23:50 | |
jaypipes | mriedem: I just want to keep the standard trait naming consistent. | 23:51 |
mriedem | alright all done | 23:52 |
*** Apoorva has quit IRC | 23:52 | |
*** hongbin_ has quit IRC | 23:53 | |
*** salv-orlando has joined #openstack-nova | 23:54 | |
*** Apoorva has joined #openstack-nova | 23:54 | |
*** salv-orlando has quit IRC | 23:58 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!