| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-01-16 | |||
| 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: SEA https://review.openstack.org/523216 | |
| 18:58:37 | openstackgerrit | Eric Berglund proposed openstack/nova master: PowerVM driver: ovs vif https://review.openstack.org/422512 | |
| 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: vSCSI volume driver https://review.openstack.org/526094 | |
| 01:03:15 | openstackgerrit | Eric Berglund proposed openstack/nova master: PowerVM Driver: SEA https://review.openstack.org/523216 | |
| 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 | |
| 05:15:21 | openstackgerrit | Merged openstack/nova master: hyper-v: Deprecates support for Windows / Hyper-V Server 2012 https://review.openstack.org/530953 | |
| 05:15:57 | openstackgerrit | Merged openstack/nova master: Fix accumulated nits https://review.openstack.org/534368 | |
| 06:03:20 | openstackgerrit | Hironori Shiina proposed openstack/nova master: Ironic: Get IP address for volume connector from ports/portgroups https://review.openstack.org/517322 | |
| 06:38:22 | ameeda | Morning :) | |
| 06:42:57 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: nova-manage db archive_deleted_rows is not multi-cell aware https://review.openstack.org/507486 | |
| 06:59:42 | openstackgerrit | Merged openstack/nova master: [placement] Add sending global request ID in delete (2) https://review.openstack.org/526263 | |
| 08:44:22 | openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: [WIP]Fix update server's wrong meta when update concurrently https://review.openstack.org/534682 | |
| 08:49:24 | openstackgerrit | Lajos Katona proposed openstack/nova master: WIP: Add nested resources to server moving tests https://review.openstack.org/527728 | |
| 08:53:41 | openstackgerrit | Eli Qiao proposed openstack/nova master: Api-guide: Add Block Device Mapping https://review.openstack.org/522084 | |
| 09:12:59 | openstackgerrit | Stephen Finucane proposed openstack/nova master: trivial: Remove 'tools/releasenotes_tox.sh' https://review.openstack.org/534383 | |
| 09:12:59 | openstackgerrit | Stephen Finucane proposed openstack/nova master: trivial: Cleanup tox.ini https://review.openstack.org/534382 | |
| 09:29:40 | ameeda | gmann: please don't forget me about this gerrit "https://review.openstack.org/#/c/526900/" | |
| 09:42:10 | kashyap | stephenfin: Heya, when you get a moment, mind taking a gander at this (one of the use cases is right in your wheel house)? https://review.openstack.org/#/c/534384/ -- "[WIP] libvirt: Allow to specify granular CPU feature flags" | |
| 09:42:29 | kashyap | stephenfin: It's still WIP, I need to fix the 11 or so broken tests | |
| 09:42:58 | kashyap | But thought I'd put it out there, as we should it'll alleviate the performance impact that the recent "Meltdown" CVE fixes cause. | |
| 09:43:11 | kashyap | The commit message tells it all (I mean, all) :-) | |
| 09:47:06 | stephenfin | kashyap: Sounds good | |
| 09:48:18 | bauzas | stephenfin: just in case, you could also be interested in https://review.openstack.org/#/c/528832/ | |
| 09:48:20 | kashyap | stephenfin: Just to note --- next two days I'm a bit sporadic in my availability, Friday night, I'lll be back home | |
| 09:48:31 | bauzas | stephenfin: ;) | |
| 09:56:42 | BKaminski | There any etiquette on adding someone as a reviewer to a commit if you know they know that code well? | |
| 09:58:03 | kashyap | BKaminski: Sure, it's fine. | |
| 09:58:28 | BKaminski | Alrighty, thanks :) | |
| 09:59:38 | kashyap | BKaminski: E.g. sometimes I run this command: git shortlog -nse nova/$File-of-Interest | |