Earlier  
Posted Nick Remark
#openstack-nova - 2018-01-16
18:35:55 efried agreeeeed.
18:36:10 efried edleafe I think what I'm actually coming around to is that we shouldn't do the deep copy of the nested children in .snapshot() - at least not by default.
18:36:46 efried edleafe Then at least we're only throwing away one provider's worth of stuff we copied for no reason.
18:38:11 sean-k-mooney stephenfin: jaypipes i ran all the tempest smoke test + senario test + tox -e py35,py27,functional,pep8,docs locally on https://review.openstack.org/#/c/534307/2 here http://paste.openstack.org/show/645744/ all passed so just waiting on the gate to do the same.
18:38:15 edleafe or have snapshot() take an optional rp_uuid
18:38:32 edleafe and only return that provider w/o children
18:38:54 efried edleafe It already does take rp_uuid; that's how you tell it which provider you want to snapshot.
18:41:23 esberglu stephenfin: Asked around and it turns out that nova-networking isn't supported. You okay with leaving bit in for now and removing it in a follow up?
18:41:34 esberglu Trying to avoid respinning the series if possible
18:42:02 sean-k-mooney esberglu: nova-networking has not been removed yet. it will be in early rocky
18:42:28 esberglu sean-k-mooney: This is in regards to powervm virt driver support
18:42:52 esberglu It only supports neutron
18:43:27 sean-k-mooney esberglu: oh well if povervm never supported nova-net before or you have already dropped(or never supported) cells v1 then you dont need nova-networks support
18:44:02 efried esberglu fyi stephenfin is on UK time, so may be gone for the evening.
18:44:22 esberglu efried: ahh thanks for the info
18:45:28 efried esberglu And gibi is also euro. So you might as well respin the series and they should be able to nail 'em quick in the morning (like 4am for us).
18:46:53 sean-k-mooney efried: are you east coast or west?
18:47:12 efried sean-k-mooney Central (Texas)
18:47:27 sean-k-mooney ah ok
18:47:31 efried I think esberglu is either Central or East
18:47:47 esberglu central
18:48:26 openstackgerrit Eric Berglund proposed openstack/nova master: PowerVM Driver: vSCSI volume driver https://review.openstack.org/526094
18:49:10 sean-k-mooney ya unlike me stephen starts early and finishes early. i generally stay later but start later to give more us overlap in my day. also i hate have 2 7 oclocks in my day unless the frist on is 7 pm
18:50:10 efried sean-k-mooney Sounds like you coordinate way more than we do :)
18:51:14 efried As far as meetings go, I consider a successful week to be one with three IRC meetings and zero in-person or phone meetings.
18:52:28 efried (at least, zero in-person meetings that don't involve choking the persons involved)
18:52:52 openstackgerrit Claudiu Belu proposed openstack/nova master: tests: fixes mock autospec usage https://review.openstack.org/447505
18:53:18 sean-k-mooney efried: haha i would love that. lately i have not had much internal meeting but in the past i had. i like beeing on irc when most of the nova,neutron and kolla core team are about too so haveing a slightly later day helps
18:53:41 efried Mm. I believe cdent is of similar philosophy.
18:54:03 edleafe efried: I have 3 phone meetings *today*
18:54:28 cdent ... that's more side-effect than goal
18:58:37 openstackgerrit Eric Berglund proposed openstack/nova master: PowerVM driver: ovs vif https://review.openstack.org/422512
18:58:37 openstackgerrit Eric Berglund proposed openstack/nova master: PowerVM Driver: SEA https://review.openstack.org/523216
18:58:38 openstackgerrit Eric Berglund proposed openstack/nova master: PowerVM Driver: vSCSI volume driver https://review.openstack.org/526094
19:01:10 sean-k-mooney efried_nomnom: esberglu is there a reason you are not using and os-vif plugin intead of https://review.openstack.org/#/c/422512/40/nova/virt/powervm/vif.py
19:02:17 efried_nomnom sean-k-mooney The main reason is probably because os-vif didn't exist when we wrote this stuff, or even when we first started porting it in tree.
19:02:51 sean-k-mooney efried_nomnom: os-vif has been around for 3 maybe 4 cycles now
19:03:11 sean-k-mooney efried_nomnom: it would be nice to convert in rocky
19:03:23 efried_nomnom Duly noted. esberglu ^ for the to-do list?
19:03:59 efried_nomnom Also sean-k-mooney we wouldn't know how to do that offhand. But since you've volunteered to guide us.... :)
19:04:06 cfriesen I'm trying to figure out if a bug is in nova or libvirt/qemu. Do we have any functional/tempest tests of live migration with a config drive and an attached cinder volume?
19:04:45 Roamer` hmm, if mriedem is really on vacation, could I interest any of you in taking a quick look at https://review.openstack.org/140733/ - the StorPool libvirt volume attachment driver? :) The Cinder and os-brick support is in, and Nova's requirements.txt has a dependency on the new version of os-brick now.
19:05:31 Roamer` of course, "leave it till he gets back" might also be kind of reasonable... although it may be cutting it a bit too close to the January 25th deadline
19:06:34 sean-k-mooney efried_nomnom: sure im happy to help. its not a huge amount of work to do but it means you dont have to submit patches to nova to take advantage of things like ovs now support jumbo frames with datapath x or other network backend specific features
19:07:15 efried_nomnom sean-k-mooney Thanks in advance
19:08:25 sean-k-mooney efried_nomnom: other then requireing a powerpc system is there any other requirement to be able to deploy powervm. e.g. is it opensouce?
19:10:10 efried_nomnom sean-k-mooney As long as "having a POWER system" includes relevant licensing (e.g. for VIOS), no other requirements I believe. Everything else is free (though not all of it is open source).
19:11:21 sean-k-mooney efried_nomnom: hum ok i was wondering if i could emulate PPC under kvm and in stall powervm hyperviors under something like ubunut but i guess not
19:11:30 cfriesen melwitt: do you know what the fix was for https://bugzilla.redhat.com/show_bug.cgi?id=1394964 (I'm guessing "mwitt@redhat.com" is you...)
19:11:31 openstack bugzilla.redhat.com bug 1394964 in openstack-nova "Live migration with config-drive fails with InvalidSharedStorage error" [High,Closed: errata] - Assigned to mwitt
19:11:52 efried_nomnom sean-k-mooney No, fraid not.
19:12:23 efried_nomnom sean-k-mooney I mean, if you *could* actually emulate POWER (which isn't PowerPC, btw) then I imagine you could do that.
19:13:00 efried_nomnom sean-k-mooney But emulating POWER is very, very hard. There is/was a project around here to do it, but I don't think it's up to date; and it's dogggggg slow.
19:13:15 sean-k-mooney oh i did not know they were different. PPC is ibm right is POWER openPOWER
19:13:37 efried_nomnom I'm not sure what the status of ppc is actually.
19:13:54 efried_nomnom And POWER is *almost* openPOWER.
19:14:09 melwitt cfriesen: yes, that is me. looking ...
19:14:20 efried_nomnom I think openpower is a way for not-IBM to build POWER-compatible systems.
19:15:12 efried_nomnom But I'm totally not a terminology/marketing guy. I just try to write as much code as possible.
19:16:25 cfriesen melwitt: maybe bc4b9980f ?
19:16:26 sean-k-mooney til well i shoudl be able to help in anycase. there is a lot you can do with unitest to fake things out + there is the IBM PowerVM CI so if you remind me/ i get time i might take a go at it but not before the PTG/end of Q1
19:19:07 melwitt cfriesen: so it was fixed by having https://review.openstack.org/#/c/227278 and removing VIR_MIGRATE_TUNNELLED from the block_migration_flag in nova.conf
19:19:29 melwitt and I think config drive has to be vfat also
19:20:49 cfriesen melwitt: okay, was wondering about that last part. we're still (in Pike) hitting issues with iso9660 config drives and boot-from-volume on a compute node with local storage.
19:21:45 cfriesen melwitt: specifically with block-live-migrating such instances
19:22:03 cfriesen I'm trying to see if it's something we broke or an upstream issue.
19:22:07 melwitt cfriesen: yeah, I can't remember the context now but if you dig into the libvirt driver code there are a lot of conditions that have to be met before live block migration will work, and if you have a config drive, that's local storage so has to be a live BLOCK migration
19:22:43 melwitt you can't have the tunneling but I don't remember why
19:23:48 cfriesen melwitt: that part I know...tunnelling prevents the "block migrate some disks but not others" behaviour in libvirt
19:23:56 melwitt oh, okay. cool
19:24:12 melwitt now I know then :)
19:29:10 melwitt cfriesen: I know that in the past iso9660 config drives couldn't be live migrated, I don't know when/if that was fixed. based on the absence of 'vfat' in the pike code I'm guessing it was fixed
19:30:12 melwitt I just updated the BZ to make a lot of the comments public that didn't need to be private
19:30:32 melwitt in case that might help ppl that happen upon it
19:36:37 melwitt looks like iso9660 migration was enabled by https://review.openstack.org/#/c/234659
20:09:20 openstackgerrit Merged openstack/nova master: Rename block_device_info_get_root https://review.openstack.org/529028
20:21:50 sean-k-mooney jaypipes: looks like the gate is happy with it. https://review.openstack.org/#/c/534307/2 once that is merged im happy to go with a 1.8.1 release.
20:24:07 jaypipes sean-k-mooney: question... why don't we just straight revert the pyroute patch from rodolfo>?
20:46:03 openstackgerrit Eric Berglund proposed openstack/nova master: PowerVM driver: ovs vif https://review.openstack.org/422512
20:46:36 openstackgerrit Eric Berglund proposed openstack/nova master: PowerVM Driver: SEA https://review.openstack.org/523216
20:47:07 openstackgerrit Eric Berglund proposed openstack/nova master: PowerVM Driver: vSCSI volume driver https://review.openstack.org/526094
20:47:51 sean-k-mooney2 jaypipes rodolfo patch also introduces the infra for functional testing and deduplicates the logic from linux bridge plugin and ovs
20:48:52 sean-k-mooney2 So i would like to keep that and just delay moving to pyroute2 until a new release is out
20:49:52 jaypipes sean-k-mooney2: ok...
21:07:19 openstackgerrit Kashyap Chamarthy proposed openstack/nova master: [WIP] libvirt: Allow to specify granular CPU feature flags https://review.openstack.org/534384
21:42:50 matrohon sean-k-mooney: hi
21:45:33 cfriesen melwitt: mdbooth: it turns out that if you have an instance with an iso9660 config drive and a ceph-backed attached cinder volume, then block-live-migration fails in libvirt with "qemuMigrationPrecreateStorage:463 : Operation not supported: pre-creation of storage targets for incremental storage migration is not supported". With iscsi-backed cinder volumes it works fine.
21:46:51 cfriesen At least in our code....still trying to track down whether we broke something or if it's upstream.
22:53:55 openstackgerrit Jacek Tomasiak proposed openstack/python-novaclient master: Fix listing of instances above API max_limit https://review.openstack.org/534222
22:59:14 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Separate API schemas (resource_provider) https://review.openstack.org/528629
22:59:58 openstackgerrit Takashi NATSUME proposed openstack/nova stable/pike: Add 'delete_host' command in 'nova-manage cell_v2' https://review.openstack.org/513717
#openstack-nova - 2018-01-17
00:20:19 openstackgerrit Merged openstack/nova master: archive_deleted_instances is not atomic for insert/delete https://review.openstack.org/526065
00:20:32 openstackgerrit Merged openstack/nova master: Enable py36 unit tests in tox https://review.openstack.org/534223
01:03:14 openstackgerrit Eric Berglund proposed openstack/nova master: PowerVM driver: ovs vif https://review.openstack.org/422512
01:03:15 openstackgerrit Eric Berglund proposed openstack/nova master: PowerVM Driver: SEA https://review.openstack.org/523216
01:03:15 openstackgerrit Eric Berglund proposed openstack/nova master: PowerVM Driver: vSCSI volume driver https://review.openstack.org/526094
01:30:26 openstackgerrit Merged openstack/nova master: Fix missing rps in allocation candidates https://review.openstack.org/522409
03:19:40 openstackgerrit nafeiyang proposed openstack/nova master: z/VM driver: add snapshot function https://review.openstack.org/534240
03:57:46 openstackgerrit Merged openstack/nova master: func tests for nested providers in alloc candidate https://review.openstack.org/531443
05:01:14 openstackgerrit Merged openstack/nova master: Updated from global requirements https://review.openstack.org/533046

Earlier   Later