Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-12-18
05:23:37 openstackgerrit Sundar Nadathur proposed openstack/nova master: ksa auth conf and client for Cyborg access https://review.opendev.org/631242
05:23:38 openstackgerrit Sundar Nadathur proposed openstack/nova master: Define Cyborg ARQ binding notification event. https://review.opendev.org/692707
05:23:38 openstackgerrit Sundar Nadathur proposed openstack/nova master: Add Cyborg device profile groups to request spec. https://review.opendev.org/631243
05:23:39 openstackgerrit Sundar Nadathur proposed openstack/nova master: Pass accelerator requests to each virt driver from compute manager. https://review.opendev.org/698581
05:23:39 openstackgerrit Sundar Nadathur proposed openstack/nova master: Create and bind Cyborg ARQs. https://review.opendev.org/631244
05:23:40 openstackgerrit Sundar Nadathur proposed openstack/nova master: Delete ARQs for an instance when the instance is deleted. https://review.opendev.org/673735
05:23:40 openstackgerrit Sundar Nadathur proposed openstack/nova master: Compose accelerator PCI devices into domain XML in libvirt driver. https://review.opendev.org/631245
05:23:41 openstackgerrit Sundar Nadathur proposed openstack/nova master: Enable start/stop of instances with accelerators. https://review.opendev.org/699553
05:23:41 openstackgerrit Sundar Nadathur proposed openstack/nova master: Add cyborg tempest job. https://review.opendev.org/670999
05:23:42 openstackgerrit Sundar Nadathur proposed openstack/nova master: Enable and use COMPUTE_ACCELERATORS trait. https://review.opendev.org/699554
07:02:11 openstackgerrit Shilpa Devharakar proposed openstack/nova master: Handle new is_volume_backend join column query https://review.opendev.org/694462
07:02:12 openstackgerrit Shilpa Devharakar proposed openstack/nova master: Ignore root_gb if instance is booted from volume https://review.opendev.org/612626
07:02:12 openstackgerrit Shilpa Devharakar proposed openstack/nova master: Instance object changes for the new 'is_volume_backed' expected_attr https://review.opendev.org/694463
07:30:17 openstackgerrit Takashi NATSUME proposed openstack/nova master: Update keypairs in saving an instance object https://review.opendev.org/683043
07:30:50 openstackgerrit ya.wang proposed openstack/nova-specs master: Add "live migration without performance impact" spec. https://review.opendev.org/693655
09:25:01 openstackgerrit Merged openstack/nova stable/train: Cache security group driver https://review.opendev.org/697475
10:40:56 frickler KeithMnemonic: the sorting works fine for me, both with master and rocky.
11:35:22 openstackgerrit Brin Zhang proposed openstack/nova-specs master: Add action event fault details https://review.opendev.org/699669
11:56:20 stephenfin lyarwood: Could you hit https://review.opendev.org/#/c/699522/ please?
11:59:15 lyarwood stephenfin: I'm only nova-stable not stable-maint so I can't +2 that sorry
11:59:26 stephenfin ah, np
12:06:02 stephenfin melwitt: Could you review https://review.opendev.org/#/c/687909/ today when you're around. Would love to close that BP out before the end of the year
12:06:08 stephenfin will ask mriedem too when he's about
13:00:13 openstackgerrit Nathan Kinder proposed openstack/nova master: Allow TLS ciphers/protocols to be configurable for console proxies https://review.opendev.org/679502
13:37:23 openstackgerrit Merged openstack/nova stable/pike: Delete resource providers for all nodes when deleting compute service https://review.opendev.org/666865
13:37:28 openstackgerrit Merged openstack/nova stable/ocata: Exclude build request marker from server listing https://review.opendev.org/677201
14:19:39 KeithMnemonic1 frickler thanks, i found the problem i was seeing at least, when you bulk create 5 instances for example unless you are really lucky the created time is the same
14:30:38 KeithMnemonic1 frickler, it is broken when you use a marker
14:46:30 KeithMnemonic1 frickler ok i figured it out now, sorry for the noise, it is back to a horizon issue
14:56:28 frickler KeithMnemonic1: ok, great, thanks for the update
15:03:20 openstack bug 1829479 in OpenStack Compute (nova) "The allocation table has residual records when instance is evacuated and the source physical node is removed" [Medium,In progress] https://launchpad.net/bugs/1829479 - Assigned to Matt Riedemann (mriedem)
15:03:20 openstackgerrit Matt Riedemann proposed openstack/nova stable/queens: Add functional recreate test for bug 1829479 and bug 1817833 https://review.opendev.org/699698
15:03:21 openstack bug 1817833 in OpenStack Compute (nova) "Check compute_id existence when nova-compute reports info to placement" [Medium,In progress] https://launchpad.net/bugs/1817833 - Assigned to Matt Riedemann (mriedem)
15:35:50 openstack bug 1852610 in OpenStack Compute (nova) rocky "API allows source compute service/node deletion while instances are pending a resize confirm/revert" [Medium,In progress] https://launchpad.net/bugs/1852610 - Assigned to Matt Riedemann (mriedem)
15:35:50 openstackgerrit Matt Riedemann proposed openstack/nova stable/queens: Add functional recreate test for bug 1852610 https://review.opendev.org/699705
15:40:27 mriedem stephenfin: same vendor approval huh https://review.opendev.org/#/c/679502/
15:41:06 stephenfin I didn't even see they worked for Red Hat /o\
15:41:54 stephenfin I can pull it out of the gate, but do we have anyone other than Mel and I that can review console stuff now?
15:43:11 openstack bug 1852610 in OpenStack Compute (nova) rocky "API allows source compute service/node deletion while instances are pending a resize confirm/revert" [Medium,In progress] https://launchpad.net/bugs/1852610 - Assigned to Matt Riedemann (mriedem)
15:43:11 openstackgerrit Matt Riedemann proposed openstack/nova stable/queens: Add functional recreate revert resize test for bug 1852610 https://review.opendev.org/699708
15:43:45 mriedem i reviewed that once before and spotted some stuff - don't have to be a console expert to be a core and review things
15:44:21 openstackgerrit Stephen Finucane proposed openstack/nova master: Allow TLS ciphers/protocols to be configurable for console proxies https://review.opendev.org/679502
15:44:22 mriedem pull it out, +2 it and i'll look at it today
15:44:35 stephenfin fair. done and done
15:44:41 mriedem thanks
15:45:08 stephenfin while you're on console stuff, could you add [1] to the queue too? https://review.opendev.org/#/c/687909/
15:45:30 stephenfin I pinged Mel about it earlier too
15:47:37 mriedem have you pushed stable branch changes to enable xvpvnc? https://review.opendev.org/#/c/695853/
15:48:31 mriedem i see https://review.opendev.org/#/c/698591/1 but it's wrong
15:55:50 lyarwood mriedem: I'll rework that now
15:56:51 mriedem the tempest patch has issues as well
15:57:41 mriedem btw, i thought you guys were all on pto?
15:58:06 lyarwood ha I wish
15:58:10 lyarwood zero PTO left for the year
15:58:18 lyarwood and I think stephenfin is the same
15:58:30 mriedem huh, ok
15:59:04 stephenfin Sorry, had to sanity check how that Tempest check was actually working
15:59:44 lyarwood stephenfin: I'll respin after the standup
15:59:57 stephenfin Nah, I mean the original test
16:00:19 stephenfin I still don't know how it does. XVP console is Xen-only. That shouldn't work unless Tempest was run with a XenServer backend
16:00:50 openstackgerrit Matt Riedemann proposed openstack/nova stable/queens: Block deleting compute services with in-progress migrations https://review.opendev.org/699718
16:03:59 mriedem stephenfin: it's not actually trying to get a console i don't think, just a url
16:04:44 mriedem so if there isn't any special token handling for xvpvnc it doesn't seem like it is treated any different from vnc
16:04:53 mriedem the url would be different, but the test isn't checking to see if the url can be used
16:15:17 lyarwood so is it even worth keeping the tempest test in tree for older releases in that case?
16:15:51 lyarwood stephenfin: ^ ?
16:26:20 stephenfin Sorry, meeting
16:26:34 stephenfin lyarwood, mriedem: So I think that test is junk because nova doesn't seem to check whether it can actually support the requested console type
16:27:12 stephenfin nova just requests _a_ console type from the virt driver https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L6681
16:27:57 stephenfin so if you request the xvp type on the libvirt driver, it'll return a connection_info object with a URL that would never work
16:28:31 mriedem i think i said all of that above
16:28:32 lyarwood yup, do you want to push something to nuke it and I'll drop my change?
16:28:49 stephenfin lyarwood: To answer your question, given that we're deprecating the XVP console type and using it previously would require Xen, which no one's tested in who knows how long, I'd nuke it from Tempest
16:28:54 stephenfin no need for the config option
16:29:16 stephenfin mriedem: and I'm agreeing with you :)
16:29:18 lyarwood stephenfin: ack, I'll do that now then
16:29:19 mriedem i'd support nuking that from tempest
16:29:32 mriedem if someone using xen comes along and wants that later, they could write a tempest plugin
16:29:46 stephenfin If they want it on < Ussuri, that is
16:29:52 stephenfin since it'll be gone from then on, hopefully
16:29:56 mriedem sure
16:30:05 mriedem the plugin is branchless as well
16:30:19 stephenfin doubtful, given Bob Ball (?) said they're using noVNC now too
16:30:21 lyarwood goes back to PS1
16:41:10 lyarwood stephenfin: sorry side tracked, https://review.opendev.org/695853
16:42:00 mriedem so uh, we have like no end in sight for fixing http://status.openstack.org/elastic-recheck/#1844929
16:42:07 mriedem it continues to cripple the gate,
16:42:30 mriedem i think i'm going to hack a retry attempt into the scheduler code that continually hits that in the off chance it's something like eventlet that is screwing things up here and a retry will recover
16:45:02 stephenfin lyarwood: reviewed
16:45:32 stephenfin not sure about that schema change, though maybe I misunderstood what it was doing
16:52:52 lyarwood stephenfin: ack, sorted.
17:00:04 openstackgerrit Lee Yarwood proposed openstack/nova master: Remove 'nova-xvpvncproxy' https://review.opendev.org/687909
17:00:19 lyarwood stephenfin: ^ rebased so it's tested against that change
17:01:10 stephenfin ack, thanks
17:15:30 openstackgerrit Matt Riedemann proposed openstack/nova master: Retry on cell timeout failures getting nodes during scheduling https://review.opendev.org/699735
17:15:37 mriedem slaweq: ^ not sure if this will help with http://status.openstack.org/elastic-recheck/#1844929 but we can see
17:22:50 openstackgerrit Merged openstack/nova master: api-ref: avoid mushy wording around server.image description https://review.opendev.org/698976
17:28:08 openstackgerrit Matt Riedemann proposed openstack/nova master: Use COMPUTE_SAME_HOST_COLD_MIGRATE trait during migrate https://review.opendev.org/695220
17:44:17 openstackgerrit Merged openstack/nova master: Zuul v3: use devstack-plugin-nfs-tempest-full https://review.opendev.org/699516
17:44:23 openstackgerrit Merged openstack/nova master: Add troubleshooting doc about rebuilding the placement db https://review.opendev.org/698517

Earlier   Later