| *** tosky has quit IRC | 00:02 | |
| *** jangutter_ has joined #openstack-nova | 00:16 | |
| *** jangutter has quit IRC | 00:19 | |
| *** dcapone2004 has quit IRC | 00:20 | |
| *** adrian-a has quit IRC | 00:25 | |
| *** macz_ has quit IRC | 00:25 | |
| *** mlavalle has quit IRC | 00:30 | |
| *** brinzhang has joined #openstack-nova | 00:38 | |
| openstackgerrit | Wenping Song proposed openstack/nova master: Remove redundant code in instance delete https://review.opendev.org/c/openstack/nova/+/759577 | 00:45 |
|---|---|---|
| *** aj_mailing has quit IRC | 00:52 | |
| *** benj_- has joined #openstack-nova | 00:53 | |
| *** aj_mailing has joined #openstack-nova | 00:54 | |
| *** benj_ has quit IRC | 00:54 | |
| *** benj_- is now known as benj_ | 00:54 | |
| *** littleboyfury has joined #openstack-nova | 00:54 | |
| *** nweinber has joined #openstack-nova | 01:01 | |
| *** nweinber has quit IRC | 01:01 | |
| *** LinPeiWen has joined #openstack-nova | 01:09 | |
| openstackgerrit | Brin Zhang proposed openstack/nova master: Replaces tenant_id with project_id from Rebuild Server API https://review.opendev.org/c/openstack/nova/+/766380 | 02:17 |
| *** k_mouza has joined #openstack-nova | 02:28 | |
| brinzhang | gmann: hi, I have a question with remove tenant_id | 02:29 |
| brinzhang | gmann: the Servers Security Groups API, just need to change this apihttps://docs.openstack.org/api-ref/compute/?expanded=list-security-groups-by-server-detail#servers-security-groups-servers-os-security-groups | 02:30 |
| brinzhang | but it will be impact the deprecated apis, https://docs.openstack.org/api-ref/compute/?expanded=list-security-groups-by-server-detail#security-groups-os-security-groups-deprecated | 02:31 |
| brinzhang | does it need to be consider to change the deprecated apis? | 02:31 |
| brinzhang | I saw the refresh policies changed the deprecated APIs too | 02:32 |
| *** k_mouza has quit IRC | 02:33 | |
| *** mkrai has joined #openstack-nova | 02:57 | |
| *** priteau has quit IRC | 03:03 | |
| *** hemanth_n has joined #openstack-nova | 03:10 | |
| *** tbachman has quit IRC | 03:18 | |
| *** tbachman_ has joined #openstack-nova | 03:18 | |
| *** openstackgerrit has quit IRC | 03:22 | |
| JamesBenson | sean-k-mooney: originally I had tried `custom` and posted `cpu_models = nehalem,westmere` & `cpu_model_extra_flags = pcid, vmx, pdpe1gb` but then I couldn't boot up VM's. Those three flags are the ones of interest, but aren't in both models. | 03:34 |
| *** hemanth_n has quit IRC | 03:35 | |
| JamesBenson | sean-k-mooney: I thought that the way mentioned above was the right way, but couldn't get it working. Maybe the flags need to be there for all of them? | 03:36 |
| *** psachin has joined #openstack-nova | 03:45 | |
| *** hamalq_ has quit IRC | 03:45 | |
| *** hemanth_n has joined #openstack-nova | 03:50 | |
| *** dave-mccowan has quit IRC | 03:54 | |
| *** gyee has quit IRC | 04:00 | |
| *** littleboyfury has quit IRC | 04:05 | |
| *** ratailor has joined #openstack-nova | 04:54 | |
| *** sapd1 has joined #openstack-nova | 04:55 | |
| *** bauzas has quit IRC | 05:02 | |
| *** zzzeek has quit IRC | 05:06 | |
| *** zzzeek has joined #openstack-nova | 05:08 | |
| *** bauzas has joined #openstack-nova | 05:09 | |
| *** dasp has quit IRC | 05:18 | |
| *** evrardjp has quit IRC | 05:33 | |
| *** evrardjp has joined #openstack-nova | 05:33 | |
| *** JamesBenson has quit IRC | 05:33 | |
| *** dasp has joined #openstack-nova | 05:34 | |
| *** JamesBenson has joined #openstack-nova | 05:35 | |
| *** JamesBenson has quit IRC | 05:41 | |
| *** rpittau|afk has quit IRC | 06:11 | |
| *** mnaser has quit IRC | 06:11 | |
| *** mnaser has joined #openstack-nova | 06:11 | |
| *** rpittau|afk has joined #openstack-nova | 06:11 | |
| *** sapd1 has quit IRC | 06:15 | |
| *** tbachman_ has quit IRC | 06:16 | |
| *** zzzeek has quit IRC | 06:17 | |
| *** JamesBenson has joined #openstack-nova | 06:17 | |
| *** zzzeek has joined #openstack-nova | 06:18 | |
| *** tbachman has joined #openstack-nova | 06:19 | |
| *** JamesBenson has quit IRC | 06:22 | |
| *** tbachman has quit IRC | 06:35 | |
| *** tbachman has joined #openstack-nova | 06:36 | |
| *** links has joined #openstack-nova | 06:40 | |
| *** evrardjp has quit IRC | 06:49 | |
| *** evrardjp_ has joined #openstack-nova | 06:49 | |
| *** lpetrut has joined #openstack-nova | 07:00 | |
| *** rcernin has quit IRC | 07:10 | |
| *** slaweq has joined #openstack-nova | 07:18 | |
| *** ralonsoh has joined #openstack-nova | 07:25 | |
| *** links has quit IRC | 07:28 | |
| *** rcernin has joined #openstack-nova | 07:29 | |
| *** dklyle has quit IRC | 07:37 | |
| *** macz_ has joined #openstack-nova | 07:51 | |
| *** openstackgerrit has joined #openstack-nova | 07:53 | |
| openstackgerrit | Wenping Song proposed openstack/nova master: Nova supports password encrypted VNC https://review.opendev.org/c/openstack/nova/+/622336 | 07:53 |
| *** macz_ has quit IRC | 07:55 | |
| *** hamalq has joined #openstack-nova | 08:06 | |
| *** mkrai has quit IRC | 08:07 | |
| *** rcernin has quit IRC | 08:07 | |
| *** andrewbonney has joined #openstack-nova | 08:11 | |
| openstackgerrit | Wenping Song proposed openstack/nova master: Nova supports password encrypted VNC https://review.opendev.org/c/openstack/nova/+/622336 | 08:12 |
| *** rpittau|afk is now known as rpittau | 08:17 | |
| *** teoobo_ has joined #openstack-nova | 08:18 | |
| *** JamesBenson has joined #openstack-nova | 08:18 | |
| openstackgerrit | Wenping Song proposed openstack/nova master: Nova supports password encrypted VNC https://review.opendev.org/c/openstack/nova/+/622336 | 08:19 |
| *** tesseract has joined #openstack-nova | 08:22 | |
| *** JamesBenson has quit IRC | 08:23 | |
| *** mkrai has joined #openstack-nova | 08:29 | |
| *** links has joined #openstack-nova | 08:34 | |
| *** larainema has joined #openstack-nova | 08:35 | |
| *** zzzeek has quit IRC | 08:36 | |
| bauzas | good morning Nova | 08:38 |
| *** zzzeek has joined #openstack-nova | 08:39 | |
| openstackgerrit | Brin Zhang proposed openstack/nova master: Replaces tenant_id with project_id from Rebuild Server API https://review.opendev.org/c/openstack/nova/+/766380 | 08:44 |
| *** tosky has joined #openstack-nova | 08:47 | |
| *** spatel has joined #openstack-nova | 08:52 | |
| *** spatel has quit IRC | 08:56 | |
| *** rcernin has joined #openstack-nova | 08:56 | |
| *** derekh has joined #openstack-nova | 08:58 | |
| *** rcernin has quit IRC | 09:13 | |
| *** links has quit IRC | 09:16 | |
| *** martinkennelly has joined #openstack-nova | 09:16 | |
| *** k_mouza has joined #openstack-nova | 09:17 | |
| *** aj_mailing has quit IRC | 09:31 | |
| *** zzzeek has quit IRC | 09:32 | |
| *** zzzeek has joined #openstack-nova | 09:33 | |
| gibi | melwitt: hi | 09:34 |
| melwitt | gibi: hi | 09:35 |
| gibi | melwitt: I saw your comment about the chicken-egg | 09:35 |
| melwitt | I figured :) I'm tired so I might be wrong but afaict it is chicken and egg | 09:36 |
| gibi | melwitt: I guess we need to pull the fixes top of each other and then ask infra to promote | 09:36 |
| gibi | melwitt: I will add a unit test coverage for your qos fix during my day (I assume you will go to sleep soon) | 09:36 |
| melwitt | yeah, either that or disable the min bw tempest tests to let the lower-constraints change through and then re-enable it | 09:37 |
| lyarwood | ah right I was going to ask about the bw failures in nova-next | 09:37 |
| brinzhang | gibi, dansmith: could you please review cyborg shelve/unshelve patch again https://review.opendev.org/c/openstack/nova/+/729563 ? thanks | 09:37 |
| lyarwood | brinzhang: dansmith is out until 2021 FYI | 09:37 |
| melwitt | gibi: cool yes that would be appreciated, I am going to sleep soon. also I didn't find anything as to the cause for the failure on the master branch with the port binding failure too so that one needs eyes on it too | 09:38 |
| brinzhang | lyarwood: ack, a shocking news, dansmith will work to k8s? | 09:39 |
| melwitt | gibi: for my patch I had been thinking to add resource requests to one of the nova/tests/functional/cross_cell_migrate.py but it would take me a bit to learn how to incorporate that | 09:39 |
| lyarwood | brinzhang: umm not that I'm aware of, he's just out on PTO until the new year. | 09:39 |
| *** xek_ has joined #openstack-nova | 09:40 | |
| melwitt | *test_cross_cell_migrate.py | 09:40 |
| melwitt | maybe that would be overkill anyway, I dunno | 09:41 |
| gibi | melwitt: I think this upgrade check that breaks in stable is removed from master | 09:41 |
| melwitt | gibi: it is but it's failing in a different way for the master branch | 09:41 |
| brinzhang | lyarwood: yeah, I like that job too^, thanks | 09:41 |
| gibi | melwitt: ohh, then I will check that too | 09:41 |
| melwitt | gibi: here's my ML post with links to more info http://lists.openstack.org/pipermail/openstack-discuss/2020-December/019319.html | 09:42 |
| melwitt | lajoskatona is looking at the neutron side but I doubt it's neutron. it's just that when we request the port binding it results in neutron getting UnknownResourceProvider, I have no idea how that can happen | 09:43 |
| gibi | melwitt: sorry I was pulled in a downstream call | 09:53 |
| gibi | I will pick up the threads after it | 09:54 |
| melwitt | thanks gibi++ | 09:54 |
| melwitt | o/ | 09:54 |
| gibi | melwitt: have a nice evening | 09:54 |
| *** ralonsoh_ has joined #openstack-nova | 10:00 | |
| stephenfin | sean-k-mooney: The issue with https://docs.python.org/3/library/unittest.html#unittest.TestCase.assertRaises is nothing to do with mock the lib | 10:00 |
| stephenfin | sean-k-mooney: that happens because of testtools | 10:00 |
| stephenfin | sean-k-mooney: https://github.com/testing-cabal/testtools/issues/113 https://github.com/testing-cabal/testtools/issues/275 | 10:01 |
| *** ralonsoh has quit IRC | 10:01 | |
| gibi | lyarwood: regarding qos stable failures: other than the the packaging 20.4 bump and the qos break, is there any other know break on stable/victoria at the moment? | 10:02 |
| lyarwood | gibi: not on stable/victoria that I'm aware of | 10:04 |
| lyarwood | gibi: <= stable/train has other issues | 10:04 |
| brinzhang | hi all, the nova-multi-cell task always failed, is there a bug tracing? | 10:07 |
| *** hamalq has quit IRC | 10:07 | |
| brinzhang | https://8881082f52814b33cb4d-7716d10619577c42ab1386a760b8bb5c.ssl.cf2.rackcdn.com/764292/8/check/nova-multi-cell/9429107/testr_results.html | 10:07 |
| gibi | lyarwood: ack, I saw the bandit affecting train and older | 10:10 |
| gibi | brinzhang: yes, the nova-cell-multi is broken right now | 10:10 |
| gibi | brinzhang: I have to look at it after my current call | 10:11 |
| gibi | brinzhang: here is melwitt's post about it from the mL http://lists.openstack.org/pipermail/openstack-discuss/2020-December/019319.html | 10:11 |
| brinzhang | gibi: ack | 10:11 |
| brinzhang | gibi: I saw your talking about above, sorry to mention again | 10:12 |
| *** zzzeek has quit IRC | 10:13 | |
| lyarwood | gibi: ack kk, we could switch both jobs to NV and then land things? | 10:13 |
| *** zzzeek has joined #openstack-nova | 10:14 | |
| gibi | lyarwood: both job is nova specific? then yes. NV->fix1->fix2->revert NV could be a solution | 10:16 |
| gibi | lyarwood: sorry I have to be off for an hour or so | 10:17 |
| lyarwood | gibi: np | 10:19 |
| lyarwood | gibi: pushing now, I'll ask elod and others to review | 10:19 |
| *** JamesBenson has joined #openstack-nova | 10:19 | |
| lyarwood | git-review really takes a while now | 10:19 |
| openstackgerrit | Lee Yarwood proposed openstack/nova stable/victoria: lower-constraints: Bump packaging to 20.4 https://review.opendev.org/c/openstack/nova/+/765749 | 10:20 |
| openstackgerrit | Lee Yarwood proposed openstack/nova stable/victoria: zuul: Temporarily make nova-multi-cell non-voting https://review.opendev.org/c/openstack/nova/+/766440 | 10:20 |
| lyarwood | oh lyarwood-- | 10:20 |
| *** JamesBenson has quit IRC | 10:23 | |
| openstackgerrit | MIKI Nobuhiro proposed openstack/nova master: libvirt: add IP address to libvirt guest metadata https://review.opendev.org/c/openstack/nova/+/750552 | 10:24 |
| openstackgerrit | Lee Yarwood proposed openstack/nova stable/victoria: [stable-only] Temporarily make nova-multi-cell non-voting https://review.opendev.org/c/openstack/nova/+/766440 | 10:25 |
| openstackgerrit | Lee Yarwood proposed openstack/nova stable/victoria: lower-constraints: Bump packaging to 20.4 https://review.opendev.org/c/openstack/nova/+/765749 | 10:25 |
| *** macz_ has joined #openstack-nova | 10:33 | |
| *** ralonsoh_ is now known as ralonsoh | 10:33 | |
| *** bbowen has quit IRC | 10:34 | |
| lyarwood | elod / bauzas ^ would you mind taking a look at the above stable changes | 10:34 |
| *** bbowen has joined #openstack-nova | 10:35 | |
| *** songwenping_ has quit IRC | 10:35 | |
| elod | lyarwood: sure, looking | 10:35 |
| *** songwenping_ has joined #openstack-nova | 10:35 | |
| melwitt | gibi: one last update before I really go, I can see on zuul status that my stable/victoria change minbw tests have failed still but now for a different reason. the logs are not yet available but I suspect that now that the cell is correctly targeted, it's now hitting the same bug that exists on the master branch. just fyi | 10:37 |
| lyarwood | can someone kick melwitt so she can get some sleep? | 10:38 |
| *** macz_ has quit IRC | 10:38 | |
| lyarwood | :) | 10:38 |
| melwitt | D: | 10:38 |
| lyarwood | only joking | 10:38 |
| lyarwood | but seriously get some sleep :D | 10:38 |
| melwitt | I know, that was my OMG I CANT BELIEVE YOU SAID THAT look | 10:38 |
| melwitt | :) | 10:38 |
| * melwitt jokes without skill | 10:39 | |
| *** zzzeek has quit IRC | 10:39 | |
| melwitt | ok, gnight for reals | 10:39 |
| lyarwood | hehe twas a good one :) | 10:39 |
| lyarwood | \o | 10:39 |
| *** ociuhandu has joined #openstack-nova | 10:40 | |
| *** zzzeek has joined #openstack-nova | 10:41 | |
| *** mkrai has quit IRC | 10:42 | |
| *** k_mouza has quit IRC | 10:46 | |
| elod | lyarwood: will the two patch work separately? Don't they need to be squashed to fix every issue? (or will the .zuul.yaml change only run limited tests that could pass? O.o) | 10:48 |
| *** k_mouza has joined #openstack-nova | 10:48 | |
| * lyarwood gulps coffee | 10:49 | |
| lyarwood | elod: yup I'm an idiot | 10:49 |
| openstackgerrit | Lee Yarwood proposed openstack/nova stable/victoria: lower-constraints: Bump packaging to 20.4 https://review.opendev.org/c/openstack/nova/+/765749 | 10:51 |
| elod | lyarwood: I'm not sure, just asking | 10:51 |
| lyarwood | elod: no no you are right, lower-constraints would fail in the standalone change | 10:52 |
| lyarwood | elod: so I've squashed it into the fix for that job | 10:52 |
| lyarwood | elod: and once we have a fix for the multi-cell job we should enable it again in that change | 10:53 |
| lyarwood | elod: the only other way of fixing this would be to squash both fixes into one change | 10:53 |
| lyarwood | elod: that I'm not a fan of tbh | 10:54 |
| *** ociuhandu has quit IRC | 10:54 | |
| elod | lyarwood: ok, good, thx! | 10:55 |
| *** rcernin has joined #openstack-nova | 10:57 | |
| *** adrian-a has joined #openstack-nova | 11:08 | |
| *** hamalq has joined #openstack-nova | 11:09 | |
| *** links has joined #openstack-nova | 11:12 | |
| *** hamalq has quit IRC | 11:14 | |
| *** jangutter has joined #openstack-nova | 11:28 | |
| *** jangutter_ has quit IRC | 11:32 | |
| *** dtantsur|afk is now known as dtantsur | 11:33 | |
| openstackgerrit | Lucian Petrut proposed openstack/nova master: Fix unshelved image ref https://review.opendev.org/c/openstack/nova/+/765924 | 11:40 |
| *** larainema has quit IRC | 11:44 | |
| *** tbachman has quit IRC | 11:55 | |
| * gibi is back | 11:57 | |
| openstackgerrit | Lee Yarwood proposed openstack/nova-specs master: DNM/WIP Remove device from attach_volume request response https://review.opendev.org/c/openstack/nova-specs/+/765097 | 11:58 |
| gibi | lyarwood: I will start looking into the qos failure on the master as melwitt said we propably hit that on stable after we fix the context targeting issue | 12:00 |
| *** JamesBenson has joined #openstack-nova | 12:00 | |
| gibi | hm, first, I will confirm that the stable failure after the context targeting fix is the same as the master failure | 12:01 |
| *** ociuhandu has joined #openstack-nova | 12:02 | |
| *** ociuhandu has quit IRC | 12:03 | |
| *** ociuhandu has joined #openstack-nova | 12:03 | |
| *** JamesBenson has quit IRC | 12:06 | |
| *** JamesBenson has joined #openstack-nova | 12:06 | |
| *** ratailor has quit IRC | 12:08 | |
| *** larainema has joined #openstack-nova | 12:12 | |
| gibi | OK, I confirm that after https://review.opendev.org/c/openstack/nova/+/766364 on stable we hit the same issue as visible on msater | 12:12 |
| gibi | master | 12:12 |
| gibi | so on master we need to fix https://bugs.launchpad.net/nova/+bug/1907522 and then backport the fix to stable on top of https://review.opendev.org/c/openstack/nova/+/766364 which is only needed on stable | 12:13 |
| openstack | Launchpad bug 1907522 in OpenStack Compute (nova) "test_*_with_qos_min_bw_allocation fails in the nova-multi-cell job with: nova.exception.MigrationPreCheckError: Migration pre-check error: Failed to create port bindings for host <host>" [Undecided,New] | 12:13 |
| *** rcernin has quit IRC | 12:13 | |
| gibi | this will be fun | 12:13 |
| *** zzzeek has quit IRC | 12:14 | |
| *** zzzeek has joined #openstack-nova | 12:17 | |
| *** ociuhandu has quit IRC | 12:23 | |
| *** hamalq has joined #openstack-nova | 12:28 | |
| *** ociuhandu has joined #openstack-nova | 12:31 | |
| *** hamalq has quit IRC | 12:33 | |
| *** mkrai has joined #openstack-nova | 12:36 | |
| *** zzzeek has quit IRC | 12:40 | |
| *** zzzeek has joined #openstack-nova | 12:41 | |
| *** raildo has joined #openstack-nova | 12:42 | |
| *** ociuhandu has quit IRC | 12:55 | |
| *** dave-mccowan has joined #openstack-nova | 12:56 | |
| *** adrian-a has quit IRC | 12:57 | |
| gibi | lyarwood, melwitt : I'm affraid that we missed the qos support for cross cell resize. I think qos and cross cell work was done in parallel. | 13:00 |
| *** adrian-a has joined #openstack-nova | 13:00 | |
| gibi | as the migration uses different conductor tasks and ComputeManager methods we simply missing the qos code from the cross cell path | 13:00 |
| *** adrian-a has quit IRC | 13:01 | |
| *** adrian-a has joined #openstack-nova | 13:01 | |
| *** ociuhandu has joined #openstack-nova | 13:05 | |
| openstackgerrit | Illes Elod proposed openstack/nova stable/train: DNM: tox constraints test https://review.opendev.org/c/openstack/nova/+/766463 | 13:07 |
| *** teoobo_ has quit IRC | 13:08 | |
| *** hemanth_n has quit IRC | 13:09 | |
| *** priteau has joined #openstack-nova | 13:11 | |
| *** ociuhandu has quit IRC | 13:15 | |
| *** hemna has quit IRC | 13:28 | |
| *** slaweq has quit IRC | 13:29 | |
| *** hemna has joined #openstack-nova | 13:31 | |
| *** k_mouza_ has joined #openstack-nova | 13:33 | |
| *** k_mouza has quit IRC | 13:33 | |
| *** teoobo_ has joined #openstack-nova | 13:33 | |
| *** slaweq has joined #openstack-nova | 13:33 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Disable qos resize tempest test for nova-multi-cell job https://review.opendev.org/c/openstack/nova/+/766471 | 13:38 |
| gibi | lyarwood, melwitt: ^^ this is my way forward on the master and this needs to be backported to stable | 13:38 |
| bauzas | I agree | 13:44 |
| bauzas | we need to revert the tests | 13:44 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Disable qos resize tempest test for nova-multi-cell job https://review.opendev.org/c/openstack/nova/+/766471 | 13:44 |
| gibi | bauzas: the test works fine with same cell resize so I would not revert the whole test | 13:45 |
| bauzas | yeah, by reverting, I meant disabling the faulty ones | 13:47 |
| bauzas | not the whole patch | 13:47 |
| bauzas | anyway +2d with comments | 13:47 |
| gibi | let me quicly fix those nits | 13:49 |
| *** teoobo_ has quit IRC | 13:49 | |
| *** _erlon_ has joined #openstack-nova | 13:50 | |
| *** ociuhandu has joined #openstack-nova | 13:52 | |
| *** zigo has joined #openstack-nova | 13:52 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Disable qos resize tempest test for nova-multi-cell job https://review.opendev.org/c/openstack/nova/+/766471 | 13:52 |
| gibi | bauzas: ^^ | 13:53 |
| *** tbachman has joined #openstack-nova | 13:53 | |
| bauzas | ack | 13:54 |
| *** nweinber has joined #openstack-nova | 13:56 | |
| *** sapd1 has joined #openstack-nova | 13:59 | |
| *** sapd1 has quit IRC | 14:00 | |
| *** sapd1 has joined #openstack-nova | 14:01 | |
| elod | lyarwood: I think the bandit patch needs to be merged starting from pike, and then to newer branches, branch by branch... due to grenade job. (which would be unnecessary if bandit 1.6.3 was yanked, but that might not happen soon...) | 14:05 |
| *** mkrai has quit IRC | 14:06 | |
| *** ociuhandu has quit IRC | 14:06 | |
| *** mkrai has joined #openstack-nova | 14:07 | |
| elod | lyarwood: also, I'm experimenting with the 'general' lower-constraint fix ( https://review.opendev.org/q/topic:"constrain-tox-install" ) but it might be better to try to fix the lower-constraints.txt instead, if it does not bring up infinite package version bumps there... | 14:08 |
| *** ociuhandu has joined #openstack-nova | 14:09 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova stable/victoria: Disable qos resize tempest test for nova-multi-cell job https://review.opendev.org/c/openstack/nova/+/766474 | 14:10 |
| *** ociuhandu has quit IRC | 14:13 | |
| *** ociuhandu has joined #openstack-nova | 14:15 | |
| *** adrian-a has quit IRC | 14:17 | |
| *** teoobo_ has joined #openstack-nova | 14:19 | |
| *** ociuhandu has quit IRC | 14:19 | |
| *** adrian-a has joined #openstack-nova | 14:21 | |
| gibi | elod: lyarwood: now I two possible way forward for stable. a) disable nova-multi-cell job b) disable the qos tests in the nova-multi-cell job | 14:23 |
| lyarwood | hey sorry just back from lunch | 14:24 |
| lyarwood | elod: ack I can work my way forwards from pike if it's going to be easier | 14:25 |
| lyarwood | gibi: https://review.opendev.org/c/openstack/nova/+/765749 is alreadying in the gate for victoria | 14:25 |
| lyarwood | gibi: making the entire job non-voting | 14:25 |
| gibi | lyarwood: OK to merge that. Then to re-enable the job we need https://review.opendev.org/c/openstack/nova/+/766474 | 14:26 |
| gibi | then the question what to do https://review.opendev.org/c/openstack/nova/+/766364/ | 14:27 |
| *** psachin has quit IRC | 14:28 | |
| *** ociuhandu has joined #openstack-nova | 14:28 | |
| *** hamalq has joined #openstack-nova | 14:29 | |
| lyarwood | gibi: https://review.opendev.org/c/openstack/nova/+/765749 isn't actually in the gate yet so we could yank it out now | 14:29 |
| elod | lyarwood: I've just checked that there's no lower-constraint job @ pike, so bandit patch could be merged easier (fingers crossed) :] | 14:29 |
| gibi | lyarwood: keep it as it is closer to merge | 14:30 |
| lyarwood | elod: cool, I'll try to make a start shortly | 14:30 |
| lyarwood | gibi: kk | 14:30 |
| gibi | lyarwood: then I will try to reenable the gate with https://review.opendev.org/c/openstack/nova/+/766474 | 14:30 |
| gibi | regarding https://review.opendev.org/c/openstack/nova/+/766364/ I have to figure out if we need this to avoid leaking something instead of clearly rolling back the migration | 14:31 |
| *** hamalq_ has joined #openstack-nova | 14:32 | |
| *** hamalq has quit IRC | 14:34 | |
| *** hamalq_ has quit IRC | 14:36 | |
| *** k_mouza_ has quit IRC | 14:38 | |
| *** tobberydberg has quit IRC | 14:38 | |
| *** k_mouza has joined #openstack-nova | 14:39 | |
| *** tobberydberg has joined #openstack-nova | 14:39 | |
| gmann | brinzhang: ah that is good question, I do not think we need to do any improvement on deprecated APIs. We have not done any update on deprecated API in past too. policy changes were on policy side only so we thought of keeping consistency. But I will discuss it in today meeting to get more opinion. | 14:40 |
| elod | lyarwood: btw, if you are busy, I can do the bandit backports till pike | 14:41 |
| elod | just let me know :) | 14:42 |
| lyarwood | elod: if you have time then please go ahead and I can review | 14:44 |
| gmann | brinzhang: but server sec group is not deprecated API -https://docs.openstack.org/api-ref/compute/?expanded=list-security-groups-by-server-detail#servers-security-groups-servers-os-security-groups | 14:44 |
| elod | lyarwood: ok, doing it now | 14:45 |
| gmann | brinzhang: ah just saw that we did mention about not to change deprecated APIs for tenant-id - https://specs.openstack.org/openstack/nova-specs/specs/wallaby/approved/remove-tenant-id.html | 14:45 |
| gmann | brinzhang: so we are good on that and let's target only non-deprecated APIs only | 14:46 |
| gmann | brinzhang: also our all deprecated APIs are capped with corresponding microversion and they are 404 for higher than deprecated microversion so technically we cannot do any change in those API which include the microversion bump. | 14:49 |
| openstackgerrit | Illes Elod proposed openstack/nova stable/stein: [stable-only] Cap bandit to 1.6.2 https://review.opendev.org/c/openstack/nova/+/766487 | 14:49 |
| *** k_mouza has quit IRC | 14:52 | |
| *** k_mouza has joined #openstack-nova | 14:56 | |
| *** ociuhandu has quit IRC | 14:57 | |
| openstackgerrit | Illes Elod proposed openstack/nova stable/stein: [stable-only] Cap bandit to 1.6.2 https://review.opendev.org/c/openstack/nova/+/766487 | 15:00 |
| openstackgerrit | Illes Elod proposed openstack/nova stable/rocky: [stable-only] Cap bandit to 1.6.2 https://review.opendev.org/c/openstack/nova/+/766492 | 15:03 |
| *** links has quit IRC | 15:04 | |
| *** ociuhandu has joined #openstack-nova | 15:05 | |
| *** ociuhandu has quit IRC | 15:10 | |
| lyarwood | elod: Bernard just replied on the ML about the cross project issue with the bandit cap | 15:11 |
| *** ociuhandu has joined #openstack-nova | 15:11 | |
| lyarwood | elod: and thinking about it more an overall cap in requirements might be the only way to unblock things here | 15:11 |
| *** songwenping_ has quit IRC | 15:11 | |
| lyarwood | elod: but landing from stable/pike as you suggested for the project specific changes | 15:11 |
| lyarwood | elod: so grenade works etc | 15:11 |
| *** songwenping_ has joined #openstack-nova | 15:12 | |
| lyarwood | `ERROR Failed to update project None in 3s` oh cool a new zuul error | 15:12 |
| openstackgerrit | Illes Elod proposed openstack/nova stable/pike: [stable-only] Cap bandit to 1.6.2 https://review.opendev.org/c/openstack/nova/+/766497 | 15:13 |
| lyarwood | https://github.com/PyCQA/bandit/issues/663 that said it looks like they might yank 1.6.3 after all | 15:14 |
| *** ociuhandu has quit IRC | 15:15 | |
| *** ociuhandu has joined #openstack-nova | 15:20 | |
| gibi | lyarwood: this is not the first time I saw that zuul ERROR | 15:23 |
| *** k_mouza has quit IRC | 15:24 | |
| gibi | lyarwood: at the previous time it was around a gerrit restart so I assmue that caused it | 15:24 |
| lyarwood | ah kk | 15:24 |
| bauzas | ergh, call me stupid | 15:24 |
| bauzas | requested_networks can be None | 15:25 |
| * bauzas hides | 15:25 | |
| gibi | lyarwood: pinged infra about it | 15:26 |
| *** dklyle has joined #openstack-nova | 15:27 | |
| lyarwood | gibi: ah thanks, I was just going to recheck tbh | 15:28 |
| gibi | lyarwood: sure, | 15:28 |
| gibi | lyarwood: infra increased some git cloning timeouts in the zuul executors | 15:29 |
| gibi | recently | 15:29 |
| gibi | they expect that it helps | 15:30 |
| elod | lyarwood: if they would yank bandit 1.6.3 that would be the best :) | 15:35 |
| elod | lyarwood: until that, let's see if we can merge the bandit cap patches, as it might take some time | 15:35 |
| lyarwood | elod: ack lets keep trying | 15:36 |
| lyarwood | elod: I'm just worried that we will need to land multiple caps across multiple projects to allow the gate to work again | 15:36 |
| elod | lyarwood: yes, that's possible :S | 15:37 |
| *** k_mouza has joined #openstack-nova | 15:49 | |
| *** sapd1 has quit IRC | 15:52 | |
| *** adrian-a has quit IRC | 15:57 | |
| *** lpetrut has quit IRC | 15:57 | |
| *** links has joined #openstack-nova | 16:00 | |
| *** sapd1 has joined #openstack-nova | 16:01 | |
| *** macz_ has joined #openstack-nova | 16:01 | |
| *** sapd1 has quit IRC | 16:07 | |
| *** mkrai has quit IRC | 16:26 | |
| *** hamalq has joined #openstack-nova | 16:30 | |
| *** ociuhandu has quit IRC | 16:31 | |
| *** teoobo_ has left #openstack-nova | 16:31 | |
| *** k_mouza has quit IRC | 16:33 | |
| *** k_mouza has joined #openstack-nova | 16:35 | |
| *** ociuhandu has joined #openstack-nova | 16:36 | |
| *** songwenping__ has joined #openstack-nova | 16:46 | |
| *** ociuhandu has quit IRC | 16:48 | |
| *** songwenping_ has quit IRC | 16:48 | |
| *** ociuhandu has joined #openstack-nova | 16:53 | |
| *** ociuhandu_ has joined #openstack-nova | 16:59 | |
| gibi | lyarwood, melwitt: I rechecked https://review.opendev.org/c/openstack/nova/+/766471 (master unblock patch) as it bounced due to zuul " ERROR Failed to update project None in" | 17:00 |
| gibi | I have to leave in ~15 minutes so I hope melwitt can take over rechecking it if needed | 17:01 |
| *** ociuhandu has quit IRC | 17:03 | |
| *** links has quit IRC | 17:09 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: DNM https://review.opendev.org/c/openstack/nova/+/766509 | 17:12 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova stable/victoria: DNM https://review.opendev.org/c/openstack/nova/+/766511 | 17:13 |
| lyarwood | gibi: ack thanks I'll also be around for a while to recheck when required | 17:15 |
| gibi | lyarwood: thanks I will check back too before I go to sleep but now dinner time | 17:16 |
| gibi | o/ | 17:16 |
| gibi | lyarwood: ahh, it is already got errors from zuul on https://zuul.opendev.org/t/openstack/status | 17:17 |
| gibi | :/ | 17:17 |
| gibi | so I think we have to wait for infra | 17:17 |
| *** adrian-a has joined #openstack-nova | 17:19 | |
| lyarwood | ack np enjoy dinner \o | 17:22 |
| bauzas | sean-k-mooney: around ? | 17:29 |
| bauzas | sean-k-mooney: I found why my functest is not working, I need to mock the neutron extension for segments | 17:29 |
| sean-k-mooney | bauzas: ah yes you do | 17:30 |
| sean-k-mooney | i think i have an example of that | 17:30 |
| sean-k-mooney | in one of teh regression test i wrote | 17:30 |
| sean-k-mooney | bauzas: https://github.com/openstack/nova/commit/71bc6fc9b89535679252ffe5a737eddad60e4102#diff-4763b3c03cb51e2efd60e19a11af59d4bfb76fbca93c953dfd9201eb5187ee88 | 17:32 |
| sean-k-mooney | bauzas: you can just do https://github.com/openstack/nova/blob/master/nova/tests/functional/regressions/test_bug_1888395.py#L38-L55 | 17:33 |
| *** dave-mccowan has quit IRC | 17:33 | |
| sean-k-mooney | bauzas: you could also update the fixture | 17:33 |
| *** hamalq has quit IRC | 17:34 | |
| sean-k-mooney | bauzas: but im not sure we want it there by default | 17:34 |
| sean-k-mooney | https://github.com/openstack/nova/blob/master/nova/tests/fixtures.py#L1891-L1905 | 17:35 |
| sean-k-mooney | personally i would not add it there since most deployment wont have it and just add it in your test class | 17:35 |
| bauzas | ack thanks | 17:35 |
| bauzas | sorry I had to drop because of an Amazon delivery for my new headset \o/ | 17:36 |
| sean-k-mooney | oh nice | 17:36 |
| * bauzas will test the Plantronics RIG800HS for tomorrow's meeting :) | 17:37 | |
| JamesBenson | sean-k-mooney: originally I had tried `custom` and posted `cpu_models = nehalem,westmere` & `cpu_model_extra_flags = pcid, vmx, pdpe1gb` but then I couldn't boot up VM's. Those three flags are the ones of interest, but aren't in both models. I thought that the way mentioned above was the right way, but couldn't get it working. Maybe the flags need to be there for all of them? | 17:38 |
| *** ociuhandu_ has quit IRC | 17:42 | |
| sean-k-mooney | JamesBenson: that should be valid | 17:45 |
| sean-k-mooney | pdep1GB is for 1G hugepages | 17:45 |
| sean-k-mooney | nehalem does not actully support that if i remeber correctly | 17:45 |
| sean-k-mooney | you need host supprot to use it | 17:46 |
| JamesBenson | correct, nehalem doesn't support any of those three. | 17:46 |
| JamesBenson | but westmere does | 17:46 |
| sean-k-mooney | wel it support vmx | 17:46 |
| sean-k-mooney | not in the model defieniotn but in the cpu | 17:46 |
| sean-k-mooney | wehn using multiple models the flags must be valid for all models in the list | 17:47 |
| sean-k-mooney | on your westmer host that should work | 17:47 |
| JamesBenson | ah, okay, so that's the trick then. And just verifying, to check for the flags, I just issued `cat /proc/cpuinfo` | 17:48 |
| sean-k-mooney | but on your nehalem host it will fail | 17:48 |
| JamesBenson | let me double check the flags on my hosts. | 17:48 |
| sean-k-mooney | yep cat /proc/cpuinfo or use lscpu | 17:48 |
| sean-k-mooney | for vmx you need to enable nested virt too in the hosts kernel | 17:49 |
| JamesBenson | okay, yeah only vmx on the nehalem, westmere has all three. | 17:50 |
| sean-k-mooney | cat /sys/module/kvm_intel/parameters/nested | 17:50 |
| sean-k-mooney | Y | 17:50 |
| JamesBenson | just change that to Y? | 17:50 |
| sean-k-mooney | Y or 1 | 17:50 |
| sean-k-mooney | that is showing the current nested virt status | 17:51 |
| sean-k-mooney | if that is N or 0 then you dont have it enabled | 17:51 |
| JamesBenson | okay, cool. So which is better: cpu_models = nehalem,westmere with the vmx flag or host-model? Or should I get a list of flags that are on both and put all of the flags in there? | 17:51 |
| JamesBenson | It is an `N` | 17:52 |
| JamesBenson | Thanks for the help & insight be the way. Super helpful :-) | 17:52 |
| sean-k-mooney | ya so you need to drop a file into /etc/modeprobe.d | 17:52 |
| sean-k-mooney | what i would do is as follows | 17:53 |
| sean-k-mooney | on the nehelm host set cpu_models = nehalem and cpu_model_extra_flags = vmx | 17:54 |
| *** ociuhandu has joined #openstack-nova | 17:54 | |
| sean-k-mooney | on the westmear host do cpu_model_extra_flags = pcid, vmx, pdpe1gb cpu_models = nehalem,westmere | 17:54 |
| sean-k-mooney | in the flavor then add a required trait for pdep1gb on flavor that use 1G hugepages | 17:55 |
| sean-k-mooney | JamesBenson: the feature flags are reported as traits to placement so you can express what you need in both the flavor and image | 17:56 |
| JamesBenson | okay. Is this the recommended path for mixed architectures to get the most out of them? | 17:56 |
| JamesBenson | great, we have windows VM's so that's what I'm hoping to speed up with using those flags. | 17:56 |
| *** raildo has quit IRC | 17:57 | |
| sean-k-mooney | normaly you would use something like host aggreates. PCID should acclerate gust where you have mitigation enabled somewhat | 17:57 |
| *** raildo has joined #openstack-nova | 17:58 | |
| sean-k-mooney | vmx is only useful if those windows instnace have hyperv or are using docker with linux instance | 17:58 |
| *** ociuhandu has quit IRC | 17:58 | |
| *** tesseract has quit IRC | 17:58 | |
| sean-k-mooney | on host without pdep1gb you can still use 2MB hugepages | 17:59 |
| sean-k-mooney | so that is what i advise on nehelem | 17:59 |
| sean-k-mooney | JamesBenson: http://paste.openstack.org/show/800947/ | 18:01 |
| *** derekh has quit IRC | 18:01 | |
| sean-k-mooney | you shoudl have a file that looks like that or you can create one | 18:01 |
| sean-k-mooney | options kvm_intel nested=1 is what you need to uncomment unless you have a kernel newwer then 4.19 | 18:01 |
| sean-k-mooney | in which case its the default value | 18:01 |
| sean-k-mooney | although your proably want a kernel newer then 5.1 unless it has backports | 18:02 |
| JamesBenson | freshly deployed ubuntu 18.04 with 4.15.0 | 18:02 |
| sean-k-mooney | i would sugges using the 5.3 or 5.4 kernel they provide form focal instead | 18:03 |
| sean-k-mooney | its the hwe kernel | 18:03 |
| *** hamalq has joined #openstack-nova | 18:03 | |
| sean-k-mooney | https://wiki.ubuntu.com/Kernel/LTSEnablementStack | 18:03 |
| *** rpittau is now known as rpittau|afk | 18:04 | |
| JamesBenson | okay :-) | 18:05 |
| sean-k-mooney | out of interst is tehre a reason you are not usin g2.04 | 18:05 |
| sean-k-mooney | *20.04 | 18:05 |
| sean-k-mooney | if its a fresh install | 18:05 |
| JamesBenson | kolla doesn't support it I think fully | 18:06 |
| JamesBenson | That's my deployment method. | 18:06 |
| sean-k-mooney | yep i use kolla too. althouhg im currentoy using centos kind of | 18:07 |
| *** hamalq_ has joined #openstack-nova | 18:07 | |
| sean-k-mooney | JamesBenson: depending on the release they have different levels of supprot for 20.04 | 18:07 |
| JamesBenson | yeah, I'm on ussuri, I thought victoria was the first one to support 20 fully. | 18:08 |
| sean-k-mooney | JamesBenson: yes it is | 18:09 |
| sean-k-mooney | JamesBenson: i guess they have not had there offical release yet technically | 18:09 |
| JamesBenson | ah, well, then I can definitely try that out. We do it all, including the OS deploy, in a CI/CD environment, so easy enough ;-) | 18:09 |
| JamesBenson | yeah, not yet, soon I think | 18:10 |
| *** hamalq has quit IRC | 18:10 | |
| JamesBenson | but for victoria, we do plan on pushing to ubuntu 20.04 | 18:10 |
| JamesBenson | but I'll see about bumping the kernel, I've just been using what it deploys with, having tried to get it to a newer version. | 18:10 |
| sean-k-mooney | the nested virt support signifcantly improve after kernel 4.19 so if you use it i always recommend having 4.19 and preferbly 5.1+ as you min kernel | 18:11 |
| sean-k-mooney | JamesBenson: the hwe kernel works well for nested virt without needing the os bump | 18:12 |
| sean-k-mooney | its 5.3 i belive | 18:12 |
| JamesBenson | cool, have you ever tried low-latency kernels in openstack? | 18:12 |
| sean-k-mooney | i have but my experice was tainted. i deployed with the low-latency kernel but i later found out that my system hangs which i attibuted to it was due to 3 failing dimms | 18:13 |
| ganso | elod, bauzas, lyarwood: Hi! do you think those patches could be allowed to be backported from rocky to queens: https://review.opendev.org/c/openstack/nova/+/580720 and https://review.opendev.org/c/openstack/nova/+/583715 ? | 18:13 |
| sean-k-mooney | JamesBenson: so i have not really looked at it fairly. | 18:14 |
| sean-k-mooney | JamesBenson: my deployemtn is just my home test/dev cloud so its currently runing the mainline 5.9 kernel | 18:14 |
| sean-k-mooney | i was hoping to move to the 5.10 upstream lts and do a ussuri to victoria upgrade over the holiday break but that is still tbd | 18:15 |
| JamesBenson | okay, looks like linux-lowlatency-hwe-18.04 is 5.4.0.53.59 | 18:15 |
| JamesBenson | so maybe I'll try that out :-) | 18:15 |
| sean-k-mooney | cool with that you wont need t o set the modeprobe paramter eitehr since its enabel by default | 18:17 |
| ganso | elod, bauzas, lyarwood, nevermind I kind of got the answer myself by cherry-picking and seeing that queens is missing 2 other RequestSpec updates | 18:17 |
| sean-k-mooney | ganso: we cant backport modifcation to any nova objects | 18:17 |
| ganso | sean-k-mooney: thanks! | 18:18 |
| sean-k-mooney | ganso: well we cant backport new fields | 18:18 |
| sean-k-mooney | we can backport change that dont modify the serialsed format | 18:18 |
| sean-k-mooney | ganso: you wanted this in queens | 18:20 |
| ganso | sean-k-mooney: yes, in queens | 18:20 |
| ganso | sean-k-mooney: since the fix requires the new field, we wouldn't be able to adapt it to queens, as I understand | 18:20 |
| sean-k-mooney | that is correct unfortunetly however we coudl do something else possibly | 18:22 |
| sean-k-mooney | if the places that need is_bfv hae a context and instnace tehy could use is_volume_backed_instance(ctxt, instance) instead directly | 18:22 |
| *** jangutter_ has joined #openstack-nova | 18:23 | |
| ganso | sean-k-mooney: hmmm I will investigate this in the code. Thanks! Although, I'm aware it would make the backport with new code with extra risk, and then the stable maintainers may disagree with merging it | 18:24 |
| sean-k-mooney | yes you would basically have to modeify https://review.opendev.org/c/openstack/nova/+/580720/6/nova/scheduler/utils.py | 18:24 |
| sean-k-mooney | unfortunetly that does not have th instance or a keystone context | 18:25 |
| ganso | oh I see | 18:25 |
| ganso | yea, bummer | 18:25 |
| ganso | thanks again sean-k-mooney ! =) | 18:25 |
| *** jangutter has quit IRC | 18:26 | |
| *** CeeMac has joined #openstack-nova | 18:34 | |
| *** jangutter_ has quit IRC | 18:40 | |
| *** dtantsur is now known as dtantsur|afk | 18:40 | |
| *** jangutter has joined #openstack-nova | 18:41 | |
| *** k_mouza has quit IRC | 18:48 | |
| *** efried1 has joined #openstack-nova | 19:13 | |
| *** efried has quit IRC | 19:15 | |
| *** efried1 is now known as efried | 19:15 | |
| *** adrian-a has quit IRC | 19:20 | |
| *** adrian-a has joined #openstack-nova | 19:24 | |
| melwitt | gibi, lyarwood: yes I can take over babysitting those patches once yall are gone | 19:44 |
| *** xek_ has quit IRC | 19:49 | |
| *** xek has joined #openstack-nova | 20:04 | |
| *** andrewbonney has quit IRC | 20:05 | |
| *** jangutter has quit IRC | 20:05 | |
| *** martinkennelly has quit IRC | 20:06 | |
| *** jangutter has joined #openstack-nova | 20:06 | |
| *** tbachman has quit IRC | 20:09 | |
| openstackgerrit | Lance Bragstad proposed openstack/placement master: Implement secure RBAC for resource providers https://review.opendev.org/c/openstack/placement/+/760240 | 20:10 |
| *** martinkennelly has joined #openstack-nova | 20:10 | |
| *** martinkennelly has quit IRC | 20:19 | |
| *** rcernin has joined #openstack-nova | 20:21 | |
| *** nweinber has quit IRC | 20:26 | |
| *** martinkennelly has joined #openstack-nova | 20:27 | |
| *** nweinber has joined #openstack-nova | 20:47 | |
| *** xek has quit IRC | 20:50 | |
| *** xek has joined #openstack-nova | 20:54 | |
| *** slaweq has quit IRC | 21:16 | |
| JamesBenson | sean-k-mooney, one follow up question. Is there a setting (or benefit) to passing all of the CPU flags in the nova.conf? | 21:21 |
| *** songwenping_ has joined #openstack-nova | 21:28 | |
| *** songwenping__ has quit IRC | 21:31 | |
| *** rcernin has quit IRC | 21:31 | |
| *** rcernin has joined #openstack-nova | 21:34 | |
| *** eharney has quit IRC | 21:38 | |
| *** ociuhandu has joined #openstack-nova | 21:40 | |
| lyarwood | elod: https://review.opendev.org/q/I8f24b839bf42e2fb9803dc7df3a30ae20cf264eb lets see what that breaks | 21:50 |
| lyarwood | melwitt: ack sorry had already gone, just wanted to check on the bandit thing and it looks like we might be able to get away with backporting the above to workaround it | 21:50 |
| *** xek has quit IRC | 21:51 | |
| *** rcernin has quit IRC | 21:51 | |
| melwitt | lyarwood: is there a bug link for the bandit thing? sorry I fell too far behind with knowing what was going on there | 21:53 |
| *** rcernin has joined #openstack-nova | 21:53 | |
| lyarwood | melwitt: http://lists.openstack.org/pipermail/openstack-discuss/2020-December/019292.html and https://bugs.launchpad.net/nova/+bug/1907438 | 21:55 |
| openstack | Launchpad bug 1907438 in OpenStack Object Storage (swift) "ERROR: Package 'bandit' requires a different Python: 2.7.17 not in '>=3.5'" [Undecided,Confirmed] | 21:55 |
| melwitt | ah thanks | 21:55 |
| melwitt | I don't know how I missed that on the ML | 21:55 |
| lyarwood | there's so many fires at the moment, I wouldn't worry about it :) | 21:56 |
| melwitt | you can say that again | 21:56 |
| lyarwood | there's so many fires at the moment, I wouldn't worry about it :) | 21:56 |
| * lyarwood shows himself out | 21:56 | |
| melwitt | last night I almost tweeted "openstack ci right now is" with a mushroom cloud gif | 21:56 |
| melwitt | hah xD | 21:56 |
| melwitt | or the gif of elmo with the fire behind him | 21:57 |
| *** nweinber has quit IRC | 21:58 | |
| lyarwood | hehe yeah both would still work right now :D | 21:59 |
| lyarwood | ah cool the stable/victoria LC change made it to the gate | 22:00 |
| melwitt | hot dog! | 22:00 |
| *** eharney has joined #openstack-nova | 22:02 | |
| melwitt | lyarwood: hm something I don't get though is how the LC change is having nova-multi-cell as non-voting if the disable voting change isn't in the gate? | 22:06 |
| lyarwood | melwitt: different branches, I wrote the LC change switching nova-multi-cell to non-voting before gibi pushed his master change | 22:08 |
| lyarwood | melwitt: I was going to pull it out and backport squash his into the stable/victoria LC fix but he said to just go on with the non-voting change | 22:09 |
| lyarwood | melwitt: either way it's only to allow the LC change to land in stable/victoria, we will need to follow up once it's in | 22:10 |
| melwitt | lyarwood: yeah but https://review.opendev.org/c/openstack/nova/+/766440 isn't approved? maybe I'm just missing something dumb | 22:10 |
| lyarwood | melwitt: oh I need to close that, I squashed that into the actual LC change, that's an old PS that isn't used anymore | 22:11 |
| melwitt | oh you squashed it. sorry /facepalm | 22:11 |
| melwitt | yeah I see now. guhhhhh | 22:11 |
| lyarwood | yeah sorry I've got dead changes all over the place from the last few days | 22:11 |
| melwitt | no it's not your fault that I can't see things :) | 22:11 |
| melwitt | ok, so iiuc we'll do this and then after gibi's change lands, we need to backport it to stable/victoria, then we re-enable voting | 22:12 |
| *** mlavalle has joined #openstack-nova | 22:13 | |
| lyarwood | yeah I think so | 22:14 |
| lyarwood | okay bed time for me now, good luck with the gate :D | 22:14 |
| *** bbowen has quit IRC | 22:15 | |
| melwitt | ok cool, gnight o/ | 22:15 |
| melwitt | argh nova-ceph-multistore failed in the gate queue. of course it did | 22:17 |
| *** adrian-a has quit IRC | 22:38 | |
| *** martinkennelly has quit IRC | 22:48 | |
| *** ociuhandu has quit IRC | 22:59 | |
| *** efried has quit IRC | 23:04 | |
| *** bbowen has joined #openstack-nova | 23:37 | |
| *** tosky has quit IRC | 23:53 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!