Earlier  
Posted Nick Remark
#openstack-nova - 2018-10-11
18:10:16 aspiers and we all know success in life is defined by stackalytics
18:10:59 mriedem dansmith: i'll be the judge of how quickly i can judge something
18:11:11 dansmith aspiers: yeah my last loan officer said "aand, just one more thing, what's your stackalytics ranking?"
18:11:19 aspiers hahaha
18:11:29 dansmith mriedem: I should hope so
18:12:10 aspiers I barely squeaked through CBP at Denver airpor because my stackalytics were so poor
18:13:55 imacdonn dansmith sean-k-mooney mriedem: please rereview ... note that I had to tweak the logic a bit (see comment in review) ... https://review.openstack.org/#/c/608091/
18:14:45 aspiers dansmith: luckily my UK passport will let me into Berlin for the next few months at least :-/
18:14:55 dansmith heh
18:15:05 imacdonn you non-european european, you
18:15:24 aspiers actually I'm technically also an Irish citizen, but haven't got around to applying for that passport yet
18:15:34 aspiers (thanks mum)
18:15:40 imacdonn I have both UK and Irish passports .. it's been handy in the past
18:15:46 aspiers oh nice
18:16:07 imacdonn and a green card, so I have a few bases covered ;)
18:16:33 aspiers wow, good work
18:16:39 aspiers I have Global Entry at least
18:20:48 aspiers hmm, I guess I should submit the blueprint first, otherwise the spec will point to a non-existing bp
18:23:13 openstackgerrit melanie witt proposed openstack/nova master: Use nova-consoleauth only if workaround enabled https://review.openstack.org/607068
18:23:35 melwitt attempt number 29413059 ^
18:23:41 dansmith aspiers: yeah, which is auto-minus-2-auto-abandon
18:24:09 aspiers uh-oh ... well, that's still better than -3, so I'm improving ;-)
18:24:29 artom As long as it's not -3.14, no one should be without pie
18:24:29 openstackgerrit Adam Spiers proposed openstack/nova-specs master: Add spec for libvirt driver launching AMD SEV-encrypted instances https://review.openstack.org/609779
18:24:46 aspiers artom: I'm with you on that
18:25:04 aspiers haha
18:25:18 artom dansmith, I mean, I'll take the "giant" as a compliment
18:25:35 aspiers melwitt: here it is finally ... https://blueprints.launchpad.net/nova/+spec/amd-sev-libvirt-support
18:26:03 aspiers my very first nova spec, so I'm prepared for the -3s to roll in
18:26:12 dansmith artom: heh
18:26:24 aspiers artom: ROFL
18:27:05 aspiers melwitt: one of the key questions is whether the built-in trait:HW_CPU_x86_SEV support should be included in this spec, or split off into a separate spec
18:27:13 aspiers melwitt: I'll gratefully take guidance on that
18:27:25 melwitt that's a jaypipes question
18:27:30 aspiers OK thanks
18:27:45 melwitt but I think in this spec would be fine
18:27:51 aspiers also, as a distinct nova newb, I've probably got the wrong end of the stick on a whole bunch of other stuff
18:28:04 jaypipes aspiers: I'll have a gander later today.
18:28:08 aspiers a lot of it is based on reading source code over the last few days
18:28:13 aspiers jaypipes: awesome, thanks!
18:29:09 aspiers intention is for me (and/or maybe colleagues) to do the heavy lifting, so we're not expecting anyone else to come in and do the hard work for us
18:29:12 mriedem dansmith: i left some replies in https://review.openstack.org/#/c/605573/15
18:29:20 aspiers ... although of course we wouldn't object to that ;-)
18:30:40 aspiers oh crap, I forgot to finish the work items and dependencies sections
18:31:47 imacdonn -4 ?
18:32:13 mriedem ah traits, the new flavor extra spec
18:32:14 aspiers I'm OK with that if I get in the Guinness World Records for lowest Gerrit score
18:32:45 mriedem aspiers: have you read http://lists.openstack.org/pipermail/openstack-dev/2018-October/135446.html ?
18:32:52 aspiers almost certainly not
18:33:01 mriedem probably answers your trait questoin
18:33:03 mriedem *question
18:33:21 aspiers cool, thanks
18:34:40 mriedem tl;dr is we'll translate a flavor extra spec / image property to a trait for scheduling and the virt driver would use the extra spec/image property
18:34:41 aspiers mriedem: hooray, it seems my proposal accidentally aligns with upstream consensus on this
18:35:28 aspiers wait - doesn't that translation correspond to the first approach listed in that mail, rather than the second?
18:35:40 openstackgerrit Jim Rollenhagen proposed openstack/nova-specs master: Use conductor groups to partition nova-compute services for Ironic https://review.openstack.org/609709
18:37:08 mriedem i guess
18:37:20 mriedem i don't remember being around for the decision
18:37:51 aspiers that mail says "Ultimately, the decision was made to go with the second approach."
18:38:35 mriedem yeah i just read it
18:38:50 mriedem hopefully that makes its way into developer docs somewhere at some point
18:38:58 aspiers +1 for that
18:39:13 aspiers but that's super-helpful - I'm gonna link it from the spec. thanks!
18:39:25 melwitt yeah, in the spec https://review.openstack.org/#/c/607989/1/specs/stein/approved/support-hpet-on-guest.rst it's the "explicit trait, implicit config" approach
18:42:44 mriedem dansmith: on that cache thing, can we do that in a follow up?
18:47:14 dansmith mriedem: I was even going to volunteer to do it as a follow-up, but then you hurt my feelings by calling it excessive
18:47:41 mriedem "all nova devs are giant dicks"
18:47:58 mriedem just trying to stay true to form...
18:48:14 dansmith I would like to appeal to our guidance counselor
18:49:40 mriedem is it still mnaser?
18:50:16 mnaser mriedem: you're on timeout
18:50:19 mnaser 4 hours of typo fixing
18:50:21 dansmith they're drawing new names out of a hat to ensure fairness I think
18:50:23 mnaser be nice >:(
18:51:07 mnaser also now that i've been highlighted
18:51:09 openstackgerrit Adam Spiers proposed openstack/nova-specs master: Add spec for libvirt driver launching AMD SEV-encrypted instances https://review.openstack.org/609779
18:51:21 mnaser eyes on https://review.openstack.org/#/c/602384/ would be important/nice considering you can kinda snoop and listen to other vms traffic
18:51:31 mnaser which is kind of a big deal™
18:51:50 mriedem sean-k-mooney was supposed to be working on backports
18:52:09 mriedem to see if older versions of libvirt had issues with that
18:53:03 mnaser "someone" i know has done extensive testing
18:53:13 mnaser and would be likely happy to do local testing
18:53:20 mnaser so i'll try to ask/get them here to help sean-k-mooney
18:53:50 mriedem mnaser: on https://review.openstack.org/#/c/609330/1/specs/stein/approved/remove-force-flag-from-live-migrate-and-evacuate.rst@93 you don't know of any operators that disable computes and then force live migrations to those hosts to rebalance or during rolling upgrades?
18:55:45 mnaser mriedem: we do a bit of the opposite. we disable the host being evacuated and then evacuate that one
18:55:57 mnaser that way the scheduler just distributes things the way they're suppoesd to be
18:56:20 mnaser i'll reply to the comment
18:59:28 mnaser mriedem: added my comment
18:59:28 mriedem mnaser: btw, you don't get "on" timeout, you get "in" timeout
18:59:36 mriedem unless on timeout is a canadian thing
18:59:45 mriedem it's like kid jail
18:59:53 mnaser sorry i haven't done the whole raising a kid/pet/thing
19:00:54 mnaser whats wrong with kids napping
19:02:41 dansmith mriedem: hmm, does the context we're passing to get_all_volume_types potentially limit the result to what you have available based on who you are?
19:02:51 mriedem dansmith: yes
19:02:57 mriedem so no caching
19:03:10 mriedem unless you use cinder admin config and list all volume types
19:03:19 mriedem which we have, but it's optional
19:04:18 dansmith I'll just stop trying to be smart
19:05:03 mriedem that's ok, i apparently don't understand json schema https://review.openstack.org/#/c/606398/10/nova/tests/unit/api/openstack/compute/test_serversV21.py

Earlier   Later