Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-03-07
20:24:29 mriedem dansmith: that would be lovely
20:25:07 mriedem i'm going to start reviewing https://review.openstack.org/#/c/440856/ and up, do i need any pep talk before i start?
20:25:32 dansmith mriedem: you can do it, you big code reviewing stud muffin you
20:25:42 mriedem heh
20:25:56 melwitt you might need a montage
20:26:41 mriedem thanks miyagi https://www.youtube.com/watch?v=iBktYJsJq-E
20:26:51 dansmith mriedem: actually the pep talk is: consider all the infra puppies you kill by asking for nitty changes at the bottom, knowing that I'll follow up with legit stuff :P
20:27:12 mriedem psh
20:27:18 mriedem just for saying that, i'm going to be extra critical
20:27:19 dansmith I'm shocked I haven't gotten a citation already today with all the resources this set is consuming
20:27:47 mriedem sdague removed all of the scenario tests so no one cares anymore
20:27:55 dansmith heh
20:31:47 dansmith I'd like to add up my "bill" for all the resources I use in nodepool for a month
20:31:51 dansmith just for shits and giggles
20:38:56 migi dansmith: ok one more question for today. Let's say I have this dump and will run nova-manage. Do I need to know for which nova version the sql was created or can I use nova-manage from Liberty against db created by master?
20:39:23 dansmith migi: they have to match
20:39:41 mriedem dansmith: blam https://review.openstack.org/#/c/441388/5
20:39:43 migi dansmith: and this info is somewhere in sql or do I need to store it in metadata somewhere outisde
20:39:56 migi dansmith: that was too much of questions as I said last one one before
20:40:13 dansmith migi: you can get the version number out of the database somewhere, yeah
20:40:22 migi dansmith: ok will find it then, thanks
20:40:38 dansmith migi: I don't recall where, but it's like a single-row table that sqla-migrate uses
20:40:52 mriedem it's the 'nova.migrations' table i believe
20:41:18 migi mriedem: thanks, will find it :)
20:44:30 dansmith mriedem: ack
20:44:33 mriedem https://www.youtube.com/watch?v=X9UM4N5sGBs
20:44:33 dansmith well, that was qick
20:44:37 mriedem ha
20:44:53 dansmith here and I was being nice and writing your nova-status thing
20:44:54 mriedem i'm not sure i can put links to judas priest songs in reviews
20:45:05 jaypipes melwitt: sure, no prob. I was just reviewing the migration one. you want me to hold off?
20:45:22 melwitt jaypipes: hold off on the top patch, the third one
20:45:29 jaypipes gotcha.
20:45:30 jaypipes will do.
20:45:48 melwitt thanks :)
21:07:13 openstackgerrit Dan Smith proposed openstack/nova master: Add nova-status check for minimum service version https://review.openstack.org/442787
21:07:25 dansmith mriedem: ^
21:14:27 mriedem just get in line
21:19:33 dansmith mriedem: guh, that breaks my math doubling even for the other things
21:19:40 dansmith mriedem: and if I have only one cell, then we don't poke this
21:30:29 openstackgerrit Dan Smith proposed openstack/nova master: Get instance availability_zone without hitting the api db https://review.openstack.org/439754
21:30:29 openstackgerrit Dan Smith proposed openstack/nova master: Teach HostAPI about cells https://review.openstack.org/442162
21:30:29 openstackgerrit Dan Smith proposed openstack/nova master: Make os-availability-zones know about cells https://review.openstack.org/442163
21:30:30 openstackgerrit Dan Smith proposed openstack/nova master: Target cell in super conductor operations https://review.openstack.org/438022
21:30:31 openstackgerrit Dan Smith proposed openstack/nova master: Change MQ targeting to honor only what is in the context https://review.openstack.org/438023
21:30:31 openstackgerrit Dan Smith proposed openstack/nova master: Teach os-migrations about cells https://review.openstack.org/442685
21:30:32 openstackgerrit Dan Smith proposed openstack/nova master: Make scheduler target cells to get compute node instance info https://review.openstack.org/439891
21:30:32 openstackgerrit Dan Smith proposed openstack/nova master: Make consoleauth target the proper cell https://review.openstack.org/441461
21:30:33 openstackgerrit Dan Smith proposed openstack/nova master: Add cell field to Destination object https://review.openstack.org/438024
21:30:33 openstackgerrit Dan Smith proposed openstack/nova master: WIP: Make conductor ask scheduler to limit migrates to same cell https://review.openstack.org/438025
21:30:34 openstackgerrit Dan Smith proposed openstack/nova master: Teach simple_tenant_usage about cells https://review.openstack.org/441388
21:30:34 openstackgerrit Dan Smith proposed openstack/nova master: Teach os-aggregates about cells https://review.openstack.org/442200
21:30:36 openstackgerrit Dan Smith proposed openstack/nova master: Add workaround to disable group policy check upcall https://review.openstack.org/442736
21:30:51 dansmith mriedem: look at all that puppy blood you spilled
21:30:53 dansmith jaypipes: shake it at mriedem !
21:30:57 mriedem dude
21:30:59 jaypipes :)
21:30:59 mriedem clearly
21:31:08 dansmith mriedem: when I get back I'll remove that check behind the nova-status change
21:31:10 mriedem maybe if you guys didn't break the api contract
21:31:12 mriedem ...
21:31:29 dansmith nobody uses simple_tenant_usage
21:31:30 mriedem dansmith: comments are in the nova-status change also
21:31:34 dansmith yep, I saw
21:34:15 openstackgerrit Sarafraj Singh proposed openstack/nova-specs master: Add live-migration-force-after-timeout spec https://review.openstack.org/438467
21:38:35 openstackgerrit Artom Lifshitz proposed openstack/nova master: Plumbing for tagged nic attachment https://review.openstack.org/415622
21:38:36 openstackgerrit Artom Lifshitz proposed openstack/nova master: Delete nic metadata when detaching interface https://review.openstack.org/435387
21:38:38 openstackgerrit Artom Lifshitz proposed openstack/nova master: Libvirt support for tagged volume attachment https://review.openstack.org/418100
21:38:39 openstackgerrit Artom Lifshitz proposed openstack/nova master: Delete disk metadata when detaching volume https://review.openstack.org/435082
21:38:41 openstackgerrit Artom Lifshitz proposed openstack/nova master: Plumbing for tagged volume attachment https://review.openstack.org/391941
21:38:41 openstackgerrit Artom Lifshitz proposed openstack/nova master: API support for tagged device attachment https://review.openstack.org/376021
21:38:44 openstackgerrit Artom Lifshitz proposed openstack/nova master: Libvirt support for tagged nic attachment https://review.openstack.org/415520
21:47:13 mriedem dansmith: questions about sort order and cell0 in here https://review.openstack.org/#/c/441388/
21:47:21 mriedem i think cell0 is fine but wanted to ask it out loud
21:52:52 openstackgerrit melanie witt proposed openstack/nova master: Count resources to check quota for cells https://review.openstack.org/416521
21:52:53 openstackgerrit melanie witt proposed openstack/nova master: Make Quotas object favor the API database https://review.openstack.org/410945
21:52:53 openstackgerrit melanie witt proposed openstack/nova master: Add online migration to move quotas to API database https://review.openstack.org/410946
21:58:26 mtreinish mriedem: good waffles do stick together :)
22:07:44 openstackgerrit Sujitha proposed openstack/nova master: [WIP] Reduce calls to load_tags() to 0 https://review.openstack.org/435146
22:08:10 openstackgerrit Sujitha proposed openstack/nova master: Reduce calls to load_tags() to 0 https://review.openstack.org/435146
22:12:20 inc0 hey guys, so we've run into serious issue with new libvirt 2.5
22:12:55 inc0 issue being nova, when creating files for vms (like console.log), uses creates directory with permissions 744
22:13:12 inc0 so if you run nova-compute with different user than qemu, what we do in Kolla
22:13:30 inc0 qemu can't access console.log which makes spawning vm impossible
22:14:02 inc0 and with libvirt 2.5 for reasons unknown to me, dynamic permissiosn seems to be failing to chown these files
22:14:55 inc0 anyone seen that happening? can we specify permissions for vm files (i.e. change them to 774)?
22:32:17 openstackgerrit Sarafraj Singh proposed openstack/nova-specs master: Spec to add per instance timeout for LM https://review.openstack.org/442809
22:33:10 mriedem inc0: you'll probably need to talk to kashyap and/or sgordon on that one, maybe mdbooth - some of the red hat people,
22:33:22 mriedem given the time of day kashyap and mdbooth are probably no longer around, not sure about sgordon
22:33:48 mriedem we're nowhere near testing with libvirt 2.5 in the gate
22:34:18 mriedem we test with 1.3.1 which is what's in xenial
22:34:43 inc0 yes...but not in cloud archive
22:34:49 openstackgerrit Sarafraj Singh proposed openstack/nova-specs master: Add live-migration-force-after-timeout spec https://review.openstack.org/438467
22:34:56 inc0 we will probably fall back to 1.3 ourselves
22:35:13 openstackgerrit Sarafraj Singh proposed openstack/nova-specs master: Spec to add per instance timeout for LM https://review.openstack.org/442809
22:35:38 mriedem inc0: we == kolla?
22:35:46 inc0 mriedem: correct
22:36:22 inc0 I'm not sure when canonical bumped libvirt version this high..but here we are
22:36:24 mriedem ok. i'm not sure how you guys pick what your use or support, but definitely using anything beyond what we test with in the integrated gate using xenial opens you up to risk for stuff like this
22:36:45 mriedem inc0: might as well report a bug regardless

Earlier   Later