Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-01-22
16:17:42 jaypipes zzzeek: another annoying this about oslo.cfg: you can't tell whether a CONF option has been set or not. i.e. has the CONF option value been set (via either argparse or configparser) to any value.
16:17:44 sean-k-mooney stephenfin: just looking at https://review.openstack.org/#/c/449257/52/nova/pci/request.py@93 the clean up you are asking for are not directly related to the patch rodlfo is doing so i should probably pull it out into another patch above or below rodolfos. is that ok? and do you have a prefernece?
16:17:55 jaypipes zzzeek: s/this/thing
16:18:28 zzzeek jaypipes: I might have observed the other day that putting something in [DEFAULT] doesn't actually "default" the value elsewhere? i struggled for days getting region_name / os_region_name setup to work
16:18:57 jaypipes zzzeek: yeah
16:19:13 bauzas does anyone know more than just by the name the "nova backup" command ?
16:19:33 zzzeek jaypipes: and another thing! why is the keystone auth URL in nova.conf but then if my region is wrong it seems to ask keystone for the *other* URL then it starts using that one (and fails) ?
16:19:37 bauzas I guess the retention period is just for calculating whether we should delete an old image or not
16:19:56 bauzas but hopefully nova isn't periodically running a task to clean that up
16:20:13 bauzas (ie. that's a per-call retention calculation)
16:20:18 stephenfin sean-k-mooney: What cleanup is this now?
16:20:44 bauzas I'm just amazed to discover such API after 5 years working for Nova
16:21:17 bauzas nevermind, got my answer https://elastx.se/en/blog/backups-openstack-cloud
16:21:22 sean-k-mooney stephenfin: basically the comment here https://review.openstack.org/#/c/449257/52/nova/pci/request.py@93
16:22:28 stephenfin sean-k-mooney: Aye, the lines I actually commented on are a cleanup task. However, my comments about what the object should actually look like are not
16:22:32 sean-k-mooney stephenfin: they are against lines not modified by rodolfos patch so im wondering should it be a seperate patch or in this one
16:22:50 mriedem bauzas: note you can't use backup with a volume-backed instance, but you can snapshot a volume-backed instance, another fun wrinkle, and something people have wanted to fix for probably 5 years as well
16:23:00 stephenfin sean-k-mooney: What I was suggesting is that rather than having two generic container fields on the object, we actually define the fields we want
16:23:14 bauzas mriedem: but you can backup a volume, right?
16:23:22 bauzas not with nova, of course
16:23:30 stephenfin sean-k-mooney: The two generic container fields being 'dict_of_lists' and 'dict_of_strings'
16:23:53 bauzas anyway, I'm just testing my patches against https://developer.openstack.org/api-guide/compute/server_concepts.html#server-actions
16:24:06 bauzas whatever the backup does, it works
16:24:11 bauzas period.
16:24:23 mriedem backup creates a snapshot with a rotating retention period yeah
16:24:36 jaypipes zzzeek: see: vestigial tail? :)
16:24:38 stephenfin sean-k-mooney: Rather than using 'dict_of_lists', add a ListOpt for each key that we'd expect to store in there. Similarly, instead of 'dict_of_strings', add a ListOpt for each key
16:24:41 stephenfin Does that make sense?
16:25:20 bauzas mriedem: it tags the snapshot, I see
16:25:34 bauzas mriedem: my question was just about what was enforcing that retention
16:25:39 bauzas but that's fine, I see that
16:25:45 sean-k-mooney ok i can do that would you like it in this patch or a seperate one
16:26:28 stephenfin sean-k-mooney: That one, please. If not, we're going to have to immediately issue a MAJOR version bump on the object so we can remove the 'dict_of_*' fields
16:26:36 stephenfin Which seems awfully silly :)
16:27:24 sean-k-mooney stephenfin: the network capablites will still need to remain a list of stings in the object as it can technicall by any trait includeing custom_ ones
16:27:37 sean-k-mooney stephenfin: ok will do.
16:27:56 stephenfin sean-k-mooney: Yup, I'd expect to see 'capabilities = ListOpt(...)'
16:28:00 stephenfin sean-k-mooney: Shhhhooound
16:29:01 sean-k-mooney am that might also want to be an object actully capablities:{ network = listOpts(...);}
16:29:21 sean-k-mooney for pci device we could have compute or other non netowrk capablites later
16:30:53 openstackgerrit Matt Riedemann proposed openstack/python-novaclient master: Microversion 2.59 - Migrations list pagination https://review.openstack.org/516168
16:30:55 sean-k-mooney e.g. gpus could have opengl/cuda level as a trait but they are still pcidevices.
16:31:23 lyarwood kashyap: random question, have you ever had to file a bug against libvirt as shipped in UCA?
16:31:51 kashyap lyarwood: Good random question
16:32:00 kashyap lyarwood: I haven't, but what I did was---
16:32:03 openstackgerrit Ed Leafe proposed openstack/nova master: Change compute RPC to use alternates for resize https://review.openstack.org/526436
16:32:08 lyarwood kashyap: hitting this http://logs.openstack.org/50/536350/1/check/legacy-grenade-dsvm-neutron-multinode-live-migration/5f7ed57/logs/screen-n-cpu.txt.gz?level=ERROR#_Jan_22_15_09_28_467904 that I think is the result of libvirt being compiled without gnutls installed, somehow.
16:32:10 kashyap Jump on their IRC channel, and some friendly Ubuntu person did i
16:32:13 edleafe mriedem: jaypipes: ^^ rebased w/new RPC version
16:32:15 kashyap s/did i/it/
16:32:29 lyarwood kashyap: kk, which channel?
16:32:37 edmondsw zzzeek if you're referring to the keystone_authtoken section, keystone url has to be there so that nova knows how to talk to keystone at all
16:32:38 kashyap lyarwood: #ubuntu-dev
16:32:49 kashyap lyarwood: If it's a packaging but, #ubuntu-packaging
16:32:52 kashyap s/but/bug/
16:33:11 zzzeek edmondsw: yes, but I have observed that once it does that initial talk to keystone thing, it looks into endpoints and gets whatever it finds there for keystone, then starts talking to that
16:33:19 kashyap Why are they compiling _out_ GnuTLS? Libvirt depends on it, as you know
16:33:39 edmondsw zzzeek yep. Which should be all well and good unless you've got something misconfigured
16:33:48 lyarwood kashyap: no idea :)
16:34:20 kashyap lyarwood: Try if you can find Christian Ehrhardt from Canonical on those channels
16:34:30 kashyap lyarwood: He's the active libvirt dev from Canononical that I see
16:34:38 kashyap (On the upstream lists)
16:34:57 zzzeek edmondsw: sure. these things are just v hard to debug in an HA setup
16:35:54 edmondsw zzzeek yeah :( I'm not sure the exact reason there (or if there is a good reason). lbragstad would have a better idea
16:36:31 openstackgerrit Merged openstack/python-novaclient master: Add support for the 2.57 microversion https://review.openstack.org/528128
16:36:32 openstackgerrit Merged openstack/python-novaclient master: Microversion 2.58 - Instance actions list pagination https://review.openstack.org/528601
16:37:29 zzzeek edmondsw: well w regions I guess you are saying "here we'd prefer you use this address for this service" so even though you already gave it a "bootstrap" keystone, sure I'd probably like that it does that. depends on wht you're using regions for
16:38:14 edmondsw zzzeek yeah, regions get all kinds of funky
16:41:36 kashyap lyarwood: Probably best to report it on the tracker, to save your time
16:41:46 openstackgerrit Jackie Truong proposed openstack/python-novaclient master: Microversion 2.60 - Add trusted_image_certificates https://review.openstack.org/500396
16:41:47 kashyap lyarwood: I see UCA libvirt bugs are filed just "normally" like this -- https://bugs.launchpad.net/fuel/+bug/1670220
16:41:49 openstack Launchpad bug 1670220 in Fuel for OpenStack ocata "Libvirt service name is different for UCA" [High,Fix committed] - Assigned to Vladimir Kuklin (vkuklin)
16:42:17 lyarwood kashyap: kk, I created https://bugs.launchpad.net/ubuntu/+source/libvirt/+bug/1744758
16:42:18 openstack Launchpad bug 1744758 in libvirt (Ubuntu) "libvirt 2.5.0 appears to be compiled without gnutls" [Undecided,New]
16:43:26 kashyap lyarwood: Yeah, checking w/ a libvirt upstream dev, seems there's no 'special' flag, but certainly something is wonky in the build
16:43:31 kashyap So your bug looks fine
16:45:26 lyarwood kashyap: yeah, there's no flag for this, we just check if gnutls_cipher_encrypt is available during the build.
16:45:32 lyarwood kashyap: thanks for asking :)
16:47:05 jaypipes lyarwood: quick ? for you on https://review.openstack.org/#/c/460243/
16:47:24 lyarwood jaypipes: looking
16:50:53 lyarwood jaypipes: yeah correct, _get_guest_xml calls down into _get_guest_storage_config that eventually calls _connect_volume
16:52:50 bauzas cdent: saw your -1 but that seems a bit harsh given it's for grammatical issues
16:53:06 bauzas cdent: would you like to see the fixes by a follow-up ?
16:53:13 bauzas cdent: context is https://review.openstack.org/#/c/533642/5
16:53:31 cdent bauzas: is fine as a followup, but it seemed pretty relevant since it was a release note
16:53:47 cdent however, since there are later changes to the same release note, followups would be fine
16:53:54 dansmith bauzas: I think getting the reno right is worthwhile, FWIW
16:54:13 dansmith it's not just one typo
16:54:21 bauzas dansmith: cdent: okay, I'm just afraid of any possible gate slowness so close by the milestone
16:54:31 bauzas but I'll spin another rev
16:54:42 dansmith bauzas: I'll pounce on it as soon as you do
16:54:59 bauzas I'll also clarify what has been tested and what not
16:56:45 cdent jaypipes: feh, I struggle to remember '"special" filters and weighers'
16:57:17 bauzas also, I'm interested in getting some Oxford/Thesaurus link about the difference between a "this" and a "that" :p
16:57:44 bauzas because we don't have that with "ce truc" :p
16:58:10 cdent this is here, that is there
16:58:13 bauzas but we have a "tu/vous" rule that is silly
16:58:23 cdent english is full of silly
16:58:25 dansmith cdent: heh, was trying to think of a way to explain that
16:58:51 bauzas I have a 7-yr old daughter just learning French grammar

Earlier   Later