Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-01-22
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 bauzas this = person, that = object ?
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: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
17:06:12 bauzas I'll try to play with this
17:06:17 bauzas (and not that)
17:06:39 bauzas thanks
17:06:48 efried Bonne chance.
17:06:55 edleafe mriedem: will review in a bit
17:07:00 efried I'll argue English-is-sillier-than-French any day.
17:11:24 jaypipes lyarwood: thx. just wanted to be sure.
17:16:26 mriedem stvnoyes: your swap volume multiattach tempest test patch is a merge conflict with the resize test, i'm going to rebase those so they are in a series, otherwise i can't test them together
17:17:05 stvnoyes ok
17:19:45 openstackgerrit Eric Berglund proposed openstack/nova master: PowerVM Driver: SEA https://review.openstack.org/523216
17:22:31 esberglu gibi: Removed the nova-net references from ^ so it's ready whenever you have a chance. Thanks for the reviews!
17:25:16 openstackgerrit Matt Riedemann proposed openstack/nova master: [api] Allow multi-attach in compute api https://review.openstack.org/271047
17:25:17 openstackgerrit Matt Riedemann proposed openstack/nova master: Add the nova-multiattach job https://review.openstack.org/532689
17:25:22 mriedem ah crap
17:25:45 mriedem gibi: accidental rebase on https://review.openstack.org/#/c/271047/ - want to +W again?
17:32:47 dansmith jroll: is that a thing you want separate from working around the segv issue? because I thought we always expect ironic to be newer and this changes that
17:34:24 jroll dansmith: yes, we want to be able to go one release either way between ironic and nova. click the bug there, there's a plan where we drop most of the negotiation in the client to do so, this is just the quick way to do it
17:35:07 openstackgerrit Merged openstack/nova master: doc: Document TLS security setup for noVNC proxy https://review.openstack.org/500544
17:35:17 mriedem efried: thanks for running the nova meeting last week
17:35:49 efried mriedem Sure thing, for what it was worth :) I didn't update the agenda or anything, so it was a bit slipshod.
17:36:10 mriedem was fine
17:36:35 mriedem you are now, incidentally, the PTL for Rocky, congratulations; it was a trap.
17:37:00 dansmith woohoo!
17:37:02 efried hahahaha, so THAT's all it takes?
17:37:36 dansmith jroll: mkay, I'm not sure how I feel about this.. the mechanism hasn't changed since this was titled "hacky negotiation to let us do a thing" and ... I agree
17:38:38 jroll dansmith: I mean, it's a bit of a hack but it works. the client patches are nearly done to do the better way, but it likely won't make this cycle. our options are basically do this or release without tested rolling upgrades, AIUI
17:38:58 dansmith mriedem: you marked this bug as wishlist so I assume you're okay with it? (having a random previous ironic microversion the driver is willing to use)
17:39:19 dansmith jroll: because some library is crashing conductor?
17:39:30 jroll dansmith: suggestions welcome. especially if you can figure out the conductor exploding
17:40:05 dansmith jroll: everything we add after this will have to either bump that "previous" thing or add infrastructure and handling for the newer feature, IFF the version is new enough
17:40:44 edleafe efried: it's so cute that you think mriedem is kidding.
17:40:57 dansmith jroll: is it some action that causes conductor to crash and is a restart enough? It's odd since conductor shouldn't be related to ironic at all
17:41:09 mriedem dansmith: which bug?
17:41:55 dansmith mriedem: well, it's a combo thing.. they want "N-1 ironic support in the driver" as well as this configdrive optionality which uses it, which I guess works around some other unrelated issue by chance
17:42:02 dansmith mriedem: https://review.openstack.org/#/c/535786/5
17:42:23 jroll dansmith: debugging notes so far are at line 47 here: https://etherpad.openstack.org/p/IronicWhiteBoard . as far as we can tell, upon upgrade, some shared library is being deleted, that is being loaded at runtime in the conductor
17:42:32 jroll see also https://bugs.launchpad.net/ironic/+bug/1744139
17:42:33 openstack Launchpad bug 1744139 in Ironic "Grenade job fails due to systemd stopping n-cpu" [Critical,In progress] - Assigned to Julia Kreger (juliaashleykreger)
17:42:54 dansmith jroll: right, so just a restart of the service because of that issue would be all we need right?
17:43:12 dansmith that seems like a reasonable thing to have in a from-foo script in grenade with a note about the shared library thing
17:43:16 jroll dansmith: pike conductor won't start with queens version of global-reqs

Earlier   Later