| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-12-18 | |||
| 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 | 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: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: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 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/queens: Add functional recreate test for bug 1852610 https://review.opendev.org/699705 | |
| 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: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 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/queens: Add functional recreate revert resize test for bug 1852610 https://review.opendev.org/699708 | |
| 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: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 | |
| 17:44:30 | openstackgerrit | Merged openstack/nova master: Remove dead code from MigrationTask._execute https://review.opendev.org/697699 | |
| 18:01:36 | mriedem | stephenfin: reviewed https://review.opendev.org/#/c/679502/ | |
| 18:03:34 | stephenfin | thanks | |
| 18:21:22 | openstackgerrit | Luigi Toscano proposed openstack/nova stable/train: Zuul v3: use devstack-plugin-nfs-tempest-full https://review.opendev.org/699753 | |
| 18:28:06 | openstackgerrit | Merged openstack/nova master: Optimization for nova-api _checks_for_create_and_rebuild https://review.opendev.org/698125 | |
| 18:36:43 | KeithMnemonic1 | I know the holidays are coming up, but i appreciate any more reviews to help this get merged before the year ends https://review.opendev.org/691045 | |
| 19:57:59 | mriedem | melwitt: if you're around the bottom 4 nova-net removal patches are pretty easy, starts at https://review.opendev.org/#/c/696510/ | |
| 21:17:36 | gmann | stephenfin: did not get your comment on this - https://review.opendev.org/#/c/657698/14 | |
| 22:07:47 | mriedem | get yer gate bugs hyeah https://bugs.launchpad.net/nova/+bug/1856902 | |
| 22:07:47 | openstack | Launchpad bug 1856902 in OpenStack Compute (nova) "nova.tests.functional.libvirt.test_vpmem.VPMEMTests.test_create_servers_with_vpmem fails intermittently" [Medium,Confirmed] | |
| #openstack-nova - 2019-12-19 | |||
| 00:06:41 | openstackgerrit | Merged openstack/nova master: Revert cross-cell resize from the API https://review.opendev.org/638048 | |
| 02:27:36 | luyao | artom, stephenfin, sean-k-mooney: spec 'support live migration with vpmem' updated. https://review.opendev.org/695863 | |
| 02:28:24 | artom | lyarwood, ack, hopefully I can get to it before my holiday PTO on friday | |
| 02:29:07 | luyao | artom: thanks | |
| 04:38:07 | openstackgerrit | Merged openstack/nova master: Remove now invalid cells v1 comments from conductor code https://review.opendev.org/697670 | |
| 06:07:30 | openstackgerrit | Brin Zhang proposed openstack/nova-specs master: Add action event fault details https://review.opendev.org/699669 | |
| 06:58:54 | openstackgerrit | Merged openstack/nova stable/rocky: Add --dry-run option to heal_allocations CLI https://review.opendev.org/698525 | |
| 06:58:59 | openstackgerrit | Merged openstack/nova stable/rocky: Add --instance option to heal_allocations https://review.opendev.org/698529 | |
| 06:59:05 | openstackgerrit | Merged openstack/nova stable/rocky: Add BFV wrinkle to TestNovaManagePlacementHealAllocations https://review.opendev.org/698531 | |
| 07:15:59 | openstackgerrit | Merged openstack/nova stable/train: Zuul v3: use devstack-plugin-nfs-tempest-full https://review.opendev.org/699753 | |
| 07:41:17 | openstackgerrit | Luigi Toscano proposed openstack/nova stable/stein: Zuul v3: use devstack-plugin-nfs-tempest-full https://review.opendev.org/699924 | |