| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-01-11 | |||
| 14:15:56 | mdbooth | bauzas: I agree that moving that stuff is less important, though. | |
| 14:16:21 | mdbooth | I think where we differ is much less important we think it is ;) | |
| 14:16:37 | cdent | "source of truth" is definitely a common issue | |
| 14:17:09 | bauzas | distributed systems is a NP-problem by definition | |
| 14:17:18 | bauzas | so I'm not afraid | |
| 14:20:13 | mdbooth | bauzas: The other things are bugs, anyway. This is an issue of ongoing maintainability. | |
| 14:21:15 | kashyap | stephenfin: Hey, have you setup DevStack lately on an older branch (Mitaka) on CentOS? | |
| 14:21:41 | stephenfin | kashyap: Newton is as far back I've gone in a while, I think | |
| 14:21:42 | kashyap | I don't see Mitaka branch in DevStack, do they "delete" the older branches? | |
| 14:21:46 | stephenfin | and that was before it was EOL | |
| 14:21:56 | stephenfin | I guess they do. Look for the mitaka-eol tag | |
| 14:22:22 | kashyap | stack@devstack-centos ~/src/cloud/devstack (master) $ git branch -a | grep -i mitaka | |
| 14:22:25 | kashyap | stack@devstack-centos ~/src/cloud/devstack (master) $ echo $? | |
| 14:22:28 | kashyap | 1 | |
| 14:22:42 | stephenfin | git tag -a | grep -i mitaka | |
| 14:23:01 | kashyap | s/-a/-l/ | |
| 14:23:25 | stephenfin | Indeed | |
| 14:23:26 | kashyap | Yep, could fetch from the tag. Thx. | |
| 14:23:35 | stephenfin | Once you've that checked out, you'll also want to modify the branches that are checked out, e.g. https://gist.github.com/stephenfin/d1c000f7e52ff181c4673322b28eaf47#file-local-conf-L71-L81https://gist.github.com/stephenfin/d1c000f7e52ff181c4673322b28eaf47#file-local-conf-L71-L81 | |
| 14:23:46 | stephenfin | I guess they should all be 'mitaka-eol' | |
| 14:24:25 | kashyap | stephenfin: Yeah, my attempt on Fedora for Mitaka failed | |
| 14:25:07 | kashyap | Failed as in, the libvirt-python wonkiness: http://paste.openstack.org/show/642707/ | |
| 14:25:08 | stephenfin | Aye, an LTS distro is probably the way to go. Guess either Ubuntu 14.04 or CentOS 7.4 is your best bet | |
| 14:25:29 | kashyap | It's missing two of the events, when building libvirt-python | |
| 14:25:59 | kashyap | Didn't wanted to duke around there, so just went w/ CentOS. Let's see how that goes. | |
| 14:27:12 | kashyap | stephenfin: I used to use -- https://kashyapc.fedorapeople.org/virt/openstack/Minimal-DevStack-local.conf | |
| 14:27:26 | kashyap | I know it could be improved (Neutron is now default, etc). | |
| 14:50:09 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Collocate encryptor and volume driver calls https://review.openstack.org/460243 | |
| 14:50:10 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Introduce disk encryption config classes https://review.openstack.org/464008 | |
| 14:50:10 | openstackgerrit | Lee Yarwood proposed openstack/nova master: WIP libvirt: QEMU native LUKS decryption for volumes https://review.openstack.org/523958 | |
| 15:03:13 | mriedem | oops, totally forgot about the team meeting | |
| 15:03:51 | mriedem | gibi: thanks for running that | |
| 15:05:04 | openstackgerrit | sahid proposed openstack/nova master: libvirt: slow live-migration to ensure network is ready https://review.openstack.org/497457 | |
| 15:05:05 | gibi | mriedem: no problem, I was not prepared but I could run it | |
| 15:10:30 | efried | Hi cores, would anyone be willing to look at the first two patches for the PowerVM integration bp? https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/powervm-nova-it-compute-driver | |
| 15:11:00 | efried | stephenfin gibi bauzas perhaps if you have time? | |
| 15:11:43 | stephenfin | efried: I've been meaning to review the OVS VIF one, but I'll have to leave the storage patches to someone else | |
| 15:11:45 | bauzas | efried: I'm still rushing on my feature branch :/ | |
| 15:11:52 | bauzas | efried: blame mdbooth | |
| 15:12:09 | mdbooth | bauzas: Sorry :P | |
| 15:12:52 | bauzas | mdbooth: no worries, you just owe me a beer :p | |
| 15:12:58 | bauzas | et voila! | |
| 15:13:16 | bauzas | interesting, creating a new mdev even if you have a kernel race works | |
| 15:13:25 | bauzas | that's awesome news | |
| 15:13:37 | bauzas | since the mdev is *really* created, you can ask libvirt to use it | |
| 15:13:46 | bauzas | even if libvirt isn't able to show it per its API | |
| 15:13:53 | bauzas | amazing | |
| 15:14:10 | bauzas | so, one big caveat isn't really a big deal and not requiring a separate change | |
| 15:14:14 | bauzas | just docs | |
| 15:14:22 | bauzas | huzzah | |
| 15:14:24 | efried | stephenfin Thanks, anything helps. TBH, there's not a lot of storage (or network) understanding required for these. The guts are in the pypowervm lib; these patches are really just translating the nova-ese to powervm-ese. | |
| 15:19:14 | mriedem | stephenfin: do you see anything obviously wrong with https://review.openstack.org/#/c/532689/ ? i ran it last night through the experimental queue but results were never posted back on the patch | |
| 15:20:05 | mriedem | i can see it queued up in the experimental queue in zuul right now | |
| 15:22:24 | mriedem | gmann: https://review.openstack.org/#/c/330406/85/nova/api/openstack/compute/migrations.py@119 | |
| 15:22:32 | mriedem | we have 2.0 and 2.1 all over the API, it doesn't seem to make a difference | |
| 15:24:07 | openstackgerrit | sahid proposed openstack/nova master: hardware: only take into account small pages https://review.openstack.org/532168 | |
| 15:26:57 | hongbin | mrodden: mlavalle : want to make a call on the full substring or partial substring debate https://review.openstack.org/#/c/521683/ ? | |
| 15:27:18 | mlavalle | hongbin: I will later today. Thanks for the heads up | |
| 15:29:15 | gibi | efried: opened the patches, but it won't fitt to my today's schedule | |
| 15:29:27 | efried | gibi Thanks in advance | |
| 15:30:01 | mriedem | hongbin: full substring | |
| 15:30:10 | hongbin | mlavalle: mriedem : ack | |
| 15:32:24 | kashyap | Does anyone know what provides this one? I'm on CentOS 7.4, and DevStack fails with: | |
| 15:32:28 | kashyap | $ rpm -qf `which generate-subunit` | |
| 15:32:30 | kashyap | file /usr/bin/generate-subunit is not owned by any package | |
| 15:32:43 | kashyap | (Err the failure message was: "./stack.sh: line 488: generate-subunit: command not found") | |
| 15:33:32 | mriedem | ask in qa | |
| 15:33:49 | kashyap | Yep, checking there | |
| 15:35:43 | kashyap | mriedem: Hehe, I found a bug with your comment (#5) -- https://bugs.launchpad.net/devstack/+bug/1547379 | |
| 15:35:47 | openstack | Launchpad bug 1547379 in devstack "./stack.sh: line 488: generate-subunit: command not found" [Undecided,Invalid] | |
| 15:36:09 | kashyap | It's indeed a mistake in my local.conf! | |
| 15:36:17 | mriedem | exellent | |
| 15:36:20 | mriedem | *excellent event | |
| 15:36:21 | mriedem | *gdi | |
| 15:36:32 | kashyap | (Because I copied my F27 local.conf to the old and sloth-like CentOS) | |
| 15:40:23 | openstackgerrit | Eric Berglund proposed openstack/nova master: PowerVM Driver: vSCSI volume driver https://review.openstack.org/526094 | |
| 15:44:14 | stephenfin | mriedem: I can't see anything obvious, no. I'm guessing it's just zuul funkiness | |
| 15:44:37 | stephenfin | There's been a lot of that this week | |
| 15:48:41 | mriedem | stvnoyes: ildikov: jgriffith: about 10pm last night i thought about this https://etherpad.openstack.org/p/multi-attach-volume-queens - see L48 | |
| 15:48:55 | mriedem | stephenfin: yeah, thanks for looking. have'nt merged any patches in 2 days... | |
| 15:50:38 | ildikov | mriedem: does that apply to live_migrate as well? | |
| 15:51:11 | openstackgerrit | Eric Berglund proposed openstack/nova master: PowerVM Driver: vSCSI volume driver https://review.openstack.org/526094 | |
| 15:51:14 | mriedem | ildikov: no | |
| 15:51:26 | mriedem | ildikov: with live migration, it's the same volume attached on both the source and dest nodes, | |
| 15:51:39 | mriedem | and it's using the same disk config xml which already has the shareable flag set for the source guest xml | |
| 15:51:55 | ildikov | ok, I wasn't sure about the config on the dest | |
| 15:51:58 | mriedem | i think this is purely a swap volume wrinkle | |
| 15:52:21 | mriedem | so i think i need to handle that in between the libvirt patch and https://review.openstack.org/#/c/531244/ | |
| 15:52:28 | mriedem | since https://review.openstack.org/#/c/531244/ bumps the rpc version | |
| 15:52:54 | stvnoyes | mriedem: at this point in Queens, what about explicitly not allowing swap volume if the volume is MA'd? Then we can test swap more exhaustively for rocky... | |
| 15:53:47 | mriedem | stvnoyes: because that means you'd need a new microversion for swap volume with multiattach support in rocky, and that would mean cinder (using nova client) would have to know to send that microversoin if swapping with a MA volume | |
| 15:54:04 | mriedem | stvnoyes: what needs to happen is like a 3 line code change i think | |
| 15:54:07 | ildikov | mriedem: I'm good with your solution | |
| 15:54:11 | stvnoyes | ah ok | |
| 15:54:21 | mriedem | if new_volume['multattach']: new_connection_info['multiattach'] = True | |
| 15:54:39 | ildikov | yep | |
| 15:54:41 | mriedem | thinking about a tempest test scenario to tickle this... | |
| 15:55:07 | mriedem | attach non-multiattach volume A, swap to multiattach volume B, then attach B to a 2nd serer | |
| 15:55:08 | mriedem | *server | |
| 15:55:31 | mriedem | if we didn't fix the swap volume piece, the 2nd attach for B on the 2nd server should be wonky | |