Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-01-22
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
09:13:04 gameon kashyap: Thank you - I understand live migration wouldn't work, but the AMD pricing seems very keen these days!
09:13:15 kashyap E.g. also check out: https://docs.openstack.org/nova/rocky/user/aggregates.html
09:13:49 kashyap gameon: Heh, yeah. Also recall that last year we had a string of hardware CVEs
09:13:56 kashyap So ... that makes things "interesting".
09:14:40 kashyap gameon: FWIW, I gave a talk at the recently concluded Berlin Summit on that: https://www.openstack.org/summit/berlin-2018/summit-schedule/events/22105/effective-virtual-cpu-configuration-in-nova
09:15:29 kashyap The slides are fairly self contained, and should give a sense of what's involved
09:15:53 kashyap (For Operators, particularly this section is relevant: "x86: Updating to patched vCPU models")
09:17:26 gameon Thank you very much, that's really helpful. I was at the Summit but didn't catch your talk :)
09:17:38 kashyap gameon: Also Nova has this thing called "host aggregates", that lets the scheduler make more sensible decisions based on certain properties.
09:18:11 kashyap gameon: Yeah, no worries. There's too much input at these events, and human attention is limited :-)
09:56:49 kashyap mdbooth: Hi, when you get a moment, I'm stumped with this unit test failure (after MIN{LIBVIRT_QEMU}_VERSION bump): http://logs.openstack.org/07/632507/2/check/openstack-tox-py27/6f87de4/testr_results.html.gz
09:57:18 kashyap (See the first one: test_get_guest_config_ppc64_through_image_meta_spice_enabled)
09:57:28 kashyap [...]
09:57:28 kashyap raise mismatch_error
09:57:28 kashyap testtools.matchers._impl.MismatchError: '<nova.virt.libvirt.config.LibvirtConfigMemoryBalloon object at 0x7f10ac940150>' is not an instance of LibvirtConfigGuestVideo
09:57:31 kashyap [...]
09:57:43 openstackgerrit caoxufeng proposed openstack/nova master: oslo.config==6.8.0 requires rfc3986>=1.2.0 and requirements project had made rfc3986==1.2.0 in upper-constraints.txt,so modify it . https://review.openstack.org/632391
09:58:55 mdbooth kashyap: Looking at your patch
10:02:14 mdbooth kashyap: Ok, I can reproduce the failure locally
10:03:07 kashyap Nod
10:03:51 kashyap (There's a typo in the commit message: 1.3.1 --> 3.0.0)
10:04:01 kashyap (But that's besides the point.)
10:05:35 mdbooth kashyap: Ok, to start off with this is just a bad test.
10:05:53 mdbooth self._test_get_guest_config_ppc64(8)
10:06:17 mdbooth 8 is an uncommented bare constant, the best kind of constant
10:07:01 mdbooth It appears to describe the expected location of the video device in the ordered list of devices generates by _get_guest_config
10:07:17 mdbooth I can't imagine why that's a good idea
10:07:39 kashyap Yeah, there's many of those undocumented constants :-(
10:08:12 kashyap There are in total 33 failures, though. Still ploughing through

Earlier   Later