Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-02-08
21:22:45 mriedem was just looking at that
21:23:15 sdague ah, ok
21:23:24 mriedem i was recently complaining about reclaim_instance_interval
21:23:46 mriedem but in my old age i've forgotten why
21:23:55 dansmith because it's stupid?
21:24:14 mriedem well, my complaint was if you set reclaim_instance_interval in the api but not on the compute hosting the instance, the instance won't actually be deleted
21:24:20 mriedem but, we assume you don't do that
21:24:24 dansmith ah yep
21:24:49 mriedem we assume you've correctly configured nova.conf's 5K options and deployed it correctly on all of your services
21:24:59 mriedem oops mine just blew off your toe!
21:25:38 mriedem at some point i think capabilities will bridge that gap
21:25:48 mriedem since the api could tell what the capabilities are on the host
21:25:51 cfriesen looks like only xen supports soft delete
21:26:17 cfriesen oh, I guess we fall back to just power_off
21:26:18 dansmith cfriesen: we fall back to poweroff if the driver doesn't do anything
21:26:20 dansmith yeah
21:28:08 openstack Launchpad bug 1663036 in OpenStack Compute (nova) "api-ref: delete server async postcondition doc is missing some info" [Medium,Confirmed]
21:28:08 mriedem https://bugs.launchpad.net/nova/+bug/1663036
21:28:17 mriedem for any low-hanging-fruit grabbers
21:28:53 openstackgerrit Matt Riedemann proposed openstack/nova master: Allow None for block_device_mapping_v2.boot_index https://review.openstack.org/430497
21:28:57 cfriesen soft-delete can be undone though, so you can't really re-use any attached volumes until it's actually deleted, right?
21:29:00 mriedem sdague: assuming you're not tired of us, ^ is controversial
21:29:26 mriedem cfriesen: correct
21:31:00 dansmith cfriesen: but that's what you want if you want to wait
21:31:13 dansmith cfriesen: so you set the interval short, to your max timeout, and then you get what you want
21:35:19 sdague mriedem: so... if we aren't doing a microversion then we are backporting to all active stable branches right?
21:35:41 mriedem sdague: that was my plan yes
21:36:07 sdague including mitaka?
21:36:14 mriedem that was my plan yes
21:36:26 sdague if you are going to take it back to there, I'd be fine with it
21:36:36 mriedem ok
21:36:37 sdague it definitely is a bug to have busted that
21:37:12 mriedem when mordred and dhellmann were reporting it, they said it used to work, which i assume meant when they were talking to older clouds that were only exposing v2
21:37:37 mriedem and i also assume that most clouds are around 18 months behind trunk
21:37:45 dhellmann yes, it works on dreamhost's deployment but not devstack
21:37:53 dhellmann I believe dreamhost is running newton by now
21:38:05 mriedem we dropped the v2 code in newton,
21:38:09 mriedem so i doubt it
21:38:12 dhellmann oh, maybe not then
21:38:28 dhellmann last I heard they were upgrading, but maybe they haven't rolled out the results
21:38:36 dhellmann it's likely mitaka, then
21:38:40 sdague mriedem: yeh
21:38:44 dhellmann I can find out if it's important
21:38:54 mriedem i don't think it's hugely important
21:38:57 sdague mriedem: yeh, I do assume that same kind of delay
21:39:21 sdague so covering the bases across 4 releases with the fix, it's probably good enough for folks
21:39:58 sdague mriedem: we should make a pretty big release note on the stable branches so it's clear to people we're restoring previous behavior
21:40:22 mriedem sdague: i can put the release note in this change itself
21:40:27 mriedem i thought about that actually but apparently forgot
21:40:31 sdague mriedem: that's fine as well
21:40:38 sdague on master, I'm slightly less concerned
21:41:22 sdague but the api fix on stable is a little bigger of a thing, because the behavior we are trying to do is get ahead of people before they upgrade, or if they upgraded, get them to bump to latest on their stable branch
21:42:40 openstackgerrit Matt Riedemann proposed openstack/nova master: api-ref: fix delete server async postcondition docs https://review.openstack.org/431190
21:45:57 openstackgerrit melanie witt proposed openstack/nova master: Add Cells V1 --> Cells V2 step-by-step example https://review.openstack.org/429075
21:45:58 openstackgerrit melanie witt proposed openstack/nova master: Edits for Cells V2 step-by-step examples https://review.openstack.org/431115
21:55:00 openstackgerrit melanie witt proposed openstack/nova master: Add Cells V1 --> Cells V2 step-by-step example https://review.openstack.org/429075
21:55:01 openstackgerrit melanie witt proposed openstack/nova master: Edits for Cells V2 step-by-step examples https://review.openstack.org/431115
21:58:57 openstackgerrit Maciej Szankin proposed openstack/nova master: config options: remove libvirt imagecache https://review.openstack.org/378497
22:02:47 melwitt dansmith, mriedem: the boot-from-volume patches for rc2 nice-to-have are ready for review. I updated the rc1 todos etherpad to indicate
22:06:42 openstackgerrit Maciej Szankin proposed openstack/nova master: conf: mark glance.api_servers opt as required https://review.openstack.org/428241
22:08:17 openstackgerrit Matt Riedemann proposed openstack/nova master: Allow None for block_device_mapping_v2.boot_index https://review.openstack.org/430497
22:08:37 mriedem sdague: see if this makes sense https://review.openstack.org/#/c/430497/6/releasenotes/notes/bug-1662699-06203e7262e02aa6.yaml
22:29:28 mordred mriedem, sdague: did you see that list of public nova versions I made for cdent a few weeks ago?
22:29:42 mriedem don't think so
22:29:59 mordred there are basically 2 versions in the wild ...
22:31:36 mordred mriedem, sdague, dhellmann: http://paste.openstack.org/show/598149/
22:33:04 mriedem 2.25 is mitaka
22:33:23 mordred cool.
22:33:31 mriedem so v2.0 or mitaka
22:33:51 mordred yah. havent' found anybody newer yet
22:34:10 macsz mriedem: if you have a sec, can you take a look into https://review.openstack.org/#/c/397800/ - it already has 2x +2 and yours -2 from few weeks back
22:34:12 mriedem because mitaka isn't eol yet
22:34:15 mriedem mordred: ^ :)
22:34:46 mriedem macsz: i've dropped the -2
22:34:54 macsz mriedem: thanks :)
22:34:57 mriedem other cores can take the heat for merging that if it turns out to be bad (in 2 years from now)
22:35:06 macsz mriedem: :D
22:35:21 mriedem rax is still on v2.0 huh
22:35:33 mriedem all of those juicy API extensions were just too much to move forward
22:36:40 mtreinish mriedem: they still use a bunch of out of tree extensions last time I looked
22:36:46 mtreinish it's how they do networking
22:39:11 mordred "that's how they do PANTS!"
22:41:04 mordred also ... in case anyone cares, I ran this: http://paste.openstack.org/show/598150/ to get the above paste
22:42:59 dansmith mordred: excellent reference
22:43:50 mtreinish mordred: cool
22:44:41 dansmith melwitt: https://www.youtube.com/watch?v=fTzEybULzRg
22:44:52 mtreinish every cloud I have creds on is also 2.25
22:45:00 melwitt thanks
22:46:17 dansmith oh yeah, I'm an addict
22:46:28 dansmith i see the entire world through the eyes of chandler's humor
22:46:43 mordred dansmith: I had no idea we were that much the same human.
22:47:27 melwitt lol "move it over"
22:47:32 dansmith mordred: agreed, my stereotyping engine did not peg you for a friends fan
22:47:36 mordred dansmith: this couch. is cut. in half. I'd like to return it for one that is not ... cut in half.
22:47:48 dansmith melwitt: heh
22:47:52 dansmith mordred: seems like a straightforward request
22:47:53 mordred melwitt: right?
23:08:11 JayF jaypipes: tox -py27 -- --failing? That's a new one on me? Does it only retry failing tests?
23:08:32 jaypipes JayF: yup, just re-runs the tests that failed from the last run :)
23:08:41 jaypipes JayF: it's a testr feature.

Earlier   Later