| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-02-13 | |||
| 10:05:06 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Calculate RequestGroup resource provider mapping https://review.openstack.org/616239 | |
| 10:05:07 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Recalculate request group - RP mapping during re-schedule https://review.openstack.org/619529 | |
| 10:05:07 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Pass resource provider mapping to neutronv2 api https://review.openstack.org/616240 | |
| 10:05:08 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Send RP uuid in the port binding https://review.openstack.org/569459 | |
| 10:05:09 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Record requester in the InstancePCIRequest https://review.openstack.org/625310 | |
| 10:05:09 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Remove port allocation during detach https://review.openstack.org/622421 | |
| 10:05:09 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Test boot with more ports with bandwidth request https://review.openstack.org/573317 | |
| 10:05:10 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Ensure that bandwidth and VF are from the same PF https://review.openstack.org/623543 | |
| 10:05:10 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add pf_interface_name tag to passthrough_whitelist https://review.openstack.org/625311 | |
| 10:05:11 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Support server create with ports having resource request https://review.openstack.org/636360 | |
| 10:12:28 | openstackgerrit | Silvan Kaiser proposed openstack/nova master: Added mount fstype based validation of Quobyte mounts https://review.openstack.org/522245 | |
| 10:13:01 | kashyap | gibi: One thing the s390x maintainer in QEMU confirmed is that: 'pty' for "sclplm" device will work. | |
| 10:13:20 | kashyap | But I want to get one more view from DanPB, who wrote the 'virtlogd' daemon. | |
| 10:18:29 | gibi | kashyap: cool | |
| 10:18:50 | gibi | kashyap: you also have a nit in the commit message, about the new min Qemu version which should be 2.8.0 not 2.5.0 | |
| 10:18:56 | gibi | besides these I'm OK with your patch | |
| 10:18:58 | kashyap | Ah, thank you | |
| 10:19:11 | kashyap | gibi: Yeah, getting this will allow me to clean up all the now needless version constants | |
| 10:19:26 | kashyap | And getting that unit tests fixed took 8 days :/ | |
| 10:19:43 | gibi | yeah, I can imagine the pain | |
| 10:20:21 | openstackgerrit | Lee Yarwood proposed openstack/nova master: compute: Reject resize requests when the source host is down https://review.openstack.org/623489 | |
| 10:20:33 | stephenfin | kashyap: Sorry, forgot to start HexChat. I thought mriedem had suggested we do that min version bump early in Train instead? | |
| 10:20:53 | kashyap | stephenfin: He was ambivalent on it. | |
| 10:21:36 | kashyap | stephenfin: IMHO, don't see any compelling blockers (/me hopes this statement doesn't come back to bite him) | |
| 10:22:02 | stephenfin | kashyap: Cool. I'll hold of +2ing personally until I double check, though I'd be happy to get it in of course | |
| 10:22:08 | kashyap | Re-working / rebasing the unit tests again for Train becomes a lot more more painful | |
| 10:22:29 | kashyap | stephenfin: I actually asked him (MattR) last night here, but his nick was _afk | |
| 10:24:20 | kaisers | stephenfin: Thanks, i updated the CL. If you're rolling I'd also be happy to have somebody revisit https://review.openstack.org/#/c/554195/ | |
| 10:24:24 | kashyap | stephenfin: gibi: Release-wise, we're just about to reach M3 (Mar 04-08), and the GA is aroud 12-Apr — we still have decent time for this to soak in | |
| 10:26:10 | openstackgerrit | Silvan Kaiser proposed openstack/nova master: Exec systemd-run without --user flag in Quobyte driver https://review.openstack.org/554195 | |
| 10:31:37 | openstackgerrit | Silvan Kaiser proposed openstack/nova master: Exec systemd-run without --user flag in Quobyte driver https://review.openstack.org/554195 | |
| 10:45:18 | stephenfin | kashyap: Comments left. Just the commit message fix needed, fwict | |
| 10:45:29 | kashyap | stephenfin: Most excellent. | |
| 10:45:58 | kashyap | gibi: stephenfin: Meanwhile, I double-checked with DanPB, and what I'm doing is "sane". Here's the conversation snippet: | |
| 10:46:01 | kashyap | < kashyap> danpb: Is changing the behaviour in this case, by using the console type 'pty' instead of 'file' for s390x's "sclplm" okay? | |
| 10:46:04 | kashyap | < kashyap> (FWIW, checking with Corneila of s390x, she said using 'pty' type for "sclplm" device will work.) | |
| 10:46:07 | kashyap | < kashyap> If you're wondering WTH is this "sclplm", s390x has two types of console devices: | |
| 10:46:10 | kashyap | < danpb> the guest doesnt' care what's done in the backend | |
| 10:46:13 | kashyap | < kashyap> (1) "sclp" -- for interactive consoles | |
| 10:46:15 | kashyap | < kashyap> (2) "sclplm" -- for log files | |
| 10:46:18 | kashyap | < kashyap> danpb: Ah, okay, then. | |
| 10:46:20 | kashyap | < danpb> personally i'd just get rid of that duplication | |
| 10:46:23 | kashyap | < kashyap> Okay, I'll make that in a separate logical change | |
| 10:46:36 | kashyap | stephenfin: Thanks for the quick review! | |
| 10:47:36 | kashyap | stephenfin: Related to your comment, look up "negative code" here :-) -- https://en.wikipedia.org/wiki/Douglas_McIlroy#Views_on_computing | |
| 10:49:39 | stephenfin | kashyap: Very true, but there's no reason removing code can't be done in multiple steps to help grok'ability. See https://review.openstack.org/#/c/636384/ and the preceding patch | |
| 10:50:05 | kashyap | stephenfin: Oh, absolutely: I try to be iron-clad on the "one logical change per commit" | |
| 10:57:29 | kashyap | stephenfin: Nice work there. | |
| 10:59:01 | openstackgerrit | Sylvain Bauza proposed openstack/nova master: libvirt: implement reshaper for vgpu https://review.openstack.org/599208 | |
| 10:59:02 | openstackgerrit | Sylvain Bauza proposed openstack/nova master: WIP: Use the correct mdev allocated from the pGPU https://review.openstack.org/636591 | |
| 10:59:02 | openstackgerrit | Sylvain Bauza proposed openstack/nova master: Add functional test for libvirt vgpu reshape https://review.openstack.org/631559 | |
| 10:59:24 | bauzas | gibi: ^ | |
| 10:59:48 | bauzas | gibi: I thought about the fact that we blindly use mdevs without checking the pGPU | |
| 10:59:55 | bauzas | gibi: and I think it's not a problem | |
| 11:00:07 | bauzas | at least until we support multiple types | |
| 11:00:15 | bauzas | since libvirt will be able to find a specific mdev | |
| 11:00:40 | bauzas | that said, I'll take my old internal hardware and do a lot of checks | |
| 11:01:03 | stephenfin | kaisers: More comments on https://review.openstack.org/#/c/522245/ after a second reading | |
| 11:04:27 | openstackgerrit | Kashyap Chamarthy proposed openstack/nova master: libvirt: Bump MIN_{LIBVIRT,QEMU}_VERSION for "Stein" https://review.openstack.org/632507 | |
| 11:04:46 | kashyap | gibi: stephenfin: When you can, addressed both your remarks ^ | |
| 11:06:43 | kaisers | stephenfin: thnx, checkin' in a min | |
| 11:14:57 | stephenfin | kaisers: And on https://review.openstack.org/#/c/554195/2 | |
| 11:28:06 | kaisers | stephenfin: yup, thnx again! | |
| 13:00:45 | openstackgerrit | Surya Seetharaman proposed openstack/nova master: API microversion 2.69: Handles Down Cells https://review.openstack.org/591657 | |
| 13:00:45 | openstackgerrit | Surya Seetharaman proposed openstack/nova master: Plumbing required in servers ViewBuilder to construct partial results https://review.openstack.org/635146 | |
| 13:00:46 | openstackgerrit | Surya Seetharaman proposed openstack/nova master: API microversion 2.69: Handles Down Cells Documentation https://review.openstack.org/635147 | |
| 13:13:29 | openstackgerrit | Surya Seetharaman proposed openstack/nova master: API microversion 2.69: Handles Down Cells Documentation https://review.openstack.org/635147 | |
| 13:15:22 | openstackgerrit | Jim Rollenhagen proposed openstack/nova master: ironic: partition compute services by conductor group https://review.openstack.org/635006 | |
| 13:18:27 | openstackgerrit | Jim Rollenhagen proposed openstack/nova master: Ensure config regexes match the entire string https://review.openstack.org/636627 | |
| 13:22:24 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Change nova-next tempest test regex https://review.openstack.org/636459 | |
| 13:28:12 | Shilpa | stephenfin: hi, under Taskflow, added https://bugs.launchpad.net/taskflow/+bug/1815738, do let me know is this active channel? Unable to trace PTL for this, but https://pypi.org/project/taskflow/ reports latest release in Jan'19 | |
| 13:28:13 | openstack | Launchpad bug 1815738 in taskflow "Taskflow doesn't support to retrieve tasks progress details in the order of workflow task execution" [Undecided,New] | |
| 13:30:09 | mriedem | Shilpa: i think that would be in the #openstack-oslo channel now | |
| 13:31:11 | mriedem | or i guess #openstack-state-management https://wiki.openstack.org/wiki/TaskFlow#Contact_us.21 | |
| 13:31:26 | mriedem | tssurya: did you enjoy my tales from going down cell down street? | |
| 13:31:53 | Shilpa | mriedem: #openstack-state-management here no one responding, will check #openstack-oslo channel | |
| 13:32:14 | mriedem | tssurya: belmoreira: i'm interested - what do you configure [upgrade_levels]/compute to in nova.conf? do you pin to a release or have it set to auto? | |
| 13:32:23 | bauzas | mriedem: FWIW, I uploaded the new rev for reshape | |
| 13:32:36 | bauzas | mriedem: I found a potential blocker, but eventually it's not a problem | |
| 13:32:55 | mriedem | ok cool | |
| 13:33:58 | belmoreira | mriedem: in the past used to pin it to a release. Now we use auto | |
| 13:36:56 | openstackgerrit | Lee Yarwood proposed openstack/nova master: Avoid redundant initialize_connection on source post live migration https://review.openstack.org/551302 | |
| 13:37:37 | mriedem | belmoreira: ok if you have this change from stable/rocky https://review.openstack.org/#/c/624982/ then you might want to be careful, | |
| 13:37:43 | mriedem | i came across something nasty last night in down cell testing | |
| 13:38:02 | mriedem | https://bugs.launchpad.net/nova/+bug/1815697 | |
| 13:38:03 | openstack | Launchpad bug 1815697 in OpenStack Compute (nova) "[upgrade_levels]compute=auto grinds the API response times when a cell is down" [Medium,Confirmed] | |
| 13:38:52 | mriedem | at this point i don't have any great solutions for that bug | |
| 13:40:00 | sean-k-mooney | mriedem: as a workaournd you set the retry interval to 1 and retries to 1 and set the exact version | |
| 13:40:29 | sean-k-mooney | that could lead to more api request failing in other cases but it should not be inherintly dangourous right | |
| 13:40:32 | mriedem | sean-k-mooney: no, the max_retries and retry_interval was just to quiet the logs | |
| 13:40:39 | sean-k-mooney | oh ok | |
| 13:40:46 | mriedem | the workaround for hanging the api on startpu was setting [upgrade_levels]/compute=rocky | |
| 13:42:11 | tssurya | mriedem: I did go through your tales :D I absolutely appreciate the testing :) I am adding more test cases for the parts you felt are lacking | |
| 13:42:51 | belmoreira | mriedem: we don't have that commit, yet. Thanks for the heads up. | |
| 13:42:53 | tssurya | and I am not so sure I get your worry here: https://review.openstack.org/#/c/591657/39/nova/api/openstack/compute/servers.py@135 | |
| 13:43:38 | Shilpa | mriedem: no one replies here too #openstack-oslo, will wait for more time | |
| 13:45:41 | mriedem | Shilpa: ok well nova doesn't use taskflow. harlowja was the primary maintainer but i'm not sure if he's around anymore. oslo team would be your best bet i'd think, otherwise maybe #openstack-dev | |
| 13:45:58 | mriedem | efried loves taskflow though | |
| 13:46:49 | Shilpa | mriedem: ok, hope he will help me there | |
| 13:47:37 | mriedem | tssurya: my concern is if tooling is passing filter parameters by default, | |