Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-01-22
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
16:59:12 bauzas never say English is full of silly, or I'll just compare with French
16:59:24 efried Most Americans never learn English grammar, so...
17:00:18 efried bauzas this ~= celui-çi; that ~= celui-là
17:00:35 openstackgerrit Jim Rollenhagen proposed openstack/nova master: Ironic: negotiate microversion to allow downgrade to Pike https://review.openstack.org/535786
17:00:35 bauzas this = person, that = object ?
17:00:40 efried no
17:00:49 efried just ceci vs cela
17:00:55 bauzas mmmm
17:01:10 jroll hey (again) nova friends, the ironic rolling upgrade testing is down. while we try to track down why nova-conductor segfaults after upgrading ironic (without restarting n-cond), this patch will help us work around it by allowing nova queens to work with ironic pike (which is a good thing for users anyway). reviews would be super helpful, thank you :) https://review.openstack.org/#/c/535786/
17:01:50 efried bauzas Mais je peux voir que "ça" peut servir pour les deux
17:02:00 bauzas efried: 'Remove that code' looks to be wrong
17:02:06 bauzas that means "ce" to me
17:02:22 bauzas anyway
17:02:29 bauzas just rebasing now
17:02:53 mriedem edleafe: ok comments inline
17:03:17 efried bauzas Depends on the context. And it's definitely subtle. If I highlighted a chunk and made a gerrit comment on it, I would sai "Remove this code". If I was talking about some code somewhere else, or in the future, I would say "Remove that code".
17:05:50 bauzas efried: I see, it's more the intent itself

Earlier   Later