| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-01-22 | |||
| 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 | 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 | |