Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-04-11
19:12:30 mordred dansmith: it's 19:12 UTC, which everyone knows is crack-o'clock
19:12:38 dansmith obvi
19:16:27 mordred of course!
19:16:40 mordred dansmith: is there some other type I should know about?
19:16:54 dansmith mordred: nah, stick to the safe stuff
19:18:56 openstackgerrit Matt Riedemann proposed openstack/nova master: Deprecate CONF.api.allow_instance_snapshots https://review.openstack.org/455100
19:27:42 openstackgerrit Mathieu Gagné proposed openstack/nova-specs master: Nova support for attached volume extend https://review.openstack.org/453272
19:35:09 mgagne gibi: mriedem asked me to ask you about your input/opinion on notification impact on this proposed spec: https://review.openstack.org/#/c/453272/
19:35:23 mgagne gibi: in PS3: "Hmm, this seems like something for which we should send a notification. I don't think we send notifications for any external events that we handle today though. It might be worth talking to gibi about this. I wouldn't block this spec on it, but it seems useful."
19:39:28 cfriesen does anything test with "force_raw_images" set to false, or with glance image formats other than raw/qcow2 ?
19:41:00 mriedem of course not http://codesearch.openstack.org/?q=force_raw_images&i=nope&repos=nova
19:41:12 mriedem that reminds me, i noticed we don't test with that as False once
19:43:51 cfriesen it seems to work okay for raw/qcow2, but apparently it doesn't work with a vmdk format image file. (admittedly a qcow2 image with a vmdk backing file is a bit odd, but we don't do anything to prevent it)
19:45:41 mriedem cfriesen: did you mean integration test? because we don't even test that with a unit test
19:45:55 mriedem vmdk with libvirt imagebackend?
19:46:38 mriedem vmdk is all vmware
19:46:45 mordred cfriesen: we do vhd uploads in production
19:47:19 mordred but that's not helpful for gate testing :)
19:47:48 openstackgerrit Matt Riedemann proposed openstack/nova master: Add uuid to Service model https://review.openstack.org/453025
19:47:49 openstackgerrit Matt Riedemann proposed openstack/nova master: Add online data migration for populating services.uuid https://review.openstack.org/454899
19:48:51 openstackgerrit Jay Pipes proposed openstack/nova master: Read Neutron port 'binding_profile' during boot https://review.openstack.org/449257
19:49:09 openstackgerrit Jay Pipes proposed openstack/nova master: Modify PciPassthroughFilter to accept lists https://review.openstack.org/444946
19:49:21 openstackgerrit Jay Pipes proposed openstack/nova master: Read Neutron port 'binding_profile' during boot https://review.openstack.org/449257
19:53:27 mriedem cfriesen: you've got a little unit test failure to fix up in https://review.openstack.org/#/c/447055/
19:59:24 cfriesen mriedem: okay, will sort it out. I'm putting out customer fires.
20:00:22 cfriesen mriedem: mordred: apparently libvirt/qemu will boot a vmdk-format glance image.
20:01:31 mriedem https://en.wikibooks.org/wiki/QEMU/Images#Image_types
20:02:01 mriedem i don't see ploop in that list
20:02:16 mordred cfriesen: fascinating
20:03:04 cfriesen I have a report that it boots up, but then fails when we try to live-migrate the instance.
20:03:26 cfriesen I'll raise a bug once we have the info nailed down a bit more securely
20:03:50 mriedem please include customer credit card numbers in the bug report thanks
20:04:00 cfriesen wouldn't that be nice. :)
20:04:26 cfriesen not actually expecting it to get fixed, but at least someone else can see the bug report and know it's broken. :)
20:07:21 jaypipes wait, mriedem, your customers pay you?
20:07:53 mriedem i don't have customers
20:08:47 d9k is there any code for different components to share locks?, i dont see anything
20:09:29 d9k also oslo, which uses fasteners does not have any code for shared locks either
20:10:26 d9k i posted earlier about a race nova and neutron introduce with iptables and ipsets, both need to acquire shared locks from each other
20:11:06 sdague d9k: there is no mechanism for that today, is there an open bug on that issue?
20:16:54 d9k no not yet, acquiring neutron-ipset lock in the iptables-save/restore bit fixes https://ptpb.pw/16S8, but https://ptpb.pw/aukQ needs locks too
20:19:59 mriedem http://forumtopics.openstack.org/cfp/details/11
20:21:51 d9k also this happens fairly often when running tempest compute.servers
20:30:13 sdague d9k: a bug with details would be good, sounds like you've done a lot of the debugging, and it would be great to get all those details down
20:38:56 swebster Has anybody run the latest py27 unit tests on a Centos system? I'm getting a failure on the nova.tests.unit.objects.test_objects.TestObjectVersions.test_versions test which seems to be related to the recent "Use cursive for signature verification" change: https://review.openstack.org/#/c/351232/. The test passes on my Ubuntu Xenial box, with cursive versions both at 0.1.2.
20:44:57 mriedem swebster: it's likely something in the cursive library, what is the actual error? (in a paste)
20:46:10 swebster mriedem: Here's a paste of the error(s) seen https://pastebin.com/1rqJaTPs
20:48:55 mriedem my guess would be something with this https://github.com/openstack/cursive/blob/master/cursive/signature_utils.py#L177
20:49:08 mriedem that changes what's in REGISTERED_TYPES
20:49:14 mriedem which changes our enum list in nova
20:50:48 mriedem this https://github.com/openstack/nova/blob/master/nova/objects/fields.py#L430
20:51:17 mriedem swebster: to find out, import cursive 0.1.2 in centos and xenial and compare signature_utils.SignatureKeyType.REGISTERED_TYPES.keys()
20:54:32 openstackgerrit Matt Riedemann proposed openstack/nova-specs master: Deprecate multinic, os-virtual-interfaces, and floating IP action APIs https://review.openstack.org/384261
20:54:41 mriedem sdague: jaypipes: mordred: alex_xu: johnthetubaguy: ^ there you go
20:54:45 mriedem big bag of deprecation
20:55:09 jaypipes mriedem: cool.
20:55:29 swebster mriedem: Very right. Centos: ['RSA-PSS', 'ECC_SECP521R1', 'ECC_SECP384R1', 'DSA']. Ubuntu: ['RSA-PSS', 'ECC_SECP521R1', 'ECC_SECT409R1', 'ECC_SECT409K1', 'ECC_SECT571R1', 'ECC_SECP384R1', 'ECC_SECT571K1', 'DSA']
20:56:46 mriedem swebster: that's super fun
20:56:51 mriedem they used to be hardcoded https://review.openstack.org/#/c/351232/16/nova/objects/fields.py
20:56:56 mriedem before we were using cursive
20:57:40 mriedem swebster: please report a bug
20:57:48 mriedem https://bugs.launchpad.net/nova/
20:57:51 swebster mriedem: And now it looks like they could be different system to system. Will do, and thanks
20:58:25 mriedem yeah that sucks, and i was afraid of something like that happening
20:58:33 mriedem we should probably just hard-code the superset
21:01:46 mriedem melwitt: so you hit "sphinx.errors.SphinxWarning: WARNING: no "man_pages" config value found; no manual pages will be written" right?
21:01:49 mriedem did you just recheck it?
21:02:04 melwitt mriedem: cdent rechecked it and it failed again
21:03:07 mriedem well that's just great
21:03:12 mriedem i'm going to go eat my feelings
21:13:22 dansmith last time I tasted feelings they had a distinct "like butt" quality
21:13:33 dansmith but, I'm not very adventurous with my mouth
21:13:39 dansmith er, tastes
21:25:02 mriedem these feelings tasted like pretzels
21:36:47 mriedem ok, on to writing a spec to deprecate os-hosts now?!
21:48:42 dansmith mriedem: uuid should be nullable on the model but not on the object right? https://review.openstack.org/#/c/453025/6/nova/objects/service.py
21:49:20 dansmith AFAICT, there's no need for it to actually be None anywhere
21:49:28 dansmith but if it's nullable everything needs to be defensive
21:52:15 openstackgerrit Matt Riedemann proposed openstack/nova master: Add uuid to Service model https://review.openstack.org/453025
21:52:18 mriedem yes
21:52:20 mriedem you're right
21:52:48 mriedem compute nodes uuid is read-only https://github.com/openstack/nova/blob/master/nova/objects/compute_node.py#L55
21:52:50 mriedem but not nullable
21:53:14 mriedem aggregate uuid is nullable=False, but not read-only https://github.com/openstack/nova/blob/master/nova/objects/aggregate.py#L244
21:53:34 dansmith yeah
21:53:48 dansmith there was a reason for the agg one not being read-only
21:53:56 mriedem so should be nullable=False and read_only=True?
21:54:17 dansmith tbh, just make it nullable=False (i.e. don't specify it)
21:54:22 dansmith the readonly has been problematic recently,
21:54:25 mriedem ok
21:54:37 dansmith so I'd leave that off, which is why it is in aggregate
21:57:11 openstackgerrit Eric Fried proposed openstack/nova master: PowerVM Driver: spawn/destroy #2: functional https://review.openstack.org/438598
21:57:35 efried mriedem ^^ back atcha.
21:58:01 efried Still a couple uncertainties inline.
21:59:26 openstackgerrit Matt Riedemann proposed openstack/nova master: Add uuid to Service model https://review.openstack.org/453025
21:59:26 openstackgerrit Matt Riedemann proposed openstack/nova master: Add online data migration for populating services.uuid https://review.openstack.org/454899
22:36:03 mriedem mgagne: http://lists.openstack.org/pipermail/openstack-dev/2017-April/115292.html
22:37:03 mriedem browne: efried: BobBall: would be good to check out https://review.openstack.org/#/c/453272/ and see if vmware, powervm and xen drivers can support this API eventually
22:37:08 mgagne mriedem: Thanks for starting the conversation and doing proper follow up
22:37:11 mriedem claudiub: you too but i think hyperv already does

Earlier   Later