Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-09-29
19:43:56 superdan because I thought it was set there too
19:44:05 mriedem _live_migrate in conductor task manager
19:44:16 mriedem we set the compute attributes, but not the node ones
19:44:23 mriedem was just looking at using those yesterday for something else
19:44:51 mriedem if those do get set, it would happen later in the compute probably
19:44:58 superdan oh the hosts I see
19:46:32 superdan well, I expect that just means we don't have coverage for live migrations failing in a way that will result in this getting called,
19:46:44 superdan and/or I wasn't consistent in that patch at the top,
19:46:53 superdan but I haven't really gone over that one in as much detail
19:46:59 penick pino: There's a couple ways to do this, one is to completely generate a ssh host key and sign it, then stuff it into the instance through vendordata. Another is.. more complex.
19:47:01 superdan keeping these two sets straight has been hard
19:47:08 mriedem superdan: i bet
19:47:13 mriedem don't forget about evacuate and unshelve
19:47:46 mriedem oh yeah, plus the migration object has dest_compute, dest_host, and dest_node, where one of those is the hostname and one is the host IP
19:47:47 mriedem yay!
19:48:10 penick pino: the more complex version means running a whole AuthNG infrastructure to handle trusted boot of an instance and giving it one-time credentials which allow it to bootstrap on boot and get its key signed. Heavy to implement, useful long term for secure infrastructure. Probably way heavier than you need
19:48:32 superdan mriedem: not sure unshelve is a thing here.. once you're offloaded we shouldn't keep an allocation for you
19:49:04 mriedem oh right
19:49:04 mriedem yeah
19:49:29 superdan and for evacuate, we don't need to keep a claim on the old host, so ... also not sure that's a thing
19:49:52 superdan I originally had them on my radar, but thinking more I'm not sure that's the right thing to do
20:02:32 dims mriedem : if i can get to the browser based vnc for a nova vm ... is there a way to attach a real vncviewer instead of the browser? (is there enough information in the browser url?)
20:03:14 superdan dims: you need a websocket client proxy
20:03:29 superdan dims: the browser client isn't connecting to the vnc server, but getting the stream over a websocket
20:04:31 dims superdan : i see, i have to find a vnc client that supports that ..
20:04:38 dims thanks superdan. will look
20:04:39 superdan dims: it's called novnc :P
20:04:54 dims which is browser based :)
20:04:58 superdan right
20:05:13 superdan I expect you won't find another, but .. good luck :)
20:05:21 superdan you could build a ws proxy that listens on a tcp port
20:05:31 superdan client->proxy->proxy->server
20:07:04 dims hmm, i might just stick to the browser :)
20:07:13 superdan good plan :)
20:12:48 mriedem dims: are you trying to circumvent zones
20:13:28 dims mriedem : not really, i can use the browser on my mac already, its just not very convenient for long use
20:14:01 mriedem s/*/i can use my mac already, it's just not very convenient/
20:15:09 dims mriedem : bad regex? :)
20:15:15 mriedem yeah maybe
20:16:40 superdan dims: vnc consoles are for disaster recovery and windows boxes, neither of which should be prolonged use
20:17:22 dims right superdan. there's just no other way to get to these vm(s) currently (multi-level protected zones)
20:17:39 superdan ouch
20:17:47 dims y sigh
20:21:38 openstackgerrit Matt Riedemann proposed openstack/nova master: Add hints to what the Migration attribute values are https://review.openstack.org/508603
20:21:38 openstackgerrit Matt Riedemann proposed openstack/nova master: Refactor duplicate code for looking up the compute node name https://review.openstack.org/508604
20:25:35 cfriesen not strictly a nova question, but hoping someone knows. If I monkey_patch the world with eventlet, will threading.Thread() actually give me a real thread or will it give me a greenthread?
20:27:01 superdan a greenthread
20:29:46 cfriesen thanks. I was expecting a real thread and couldn't figure out why I didn't see it on the system.
20:32:01 cfriesen do Condition and RLock and friends all work properly with the greenthread?
20:32:21 superdan yes
20:43:49 openstackgerrit Ed Leafe proposed openstack/nova-specs master: Return Alternate Hosts https://review.openstack.org/504275
21:04:04 mriedem what was the big reason why baremetal needs to have reschedules again? came up in boston and the ironic multinode jobs rely on reschedules right now - is it something with node contention? or was it also something with just failed hardware builds?
21:05:27 superdan flakiness of the management controllers
21:05:50 superdan before claims in the scheduler, it was also that only one instance can ever be there,
21:06:06 superdan so if we race for resources with virt, there is a possibility that both instances can fit, but that never happens with bm
21:24:02 mriedem figleaf: some comments inline https://review.openstack.org/#/c/504275/7
21:24:04 mriedem but overall good
21:24:27 mriedem superdan: ok, that was something i mentioned in ^ as a missing use case
21:24:35 mriedem since it seemed like a pretty critical issue for ironic
21:29:29 openstackgerrit ayoung proposed openstack/nova master: Admin API Policy contingent on is_admin_project https://review.openstack.org/384148
21:41:13 mriedem leakypipes: fried_rice: is there a reason why the traits parameter in alex's spec for GET /allocation_candidates is just called "required" rather than "required_traits"?
21:41:34 mriedem "Is the 'required' parameter required? No, it's optional. OK?"
21:43:24 figleaf mriedem: thanks
21:45:12 figleaf mriedem: and I wasn't at Boston, but I remember it from Atlanta
21:45:33 mriedem ok
21:45:40 mriedem let's combine our forgetfulness
21:45:43 mriedem for max recollection
21:46:06 figleaf It was in Austin!!
21:50:42 mriedem leakypipes: figleaf: superdan: johnthetubaguy: fried_rice: left some comments in the traits spec https://review.openstack.org/#/c/497713/ - two things i don't like, but i'm assuming have been discussed to death already
21:51:08 superdan I won't get back to it until monday
21:51:12 superdan but... cool
21:51:58 figleaf mriedem: I don't like 'required' or 'requires' either
21:52:17 superdan required= is the only way to go, IMHO
21:52:31 figleaf mriedem: heading out soon; will comment by Monday
21:58:39 mriedem ?required_foobars
22:05:44 mriedem ok seeing the bike shedding on this in the previous comments
22:06:41 fried_rice mriedem I abstain. I'd be fine with whatever y'all land on wrt those two points.
22:28:16 mriedem tested out GET /servers?status=''
22:28:28 mriedem it definitely has a behavior change from what the spec is proposing
22:28:39 mriedem it applies that as the filter, which means you don't get any instances back that have a vm_state set
22:41:08 fried_rice mriedem But does status= take a list?
22:41:23 fried_rice It doesn't appear to, in my trials.
22:41:59 fried_rice mriedem Which makes it apples:oranges compared to traits.
22:44:30 thingee mriedem: there are 69 total forum sessions
22:44:43 thingee mriedem: was waiting all day for that answer myself :)
22:45:55 mriedem ok and 73 proposed
22:46:11 mriedem so i guess chances are good to get stuff in, especially since there are some duplicates
22:46:33 thingee hey I'll cheers my beer to that
23:21:57 openstackgerrit Eric Fried proposed openstack/nova master: Use ksa adapter for cinder client https://review.openstack.org/508345
23:22:32 fried_rice mriedem Since you're the cinder guy? ^
23:37:43 superdan mriedem: they just restarted zuul with some fixes including one that makes it not do the thing your patch was trying to avoid
23:37:47 superdan so we might be unblocked
23:51:33 superdan mriedem: strike that.
#openstack-nova - 2017-09-30
01:25:14 openstackgerrit Liping Mao proposed openstack/nova stable/ocata: Stop _undefine_domain erroring if domain not found https://review.openstack.org/508640
01:26:04 openstackgerrit Liping Mao proposed openstack/nova stable/newton: Stop _undefine_domain erroring if domain not found https://review.openstack.org/508641
03:41:12 openstackgerrit Dan Smith proposed openstack/nova master: Fix a pagination logic bug in test_bug_1689692 https://review.openstack.org/505661
03:41:12 openstackgerrit Dan Smith proposed openstack/nova master: Use improved instance_list module in compute API https://review.openstack.org/505418
03:41:13 openstackgerrit Dan Smith proposed openstack/nova master: Fix CellDatabases fixture swallowing exceptions https://review.openstack.org/506312
03:41:13 openstackgerrit Dan Smith proposed openstack/nova master: Fix minor input items from previous patches https://review.openstack.org/506416
03:41:14 openstackgerrit Dan Smith proposed openstack/nova master: Merge build requests into the sortmaster 9000 https://review.openstack.org/508595

Earlier   Later