| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-07-13 | |||
| 20:27:46 | mriedem | 1 | |
| 20:27:48 | mriedem | "Specify whether the result of resource total includes reserved resources or not." | |
| 20:29:33 | melwitt | well, I see that resize reservation is committed in compute after the resize request. so that happens before confirm/revert | |
| 20:29:50 | melwitt | so when it's in verify_resize it should have reserved == usage currently, right? | |
| 20:30:00 | melwitt | sorry | |
| 20:30:03 | melwitt | reserved will be 0 | |
| 20:30:09 | mriedem | but in_use would be incremented | |
| 20:30:12 | melwitt | right | |
| 20:30:13 | mriedem | ok | |
| 20:30:25 | mriedem | so what will happen with counting quotas? | |
| 20:30:47 | melwitt | usage should be incremented and reserved will be 0 (reserved is hard-coded to 0 always) | |
| 20:31:27 | mriedem | because instance.memory_mb is taken from new_instance_type['memory_mb'] somewhere? | |
| 20:31:40 | mriedem | b/c the count is now taken from instance.memory_mb right? | |
| 20:31:40 | melwitt | lemme see ... | |
| 20:31:53 | melwitt | yeah, as long as the new flavor has been saved to the instance | |
| 20:32:11 | melwitt | I'm looking where that happens. pretty sure it's regardless of confirm/revert | |
| 20:32:47 | melwitt | yeah, in finish_resize() (before confirm/revert) it sets the new flavor. so at that time the new usage should be counted | |
| 20:33:04 | melwitt | if an instance is sitting in verify_resize it's consuming more resources (if a resize up) | |
| 20:33:48 | mriedem | ah Specify whether the result of resource total includes reserved resources or not. | |
| 20:33:50 | mriedem | oops | |
| 20:33:53 | mriedem | _set_instance_info | |
| 20:33:55 | mriedem | in the compute manager | |
| 20:34:00 | melwitt | yeah | |
| 20:34:29 | mriedem | we should probably remove this comment then :) https://github.com/openstack/nova/blob/ac20980702030424ce4bb3ddf2610d31716a8bd7/nova/compute/manager.py#L4006 | |
| 20:34:36 | mriedem | since the quota count is goign to be completely based on these now | |
| 20:35:22 | melwitt | yeah, true | |
| 20:35:40 | mriedem | i've noted it in your change | |
| 20:35:42 | mriedem | ok on with the test | |
| 20:35:52 | melwitt | I'm trying to think how a confirm could ever fail quota check. I think it couldn't because all confirm does is remove the stuff from the source host and other cleanup. it wouldn't do anything to change the usage | |
| 20:36:18 | melwitt | just thinking out loud | |
| 20:36:20 | mriedem | that sounds right | |
| 20:36:48 | mriedem | so i'll get this test up and it will test the existing behavior, and then put a change on top to depend on your patch to test the new behavior | |
| 20:36:51 | melwitt | so just makes me wonder why it checks quota on a confirm at all right now | |
| 20:37:00 | melwitt | k, cool. | |
| 20:37:02 | mriedem | shrug | |
| 20:38:59 | melwitt | in a code comment says it does it only for a decrease in usage. ah, because on resize it only makes a reservation for an upsize. never for a downsize | |
| 20:39:11 | melwitt | so this is adjusting usage for a downsize only on confirm. guh, how confusing | |
| 20:39:38 | melwitt | so for our new counting stuff, we shouldn't need to check quota on confirm. only on revert | |
| 20:44:04 | mriedem | and only on revert when upsizing? | |
| 20:44:16 | mriedem | or we just won't make a distinction? | |
| 20:44:48 | melwitt | yeah. it could save wasted query to only do it on a revert of a downsize | |
| 20:45:08 | melwitt | so I think that's a good idea | |
| 21:16:16 | openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: Add functional test for resize-confirm plus quota validation https://review.openstack.org/483559 | |
| 21:31:23 | mriedem | https://bugs.launchpad.net/cinder/+bug/1694509 | |
| 21:31:24 | openstack | Launchpad bug 1694509 in Cinder "eventlet errors and cannot switch to a different thread" [Undecided,Confirmed] | |
| 21:31:24 | mriedem | oops | |
| 21:33:26 | openstackgerrit | Jay Pipes proposed openstack/nova master: scheduler: isolate _get_sorted_hosts() https://review.openstack.org/483564 | |
| 21:33:27 | openstackgerrit | Jay Pipes proposed openstack/nova master: add dict of allocation requests to select_dests() https://review.openstack.org/483565 | |
| 21:33:27 | openstackgerrit | Jay Pipes proposed openstack/nova master: claim resources in placement API during schedule() https://review.openstack.org/483566 | |
| 21:33:55 | jaypipes | cdent, mriedem, dansmith, edleafe, bauzas: ok, I know it's late in the day, but I got it pushed ^^ | |
| 23:01:31 | openstackgerrit | Merged openstack/nova master: Fix wrong log parm. https://review.openstack.org/482832 | |
| 23:36:33 | openstackgerrit | Merged openstack/nova master: Remove mox from nova.tests.unit.virt.xenapi.test_vm_utils.py https://review.openstack.org/478749 | |
| #openstack-nova - 2017-07-14 | |||
| 00:47:45 | openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: Add functional test for resize-confirm plus quota validation https://review.openstack.org/483559 | |
| 01:03:11 | openstackgerrit | zhangdaolong proposed openstack/python-novaclient master: Fix the inappropriate parameter name https://review.openstack.org/480826 | |
| 01:33:37 | mriedem | mysql is case insensitive by default, right? | |
| 01:33:44 | mriedem | isn't that one of the mysql vs postgresql issues? | |
| 01:33:56 | mriedem | b/c https://developer.openstack.org/api-ref/compute/?expanded=add-a-single-tag-detail says "Tags are case sensitive." | |
| 01:33:59 | mriedem | so i'm not sure that's true | |
| 01:34:03 | mriedem | depending on db backend | |
| 01:34:24 | melwitt | mysql is case insensitive by default | |
| 01:34:33 | mriedem | yeah, so i guess that's a bogus statement | |
| 01:34:47 | mriedem | FOO and foo would end up the same | |
| 01:35:30 | melwitt | if there's any python caching in between that could be a case sensitive issue | |
| 01:36:28 | melwitt | I know that flavor names are case sensitive for that reason | |
| 01:36:55 | melwitt | request caching in the API by flavor ID | |
| 01:36:55 | mriedem | i'm not following | |
| 01:37:01 | mriedem | oh that | |
| 01:37:17 | mriedem | yeah that's also this bug https://review.openstack.org/#/c/467772/ | |
| 01:37:40 | melwitt | yeah. a long time ago someone tried lowercasing flavor IDs before sending to the REST API and it broke people that had mixed case flavor IDs bc things weren't found in the cache that way | |
| 01:37:58 | melwitt | *lowercasing in novaclient | |
| 01:38:05 | mriedem | so if i set a tag of FOO on my server, and then check if FOO exists, it won't because the db says it's foo? | |
| 01:38:29 | melwitt | no that should match | |
| 01:38:48 | mriedem | ok | |
| 01:39:03 | melwitt | if there's request caching in the way you might not match FOO with Foo even though it would match in the DB is what I'm trying to say | |
| 01:39:29 | mriedem | what i'm hearing you saying is we should drop mysql support and only support postgresql | |
| 01:39:34 | melwitt | so that comment might be referring to some sort of request caching. if it's not, then it might be bogus | |
| 01:39:39 | melwitt | lol | |
| 01:40:08 | melwitt | yeah, the case insensitive default is not what I think most people would want | |
| 01:49:44 | alex_xu | oomichi: appreciate if you can help on the review of https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/api-no-more-extensions-pike, those patch are pretty easy, just need one more +2 | |
| 02:13:19 | mriedem | Kevin_Zheng: some comments in https://review.openstack.org/#/c/469800/ but overall it's really nice | |
| 02:13:20 | mriedem | bbiab | |
| 02:13:52 | Kevin_Zheng | mriedem: Just saw it, thanks for the review | |
| 02:39:48 | openstackgerrit | Tuan Luong-Anh proposed openstack/nova master: Update URL home-page in documents according to document migration https://review.openstack.org/483757 | |
| 02:42:39 | openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: Add functional test for resize-confirm plus quota validation https://review.openstack.org/483559 | |
| 02:54:04 | openstackgerrit | Merged openstack/python-novaclient master: Updated from global requirements https://review.openstack.org/483423 | |
| 03:00:46 | openstackgerrit | Merged openstack/nova master: Use plain routes list for os-cells endpoint instead of stevedore https://review.openstack.org/474869 | |
| 03:15:49 | openstackgerrit | Merged openstack/nova master: Updated from global requirements https://review.openstack.org/483337 | |
| 03:33:30 | openstackgerrit | Tuan Luong-Anh proposed openstack/nova master: Update URL home-page in documents according to document migration https://review.openstack.org/483757 | |
| 03:34:50 | openstackgerrit | Zhen Qin proposed openstack/nova master: Avoid false positives of Jinja2 in Bandit scan https://review.openstack.org/479437 | |
| 05:04:13 | gaoyuan | whoami | |
| 05:51:24 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Consistent policies https://review.openstack.org/479774 | |
| 05:53:04 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Consistent policies https://review.openstack.org/479774 | |
| 07:19:00 | openstackgerrit | Merged openstack/nova master: Fix typo https://review.openstack.org/483159 | |
| 07:20:04 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add sample test for instance audit https://review.openstack.org/480955 | |
| 07:23:34 | bauzas | stephenfin: gentle reminder for other folks that today is a French holiday in case people look at me | |
| 07:26:53 | alex_xu | bauzas: enjoy | |
| 07:33:32 | gmann | bauzas, nice. take some rest :) | |
| 07:46:43 | openstackgerrit | Gábor Antal proposed openstack/nova master: Transform aggregate.update_prop notification https://review.openstack.org/462576 | |
| 08:08:57 | openstackgerrit | Jan Gutter proposed openstack/nova master: Netronome SmartNIC Enablement https://review.openstack.org/483459 | |
| 08:16:40 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Support tag instances when boot(4/4) https://review.openstack.org/469800 | |