| openstackgerrit | melanie witt proposed openstack/nova master: Propagate exception message from _prep_block_device https://review.openstack.org/624832 | 00:01 |
|---|---|---|
| *** mriedem has quit IRC | 00:01 | |
| *** wolverineav has quit IRC | 00:01 | |
| *** wolverineav has joined #openstack-nova | 00:04 | |
| *** wolverineav has quit IRC | 00:10 | |
| *** itlinux has joined #openstack-nova | 00:11 | |
| *** itlinux has quit IRC | 00:11 | |
| *** macza has quit IRC | 00:14 | |
| *** tetsuro has joined #openstack-nova | 00:15 | |
| openstackgerrit | melanie witt proposed openstack/nova master: Add method to generate device names universally https://review.openstack.org/573066 | 00:16 |
| openstackgerrit | melanie witt proposed openstack/nova master: WIP Add configuration of maximum disk devices to attach https://review.openstack.org/616777 | 00:16 |
| *** spatel has joined #openstack-nova | 00:17 | |
| openstackgerrit | Merged openstack/nova master: refactor get_console_output() for console logfiles https://review.openstack.org/575735 | 00:20 |
| *** spatel has quit IRC | 00:22 | |
| *** rodolof has quit IRC | 00:43 | |
| *** rodolof has joined #openstack-nova | 00:44 | |
| *** Swami has quit IRC | 00:48 | |
| *** wolverineav has joined #openstack-nova | 00:50 | |
| *** dklyle has joined #openstack-nova | 00:53 | |
| *** wolverineav has quit IRC | 00:54 | |
| *** rodolof has quit IRC | 01:04 | |
| *** rodolof has joined #openstack-nova | 01:04 | |
| *** brinzhang has joined #openstack-nova | 01:08 | |
| *** gyee has quit IRC | 01:08 | |
| *** dave-mccowan has joined #openstack-nova | 01:17 | |
| openstackgerrit | sean mooney proposed openstack/nova master: Libvirt: do not set mac when unplugging macvtap vf https://review.openstack.org/624842 | 01:18 |
| *** rodolof has quit IRC | 01:30 | |
| *** yedongcan has joined #openstack-nova | 01:40 | |
| *** bhagyashris has joined #openstack-nova | 01:56 | |
| *** Dinesh_Bhor has joined #openstack-nova | 01:59 | |
| *** tetsuro has quit IRC | 02:05 | |
| *** mrsoul has joined #openstack-nova | 02:05 | |
| openstackgerrit | Brin Zhang proposed openstack/nova-specs master: Support admin to specify project to create snapshot https://review.openstack.org/616843 | 02:06 |
| *** Dinesh_Bhor has quit IRC | 02:09 | |
| *** Dinesh_Bhor has joined #openstack-nova | 02:10 | |
| *** tinwood has quit IRC | 02:10 | |
| *** tinwood has joined #openstack-nova | 02:11 | |
| *** codyhill has quit IRC | 02:13 | |
| *** dave-mccowan has quit IRC | 02:14 | |
| *** itlinux has joined #openstack-nova | 02:18 | |
| *** mhen has quit IRC | 02:27 | |
| *** wolverineav has joined #openstack-nova | 02:28 | |
| *** mhen has joined #openstack-nova | 02:28 | |
| *** wolverineav has quit IRC | 02:31 | |
| *** wolverineav has joined #openstack-nova | 02:32 | |
| *** hongbin has joined #openstack-nova | 02:45 | |
| *** wolverineav has quit IRC | 02:45 | |
| *** spatel has joined #openstack-nova | 02:56 | |
| *** cfriesen has quit IRC | 03:17 | |
| *** dklyle has quit IRC | 03:22 | |
| openstackgerrit | zhangyujun proposed openstack/nova master: ESFIX: Should not raise while restore power on failed https://review.openstack.org/624854 | 03:27 |
| *** brinzhang has quit IRC | 03:28 | |
| openstackgerrit | zhangyujun proposed openstack/nova master: Should not raise while restore power on failed https://review.openstack.org/624854 | 03:30 |
| *** wolverineav has joined #openstack-nova | 03:33 | |
| *** wolverineav has quit IRC | 03:37 | |
| *** spatel has quit IRC | 03:39 | |
| *** wolverineav has joined #openstack-nova | 03:41 | |
| *** wolverineav has quit IRC | 03:41 | |
| *** rambo_li has joined #openstack-nova | 03:41 | |
| *** wolverineav has joined #openstack-nova | 03:41 | |
| *** tetsuro has joined #openstack-nova | 03:44 | |
| *** MasterofJOKers has quit IRC | 04:01 | |
| *** tetsuro has quit IRC | 04:04 | |
| *** MasterofJOKers has joined #openstack-nova | 04:10 | |
| *** itlinux has quit IRC | 04:11 | |
| *** rambo_li has quit IRC | 04:13 | |
| *** khomesh has joined #openstack-nova | 04:16 | |
| *** khomesh has quit IRC | 04:16 | |
| *** hongbin has quit IRC | 04:20 | |
| *** sridharg has joined #openstack-nova | 04:24 | |
| *** itlinux has joined #openstack-nova | 04:43 | |
| *** udesale has joined #openstack-nova | 04:49 | |
| *** bhagyashris has quit IRC | 05:01 | |
| *** itlinux has quit IRC | 05:11 | |
| *** wolverineav has quit IRC | 05:23 | |
| *** _alastor_ has joined #openstack-nova | 05:24 | |
| *** wolverineav has joined #openstack-nova | 05:40 | |
| *** ratailor has joined #openstack-nova | 05:40 | |
| *** wolverineav has quit IRC | 05:51 | |
| *** bhagyashris has joined #openstack-nova | 05:52 | |
| *** igordc has joined #openstack-nova | 06:13 | |
| *** _alastor_ has quit IRC | 06:20 | |
| *** ccamacho has quit IRC | 06:28 | |
| *** _alastor_ has joined #openstack-nova | 06:40 | |
| *** slaweq has joined #openstack-nova | 06:57 | |
| *** _alastor_ has quit IRC | 06:59 | |
| *** rcernin has quit IRC | 07:09 | |
| *** pcaruana has joined #openstack-nova | 07:12 | |
| *** ralonsoh has joined #openstack-nova | 07:19 | |
| *** artom has quit IRC | 07:29 | |
| *** wolverineav has joined #openstack-nova | 07:30 | |
| *** artom has joined #openstack-nova | 07:31 | |
| *** artom has quit IRC | 07:31 | |
| *** artom has joined #openstack-nova | 07:32 | |
| *** igordc has quit IRC | 07:32 | |
| *** igordc has joined #openstack-nova | 07:33 | |
| *** Luzi has joined #openstack-nova | 07:33 | |
| *** wolverineav has quit IRC | 07:34 | |
| *** _alastor_ has joined #openstack-nova | 07:35 | |
| openstackgerrit | Andrey Volkov proposed openstack/nova master: Exclude build request instance with marker from listing https://review.openstack.org/624870 | 07:35 |
| *** _alastor_ has quit IRC | 07:40 | |
| openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient master: Fix flavor keyerror when nova boot vm https://review.openstack.org/582147 | 07:50 |
| *** tetsuro has joined #openstack-nova | 07:51 | |
| *** dklyle has joined #openstack-nova | 08:07 | |
| *** _alastor_ has joined #openstack-nova | 08:08 | |
| *** brault has joined #openstack-nova | 08:08 | |
| *** maciejjozefczyk has joined #openstack-nova | 08:10 | |
| openstackgerrit | Takashi NATSUME proposed openstack/os-resource-classes master: Update author-email in setup.cfg https://review.openstack.org/624885 | 08:11 |
| *** _alastor_ has quit IRC | 08:12 | |
| *** helenafm has joined #openstack-nova | 08:17 | |
| *** ccamacho has joined #openstack-nova | 08:22 | |
| *** imacdonn has quit IRC | 08:23 | |
| *** imacdonn has joined #openstack-nova | 08:24 | |
| *** ohorecny2 has joined #openstack-nova | 08:26 | |
| *** dklyle has quit IRC | 08:27 | |
| *** Dinesh_Bhor has quit IRC | 08:31 | |
| *** takashin has left #openstack-nova | 08:31 | |
| *** Dinesh_Bhor has joined #openstack-nova | 08:32 | |
| *** xek has joined #openstack-nova | 08:32 | |
| *** sahid has joined #openstack-nova | 08:41 | |
| *** moshele has joined #openstack-nova | 08:43 | |
| *** maciejjozefczyk has quit IRC | 08:44 | |
| *** maciejjozefczyk has joined #openstack-nova | 08:44 | |
| *** brinzhang has joined #openstack-nova | 08:45 | |
| *** brinzh has joined #openstack-nova | 08:48 | |
| *** brinzhang has quit IRC | 08:50 | |
| *** alexchadin has joined #openstack-nova | 08:53 | |
| *** tssurya has joined #openstack-nova | 09:09 | |
| *** k_mouza has joined #openstack-nova | 09:10 | |
| *** k_mouza has quit IRC | 09:15 | |
| *** k_mouza has joined #openstack-nova | 09:15 | |
| *** trident has quit IRC | 09:16 | |
| *** ttsiouts has joined #openstack-nova | 09:18 | |
| *** trident has joined #openstack-nova | 09:19 | |
| *** belmoreira has quit IRC | 09:22 | |
| *** belmoreira has joined #openstack-nova | 09:26 | |
| *** ttsiouts has quit IRC | 09:28 | |
| *** ttsiouts has joined #openstack-nova | 09:29 | |
| *** yan0s has joined #openstack-nova | 09:30 | |
| *** sayalilunkad has quit IRC | 09:31 | |
| *** fanzhang has quit IRC | 09:31 | |
| *** ttsiouts has quit IRC | 09:33 | |
| *** alexchadin has quit IRC | 09:33 | |
| *** ttsiouts has joined #openstack-nova | 09:40 | |
| *** markvoelker has joined #openstack-nova | 09:46 | |
| *** rambo_li has joined #openstack-nova | 09:46 | |
| *** bhagyashris has quit IRC | 09:47 | |
| *** mdbooth_ has quit IRC | 09:55 | |
| *** rodolof has joined #openstack-nova | 09:58 | |
| *** mdbooth has joined #openstack-nova | 09:58 | |
| *** sahid has quit IRC | 10:11 | |
| *** trident has quit IRC | 10:15 | |
| *** trident has joined #openstack-nova | 10:15 | |
| *** k_mouza has quit IRC | 10:21 | |
| *** mvkr has quit IRC | 10:24 | |
| *** rambo_li has quit IRC | 10:28 | |
| *** moshele has quit IRC | 10:31 | |
| *** sahid has joined #openstack-nova | 10:33 | |
| *** sayalilunkad has joined #openstack-nova | 10:35 | |
| *** k_mouza has joined #openstack-nova | 10:38 | |
| *** cdent has joined #openstack-nova | 10:42 | |
| openstackgerrit | Merged openstack/nova stable/rocky: Only warn about not having computes nodes once in rpcapi https://review.openstack.org/624677 | 10:45 |
| *** ttsiouts has quit IRC | 10:51 | |
| *** ttsiouts has joined #openstack-nova | 10:52 | |
| *** maciejjozefczyk has quit IRC | 10:52 | |
| *** dtantsur|afk is now known as dtantsur | 10:53 | |
| *** maciejjozefczyk has joined #openstack-nova | 10:53 | |
| *** mvkr has joined #openstack-nova | 10:56 | |
| *** ttsiouts has quit IRC | 10:57 | |
| *** derekh has joined #openstack-nova | 10:59 | |
| *** moshele has joined #openstack-nova | 11:03 | |
| frickler | lyarwood: could I ask you to place https://review.openstack.org/619254 onto your review list? that's the last one in my stable backport series, all the others got merged | 11:07 |
| *** wolverineav has joined #openstack-nova | 11:08 | |
| lyarwood | frickler: ack | 11:11 |
| openstackgerrit | Merged openstack/os-resource-classes master: Initial automated testing for os-resource-classes https://review.openstack.org/624366 | 11:12 |
| openstackgerrit | Merged openstack/os-resource-classes master: Rename ORDERED_CLASSES to STANDARDS https://review.openstack.org/624369 | 11:12 |
| openstackgerrit | Merged openstack/os-resource-classes master: Sync STANDARDS with placement/nova rc_fields https://review.openstack.org/624370 | 11:12 |
| *** wolverineav has quit IRC | 11:13 | |
| *** mvkr has quit IRC | 11:16 | |
| *** mvkr has joined #openstack-nova | 11:16 | |
| openstackgerrit | Tom Barron proposed openstack/nova stable/rocky: Do Not Merge, testing libvirt-python requirements https://review.openstack.org/624958 | 11:18 |
| *** tbarron has joined #openstack-nova | 11:19 | |
| *** yedongcan has left #openstack-nova | 11:24 | |
| *** _alastor_ has joined #openstack-nova | 11:25 | |
| *** _alastor_ has quit IRC | 11:30 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: WIP don't call begin_detaching when detaching volume from sheved vm https://review.openstack.org/624959 | 11:33 |
| *** tobias-urdin is now known as tobias-urdin_afk | 11:41 | |
| *** tobias-urdin_afk is now known as tobias-urdin | 11:42 | |
| *** moshele has quit IRC | 11:42 | |
| *** tobias-urdin is now known as tobias-urdin_afk | 11:43 | |
| *** tbachman has quit IRC | 11:44 | |
| *** maciejjozefczyk has quit IRC | 11:47 | |
| openstackgerrit | Chris Dent proposed openstack/os-resource-classes master: Tune up documentation to be more useful https://review.openstack.org/624384 | 11:49 |
| *** maciejjozefczyk has joined #openstack-nova | 11:53 | |
| *** moshele has joined #openstack-nova | 12:04 | |
| cdent | stephenfin: see my comment on https://review.openstack.org/624826 for something that might be of interest to you. Also I fixed the problems you identified in ^ | 12:14 |
| * stephenfin clicks | 12:14 | |
| *** Dinesh_Bhor has quit IRC | 12:14 | |
| stephenfin | cdent: Hmm, the cookiecutter? Weird. Maybe something's changed that I wasn't aware of | 12:14 |
| cdent | no, the functional tests stuff in 624826 not producing tox files because of shared envs | 12:15 |
| *** helenafm has quit IRC | 12:17 | |
| *** eharney has quit IRC | 12:24 | |
| *** markvoelker has quit IRC | 12:24 | |
| *** udesale has quit IRC | 12:25 | |
| *** udesale has joined #openstack-nova | 12:26 | |
| openstackgerrit | Chris Dent proposed openstack/os-resource-classes master: Tune up documentation to be more useful https://review.openstack.org/624384 | 12:26 |
| *** brinzh has quit IRC | 12:32 | |
| openstackgerrit | Adrian Chiris proposed openstack/nova master: Allow libvirt to properly set-up and clean-up MACs https://review.openstack.org/624973 | 12:33 |
| openstackgerrit | Georg Hoesch proposed openstack/nova master: allow tcp-based consoles in get_console_output https://review.openstack.org/624974 | 12:34 |
| sean-k-mooney | stephenfin: can you review https://review.openstack.org/#/c/624831/ ill need to do a 1.13.1 release when that lands. os-vif is currently breaking the iptable hybrid job in nueutron | 12:41 |
| stephenfin | can do | 12:42 |
| *** mriedem has joined #openstack-nova | 12:42 | |
| adrianc | sean-k-mooney: https://review.openstack.org/#/c/624973/ this should take care of the macs issue | 12:42 |
| adrianc | sean-k-mooney: also you will need to re-deploy https://review.openstack.org/#/c/620123/ (macvtap multiple port binding POC) | 12:43 |
| sean-k-mooney | adrianc: i also submitted https://review.openstack.org/#/c/624842/ | 12:43 |
| adrianc | sean-k-mooney: ahh :) should have looked more carefully on the incoming reviews | 12:44 |
| adrianc | sean-k-mooney: well i believe another WA code should be removed that is related, under nova/network/linux_utils.py | 12:45 |
| sean-k-mooney | adrianc: yes i wanted to do it in two parts. | 12:45 |
| *** ratailor has quit IRC | 12:45 | |
| adrianc | sean-k-mooney: if you have time look at my commit, ill abandon mine then | 12:46 |
| sean-k-mooney | setting the mac to 0 so we can backport and thenremove teh fuction call | 12:46 |
| sean-k-mooney | adrianc: yes i saw you removed the code here https://review.openstack.org/#/c/624973/1/nova/network/linux_utils.py | 12:46 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Fail to live migration if instance has a NUMA topology https://review.openstack.org/611088 | 12:48 |
| adrianc | sean-k-mooney: its a must as the removed code sets the 0 mac on the VF netdev, this will fail | 12:48 |
| openstackgerrit | Tom Barron proposed openstack/nova stable/rocky: Do Not Merge, testing libvirt-python requirements https://review.openstack.org/624958 | 12:49 |
| sean-k-mooney | the vf netdev perhaps the vf itself no | 12:49 |
| adrianc | sean-k-mooney: i.e not removing the code will attempt to set mac address on the VF netdev, in the case of zero mac it will fail | 12:49 |
| sean-k-mooney | i was about to use my patchset to live migrate back and foth with out issues | 12:49 |
| *** igordc has quit IRC | 12:51 | |
| adrianc | sean-k-mooney: ive also uploaded a new PS for the macVtap multiple port binding, i removed all the set/clear macs on bind/unbind as it was just covering the problems introduces in the plug/unplug hw veb | 12:52 |
| sean-k-mooney | adrianc: a good to know. we have our christmas event this evening so i may or may not get a chance to test it before i leave in 2 hours but ill use it going forward | 12:53 |
| *** tobias-urdin_afk is now known as tobias-urdin | 12:53 | |
| sean-k-mooney | and you are right we get sean@pop-os:~$ sudo ip link set enp0s31f6 address 00:00:00:00:00:00 | 12:53 |
| sean-k-mooney | RTNETLINK answers: Cannot assign requested address | 12:53 |
| adrianc | sean-k-mooney: libvirt in the case of macVtap will just set the mac on the vf netdev not the vf index (e.g 'administrative mac') and restores it to the previous value on teardown | 12:54 |
| *** rodolof has quit IRC | 12:54 | |
| *** rodolof has joined #openstack-nova | 12:54 | |
| *** tetsuro has quit IRC | 12:54 | |
| adrianc | sean-k-mooney: enjoy the christmas event ! | 12:54 |
| sean-k-mooney | ya, ill combine our two patches and also fix the unit test which strangly passed form me before but whatever | 12:54 |
| adrianc | ack thanks, on my setup i managed to live migrate with ping running within the migrating VM so it seems to be working | 12:55 |
| *** igordc has joined #openstack-nova | 12:59 | |
| openstackgerrit | Merged openstack/nova stable/pike: Make supports_direct_io work on 4096b sector size https://review.openstack.org/619254 | 12:59 |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/rocky: Make compute rpcapi version calculation check all cells https://review.openstack.org/624982 | 12:59 |
| *** igordc has quit IRC | 13:10 | |
| *** markvoelker has joined #openstack-nova | 13:10 | |
| *** tbachman has joined #openstack-nova | 13:11 | |
| *** rambo_li has joined #openstack-nova | 13:13 | |
| *** tbachman has quit IRC | 13:16 | |
| *** eharney has joined #openstack-nova | 13:16 | |
| *** tbachman has joined #openstack-nova | 13:21 | |
| stephenfin | sean-k-mooney: Before I submit the next version of this, thoughts on https://review.openstack.org/#/c/624444/1/nova/objects/instance_pci_requests.py ? | 13:27 |
| cdent | mriedem, stephenfin: jaypipes was +W on this before I found (and fixed) a race, so if you're inclined to give it a gander: https://review.openstack.org/#/c/617941/ | 13:28 |
| stephenfin | cdent: mriedem has already taken a look at that, so if he has time I'll leave it to him | 13:29 |
| cdent | ✔ | 13:30 |
| sean-k-mooney | thanks i stephenfin so the default numa afinity policy was ment to keep backward compatiblyt with existing behavior whic was the legacy policy im not sure if https://github.com/openstack/nova/blob/f572527d2b7f93b0c2108118c2f240f2dd1cf968/nova/pci/stats.py#L155 does that | 13:30 |
| mriedem | i looked at one file in there :) | 13:31 |
| sean-k-mooney | i assuem it does but i hate the fact we assuem things in the code. e.g. that if cpu_policy is not dedicated it is shared. | 13:31 |
| stephenfin | sean-k-mooney: nope, but the function called on the next line after that does https://github.com/openstack/nova/blob/f572527d2b7f93b0c2108118c2f240f2dd1cf968/nova/pci/stats.py#L240 | 13:31 |
| stephenfin | mriedem: Oh :) I can take a look so | 13:31 |
| mriedem | cdent: just this yeah? https://review.openstack.org/#/c/617941/30..31/nova/tests/functional/test_nova_status.py | 13:32 |
| cdent | mriedem: that was the latest change, yes | 13:32 |
| sean-k-mooney | stephenfin: so yes it will do the right thing eventrully i just prefer to set the vaules as early as possibel rather then doing late tests | 13:33 |
| cdent | the race there didn't show up for test after test after test, and then it was at the good slot in the gate and failed | 13:33 |
| mriedem | cdent: done | 13:34 |
| sean-k-mooney | stephenfin: its up to you ill remove my -1 but its just a style thing i guess i prefer to explitly state thing up front rather then defenively check later | 13:35 |
| stephenfin | sean-k-mooney: I can add it. No harm | 13:36 |
| *** helenafm has joined #openstack-nova | 13:36 | |
| cdent | mriedem: I and my lines removed stats thank you | 13:36 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: objects: Store InstancePCIRequest.numa_policy in DB https://review.openstack.org/624444 | 13:37 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Call out where pci.alias should be set https://review.openstack.org/624445 | 13:37 |
| sean-k-mooney | stephenfin: ok other then that i was happy with the change over all however looks like the unit tests were less so | 13:37 |
| stephenfin | sean-k-mooney: Yeah, one test that I missed. Fixed now | 13:37 |
| stephenfin | gibi, mriedem: Could you take a look at this follow up docs patch today? https://review.openstack.org/#/c/614322/ | 13:39 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add irrelevant-files for grenade-py3 jobs https://review.openstack.org/624997 | 13:39 |
| *** rambo_li has quit IRC | 13:45 | |
| gibi | stephenfin: +2 from my side | 13:46 |
| frickler | mriedem: now that my stable patches have merged, can we talk about a new round of releases? would save me from having to run on custom-built pkgs. I'd mostly be interested in pike and queens currently | 13:46 |
| stephenfin | gibi: Thank you | 13:46 |
| *** irclogbot_2 has quit IRC | 13:50 | |
| *** igordc has joined #openstack-nova | 13:52 | |
| mriedem | frickler: we normally do the releases in order starting from the most recent, so rocky->queens->pike, | 13:54 |
| mriedem | idk if melwitt was planning a stable release soon or not | 13:54 |
| frickler | melwitt: mriedem: at least I'd say it looks like a nice bunch of things has piled up at https://docs.openstack.org/releasenotes/nova/rocky.html#relnotes-18-0-3-38-stable-rocky | 13:58 |
| gibi | nova team meeting is starting on #openstack-meeting | 14:00 |
| *** markvoelker has quit IRC | 14:03 | |
| mriedem | frickler: i'll bring it up in the team meeting | 14:03 |
| frickler | mriedem: cool, thx | 14:06 |
| *** igordc has quit IRC | 14:07 | |
| *** irclogbot_2 has joined #openstack-nova | 14:08 | |
| mriedem | lyarwood: want to hit this backport https://review.openstack.org/#/c/624813/ | 14:08 |
| * lyarwood looks | 14:09 | |
| lyarwood | mriedem: LGTM, I don't think I've actually asked but you don't have an issue with me proxying stable core votes in changes like this right? | 14:13 |
| mriedem | lyarwood: you see me doing that on yours right? | 14:14 |
| lyarwood | mriedem: right, just checking. | 14:14 |
| mriedem | i think it's ok for clean backports on clear bug fixes | 14:14 |
| lyarwood | yeah agreed, I still feel slightly uneasy about it sometimes but this example is simple etc. | 14:15 |
| mriedem | if it's something like stephen's series here https://review.openstack.org/#/c/588570/ then that's a different story | 14:15 |
| lyarwood | stephenfin: ^ I was going to ask if you wanted to drop this btw | 14:15 |
| mriedem | with extended maintenance it's less bad if we screw up on stable b/c we have time to fix it, | 14:16 |
| stephenfin | lyarwood: Oh, yeah. I'll abandon that now | 14:16 |
| lyarwood | stephenfin: thanks | 14:16 |
| mriedem | plus we just have fewer people reviewing stable and i hate to let fixes sit for months | 14:16 |
| lyarwood | yup indeed, cool that's clear now thanks. | 14:16 |
| *** irclogbot_2 has quit IRC | 14:19 | |
| *** mchlumsky has joined #openstack-nova | 14:22 | |
| stephenfin | sean-k-mooney: Thoughts on this? https://review.openstack.org/#/c/625011/ Could be a more programmatic solution to https://review.openstack.org/#/c/624445/ | 14:26 |
| *** ttsiouts has joined #openstack-nova | 14:28 | |
| *** k_mouza_ has joined #openstack-nova | 14:31 | |
| *** irclogbot_2 has joined #openstack-nova | 14:31 | |
| *** rodolof has quit IRC | 14:32 | |
| *** rodolof has joined #openstack-nova | 14:32 | |
| cdent | mnaser, coreycb, anyone else: I'll spin up an etherpad on the "path internal to nova to happy placement extraction" topic and invite you to help refine it. Won't be today, but soon | 14:33 |
| *** _alastor_ has joined #openstack-nova | 14:33 | |
| *** k_mouza has quit IRC | 14:34 | |
| *** k_mouza_ has quit IRC | 14:35 | |
| *** markvoelker has joined #openstack-nova | 14:36 | |
| *** cfriesen has joined #openstack-nova | 14:36 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Summarize output of sample configuration generator https://review.openstack.org/625014 | 14:37 |
| gibi | mriedem: when you have a minute, can you check back to https://review.openstack.org/#/c/624740/ | 14:37 |
| openstackgerrit | sean mooney proposed openstack/nova master: Libvirt: do not set mac when unplugging macvtap vf https://review.openstack.org/624842 | 14:37 |
| *** _alastor_ has quit IRC | 14:38 | |
| frickler | coreycb: did you make progress with building pike with stestr? otherwise we should maybe also consider reverting https://review.openstack.org/580492 before the next stable/pike release | 14:38 |
| *** mlavalle has joined #openstack-nova | 14:38 | |
| frickler | mriedem: melwitt: ^^ not sure whether you had seen that some weeks ago. stestr isn't in the original set of pike reqs, so building the latest release there was broken on ubuntu | 14:39 |
| frickler | also sorry that I didn't remember to mention that for the meeting | 14:40 |
| frickler | oh, there's https://bugs.launchpad.net/nova/+bug/1806079 for that already | 14:41 |
| openstack | Launchpad bug 1806079 in OpenStack Compute (nova) "revert use of stestr in stable/pike" [Wishlist,In progress] - Assigned to Corey Bryant (corey.bryant) | 14:41 |
| *** priteau has joined #openstack-nova | 14:42 | |
| mriedem | gibi: looking | 14:42 |
| gibi | adrianc: hi! we talked about sriov after the bandwidth demo in Berlin. I think I see the way forward how to influence which VF nova-compute chooses for a neutron port https://review.openstack.org/#/c/623543 | 14:43 |
| mriedem | frickler: yes i specifically requested that from mtreinish for reasons stated in the patch/bug | 14:43 |
| *** liuyulong has joined #openstack-nova | 14:43 | |
| mriedem | https://review.openstack.org/#/c/621227/2//COMMIT_MSG@12 | 14:43 |
| gibi | adrianc: the patch is still WIP but the the logic is already visible | 14:43 |
| frickler | mriedem: yeah, I was just reading that. if it reverts cleanly, I guess it should be fine to carry the revert on the ubuntu branch | 14:44 |
| mriedem | frickler: as coreycb said, they can carry it as a patch for them on pike | 14:44 |
| mriedem | yeah | 14:44 |
| gibi | mriedem: futurist added to the nova lower-constraints when the global lower constraints was moved from the global requirement to per project | 14:45 |
| *** wolverineav has joined #openstack-nova | 14:45 | |
| adrianc | gibi: Hi, ill put this in my review list | 14:45 |
| gibi | adrianc: cool | 14:45 |
| mriedem | gibi: and that was done blindly via automation, | 14:45 |
| gibi | mriedem: I guess so | 14:45 |
| mriedem | before your change, nova didn't use futurist, so this is the first usage of it, | 14:45 |
| gibi | mriedem: true | 14:45 |
| mriedem | which means, we should use what we gate against, | 14:45 |
| mriedem | so use the version from u-c | 14:45 |
| gibi | mriedem: when u-c changes then our lower constraints can diverge from u-c is that OK? | 14:46 |
| mriedem | yeah | 14:46 |
| mriedem | we have a lower-constraints unit test job for that, | 14:46 |
| mriedem | but upper-constraints is used in both unit/functional/tempest integration testing | 14:46 |
| mriedem | lower-constraints is kind of a 'cover your bare ass' job | 14:46 |
| cdent | mriedem: its broken until https://review.openstack.org/#/c/622972/ | 14:47 |
| gibi | mriedem: OK, then I will fix up the patch by bumping the lower constraints of futurist to 1.8.0 | 14:47 |
| *** awaugama has joined #openstack-nova | 14:48 | |
| *** moshele has quit IRC | 14:48 | |
| *** takashin has joined #openstack-nova | 14:49 | |
| *** wolverineav has quit IRC | 14:49 | |
| mnaser | cdent: cool thanks | 14:50 |
| adrianc | gibi: is there also code that deals with the migration use-case ? (cold and live-migration) | 14:50 |
| *** ondrejme has quit IRC | 14:50 | |
| openstackgerrit | Merged openstack/os-vif master: Create iproute.IPRoute() inside a context https://review.openstack.org/624831 | 14:50 |
| *** takashin has left #openstack-nova | 14:51 | |
| gibi | adrianc: not yet. that patch series only works with boot, delete, re-schedule. I'm working on the instance move cases | 14:51 |
| gibi | cdent: I can try my patch with your lower constraint fix to see if everything works after both | 14:52 |
| adrianc | gibi: ack :) | 14:52 |
| gibi | adrianc: ohh one more thing, interface detach works as well :) | 14:53 |
| adrianc | gibi: good to know! sean-k-mooney ^ hopefully the direct support will go smoothly | 14:54 |
| *** panda|off is now known as panda | 14:58 | |
| *** Shilpa has joined #openstack-nova | 14:59 | |
| *** ShilpaSD has quit IRC | 15:02 | |
| sean-k-mooney | gibi: well i intend to detach the interface at the hypervior level but i wont actully be detaching the interface form a neutron point of view | 15:03 |
| sean-k-mooney | adrianc: ^ | 15:03 |
| sean-k-mooney | from a nova and neutron point of view the interfce need to remain attached to the instance as it does in the macvtap case for schduling to work | 15:04 |
| sean-k-mooney | so i plan on only detaching at the libvirt level | 15:05 |
| gibi | sean-k-mooney: a think we are thinking about two different use case. you are working on live-migrate and I work on detaching a port that has bandwidth allocation in placement | 15:05 |
| sean-k-mooney | gibi: yep | 15:05 |
| sean-k-mooney | gibi: but if you have live migration working with bandwidth allocation on ovs for example i think it shoudl work with sriov too once we are done | 15:06 |
| *** dpawlik has quit IRC | 15:06 | |
| *** itlinux has joined #openstack-nova | 15:07 | |
| *** dpawlik has joined #openstack-nova | 15:07 | |
| *** dpawlik has quit IRC | 15:07 | |
| sean-k-mooney | from a neutorn perspective and mostly from a nova one the detach/attach i plan to do will be hiden in the driver level | 15:07 |
| gibi | sean-k-mooney: there is one difference between ovs and sriov. for sriov there is VF resource handled by nova today and now there will be bandwidth resource handled by placement | 15:09 |
| *** maciejjozefczyk has quit IRC | 15:10 | |
| gibi | sean-k-mooney: the leads to a situation described in the patch https://review.openstack.org/#/c/623543/ | 15:10 |
| gibi | sean-k-mooney: and we discussed a bit about this in Berlin with adrianc | 15:11 |
| sean-k-mooney | gibi: ill be leaveing soon but ill take a look tomorow. i can try and test it next week. | 15:13 |
| gibi | sean-k-mooney: no rush, that patch is still WIP | 15:13 |
| gibi | sean-k-mooney: it is more like that the problem statement might be relevant for your live migration work | 15:14 |
| openstackgerrit | Jose Castro Leon proposed openstack/nova master: Fix destination_type attribute in the bdm_v2 documentation https://review.openstack.org/625020 | 15:19 |
| *** sridharg has quit IRC | 15:25 | |
| openstackgerrit | Surya Seetharaman proposed openstack/nova master: Add DownCellFixture https://review.openstack.org/614810 | 15:36 |
| openstackgerrit | Surya Seetharaman proposed openstack/nova master: API microversion 2.68: Handles Down Cells https://review.openstack.org/591657 | 15:36 |
| *** markvoelker has quit IRC | 15:38 | |
| *** dpawlik has joined #openstack-nova | 15:40 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Replace ThreadPoolExecutor with GreenThreadPoolExecutor https://review.openstack.org/624740 | 15:41 |
| mriedem | d'oh | 15:41 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Replace ThreadPoolExecutor with GreenThreadPoolExecutor https://review.openstack.org/624740 | 15:42 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add python 3.7 unit and functional tox jobs https://review.openstack.org/624055 | 15:42 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add py36/py37 functional jobs to the experimental queue https://review.openstack.org/624826 | 15:42 |
| mriedem | gibi: i was rebasing and fixing that at the same time | 15:42 |
| gibi | mriedem: no problemo | 15:43 |
| gibi | stephenfin: could you look at it? ^^ | 15:44 |
| *** dpawlik has quit IRC | 15:44 | |
| *** efried has joined #openstack-nova | 15:45 | |
| gibi | stephenfin: mriedem and I agreed on the solution | 15:45 |
| *** moshele has joined #openstack-nova | 15:51 | |
| *** Luzi has quit IRC | 15:52 | |
| *** ccamacho has quit IRC | 15:52 | |
| *** ttsiouts has quit IRC | 15:56 | |
| *** ttsiouts has joined #openstack-nova | 15:57 | |
| mriedem | ooo goody http://logs.openstack.org/82/624982/1/check/nova-next/4d4ec67/logs/screen-n-cpu.txt.gz | 15:59 |
| *** ohorecny2 has quit IRC | 16:00 | |
| dansmith | looks fine | 16:00 |
| mriedem | dansmith: that's the backport of your multi-cell n-api rpc version check | 16:03 |
| mriedem | the n-cpu failures there have to be unrelated | 16:03 |
| mriedem | i thought we collected mysql logs but maybe not anymore | 16:04 |
| dansmith | hmm, it shouldn't check all cells if the api connection isn't set | 16:04 |
| mriedem | this is n-cpu failing to start | 16:04 |
| *** yan0s has quit IRC | 16:04 | |
| mriedem | File "/opt/stack/new/nova/nova/objects/cell_mapping.py", line 257, in get_all\n | 16:05 |
| mriedem | wtf | 16:05 |
| mriedem | oooo | 16:05 |
| mriedem | http://logs.openstack.org/82/624982/1/check/nova-next/4d4ec67/logs/etc/nova/nova-cpu.conf.txt.gz | 16:05 |
| mriedem | we still have the api db conf set in rocky devstack | 16:05 |
| *** wwriverrat_ has quit IRC | 16:05 | |
| dansmith | but.. then it shouldn't fail right? | 16:06 |
| mriedem | unless it's starting n-cpu before the api db is up? | 16:06 |
| mriedem | but i thought n-cpu went last | 16:06 |
| dansmith | the error says no connection parameter, right? | 16:06 |
| dansmith | oh, this is a RemoteError btw | 16:06 |
| *** alexchadin has joined #openstack-nova | 16:07 | |
| dansmith | so the cell conductor doesn't have it, | 16:07 |
| dansmith | but cpu does, so it tried a remotable all_cells call maybe? | 16:07 |
| dansmith | no, that doesn't make sense | 16:07 |
| mriedem | right nova_cell1.conf (conductor) doesn't have api_database http://logs.openstack.org/82/624982/1/check/nova-next/4d4ec67/logs/etc/nova/nova_cell1.conf.txt.gz | 16:08 |
| dansmith | ph. | 16:08 |
| dansmith | yeah | 16:08 |
| dansmith | it's doing load_cells, which is where the remotable comes in | 16:08 |
| mriedem | https://github.com/openstack/nova/blob/stable/rocky/nova/compute/manager.py#L505 | 16:08 |
| dansmith | so yeah just nuking the cpu config api database should fix it I think | 16:08 |
| mriedem | compute manager init creates compute.API which creates the compute rpcapi client which checks the api db conf | 16:08 |
| dansmith | File "/opt/stack/new/nova/nova/context.py", line 476, in load_cells | 16:09 |
| mriedem | https://github.com/openstack-dev/devstack/commit/7d0003ef7ed2c2609ffe6782e0ce6147f9c9a3bf#diff-665a24457e945ac31372cf63d00a4080 | 16:09 |
| mriedem | just need that | 16:09 |
| dansmith | right, which ends up calling into load_cells, which does the remotable I think | 16:09 |
| dansmith | yar | 16:09 |
| mriedem | i think before rocky, devstack didn't use upgrade_levels=auto | 16:09 |
| dansmith | ack | 16:09 |
| *** morgan is now known as kmalloc | 16:09 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/rocky: Make compute rpcapi version calculation check all cells https://review.openstack.org/624982 | 16:10 |
| *** alexchadin has quit IRC | 16:10 | |
| *** ttsiouts has quit IRC | 16:16 | |
| *** eharney has quit IRC | 16:22 | |
| *** eharney_ has joined #openstack-nova | 16:22 | |
| *** moshele has quit IRC | 16:23 | |
| mriedem | dansmith: hmm, thinking about that... | 16:25 |
| mriedem | people are setting api db in cell configs, | 16:25 |
| mriedem | so they can do the up-call stuff | 16:25 |
| mriedem | so when nova-compute starts up in cell1, it could be determining it's min version from computes in cell50 | 16:25 |
| dansmith | as long as it's the same on the compute and corresponding conductor it'll befine | 16:25 |
| mriedem | in cerns case | 16:25 |
| dansmith | oh | 16:26 |
| dansmith | I see what you mean, but that probably isn't terrible | 16:26 |
| dansmith | if you're giving the cell visibility to the whole deployment, then .. you get the whole deployment's min version | 16:26 |
| dansmith | this is exactly why I *don't* want to make this a common practice, fwiw, | 16:26 |
| dansmith | because now you're talking about teaching the cells about their place in the world and that gets icky | 16:27 |
| mriedem | i just don't know if it could throw anyone off, i.e. upgrade reno or anything, | 16:28 |
| mriedem | but if you're doing upgrade_levels=auto, | 16:28 |
| mriedem | you're opting into the system figuring this out | 16:28 |
| mriedem | and can pin manually if you don't trust the system | 16:28 |
| mriedem | let's pretend i never mentioned this | 16:28 |
| dansmith | yep | 16:28 |
| *** tbachman has quit IRC | 16:32 | |
| *** moshele has joined #openstack-nova | 16:33 | |
| *** _alastor_ has joined #openstack-nova | 16:34 | |
| cdent | mriedem: I've added some links to https://bugs.launchpad.net/openstack-gate/+bug/1808063 for the recent failure that you rechecked. g-api is having a stackoverflow | 16:35 |
| openstack | Launchpad bug 1808063 in OpenStack-Gate "glanceclient.exc.HTTPBadGateway: 502 Proxy Error during server snapshot" [Undecided,Confirmed] | 16:35 |
| mriedem | oh nice | 16:38 |
| mriedem | http://logs.openstack.org/41/617941/31/gate/grenade-py3/7909f30/logs/screen-g-api.txt.gz#_Dec_13_14_43_21_414511 | 16:38 |
| mriedem | Dec 13 14:43:21.414511 ubuntu-xenial-rax-ord-0001176665 devstack@g-api.service[6990]: File "/usr/local/lib/python3.5/dist-packages/oslo_messaging/_drivers/impl_rabbit.py", line 926 in _heartbeat_thread_job | 16:38 |
| mriedem | dansmith: look familiar? ^ | 16:39 |
| *** efried has quit IRC | 16:39 | |
| cdent | and a second comment | 16:39 |
| mriedem | File "/opt/stack/new/glance/glance/quota/__init__.py", line 168 in __getattr__ looks like infinite loop? | 16:39 |
| cdent | eyes | 16:39 |
| cdent | yes | 16:39 |
| mriedem | https://github.com/openstack/glance/commit/cb45edf5c81f7d09c9ef0b88d40d56b4750beb10#diff-5afd3797ee6a482f2c0d4862140f7014 was the last change in there but awhile ago | 16:40 |
| mriedem | https://github.com/openstack/glance/blob/master/glance/quota/__init__.py#L168 | 16:40 |
| *** udesale has quit IRC | 16:41 | |
| * mriedem goes to -glance | 16:43 | |
| *** _alastor_ has quit IRC | 16:43 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/queens: Only warn about not having computes nodes once in rpcapi https://review.openstack.org/625051 | 16:44 |
| *** eharney_ is now known as eharney | 16:53 | |
| *** moshele has quit IRC | 16:59 | |
| melwitt | mriedem: +1 to flushing queens and pike backports and releasing before the holiday break. I assumed not rocky bc we did that semi recently but were you thinking rocky too? I'm sure a lot has landed since last time | 17:02 |
| melwitt | I will help flush | 17:02 |
| *** igordc has joined #openstack-nova | 17:06 | |
| *** moshele has joined #openstack-nova | 17:06 | |
| *** moshele has quit IRC | 17:07 | |
| *** helenafm has quit IRC | 17:07 | |
| *** hamzy_ has quit IRC | 17:10 | |
| *** hamzy_ has joined #openstack-nova | 17:10 | |
| mriedem | melwitt: yes otherwise we could release things in pike/queens that are not yet released in rocky | 17:11 |
| mriedem | generally need to snowball these from newest to oldest branch | 17:11 |
| melwitt | good point, I should have thought of that. thanks | 17:11 |
| openstackgerrit | Merged openstack/nova stable/rocky: Handle tags in _bury_in_cell0 https://review.openstack.org/624813 | 17:12 |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/queens: Make compute rpcapi version calculation check all cells https://review.openstack.org/625060 | 17:14 |
| *** igordc has quit IRC | 17:17 | |
| *** sahid has quit IRC | 17:25 | |
| *** dtantsur is now known as dtantsur|afk | 17:27 | |
| *** derekh has quit IRC | 17:28 | |
| *** igordc has joined #openstack-nova | 17:34 | |
| openstackgerrit | melanie witt proposed openstack/nova master: Propagate exception message from _prep_block_device https://review.openstack.org/624832 | 17:35 |
| openstackgerrit | melanie witt proposed openstack/nova master: Add method to generate device names universally https://review.openstack.org/573066 | 17:35 |
| openstackgerrit | melanie witt proposed openstack/nova master: WIP Add configuration of maximum disk devices to attach https://review.openstack.org/616777 | 17:35 |
| *** igordc has quit IRC | 17:39 | |
| *** colby_ has joined #openstack-nova | 17:39 | |
| *** igordc has joined #openstack-nova | 17:39 | |
| colby_ | Hey Eveyrone. We just upgraded to queeens. Im seeing some weird issues when creating an instance from an already created volume. Im seeing this error: | 17:40 |
| colby_ | Build of instance aborted: Nova does not support Cinder API version 2. The catalog is set to use v3 cinder as default, and in the nova config we have this set for cinder: catalog_info=volumev3:cinderv3:internalURL | 17:40 |
| colby_ | we have a volumev3 in the catalog as well that points to the v3 api | 17:41 |
| *** panda is now known as panda|off | 17:42 | |
| *** tbachman has joined #openstack-nova | 17:46 | |
| *** igordc has quit IRC | 17:47 | |
| melwitt | colby_: is it possible you have [cinder]endpoint_template set, which will override catalog_info? | 17:55 |
| *** cezary_zukowski has joined #openstack-nova | 17:58 | |
| *** wolverineav has joined #openstack-nova | 18:01 | |
| *** erlon has joined #openstack-nova | 18:01 | |
| cezary_zukowski | Hey, how to simulate/mimic a network disruption occuring between a Nova instance/server and Cinder (volume/storage)? Does detaching a volume from instance manually would be a sensible approach/substitute? | 18:01 |
| mriedem | that would be my guess, endpoint_template is set | 18:01 |
| cezary_zukowski | Command for detaching: nova volume-detach <instance_id> <volume_id> | 18:02 |
| cezary_zukowski | mriedem: What do you mean? | 18:02 |
| colby_ | melwitt: no its not set in the config anywhere | 18:03 |
| mriedem | cezary_zukowski: was answering a different question | 18:03 |
| cezary_zukowski | mriedem: ah, right | 18:03 |
| mriedem | colby_: do you have this? https://github.com/openstack/nova/commit/c7d87f6691bbc39daca918729b769cc2d2000abd | 18:04 |
| mriedem | that's in 17.0.6 | 18:04 |
| mriedem | if you're queens ga you wouldn't have that | 18:04 |
| mriedem | not sure if that's your problem though | 18:05 |
| *** Swami has joined #openstack-nova | 18:06 | |
| *** gyee has joined #openstack-nova | 18:09 | |
| *** tssurya has quit IRC | 18:16 | |
| melwitt | colby_: aside from all of that, I would just take a look at your keystone catalog again and double check there's no /v2.0 for nova (should be /v2.1) and no /v2 for cinder | 18:17 |
| melwitt | because the only other thing I see the code doing to determine the cinder version is parsing the version from the endpoint found in the catalog | 18:18 |
| colby_ | oh it has the wrong config on the actual compute nodes. Thanks that helped me narrow it down. Ill switch that config there. Apparently it just got switched on the controller services | 18:18 |
| melwitt | ok, cool | 18:20 |
| *** ralonsoh has quit IRC | 18:24 | |
| *** _alastor_ has joined #openstack-nova | 18:31 | |
| *** radez has quit IRC | 18:33 | |
| *** wolverineav has quit IRC | 18:37 | |
| *** _alastor_ has quit IRC | 18:39 | |
| *** mvkr has quit IRC | 18:40 | |
| *** wolverineav has joined #openstack-nova | 18:43 | |
| *** mpiwowarczy has joined #openstack-nova | 18:45 | |
| mpiwowarczy | hello | 18:45 |
| mpiwowarczy | can I have a question to nova deprecated personality parameter? | 18:46 |
| mpiwowarczy | seems it will be removed in microversion 2.57 | 18:47 |
| *** jmlowe has quit IRC | 18:47 | |
| mpiwowarczy | but can not found when 2.57 will be used? | 18:47 |
| mpiwowarczy | for Rocky release it was microversion 2.27 | 18:47 |
| *** wolverineav has quit IRC | 18:48 | |
| mpiwowarczy | which version will be used in Stein? | 18:48 |
| mpiwowarczy | or is there any fixed date when 2.57 version will be dropped? | 18:49 |
| *** mlavalle has quit IRC | 18:51 | |
| artom | mpiwowarczy, microversions are never dropped, we still support 2.1 if the client asks for it | 18:54 |
| artom | mpiwowarczy, in terms of releases, have a look at https://docs.openstack.org/nova/latest/reference/api-microversion-history.html | 18:54 |
| mpiwowarczy | artom: thanks for the answer | 18:56 |
| mpiwowarczy | so when personality files (files injection) feature will be dropped? | 18:56 |
| mpiwowarczy | will be this supported in Stein release? | 18:58 |
| artom | mpiwowarczy, it'll never be dropped per se | 19:01 |
| artom | mpiwowarczy, you can keep using it in Stein if the client asks for a microversion < 2.57 | 19:02 |
| mpiwowarczy | ok, thanks for your help | 19:03 |
| melwitt | mpiwowarczy: it's dropped in a sense that if/when you find yourself needing to request microversion >= 2.57 when you call the REST API to get a different feature/addition, you will not be able to use personality files at the same time/in the same call | 19:07 |
| mpiwowarczy | melwitt: I see | 19:08 |
| mpiwowarczy | do you intend to maintain all versions, starting from 2.1, in all future OpenStack releases? | 19:10 |
| mpiwowarczy | day by day it is more difficult to maintain :) | 19:11 |
| *** jmlowe has joined #openstack-nova | 19:12 | |
| mriedem | what is more difficult to maintain? writing apps against the API? or maintaining the various microversions in the API? | 19:18 |
| mriedem | the client opts into the behavior, so if you are ok with v2.1 then you don't need to ever change that | 19:18 |
| *** wolverineav has joined #openstack-nova | 19:19 | |
| mpiwowarczy | oh yeah, you're right | 19:20 |
| *** eharney has quit IRC | 19:20 | |
| *** wolverineav has quit IRC | 19:20 | |
| *** wolverineav has joined #openstack-nova | 19:20 | |
| mpiwowarczy | just one more question - latest novaclient 11.1.0, which microversion it uses? | 19:20 |
| mriedem | the alternative is we don't use microversions, change the api from release to release, and all client software everywhere in the world has to deal with that based on the cloud they are talking to | 19:20 |
| mpiwowarczy | where I can check that? | 19:21 |
| mriedem | the client-side support is defined here https://github.com/openstack/python-novaclient/blob/master/novaclient/__init__.py#L28 | 19:21 |
| *** mvkr has joined #openstack-nova | 19:21 | |
| mriedem | novaclient python APIs by default use 2.1 | 19:21 |
| mriedem | the CLI by default negotiates the highest supported microversion between the client and the server | 19:21 |
| mriedem | so if you're using novaclient as a python sdk, then it's opt-in, | 19:22 |
| mriedem | if you're using the CLI, it's negotiated unless you tell it otherwise | 19:22 |
| mpiwowarczy | it is about SDK | 19:23 |
| mriedem | then it's opt-in | 19:23 |
| mpiwowarczy | we have defined version python-novaclient==11.1.0 and we use file injection | 19:23 |
| mriedem | api methods in novaclient are wrapped / bound based on what versions they support | 19:23 |
| mriedem | as i said, unless you opt into a newer microversion, it will use 2.1 | 19:23 |
| mriedem | for the sdk | 19:23 |
| openstackgerrit | Merged openstack/nova stable/rocky: Harden placement init under wsgi https://review.openstack.org/617297 | 19:23 |
| mpiwowarczy | *python-novaclient>=9.1.0 # Apache-2.0 | 19:24 |
| *** N3l1x has joined #openstack-nova | 19:24 | |
| mpiwowarczy | thanks, you helped me a lot | 19:25 |
| mriedem | yw | 19:25 |
| *** N3l1x has quit IRC | 19:30 | |
| *** mpiwowarczy has quit IRC | 19:30 | |
| melwitt | mriedem: do happen to know if we have the opposite of --availability-zone ZONE:HOST:NODE for admins where it would mean launch the server anywhere but this node? curious because I know in the request spec we have both force_hosts and ignore_hosts | 19:33 |
| melwitt | I didn't think we did but not 100% sure | 19:33 |
| mriedem | not from the api i don't think so | 19:34 |
| mriedem | ignore_hosts is for when you're moving a server and you ignore the source host from schedluing | 19:34 |
| cdent | i've had people ask about that too, and also couldn't find it | 19:34 |
| mriedem | or ignore hosts you've rescheduled from | 19:34 |
| melwitt | thanks. someone asked recently and I told them I think forbidden traits is the closest thing to something like that | 19:35 |
| *** igordc has joined #openstack-nova | 19:35 | |
| mriedem | well, | 19:35 |
| mriedem | for that to work, you'd have to temporarily put a custom trait on the blacklisted host, schedule to !host via forbidden trait, and then remove the trait | 19:35 |
| mriedem | if you're going to do that, you might as well just disable the compute service and then re-enable it after the scheduling is done | 19:36 |
| mriedem | b/c the ComputeFilter essentially does the same thing | 19:36 |
| mriedem | unless you're cern and you drop the max placement results down to 10 | 19:36 |
| *** rodolof has quit IRC | 19:36 | |
| melwitt | yeah, well I was thinking more general if there was an underlying reason to avoid that host, like there's special hardware there that you want to avoid taking up | 19:36 |
| *** rodolof has joined #openstack-nova | 19:37 | |
| melwitt | but as a one shot thing if there's no other meaning to it, yeah, I agree disabling is easier | 19:37 |
| mriedem | i believe tetsuro's forbidden aggregates spec goes into detail about that | 19:38 |
| mriedem | for blazar | 19:38 |
| mriedem | it is essentially the same problem, you have a special pool of resources that non-special requests can't have access to | 19:38 |
| melwitt | yeah | 19:39 |
| *** N3l1x has joined #openstack-nova | 19:39 | |
| *** pcaruana has quit IRC | 19:47 | |
| *** jangutter has quit IRC | 19:49 | |
| *** igordc has quit IRC | 19:57 | |
| *** wolverineav has quit IRC | 20:05 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove GROUP BY clause from CellMapping.get_by_project_id https://review.openstack.org/619061 | 20:09 |
| mriedem | zzzeek: i did that subquery and it works ^ thanks for the tip there | 20:09 |
| zzzeek | mriedem: that's great. keep pinging me for thse so we can make the DB code better :) | 20:09 |
| *** wolverineav has joined #openstack-nova | 20:10 | |
| *** wolverineav has quit IRC | 20:15 | |
| *** priteau has quit IRC | 20:15 | |
| *** erlon has quit IRC | 20:15 | |
| *** wolverineav has joined #openstack-nova | 20:23 | |
| *** eharney has joined #openstack-nova | 20:24 | |
| *** wolverineav has quit IRC | 20:25 | |
| *** wolverineav has joined #openstack-nova | 20:25 | |
| *** munimeha1 has joined #openstack-nova | 20:28 | |
| *** bauzas has quit IRC | 20:42 | |
| *** bauzas has joined #openstack-nova | 20:46 | |
| *** psachin has joined #openstack-nova | 20:52 | |
| *** lpetrut has joined #openstack-nova | 20:53 | |
| *** jmlowe has quit IRC | 21:01 | |
| *** mriedem has quit IRC | 21:02 | |
| *** wolverineav has quit IRC | 21:06 | |
| *** wolverineav has joined #openstack-nova | 21:06 | |
| *** wolverineav has quit IRC | 21:07 | |
| *** wolverineav has joined #openstack-nova | 21:07 | |
| *** macza has joined #openstack-nova | 21:09 | |
| *** markvoelker has joined #openstack-nova | 21:11 | |
| *** igordc has joined #openstack-nova | 21:13 | |
| openstackgerrit | Jack Ding proposed openstack/nova master: Preserve UEFI NVRAM variable store https://review.openstack.org/621646 | 21:15 |
| *** jmlowe has joined #openstack-nova | 21:21 | |
| *** xek has quit IRC | 21:26 | |
| *** mriedem has joined #openstack-nova | 21:26 | |
| *** jmlowe has quit IRC | 21:26 | |
| *** psachin has quit IRC | 21:27 | |
| *** tobias-urdin has quit IRC | 21:32 | |
| *** igordc has quit IRC | 21:33 | |
| *** igordc has joined #openstack-nova | 21:40 | |
| *** igordc has quit IRC | 21:45 | |
| *** jmlowe has joined #openstack-nova | 21:50 | |
| *** igordc has joined #openstack-nova | 21:55 | |
| *** wolverineav has quit IRC | 21:57 | |
| *** wolverineav has joined #openstack-nova | 21:58 | |
| *** lpetrut has quit IRC | 22:00 | |
| *** wolverineav has quit IRC | 22:02 | |
| *** N3l1x has quit IRC | 22:04 | |
| *** dklyle has joined #openstack-nova | 22:05 | |
| *** markvoelker has quit IRC | 22:06 | |
| *** david-lyle has joined #openstack-nova | 22:09 | |
| openstackgerrit | Merged openstack/python-novaclient master: Fix flavor keyerror when nova boot vm https://review.openstack.org/582147 | 22:11 |
| openstackgerrit | Merged openstack/python-novaclient stable/rocky: Improve the description of optional arguments https://review.openstack.org/602056 | 22:11 |
| *** dklyle has quit IRC | 22:12 | |
| mriedem | melwitt: you want to just fast approve this backport https://review.openstack.org/#/c/623546/ for rocky and then we can queue up a rocky release with a couple of depends on | 22:15 |
| mriedem | the only other one i'm watching is https://review.openstack.org/#/c/623507/ | 22:15 |
| melwitt | ok. depends-on being this one I'd think? https://review.openstack.org/624982 | 22:17 |
| openstackgerrit | Merged openstack/nova master: Use external placement in functional tests https://review.openstack.org/617941 | 22:18 |
| *** radez has joined #openstack-nova | 22:20 | |
| *** rcernin has joined #openstack-nova | 22:21 | |
| *** david-lyle has quit IRC | 22:23 | |
| *** wolverineav has joined #openstack-nova | 22:29 | |
| *** wolverineav has quit IRC | 22:34 | |
| *** itlinux has quit IRC | 22:36 | |
| mriedem | melwitt: i would like to let that bake a bit | 22:37 |
| melwitt | k | 22:37 |
| mriedem | dan and i were talking about something in there as an unexpected side effect this morning | 22:37 |
| mriedem | hence the depends-on | 22:37 |
| mriedem | so just the docs change and lee's backport | 22:37 |
| melwitt | ok | 22:37 |
| *** lbragstad has quit IRC | 22:37 | |
| mriedem | if you're curious, the side effect is if the cell is configured to hit the api db, which lots of people still do for up-calls to work, and upgrade levels is 'auto', it means all cells will use the minimum compute version even if a particular cell is fully upgraded | 22:38 |
| mriedem | which is different from before | 22:38 |
| mriedem | not bad, but different | 22:38 |
| melwitt | I was, thanks for saving me from having to read back in the irc log | 22:40 |
| mriedem | Kevin_Zheng: i've got some questions/confusion on yikun's live migration completion timeout action patch if you want to kick him when you're in the office https://review.openstack.org/#/c/619143/ | 22:43 |
| *** mchlumsky has quit IRC | 22:45 | |
| *** igordc has quit IRC | 22:56 | |
| openstackgerrit | Merged openstack/os-resource-classes master: Tune up documentation to be more useful https://review.openstack.org/624384 | 22:57 |
| *** tbachman has quit IRC | 22:58 | |
| *** wolverineav has joined #openstack-nova | 23:04 | |
| mriedem | dansmith: you might have an opinion on the os-vif version side-step in this spec https://review.openstack.org/#/c/607610/ but i'm not sure it's really worth holding up on that | 23:08 |
| *** wolverineav has quit IRC | 23:08 | |
| dansmith | mriedem: yeah I think I saw talk of that recently | 23:10 |
| dansmith | I'm ambivalent, although amused/bemused that it's been however many years and we still aren't sharing objects with neutron | 23:11 |
| Kevin_Zheng | mriedem: ack | 23:11 |
| mriedem | we didn't do the port binding thing, talked about since ocata, until rocky, so yeah | 23:15 |
| *** sambetts_ has quit IRC | 23:19 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/queens: Make compute rpcapi version calculation check all cells https://review.openstack.org/625060 | 23:19 |
| *** sambetts_ has joined #openstack-nova | 23:22 | |
| *** wolverineav has joined #openstack-nova | 23:29 | |
| *** _alastor_ has joined #openstack-nova | 23:30 | |
| *** aloga has quit IRC | 23:32 | |
| *** aloga has joined #openstack-nova | 23:32 | |
| *** tbachman has joined #openstack-nova | 23:36 | |
| *** tbachman_ has joined #openstack-nova | 23:38 | |
| *** slaweq has quit IRC | 23:40 | |
| *** tbachman has quit IRC | 23:40 | |
| *** tbachman_ is now known as tbachman | 23:40 | |
| *** mriedem has quit IRC | 23:41 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!