| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-01-22 | |||
| 16:49:15 | zzzeek | sean-k-mooney: the placement service was broken off from nova and uses alembic. this was already a mini-migration of former parts of nova from migrate to alembic | |
| 16:49:19 | sean-k-mooney | so i think we jsut said maybe in the futre but since its not causeing us issue not done "fix" it | |
| 16:50:08 | sean-k-mooney | zzzeek: i remember cdent and i spoke about that brefily in denver actully | |
| 16:50:38 | zzzeek | sean-k-mooney: nova cant be on sqlalhcemy-migrate forever because I can't keep supporting migrate forever from my side | |
| 16:50:49 | sean-k-mooney | zzzeek: i think in stien it just was not high enough on our priority list | |
| 16:51:29 | sean-k-mooney | zzzeek: sure and as a maintainer if that is deperacted we shoudl come up with a transition plan so you dont have too but it would need a spec and need to be a train thing i think | |
| 16:51:33 | zzzeek | sean-k-mooney: im all for it being bottom priority but that made me nervous just now when mriedem_away said it was a "never" thing | |
| 16:51:58 | zzzeek | sean-k-mooney: if you look at what placement did, it's that | |
| 16:52:51 | sean-k-mooney | nova has a lot more legacy stuff to deal with. | |
| 16:54:02 | sean-k-mooney | zzzeek: as i said its something we should talk about for train. | |
| 16:54:31 | sean-k-mooney | zzzeek: is sqlalcamy-migrate offially deprecated/unsupproted | |
| 16:54:42 | zzzeek | sean-k-mooney: if nova got off it, it sure would be | |
| 16:54:57 | zzzeek | sean-k-mooney: some other projects too. keystone cinder not sure if they are still on it | |
| 16:55:16 | zzzeek | sean-k-mooney: it's "supported" because we have no choice | |
| 16:55:32 | zzzeek | sean-k-mooney: also there are no "legacy" cases that alembic doesn't support from a DB perspective | |
| 16:55:35 | sean-k-mooney | right which give nova/openstack 2 options. 1 they fork/take over maintance of it 2 the move to something else. | |
| 16:56:02 | zzzeek | sean-k-mooney: openstack already owns sqlalchemy-migrate | |
| 16:56:20 | sean-k-mooney | oh ok | |
| 16:57:11 | zzzeek | sean-k-mooney: I only know what sqlalchemy-migrate even looks like becuase I joined red hat and got assigned to it :) | |
| 16:57:25 | sean-k-mooney | ah i see. | |
| 16:57:27 | zzzeek | the deep irony that I end up maintaining a project that I replaced w/ somehting else | |
| 16:58:24 | smcginnis | zzzeek: Cinder is still using it. What should we be switching to? | |
| 16:58:42 | zzzeek | smcginnis: alembic migrations have superseded sqlalchemy-migrate for many years | |
| 16:59:48 | smcginnis | zzzeek: Ah. I had raised that as a question ~1 year ago and was shot down as too complex to switch and not really needed. | |
| 16:59:52 | stephenfin | kashyap: Oh, that's why it's WIP. Might stick that in the commit message on the next revision, if you're doing this in a piecemeal fashion | |
| 16:59:53 | sean-k-mooney | zzzeek: its a little technology specific but perhaps it coudl be raise as a cross project goal | |
| 17:00:07 | zzzeek | smcginnis: yeah it's like I am being too nice by not just breaking compatibility with it or something | |
| 17:00:10 | smcginnis | But I can raise it again now with this. We have a midcycle coming up that will be a good venue to discuss it. | |
| 17:00:39 | zzzeek | smcginnis: im looking for reason to go back to denver in may so if you think I'd be useful send me a ping | |
| 17:00:54 | sean-k-mooney | smcginnis: i think that was the same sentime it was discussed with in nova in the past too | |
| 17:01:00 | smcginnis | sean-k-mooney: ++ | |
| 17:01:17 | zzzeek | sean-k-mooney: technically, I was supposed to be getting everyone on alembic years ago, that was part of my job duties. but I get the big wall of "we dont care" | |
| 17:01:29 | smcginnis | zzzeek: Since there are a few projects that have not switched, it might be good to have a Forum or PTG session to go over how to migrate. | |
| 17:02:01 | kashyap | stephenfin: Will add that note. I have a couple of other patches related to it. Meanwhile, I'm trying to figure out the first unit failure of "testtools.matchers._impl.MismatchError" in the test_get_guest_config. | |
| 17:02:14 | zzzeek | smcginnis: yeah I guess you are right | |
| 17:03:05 | smcginnis | zzzeek: We could also try to make it a cycle goal, so having a session to kick that off could really help there. | |
| 17:03:28 | sean-k-mooney | zzzeek: from a redhat point of view we have an lts every 3 releases so makeing the switch in Train/osp16 would be good form a support perspective on your part | |
| 17:03:52 | sean-k-mooney | zzzeek: by the way i know you know that :) | |
| 17:04:01 | zzzeek | smcginnis sean-k-mooney OK....but if I just do a session and then, everyones like OK mike go do it, thats not really going to work for me :) | |
| 17:04:25 | smcginnis | Oh come on, we would love it if you did all the work. :D | |
| 17:04:35 | zzzeek | sean-k-mooney: the problem is this is strict technical debt stuff nobody cares | |
| 17:04:52 | zzzeek | smcginnis: even if I *do* all the work, my reviews get ignored becuase they make everyone nervous | |
| 17:05:03 | zzzeek | smcginnis: so it's not even doing the work | |
| 17:05:07 | zzzeek | its the huge wall of indifference | |
| 17:05:48 | smcginnis | It's a really big wall. | |
| 17:06:19 | zzzeek | smcginnis: this is why, somehow, when they did placement, enough people on "placement" cared to do the switch, was refreshing to see | |
| 17:06:44 | zzzeek | but i think it's important that people who are core to the projet are involved in the work. nobody trusts me | |
| 17:07:43 | smcginnis | Good to start socializing the idea now then. And maybe another good reason to try for a session in Denver to make sure folks are aware and understand that this is technical debt that really does need to be addressed. | |
| 17:08:15 | sean-k-mooney | zzzeek: if you propose patches stephenfin will likely review them and if you talke about why the change is needed and point ot plamcent and other that have done it already people will trust your work | |
| 17:09:00 | stephenfin | ^ this is true | |
| 17:09:24 | stephenfin | Will need a bit of hand holding to avoid "my reviews get ignored becuase they make everyone nervous" though | |
| 17:09:48 | sean-k-mooney | zzzeek: i can try and review it too but im not a core and i usually avoid the db layer | |
| 17:10:19 | stephenfin | Should probably see if mriedem_away (or anyone) had other objections besides it being more tech debt patches to add to our already bonkers review queue too | |
| 17:11:06 | zzzeek | sean-k-mooney: the best way I work ont hese things is a core stakeholder gets the major stuff in place, comes to me with it non-working, and I work with them to get it working | |
| 17:11:17 | zzzeek | sean-k-mooney: that's how we did placement, specificalyl getting the oslo test framework running | |
| 17:12:41 | sean-k-mooney | zzzeek: this would alos be a good topic to start a ml thread fyi | |
| 17:13:50 | sean-k-mooney | it will allow people that are not here right now to express if they had concurers beyond our backlog of review and to express interest async of this irc converstation | |
| 17:13:56 | zzzeek | sean-k-mooney: yep | |
| 17:14:14 | zzzeek | sean-k-mooney: ive stayed away from this for years b.c. of above mentioned things | |
| 17:18:41 | openstackgerrit | Kashyap Chamarthy proposed openstack/nova master: [WIP] libvirt: Bump MIN_{LIBVIRT,QEMU}_VERSION for "Stein" https://review.openstack.org/632507 | |
| 17:18:42 | openstackgerrit | Kashyap Chamarthy proposed openstack/nova master: libvirt: Drop MIN_LIBVIRT_PARALLELS_SET_ADMIN_PASSWD https://review.openstack.org/632514 | |
| 17:20:22 | kashyap | stephenfin: Now read the "Bump" commit message, hope it's clearer. | |
| 17:22:57 | openstackgerrit | Elod Illes proposed openstack/nova stable/queens: Fix os-simple-tenant-usage result order https://review.openstack.org/632516 | |
| 18:09:27 | efried | sean-k-mooney: Thanks for the tip on openstack.org being a separate account for company affiliation purposes. I had missed that one. | |
| 18:10:01 | sean-k-mooney | efried: ya i was wondering why i didnt get election emails since i moved | |
| 18:25:16 | openstackgerrit | Merged openstack/nova master: Cleanup soft (anti)affinity weight multiplier options https://review.openstack.org/628205 | |
| 19:17:46 | openstackgerrit | Merged openstack/os-vif master: Import IP implementation modules outside privsep context https://review.openstack.org/629025 | |
| 19:20:50 | efried | sean-k-mooney: I had a secondary email address in there. | |
| 19:24:51 | openstackgerrit | Merged openstack/os-vif master: make kuryr-kubernetes-tempest-daemon-octavia non voting https://review.openstack.org/632484 | |
| 19:36:15 | openstackgerrit | Merged openstack/os-vif master: Add test to check os_vif.internal.command.ip.exists https://review.openstack.org/632077 | |
| 19:50:58 | jangutter | sean-k-mooney, jaypipes: ^^ the sarlacc pit claims another victim. | |
| 20:08:16 | jaypipes | jangutter: +1 | |
| 20:32:35 | mriedem | gibi: i didn't -1 but i'm not sure if https://review.openstack.org/#/c/570078/ to reject qos ports on attach is handling the case that a network is provided and nova-compute creates a port on that network and the qos policy is associated that way | |
| 20:36:22 | openstackgerrit | Eric Fried proposed openstack/nova master: WIP: ksa auth conf and client for cyborg access https://review.openstack.org/631242 | |
| 20:53:07 | openstackgerrit | Merged openstack/nova master: Fix ovo compatibility code unit tests https://review.openstack.org/631239 | |
| 21:02:53 | openstackgerrit | Paul-Emile Element proposed openstack/nova master: Add emulated TPM support to Nova https://review.openstack.org/631363 | |
| 22:15:01 | openstackgerrit | Eric Fried proposed openstack/nova master: ksa auth conf and client for cyborg access https://review.openstack.org/631242 | |
| 22:24:57 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Don't call begin_detaching when detaching volume from shelved vm https://review.openstack.org/624959 | |
| 22:36:55 | jaypipes | mriedem: any reason this wasn't backported to ocata that I should know about? https://bugs.launchpad.net/nova/+bug/1750623 | |
| 22:36:57 | openstack | Launchpad bug 1750623 in OpenStack Compute (nova) pike "rebuild to same host with different image shouldn't check with placement" [Medium,Fix committed] - Assigned to Lee Yarwood (lyarwood) | |
| 22:37:21 | jaypipes | mriedem: I don't see anything specifically about ocata but wondered if you remembered some difficulty getting that onto ocata I should be aware of... | |
| 22:48:06 | mriedem | jaypipes: ocata is in extended maintenance mode so it was probably just a matter of not doing it unless someone cared | |
| 22:49:20 | mriedem | i'm not opposed to backporting that | |
| 23:12:09 | openstackgerrit | Merged openstack/nova master: Use X-Forwarded-Proto as origin protocol if present https://review.openstack.org/631103 | |
| 23:12:17 | openstackgerrit | Merged openstack/nova master: Extend NeutronFixture to return port with resource request https://review.openstack.org/630719 | |
| 23:31:22 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Add minimum value in max_concurrent_live_migrations https://review.openstack.org/632240 | |
| #openstack-nova - 2019-01-23 | |||
| 00:21:59 | openstackgerrit | Merged openstack/nova master: Convert port to str when validate console port https://review.openstack.org/626853 | |
| 02:09:14 | openstackgerrit | Merged openstack/nova master: Move simple execute call to processutils. https://review.openstack.org/621527 | |
| 03:09:44 | openstackgerrit | yenai proposed openstack/nova master: Disconnect volume when encryption is failed https://review.openstack.org/632598 | |
| 03:46:24 | openstackgerrit | zhufl proposed openstack/nova master: Add missing ws seperator between words https://review.openstack.org/632606 | |
| 05:08:22 | openstackgerrit | Yongli He proposed openstack/nova master: Add server subresouce toplogy API https://review.openstack.org/621476 | |
| 06:01:37 | openstackgerrit | yenai proposed openstack/nova master: Disconnect volume when encryption is failed https://review.openstack.org/632598 | |
| 06:43:11 | openstackgerrit | zhufl proposed openstack/nova master: Add missing ws seperator between words https://review.openstack.org/632606 | |
| 06:44:18 | openstackgerrit | Dakshina Ilangovan proposed openstack/nova-specs master: Nova changes to support Local Resource Management that uses Resource Management Daemon https://review.openstack.org/630817 | |
| 07:40:25 | openstackgerrit | zhufl proposed openstack/nova master: Add missing ws seperator between words https://review.openstack.org/632606 | |
| 08:58:46 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Per-instance serial number https://review.openstack.org/619953 | |
| 09:03:29 | openstackgerrit | yenai proposed openstack/nova master: Ignore VolumeAttachmentNotFound exception in compute.manager https://review.openstack.org/632657 | |
| 09:04:17 | gameon | Does anyone run AMD & Intel in their OpenStack environments together? Is it inadvisable? | |
| 09:12:15 | kashyap | gameon: I'm not an operator, but as long as you ensure to schedule your guests in a "pool of AMD hosts" vs. "pool of Intel hosts", it should be fine? | |
| 09:12:29 | kashyap | To allow live migration compatibility | |