| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-04-17 | |||
| 19:15:44 | edleafe | mriedem: http://www.netfunny.com/rhf/jokes/95q2/pants.html | |
| 19:15:49 | mriedem | dansmith: haven't gotten the template yet | |
| 19:16:52 | mriedem | edleafe: ok i figured poo was involved | |
| 19:18:24 | mriedem | dansmith: jaypipes: for the project update, when it gets to the part about the R release content, that's when i hand over the mic | |
| 19:18:45 | dansmith | mriedem: and that's when the pants really help | |
| 19:19:15 | mriedem | no it's simple | |
| 19:19:20 | mriedem | "nested resource providers by R" | |
| 19:19:24 | mriedem | "maybe" | |
| 19:19:40 | dansmith | heh | |
| 19:19:43 | mriedem | "hierarchical quotas?" | |
| 19:20:37 | mriedem | jaypipes: you want me to take my turn at updating this spec? https://review.openstack.org/#/c/454983/ | |
| 19:20:51 | mriedem | i want to just say we're going to deprecate the old cinder/glance/neutron endpoint options once we are using ksa | |
| 19:21:04 | mriedem | and drop those in the next release | |
| 19:21:17 | openstackgerrit | Dan Smith proposed openstack/nova master: Sort CellMappingList.get_all() for safety https://review.openstack.org/443174 | |
| 19:21:17 | openstackgerrit | Dan Smith proposed openstack/nova master: Clean up ClientRouter debt https://review.openstack.org/444487 | |
| 19:21:19 | openstackgerrit | Dan Smith proposed openstack/nova master: Add workaround to disable group policy check upcall https://review.openstack.org/442736 | |
| 19:21:20 | openstackgerrit | Dan Smith proposed openstack/nova master: Make server groups api aware of multiple cells for membership https://review.openstack.org/457338 | |
| 19:21:21 | mriedem | jaypipes: plus probably change assignee to TBD | |
| 19:21:49 | mriedem | will need to sync up with john this week about what osic owned that now needs a new owner | |
| 19:27:18 | lifeless | efried: bugs.python.org still | |
| 19:28:21 | efried | lifeless Someone tagged an existing bug that looked likely. I've got no experience looking at these things, but it doesn't look like it's in a release yet. | |
| 19:29:09 | efried | And likelihood of backport to 2.7 seems low, so I'm not sure it's going to be much use to me in the near future. | |
| 19:31:18 | mriedem | ooo nice http://docs-draft.openstack.org/98/456198/1/check/gate-placement-api-ref-nv/deee665//placement-api-ref/build/html/ | |
| 19:32:14 | jaypipes | mriedem: yes, please go ahead on that service catalog spec. | |
| 19:37:31 | lifeless | efried: so 'mock' is the backport mechanism, if there is a fix in git, we'll pick it up and make it available | |
| 19:38:02 | efried | lifeless Okay, cool. I'll keep an eye on those two bug reports. | |
| 19:38:04 | efried | Thanks | |
| 19:39:43 | mordred | jaypipes: it's not as good as my summit talk that didn't get accepted | |
| 19:40:11 | jaypipes | lol | |
| 19:51:09 | mriedem | alex_xu: this must be yingxin's perf benchmark tool for the nova scheduler https://github.com/cyx1231st/nova-scheduler-bench | |
| 19:53:57 | mriedem | nice, readme says you can use devstack single node | |
| 20:00:58 | comstud | mriedem: quotas are not my fault. | |
| 20:01:10 | comstud | :) | |
| 20:01:16 | mriedem | comstud: is that something i said 3 weeks ago? | |
| 20:01:24 | comstud | at least | |
| 20:01:27 | mriedem | ha | |
| 20:01:38 | mriedem | comstud: but your name is all over the TODOs :) | |
| 20:01:53 | comstud | My name was over a race condition TODO re: instance delete | |
| 20:02:04 | mriedem | guilt by association | |
| 20:02:08 | comstud | but that TODO existed prior to quotas | |
| 20:02:12 | comstud | :) | |
| 20:02:24 | mriedem | comstud: riddle me this: who do i blame for instance delete? | |
| 20:02:38 | comstud | well | |
| 20:02:41 | comstud | a lot of people | |
| 20:02:49 | comstud | I did put effort into trying to resolve races | |
| 20:03:13 | jroll | whoa, comstud is here | |
| 20:03:15 | comstud | that particular TODO you were asking about... was definitely to cover the case of 2 concurrent API requests for deleting the same instance | |
| 20:03:20 | comstud | jroll: :) | |
| 20:03:31 | mriedem | comstud: you should realize by now that anything i mention and call you out is just an attempt to lure you back | |
| 20:03:43 | comstud | The quotas code in general needed a lot of love when I stopped openstacking. | |
| 20:03:49 | comstud | and I'm guessing it has never been improvemd | |
| 20:03:51 | comstud | improved | |
| 20:03:54 | mriedem | comstud: wip | |
| 20:04:03 | comstud | heheh | |
| 20:04:07 | mriedem | wip == the status melwitt gives in each week's cells meeting | |
| 20:04:21 | comstud | you asked me to smack you around... I'd gladly do it | |
| 20:04:26 | mriedem | comstud: but https://specs.openstack.org/openstack/nova-specs/specs/pike/approved/cells-count-resources-to-check-quota-in-api.html | |
| 20:04:29 | comstud | although there was no other context at the time | |
| 20:04:32 | comstud | hehe | |
| 20:05:33 | comstud | IIRC, quotas were very fragile even without cells | |
| 20:05:52 | comstud | the soft deleting of instances in particular caused issues | |
| 20:06:23 | mriedem | yes, soft delete is terrible | |
| 20:06:24 | comstud | i know I'd made a number of comments on how to fix it | |
| 20:06:27 | comstud | but | |
| 20:06:35 | comstud | i don't remember exactly what they were | |
| 20:06:35 | mriedem | well, | |
| 20:06:38 | comstud | i think it was... | |
| 20:06:44 | mriedem | soft delete + quotas + local delete code in the api | |
| 20:06:47 | comstud | move some code very close to the real delete. | |
| 20:07:01 | comstud | but that's the other issue... | |
| 20:07:10 | comstud | edge cases when compute host is down | |
| 20:07:13 | comstud | etc | |
| 20:07:29 | mriedem | yeah see https://review.openstack.org/#/c/386756/ | |
| 20:07:40 | mriedem | we've also duplicated a lot of that local delete logic too | |
| 20:07:47 | mriedem | since you've been gone | |
| 20:08:10 | mriedem | oh, let's not forget shelve offloaded | |
| 20:08:33 | comstud | ya, i dunno when shelved was added | |
| 20:08:41 | comstud | i think it was after I stopped focusing on nova | |
| 20:08:46 | mriedem | really? | |
| 20:08:48 | mriedem | i figured it was older | |
| 20:08:53 | mriedem | what i refer to as "the before times" | |
| 20:08:59 | comstud | haha | |
| 20:09:15 | comstud | well, it's new enough that I don't remember much about it | |
| 20:09:19 | mriedem | nova, BC (before comstud) | |
| 20:09:20 | comstud | it is familiar to me, so I know it must have existed | |
| 20:09:35 | comstud | but I might have been more focused on Ironic at the time | |
| 20:09:40 | mriedem | ok, | |
| 20:09:43 | mriedem | well, fyi, don't use it | |
| 20:10:03 | comstud | haha | |
| 20:10:08 | comstud | don't use openstack? | |
| 20:10:09 | comstud | yeah, I know. | |
| 20:10:23 | mriedem | ouch | |
| 20:11:29 | comstud | in any case, i remember feeling like the quota code needed a complete re-do | |
| 20:11:44 | mriedem | that's what melwitt is working on | |
| 20:12:01 | mriedem | taking out reservations and commit/rollback | |
| 20:12:34 | mriedem | if i mention her enough, she's bound to show up too :) | |
| 20:13:48 | comstud | hehe | |
| 20:16:48 | Apoorva | Cinder CI tests are failing with following error http://paste.openstack.org/show/606920/ Can someone help? | |
| 20:17:10 | Apoorva | Failures are in n-cpu.log http://openstack-ci.tintri.com/tintri/refs-changes-58-430858-3/logs/screen-n-cpu.log.txt | |
| 20:17:26 | mriedem | Apoorva: is your CI using the ubuntu cloud archive? | |