| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-01-14 | |||
| 16:45:28 | sean-k-mooney | and you get all the qos oplicys or none today. i honestly did not look quite that closely at least not in a few years | |
| 16:47:03 | gibi | sean-k-mooney: I've never looked into it deeply. I rely on rubasov's and lajoskatona's neutron experties | |
| 16:48:07 | ade_lee | hi all -- we have a grenade test that keeps failing due to an error which is related to volume attachment. Is there anyone here that knows about how nova volume attach works and can help? | |
| 16:49:33 | jangutter | moshel, lennyb: I suspect the OVS_HW_offload CI is having some issues. Would you be able to check it? | |
| 16:50:48 | ade_lee | specifically, we're running into an error where we try to write to a volume before its ready -- | |
| 16:51:33 | ade_lee | we do wait for cinder to show the volume as being "in-use", but that seems to be insufficient. | |
| 16:51:59 | ade_lee | what would we wait for to ensure that the volume is ready for use? | |
| 16:52:09 | ade_lee | and the instance knows about it. | |
| 17:01:04 | stephenfin | bauzas: I think you need to submit those gantt reviews by yourself, as it's just yourself and johnthetubaguy that have access. melwitt is OK with us retiring it | |
| 17:01:23 | bauzas | stephenfin: WDYM ? | |
| 17:01:28 | bauzas | fast-approving ? | |
| 17:01:33 | stephenfin | yup | |
| 17:02:15 | stephenfin | bauzas: Pretty much everyone else is no longer working on OpenStack https://review.openstack.org/#/admin/groups/253,members | |
| 17:02:16 | sean-k-mooney | stephenfin: well you can submit the change and bauzas and johnthetubaguy can approve | |
| 17:02:31 | mdbooth | Are folks aware that openstack-placement isn't listed in either requirements or test-requirements, btw? | |
| 17:02:47 | stephenfin | mdbooth: Neither is neutron, I'd imagine | |
| 17:03:05 | mdbooth | stephenfin: How's it supposed to be installed? | |
| 17:03:12 | stephenfin | They're all applications rather than libraries. You don't list applications in requirements.txt | |
| 17:03:32 | mdbooth | stephenfin: We pull in placement in functional | |
| 17:03:33 | stephenfin | If you're DevStack, usually by cloning the repo locally? | |
| 17:03:41 | mdbooth | Functional tests don't work | |
| 17:03:42 | stephenfin | Oh, then that's different | |
| 17:03:55 | sean-k-mooney | stephenfin: it should be in test-requiremetns if its needed to run the tests | |
| 17:04:24 | stephenfin | sean-k-mooney: Yup. It shouldn't be required to run tests though, otherwise we now have integration tests | |
| 17:04:38 | stephenfin | mdbooth: Then I can't offer any more info. Sorry for the noise :) | |
| 17:05:35 | sean-k-mooney | stephenfin: we import the test fixture form placement now so we can share it between nova and placement i think | |
| 17:05:36 | mdbooth | https://github.com/openstack/nova/blob/master/nova/tests/functional/fixtures.py#L19 | |
| 17:05:49 | sean-k-mooney | cdent: ^ you would know better then i | |
| 17:06:18 | cdent | in the gate, it is happening a a result of tox-siblings | |
| 17:06:27 | openstackgerrit | Jan Gutter proposed openstack/nova master: Convert vrouter legacy plugging to os-vif https://review.openstack.org/571325 | |
| 17:06:31 | jangutter | mriedem: would you be able to remove your procedural -2 from ^^ (a tag has been made!) | |
| 17:06:32 | mdbooth | Sounds cosy | |
| 17:07:19 | sean-k-mooney | cdent: how would you run tox locally to get the same effect. just pip install placement after the fact? | |
| 17:07:27 | cdent | one mo | |
| 17:08:02 | cdent | sean-k-mooney, mdbooth : look at the functional job def in tox.ini | |
| 17:08:21 | cdent | placement is installed explicitly for functional only because unit tests should not know about placement | |
| 17:08:31 | cdent | from git master | |
| 17:08:43 | cdent | tox-siblings makes sure in the gate that depends-on type things are respected | |
| 17:08:46 | mriedem | jangutter: done | |
| 17:08:59 | cdent | if you need that locally, then pip install the git ref | |
| 17:08:59 | sean-k-mooney | cdent: mdbout is have import errors when runing placement | |
| 17:09:07 | sean-k-mooney | * nova fucntional tests | |
| 17:09:30 | cdent | mdbooth: got any more details than that? | |
| 17:09:34 | jangutter | mriedem: thanks! (finally, the dominoes have started falling!) | |
| 17:10:35 | mdbooth | cdent: Ah, ha! | |
| 17:10:49 | mdbooth | I was running functional-py37 which looks like it's missing a line | |
| 17:11:16 | cdent | mdbooth: it does indeed appear to be missing a line | |
| 17:11:18 | mdbooth | py3{5,6} both have an explicit deps= referring to functional deps | |
| 17:11:21 | cdent | i bet that's a bad merge | |
| 17:11:34 | cdent | functiona-py37 was added around the same time the functional test changes were added | |
| 17:11:43 | cdent | but functional-py37 is not in the gate | |
| 17:12:01 | cdent | mdbooth: you happy to fix that as part of whatever else you are doing, or would like me to get it? | |
| 17:12:26 | mdbooth | cdent: 'whatever I'm doing' will merge shortly after the decay of the last proton, so you go ahead | |
| 17:15:52 | jangutter | mdbooth: you made me look up the current lower limit for proton decay. I think the only think that will beat that is COBOL code running in production. | |
| 17:16:22 | mdbooth | jangutter: lol | |
| 17:18:21 | gibi | mlavalle, sean-k-mooney, efried, mriedem: So the (hopefully) final timeslot for the bandwidth hangouts in Friday 17:00 UTC. Posted on the ML | |
| 17:18:32 | gibi | http://lists.openstack.org/pipermail/openstack-discuss/2019-January/001710.html | |
| 17:18:35 | mlavalle | gibi: ack | |
| 17:18:49 | efried | ack | |
| 17:20:59 | efried | gibi: Hangout link to be provided day-of? | |
| 17:29:20 | openstackgerrit | Chris Dent proposed openstack/nova master: Make functional-py37 job work like others https://review.openstack.org/630745 | |
| 17:41:21 | stephenfin | sean-k-mooney: thoughts on https://review.openstack.org/#/c/621528/5/nova/network/linux_net.py@1455 ? | |
| 17:42:47 | sean-k-mooney | exitcode 2 is file not found or interface not found in this case | |
| 17:42:58 | sean-k-mooney | 254 im not sure and 0 is success | |
| 17:43:38 | sean-k-mooney | what is that patch for | |
| 17:44:25 | sean-k-mooney | oh its the privesep changes im not sure if that code is used as os-vif should be used even for nova-networks | |
| 17:45:05 | stephenfin | sean-k-mooney: Aye, could be nova-network only code | |
| 17:45:37 | sean-k-mooney | you mean unrelated to vif plugging | |
| 17:46:25 | stephenfin | No I mean the file I'm linking to seems to be nova-network only code and unrelated to the neutron/os-vif code path | |
| 17:46:34 | stephenfin | We don't use os-vif for nova-net, do we? | |
| 17:46:39 | sean-k-mooney | yes we do | |
| 17:47:08 | sean-k-mooney | os-vif should be used for both nova-networks and neutorn in the libvirt dirver | |
| 17:47:58 | sean-k-mooney | this is the legacy linux bridge code path which i think is unused currently | |
| 17:48:43 | sean-k-mooney | in any case you are coorect that its a change in behavior but i think exit codes [0,2,254] would have been the correct set to use in this case | |
| 17:52:23 | jaypipes | efried: got a sec? | |
| 17:52:26 | jaypipes | efried: re: https://review.openstack.org/#/c/617042/14/nova/compute/manager.py@555 | |
| 17:52:36 | jaypipes | efried: I'm still super-confused why we need that. | |
| 17:52:55 | jaypipes | efried: and why we can't just instantiate the reportclient variable on __init__ of the compute manager | |
| 17:53:25 | jaypipes | efried: is this to minimize the number of ProviderTree caches or something? | |
| 18:06:12 | jaypipes | efried: never mind... I see you've addressed it in https://review.openstack.org/#/c/620711/13 | |
| 18:16:19 | mriedem | it's funny that an az description field has never been requested for the api | |
| 18:25:10 | jaypipes | mriedem: perhaps because nobody knows wtf an AZ is in Nova? | |
| 18:31:46 | sean-k-mooney | jaypipes: its like a aws AZ right with all the semantics or a totally seperate fault domain and is tototally not a metadata key on an aggregate | |
| 18:32:02 | efried | jaypipes: sorry was afk. Assume any remaining action items are denoted in review comments at this point? | |
| 18:36:28 | jaypipes | efried: yuupers. | |
| 18:36:31 | jaypipes | yuppers | |
| 18:36:46 | jaypipes | sean-k-mooney: lol, yeah right :) | |
| 18:36:55 | efried | jaypipes: Thanks for the review. I'm pretty excited to get this series going. | |
| 18:37:30 | efried | cdent: tape that ^ | |
| 18:40:54 | eandersson | Are there any actual dependencies on the new libvirt / qemu versions for nova-compute? | |
| 18:44:56 | cdent | efried: no sir | |
| 18:54:43 | lennyb | jangutter, moshele. yes. internal environment . I will check it | |
| 18:55:17 | moshele | lennyb: ? | |
| 18:56:58 | mriedem | jaypipes: heh, well, all the more reason to allow a cloud operator to describe their AZs | |
| 18:57:52 | mriedem | eandersson: what do you mean? the libvirt driver has a minimum required libvirt/qemu otherwise nova-compute won't start | |
| 18:58:07 | mriedem | we bump that minimum every other release or so | |
| 18:58:45 | mriedem | some features in the libvirt driver require newer versions of the binaries and those have conditional version checks on them | |
| 18:58:59 | mriedem | e.g. file-backed memory | |
| 19:41:59 | openstackgerrit | Adrian Chiris proposed openstack/nova master: Add free for claimed, allocated devices https://review.openstack.org/616120 | |
| 19:42:00 | openstackgerrit | Adrian Chiris proposed openstack/nova master: Allow per-port modification of vnic_type and profile https://review.openstack.org/607365 | |