| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-10-18 | |||
| 17:30:44 | jmccarthy | sean-k-mooney: Not sure I follow here about proxy service that is needed on xen compute ? | |
| 17:31:28 | cdent | thanks efried will pay real attention a bit later | |
| 17:31:42 | sean-k-mooney | jmccarthy: actully it does not need to be on the computes. for kvm we deploy nova_novncproxy on the contoler and it connect to the hypervisor vnc console and proxyies it to horizon via a websocket | |
| 17:31:48 | efried | cdent The bug doesn't have anything you don't already know. I'll post the reviews in a bit and poke ya. | |
| 17:32:04 | cdent | s’cool, good to have on radar | |
| 17:32:56 | jmccarthy | sean-k-mooney: Ah ok yep I have that - I have both types of computes in this setup actually | |
| 17:34:07 | sean-k-mooney | jmccarthy: ya so backically in the nova compute nodes you have to add eitehr a vnc or spice section to the nova.conf or nova-compute.conf that points the hypervior at the proxy | |
| 17:34:44 | sean-k-mooney | jmccarthy: somthing like this http://paste.openstack.org/show/624007/ | |
| 17:34:45 | mriedem | need another specs core on this one https://review.openstack.org/#/c/509326/ | |
| 17:35:04 | jmccarthy | sean-k-mooney: I think I have that tho ? The console in horizon is working for instances on kvm compute | |
| 17:35:46 | sean-k-mooney | jmccarthy: is xen configured to use a xen console for guests or a vnc console. you will have to match them up | |
| 17:38:07 | sean-k-mooney | jmccarthy: are you using xen via libvirt | |
| 17:38:37 | jmccarthy | sean-k-mooney: yep | |
| 17:38:38 | sean-k-mooney | jmccarthy: if so dump the xml of the xen instance and check if its using vnc like kvm or is it using the xen console | |
| 17:39:15 | sean-k-mooney | you have to pass the -c xen:// option to virsh to see the instance by default | |
| 17:39:52 | sean-k-mooney | so virsh -c xen:// list then virsh -c xen:// dumpxml <domain name> | |
| 17:40:36 | jmccarthy | sean-k-mooney: I see this console bit ? http://paste.openstack.org/show/624009/ | |
| 17:41:09 | sean-k-mooney | jmccarthy: can you show me the full xml | |
| 17:42:02 | sean-k-mooney | jmccarthy: it should look something like http://paste.openstack.org/show/623903/ | |
| 17:42:12 | jmccarthy | sean-k-mooney: http://paste.openstack.org/show/624010/ | |
| 17:42:52 | sean-k-mooney | <graphics type='vnc' port='5900' autoport='yes' listen='0.0.0.0' keymap='en-us'> | |
| 17:42:54 | sean-k-mooney | <listen type='address' address='0.0.0.0'/> | |
| 17:42:56 | sean-k-mooney | </graphics> | |
| 17:42:58 | sean-k-mooney | so its using vnc | |
| 17:43:09 | sean-k-mooney | so you need to configre the nova conf also for vnc | |
| 17:45:34 | jmccarthy | sean-k-mooney: Oh I think vnc is off, spice is enabled | |
| 17:46:02 | sean-k-mooney | jmccarthy: :) that would cause issues. vnc works better in general | |
| 17:46:49 | sean-k-mooney | at least via horizon | |
| 17:47:42 | openstackgerrit | Eric Fried proposed openstack/nova master: Test alloc candidates with same RC in cn & shared https://review.openstack.org/513149 | |
| 17:47:46 | efried | cdent ^ thar she blows. | |
| 17:47:55 | cdent | word | |
| 17:47:57 | efried | mriedem dansmith ^ demonstrates those two bugs I mentioned earlier. | |
| 17:49:09 | jmccarthy | sean-k-mooney: Ok I have to re-jig this and try it again | |
| 17:49:23 | jmccarthy | sean-k-mooney: Thanks :) ! | |
| 18:08:25 | jmccarthy | sean-k-mooney: Oh actually it is vnc hmm | |
| 18:13:21 | dims | edleafe : ah ivy league school :) | |
| 18:14:12 | edleafe | dims: :) | |
| 18:24:52 | openstackgerrit | Elod Illes proposed openstack/nova master: Transform keypair.import notification https://review.openstack.org/513159 | |
| 18:31:38 | openstackgerrit | melanie witt proposed openstack/nova master: DNM: Test websocketproxy with TLS https://review.openstack.org/513160 | |
| 18:34:19 | efried | edleafe Where, in that test case? I was just following examples in the rest of the file. Thought I was mimicking alex_xu - but apparently he was mimicking jaypipes :) | |
| 18:35:34 | openstackgerrit | melanie witt proposed openstack/nova master: DNM: Test websocketproxy with TLS https://review.openstack.org/513160 | |
| 18:37:48 | efried | jaypipes I was about to start tackling the functional test failures in https://review.openstack.org/#/c/385693/ but don't want to step on you if you're on it. | |
| 18:40:42 | jaypipes | efried: looking.. | |
| 18:41:32 | jaypipes | dims: that would be an i v lg schl | |
| 18:41:54 | dims | LOL | |
| 18:43:40 | jaypipes | dims: or, just to piss off edleafe and dansmith, it would be: | |
| 18:43:41 | jaypipes | an | |
| 18:43:42 | jaypipes | i | |
| 18:43:43 | jaypipes | v | |
| 18:43:44 | jaypipes | lg | |
| 18:43:45 | jaypipes | schl | |
| 18:43:50 | jaypipes | ) | |
| 18:50:36 | jaypipes | penick: :) | |
| 18:51:27 | dansmith | penick: that's jay's coding style... "indistinguishable from dog-on-keyboard" | |
| 18:52:28 | jaypipes | :) | |
| 18:52:34 | dansmith | it's like pep192 or something I think | |
| 18:52:58 | jaypipes | pep8919280afiksjhf | |
| 18:53:06 | dansmith | sounds more like it | |
| 18:53:16 | edleafe | "dog_on_keyboard" sounds about right :) | |
| 18:53:48 | jaypipes | heh | |
| 18:54:20 | openstackgerrit | Merged openstack/nova master: placement: set/check if inventory change in tree https://review.openstack.org/470575 | |
| 18:54:44 | efried | woot^ | |
| 18:54:48 | efried | next... | |
| 18:55:32 | jaypipes | efried: so, can you brief me on any changes, if any, you made in the n-r-p series please? | |
| 18:55:48 | jaypipes | efried: just rebases or anything functional? | |
| 18:56:13 | jaypipes | efried: I'm gonna pull the top patch (385693) and run tests locally. | |
| 18:56:13 | efried | jaypipes Almost entirely rebases, fixing nits, a couple of bugs. I don't think there was anything functional - lemme skim real quick... | |
| 18:56:19 | jaypipes | kk | |
| 18:56:23 | openstackgerrit | Chris Dent proposed openstack/nova master: [placement] manage cache headers for resource classes https://review.openstack.org/513161 | |
| 18:56:33 | edleafe | jaypipes: I know you got a ton to catch up with, but the alternate hosts series is ready for review, starting with https://review.openstack.org/#/c/486215/ | |
| 18:56:49 | jaypipes | edleafe: rock on, will do that as soon as I get those tests running. | |
| 18:56:51 | efried | jaypipes Okay, should I assume you're taking the reins back at this point, or do you want to do some pair dev on that series? | |
| 18:57:14 | jaypipes | efried: oh, I can take it over again, no prob. | |
| 18:57:17 | efried | jaypipes I'm totally prepared to fix those test cases if you have more important (or jaypipes-needin) things to do. | |
| 18:57:37 | jaypipes | efried: nope, I'm good to take it back | |
| 18:57:52 | jaypipes | appreciate you, edleafe and dansmith pushing on this while I was off | |
| 18:59:33 | efried | jaypipes At some point you should give https://review.openstack.org/#/c/510244/ a once-over, make sure it gels with what you conceived at the PTG. | |
| 19:00:05 | jaypipes | efried: yep, it's near the top of my queue. | |
| 19:00:09 | efried | coo | |
| 19:02:38 | efried | jaypipes The only changes of any substance were: 1) https://review.openstack.org/#/c/377138/50..51/nova/objects/resource_provider.py -- please double-check me to make sure I referenced the exception from the right spot | |
| 19:03:13 | efried | jaypipes 2) https://review.openstack.org/#/c/377215/48/nova/objects/resource_provider.py@1275 moved no-op reformatting stuff to a new patch | |
| 19:04:36 | efried | jaypipes That's it. The rest were rebases & spelling. | |
| 19:05:18 | jaypipes | efried: coolio. thx. | |
| 19:05:38 | mriedem | dansmith: this newton backport of artom's looks good to me now https://review.openstack.org/#/c/471356/ - it's kind of a frankenpatch with squashes but does the job | |
| 19:05:48 | mriedem | would be the last one for newton before we release and tag for eol | |
| 19:07:00 | efried | jaypipes And as soon as you come up for air, would like your input on the possibility of ripping out some of the existing aggregate code until we can figure out how tf it's ever going to work with traits & nested & numbered. | |
| 19:08:03 | jaypipes | efried: well, on that, if it isn't currently affecting anything (other than read queries which cdent has patches up to address perf issues), I'd just as soon leave it be for now. We have lots of other things to get done by SYD | |
| 19:08:20 | dansmith | yeah | |
| 19:08:22 | dansmith | me too | |
| 19:08:28 | efried | jaypipes That's just it: it very well may be affecting "things". | |
| 19:10:14 | jaypipes | efried: well, I'm perfectly happy to see your thoughts on it. just not a huge priority unless you can show me it's causing bugs. | |
| 19:10:33 | jaypipes | efried: besides the general "tech debt bugs" which I certainly admit there are many. | |
| 19:10:47 | jaypipes | ok, lemme read this granular spec :) | |
| 19:13:28 | mriedem | dansmith: john put up a patch for the ironic flavor migratoin allocatoins thing https://review.openstack.org/#/c/513085/2 | |
| 19:13:40 | mriedem | seems ok to me | |
| 19:14:24 | dansmith | mriedem: yeah, sorry, I'm just behind | |
| 19:14:32 | mriedem | np, i just got back from running errands | |
| 19:15:13 | mriedem | your context targeting thing in pike is also happy for ci now https://review.openstack.org/#/c/512456/ | |
| 19:15:28 | mriedem | sdague: ^ we need to get that released soon | |
| 19:16:50 | dansmith | okay I can hit that one quick | |