| *** betherly has joined #openstack-nova | 00:02 | |
| *** betherly has quit IRC | 00:07 | |
| *** slaweq has joined #openstack-nova | 00:11 | |
| gmann | nova-next job is failing on Bionic migration. i think for TLS console proxy - https://bugs.launchpad.net/nova/+bug/1819794 | 00:12 |
|---|---|---|
| openstack | Launchpad bug 1819794 in OpenStack Compute (nova) "nova-next job fail on Ubuntu Bionic" [Undecided,New] | 00:12 |
| gmann | Bionic migration will happen tomorrow EOD, either we need to make this job as n-v till fix the issue or fix it before that. | 00:13 |
| gmann | melwitt: efried dansmith bauzas gibi stephenfin ^^ if any one of you know or figure out the issue. | 00:14 |
| melwitt | gmann: thanks. I didn't realize the tls-proxy service was old and to not be used anymore | 00:15 |
| *** IvensZambrano has quit IRC | 00:15 | |
| gmann | melwitt: it is enable and working fine for devstack based job but i think nova-next is the only legacy job (running on xenial) which enbale tls-proxy | 00:15 |
| melwitt | we added it as an "easy" way to get tls certs setup in the env, for testing console proxies with tls enabled. BUT, it's been several months that I noticed the job was no longer running console proxies with tls ("cannot find cert") so we're not gaining anything by enabling it for a long time now | 00:16 |
| gmann | yeah it run on xenial. my patch move the job to run on bionic. | 00:17 |
| gmann | rest all legacy job like nova-live-migration, nova-lvm etc run fine but nova-next fail | 00:19 |
| melwitt | ok, I think we need to stop enabling it on nova-next and leave a TODO that we need to find out what's the right way to configure tls certs in test env today | 00:19 |
| gmann | melwitt: ok. let me give try with that if failure is only because of tls cert | 00:20 |
| melwitt | gmann: to be clear, the only reason we were including tls-proxy service was to get the automatic setup tasks that devstack did when it was enabled | 00:21 |
| gmann | melwitt: i see. | 00:22 |
| melwitt | it used to copy the certs to right directories etc. but several months ago, maybe even a year ago, those setup tasks no longer worked and we found our console proxies running without tls (they couldn't find cert that they used to be able to find) | 00:22 |
| melwitt | probably related to the legacy/deprecated nature of tls-proxy service | 00:23 |
| melwitt | so tls-proxy isn't really related directly to our testing in nova-next, so since it's causing a problem, we can take it out. it wasn't helping us anymore anyway | 00:23 |
| gmann | may be, we need to check if it is working in tempest-full. if so then we can wait to move nova-next on devstack based job | 00:23 |
| *** betherly has joined #openstack-nova | 00:23 | |
| gmann | sure. | 00:24 |
| *** slaweq has quit IRC | 00:24 | |
| melwitt | gmann: nova-next is the only place where we try to test console proxy with tls. because the tls certs etc has to be setup | 00:24 |
| openstackgerrit | Merged openstack/nova master: Allow utime call to fail on qcow2 image base file https://review.openstack.org/641315 | 00:25 |
| openstackgerrit | Merged openstack/nova stable/rocky: Handle missing exception in instance creation code https://review.openstack.org/641401 | 00:25 |
| openstackgerrit | Merged openstack/nova stable/rocky: Fix WeighedHost logging regression https://review.openstack.org/641355 | 00:25 |
| melwitt | we just don't know the right way to do that nowadays. we used to piggyback on tls-proxy to do it | 00:25 |
| gmann | ok | 00:26 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: DNM: Testing nova legacy jobs on bionic https://review.openstack.org/639017 | 00:26 |
| gmann | melwitt: ^^ if it work then, i will mark nova OK in https://etherpad.openstack.org/p/legacy-job-bionic | 00:27 |
| melwitt | gmann: OK, sounds good. thanks | 00:27 |
| gmann | melwitt: thanks for help and updates on tls thing. | 00:28 |
| *** betherly has quit IRC | 00:28 | |
| *** luksky has quit IRC | 00:28 | |
| melwitt | gmann: np, thanks for the heads up about the failure on bionic | 00:28 |
| *** gyee has quit IRC | 00:30 | |
| *** lbragstad has quit IRC | 00:40 | |
| openstackgerrit | Merged openstack/nova stable/rocky: ironic: stop hammering ironic API in power sync loop https://review.openstack.org/640771 | 00:42 |
| openstackgerrit | Merged openstack/nova stable/queens: Add description of custom resource classes https://review.openstack.org/619125 | 00:42 |
| *** wolverineav has quit IRC | 00:42 | |
| *** markvoelker has joined #openstack-nova | 00:43 | |
| *** betherly has joined #openstack-nova | 00:44 | |
| *** tetsuro has joined #openstack-nova | 00:44 | |
| *** wolverineav has joined #openstack-nova | 00:46 | |
| *** betherly has quit IRC | 00:49 | |
| *** wolverineav has quit IRC | 00:51 | |
| *** whoami-rajat has joined #openstack-nova | 00:51 | |
| *** wolverineav has joined #openstack-nova | 00:53 | |
| *** tbachman has quit IRC | 00:57 | |
| *** zhubx has quit IRC | 00:58 | |
| *** zhubx has joined #openstack-nova | 00:58 | |
| *** tkajinam_ has joined #openstack-nova | 00:58 | |
| *** tkajinam has quit IRC | 01:01 | |
| *** liuyulong has joined #openstack-nova | 01:03 | |
| *** betherly has joined #openstack-nova | 01:05 | |
| mnaser | efried: mind joining #openstack-tc -- just have some questions about powervm state | 01:08 |
| *** betherly has quit IRC | 01:10 | |
| *** slaweq has joined #openstack-nova | 01:11 | |
| *** marst has joined #openstack-nova | 01:14 | |
| *** tbachman has joined #openstack-nova | 01:16 | |
| *** markvoelker has quit IRC | 01:21 | |
| *** slaweq has quit IRC | 01:24 | |
| *** agopi has joined #openstack-nova | 01:24 | |
| *** Nel1x has joined #openstack-nova | 01:25 | |
| *** betherly has joined #openstack-nova | 01:26 | |
| *** betherly has quit IRC | 01:31 | |
| *** wolverineav has quit IRC | 01:31 | |
| *** wolverineav has joined #openstack-nova | 01:32 | |
| *** zbr has quit IRC | 01:34 | |
| *** tbachman has quit IRC | 01:34 | |
| *** wolverineav has quit IRC | 01:37 | |
| *** BjoernT has joined #openstack-nova | 01:38 | |
| melwitt | mriedem: I just noticed in #openstack-tc they're talking about jobs moving from xenial => bionic and project job failures. I don't understand most of what's in this email by fungi, but I think might know and be able to help: http://lists.openstack.org/pipermail/openstack-discuss/2019-March/003746.html | 01:39 |
| melwitt | *I think you might know | 01:39 |
| fungi | i am happy to explain too | 01:40 |
| fungi | right now the only failure i see for nova is your functional-py35 job | 01:40 |
| fungi | the problem is that when master branch jobs all switch to running on bionic by default, py35 jobs will still need to run on xenial because bionic doesn't provide python3.5 | 01:41 |
| fungi | we solved this for any of the shared/abstract py35-specific jobs by setting them to use an ubuntu-xenial nodeset unconditionally | 01:41 |
| fungi | but nova's functional-py35 job doesn't have any of those as a parent | 01:42 |
| fungi | the options there are to change the parent on the job, or set an explicit nodeset in it to use ubuntu-xenial | 01:42 |
| fungi | either fix should work and is only needed on master since the current stable branches already get their jobs set to use ubuntu-xenial | 01:43 |
| *** lbragstad has joined #openstack-nova | 01:44 | |
| *** tetsuro has quit IRC | 01:47 | |
| melwitt | fungi: thanks | 01:54 |
| fungi | one possible solution can be found in my edit to nova's .zuul.yaml in https://review.openstack.org/642580 | 01:56 |
| melwitt | thanks | 02:00 |
| fungi | nova gets this somewhat easier than other teams, since proposing changes to different branches of it is how i performed my preliminary testing | 02:01 |
| * melwitt nods | 02:02 | |
| *** betherly has joined #openstack-nova | 02:08 | |
| *** tbachman has joined #openstack-nova | 02:11 | |
| *** betherly has quit IRC | 02:12 | |
| openstackgerrit | Merged openstack/nova stable/queens: Make host_manager use scatter-gather and ignore down cells https://review.openstack.org/637599 | 02:18 |
| openstackgerrit | melanie witt proposed openstack/nova master: Update nova-tox-functional-py35 parent job https://review.openstack.org/642940 | 02:20 |
| *** lbragstad has quit IRC | 02:24 | |
| *** slaweq has joined #openstack-nova | 02:26 | |
| *** tbachman has quit IRC | 02:28 | |
| *** betherly has joined #openstack-nova | 02:29 | |
| *** tbachman has joined #openstack-nova | 02:29 | |
| *** betherly has quit IRC | 02:34 | |
| *** slaweq has quit IRC | 02:39 | |
| *** betherly has joined #openstack-nova | 02:50 | |
| *** betherly has quit IRC | 02:54 | |
| openstackgerrit | Boxiang Zhu proposed openstack/nova master: Trivialfix for help description of images_type https://review.openstack.org/642942 | 03:01 |
| *** psachin has joined #openstack-nova | 03:03 | |
| *** Nel1x has quit IRC | 03:05 | |
| *** betherly has joined #openstack-nova | 03:11 | |
| *** slaweq has joined #openstack-nova | 03:12 | |
| *** betherly has quit IRC | 03:15 | |
| *** slaweq has quit IRC | 03:24 | |
| *** betherly has joined #openstack-nova | 03:31 | |
| *** betherly has quit IRC | 03:36 | |
| *** marst has quit IRC | 03:37 | |
| *** itlinux has quit IRC | 03:41 | |
| *** BjoernT has quit IRC | 04:01 | |
| openstackgerrit | Merged openstack/nova stable/queens: Don't emit warning when ironic properties are zero https://review.openstack.org/608611 | 04:08 |
| *** slaweq has joined #openstack-nova | 04:11 | |
| *** udesale has joined #openstack-nova | 04:17 | |
| *** slaweq has quit IRC | 04:24 | |
| *** zhubx has quit IRC | 04:34 | |
| *** zhubx has joined #openstack-nova | 04:35 | |
| *** zhubx has quit IRC | 04:36 | |
| *** zhubx has joined #openstack-nova | 04:36 | |
| *** zhubx has quit IRC | 04:37 | |
| *** zhubx has joined #openstack-nova | 04:38 | |
| *** betherly has joined #openstack-nova | 04:44 | |
| *** zhubx has quit IRC | 04:44 | |
| *** zhubx has joined #openstack-nova | 04:45 | |
| *** betherly has quit IRC | 04:49 | |
| *** janki has joined #openstack-nova | 05:04 | |
| *** betherly has joined #openstack-nova | 05:05 | |
| *** betherly has quit IRC | 05:09 | |
| *** slaweq has joined #openstack-nova | 05:11 | |
| *** wolverineav has joined #openstack-nova | 05:12 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: DNM: Testing nova legacy jobs on bionic https://review.openstack.org/639017 | 05:14 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: DNM: Testing nova legacy jobs on bionic https://review.openstack.org/639017 | 05:15 |
| *** udesale has quit IRC | 05:16 | |
| *** udesale has joined #openstack-nova | 05:17 | |
| *** jaosorior has joined #openstack-nova | 05:22 | |
| *** slaweq has quit IRC | 05:24 | |
| *** sridharg has joined #openstack-nova | 05:27 | |
| *** igordc has quit IRC | 05:30 | |
| *** liuyulong has quit IRC | 05:41 | |
| *** zhubx has quit IRC | 05:43 | |
| *** zhubx has joined #openstack-nova | 05:44 | |
| *** markvoelker has joined #openstack-nova | 05:47 | |
| *** udesale has quit IRC | 05:51 | |
| *** betherly has joined #openstack-nova | 05:57 | |
| *** betherly has quit IRC | 06:01 | |
| *** slaweq has joined #openstack-nova | 06:11 | |
| *** udesale has joined #openstack-nova | 06:11 | |
| openstackgerrit | Chris Friesen proposed openstack/nova master: Add support for resize and cold migration of emulated TPM files https://review.openstack.org/639934 | 06:18 |
| *** pcaruana has joined #openstack-nova | 06:19 | |
| *** slaweq has quit IRC | 06:24 | |
| *** slaweq has joined #openstack-nova | 06:26 | |
| *** jdillaman has quit IRC | 06:30 | |
| *** jdillaman has joined #openstack-nova | 06:35 | |
| *** slaweq has quit IRC | 06:36 | |
| *** cfriesen has quit IRC | 06:37 | |
| *** masayukig[m]2 is now known as masayukig[m] | 06:45 | |
| *** masayukig[m] is now known as masayuki4 | 06:45 | |
| *** masayuki4 is now known as masayuki6 | 06:48 | |
| *** masayuki6 is now known as masayukig[m] | 06:48 | |
| *** Luzi has joined #openstack-nova | 06:48 | |
| *** masayukig[m] is now known as masayukig | 06:48 | |
| *** wolverineav has quit IRC | 06:53 | |
| *** ivve has joined #openstack-nova | 06:53 | |
| *** slaweq has joined #openstack-nova | 07:12 | |
| *** tonyb is now known as tonyb_gone | 07:20 | |
| *** tonyb_gone is now known as tonyb | 07:21 | |
| *** dpawlik has joined #openstack-nova | 07:23 | |
| *** slaweq has quit IRC | 07:24 | |
| *** slaweq has joined #openstack-nova | 07:35 | |
| *** luksky has joined #openstack-nova | 07:41 | |
| *** wolverineav has joined #openstack-nova | 07:51 | |
| *** dpawlik has quit IRC | 07:54 | |
| *** Dinesh_Bhor has joined #openstack-nova | 07:56 | |
| *** wolverineav has quit IRC | 07:56 | |
| *** Dinesh_Bhor has quit IRC | 07:57 | |
| *** tssurya has joined #openstack-nova | 08:02 | |
| *** lennyb has joined #openstack-nova | 08:06 | |
| *** dpawlik has joined #openstack-nova | 08:06 | |
| *** awalende has joined #openstack-nova | 08:09 | |
| *** ralonsoh has joined #openstack-nova | 08:09 | |
| *** rcernin has quit IRC | 08:09 | |
| *** rpittau|afk is now known as rpittau | 08:09 | |
| *** tesseract has joined #openstack-nova | 08:14 | |
| *** zhubx has quit IRC | 08:17 | |
| *** zhubx has joined #openstack-nova | 08:17 | |
| *** tesseract has quit IRC | 08:18 | |
| *** tesseract has joined #openstack-nova | 08:18 | |
| *** zhubx has quit IRC | 08:21 | |
| *** ccamacho has joined #openstack-nova | 08:23 | |
| *** zhubx has joined #openstack-nova | 08:23 | |
| *** zhubx has quit IRC | 08:23 | |
| *** zhubx has joined #openstack-nova | 08:23 | |
| *** zhubx has quit IRC | 08:25 | |
| *** zhubx has joined #openstack-nova | 08:26 | |
| *** helenafm has joined #openstack-nova | 08:29 | |
| *** xek has joined #openstack-nova | 08:50 | |
| *** jsheeren has joined #openstack-nova | 08:51 | |
| *** ttsiouts has joined #openstack-nova | 08:53 | |
| *** zbr has joined #openstack-nova | 09:06 | |
| *** ttsiouts has quit IRC | 09:06 | |
| Kevin_Zheng | ping gmann | 09:07 |
| *** ttsiouts has joined #openstack-nova | 09:07 | |
| *** ttsiouts has quit IRC | 09:11 | |
| *** dtantsur|afk is now known as dtantsur | 09:14 | |
| *** kukacz has quit IRC | 09:15 | |
| dtantsur | efried: agreed! | 09:15 |
| *** kukacz has joined #openstack-nova | 09:17 | |
| *** panda|rover|off is now known as panda|rover | 09:17 | |
| *** tkajinam__ has joined #openstack-nova | 09:22 | |
| *** FlorianFa has joined #openstack-nova | 09:23 | |
| *** tkajinam_ has quit IRC | 09:25 | |
| *** ttsiouts has joined #openstack-nova | 09:26 | |
| *** cdent has joined #openstack-nova | 09:32 | |
| *** IvensZambrano has joined #openstack-nova | 09:34 | |
| *** derekh has joined #openstack-nova | 09:35 | |
| *** mikal has joined #openstack-nova | 09:40 | |
| *** mikal has quit IRC | 09:45 | |
| openstackgerrit | zhaixiaojun proposed openstack/nova master: Fix bug: non-admin user can not filter instance by instance-uuid https://review.openstack.org/642403 | 09:51 |
| gibi | stephenfin: I'm wondering about the interference between the bandwidth feature and the numa aware vswitches feature | 09:55 |
| gibi | stephenfin: is it a supported case that there are a single physnet with two ovs bridges one bridge connected to a nic on numa0 and the other bridge is connected to a nic on numa1? | 09:56 |
| gibi | stephenfin: if it is supported how nova selects which bridge a port shall be plugged for this physnet? | 09:57 |
| *** jangutter has joined #openstack-nova | 09:59 | |
| openstackgerrit | Merged openstack/nova stable/pike: Lock detach_volume https://review.openstack.org/637536 | 10:16 |
| *** zhubx has quit IRC | 10:19 | |
| *** zhubx has joined #openstack-nova | 10:20 | |
| *** jsheeren has quit IRC | 10:28 | |
| frickler | in case others have - like me - difficulties to decide between melwitt and efried for PTL, watch this ML thread before you vote: http://lists.openstack.org/pipermail/openstack-discuss/2019-March/003765.html | 10:43 |
| openstackgerrit | Neil Jerram proposed openstack/nova stable/queens: Allow utime call to fail on qcow2 image base file https://review.openstack.org/642997 | 10:45 |
| *** priteau has joined #openstack-nova | 10:45 | |
| openstackgerrit | Neil Jerram proposed openstack/nova stable/rocky: Allow utime call to fail on qcow2 image base file https://review.openstack.org/642998 | 10:47 |
| cdent | frickler: you might have more success if you ask some specific questions. Their nomination statements amount to "some statements explaining why they would be the better PTL" | 10:49 |
| cdent | if you want them to differentiate themselves more, you need to ask them about a specific concern you have or present a scenario that you wonder how they would deal with | 10:49 |
| *** tbachman has quit IRC | 10:52 | |
| *** maciejjozefczyk has quit IRC | 10:56 | |
| *** cdent has quit IRC | 11:09 | |
| *** tosky has joined #openstack-nova | 11:17 | |
| *** mriedem has joined #openstack-nova | 11:26 | |
| mriedem | . | 11:26 |
| *** Alon_KS has joined #openstack-nova | 11:29 | |
| *** belmoreira has quit IRC | 11:31 | |
| *** belmoreira has joined #openstack-nova | 11:32 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/rocky: Allow utime call to fail on qcow2 image base file https://review.openstack.org/643011 | 11:33 |
| *** yaawang has quit IRC | 11:33 | |
| *** IvensZambrano has quit IRC | 11:35 | |
| *** yaawang has joined #openstack-nova | 11:42 | |
| openstackgerrit | Adrian Chiris proposed openstack/nova master: Move get_pci_mapping_for_migration to MigrationContext https://review.openstack.org/643023 | 11:50 |
| openstackgerrit | Adrian Chiris proposed openstack/nova master: Allow driver to properly unplug VIFs on destination on confirm resize https://review.openstack.org/643024 | 11:50 |
| *** cdent has joined #openstack-nova | 11:51 | |
| *** ttsiouts has quit IRC | 12:06 | |
| *** ttsiouts has joined #openstack-nova | 12:06 | |
| *** zhubx has quit IRC | 12:07 | |
| openstackgerrit | Merged openstack/nova master: pass endpoint interface to Ironic client https://review.openstack.org/640879 | 12:08 |
| *** ttsiouts has quit IRC | 12:11 | |
| openstackgerrit | Lee Yarwood proposed openstack/nova stable/queens: libvirt: Add workaround to cleanup instance dir when using rbd https://review.openstack.org/628726 | 12:11 |
| *** markvoelker has quit IRC | 12:12 | |
| *** markvoelker has joined #openstack-nova | 12:13 | |
| *** tbachman has joined #openstack-nova | 12:24 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/queens: Allow utime call to fail on qcow2 image base file https://review.openstack.org/633493 | 12:24 |
| *** ttsiouts has joined #openstack-nova | 12:28 | |
| *** wolverineav has joined #openstack-nova | 12:29 | |
| mriedem | tonyb: what do you think about backporting this? https://review.openstack.org/#/c/641351/ | 12:30 |
| mriedem | i asked in the ML as well, but no bites. | 12:30 |
| mriedem | it's an ABI change but does fix a long-standing bug so admins don't shoot themselves | 12:30 |
| mriedem | so i could go either way | 12:30 |
| *** panda|rover is now known as panda|rover|eat | 12:30 | |
| openstackgerrit | chenker proposed openstack/python-novaclient master: Optimize limit option docs Description https://review.openstack.org/643035 | 12:32 |
| *** wolverineav has quit IRC | 12:34 | |
| *** janki has quit IRC | 12:34 | |
| openstackgerrit | chenker proposed openstack/python-novaclient master: Optimize limit option docs string description for servers.py https://review.openstack.org/643035 | 12:35 |
| *** mchlumsky has joined #openstack-nova | 12:38 | |
| *** awalende has quit IRC | 12:43 | |
| *** awalende has joined #openstack-nova | 12:45 | |
| *** luksky has quit IRC | 12:50 | |
| openstackgerrit | chenker proposed openstack/python-novaclient master: Optimize limit option docs string description for servers.py https://review.openstack.org/643035 | 12:57 |
| *** janki has joined #openstack-nova | 12:58 | |
| *** panda|rover|eat is now known as panda|rover | 13:00 | |
| cdent | that's werid mriedem. is something perhaps sending allocations at the wrong microversion? | 13:01 |
| mriedem | well it looks like it's sending a PUT /allocations request with no resources right? | 13:09 |
| mriedem | and it also looks like it's only hitting in queens | 13:09 |
| mriedem | http://logstash.openstack.org/#dashboard/file/logstash.json?query=message%3A%5C%22Failed%20to%20save%20allocation%20for%5C%22%20AND%20message%3A%5C%22DestinationHypervisorTooOld%5C%22%20AND%20tags%3A%5C%22screen-n-super-cond.txt%5C%22&from=7d | 13:10 |
| mriedem | i'm not sure why the hell it's logging DestinationHypervisorTooOld | 13:10 |
| mriedem | Mar 13 01:08:15.816855 ubuntu-xenial-inap-mtl01-0003740112 nova-conductor[25115]: DEBUG nova.scheduler.client.report [None req-a9345308-4e20-4976-ac6d-a1c529d14b16 tempest-TestNetworkAdvancedServerOps-840936673 tempest-TestNetworkAdvancedServerOps-840936673] Sending updated allocation [{'resource_provider': {'uuid': u'13509a14-0b25-4020-b1d6-2c56641a4d82'}, 'resources': {}}] for instance c2fd36f9-0b41-439c-9700-17aa0abe13c2 a | 13:11 |
| mriedem | removing resources for 13509a14-0b25-4020-b1d6-2c56641a4d82. {{(pid=26101) remove_provider_from_instance_allocation /opt/stack/nova/nova/scheduler/client/report.py:1616}} | 13:11 |
| mriedem | wtf, it thinks it's doing a same-host resize or something | 13:12 |
| *** udesale has quit IRC | 13:16 | |
| *** udesale has joined #openstack-nova | 13:17 | |
| openstackgerrit | melanie witt proposed openstack/nova master: DNM: Testing nova legacy jobs on bionic https://review.openstack.org/639017 | 13:19 |
| *** IvensZambrano has joined #openstack-nova | 13:21 | |
| *** luksky has joined #openstack-nova | 13:22 | |
| *** tosky has quit IRC | 13:22 | |
| mriedem | oh heh, qemu is at different versions in this job | 13:26 |
| mriedem | gmann: the tempest-slow job on stable/queens is running with different versions of ubuntu on the compute nodes | 13:26 |
| *** priteau has quit IRC | 13:27 | |
| mriedem | https://bugs.launchpad.net/tempest/+bug/1819910 | 13:28 |
| openstack | Launchpad bug 1819910 in OpenStack Compute (nova) "test_server_connectivity_live_migration intermittently fails with NoValidHost due to PUT /allocations failure" [Undecided,Confirmed] | 13:28 |
| *** marst has joined #openstack-nova | 13:28 | |
| *** lbragstad has joined #openstack-nova | 13:31 | |
| *** IvensZambrano has quit IRC | 13:33 | |
| mriedem | cdent: ok i figured out where that weird same-host allocation goofiness happens | 13:33 |
| cdent | mriedem: sorry, only pay half attention, doing my taxes, which was a surprise gift for today | 13:34 |
| mriedem | that is a fun gift | 13:34 |
| mriedem | anyway, the allocations thing was a red herring | 13:34 |
| mriedem | during live migration we call the scheduler for dest hosts and then do some checks on them in the conductor task, like the hypervisor version comparison between the source and dest, | 13:35 |
| mriedem | if that fails, we call https://github.com/openstack/nova/blob/stable/queens/nova/scheduler/client/report.py#L1527 to remove the allocations created on the dest provider by the scheduler, | 13:35 |
| *** tbachman has quit IRC | 13:36 | |
| mriedem | if we get to https://github.com/openstack/nova/blob/stable/queens/nova/scheduler/client/report.py#L1593 it assumes we're in a same-host resize situation, which is wrong | 13:36 |
| mriedem | we end up sending {} for resource allocatoins and placement pukes on that https://github.com/openstack/nova/blob/stable/queens/nova/scheduler/client/report.py#L1620 | 13:36 |
| mriedem | we log a warning, and b/c we're doing this from an except block, oslo.log is also logging the original exception DestinationHypervisorTooOld - which makes it all very confusing | 13:36 |
| *** eharney has joined #openstack-nova | 13:37 | |
| frickler | mriedem: https://opendev.org/openstack-dev/devstack/src/branch/stable/queens/tools/fixup_stuff.sh#L86 | 13:38 |
| *** tbachman has joined #openstack-nova | 13:38 | |
| frickler | seems ENABLE_VOLUME_MULTIATTACH is only set on controller and not on compute1 | 13:38 |
| mriedem | frickler: gah that would do it | 13:39 |
| *** tbachman has quit IRC | 13:43 | |
| *** zhubx has joined #openstack-nova | 13:43 | |
| *** marst has quit IRC | 13:44 | |
| *** zhubx has quit IRC | 13:48 | |
| *** cdent_ has joined #openstack-nova | 13:54 | |
| *** cdent has quit IRC | 13:54 | |
| *** cdent_ is now known as cdent | 13:54 | |
| *** awaugama has joined #openstack-nova | 13:56 | |
| gmann | mriedem: is it? | 14:00 |
| *** amodi has quit IRC | 14:00 | |
| *** mlavalle has joined #openstack-nova | 14:00 | |
| *** marst has joined #openstack-nova | 14:01 | |
| mriedem | gmann: yeah https://bugs.launchpad.net/tempest/+bug/1819910 | 14:05 |
| openstack | Launchpad bug 1819910 in tempest "test_server_connectivity_live_migration intermittently fails with NoValidHost due to DestinationHypervisorTooOld" [High,Triaged] | 14:05 |
| *** zhubx has joined #openstack-nova | 14:07 | |
| mriedem | gmann: i'll try pushing a fix to tempest | 14:09 |
| *** irclogbot_2 has quit IRC | 14:09 | |
| gmann | mriedem: thanks, is UCA thing will fix it. | 14:11 |
| *** irclogbot_2 has joined #openstack-nova | 14:11 | |
| mriedem | gmann: ok i was working a patch but are you doing it now? | 14:14 |
| gmann | mriedem: no, i am doing legacy job migration as of now. if you are working please push. | 14:15 |
| mriedem | https://review.openstack.org/643052 | 14:20 |
| mriedem | i'll push a nova stable/queens change that depends on ^ to make sure it works | 14:20 |
| *** bnemec has quit IRC | 14:22 | |
| *** eharney has quit IRC | 14:23 | |
| *** zhubx has quit IRC | 14:24 | |
| *** irclogbot_2 has quit IRC | 14:25 | |
| *** irclogbot_2 has joined #openstack-nova | 14:27 | |
| *** BjoernT has joined #openstack-nova | 14:27 | |
| *** bnemec has joined #openstack-nova | 14:27 | |
| *** eharney has joined #openstack-nova | 14:28 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Fix live migration _remove_host_allocations warnings https://review.openstack.org/643055 | 14:28 |
| *** tbachman has joined #openstack-nova | 14:36 | |
| openstackgerrit | Corey Bryant proposed openstack/nova master: add python 3.7 unit test job https://review.openstack.org/610694 | 14:36 |
| *** brtknr has joined #openstack-nova | 14:38 | |
| brtknr | hello! | 14:38 |
| brtknr | i have an instance stuck in BUILD which I do not seem to be able to delete... | 14:39 |
| brtknr | whats the remedy | 14:39 |
| brtknr | it appears when i do `openstack server list` but when i run `openstack server delete UUID`, i get: No server with a name or ID of '1a141848-c91a-450b-b4ad-6d2f81e4125e' exists. | 14:39 |
| mriedem | brtknr: hmm, is there an entry for that instance_uuid in the nova_api.build_requests table? | 14:40 |
| *** tbachman has quit IRC | 14:40 | |
| brtknr | in the database? | 14:41 |
| brtknr | probably... | 14:41 |
| brtknr | it exists as an entity somewhere... | 14:41 |
| brtknr | but its not deletable | 14:41 |
| mriedem | can you actually check the nova_api.build_requests table for a record with that instance_uuid? | 14:41 |
| mriedem | because it should be temporary | 14:41 |
| *** agopi has quit IRC | 14:42 | |
| *** agopi has joined #openstack-nova | 14:42 | |
| brtknr | mriedem: im checking, one sec | 14:43 |
| brtknr | mysql> select * from build_requests; | 14:44 |
| brtknr | Empty set (0.00 sec) | 14:44 |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/queens: DNM: testing tempest-slow fix for subnode config https://review.openstack.org/643059 | 14:46 |
| *** mlavalle has quit IRC | 14:47 | |
| brtknr | mriedem: ive just deleted another server and its gone into the same state... still shows up under server list but cannot be deleted... | 14:47 |
| *** tbachman has joined #openstack-nova | 14:47 | |
| mriedem | brtknr: so it's probably in a cell but the instance_mappings record in the nova_api database for it does not have cell_id set for some reason | 14:48 |
| mriedem | check the nova_api.instance_mappings table for the instance_uuid in question, | 14:48 |
| mriedem | the cell_id column should be set, but i'm guessing it's not in your case | 14:49 |
| mriedem | that would suggest something is failing in your nova-conductor service during scheduling where the instance mapping is not updated to point at the cell in which the instance record actually exists | 14:49 |
| mriedem | which is i'm guessing either cell0 or the 'nova' database (cell1) for you | 14:50 |
| *** BjoernT_ has joined #openstack-nova | 14:56 | |
| openstackgerrit | Merged openstack/nova stable/pike: [pike-only] Fix resize_instance rpcapi call https://review.openstack.org/603439 | 14:57 |
| openstackgerrit | Merged openstack/nova stable/pike: tox: Don't write byte code (maybe) https://review.openstack.org/636919 | 14:57 |
| openstackgerrit | Merged openstack/nova stable/pike: PCI: do not force remove allocated devices https://review.openstack.org/635074 | 14:57 |
| openstackgerrit | Merged openstack/nova stable/pike: Note the aggregate allocation ratio restriction in scheduler docs https://review.openstack.org/623552 | 14:57 |
| dansmith | wow, pike | 14:58 |
| *** BjoernT has quit IRC | 14:58 | |
| *** psachin has quit IRC | 15:07 | |
| *** mriedem has quit IRC | 15:10 | |
| *** mriedem has joined #openstack-nova | 15:11 | |
| *** _alastor_ has quit IRC | 15:13 | |
| *** _alastor_ has joined #openstack-nova | 15:14 | |
| *** lpetrut has joined #openstack-nova | 15:14 | |
| *** Luzi has quit IRC | 15:14 | |
| mriedem | dansmith: if you like those, mayhap you want to hit these other rocky +2'ed changes https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:stable/rocky+label:Code-Review=2 | 15:15 |
| dansmith | no good deed | 15:16 |
| *** BjoernT_ has quit IRC | 15:18 | |
| *** BjoernT has joined #openstack-nova | 15:19 | |
| *** dpawlik has quit IRC | 15:22 | |
| openstackgerrit | Merged openstack/nova stable/rocky: Handle templated cell mappings in nova-status https://review.openstack.org/642611 | 15:23 |
| *** igordc has joined #openstack-nova | 15:23 | |
| *** _alastor_ has quit IRC | 15:24 | |
| *** _alastor_ has joined #openstack-nova | 15:24 | |
| dansmith | mriedem: I got stopped yesterday with that bigger ironic change at the top there | 15:26 |
| mriedem | just plow right through those concerns | 15:27 |
| *** cdent has quit IRC | 15:28 | |
| *** awalende has quit IRC | 15:28 | |
| *** erlon has joined #openstack-nova | 15:29 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Refactor _heal_allocations_for_instance to make place for port healing https://review.openstack.org/637953 | 15:29 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Refactor _heal_allocations_for_instance (2) https://review.openstack.org/637954 | 15:29 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: nova-manage: heal port allocations https://review.openstack.org/637955 | 15:29 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: cache neutron ports in heal allocation https://review.openstack.org/638207 | 15:29 |
| *** zhubx has joined #openstack-nova | 15:32 | |
| *** mikal has joined #openstack-nova | 15:34 | |
| *** irclogbot_2 has quit IRC | 15:36 | |
| *** irclogbot_2 has joined #openstack-nova | 15:38 | |
| *** mikal has quit IRC | 15:39 | |
| openstackgerrit | Merged openstack/nova master: Remove duplicate cleanup in functional tests https://review.openstack.org/636996 | 15:43 |
| *** luksky has quit IRC | 15:47 | |
| mriedem | gibi: i went over the docs for bw providers https://review.openstack.org/#/c/642064/ some comments inline | 15:47 |
| mriedem | let me know if you want to discuss my comments | 15:47 |
| gibi | mriedem: thanks. looking | 15:47 |
| *** IvensZambrano has joined #openstack-nova | 15:49 | |
| *** eharney has quit IRC | 15:49 | |
| *** irclogbot_2 has quit IRC | 15:49 | |
| gmann | melwitt: mriedem nova-next is failing with same reason - http://logs.openstack.org/17/639017/8/check/nova-next/21327c4/logs/screen-n-cpu.txt.gz#_Mar_13_14_31_14_886123 | 15:49 |
| *** irclogbot_2 has joined #openstack-nova | 15:50 | |
| mriedem | gmann: i don't think that's the same, | 15:51 |
| mriedem | and i don't think nova-next is multinode | 15:51 |
| mriedem | http://logs.openstack.org/17/639017/8/check/nova-next/21327c4/logs/libvirt/qemu/instance-00000001.txt.gz | 15:52 |
| gmann | mriedem: not same as tempest-slow. i mean cert issue which was occurring early starting from when migrating it to bionic | 15:52 |
| mriedem | -vnc 127.0.0.1:0,tls,x509verify=/etc/pki/libvirt-vnc: Failed to start VNC server: Cannot load certificate '/etc/pki/libvirt-vnc/server-cert.pem' & key '/etc/pki/libvirt-vnc/server-key.pem': Error while reading file. | 15:52 |
| gmann | yeah this one. | 15:52 |
| gmann | melwitt: was trying it with 1024 bit. | 15:52 |
| *** irclogbot_2 has quit IRC | 15:52 | |
| mriedem | ok i don't know about the vnc / cert stuff | 15:53 |
| gmann | mriedem: sorry for mixing it. it is for https://review.openstack.org/#/c/639017/ | 15:53 |
| *** cfriesen has joined #openstack-nova | 15:54 | |
| cfriesen | I think this should be a fairly uncontroversial bugfix (has +2 from efried): https://review.openstack.org/#/c/641480/ | 15:55 |
| *** irclogbot_2 has joined #openstack-nova | 15:55 | |
| mriedem | dansmith: you'll never believe it by the $11 lady won | 15:58 |
| mriedem | *but the | 15:58 |
| mriedem | i've been carole'ed | 15:58 |
| dansmith | mriedem: now you have to explain to maya why she's not an idiot after all? | 15:58 |
| mriedem | i think she's sleeping | 15:58 |
| dansmith | heh | 15:59 |
| mriedem | too much colonial penn commercials | 16:00 |
| mriedem | and oxygen tanks getting you down | 16:00 |
| dansmith | is the oatmeal guy still double-dipping on insurance commercials? | 16:01 |
| dansmith | oh I guess colonial penn was someone else | 16:02 |
| mriedem | alex trebek | 16:02 |
| mriedem | RIP | 16:02 |
| mriedem | oatmeal was wilford brimley | 16:02 |
| mriedem | from "the thing" fame | 16:02 |
| mriedem | https://www.youtube.com/watch?v=3qSbYEbtfoY | 16:03 |
| dansmith | yeah but he also did Liberty Mutual insurance, another daytime TV staple | 16:03 |
| mriedem | here is now doing heavy computer work | 16:03 |
| mriedem | note the (1) empty bottle of vodka on the desk and (2) .38 special in the drawer | 16:04 |
| dansmith | I have never seen this movie, but the highly accurate 1980s computer simulation makes me think I'm missing out | 16:05 |
| mriedem | i think that's a 38 special anyway, it's been awhile since i've listened to lynard skynard | 16:05 |
| mriedem | the thing is one of my all time favorites | 16:05 |
| dansmith | I don't recall them providing details of .38 special identification | 16:06 |
| mriedem | the 80s version of course | 16:06 |
| mriedem | i guess i'm thinking of saturday night special | 16:06 |
| dansmith | I believe there was something about technique.. perhaps holding on loosely.. but that wasn't skynard | 16:06 |
| mriedem | are those equivalent hand guns? | 16:06 |
| *** walshh___ has quit IRC | 16:07 | |
| mriedem | nobody likes the actual band "38 special" | 16:07 |
| dansmith | I believe that skynard was indeed making a reference to a .38 calibre handgun in that song yes | 16:07 |
| mriedem | I WIN! | 16:07 |
| mriedem | cfriesen: is that maybe due to a newer libvirt changing the type of error returned in that case? | 16:07 |
| *** cdent has joined #openstack-nova | 16:09 | |
| *** erlon has quit IRC | 16:11 | |
| *** mikal has joined #openstack-nova | 16:12 | |
| cfriesen | mriedem: as far as I can tell the libvirt line has been there since like 2012: https://github.com/libvirt/libvirt/blob/master/src/qemu/qemu_monitor_json.c#L392 | 16:12 |
| gibi | mriedem: if I understand you correctly then in the bandwidth doc patch then a possible split would be to only describe the API limitation in the API guide and move the rest of the doc to the admin doc of nova | 16:13 |
| mriedem | cfriesen: yeah i saw the notes in the bug report, +W | 16:13 |
| mriedem | gibi: yeah more or less | 16:14 |
| mriedem | gibi: the api guide tends to be about the non-admin API end user experience, and the stuff about allocation candidates and granular group_policy stuff and flavors and virt drivers should probably be somewhere in the admin docs | 16:14 |
| mriedem | gibi: maybe in here https://docs.openstack.org/nova/latest/admin/adv-config.html | 16:15 |
| gibi | mriedem: make sense, let me restructure the patch | 16:16 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: api-ref: Add description for BDM volume_size https://review.openstack.org/641255 | 16:17 |
| *** mikal has quit IRC | 16:17 | |
| *** b3nt_pin is now known as beagles | 16:23 | |
| *** priteau has joined #openstack-nova | 16:24 | |
| *** gyee has joined #openstack-nova | 16:25 | |
| openstackgerrit | guang-yee proposed openstack/nova stable/rocky: pass endpoint interface to Ironic client https://review.openstack.org/643098 | 16:30 |
| *** wolverineav has joined #openstack-nova | 16:36 | |
| *** wolverineav has quit IRC | 16:39 | |
| *** wolverineav has joined #openstack-nova | 16:39 | |
| *** ivve has quit IRC | 16:40 | |
| *** helenafm has quit IRC | 16:40 | |
| mriedem | dansmith: related to that az discussion we had yesterday, i realize we probably have another regression with resize revert, | 16:41 |
| *** dave-mccowan has joined #openstack-nova | 16:41 | |
| *** FlorianFa has quit IRC | 16:41 | |
| mriedem | when resizing, conductor will change the instance.az to whatever az the dest host is in, | 16:41 |
| mriedem | but on revert we don't set that back to whatever the source host is in, if it changed | 16:41 |
| mriedem | which technically it could if the server was created w/o a specific az | 16:42 |
| *** tbachman has quit IRC | 16:42 | |
| *** udesale has quit IRC | 16:42 | |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Refactor functional base test classes https://review.openstack.org/643101 | 16:43 |
| *** lpetrut has quit IRC | 16:44 | |
| *** eandersson_ is now known as eandersson | 16:54 | |
| *** FlorianFa has joined #openstack-nova | 16:54 | |
| gmann | melwitt: mriedem nova-grenade-live-migration job also failing on bionic due to no ceph hammer for bionic - http://logs.openstack.org/17/639017/8/check/nova-grenade-live-migration/735242b/ara-report/result/8816413c-7053-418f-886b-e238fdea1ec5/ | 16:56 |
| gmann | to proceed on bionic migration we can 1. disable tls-proxy in nova-next 2. make nova-grenade-live-migration n-v till frickler finds out the reason if usage of ceph hammer | 16:57 |
| gmann | if ^^ is ok for you, i can update the patch. | 16:57 |
| gmann | i have planned the bionic migration ~ 2PM PT so we need to make nova change before that to avoid the gate block | 16:58 |
| dansmith | mriedem: yeah.. looking at the az stuff more recently has me thinking we've been neglecting that stuff too much in past years | 16:59 |
| mriedem | gmann: 1 and 2 are mutually exclusive problems correct? do we have bugs reported for those? | 17:04 |
| *** lpetrut has joined #openstack-nova | 17:04 | |
| *** wolverineav has quit IRC | 17:04 | |
| mriedem | gmann: is it not an option to keep these nova-specific jobs as voting and xenial until the issues are sorted out? | 17:04 |
| mriedem | this is kind of a bad time to have to deal with this right before stein RC1 | 17:04 |
| *** wolverineav has joined #openstack-nova | 17:06 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Documentation for bandwidth support https://review.openstack.org/642064 | 17:07 |
| gibi | mriedem: fyi ^^ | 17:08 |
| gibi | mriedem: don't rush to rereview as I'm leaving for today | 17:08 |
| gibi | mriedem: btw, there is also a new version from the neutron doc https://review.openstack.org/640390 | 17:09 |
| gmann | mriedem: https://bugs.launchpad.net/nova/+bug/1819794 https://bugs.launchpad.net/nova/+bug/1819944 | 17:09 |
| openstack | Launchpad bug 1819794 in OpenStack Compute (nova) "nova-next job fail on Ubuntu Bionic" [Undecided,New] | 17:09 |
| openstack | Launchpad bug 1819944 in OpenStack Compute (nova) "nova-grenade-live-migration job failing on ubuntu bionic" [Undecided,New] | 17:09 |
| gmann | mriedem: we should make n-v or fix. keeping them on xenial makes inconsistent that few job running on xenail and few on bionic | 17:09 |
| * gibi will be off Thursday and Friday | 17:09 | |
| mriedem | gmann: i don't have a fix if ceph doesn't have packages on bionic yet, or whatever the issue is there | 17:10 |
| mriedem | we could temporarily disable the tls proxy stuff in the nova-next job on master | 17:10 |
| mriedem | but i'd need melwitt and/or stephenfin to help with that stuff | 17:10 |
| mriedem | and stephenfin is out this week | 17:10 |
| *** wolverineav has quit IRC | 17:11 | |
| *** wolverineav has joined #openstack-nova | 17:11 | |
| mriedem | dansmith: i'll write a functional test for that resize revert az thing because i got thinking about this with my cross-cell stuff so it'll be an issue there as well | 17:11 |
| dansmith | ack | 17:11 |
| mriedem | gibi: ack, but we can't wait forever since we want those docs in the stein GA | 17:11 |
| gmann | yeah melwitt is trying the tsl-proxy with 2048 bit cert but that did not pass | 17:11 |
| mriedem | does she have a patch somewhere? | 17:12 |
| mriedem | is she around? | 17:12 |
| gmann | this one (and deppends-on) - https://review.openstack.org/#/c/639017/8 | 17:12 |
| mriedem | ok https://review.openstack.org/#/c/643045/ | 17:12 |
| mriedem | honestly melwitt probably has a better handle on the ceph CI stuff as well since i'm assuming she's got downstream ceph contacts | 17:15 |
| *** dtantsur is now known as dtantsur|afk | 17:15 | |
| mriedem | or eharney etc | 17:15 |
| * mriedem jumps to cinder | 17:16 | |
| *** rpittau is now known as rpittau|afk | 17:16 | |
| mriedem | coreycb: actually you're probably the person to ask | 17:17 |
| mriedem | coreycb: https://bugs.launchpad.net/nova/+bug/1819944 | 17:18 |
| openstack | Launchpad bug 1819944 in OpenStack Compute (nova) "nova-grenade-live-migration job failing on ubuntu bionic due to missing ceph package" [Undecided,Confirmed] | 17:18 |
| *** igordc has quit IRC | 17:19 | |
| *** gyee has quit IRC | 17:20 | |
| *** gyee has joined #openstack-nova | 17:20 | |
| coreycb | mriedem: hi, we have luminous in our bionic /queensrepos | 17:23 |
| coreycb | bionic/queens repos | 17:23 |
| mriedem | yeah just saw http://download.ceph.com/debian-luminous/dists/bionic/ | 17:25 |
| coreycb | i think they should be installing from ubuntu repos rather than http://download.ceph.com/debian-hammer/dists/bionic/main/binary-amd64/Packages | 17:25 |
| mriedem | i haven't looked into the actual devstack-ceph-plugin job yet to see where it's getting packages | 17:25 |
| *** tssurya has quit IRC | 17:26 | |
| *** dave-mccowan has quit IRC | 17:27 | |
| *** ccamacho has quit IRC | 17:29 | |
| coreycb | mriedem: i commented on the bug | 17:29 |
| mriedem | so apparently this isn't enough https://review.openstack.org/#/c/611594/ | 17:29 |
| coreycb | that should at least be sleeping for 6 seconds :) | 17:30 |
| *** lpetrut has quit IRC | 17:30 | |
| mriedem | umm, so https://review.openstack.org/#/c/611594/4/devstack/lib/ceph@942 | 17:33 |
| *** Swami has joined #openstack-nova | 17:33 | |
| mriedem | frickler: probably need your help here | 17:33 |
| *** mikal has joined #openstack-nova | 17:34 | |
| mriedem | gmann: is devstack-plugin-ceph-tempest-py3 ok with bionic? that's a single-node job | 17:36 |
| mriedem | i wonder if it's something with grenade in the nova-grenade-live-migration job | 17:36 |
| gmann | mriedem: yeah that is working fine | 17:36 |
| mriedem | hmm, ok, so then it's probably something in the nova setup scripts for that live migration job | 17:36 |
| gmann | i am figuring out on 611594 that why it is not picking luminous | 17:36 |
| gmann | seems so. in live_migration/hooks/run_tests.sh | 17:37 |
| gmann | as it is failing while post_test_hook - http://logs.openstack.org/17/639017/8/check/nova-grenade-live-migration/735242b/logs/devstack-gate-post_test_hook.txt.gz | 17:37 |
| *** mikal has quit IRC | 17:38 | |
| mriedem | gmann: how about the nova-live-migration job, is that ok with bionic? | 17:39 |
| gmann | mriedem: yes, that working fine on bionic | 17:39 |
| *** eharney has joined #openstack-nova | 17:41 | |
| mriedem | hmm, that post-test hook ceph script just runs functions from devstack-plugin-ceph, | 17:42 |
| mriedem | which is branchless | 17:42 |
| gmann | nova-live-migration has same issue with that ceph pckg but use old one and process further- http://logs.openstack.org/17/639017/8/check/nova-live-migration/87084aa/logs/devstack-gate-post_test_hook.txt.gz#_2019-03-13_14_34_20_967 | 17:43 |
| gmann | is something else stopping/failing the nova-grenade-live-migration | 17:44 |
| mriedem | i think the nova-live-migration job just isn't failing properly | 17:45 |
| mriedem | and then goes on to run the evacuate stuff, which nova-grendae-live-migration doesn't run | 17:45 |
| gmann | yeah. not sure how d-g handle the failure on post_test_hook | 17:47 |
| mriedem | it must not | 17:47 |
| mriedem | gmann: to workaround bug 1819944 for now, we should probably just comment this out: | 17:49 |
| openstack | bug 1819944 in OpenStack Compute (nova) "nova-grenade-live-migration job failing on ubuntu bionic due to missing ceph package" [Undecided,New] https://launchpad.net/bugs/1819944 | 17:49 |
| mriedem | https://github.com/openstack/nova/blob/master/nova/tests/live_migration/hooks/run_tests.sh#L46-L52 | 17:50 |
| mriedem | gmann: do you want to push a patch for that and i'll +2 | 17:50 |
| mriedem | that way the job continues to run and be voting but the ceph stuff is disabled until someone fixes it | 17:50 |
| gmann | mriedem: ok thats even better, we run some stuff on that jobs | 17:50 |
| *** wolverineav has quit IRC | 17:50 | |
| gmann | i will push | 17:50 |
| mriedem | make sure to push it real good | 17:51 |
| mriedem | https://www.youtube.com/watch?v=vCadcBR95oU | 17:51 |
| dansmith | mriedem: see, I was going to say that I had noted the reference, | 17:51 |
| dansmith | but you keep going all self-serving with the youtube links | 17:51 |
| gmann | :) | 17:52 |
| dansmith | it's like laughing at your own jokes.. just.. so uncouth. | 17:55 |
| *** wolverineav has joined #openstack-nova | 17:55 | |
| *** panda|rover is now known as panda|rover|off | 17:57 | |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Remove IP proxy methods https://review.openstack.org/643115 | 17:58 |
| *** igordc has joined #openstack-nova | 17:58 | |
| *** derekh has quit IRC | 18:00 | |
| *** ttsiouts has quit IRC | 18:01 | |
| *** ttsiouts has joined #openstack-nova | 18:01 | |
| mriedem | dansmith: i did that for gmann's benefit, | 18:02 |
| mriedem | because i don't know if he grew up listening to TLC | 18:02 |
| mriedem | not that i did, but i'm aware, it was hard to avoid | 18:02 |
| dansmith | Do you mean SnP ? | 18:03 |
| dansmith | we're not discussing waterfalls here | 18:03 |
| mriedem | oh yeah heh | 18:03 |
| mriedem | or did i *really* know that but was trying to throw you off my trail? | 18:03 |
| * dansmith scowls | 18:04 | |
| mriedem | twice in one day | 18:04 |
| *** luksky has joined #openstack-nova | 18:04 | |
| dansmith | I could do this all day | 18:04 |
| *** ttsiouts has quit IRC | 18:05 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Skip the ceph based live migration testing https://review.openstack.org/643122 | 18:06 |
| gmann | mriedem: ^^ | 18:06 |
| mriedem | gmann: for bug 1819794 just do what you have here - disable tls https://review.openstack.org/#/c/639017/7 | 18:06 |
| openstack | bug 1819794 in OpenStack Compute (nova) "nova-next job fail on Ubuntu Bionic" [Undecided,New] https://launchpad.net/bugs/1819794 | 18:06 |
| *** tesseract has quit IRC | 18:06 | |
| mriedem | dansmith: want to +W https://review.openstack.org/#/c/643122/ | 18:07 |
| gmann | i have not listen TLC or SNP yet :) | 18:07 |
| gmann | mriedem: yes, doing on 643122 | 18:07 |
| mriedem | gmann: well, something for you to bring home for dinner discussion with the mrs | 18:07 |
| dansmith | gmann: mriedem really likes 80s hip-hop | 18:07 |
| gmann | humm | 18:08 |
| dansmith | mriedem: will look in a bit | 18:08 |
| *** igordc has quit IRC | 18:08 | |
| *** xek_ has joined #openstack-nova | 18:08 | |
| mriedem | this is true https://www.youtube.com/watch?v=O4o8TeqKhgY | 18:08 |
| *** xek has quit IRC | 18:10 | |
| *** igordc has joined #openstack-nova | 18:13 | |
| cfriesen | is there any way to get detailed hypervisor/host resource information in nova? Or is that a placement query now? | 18:14 |
| cfriesen | (for an admin, I mean) | 18:14 |
| *** hongbin has joined #openstack-nova | 18:15 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Disbale the tls-proxy in nova-next job for Bionic https://review.openstack.org/643129 | 18:15 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Disbale the tls-proxy in nova-next job for Bionic https://review.openstack.org/643129 | 18:16 |
| *** amodi_ has joined #openstack-nova | 18:16 | |
| *** amodi_ is now known as amodi | 18:16 | |
| gmann | mriedem: ^^ doing it separately to revert those easily. | 18:16 |
| *** priteau has quit IRC | 18:18 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Migrate legacy jobs to Ubuntu Bionic https://review.openstack.org/639017 | 18:18 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Migrate legacy jobs to Ubuntu Bionic https://review.openstack.org/639017 | 18:18 |
| mriedem | gmann: by "devstack based jobs" do you mean inheriting from the zuulv3 devstack job? | 18:20 |
| gmann | mriedem: yeah | 18:20 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Disable the tls-proxy in nova-next job for Bionic https://review.openstack.org/643129 | 18:21 |
| mriedem | ok +2 on both of those | 18:21 |
| mriedem | efried: melwitt: fyi ^ to unblock bionic job migration | 18:21 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Migrate legacy jobs to Ubuntu Bionic https://review.openstack.org/639017 | 18:21 |
| gmann | mriedem: thanks, testing this on both of those - https://review.openstack.org/#/c/639017/10 | 18:22 |
| mriedem | i'll throw these on the nova meeting agenda for gate bugs | 18:24 |
| gmann | +1. thanks | 18:24 |
| *** pcaruana has quit IRC | 18:29 | |
| *** sridharg has quit IRC | 18:29 | |
| *** wolverineav has quit IRC | 18:30 | |
| *** ralonsoh has quit IRC | 18:31 | |
| *** wolverineav has joined #openstack-nova | 18:31 | |
| *** wolverineav has quit IRC | 18:31 | |
| *** wolverineav has joined #openstack-nova | 18:31 | |
| mriedem | gmann: https://review.openstack.org/#/c/643052/ works | 18:36 |
| mriedem | http://logs.openstack.org/59/643059/1/check/tempest-slow/095dc55/compute1/logs/local_conf.txt | 18:36 |
| mriedem | ENABLE_VOLUME_MULTIATTACH=True shows up in the subnode on the tempest-slow job in stable/queens now | 18:36 |
| mriedem | per https://review.openstack.org/#/c/643059/1 | 18:36 |
| gmann | mriedem: ok. i think it was not failing always. right | 18:39 |
| melwitt | mriedem: ack. I think we'll also need this https://review.openstack.org/642940 | 18:39 |
| *** priteau has joined #openstack-nova | 18:41 | |
| gmann | i saw you added the query for that bug in case. +2 on tempest patch. | 18:42 |
| *** priteau has quit IRC | 18:44 | |
| *** wolverineav has quit IRC | 18:44 | |
| mriedem | melwitt: got it | 18:45 |
| *** gmann is now known as gmann_afk | 18:48 | |
| *** amodi has quit IRC | 18:49 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Trivial: fix typo in reno https://review.openstack.org/643137 | 18:52 |
| melwitt | gmann_afk: thanks for the heads up on the ceph thing. I'll look into how to fix it | 18:54 |
| melwitt | dansmith: can you get this one too? need for bionic https://review.openstack.org/642940 | 18:57 |
| dansmith | ack | 18:57 |
| melwitt | thanks | 18:58 |
| *** jding1_ has joined #openstack-nova | 19:03 | |
| *** mchlumsky_ has joined #openstack-nova | 19:03 | |
| *** jding1__ has quit IRC | 19:04 | |
| *** mchlumsky has quit IRC | 19:04 | |
| *** zzzeek has quit IRC | 19:04 | |
| *** zzzeek has joined #openstack-nova | 19:07 | |
| *** wolverineav has joined #openstack-nova | 19:15 | |
| *** EmilienM has joined #openstack-nova | 19:19 | |
| EmilienM | hello | 19:19 |
| EmilienM | I've been playing with ironic/nova on stein in tripleo environment (don't be scared :D) && I'm having an issue to register the nova compute as an hypervisor | 19:20 |
| EmilienM | http://paste.openstack.org/show/Bz6gl5oFokoah9Ug3VZc/ | 19:21 |
| EmilienM | I'm wondering if this kind of error rings a bell to someone :) | 19:21 |
| EmilienM | (also asked on #openstack-ironic channel) | 19:21 |
| mriedem | jroll: ^ | 19:21 |
| mriedem | that RecursionError does not ring a bell | 19:22 |
| mriedem | but jroll added the virtdrivernotready stuff i think | 19:22 |
| *** wolverineav has quit IRC | 19:22 | |
| tonyb | EmilienM: I hit that and was told to disable SSL in the undercloud | 19:24 |
| dansmith | yeah, interesting | 19:24 |
| EmilienM | tonyb: which I did | 19:24 |
| tonyb | EmilienM: rats, but it's still using SSL | 19:24 |
| EmilienM | tonyb: well actually, not for public endpoint | 19:24 |
| EmilienM | damn | 19:24 |
| EmilienM | tonyb: https://bugzilla.redhat.com/show_bug.cgi?id=1686817 | 19:25 |
| openstack | bugzilla.redhat.com bug 1686817 in openstack-nova "[osp15] nova is broken on the undercloud with ssl" [Urgent,Assigned] - Assigned to mbooth | 19:25 |
| tonyb | EmilienM: I use: openstack-config --set undercloud.conf DEFAULT generate_service_certificate false | 19:25 |
| *** gibi has quit IRC | 19:26 | |
| EmilienM | tonyb: I'll try again without SSL and report back | 19:27 |
| EmilienM | mriedem: not sure if someone from Nova team is working on that bug | 19:27 |
| EmilienM | are you folks testing SSL (I guess yes?) | 19:27 |
| tonyb | EmilienM: I htink it's tested in the gate but I don't know that ironic+nova is | 19:28 |
| tonyb | I'll check later today | 19:28 |
| *** BjoernT has quit IRC | 19:30 | |
| EmilienM | tonyb: thanks! | 19:30 |
| tonyb | EmilienM: I don't think I'll *fix* it but I can poke at least to see if it's a gap in our testing | 19:31 |
| *** mikal has joined #openstack-nova | 19:35 | |
| tonyb | mriedem: I thinkI'm okay backporting that. | 19:35 |
| tonyb | mriedem: I'll reply on list | 19:35 |
| mriedem | tonyb: ok thanks - bauzas is as well so wfm | 19:36 |
| tonyb | mriedem: I need to check if any other data to those calls would cause a 400 status code | 19:36 |
| tonyb | and I guess it will surprise operators that until now didn't nee to empty an aggregate to rename it | 19:37 |
| *** gibi has joined #openstack-nova | 19:38 | |
| mriedem | you can rename an aggregate, | 19:39 |
| mriedem | just no the az | 19:39 |
| mriedem | *not | 19:39 |
| *** mikal has quit IRC | 19:39 | |
| openstackgerrit | Kevin Rasmussen proposed openstack/nova master: created AggregateSubnetAvailabilityFilter https://review.openstack.org/633178 | 19:39 |
| mriedem | if they are not surprised, they were probably dealing with fallout maybe without even knowing it, i.e. rename az, resize a server and now it's back in the old az - or novalidhost | 19:39 |
| mriedem | dansmith: confirmed that revert resize az bug | 19:40 |
| mriedem | got a test, will post with a bug shortly | 19:40 |
| tonyb | Yeah okay. | 19:40 |
| dansmith | mriedem: sweet | 19:41 |
| dansmith | "sweet" | 19:41 |
| *** wolverineav has joined #openstack-nova | 19:45 | |
| mriedem | https://bugs.launchpad.net/nova/+bug/1819963 | 19:45 |
| openstack | Launchpad bug 1819963 in OpenStack Compute (nova) "Reverting a resize does not update the instance.availability_zone value to the source az" [Low,Triaged] - Assigned to Matt Riedemann (mriedem) | 19:45 |
| mriedem | my test shows that the api hides the incorrect instance.availability_zone value | 19:46 |
| mriedem | because of https://github.com/openstack/nova/blob/482f4fed654f384e8fb277c504a14a6407ba2e7b/nova/availability_zones.py#L179-L194 | 19:46 |
| mriedem | makes me wonder wtf we even have instance.availability_zone | 19:46 |
| *** awaugama has quit IRC | 19:46 | |
| dansmith | do we have an az filter for listing instances? | 19:47 |
| dansmith | we use it for metadata at least | 19:47 |
| mriedem | yeah we have an az filter when listing instances | 19:48 |
| dansmith | presumably we use that there? | 19:49 |
| dansmith | obviously much of our model for everything has evolved since that was a thing though | 19:49 |
| *** tbachman has joined #openstack-nova | 19:50 | |
| openstackgerrit | melanie witt proposed openstack/nova master: WIP Use Ceph Jewel release for live migration testing https://review.openstack.org/643150 | 19:51 |
| openstackgerrit | melanie witt proposed openstack/nova master: WIP Use Ceph Jewel release for live migration testing https://review.openstack.org/643150 | 19:52 |
| *** hongbin has quit IRC | 19:53 | |
| *** hongbin has joined #openstack-nova | 19:54 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add functional recreate test for bug 1819963 https://review.openstack.org/643151 | 19:55 |
| openstack | bug 1819963 in OpenStack Compute (nova) "Reverting a resize does not update the instance.availability_zone value to the source az" [Low,Triaged] https://launchpad.net/bugs/1819963 - Assigned to Matt Riedemann (mriedem) | 19:55 |
| mriedem | dansmith: so i built ^ on top of my live migration functional test with azs, which may or may not be a problem | 19:55 |
| openstackgerrit | melanie witt proposed openstack/nova master: WIP Use Ceph Jewel release for live migration testing https://review.openstack.org/643150 | 19:55 |
| mriedem | and yeah i assume the GET /servers?availability_zone=foo filtering uses the instance.availability_zone value in the DB | 19:56 |
| mriedem | otherwise it would be crazy expensive to do that filtering in the API | 19:56 |
| dansmith | yeah | 19:56 |
| dansmith | mriedem: you should have been a lawyer | 19:59 |
| dansmith | "Well, your honor, as you can see I have reproduced the crime in a functional test, exhibit A, which shows that the defendant obviously committed said crime. Once sentenced, we can remove this assertion, exhibit B" | 20:00 |
| mriedem | i'd have to wear adult mens clothing then | 20:01 |
| mriedem | no thanks | 20:01 |
| dansmith | maybe not if you were a lawyer in california, although that would have other downsides | 20:02 |
| dansmith | I hear they have silly magazine limits down there | 20:02 |
| * mriedem alerts sdague | 20:03 | |
| dansmith | heh | 20:05 |
| *** dklyle has quit IRC | 20:09 | |
| mriedem | in all seriousness, being a lawyer in MN would allow me to accept gifts of bars and hotdish from my defendents | 20:11 |
| mriedem | so it might be worth it | 20:11 |
| *** ivve has joined #openstack-nova | 20:15 | |
| *** wolverineav has quit IRC | 20:15 | |
| *** dklyle has joined #openstack-nova | 20:16 | |
| *** wolverineav has joined #openstack-nova | 20:16 | |
| *** amodi has joined #openstack-nova | 20:17 | |
| melwitt | mriedem: what did you mean by ubuntu queens/bionic is using luminous? the version in the uca? | 20:18 |
| mriedem | yeah i think that's what coreycb meant | 20:19 |
| mriedem | when i asked him earlier today | 20:19 |
| melwitt | bc I see luminous for queens xenial in the uca http://ubuntu-cloud.archive.canonical.com/ubuntu/dists/xenial-proposed/queens/main/binary-amd64/Packages | 20:19 |
| melwitt | ok, cool | 20:19 |
| coreycb | melwitt: yes and that's the same version that comes in bionic by default | 20:22 |
| melwitt | coreycb: ah, thank you | 20:22 |
| coreycb | np | 20:23 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Update instance.availability_zone on revertResize https://review.openstack.org/643155 | 20:23 |
| mriedem | dansmith: exhibit B ^ | 20:23 |
| dansmith | that's an upcall no? | 20:23 |
| * dansmith reads the comment | 20:23 | |
| dansmith | oh sorry | 20:24 |
| dansmith | god I'm stupid, sorry | 20:24 |
| *** s10 has joined #openstack-nova | 20:24 | |
| mriedem | i can optimize this | 20:25 |
| mriedem | left a comment | 20:25 |
| dansmith | was about to call your typo | 20:25 |
| mriedem | i know :) | 20:25 |
| mriedem | i could feel it | 20:26 |
| dansmith | so | 20:26 |
| dansmith | talk me through this on your reqspec optimization | 20:26 |
| dansmith | let's say... | 20:26 |
| dansmith | I have an instance with no specific az, | 20:27 |
| *** lpetrut has joined #openstack-nova | 20:27 | |
| dansmith | I move it to another host, and it's mid-resize | 20:27 |
| dansmith | or, waiting for confirm/revert I mean | 20:27 |
| dansmith | the operator could change the az affiliation of that host in the middle right? | 20:27 |
| mriedem | of the source? | 20:27 |
| dansmith | of either | 20:27 |
| mriedem | not the target, | 20:28 |
| dansmith | it wouldn't be smart, but.. | 20:28 |
| mriedem | because of avolkov's recent fix | 20:28 |
| mriedem | but they could rename the az of the source yeah | 20:28 |
| dansmith | barring the source change would be hard, looking up instances that still have residue on hosts | 20:28 |
| dansmith | anyway, | 20:28 |
| dansmith | seems like maybe not worth the optimization, | 20:28 |
| dansmith | if it's still possible to be wrong but creates a much more obscure path to get there | 20:29 |
| mriedem | right his change only deals with the az that contains the instance based on instance.host https://review.openstack.org/#/c/641351/1/nova/objects/instance.py | 20:30 |
| mriedem | which would be the target at the time of the VERIFY_RESIZE status | 20:30 |
| mriedem | i'm fine with not trying to optimize it | 20:30 |
| mriedem | i could leave a comment to that effect as well if it's something we think we'll forget and care about in a year :) | 20:30 |
| *** lpetrut has quit IRC | 20:31 | |
| dansmith | I say just don't optimize it now, regardless of how you spin the comments | 20:31 |
| mriedem | melwitt: you were +2 on this before https://review.openstack.org/#/c/567701/ if you can check it again, i think it was just rebased | 20:31 |
| melwitt | mriedem: sure | 20:32 |
| mriedem | dansmith: ok if you're ok with it as is i can just leave it alone, or fix the typo in the commit message | 20:32 |
| dansmith | oh I'm not fine with the typo | 20:32 |
| dansmith | you must fix that | 20:32 |
| mriedem | ok then | 20:32 |
| dansmith | there, made explicit | 20:32 |
| dansmith | I'll also explain why no opt I gess | 20:33 |
| * dansmith notes his typing skills are on point today | 20:33 | |
| mriedem | actually, | 20:34 |
| mriedem | no, | 20:34 |
| dansmith | mriedem: if I'm using default_availability_zone, I can change the target's affiliation with a restart right? | 20:34 |
| mriedem | if RequestSpec.availability_zone is not None, the server *has* to migrate within the AZ, | 20:34 |
| mriedem | meaning while the server is in VERIFY_RESIZE, it's in the same az on the dest host as the source host, so the admin can't rename the az | 20:34 |
| mriedem | and if RequestSpec.availability_zone is None, then we'd end up updating the instance.availability_zone anyway | 20:35 |
| mriedem | i mean with my optimization | 20:35 |
| dansmith | yeah, but what about my CONF-based attack? | 20:36 |
| dansmith | two hosts, both in default_availability_zone=foo, then I can change that conf to =bar on one and restart it and you'll do the wrong thing | 20:36 |
| dansmith | (and instance.az will be wrong) | 20:36 |
| mriedem | yeah i suppose | 20:38 |
| dansmith | it just doesn't seem like enough optimization to worry about | 20:38 |
| dansmith | to me anyway | 20:39 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Update instance.availability_zone on revertResize https://review.openstack.org/643155 | 20:39 |
| mriedem | where is jaypipes when i want to hear someone say we should just delete availability zones as a concept from nova | 20:39 |
| *** tbachman has quit IRC | 20:40 | |
| dansmith | pfft, he'll say that about anything | 20:40 |
| edleafe | (uses my jaypipes imitation) Availability zones suck! Let's just get rid of them! | 20:40 |
| *** gmann_afk is now known as gmann | 20:41 | |
| mriedem | * suck! Let's just get rid of them! | 20:41 |
| mriedem | volumes | 20:41 |
| mriedem | quotas | 20:41 |
| mriedem | everything | 20:41 |
| mriedem | *root volumes | 20:41 |
| dansmith | instances kinda suck too | 20:41 |
| dansmith | we should nix those | 20:41 |
| *** weshay is now known as other_guys | 20:46 | |
| *** other_guys is now known as other_guy | 20:46 | |
| openstackgerrit | Merged openstack/nova master: Trivialfix for help description of images_type https://review.openstack.org/642942 | 20:48 |
| openstackgerrit | Merged openstack/nova stable/rocky: Fix an error when generating a host ID https://review.openstack.org/641173 | 20:48 |
| openstackgerrit | Merged openstack/nova stable/rocky: Move legacy-grenade-dsvm-neutron-multinode-live-migration in-tree https://review.openstack.org/640191 | 20:48 |
| openstackgerrit | Merged openstack/nova stable/rocky: Fix resource tracker updates during instance evacuation https://review.openstack.org/640982 | 20:48 |
| *** other_guy is now known as weshay | 20:55 | |
| openstackgerrit | Merged openstack/nova master: Add missing libvirt exception during device detach https://review.openstack.org/641480 | 21:00 |
| openstackgerrit | Merged openstack/nova stable/queens: Make service all-cells min version helper use scatter-gather https://review.openstack.org/637594 | 21:03 |
| openstackgerrit | Merged openstack/nova master: api-ref: Add description for BDM volume_size https://review.openstack.org/641255 | 21:03 |
| openstackgerrit | Merged openstack/nova master: docs: Fix nits in remote console guide https://review.openstack.org/641098 | 21:03 |
| *** snevi has joined #openstack-nova | 21:04 | |
| *** IvensZambrano has quit IRC | 21:06 | |
| *** whoami-rajat has quit IRC | 21:11 | |
| *** wolverineav has quit IRC | 21:18 | |
| *** janki has quit IRC | 21:19 | |
| *** ivve has quit IRC | 21:20 | |
| *** wolverineav has joined #openstack-nova | 21:22 | |
| *** wolverineav has quit IRC | 21:24 | |
| *** wolverineav has joined #openstack-nova | 21:24 | |
| *** tbachman has joined #openstack-nova | 21:35 | |
| *** tbachman has quit IRC | 21:37 | |
| *** tbachman has joined #openstack-nova | 21:37 | |
| *** cdent has quit IRC | 21:37 | |
| *** cdent has joined #openstack-nova | 21:42 | |
| *** cdent has quit IRC | 21:42 | |
| *** tosky has joined #openstack-nova | 21:56 | |
| *** rcernin has joined #openstack-nova | 21:57 | |
| *** mikal has joined #openstack-nova | 21:57 | |
| *** s10 has quit IRC | 21:59 | |
| *** mikal has quit IRC | 22:03 | |
| gmann | legacy base jobs to Bionic are merged and nova-next job failing, hold the recheck until this workaround is merged (in gate) - https://review.openstack.org/#/c/643129/3 | 22:11 |
| gmann | sent on ML too. | 22:11 |
| *** marst has quit IRC | 22:14 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/rocky: Update instance.availability_zone during live migration https://review.openstack.org/643173 | 22:17 |
| *** mriedem is now known as mriedem_afk | 22:18 | |
| *** lifeless has quit IRC | 22:27 | |
| *** lifeless has joined #openstack-nova | 22:27 | |
| *** mikal has joined #openstack-nova | 22:34 | |
| fungi | melwitt: oops, i missed that https://review.openstack.org/642940 had a depends-on to the demonstration patch. that commit message needs to be amended to remove the depends-on line | 22:45 |
| fungi | mriedem_afk: dansmith: ^ | 22:45 |
| efried | fungi, melwitt, mriedem_afk, dansmith: I'll take care of it, stand by... | 22:48 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Update nova-tox-functional-py35 parent job https://review.openstack.org/642940 | 22:49 |
| fungi | thanks!!! | 22:49 |
| fungi | and sorry for missing that in review | 22:49 |
| efried | fungi: done | 22:49 |
| efried | I noticed that, but I thought it was intentional | 22:49 |
| efried | like, I thought the other patch was going to be non-DNM'd soon | 22:49 |
| efried | or something | 22:50 |
| efried | I didn't read the backstory closely at all | 22:50 |
| fungi | nope, it was entirely a demonstration. apologies for the confusion | 22:50 |
| fungi | we couldn't actually show the effect of the change without a depends-on chain to an alternate universe base job | 22:50 |
| fungi | at least not until the actual base job change merged | 22:51 |
| efried | ack | 22:52 |
| efried | and with that, I'm outta here o/ | 22:52 |
| *** mikal has quit IRC | 23:01 | |
| *** wolverineav has quit IRC | 23:03 | |
| *** slaweq has quit IRC | 23:05 | |
| *** sapd1 has joined #openstack-nova | 23:09 | |
| *** slaweq has joined #openstack-nova | 23:11 | |
| *** wolverineav has joined #openstack-nova | 23:15 | |
| *** slaweq has quit IRC | 23:15 | |
| *** wolverineav has quit IRC | 23:20 | |
| *** wolverineav has joined #openstack-nova | 23:22 | |
| *** tosky has quit IRC | 23:23 | |
| openstackgerrit | Merged openstack/nova master: Update instance.availability_zone during live migration https://review.openstack.org/567701 | 23:25 |
| openstackgerrit | Merged openstack/nova master: Skip the ceph based live migration testing https://review.openstack.org/643122 | 23:25 |
| *** hongbin has quit IRC | 23:26 | |
| *** dave-mccowan has joined #openstack-nova | 23:27 | |
| *** sapd1 has quit IRC | 23:30 | |
| *** xek_ has quit IRC | 23:33 | |
| *** dave-mccowan has quit IRC | 23:38 | |
| gmann | fungi: efried we have cross-dependency to unblock the gate. 643129 cannot merge which need 642940 and vice versa. | 23:40 |
| fungi | possible we'll need to squash those | 23:41 |
| gmann | fungi: can we get this forcefully in - 642940? or i can make temp patch to unblock either one | 23:41 |
| fungi | they'll be properly tested if we squash the two commits together, but would need a nova-core on hand to approve the resulting revision of whichever change we squash into | 23:43 |
| gmann | ok, let me squash first then i can catch someone alex_xu might be coming online soon | 23:43 |
| fungi | i'm just veeery hesitant to bypass ci for a change where it's not strictly necessary | 23:47 |
| fungi | pretty much every time i do, i break something a job would have caught | 23:47 |
| *** erlon has joined #openstack-nova | 23:48 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Disable the tls-proxy in nova-next & fix nova-tox-functional-py35 parent https://review.openstack.org/643129 | 23:48 |
| gmann | done ^^ | 23:48 |
| gmann | efried: melwitt dansmith alex_xu bauzas stephenfin mriedem_afk ^^ if you are around. | 23:49 |
| openstackgerrit | Merged openstack/nova master: Trivial: fix typo in reno https://review.openstack.org/643137 | 23:50 |
| *** luksky has quit IRC | 23:51 | |
| *** wolverineav has quit IRC | 23:52 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Migrate legacy jobs to Ubuntu Bionic https://review.openstack.org/639017 | 23:54 |
| *** wolverineav has joined #openstack-nova | 23:57 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!