Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-03-24
16:04:49 leakypipes mriedem: ++
16:05:07 cdent leakypipes: maybe sean-k-mooney can take a stab at describing it? otherwise, if you're still curious I'll try a very quick spike at it after I rebase some stuff to fix merge conflicts after that float fix
16:05:33 leakypipes cdent: well, lemme try one more thing and push it.
16:06:02 cdent k
16:07:50 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/nova master: Read Neutron port 'binding_profile' during boot https://review.openstack.org/449257
16:09:13 openstackgerrit Matt Riedemann proposed openstack/nova master: Regression test for local delete with an attached volume https://review.openstack.org/449334
16:09:13 openstackgerrit Matt Riedemann proposed openstack/nova master: Set 'size' during image create with FakeImageService https://review.openstack.org/449702
16:09:43 mriedem sdague: ^ you'd probably appreciate that bottom change
16:11:37 openstackgerrit Matt Riedemann proposed openstack/nova master: Regression test for local delete with an attached volume https://review.openstack.org/449334
16:11:38 openstackgerrit Matt Riedemann proposed openstack/nova master: Set 'size' during image create with FakeImageService https://review.openstack.org/449702
16:13:05 mriedem superdan: pep8 https://review.openstack.org/#/c/449640/
16:13:31 superdan mriedem: yeah I have it fixed, as waiting for the run to finish before I pushed it up again
16:13:34 superdan and was on a call
16:13:38 mriedem ok
16:13:39 sdague johnthetubaguy: while I'm doing that, you want to hit this easy thing - https://review.openstack.org/#/c/448019/
16:13:40 superdan here
16:13:47 openstackgerrit Dan Smith proposed openstack/nova master: Remove legacy regeneration of RequestSpec in MigrationTask https://review.openstack.org/449640
16:15:05 johnthetubaguy sdague: got my head in the policy spec still, adding that to the queue
16:17:58 openstackgerrit Chris Dent proposed openstack/nova master: [placement] Allow PUT and POST without bodies https://review.openstack.org/447625
16:17:59 openstackgerrit Chris Dent proposed openstack/nova master: placement: Enable attach traits to ResourceProvider https://review.openstack.org/376201
16:17:59 openstackgerrit Chris Dent proposed openstack/nova master: placement: Add Traits API to placement service https://review.openstack.org/376200
16:18:01 openstackgerrit Chris Dent proposed openstack/nova master: placement: Add Trait and TraitList objects https://review.openstack.org/376199
16:19:11 openstackgerrit Stephen Finucane proposed openstack/nova master: libvirt: Remove dead code https://review.openstack.org/449708
16:30:22 openstackgerrit Anusha Unnam proposed openstack/nova master: Add description to policies in tenant_networks.py https://review.openstack.org/449714
16:34:08 cdent leakypipes: while waiting for some tests to run I made a hack to do demo what I was thinking of. it can probably cleaned up monstrously, but shows the idea: https://github.com/cdent/pony
16:35:13 leakypipes cdent: lol, very similar to what I put together locally.
16:35:29 cdent leakypipes: ah, good, so we are understanding one another then?
16:36:06 leakypipes cdent: would seem so.\
16:36:15 openstackgerrit OpenStack Proposal Bot proposed openstack/python-novaclient master: Updated from global requirements https://review.openstack.org/446674
16:36:24 cdent praise be or something
16:40:25 openstackgerrit Sean Dague proposed openstack/nova-specs master: New intro https://review.openstack.org/449722
16:40:33 sdague johnthetubaguy: what about something like that?
16:42:14 openstackgerrit Chris Dent proposed openstack/nova master: [placement] Idempotent PUT /resource_classes/{name} https://review.openstack.org/448791
16:46:07 johnthetubaguy sdague: looking
16:48:41 johnthetubaguy sdague: thanks I will roll that into the patch I have cooking at the moment
17:02:35 tojuvone johnthetubaguy, Hi I had other things going like floorball. Looks that somehow I did not receive your comments in the patch.
17:04:27 johnthetubaguy tojuvone: added a follow on patch, but I don't really like my ideas in there now, after catch up with sdague on the direction he was suggesting before
17:05:31 tojuvone johnthetubaguy, ok, thanks
17:06:58 openstackgerrit Anthony PERARD proposed openstack/nova master: Add exclusion list for tempest for a libvirt+xen job https://review.openstack.org/449687
17:22:22 openstackgerrit Anusha Unnam proposed openstack/nova master: Add description to policies in simple_tenant_usage.py https://review.openstack.org/449742
17:27:57 EmilienM can we nova ephemeral disk using rbd backend be encrypted with key manager like Barbican? I know it's doable with LVM but not sure about rbd
17:29:22 cdent BobBall_: is this still a thing: https://review.openstack.org/#/c/394857/
17:39:36 tojuvone johnthetubaguy, sdague, bauwser Thanks and sorry if wasted your time around the host maintenance.
17:41:15 cfriesen when running "openstack server migrate" will that do a cold migration? If so, is there any way to do a live migration without specifying the destination compute node?
17:41:26 leakypipes cdent, figleaf, alex_xu, sean-k-mooney: OK, try this on for size: https://review.openstack.org/448282 and https://review.openstack.org/448283
17:41:35 sdague tojuvone: I mostly don't want to waste your time iterating here if we're not going to do it
17:41:51 sdague and I feel like we said we weren't going to do this a while ago
17:43:12 tojuvone sdague, Yes, agree
17:44:39 johnthetubaguy cfriesen: do you try passing host=None or null? it might be an openstack client limitation (https://developer.openstack.org/api-ref/compute/?expanded=live-migrate-server-os-migratelive-action-detail#live-migrate-server-os-migratelive-action)
17:46:29 johnthetubaguy tojuvone: sdague: I agree it probably makes sense for OpenStack to support that use case, its just the preferred implementation/architecture has 99% or 100% of the code changes outside of Nova.
17:46:35 tojuvone sdague, good you stop it there again and can focus doing just what talked already back in Austin without wasting anybody time on something more
17:47:07 johnthetubaguy what we said in austin was?
17:48:07 tojuvone Well, if I recall was around to have something as simple as linking
17:48:48 openstackgerrit Anusha Unnam proposed openstack/nova master: Add description to nova/policies/shelve.py https://review.openstack.org/449763
17:49:14 sean-k-mooney leakypipes: that looks really good to me.
17:49:23 tojuvone Everytim I just touched the review it went bigger and bigger
17:49:43 leakypipes sean-k-mooney: that's more in-line to what you were envisioning?
17:49:56 sean-k-mooney leakypipes: ya it is
17:50:41 johnthetubaguy tojuvone: right, I am not sure the link is useful enough by its-self, disabled host probably is enough for the state in Nova.
17:51:34 openstackgerrit Andrey Volkov proposed openstack/nova master: [placement] Split api-ref topics per file https://review.openstack.org/449764
17:51:35 openstackgerrit Andrey Volkov proposed openstack/nova master: [WIP] Add inventories https://review.openstack.org/449765
17:51:46 sean-k-mooney leakypipes: i still need to fully read trough the register and symbolize function implementations but while its a little more complicated then just a single files with hand coded constances i think is a nicer approch
17:52:26 sean-k-mooney leakypipes: hum they are even simpler then i taught
17:52:42 leakypipes sean-k-mooney: yeah, definitely not rocket science :)
17:52:44 tojuvone johnthetubaguy, if link is set, you know you should read it, if not system is up-to-date
17:53:22 cfriesen johnthetubaguy: looks like openstackclient is fubar'd. there's no way to do a live migration without specifying the dest, and the help text says the default is shared but it's actually defaulting to block.
17:53:30 cdent leakypipes: I've left comments on both. acceptable as is, but still a bit noisy ;)
17:53:31 tojuvone but yes if it is always set, it doesn't help much, but know where to look more detailed info
17:53:40 leakypipes cdent: responded.
17:53:41 figleaf leakypipes: heading out soon, so only a quick perusal so far. I think this is a step closer to what I envisioned, but it still requires more hardcoding than I imagined
17:54:08 figleaf IOW, auto-discovery of the modules and packages, vs. spelling them out
17:54:55 leakypipes figleaf: k. well, I'm happy to check out alternative approaches if you want to hack together something this weekend?
17:54:56 cdent leakypipes: I think a list (without the register) woud make things a lot easier to read: the focus would be on the info (and the associated comments) rather than the code
17:55:24 leakypipes cdent: meh, maybe? :)
17:55:39 cdent it would be more pythonic :)
17:56:41 figleaf leakypipes: ok, don't know if I'll have time, but by Monday fo sho
17:58:26 sdague tojuvone: so... I'm not sure if this helps you at all. But I did a quick exercise this morning of how I was imaging doing these flows outside of nova - https://etherpad.openstack.org/p/host-maintenance-not-in-nova
17:59:21 sdague it really feels like there is a need for a maintenance/status system which users can poll / subscribe to, but I think it lives on it's own
18:01:43 sean-k-mooney leakypipes: im just heading home but ill take a look at it again and play with it over the weekend.
18:02:01 leakypipes sean-k-mooney: cool, thanks!
18:02:03 johnthetubaguy cfriesen: we need to get osc onto the latest microversion, where we default to auto detecting if we need to do block migration or not
18:02:09 johnthetubaguy cfriesen: among other things!
18:03:47 sean-k-mooney leakypipes: cdent on the array vs register i really dont know which i prefer honestly, the register feels a bit like macrocs in c so it make complete sense to me but it may not be the most pythonic approch not sure if an array would be better though. as register could be extended to allow inline documentation
18:04:48 cfriesen johnthetubaguy: yeah, for sure.
18:04:58 sean-k-mooney i think i would be fine with either though as long as we dont have a single file that defines everything
18:05:19 cdent sean-k-mooney: yeah, I think if you're C-oriented it makes good sense, but from a python standpoint it looks like a whole lot of boilerplate. A list (with comments on each item as now) would read well.
18:05:40 cdent mostly I don't want to type register when I come along to add some traits ;)
18:05:53 cdent but yeah, it's reasonably fine as is
18:10:09 tojuvone sdague, thanks, I will take a look. Just home alone with 3 kids, Friday evening at 8 :)
18:10:22 sdague yep, enjoy :)
18:15:20 cdent see everyone in a week, gonna take a few days
18:20:41 superdan mriedem: do you want to recheck this a few times or slam it in? https://review.openstack.org/#/c/449640/
18:23:12 mriedem looking
18:23:47 mriedem let's do it...meow
18:24:04 superdan heh, okay
18:24:40 mriedem got my landscaping buddy on the way over to triage the mini washout
18:24:55 mriedem stopped by ace, the helpful hardware place, on the way home from lunch to stop up on gutter extenders
18:24:57 mriedem booya!
18:25:06 mriedem *stock up
18:25:12 superdan haha

Earlier   Later