*** Sukhdev has joined #openstack-nova | 00:03 | |
*** gjayavelu has quit IRC | 00:05 | |
*** andymaier_ has quit IRC | 00:06 | |
*** pushkaru has quit IRC | 00:07 | |
*** pushkaru has joined #openstack-nova | 00:07 | |
*** pushkaru has quit IRC | 00:08 | |
*** pushkaru has joined #openstack-nova | 00:08 | |
*** geekinutah has quit IRC | 00:09 | |
*** pushkaru has quit IRC | 00:11 | |
*** ssurana has quit IRC | 00:13 | |
*** pushkaru has joined #openstack-nova | 00:14 | |
*** Sukhdev has quit IRC | 00:21 | |
*** Sukhdev has joined #openstack-nova | 00:24 | |
*** pushkaru has quit IRC | 00:27 | |
*** Sukhdev has quit IRC | 00:27 | |
*** diogogmt has quit IRC | 00:28 | |
*** hemna is now known as hemnafk | 00:30 | |
*** andymaier_ has joined #openstack-nova | 00:31 | |
*** haukebruno has joined #openstack-nova | 00:32 | |
*** Rodrigo_BR has joined #openstack-nova | 00:33 | |
Rodrigo_BR | Live Migration failure: Unsafe migration: Migration may lead to data corruption if disks use cache != none | 00:33 |
---|---|---|
Rodrigo_BR | Can I perform live migration using NFS backend ? | 00:34 |
haukebruno | Rodrigo_BR, in general yes. | 00:35 |
Rodrigo_BR | <haukebruno> But I recive this error: Live Migration failure: Unsafe migration: Migration may lead to data corruption if disks use cache != none | 00:36 |
haukebruno | Rodrigo_BR, but not sure about that cache thing. you could read that issue 'a lot' these days, also in ask.openstack.org | 00:37 |
haukebruno | Rodrigo_BR, are you using nfsv3 or v4? we had some trouble using v4 so far and switched back to v3 | 00:38 |
Rodrigo_BR | <haukebruno> I am using nfsv3 with the netapp driver | 00:39 |
*** qeelee has joined #openstack-nova | 00:39 | |
*** angdraug has quit IRC | 00:40 | |
haukebruno | Rodrigo_BR, did you set any cache related options in your config files? we are using netapp too, but not with the specifig driver so far just mounting nfs on every compute node | 00:40 |
Rodrigo_BR | <haukebruno> I posted in https://ask.openstack.org/en/question/89302/migration-operation-has-aborted/ | 00:40 |
Rodrigo_BR | <haukebruno> nfs_mount_options=rw,bg,hard,nointr,tcp,vers=3,timeo=600,rsize=32768,wsize=32768,actimeo=0,sec=sys | 00:41 |
*** ljxiash has joined #openstack-nova | 00:42 | |
haukebruno | Rodrigo_BR, are you using nfs too for /var/lib/nova/instances? | 00:42 |
Rodrigo_BR | <haukebruno> Yes ! I am | 00:43 |
*** pwnall has quit IRC | 00:43 | |
haukebruno | Rodrigo_BR, sorry, no clue here. you could try block migration as a workaround, but in general live migration should work, especially with an underlying netapp NFS export (we are doing it without problems) | 00:44 |
Rodrigo_BR | <haukebruno> Ok ! thank you for help me | 00:45 |
*** ljxiash has quit IRC | 00:46 | |
*** ljxiash_ has joined #openstack-nova | 00:46 | |
*** ijw has joined #openstack-nova | 00:47 | |
*** ljxiash has joined #openstack-nova | 00:47 | |
*** ljxiash_ has quit IRC | 00:50 | |
*** zenoway has joined #openstack-nova | 00:52 | |
*** ljxiash has quit IRC | 00:52 | |
*** ijw has quit IRC | 00:52 | |
*** ZZelle_ has quit IRC | 00:53 | |
*** liusheng has joined #openstack-nova | 00:55 | |
*** dedery has quit IRC | 00:56 | |
*** zenoway has quit IRC | 00:56 | |
*** akshai has quit IRC | 00:58 | |
*** hoangcx has joined #openstack-nova | 00:59 | |
*** akshai has joined #openstack-nova | 01:02 | |
*** jaypipes has quit IRC | 01:04 | |
*** Swami has quit IRC | 01:07 | |
*** Swami has joined #openstack-nova | 01:07 | |
*** akshai has quit IRC | 01:08 | |
*** Swami has quit IRC | 01:11 | |
*** phuongnh has joined #openstack-nova | 01:14 | |
*** achanda has joined #openstack-nova | 01:14 | |
*** dimtruck is now known as zz_dimtruck | 01:19 | |
*** mnestratov has quit IRC | 01:22 | |
*** ducttape_ has joined #openstack-nova | 01:24 | |
*** zz_dimtruck is now known as dimtruck | 01:25 | |
*** ivase has joined #openstack-nova | 01:25 | |
*** ducttape_ has quit IRC | 01:25 | |
*** ducttape_ has joined #openstack-nova | 01:25 | |
*** EinstCrazy has joined #openstack-nova | 01:27 | |
*** thorst_ has quit IRC | 01:28 | |
*** thorst has joined #openstack-nova | 01:29 | |
*** ivase has quit IRC | 01:29 | |
*** ducttape_ has quit IRC | 01:31 | |
*** akshai has joined #openstack-nova | 01:31 | |
eliqiao | tdurakov: hi | 01:32 |
*** nic has quit IRC | 01:32 | |
*** diogogmt has joined #openstack-nova | 01:35 | |
*** thorst has quit IRC | 01:37 | |
*** mtanino has quit IRC | 01:37 | |
*** yamahata has quit IRC | 01:37 | |
*** akshai_ has joined #openstack-nova | 01:39 | |
*** akshai has quit IRC | 01:42 | |
*** apoorvad has quit IRC | 01:53 | |
*** Sukhdev has joined #openstack-nova | 01:54 | |
Rodrigo_BR | Hi someone can talk about http://libvirt.org/git/?p=libvirt.git;a=blob;f=src/qemu/qemu_migration.c;h=64cbffa59ae07c423660621c8528a39bd599ce5a;hb=HEAD#l2304 | 01:59 |
*** clduser has quit IRC | 02:03 | |
*** Sukhdev has quit IRC | 02:04 | |
*** rhagarty_ has quit IRC | 02:04 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova: Check 'destination_type' instead of 'source_type' in _check_and_transform_bdm https://review.openstack.org/252836 | 02:06 |
*** ijw has joined #openstack-nova | 02:06 | |
openstackgerrit | Nicolas Simonds proposed openstack/nova: Allow use of live snapshots with RBD snapshot/clone https://review.openstack.org/273652 | 02:08 |
*** vilobhmm11 has quit IRC | 02:09 | |
*** unicell has joined #openstack-nova | 02:09 | |
*** rhagarty has joined #openstack-nova | 02:12 | |
*** gyee has quit IRC | 02:15 | |
*** thorst has joined #openstack-nova | 02:16 | |
openstackgerrit | Tim Pownall proposed openstack/nova: Updating while loop for tar extraction in utils.py. https://review.openstack.org/287426 | 02:17 |
*** fawadkhaliq has quit IRC | 02:20 | |
*** zengine has joined #openstack-nova | 02:22 | |
*** thorst has quit IRC | 02:23 | |
*** ducttape_ has joined #openstack-nova | 02:25 | |
*** andymaier_ has quit IRC | 02:26 | |
*** gjayavelu has joined #openstack-nova | 02:32 | |
*** clduser has joined #openstack-nova | 02:32 | |
*** Rodrigo_BR has quit IRC | 02:32 | |
*** gcb has quit IRC | 02:34 | |
*** apoorvad has joined #openstack-nova | 02:36 | |
*** rhagarty has quit IRC | 02:36 | |
*** gjayavelu has quit IRC | 02:37 | |
*** zenoway has joined #openstack-nova | 02:39 | |
*** haomaiwang has joined #openstack-nova | 02:39 | |
openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Nuke cliutils from oslo-incubator https://review.openstack.org/287753 | 02:42 |
*** y_sawai has joined #openstack-nova | 02:43 | |
*** dimtruck is now known as zz_dimtruck | 02:43 | |
*** zenoway has quit IRC | 02:44 | |
*** houming has joined #openstack-nova | 02:44 | |
*** ijw has quit IRC | 02:45 | |
*** haomaiwang has quit IRC | 02:45 | |
*** unicell has quit IRC | 02:47 | |
*** qeelee has quit IRC | 02:49 | |
*** akshai_ has quit IRC | 02:51 | |
*** gcb has joined #openstack-nova | 02:53 | |
*** akshai has joined #openstack-nova | 02:56 | |
*** haomaiwang has joined #openstack-nova | 02:56 | |
*** cdelatte has joined #openstack-nova | 02:59 | |
*** akshai has quit IRC | 03:00 | |
*** haomaiwang has quit IRC | 03:01 | |
*** links has joined #openstack-nova | 03:01 | |
*** lykinsbd has joined #openstack-nova | 03:05 | |
*** lykinsbd has quit IRC | 03:06 | |
*** lykinsbd has joined #openstack-nova | 03:07 | |
*** harlowja_at_home has quit IRC | 03:07 | |
*** lykinsbd has quit IRC | 03:11 | |
*** ijw has joined #openstack-nova | 03:11 | |
*** lykinsbd has joined #openstack-nova | 03:11 | |
*** ijw has quit IRC | 03:11 | |
*** lykinsbd has quit IRC | 03:12 | |
*** ijw has joined #openstack-nova | 03:12 | |
*** haomaiwang has joined #openstack-nova | 03:12 | |
*** zenoway has joined #openstack-nova | 03:15 | |
*** vilobhmm11 has joined #openstack-nova | 03:15 | |
*** haomaiwang has quit IRC | 03:17 | |
*** zenoway has quit IRC | 03:19 | |
*** lykinsbd has joined #openstack-nova | 03:19 | |
*** lykinsbd has quit IRC | 03:19 | |
*** thorst has joined #openstack-nova | 03:21 | |
*** vilobhmm111 has joined #openstack-nova | 03:22 | |
*** vilobhmm11 has quit IRC | 03:25 | |
*** diana_clarke1 has joined #openstack-nova | 03:27 | |
*** thorst has quit IRC | 03:28 | |
*** zengine has quit IRC | 03:28 | |
*** ducttape_ has quit IRC | 03:36 | |
*** fawadkhaliq has joined #openstack-nova | 03:38 | |
alex_xu | oomichi: hi, are you around? | 03:38 |
*** baoli has joined #openstack-nova | 03:39 | |
oomichi | alex_xu: yep | 03:39 |
*** vilobhmm111 has quit IRC | 03:39 | |
*** liusheng has quit IRC | 03:39 | |
alex_xu | oomichi: i'm trying to fix this bug https://bugs.launchpad.net/nova/+bug/1552888 | 03:40 |
openstack | Launchpad bug 1552888 in OpenStack Compute (nova) "ERROR (BadRequest): expected string or buffer" [High,Confirmed] - Assigned to jichenjc (jichenjc) | 03:40 |
*** dims has quit IRC | 03:40 | |
alex_xu | oomichi: I want to write schema for availability_zone like this {'anyOf': [parameter_types.name, {'type': 'null'}]} | 03:40 |
*** liusheng has joined #openstack-nova | 03:40 | |
alex_xu | but I found it still failed at parameter_types.name | 03:40 |
oomichi | alex_xu: jichenjc posted a patch for reverting it, is it merged already now? | 03:41 |
alex_xu | when I create aggregate with None value for availability_zone | 03:41 |
alex_xu | oomichi: not yet | 03:41 |
*** yamahata has joined #openstack-nova | 03:41 | |
alex_xu | oomichi: should we revert first? or if we have a way to fix it directly? | 03:41 |
oomichi | alex_xu: ok, how about merging it at this time and thinking best way later? | 03:41 |
oomichi | alex_xu: the patch was merged yesterday, and it is easy to revert it back now | 03:42 |
*** ducttape_ has joined #openstack-nova | 03:42 | |
alex_xu | oomichi: emm... it sounds ok | 03:42 |
oomichi | then we can get enough time for finding the best way | 03:43 |
alex_xu | oomichi: ok | 03:43 |
oomichi | alex_xu: https://review.openstack.org/#/c/288125/ | 03:43 |
*** dims has joined #openstack-nova | 03:43 | |
*** dims has quit IRC | 03:44 | |
*** links has quit IRC | 03:44 | |
alex_xu | oomichi: just +2 | 03:44 |
*** apoorvad has quit IRC | 03:45 | |
oomichi | alex_xu: thanks, I'd like to confirm that is right to revert it back. | 03:45 |
alex_xu | oomichi: thanks | 03:45 |
oomichi | alex_xu: wait a minute | 03:45 |
* alex_xu go to lunch, back after hour | 03:47 | |
*** ijw has quit IRC | 03:49 | |
*** zenoway has joined #openstack-nova | 03:50 | |
oomichi | alex_xu: done, have good lunch time ;) | 03:52 |
*** zenoway has quit IRC | 03:54 | |
*** hemna has joined #openstack-nova | 03:55 | |
*** ducttape_ has quit IRC | 03:56 | |
*** jdurgin1 has joined #openstack-nova | 04:04 | |
openstackgerrit | Shiina, Hironori proposed openstack/nova: Documentation fix regarding triggering crash dump https://review.openstack.org/288242 | 04:06 |
*** gjayavelu has joined #openstack-nova | 04:13 | |
*** hemna has quit IRC | 04:16 | |
*** Jeffrey4l has joined #openstack-nova | 04:22 | |
*** tojuvone_ has joined #openstack-nova | 04:27 | |
*** zz_dimtruck is now known as dimtruck | 04:27 | |
*** zenoway has joined #openstack-nova | 04:27 | |
*** thorst has joined #openstack-nova | 04:27 | |
*** agireud has quit IRC | 04:27 | |
*** links has joined #openstack-nova | 04:27 | |
*** baoli_ has joined #openstack-nova | 04:27 | |
*** openstack has joined #openstack-nova | 04:29 | |
*** baoli_ has quit IRC | 04:30 | |
*** zenoway has quit IRC | 04:30 | |
*** openstack has quit IRC | 04:30 | |
*** openstack has joined #openstack-nova | 04:32 | |
*** thorst has quit IRC | 04:33 | |
alex_xu | oomichi: thanks a lot | 04:33 |
*** openstack has quit IRC | 04:34 | |
*** openstack has joined #openstack-nova | 14:07 | |
bauwser | lascii: I was thinking of ops either Ctrl-C'ing or wanting to only do that by steps | 14:07 |
*** cdelatte has quit IRC | 14:08 | |
*** zenoway has quit IRC | 14:09 | |
*** ctrath has joined #openstack-nova | 14:09 | |
*** mrkz has joined #openstack-nova | 14:09 | |
markus_z | sdague: Please correct me if I'm wrong: http://lists.openstack.org/pipermail/openstack-dev/2016-March/088382.html | 14:10 |
*** haomaiwang has joined #openstack-nova | 14:13 | |
andrearosa | do you know when we are going to remove the loading of the API extension? | 14:14 |
*** mriedem has joined #openstack-nova | 14:15 | |
*** anush has joined #openstack-nova | 14:16 | |
*** RuiChen has quit IRC | 14:19 | |
lascii | bauwser: The concern about Ctrl-C is reasonable, but I don't think we want to map part of the set of hosts to a second defined cell. | 14:19 |
*** RuiChen has joined #openstack-nova | 14:19 | |
lascii | bauwser: I would say that if none are mapped create the cell, if some are mapped error | 14:19 |
bauwser | lascii: sure, hence me getting the previous cell mapping | 14:20 |
lascii | ahh, I missed that part | 14:20 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/288357 | 14:20 |
*** knikolla has joined #openstack-nova | 14:21 | |
bauwser | lascii: I actually provided also a comment saying that's IMHO acceptable to use the host_mapping from the latest host, because we shouldn't support 2 cell mappings for 2 different calls | 14:21 |
*** amotoki has quit IRC | 14:21 | |
bauwser | lascii: I just want to make sure that the run is idempotent, that's it | 14:21 |
bauwser | like with puppet or other tools | 14:21 |
*** ducttape_ has joined #openstack-nova | 14:21 | |
*** diana_clarke1 has joined #openstack-nova | 14:22 | |
bauwser | lascii: https://review.openstack.org/#/c/270565/11/nova/cmd/manage.py@1315 | 14:22 |
*** edtubill has joined #openstack-nova | 14:22 | |
*** Daisy has joined #openstack-nova | 14:22 | |
*** sridharg has quit IRC | 14:23 | |
*** ildikov has joined #openstack-nova | 14:24 | |
*** zenoway has joined #openstack-nova | 14:24 | |
openstackgerrit | Thomas Bachman proposed openstack/nova: VMware: Use Port Group and Key in binding details https://review.openstack.org/288076 | 14:24 |
*** macsz1 has joined #openstack-nova | 14:25 | |
*** ducttape_ has quit IRC | 14:26 | |
*** jschlueter has quit IRC | 14:28 | |
*** dansmith is now known as superdan | 14:29 | |
mriedem | cdent: do you have any interest in backporting https://review.openstack.org/#/c/264349/ to stable/liberty? the bug is marked as liberty-backport-potential | 14:31 |
mriedem | there is a merge conflict on the backport though | 14:32 |
bauwser | mriedem: I feel it's a reasonable backport, IMHO | 14:32 |
bauwser | not that critical, but useful still | 14:32 |
mriedem | i agree | 14:33 |
*** macsz1 has quit IRC | 14:33 | |
*** erno has quit IRC | 14:33 | |
mriedem | i'm just looking for volunteers to backport it :) | 14:33 |
sdague | andrearosa: it's deprecated now ... I don't know we have a full timetable on killing it off entirely | 14:33 |
*** rodrigods has quit IRC | 14:34 | |
sdague | markus_z: that looks about right | 14:34 |
*** rodrigods has joined #openstack-nova | 14:34 | |
markus_z | thanks | 14:34 |
mriedem | bauwser: i was just going through https://goo.gl/8F0VMI | 14:34 |
cdent | mriedem: when would it be needed by? | 14:35 |
mriedem | and looking for things that don't have backports yet. several of the bugs in the list are actually already fixed on liberty so it's just a matter of removing the liberty-backport-potential tag | 14:35 |
mriedem | cdent: whenevs | 14:35 |
mriedem | not today | 14:35 |
mriedem | i hope to request a liberty release on monday | 14:35 |
cdent | mriedem: then yeah, I'll do it | 14:35 |
bauwser | mriedem: yeah, I appreciate that | 14:36 |
cdent | if it was "today" then "nnngh" but monday is fine | 14:36 |
bauwser | mriedem: I need to do my bug skimming duty hopefully next week | 14:36 |
bauwser | well, if cutting on monday, that just means that it would wait another point release to be delivered, right ? | 14:36 |
*** openstackgerrit_ has quit IRC | 14:36 | |
bauwser | I don't see that one as a blocker for the monday release | 14:37 |
*** dave-mccowan has joined #openstack-nova | 14:37 | |
*** openstackgerrit_ has joined #openstack-nova | 14:37 | |
*** ivase has joined #openstack-nova | 14:38 | |
*** smatzek has quit IRC | 14:39 | |
*** eharney has joined #openstack-nova | 14:39 | |
markus_z | sdague: ec2 is not a thing anymore, right? I would remove it from the official bug tags. | 14:40 |
*** links has quit IRC | 14:41 | |
*** diogogmt has quit IRC | 14:42 | |
*** PaulMurray has joined #openstack-nova | 14:43 | |
sdague | markus_z: it's still a thing on stable branches we support | 14:43 |
mriedem | bauwser: it's not a blocker for the monday liberty release | 14:43 |
*** gszasz has quit IRC | 14:43 | |
mriedem | bauwser: but i'll let cdent think i'm holding up the world for him :) | 14:44 |
sdague | markus_z: and there are still remnant bits we need to scrub out | 14:44 |
sdague | so I'd leave it for now | 14:44 |
bauwser | mriedem: I agree, I was just pointing to cdent that it won't be shipped in that particular dot release | 14:44 |
mriedem | ssshhhh | 14:44 |
bauwser | heh | 14:44 |
bauwser | cdent: forget what I said, and work, dude ! | 14:44 |
markus_z | sdague: right, I forgot the stable branches, thanks | 14:44 |
sdague | looks like xenproject ci went bonkers again | 14:44 |
* cdent thinks everyone should run off trunk | 14:44 | |
cdent | ;) | 14:44 |
*** rhagarty has joined #openstack-nova | 14:45 | |
*** thumpba has quit IRC | 14:46 | |
sdague | mriedem: the deprecations are all approved and in the gate but as you had comments yesterday if you want to look them over before they land - https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:deprecate_managers | 14:46 |
*** rhagarty_ has joined #openstack-nova | 14:46 | |
sdague | the only substantial change from yesterday is adding in 'use_neutron' new config variable | 14:46 |
sdague | in https://review.openstack.org/#/c/288077/ | 14:46 |
*** diana_clarke1 has quit IRC | 14:47 | |
ndipanov | mriedem, from yesterday - it was updated so I +2ed it https://review.openstack.org/#/c/252836/ | 14:47 |
mriedem | sdague: ndipanov: ok | 14:47 |
*** rhagarty has quit IRC | 14:48 | |
*** zz_dimtruck is now known as dimtruck | 14:49 | |
mriedem | sdague: comment in https://review.openstack.org/#/c/288077/5/nova/network/__init__.py | 14:50 |
mriedem | sdague: do you plan on using the use_neutron option to also remove the security group driver option? | 14:50 |
PaulMurray | sdague, do you know what happens with tempest re freeze - we have some LM tests to add and I want to understand deadlines | 14:50 |
sdague | mriedem: yeh, I think that's the direction we should go | 14:50 |
sdague | so that in newton we deprecate the security group option and key it off of this | 14:50 |
sdague | so there is just 1 flag | 14:51 |
mriedem | yeah, works for me | 14:51 |
sdague | because that's almost even user friendly | 14:51 |
*** atuvenie_ has quit IRC | 14:51 | |
sdague | PaulMurray: because tempest is branchless, there isn't really a freeze in the same way | 14:52 |
sdague | but I'd try to get them sorted in the next couple of weeks | 14:52 |
PaulMurray | sdague, I think they try to get some kind of stability coming up to a release don't they (not that tempest isn't always stable of course) | 14:53 |
sdague | honestly, I'm only lightly involved in tempest at this point, I'd ask mtreinish directly | 14:53 |
PaulMurray | sdague, yeah, I guess I'm surrounded by tempest people, so I can ask them | 14:54 |
mriedem | BobBall_AWOL: xenproject ci is off the rails it looks like | 14:55 |
mriedem | hitting novalid host | 14:55 |
mriedem | would be super sweet if the xen ci had the log formatting such that we can filter by log level | 14:55 |
*** jwcroppe has joined #openstack-nova | 14:55 | |
*** ducttape_ has joined #openstack-nova | 14:56 | |
tdurakov | PaulMurray, hey, about https://review.openstack.org/#/c/287997/3 danpb proposed alternative approach, working on implementation... | 14:56 |
mriedem | BobBall_AWOL: sdague: oh i think the xenproject ci failure is just the libxenlight bug that has been around for a looong time | 14:57 |
*** diogogmt has joined #openstack-nova | 14:57 | |
mriedem | sdague: i would have asked jaypipes about https://review.openstack.org/#/c/288421/ | 14:59 |
mriedem | since i thought there was something related to that which was extendable | 14:59 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/288357 | 14:59 |
mriedem | i could be thinking of the nova.compute.monitors.cpu extension point though | 15:00 |
*** haomaiwang has quit IRC | 15:01 | |
*** tonyb has quit IRC | 15:01 | |
mriedem | PaulMurray: can this be removed? https://github.com/openstack/nova/blob/master/setup.cfg#L44 | 15:01 |
*** tonyb has joined #openstack-nova | 15:01 | |
mriedem | bauwser: ^ | 15:01 |
mriedem | looks like an artifact of ERT | 15:01 |
*** haomaiwang has joined #openstack-nova | 15:01 | |
bauwser | mriedem: I think it's currrently just deprecated | 15:01 |
mriedem | ok | 15:02 |
PaulMurray | mriedem, that stuff is deprecated - I'm going to remove it in the next cycle | 15:02 |
mriedem | i wasn't sure which yet | 15:02 |
mriedem | ok | 15:02 |
*** pwnall has joined #openstack-nova | 15:02 | |
mriedem | sdague: are you going to deprecate the quota driver? | 15:03 |
mriedem | and make it a choice | 15:03 |
sdague | mriedem: I wasn't going there yet | 15:03 |
*** smatzek has joined #openstack-nova | 15:03 | |
PaulMurray | danpb, tdurakov I think the migration monitor gets lost if nova compute is restarted | 15:04 |
sdague | mriedem: I asked bauwser about https://review.openstack.org/#/c/288421/ and he said it was unused | 15:04 |
sdague | and legacy | 15:04 |
danpb | PaulMurray: so don't do that then :-) | 15:04 |
PaulMurray | danpb, tdurakov if force complete and pause are put done by that thread they will not be done if restarted | 15:04 |
mriedem | libvirtError: internal error: libxenlight failed to create new domain 'instance-0000003c' | 15:04 |
PaulMurray | danpb, I wonder if there is a way to restart monitors on restart? | 15:04 |
mriedem | that should be a priority for the xen project ci to fix | 15:04 |
ndipanov | PaulMurray, a gentle reminder https://review.openstack.org/244489 | 15:04 |
danpb | PaulMurray: sure, it could be done...mere matter of coding | 15:05 |
mriedem | sdague: yeah, i was thinking of the cpu monitor stuff | 15:05 |
mriedem | *confused cpu monitor with compute stats | 15:05 |
bauwser | PaulMurray: heard anything about live-migrate loosing neutron floating-ip connection ? | 15:05 |
bauwser | mriedem: the monitors are in use | 15:05 |
PaulMurray | danpb, I think what you proposed is a very good idea but may be better to make the monitor recover from restart | 15:06 |
PaulMurray | first | 15:06 |
PaulMurray | bauwser, yes | 15:06 |
mriedem | bauwser: there is a series for that with dvr | 15:06 |
PaulMurray | bauwser, I think - depends - are you thinking of somethign specific | 15:06 |
mriedem | bauwser: PaulMurray: starts here https://review.openstack.org/#/c/260738/ | 15:06 |
mriedem | johnthetubaguy: the xenproject ci still has a pretty high failure rate on this bug https://bugs.launchpad.net/nova/+bug/1461642 | 15:08 |
openstack | Launchpad bug 1461642 in OpenStack Compute (nova) "libvirt-xen: Race between nova and a Xen script for updating the iptables" [High,In progress] - Assigned to Anthony PERARD (anthony-perard) | 15:08 |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/nova: Fix config generation for Neutron auth options https://review.openstack.org/288513 | 15:08 |
mriedem | johnthetubaguy: any idea if anthony perard is still working on that since there was a partial fix | 15:08 |
mtreinish | PaulMurray: sdague is correct, there isn't really a freeze on tempest like on the other projects. So you can submit changes or new tests whenever | 15:09 |
PaulMurray | mtreinish, thanks | 15:09 |
mtreinish | this is partly because activity on the qa projects normally spikes during the FF window | 15:09 |
johnthetubaguy | mriedem: I thought anthonyper said he was hoping to look at that, not sure if BobBall_AWOL has any more info on that. | 15:09 |
mriedem | there were a couple of fixes | 15:09 |
mriedem | but it's apparently still a problem | 15:10 |
*** markvoelker has quit IRC | 15:10 | |
*** markvoelker has joined #openstack-nova | 15:10 | |
johnthetubaguy | yeah, I think he mentioned something about looking into that at FOSDEM when I last saw him, not sure how thats been going | 15:10 |
tdurakov | PaulMurray, restart of compute during live-migration will produce lot's of problems anyway, not sure is it really critical right now? | 15:11 |
doffm | sdague: Any chance you could take another look at? https://review.openstack.org/#/c/224216/ You were -1 only for needing rebase. | 15:11 |
*** claudiub has joined #openstack-nova | 15:12 | |
PaulMurray | bauwser, the connection loss I know about (related to that neutron patch) is because DVR has to setup the dest host - at the moment this id done post migration when ports are rebound, which means some network traffic is lost and disrupts the connections | 15:12 |
sdague | doffm: I'll just 0 out my review, still working through a thing on the use_neutron option | 15:12 |
doffm | sdague: Thanks. | 15:12 |
PaulMurray | bauwser, we discussed it at the mid-cycle with carl_baldwin and armax | 15:14 |
bauwser | PaulMurray: sorry on a call atm, but thanks for giving me feedback, will get more context in the bug report | 15:17 |
*** mjura has quit IRC | 15:17 | |
*** thumpba has joined #openstack-nova | 15:18 | |
*** thumpba has quit IRC | 15:20 | |
*** thumpba has joined #openstack-nova | 15:20 | |
*** thumpba has quit IRC | 15:20 | |
*** thumpba_ has joined #openstack-nova | 15:22 | |
*** thumpba_ has quit IRC | 15:22 | |
*** mvk has joined #openstack-nova | 15:22 | |
*** thumpba has joined #openstack-nova | 15:22 | |
*** jed56 has quit IRC | 15:23 | |
anthonyper | johnthetubaguy, mriedem: I'll give a look at those patch to fix the libvirt-xen race | 15:24 |
mriedem | anthonyper: thanks | 15:26 |
mriedem | i've seen a few xenproject ci failures this morning on that issue | 15:26 |
*** yamahata has joined #openstack-nova | 15:28 | |
*** Daisy has quit IRC | 15:33 | |
*** david_lyle__ has joined #openstack-nova | 15:34 | |
*** ZZelle has quit IRC | 15:34 | |
*** rhagarty_ has quit IRC | 15:36 | |
*** david-lyle has quit IRC | 15:37 | |
*** aysyd has quit IRC | 15:38 | |
*** aysyd has joined #openstack-nova | 15:38 | |
*** lpetrut has quit IRC | 15:39 | |
*** lpetrut has joined #openstack-nova | 15:39 | |
*** yamahata has quit IRC | 15:40 | |
markus_z | superdan: I'm rephrasing the wiki page about bug tags and owner. Your name is there for "unified-objects". Does this also include all RPC related things? | 15:41 |
*** Bjoern has joined #openstack-nova | 15:42 | |
*** paul-carlton2 has quit IRC | 15:45 | |
superdan | markus_z: not really, but you can add me there for that too | 15:45 |
*** smatzek_ has joined #openstack-nova | 15:46 | |
*** lpetrut has quit IRC | 15:47 | |
markus_z | superdan: We have nothing extra for RPC related issue, or is that meant with the "upgrades" tag? | 15:47 |
markus_z | superdan: What's the focus of the "unified-objects" then? | 15:47 |
superdan | markus_z: upgrades, objects, and rpc are all relatedish, but separate topics, but you can add me for all of them | 15:48 |
superdan | in the early days, we called nova.objects.* "unified-objects" | 15:48 |
*** jschlueter has joined #openstack-nova | 15:49 | |
*** mtanino has joined #openstack-nova | 15:49 | |
*** andreykurilin has left #openstack-nova | 15:49 | |
*** efried has quit IRC | 15:50 | |
markus_z | Some day I need a history lesson of the early days around a camp fire | 15:50 |
*** smatzek has quit IRC | 15:50 | |
dims | markus_z : save a seat for me too :) | 15:50 |
markus_z | dims: As a story teller, sure ;) | 15:51 |
*** kiran-r has quit IRC | 15:51 | |
*** efried has joined #openstack-nova | 15:51 | |
*** rhagarty has joined #openstack-nova | 15:51 | |
mriedem | danpb: i came across a bug yesterday that prompted me to push this up - is it completely wrong? https://review.openstack.org/#/c/288109/ | 15:52 |
*** Sukhdev has joined #openstack-nova | 15:52 | |
dims | markus_z : i feel i am a late arrival too :) i have heard a lot of stories for sure well before i got started | 15:52 |
*** rhagarty_ has joined #openstack-nova | 15:52 | |
figleaf | dims: it's worse if you were there at the beginning, and then missed a few years. | 15:53 |
*** shangxdy has joined #openstack-nova | 15:53 | |
dims | figleaf :) | 15:53 |
*** liusheng has quit IRC | 15:53 | |
dims | where are my manners | 15:53 |
*** dims is now known as dimsum_ | 15:53 | |
*** kiran-r has joined #openstack-nova | 15:53 | |
danpb | mriedem: not sure to be honest | 15:54 |
*** liusheng has joined #openstack-nova | 15:54 | |
mriedem | danpb: ok, i was hoping the reporter would try the patch and let me konw | 15:54 |
*** sridharg has joined #openstack-nova | 15:54 | |
mriedem | it seems to make sense...if you're rebuilding with disk on shared storage, don't destroy the disks if vif plugging fails or times out with neutron while rebuilding | 15:54 |
markus_z | figleaf: really? I thought you know where all the bodies lay and it's easy now. | 15:54 |
figleaf | dimsum_: much better! | 15:55 |
figleaf | markus_z: nah, they all got moved between Essex and Kilo | 15:55 |
markus_z | hehe | 15:55 |
*** mriedem is now known as mriedem_meeting | 15:55 | |
*** rhagarty has quit IRC | 15:56 | |
*** cdent has quit IRC | 15:57 | |
*** irenab has quit IRC | 15:59 | |
openstackgerrit | Paul Griffin proposed openstack/nova: List instances for secgroup without joining on rules https://review.openstack.org/288548 | 15:59 |
*** yamahata has joined #openstack-nova | 15:59 | |
*** irenab has joined #openstack-nova | 16:00 | |
markus_z | Anyone a clue what the "ops" bug tag is used for? Bug reports from operators? | 16:00 |
*** thorst has quit IRC | 16:00 | |
*** haomaiwang has quit IRC | 16:01 | |
johnthetubaguy | markus_z: there was that ops subteam that were meant to tell us the most important bugs for ops folks | 16:01 |
johnthetubaguy | markus_z: duno if thats related | 16:01 |
*** thorst has joined #openstack-nova | 16:01 | |
openstackgerrit | Sean Dague proposed openstack/nova: update tests for use_neutron=True; fix exposed bugs https://review.openstack.org/288552 | 16:01 |
sdague | mriedem_meeting / johnthetubaguy ^^^ | 16:02 |
sdague | couple of things fell out when I was trying to update all our tests to the new option | 16:02 |
*** vishwanathj_zzz is now known as vishwanathj | 16:02 | |
mriedem_meeting | sdague: so how did the other change merge? | 16:02 |
markus_z | johnthetubaguy: hm, we have a section for that in the priorities etherpad | 16:02 |
markus_z | johnthetubaguy: "Operators Bug Triage subteam" in https://etherpad.openstack.org/p/mitaka-nova-priorities-tracking | 16:03 |
sdague | mriedem_meeting: because apparently where we use is_neutron is pretty sparse an untested | 16:03 |
markus_z | well, ok, I just leave the tag there for now. | 16:03 |
mriedem_meeting | sdague: ok, well, on a call for an hour | 16:04 |
*** hemnafk is now known as hemna | 16:04 | |
sdague | there is a multiple create path | 16:04 |
sdague | mriedem_meeting: oh, I know | 16:05 |
sdague | so... all our devstack runs are still setting the legacy variable | 16:05 |
sdague | so it passes all there | 16:05 |
*** irenab has quit IRC | 16:05 | |
*** tanee is now known as away_taane | 16:05 | |
*** away_taane is now known as tanee | 16:05 | |
mriedem_meeting | i was wondering about the unit tests | 16:06 |
mriedem_meeting | why weren't those failing | 16:06 |
mriedem_meeting | or are they racing now? | 16:06 |
sdague | because they are all mocked | 16:06 |
*** irenab has joined #openstack-nova | 16:06 | |
sdague | is_neutron gets mocked all the time because it's globally cached | 16:06 |
*** irenab has quit IRC | 16:07 | |
sdague | let me get the devstack change posted to use this new flag to ensure there isn't anything else missing | 16:07 |
*** belmoreira has quit IRC | 16:07 | |
*** dedery has joined #openstack-nova | 16:08 | |
*** mdrabe has quit IRC | 16:08 | |
*** irenab has joined #openstack-nova | 16:09 | |
sdague | https://review.openstack.org/288556 will expose any follow on issues | 16:09 |
*** Drankis has quit IRC | 16:10 | |
*** e0ne has quit IRC | 16:12 | |
*** busterswt has joined #openstack-nova | 16:13 | |
*** irenab has quit IRC | 16:13 | |
*** rdopiera has quit IRC | 16:14 | |
*** thorst is now known as thorst_afk | 16:15 | |
mriedem_meeting | sdague: unrelated, but can you peak at this backport proposed to liberty? the rootwrap/upgrade implications worry me https://review.openstack.org/#/c/278928/ | 16:16 |
*** Marga_ has quit IRC | 16:17 | |
*** david_lyle__ is now known as david_lyle | 16:17 | |
*** jichen has quit IRC | 16:17 | |
*** kmartin__ has quit IRC | 16:17 | |
sdague | mriedem_meeting: oh, right, that one | 16:18 |
sdague | so, yes, it requires manual intervention by the user | 16:18 |
sdague | by the operator to update the rootwrap config | 16:18 |
sdague | I honestly would reject it as a backport for that reason | 16:18 |
mriedem_meeting | sdague: ok, that's what i was leaning to | 16:19 |
*** pushkaru has joined #openstack-nova | 16:19 | |
mriedem_meeting | claudiub: can you take a look at this backport? https://review.openstack.org/#/c/287831/ | 16:19 |
superdan | lascii: can you answer a question about flavor_access stuff? not sure if rax only ever used public flavors or not | 16:19 |
sdague | I guess we could have made him use dd instead of touch and then it would have been fine :) | 16:19 |
*** irenab has joined #openstack-nova | 16:19 | |
*** achanda has joined #openstack-nova | 16:19 | |
sdague | I remember that master patch quite a bit | 16:19 |
*** achanda has quit IRC | 16:20 | |
lascii | superdan: mostly they did. flavor_access was used to beta test a few flavors a while back, but it's wasn't a standard thing to do that | 16:21 |
superdan | okay.. I wonder who knows about that stuff | 16:21 |
superdan | maybe cburgess ? | 16:21 |
*** jschlueter has quit IRC | 16:21 | |
*** jaosorior has quit IRC | 16:24 | |
*** tongli has joined #openstack-nova | 16:26 | |
*** thorst_afk has quit IRC | 16:26 | |
*** anush has quit IRC | 16:27 | |
*** andymaier_ has quit IRC | 16:27 | |
*** cdent has joined #openstack-nova | 16:27 | |
*** rhagarty has joined #openstack-nova | 16:28 | |
*** thorst_afk has joined #openstack-nova | 16:29 | |
*** armax has quit IRC | 16:29 | |
markus_z | danpb: Do you have a minute? | 16:29 |
*** armax has joined #openstack-nova | 16:29 | |
*** rhagarty_ has quit IRC | 16:30 | |
*** penick has joined #openstack-nova | 16:30 | |
*** daemontool__ has joined #openstack-nova | 16:30 | |
*** ssurana has joined #openstack-nova | 16:31 | |
*** efried has quit IRC | 16:32 | |
auggy | sheel: I just caught up with backscroll, i would update the bug with your questions | 16:32 |
auggy | sheel: I think this might be something that either should be discussed on the mailing list or at the design summit, but I could be wrong :) | 16:33 |
*** anush has joined #openstack-nova | 16:33 | |
sheel | auggy: thanks | 16:33 |
*** daemontool_ has quit IRC | 16:34 | |
*** haukebruno has quit IRC | 16:34 | |
*** daemontool__ is now known as daemontool | 16:34 | |
auggy | sheel: those are both good questions. i think the feeling is that it's probably an edge case affecting very few users since this issue is only coming up now | 16:34 |
sheel | auggy: i pointed this out as this will create problem in existing setups | 16:35 |
auggy | sheel: yeah definitely worth hashing out more, thanks :) | 16:35 |
openstackgerrit | sahid proposed openstack/nova: network: make nova to handle port_security_enabled=False https://review.openstack.org/284095 | 16:35 |
*** thorst_afk is now known as thorst | 16:35 | |
*** zenoway has quit IRC | 16:36 | |
sheel | auggy: thanks!! | 16:36 |
*** anush has quit IRC | 16:36 | |
markus_z | superdan: Let me know if your name is at the right places: https://wiki.openstack.org/wiki/Nova/BugTriage | 16:37 |
*** zenoway has joined #openstack-nova | 16:37 | |
*** zenoway has quit IRC | 16:37 | |
*** zenoway has joined #openstack-nova | 16:38 | |
*** mriedem_meeting is now known as mriedem | 16:40 | |
superdan | markus_z: sure | 16:41 |
*** jaypipes has joined #openstack-nova | 16:41 | |
*** jaypipes is now known as leakypipes | 16:42 | |
*** zenoway has quit IRC | 16:42 | |
*** mdrabe has joined #openstack-nova | 16:43 | |
cdent | jaypipes: don't strictly disagree with you on https://twitter.com/jaypipes/status/705794176491753473 and the kit thing, just think that are a lot of logical inconsistencies in the current approach that require some critical thinking | 16:43 |
cdent | oops, sorry, leakypipes ^ | 16:43 |
openstackgerrit | sahid proposed openstack/nova: network: make nova to handle port_security_enabled=False https://review.openstack.org/284095 | 16:44 |
*** yamahata has quit IRC | 16:44 | |
openstackgerrit | Chris St. Pierre proposed openstack/nova: Fix deleted non-public flavor lookups for non-admin users https://review.openstack.org/211809 | 16:45 |
*** rhagarty_ has joined #openstack-nova | 16:45 | |
*** yamahata has joined #openstack-nova | 16:45 | |
*** auggy is now known as frauggy | 16:45 | |
*** anush has joined #openstack-nova | 16:45 | |
*** efried has joined #openstack-nova | 16:46 | |
*** penick has quit IRC | 16:46 | |
*** rhagarty has quit IRC | 16:47 | |
PaulMurray | bauwser, FYI from yesterdays conversation about disabled hosts: https://review.openstack.org/#/c/288373/ | 16:47 |
PaulMurray | johnthetubaguy, ^^ | 16:48 |
*** ljxiash has quit IRC | 16:48 | |
PaulMurray | thought you might be interested | 16:48 |
*** apoorvad has joined #openstack-nova | 16:49 | |
mriedem | lascii: it scares me that the with_entities thing in the list object here didn't fail tests https://review.openstack.org/#/c/286309/5/nova/objects/instance_mapping.py | 16:49 |
mriedem | mtreinish: do you use with_entities for sqlalchemy stuff in subunit2sql? | 16:50 |
*** pece has quit IRC | 16:50 | |
mriedem | i remember you having a huge join with indexes to get the results | 16:50 |
leakypipes | cdent: sure, no disagreement there. | 16:50 |
*** zengine has quit IRC | 16:51 | |
PaulMurray | bauwser, I have a question about your patch https://review.openstack.org/#/c/284974/2 | 16:51 |
cdent | leakypipes: I also think that if there isn't a product _somewhere_ it's hard to build anything good | 16:51 |
cdent | because everything is trying to solve all cases | 16:51 |
*** rcernin has quit IRC | 16:51 | |
mtreinish | mriedem: no, I'm not. That looks useful | 16:51 |
mriedem | mtreinish: well, useful if it works... | 16:52 |
mtreinish | mriedem: I do something similar with: https://github.com/openstack-infra/subunit2sql/blob/master/subunit2sql/db/api.py#L968-L971 | 16:52 |
mriedem | leakypipes: if this is your stance https://twitter.com/jaypipes/status/705794176491753473 | 16:53 |
*** ssurana has quit IRC | 16:53 | |
mriedem | leakypipes: then shouldn't you have been anti-deprecation of hooks and managers and other plug points? | 16:53 |
*** pece has joined #openstack-nova | 16:54 | |
johnthetubaguy | PaulMurray: Doesn't that break the service object interface? it kinda feels odd to leave the disabled in for the other ones too, in a way? | 16:55 |
*** sahid has quit IRC | 16:55 | |
cdent | mriedem has stolen my eyes! | 16:55 |
*** scheuran has quit IRC | 16:55 | |
PaulMurray | johnthetubaguy, hmm, looking | 16:56 |
mriedem | i just point it out b/c in the ML thread on removing hooks support it was pointed out that openstack used to be a kit to build cloud software but now it's not | 16:56 |
PaulMurray | johnthetubaguy, bauwser did mention that those functions are all inconsistent in this | 16:56 |
johnthetubaguy | PaulMurray: honestly, it looks like service_get_by_compute_host is what the scheduler was meant to use | 16:57 |
figleaf | mriedem: yeah, the original idea was OpenStack was the basic plumbing, and each deployment could customize it to their heart's content | 16:57 |
*** sc68cal has joined #openstack-nova | 16:57 | |
*** smatzek_ is now known as smatzek | 16:57 | |
figleaf | mriedem: Hence the focus on pluggable points | 16:58 |
*** ihrachys has quit IRC | 16:58 | |
*** _pece has joined #openstack-nova | 16:58 | |
johnthetubaguy | PaulMurray: I wonder if that is used actually, given the changes in the object for the service object | 16:58 |
figleaf | interop wasn't even a thing | 16:58 |
leakypipes | mriedem: no, I don't see the correlation there. | 16:58 |
PaulMurray | johnthetubaguy, service_get_by_compute_host gets for a particular host, we want all compute-managers | 16:59 |
openstackgerrit | John Garbutt proposed openstack/nova: xenapi: fix when tar exits early during download https://review.openstack.org/287426 | 16:59 |
johnthetubaguy | PaulMurray: doh, yes, I am going totally blind | 17:00 |
PaulMurray | johnthetubaguy, it looks like its only used by the ServiceList object | 17:00 |
*** angdraug has joined #openstack-nova | 17:00 | |
PaulMurray | johnthetubaguy, and that's only used for this purpose | 17:01 |
PaulMurray | (that function is only used for this) | 17:01 |
cdent | figleaf: do you have a history or references on when/where the interop thing entered the pic? | 17:01 |
leakypipes | mriedem, cdent: I think that companies should build products *on top of* OpenStack, and those companies should contribute to making OpenStack services/tools as stable and rock-solid and singlular-focused as possible. I am anti- the inclusion of monolithic applications into the big tent (this includes Tacker, Murano, Fuel BTW) and would prefer to see a big tent of small-focused, stable, well-maintained tools that provide cloud management services | 17:01 |
leakypipes | that products can be built around and on top of. | 17:01 |
*** pece has quit IRC | 17:01 | |
*** kmartin has joined #openstack-nova | 17:02 | |
cdent | that all makes sense | 17:02 |
*** matrohon has quit IRC | 17:02 | |
PaulMurray | johnthetubaguy, in each place its used the code immediately searches the list for disabled hsots - so its clearly expecting them to still be in there | 17:02 |
figleaf | cdent: no, I don't. I think it's a result of growth, as it became impossible to test every permuatation | 17:03 |
*** Jeffrey4l has quit IRC | 17:03 | |
*** cfarquhar has quit IRC | 17:03 | |
mriedem | cdent: i think interop probably originally started from infra | 17:03 |
leakypipes | mriedem, cdent: I also believe that *if* an OpenStack service exposes a REST API, that that REST API should be curated by a single team to ensure no overlap with other OpenStack REST API endpoints. And finally, I don''t believe that the free-form hooks and manager plug points in Nova furthered a goal of consistency in Nova. | 17:03 |
mriedem | b/c infra is using multiple public clouds | 17:03 |
*** ljxiash has joined #openstack-nova | 17:03 | |
*** amotoki has joined #openstack-nova | 17:03 | |
figleaf | mriedem: good point | 17:04 |
mriedem | cdent: and probably as a way to put the hammer down on forks of the api with defcore | 17:04 |
*** zengine has joined #openstack-nova | 17:04 | |
*** efried has quit IRC | 17:05 | |
cdent | leakypipes: I reckon maybe we agree about what to do, but not about what "product" means | 17:05 |
mriedem | but defcore was being talked about 2 years ago | 17:05 |
cdent | (or "kit") | 17:05 |
leakypipes | mriedem: sorry I'm not following you. what does defcore have to do with my statement? | 17:05 |
mriedem | "(11:01:28 AM) cdent: figleaf: do you have a history or references on when/where the interop thing entered the pic?" | 17:05 |
*** zengine_ has joined #openstack-nova | 17:06 | |
mriedem | leakypipes: 2 conversations | 17:06 |
cdent | kit, to me, means something I can use to create different apis over the same domain of behavior | 17:06 |
cdent | product means: a (just one) compute api | 17:07 |
PaulMurray | johnthetubaguy, just a thought... but I reckon this counts as a bug fix now: https://review.openstack.org/#/c/284974 | 17:07 |
*** jed56 has joined #openstack-nova | 17:07 | |
*** jlanoux has quit IRC | 17:07 | |
mriedem | cdent: well a product can change the api if they want, but if that api is part of defcore they can't be considered refstack compliant | 17:07 |
mriedem | and don't get the seal of approval, or whatever is on the website | 17:07 |
leakypipes | mriedem: oh, sorry :( | 17:08 |
*** ljxiash has quit IRC | 17:08 | |
*** kiran-r has quit IRC | 17:08 | |
cdent | mriedem: sorry, I was switched over the leakypipes side of the conversation. defcore is something that is _completely_ invisible to me | 17:08 |
leakypipes | let's move this particular chat to somewhere else to abide by /topic, eh? :) | 17:08 |
mriedem | boo | 17:09 |
*** zengine has quit IRC | 17:09 | |
*** zengine_ is now known as zengine | 17:09 | |
mriedem | i just started up folsom and can't boot an instance, leakypipes, can you help plz? | 17:09 |
leakypipes | mriedem: Mortimer just shat on the floor. Can you go clean that up plz? | 17:09 |
PaulMurray | ndipanov, I have had a couple of people say they don't really understand https://review.openstack.org/#/c/244489/ | 17:09 |
mriedem | leakypipes: that's laura's responsibility | 17:10 |
*** Sukhdev has quit IRC | 17:10 | |
PaulMurray | ndipanov, I will have a good bash at it though | 17:10 |
mriedem | the deal is, it's her dog so she deals with the mess | 17:10 |
ndipanov | PaulMurray, they don't understand it how? | 17:10 |
ndipanov | are there specific questions they have | 17:10 |
openstackgerrit | John Garbutt proposed openstack/nova: xenapi: fix when tar exits early during download https://review.openstack.org/287426 | 17:10 |
ndipanov | or the whole thing is wtf? :) | 17:11 |
ndipanov | PaulMurray, if there are specific questions - I'd be happy to answer them | 17:11 |
PaulMurray | ndipanov, similar to mdbooth_ comments in the review - I'm only just starting | 17:11 |
PaulMurray | ndipanov, I will encourage them to comment | 17:11 |
*** mdbooth_ is now known as mdbooth | 17:11 | |
ndipanov | PaulMurray, excellent | 17:12 |
PaulMurray | ndipanov, in the meantime I'm starting on it | 17:12 |
cdent | mriedem: I hope I did this right: https://review.openstack.org/#/c/288594/ | 17:14 |
mriedem | cdent: looks like the conflict in nova.virt.images was on this? https://github.com/openstack/nova/commit/915fdbbfb82272b87cd80210943372b09351cf88 | 17:18 |
cdent | pretty much | 17:18 |
mriedem | which we already backported to liberty, | 17:19 |
mriedem | and your fix in the same file came after it, so we didn't backport in order | 17:19 |
*** paul-carlton2 has joined #openstack-nova | 17:19 | |
cdent | yes | 17:20 |
sdague | mriedem: https://review.openstack.org/#/c/288556/ passes on neutron, so that set of fixes in https://review.openstack.org/#/c/288552/ should be all of them | 17:22 |
sdague | going afk for a bit | 17:23 |
*** shaohe_feng_ has joined #openstack-nova | 17:25 | |
mriedem | cdent: ok, +2, thanks | 17:25 |
cdent | happy to be of service | 17:25 |
*** dedery has quit IRC | 17:26 | |
*** Sukhdev has joined #openstack-nova | 17:27 | |
*** shaohe_feng has quit IRC | 17:28 | |
*** haplo37 has joined #openstack-nova | 17:28 | |
*** shaohe_feng_ is now known as shaohe_feng | 17:28 | |
mriedem | superdan: can you poke on https://review.openstack.org/#/c/288594/ ? if we get that approved i'll base the 12.0.2 release on it | 17:29 |
sdague | mriedem: you know, the security groups deprecation is actually pretty small | 17:30 |
sdague | I'm going to push up a WIP for that, because that would just be so much more sane if we only have 'use_neutron' | 17:30 |
*** rudolfvriend has quit IRC | 17:31 | |
*** efried has joined #openstack-nova | 17:31 | |
openstackgerrit | Sean Dague proposed openstack/nova: WIP: deprecate security_group_api class https://review.openstack.org/288602 | 17:31 |
*** harlowja_at_home has joined #openstack-nova | 17:33 | |
*** lunarlamp has joined #openstack-nova | 17:33 | |
*** yamahata has quit IRC | 17:33 | |
lascii | mriedem: yeah, the fact that tests didn't fail bothers me about the with_entities thing too. I'm going to investigate at some point, but ultimately just remove that list object for now | 17:36 |
lascii | and I'll add the backrefs at some point before we get moving again in N | 17:36 |
*** browne has joined #openstack-nova | 17:37 | |
*** daemontool_ has joined #openstack-nova | 17:37 | |
*** anush has quit IRC | 17:38 | |
*** PaulMurray has quit IRC | 17:38 | |
*** daemontool has quit IRC | 17:41 | |
*** nic has joined #openstack-nova | 17:42 | |
*** ndipanov has quit IRC | 17:43 | |
*** Daisy has joined #openstack-nova | 17:45 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/288357 | 17:46 |
*** jistr has quit IRC | 17:46 | |
*** vilobhmm11 has joined #openstack-nova | 17:46 | |
*** jwcroppe has quit IRC | 17:47 | |
*** jwcroppe has joined #openstack-nova | 17:48 | |
*** vilobhmm111 has joined #openstack-nova | 17:49 | |
*** Daisy has quit IRC | 17:50 | |
*** vilobhmm11 has quit IRC | 17:51 | |
*** salv-orlando has joined #openstack-nova | 17:52 | |
*** paul-carlton2 has quit IRC | 17:53 | |
*** lucasagomes is now known as lucas-beer | 17:54 | |
*** salv-orl_ has quit IRC | 17:55 | |
*** leakypipes has quit IRC | 17:56 | |
*** dimtruck is now known as zz_dimtruck | 17:58 | |
*** Swami_ has joined #openstack-nova | 17:59 | |
*** Swami has joined #openstack-nova | 17:59 | |
*** geekinutah has joined #openstack-nova | 18:02 | |
*** yamahata has joined #openstack-nova | 18:04 | |
*** paul-carlton2 has joined #openstack-nova | 18:05 | |
*** danpb has quit IRC | 18:05 | |
ansiwen | superdan: so indeed the empty access list defines a public flavor. that means, in the end it is just the request to rephrase the error message into somethong less dramatic. | 18:09 |
*** shangxdy has quit IRC | 18:09 | |
*** sridharg has quit IRC | 18:10 | |
superdan | ansiwen: okay seems reasonable | 18:10 |
*** derekh has quit IRC | 18:11 | |
*** yassine has quit IRC | 18:11 | |
*** permalac has quit IRC | 18:12 | |
ansiwen | superdan: you think so? which message would you recommend? Instead of filing a bug I would just push a change then. | 18:12 |
*** yarkot_ has joined #openstack-nova | 18:13 | |
openstackgerrit | John Garbutt proposed openstack/nova: xenapi: fix when tar exits early during download https://review.openstack.org/287426 | 18:16 |
*** vilobhmm111 has quit IRC | 18:16 | |
*** zz_dimtruck is now known as dimtruck | 18:17 | |
*** harlowja_at_home is now known as hollaja | 18:17 | |
*** jdurgin1 has quit IRC | 18:17 | |
openstackgerrit | John Garbutt proposed openstack/nova: xenapi: fix when tar exits early during download https://review.openstack.org/287426 | 18:19 |
superdan | ansiwen: we can detect that the flavor_access is being called on a public flavor from the client and just emit something like "duh, you're asking about a public flavor" right? | 18:19 |
ansiwen | yes, it's a bool attribute of the return value | 18:20 |
ansiwen | superdan^ | 18:20 |
*** tongli has quit IRC | 18:21 | |
*** yarkot_ has quit IRC | 18:21 | |
*** _pece has quit IRC | 18:23 | |
*** ducttape_ has quit IRC | 18:26 | |
*** amotoki has quit IRC | 18:28 | |
*** shakamunyi has joined #openstack-nova | 18:29 | |
*** Sukhdev has quit IRC | 18:33 | |
*** eharney has quit IRC | 18:35 | |
*** pwnall has quit IRC | 18:37 | |
*** penick has joined #openstack-nova | 18:37 | |
*** ducttape_ has joined #openstack-nova | 18:38 | |
*** ijw has joined #openstack-nova | 18:41 | |
*** jichen has joined #openstack-nova | 18:41 | |
*** pcaruana has quit IRC | 18:41 | |
*** achanda has joined #openstack-nova | 18:44 | |
*** paul-carlton2 has quit IRC | 18:44 | |
openstackgerrit | Sven Anderson proposed openstack/python-novaclient: Return a less dramatic message for public flavors. https://review.openstack.org/288639 | 18:45 |
*** ijw_ has joined #openstack-nova | 18:46 | |
*** ijw has quit IRC | 18:46 | |
openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Fix resize of instance with deleted glance image https://review.openstack.org/288640 | 18:47 |
openstackgerrit | Sean Dague proposed openstack/nova: deprecate security_group_api config option https://review.openstack.org/288602 | 18:47 |
*** mriedem has quit IRC | 18:47 | |
*** mriedem has joined #openstack-nova | 18:48 | |
*** vilobhmm11 has joined #openstack-nova | 18:48 | |
ansiwen | superdan: how about that? https://review.openstack.org/288639 | 18:49 |
superdan | ansiwen: oh wow, it was just being scary on purpose | 18:50 |
*** doug-fis_ is now known as doug-fish | 18:50 | |
superdan | ansiwen: I feel like just "Flavor is public" might be a little too subtle | 18:50 |
superdan | what about "Flavor is public; no access list available" or something? | 18:51 |
ansiwen | superdan: well the function claims to "Print access information about the given flavor." "Flavor is public." compies to that, no? ;-) | 18:52 |
*** hollaja has quit IRC | 18:52 | |
*** harlowja_at_home has joined #openstack-nova | 18:52 | |
superdan | well, I just think it's too subtle to be obvious, but maybe | 18:53 |
superdan | get some other opinions | 18:53 |
ansiwen | I can add "no access list available", np | 18:53 |
superdan | I'm serious.. ask some other people to see what they say | 18:54 |
superdan | the existing one was so confusing based on what was really happening I might just be over compensating | 18:54 |
superdan | I'd ask melwitt and mriedem | 18:54 |
*** jschlueter has joined #openstack-nova | 18:55 | |
mriedem | so this https://review.openstack.org/#/c/288639/ | 18:55 |
ansiwen | mriedem: yep | 18:55 |
*** vdrok has quit IRC | 18:56 | |
mriedem | i like "Flavor is public; no access list available" | 18:56 |
mriedem | or | 18:57 |
mriedem | "Access list is not available for public flavors." | 18:57 |
superdan | yeah, that's better probably | 18:58 |
mriedem | this is the api code https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/flavor_access.py#L59 | 18:58 |
superdan | what is there now is pretty strange | 18:58 |
mriedem | holy balls | 18:58 |
mriedem | i totally had no idea that's what the rest api returned before i suggested that | 18:58 |
superdan | hah | 18:58 |
superdan | mirroring the server-side message seems like a no brainer to me | 18:59 |
mriedem | yeah | 18:59 |
mriedem | -1 in favor of that | 18:59 |
mriedem | -1 on the change i mean | 18:59 |
*** anush has joined #openstack-nova | 18:59 | |
ansiwen | yo, we leave it as it is? | 19:00 |
ansiwen | so, ... | 19:00 |
melwitt | mriedem: maybe you wrote the server-side message and forgot? jk | 19:00 |
*** rlrossit has quit IRC | 19:00 | |
mriedem | ansiwen: no, | 19:00 |
mriedem | because we didn't fail | 19:00 |
mriedem | it's just not a thing you can do | 19:00 |
mriedem | fail connotes something shart it's pants | 19:01 |
mriedem | try again | 19:01 |
*** zenoway has joined #openstack-nova | 19:01 | |
*** pece has joined #openstack-nova | 19:01 | |
superdan | mriedem: am I reading that right? 400 == shart, 500 == shit? | 19:01 |
*** smatzek_ has joined #openstack-nova | 19:02 | |
mriedem | ha | 19:02 |
mriedem | the 400 range is yes, gambled and lost | 19:02 |
mriedem | 500+ is full out shit your pants | 19:02 |
superdan | lolol | 19:03 |
mriedem | i hope that makes it's way to the twitter | 19:03 |
melwitt | lol | 19:03 |
*** smatzek__ has joined #openstack-nova | 19:04 | |
*** zenoway has quit IRC | 19:05 | |
figleaf | mriedem: that's directly from the RFC | 19:05 |
* dimsum_ counters with cat pictures | 19:05 | |
dimsum_ | https://www.flickr.com/photos/girliemac/sets/72157628409467125/ | 19:05 |
dimsum_ | "HTTP Status Cats" | 19:05 |
*** smatzek has quit IRC | 19:05 | |
*** rk4n_ has joined #openstack-nova | 19:07 | |
*** smatzek_ has quit IRC | 19:07 | |
mriedem | those are pretty good | 19:08 |
figleaf | especially 405 | 19:08 |
*** rlrossit has joined #openstack-nova | 19:08 | |
*** rlrossit_ has joined #openstack-nova | 19:09 | |
mriedem | george bush hates black people, not cats | 19:09 |
mriedem | geez | 19:09 |
mriedem | even kanye knows that | 19:09 |
*** isq has joined #openstack-nova | 19:09 | |
*** isq has quit IRC | 19:09 | |
ansiwen | mriedem: but CommandError returns "ERROR" with bells and whistles, not sure is that's appropriat either. | 19:09 |
*** isq has joined #openstack-nova | 19:09 | |
figleaf | mriedem: obviously W *loves* cats! | 19:10 |
*** ihrachys has joined #openstack-nova | 19:10 | |
mriedem | ansiwen: hmm, it's not great, but i'm not sure if we should change the exit code either | 19:10 |
mriedem | especially since the API returns 404 | 19:10 |
*** rlrossit has quit IRC | 19:10 | |
*** pece has quit IRC | 19:10 | |
*** rk4n has quit IRC | 19:10 | |
ansiwen | I like "402 Payment Required"... has that ever been used? :-) | 19:11 |
*** rk4n_ has quit IRC | 19:12 | |
*** ociuhandu has quit IRC | 19:13 | |
ansiwen | mriedem: ok, so I will compensate the ERROR with the message afterwards, like "... well not really an error, rather a warning: Access list not available for public flavors." | 19:13 |
*** eharney has joined #openstack-nova | 19:13 | |
mriedem | ansiwen: please don't | 19:14 |
mriedem | as you can tell, we are serious developers working on serious things | 19:15 |
*** ljxiash has joined #openstack-nova | 19:15 | |
dimsum_ | :) | 19:16 |
*** pwnall has joined #openstack-nova | 19:16 | |
openstackgerrit | Matt Riedemann proposed openstack/nova: Add release note for nova using neutron mtu value for vif plugging https://review.openstack.org/287344 | 19:17 |
*** pece has joined #openstack-nova | 19:17 | |
ansiwen | mriedem: yes, I forgot | 19:18 |
*** zenoway has joined #openstack-nova | 19:18 | |
*** ljxiash has quit IRC | 19:19 | |
*** apoorvad has quit IRC | 19:22 | |
*** jed56 has quit IRC | 19:23 | |
*** apoorvad has joined #openstack-nova | 19:26 | |
*** frauggy is now known as auggy | 19:30 | |
*** ociuhandu has joined #openstack-nova | 19:31 | |
openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: clean up help text for live_migration_inbound_addr option https://review.openstack.org/288695 | 19:31 |
*** sdague has quit IRC | 19:32 | |
*** ctrath has quit IRC | 19:34 | |
*** smatzek__ is now known as smatzek | 19:37 | |
*** ctrath has joined #openstack-nova | 19:37 | |
*** ihrachys has quit IRC | 19:38 | |
*** eglynn has joined #openstack-nova | 19:40 | |
*** pushkaru has quit IRC | 19:40 | |
openstackgerrit | Nicolas Simonds proposed openstack/nova: Allow use of live snapshots with RBD snapshot/clone https://review.openstack.org/273652 | 19:41 |
*** vilobhmm111 has joined #openstack-nova | 19:42 | |
mriedem | lascii: ha, i'm surprised this worked https://review.openstack.org/#/c/287830/ | 19:44 |
mriedem | mgagne: ^ | 19:44 |
*** rk4n has joined #openstack-nova | 19:45 | |
*** vilobhmm11 has quit IRC | 19:45 | |
melwitt | mriedem: that's cool! | 19:46 |
lascii | I wouldn't have expected those to be related, but that's great | 19:48 |
*** Sukhdev has joined #openstack-nova | 19:49 | |
*** rk4n has quit IRC | 19:50 | |
mriedem | maybe we should get that in, or should we recheck a few times? | 19:50 |
melwitt | I trust it. YOLO | 19:50 |
lascii | hah | 19:51 |
melwitt | it says it's a test though, so we might want to change the commit message | 19:51 |
*** fawadkhaliq has joined #openstack-nova | 19:52 | |
mriedem | i could | 19:54 |
mriedem | and add the bugs that it's related to | 19:54 |
mriedem | also, unrelated, this doesn't look good http://tinyurl.com/gvt5h56 | 19:54 |
openstackgerrit | Sven Anderson proposed openstack/python-novaclient: Return a less dramatic message for public flavors. https://review.openstack.org/288639 | 19:58 |
openstackgerrit | Tim Pownall proposed openstack/nova: Updating while loop for tar extraction in utils.py. https://review.openstack.org/287426 | 19:58 |
openstackgerrit | Matt Riedemann proposed openstack/nova: Enable rebuild tests in cellsv1 job https://review.openstack.org/287830 | 19:59 |
mriedem | lascii: melwitt: ^ | 19:59 |
*** jschlueter has quit IRC | 20:00 | |
mriedem | doffm: https://github.com/openstack/nova/blob/master/nova/cmd/network.py#L65 | 20:00 |
ansiwen | mriedem: with the message from server API, with error exit code, but without "ERROR": https://review.openstack.org/288639 | 20:01 |
openstackgerrit | Tim Pownall proposed openstack/nova: Updating while loop for tar extraction in utils.py. https://review.openstack.org/287426 | 20:01 |
*** Sukhdev has quit IRC | 20:02 | |
mriedem | ansiwen: why add the new exception type? | 20:02 |
*** anush has quit IRC | 20:02 | |
ansiwen | because it doesn't print the ERROR | 20:02 |
ansiwen | it was there already | 20:02 |
ansiwen | I just made use of it | 20:02 |
mriedem | https://review.openstack.org/#/c/288639/2/novaclient/exceptions.py is new... | 20:03 |
ansiwen | yes, but it's handler existed | 20:03 |
mriedem | where is the handler? | 20:06 |
*** derekh has joined #openstack-nova | 20:06 | |
*** sheel has quit IRC | 20:07 | |
mriedem | ansiwen: you're still going to get an ERROR with BadCommand | 20:07 |
mriedem | https://github.com/openstack/python-novaclient/blob/master/novaclient/shell.py#L1083 | 20:07 |
ansiwen | mriedem: yes, you are right | 20:08 |
mriedem | so why add a new type of exception is my question | 20:09 |
mriedem | just use CommandError | 20:09 |
*** Marga_ has joined #openstack-nova | 20:10 | |
ansiwen | mriedem: I grepped for "except.*CommandError" and found the file basecommand.py which handles it, and prints an ERROR in front of the message. BadCommand did the same without error. but now I realized, it was a file from pip in the .tox venv (>_<) | 20:10 |
ansiwen | mriedem: so, there is no reason to add another exception | 20:11 |
*** vilobhmm111 has quit IRC | 20:13 | |
*** vilobhmm11 has joined #openstack-nova | 20:13 | |
*** ctrath has quit IRC | 20:15 | |
*** derekh has quit IRC | 20:18 | |
ansiwen | mriedem: so, I could add the handler for BadCommand to get rid of the misleading ERROR... if I just change the message from "ERROR: Failed to get access list for public flavor type." to "ERROR: Access list not available for public flavors." there is not mich difference, WDYT? | 20:20 |
mriedem | ansiwen: which lib is basecommand in? | 20:22 |
*** ctrath has joined #openstack-nova | 20:22 | |
ansiwen | mriedem: in pip itself. forget it. it was my mistake. | 20:22 |
ansiwen | mriedem: my grep command found it by accident, because it was in the .tox directory, and I didn't pay attention | 20:23 |
*** rfolco has quit IRC | 20:23 | |
*** rfolco has joined #openstack-nova | 20:24 | |
mriedem | ansiwen: so to clarify, the handler for CommandError is in some other library that isn't python-novaclient? | 20:24 |
ansiwen | mriedem: yes, it's a completely different handler for an exception that happens to also be named "CommandError" | 20:25 |
mriedem | so by adding a handler, you mean in the nova shell code https://github.com/openstack/python-novaclient/blob/master/novaclient/shell.py#L1083 to handle BadCommand and do the same as Exception but not prefix the message with ERROR? | 20:26 |
ansiwen | exactly | 20:26 |
*** lpetrut has joined #openstack-nova | 20:27 | |
*** sc68cal has quit IRC | 20:29 | |
mriedem | idk | 20:30 |
openstackgerrit | Matt Riedemann proposed openstack/nova: Tweak the resize_confirm_window help text https://review.openstack.org/288723 | 20:30 |
mriedem | if i saw except BadCommand in there it could be a bit confusing, but if you doc'ed the code well enough to signal why it's different and special then it might be fine | 20:30 |
*** pushkaru has joined #openstack-nova | 20:30 | |
mriedem | maybe i'll return the favor to superdan and ask for his opinion | 20:31 |
cfriesen_ | mriedem: The idea behind https://review.openstack.org/288109 seems reasonable to me. But if we're rebuilding on the same host we won't be able to tell if it's shared storage so it'll always keep it around. | 20:31 |
ansiwen | well, I also don't care too much about such a sub-low-prio thing. I just know there are users that find this ERROR irritating. | 20:32 |
mriedem | s/users/support engineers that have to explain it's a usage error rather than a real failure/ | 20:32 |
*** salv-orlando has quit IRC | 20:33 | |
cfriesen_ | mriedem: I've asked lbeliveau to take a look at https://bugs.launchpad.net/nova/+bug/1551504 since it's more his area of expertise | 20:33 |
openstack | Launchpad bug 1551504 in OpenStack Compute (nova) "I/O (PCIe) Based NUMA Scheduling can't really achieve pci numa binding in some cases." [Undecided,In progress] - Assigned to jinquanni(ZTE) (ni-jinquan) | 20:33 |
*** salv-orlando has joined #openstack-nova | 20:33 | |
superdan | mriedem: ansiwen you guys have beat this to death farther than I think is necessary, so I'm not sure it matters. Just changing the text so it's not very confusing seems fine to me, ERROR prefix or not | 20:33 |
mriedem | cfriesen_: couldn't we just add a host check? (and instance.host != self.host)? | 20:33 |
mriedem | superdan: i think it's probably a good topic for the TC or cross-project meeting | 20:34 |
mriedem | the osc guys probably know what to do! | 20:34 |
cfriesen_ | mriedem: yes, I think that'd work. but we'd end up deleting the disks in the case where we are actually on shared storage but happen to rebuild on the same host. | 20:34 |
superdan | mriedem: barf | 20:34 |
ansiwen | hehe | 20:34 |
mriedem | cfriesen_: so.... | 20:35 |
mriedem | what i'm hearing is, | 20:35 |
mriedem | if you're using ceph, | 20:35 |
mriedem | don't ever rebuild your instance | 20:35 |
mriedem | while using neutron | 20:35 |
mriedem | b/c if the vif plug fails, your disks are toast | 20:35 |
*** zengine has quit IRC | 20:36 | |
ansiwen | mriedem, superdan: well, if at least you guys have some fun it worth it spending my friday evening with something I found nt necessary in the first place... :-) | 20:36 |
cfriesen_ | mriedem: lol. so with ceph we don't ever want to destroy the disks in this case. with non-ceph shared storage it's tricky since we want to preserve the disks if we're on shared storage, but we can't reliably tell if that's the case. | 20:36 |
cfriesen_ | mriedem: but what you propose (with the host check) is better than what we have now | 20:37 |
cfriesen_ | mriedem: why do we want to destroy the disks in the failure case? | 20:38 |
cfriesen_ | mriedem: if _create_domain_and_network fails does it revert the instance.host back to the previous value? (or have we not stored the new host yet?) | 20:39 |
*** cfarquhar has joined #openstack-nova | 20:40 | |
*** cfarquhar has quit IRC | 20:40 | |
*** cfarquhar has joined #openstack-nova | 20:40 | |
mriedem | ansiwen: so i think we should at least update the error message to match what's in the rest api | 20:42 |
mriedem | if you care about the ERROR prefix, feel free to handle it, i don't really care about that | 20:42 |
mriedem | and it would make the change more complicated | 20:42 |
openstackgerrit | Sven Anderson proposed openstack/python-novaclient: Return a less dramatic message for public flavors. https://review.openstack.org/288639 | 20:44 |
mriedem | ansiwen: and finally | 20:44 |
mriedem | is there an LP bug for your BZ? | 20:44 |
mriedem | not a huge deal if not | 20:44 |
*** ericksonsantos has quit IRC | 20:45 | |
mriedem | cfriesen_: yeah i could add the host check....i'd appreciate if you have a guy that can help look at that | 20:46 |
ansiwen | mriedem: so, originally my task was to file a bug because of this, and I thought, no hell, it's even less work if I just immediately push a change... well, I was wrong... :-))) | 20:46 |
mriedem | cfriesen_: i really just put it up so the bug reporter could test it out | 20:46 |
*** Sukhdev has joined #openstack-nova | 20:46 | |
mriedem | ansiwen: +2 | 20:47 |
ansiwen | mriedem: so, no, there is no bug, but if you think it is useful, I'm happy to create one... *smirk* | 20:47 |
mriedem | no i don't really care | 20:47 |
mriedem | i feel the message is improved in this case | 20:47 |
ansiwen | ok, great | 20:47 |
mriedem | let us never speak of this ever again | 20:47 |
ansiwen | agreed | 20:48 |
ansiwen | ok, i'm off, have a great weekend | 20:48 |
*** thorst_ has joined #openstack-nova | 20:48 | |
*** ivase_ has joined #openstack-nova | 20:50 | |
*** ivase has quit IRC | 20:50 | |
*** thorst has quit IRC | 20:52 | |
*** rk4n has joined #openstack-nova | 20:54 | |
*** thumpba has quit IRC | 20:54 | |
*** thumpba has joined #openstack-nova | 20:55 | |
*** geekinutah has quit IRC | 20:55 | |
*** rk4n has quit IRC | 20:57 | |
*** delattec has quit IRC | 20:58 | |
*** rk4n has joined #openstack-nova | 20:58 | |
*** Sukhdev has quit IRC | 20:59 | |
*** thumpba has quit IRC | 21:00 | |
*** rk4n has quit IRC | 21:02 | |
*** MVenesio has quit IRC | 21:04 | |
*** rhagarty_ has quit IRC | 21:07 | |
*** rhagarty_ has joined #openstack-nova | 21:07 | |
openstackgerrit | Nicolas Simonds proposed openstack/nova: Allow use of live snapshots with RBD snapshot/clone https://review.openstack.org/273652 | 21:09 |
auggy | i have a question about string freeze, is it technically today? | 21:09 |
auggy | or is it after midnight today? | 21:09 |
*** apoorvad has quit IRC | 21:10 | |
mriedem | jwcroppe: do you have a fix for this else i'll push one up quick https://bugs.launchpad.net/nova/+bug/1553319 | 21:14 |
openstack | Launchpad bug 1553319 in OpenStack Compute (nova) "When CPU metric collection fails, stack trace not in nova logs" [Undecided,New] - Assigned to Joe Cropper (jwcroppe) | 21:14 |
mriedem | auggy: i think we're in soft string freeze now https://wiki.openstack.org/wiki/StringFreeze#.28Soft.29_String_Freeze | 21:15 |
mriedem | so no user-facing translatable string changes | 21:15 |
*** ivase_ is now known as ivase | 21:16 | |
auggy | right, i just wasn't sure if the date meant right now or not | 21:16 |
auggy | mriedem: do you know if translations are happening on the novaclient? i don't see a locale folder | 21:16 |
mriedem | yes, now | 21:16 |
mriedem | m-3 is out so we're in FF | 21:16 |
*** penick has quit IRC | 21:16 | |
mriedem | auggy: i don't think they are | 21:16 |
mriedem | even if they were, i don't know that ansiwen's change would fall under that since clients release on their own schedule.... | 21:16 |
mriedem | but it's a decent question | 21:17 |
auggy | yeah that's why i was asking | 21:17 |
mriedem | however, | 21:17 |
*** raildo is now known as raildo-afk | 21:17 | |
mriedem | that would be a great excuse to make ansiwen's head explode right now :) | 21:17 |
auggy | hahahahahaahahah so mean | 21:17 |
auggy | so i should probably find out what the internationalization folks would need from us for the novaclient | 21:18 |
auggy | unless all the clients have some other translation process? sorry i warned you all was still new at this ;) | 21:19 |
openstackgerrit | jichenjc proposed openstack/nova: Add functional regression test for bug 1552888 https://review.openstack.org/288746 | 21:19 |
openstack | bug 1552888 in OpenStack Compute (nova) "ERROR (BadRequest): expected string or buffer" [High,In progress] https://launchpad.net/bugs/1552888 - Assigned to Alex Xu (xuhj) | 21:19 |
*** edtubill has quit IRC | 21:20 | |
melwitt | I think last I heard it was thought not worth the effort to translate novaclient as the openstackclient shell is the way forward to be used | 21:20 |
auggy | thanks melwitt | 21:21 |
*** smatzek has quit IRC | 21:21 | |
melwitt | but since openstackclient uses each client library, any messages inside the python API would still need translation. but I don't think things got that specific yet | 21:21 |
mriedem | zzzeek: lascii: interested bug reported today which talks about adding an index to significantly drop the time it takes to run a big instances join query https://bugs.launchpad.net/nova/+bug/1553184 | 21:21 |
openstack | Launchpad bug 1553184 in OpenStack Compute (nova) "Slow Query on Instances Table " [Undecided,New] | 21:21 |
mriedem | zzzeek: lascii: i don't follow all of the 'explain' details in there, but if it makes sense we could see if it helps | 21:22 |
mriedem | would be nice to run a test on a large data set like we used to be able to do with rax | 21:22 |
auggy | melwitt: is that just an open discussion or is there a group or a person heading up that whole decision? | 21:22 |
auggy | like who do i bug for a clear answer or at least clear non-answer ;D | 21:23 |
melwitt | auggy: I think it's still an open question how the clients translation should be done. I'm not aware that any group has dug into it enough. let me do a quick check for lp bugs/emails | 21:24 |
openstackgerrit | Joe Cropper proposed openstack/nova: Dump metric exception text to logs https://review.openstack.org/288750 | 21:24 |
jwcroppe | mriedem: ^^ | 21:25 |
melwitt | auggy: this is the ML thread I was thinking of http://lists.openstack.org/pipermail/openstack-dev/2015-September/074922.html | 21:26 |
auggy | kk thx melwitt, i'm happy to look too just still figuring out how to find things in this crazy ecosystem we have here | 21:26 |
auggy | melwitt: perfect | 21:27 |
auggy | i'll post something to the i18n list to confirm that, especially since i might not be the only new liaison with that question | 21:27 |
melwitt | auggy: sure. I thought I can help give you a starting point since I remember seeing some discussion in the past. good luck :) | 21:27 |
auggy | haha thanks | 21:27 |
lascii | mriedem: I don't really understand how indexes and ORDER BY clauses work together, maybe jay can comment when he's around | 21:27 |
mriedem | yeah i was going to ping him but he's gone | 21:28 |
*** rfolco has quit IRC | 21:32 | |
auggy | seems just adding the index on a field you're filtering by would cause a performance gain independent of the order by? but maybe i'm just confused | 21:34 |
auggy | oh right he said that in the bug lol | 21:35 |
* auggy can read, there was no gain | 21:35 | |
lascii | oh I see, I got lost in the query | 21:35 |
*** apoorvad has joined #openstack-nova | 21:36 | |
lascii | the order by isn't relevant it seems | 21:36 |
lascii | yeah, the index makes sense. but it would be good to understand why that subquery is even there | 21:37 |
*** penick has joined #openstack-nova | 21:38 | |
lascii | nvm, I'm confused again | 21:42 |
auggy | i think if order by creates a virtual table based on an unindexed field then indexing the field you're trying to order by on would make sense | 21:42 |
openstackgerrit | jichenjc proposed openstack/nova: Add functional regression test for bug 1552888 https://review.openstack.org/288746 | 21:44 |
openstack | bug 1552888 in OpenStack Compute (nova) "ERROR (BadRequest): expected string or buffer" [High,In progress] https://launchpad.net/bugs/1552888 - Assigned to Alex Xu (xuhj) | 21:44 |
lascii | it's not filtering on created_at though, so I don't see how that reduces the number of scanned rows | 21:44 |
auggy | lascii: i'm looking at this, maybe it's relevant? http://stackoverflow.com/questions/21758085/how-order-by-works-in-mysql | 21:45 |
lascii | I'm thinking that it just needs an index on deleted, and it's just using the deleted, created_at index partially | 21:45 |
lascii | ahh, the limit | 21:46 |
*** salv-orlando has quit IRC | 21:47 | |
*** knikolla has quit IRC | 21:48 | |
*** salv-orlando has joined #openstack-nova | 21:48 | |
lascii | I could see the index helping with sorting, I was missing how it reduced the scanned rows. but the limit does it | 21:48 |
*** jichen has quit IRC | 21:48 | |
lascii | auggy: thanks for the link, that helped clear it up | 21:50 |
auggy | lascii: yeah i was just confused by the composite index thing myself | 21:50 |
*** rtheis has quit IRC | 21:51 | |
*** salv-orlando has quit IRC | 21:51 | |
*** kfarr has joined #openstack-nova | 21:51 | |
*** salv-orlando has joined #openstack-nova | 21:52 | |
*** aysyd has quit IRC | 21:53 | |
auggy | i really need to learn how to read ;) | 21:55 |
mriedem | lascii: auggy: so if either of you know the change to make, the reporter was new and didn't know how to add a migration | 21:57 |
mriedem | might be worth just throwing something up | 21:57 |
auggy | like a hairball? | 21:57 |
*** pcaruana has joined #openstack-nova | 21:58 | |
lascii | I added some hints to the bug report. I could look at it Monday though, trying to finish something else up | 21:59 |
auggy | i have never done a database migration, i could give it a shot but would rather have someone around to assist if i get stuck on something | 22:01 |
auggy | lascii: if you do end up working on it i'd be happy to take a look to understand what you did so i'd have a better idea of how to do that in the future | 22:01 |
lascii | auggy: sure. or if you give it a shot I can help you through it | 22:02 |
auggy | lascii: alright i'll see if i can figure it out based on your hints :) | 22:02 |
mriedem | i was just adding it | 22:03 |
mriedem | we'll see who wins | 22:03 |
auggy | mriedem: probably you! | 22:03 |
auggy | but if you're doing it then i'll look over it so i see what to do in the futre | 22:04 |
*** thorst_ has quit IRC | 22:05 | |
*** dimtruck is now known as zz_dimtruck | 22:07 | |
*** thorst has joined #openstack-nova | 22:07 | |
openstackgerrit | Matt Riedemann proposed openstack/nova: Add index on instances table across deleted/created_at columns https://review.openstack.org/288765 | 22:10 |
mriedem | raginbajin: auggy: ^ needs tests still but that's the start | 22:10 |
*** e0ne has joined #openstack-nova | 22:11 | |
*** thorst has quit IRC | 22:12 | |
auggy | mriedem: i can add tests if you want, i'm looking for examples | 22:12 |
*** cdent has quit IRC | 22:12 | |
mriedem | i'm a bit puzzled which db api is doing this | 22:14 |
*** baoli_ has quit IRC | 22:14 | |
mriedem | i guess instance_get_all_by_filters | 22:14 |
*** jwcroppe has quit IRC | 22:16 | |
*** jwcroppe has joined #openstack-nova | 22:16 | |
mriedem | ah https://github.com/openstack/oslo.db/blob/master/oslo_db/sqlalchemy/utils.py#L149 | 22:17 |
mriedem | when you pass a sort key | 22:17 |
mriedem | but.... | 22:17 |
mriedem | hmm | 22:17 |
*** Sukhdev has joined #openstack-nova | 22:19 | |
*** Sukhdev has quit IRC | 22:20 | |
*** Sukhdev has joined #openstack-nova | 22:20 | |
*** thorst has joined #openstack-nova | 22:23 | |
mriedem | WIP | 22:23 |
mriedem | if the query is slow just because they used created_at as the sort key in the API, then adding the index on that is arbitrary | 22:24 |
*** akshai has quit IRC | 22:25 | |
lascii | mriedem: it's a default that comes from http://git.openstack.org/cgit/openstack/nova/tree/nova/db/sqlalchemy/api.py#n2182 | 22:27 |
lascii | http://git.openstack.org/cgit/openstack/nova/tree/nova/db/sqlalchemy/api.py#n2463 | 22:27 |
*** lascii is now known as alaski | 22:28 | |
*** busterswt has quit IRC | 22:28 | |
mriedem | alaski: nice find | 22:29 |
mriedem | do we need to add id to the index then? | 22:29 |
*** kfarr has quit IRC | 22:31 | |
alaski | that may help a small amount, but I think created_at is enough. I'm speculating though so it would be better to pull in an expert | 22:33 |
*** rlrossit_ has quit IRC | 22:34 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/288357 | 22:34 |
*** thorst has quit IRC | 22:35 | |
mriedem | alaski: yeah, i'll bug his jayness on monday | 22:36 |
mriedem | and zzzeek | 22:36 |
*** jschlueter has joined #openstack-nova | 22:36 | |
*** ircuser-1 has quit IRC | 22:39 | |
openstackgerrit | Matt Riedemann proposed openstack/nova: Add index on instances table across deleted/created_at columns https://review.openstack.org/288765 | 22:41 |
*** yonglihe has quit IRC | 22:42 | |
cburgess | I | 22:42 |
cburgess | superdan Oh sorry I missed the ping from earlier. Whats up? | 22:42 |
*** ctrath has quit IRC | 22:43 | |
*** ircuser-1 has joined #openstack-nova | 22:43 | |
*** pushkaru has quit IRC | 22:44 | |
*** pushkaru has joined #openstack-nova | 22:44 | |
*** devananda has quit IRC | 22:45 | |
*** sheel has joined #openstack-nova | 22:45 | |
*** mriedem has quit IRC | 22:46 | |
*** erlon has quit IRC | 22:46 | |
*** penick has quit IRC | 22:48 | |
*** zz_dimtruck is now known as dimtruck | 22:51 | |
*** e0ne has quit IRC | 22:55 | |
*** geekinutah has joined #openstack-nova | 22:55 | |
*** Sukhdev has quit IRC | 22:56 | |
*** haplo37 has quit IRC | 22:57 | |
*** devananda has joined #openstack-nova | 22:58 | |
*** ivase has quit IRC | 23:00 | |
*** geekinut1h has joined #openstack-nova | 23:00 | |
*** ivase has joined #openstack-nova | 23:00 | |
*** geekinutah has quit IRC | 23:00 | |
*** mdrabe has quit IRC | 23:00 | |
*** Sukhdev has joined #openstack-nova | 23:02 | |
*** lpetrut has quit IRC | 23:03 | |
*** geekinutah has joined #openstack-nova | 23:05 | |
*** edmondsw has quit IRC | 23:05 | |
*** Sukhdev has quit IRC | 23:06 | |
*** geekinut1h has quit IRC | 23:06 | |
*** bauwser is now known as bauzas | 23:09 | |
*** geekinutah has quit IRC | 23:14 | |
*** e0ne has joined #openstack-nova | 23:15 | |
*** angdraug has quit IRC | 23:16 | |
*** xyang1 has quit IRC | 23:29 | |
*** Sukhdev has joined #openstack-nova | 23:32 | |
*** thorst has joined #openstack-nova | 23:35 | |
*** Sukhdev has quit IRC | 23:37 | |
*** fawadkhaliq has quit IRC | 23:38 | |
*** fawadkhaliq has joined #openstack-nova | 23:39 | |
*** Jeffrey4l has joined #openstack-nova | 23:43 | |
*** thorst has quit IRC | 23:44 | |
*** mrkz has quit IRC | 23:48 | |
*** ctrath has joined #openstack-nova | 23:52 | |
*** ctrath has quit IRC | 23:52 | |
*** salv-orl_ has joined #openstack-nova | 23:52 | |
*** Bjoern has quit IRC | 23:53 | |
*** salv-orlando has quit IRC | 23:55 | |
*** ducttape_ has quit IRC | 23:55 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!