Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-06-12
21:18:32 dansmith if you want
21:22:48 melwitt dansmith: I'll do it real quick. oomichi had one other note too
21:22:57 dansmith melwitt: already partway through
21:23:15 dansmith melwitt: I was mostly asking if you had pending local changes so as not to mess those up
21:23:20 melwitt hm, okay
21:23:31 dansmith melwitt: if you want me not to push I shan't :)
21:24:06 melwitt y'all are itching to review these huh :P
21:24:35 dansmith well, they need get-in'ing right?
21:24:43 openstackgerrit Matt Riedemann proposed openstack/nova master: Add bool_from_string for force-down action https://review.openstack.org/473564
21:25:15 melwitt yup. gimme a minute
21:26:18 melwitt dansmith: I'll push this in a minute, if that's okay
21:26:23 dansmith okay
21:26:49 melwitt thanks
21:29:26 openstackgerrit melanie witt proposed openstack/nova master: placement: Add GET /usages to placement API https://review.openstack.org/469635
21:29:26 openstackgerrit melanie witt proposed openstack/nova master: placement project_id, user_id in PUT /allocations https://review.openstack.org/469634
21:39:26 jaypipes bauzas: would appreciate a review on https://review.openstack.org/#/c/469634/ if you can. I can't +2 since I wrote a chunk of it.
21:39:44 jaypipes alex_xu: you too :) ^^
21:46:34 openstackgerrit Merged openstack/nova master: libvirt: handle missing rbd_secret_uuid from old connection info https://review.openstack.org/472266
21:51:35 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Handle uuids in os-hypervisors API https://review.openstack.org/471899
21:51:35 openstackgerrit Matt Riedemann proposed openstack/nova master: Handle uuid in HostAPI.compute_node_get https://review.openstack.org/461532
21:51:36 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Use uuid for id in Services API https://review.openstack.org/464280
21:51:37 mriedem dtp: here are the updates ^ can you look over the PUT /os-services/{service_id} changes in there? I've got the tests done.
21:51:46 mriedem dtp: just need to work on the api-ref updates, probably tomorrow
21:53:26 openstackgerrit Matt Riedemann proposed openstack/nova master: Add bool_from_string for force-down action https://review.openstack.org/473564
21:54:29 dtp mriedem sure
21:56:42 mriedem it's going to *suck* squashing in the os-hypervisors change when that happens
21:56:51 mriedem but that's another battle
22:31:10 cfriesen if anyone's on....do we create a migration record for a "rebuild", and if so where is that in the code?
22:31:47 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/472814
22:54:11 openstackgerrit Merged openstack/nova master: Add service_token for nova-glance interaction https://review.openstack.org/419662
22:59:03 mriedem efried: does ^ close the service token bp?
22:59:15 mriedem the rest looked like dnm testing patches
23:07:41 openstackgerrit Takashi NATSUME proposed openstack/nova master: Avoid forcing translation on logging calls https://review.openstack.org/413876
23:12:32 mriedem good god has anyone read the "Strawman Queens PTG week slicing" thread yet and can summarize?
23:12:37 mriedem because i don't want to start it
23:37:49 openstackgerrit Ed Leafe proposed openstack/nova master: Extract custom resource classes from flavors https://review.openstack.org/473627
23:41:59 mriedem dansmith: i think i just realized something we broke with the os-services API being cellsified
23:42:13 mriedem broke/made more obviously not dumb
23:42:33 mriedem for things like enable/disable, today you pass in the host/binary to lookup the service
23:42:45 mriedem with multi-cells, we now lookup the host mapping by the host
23:42:51 mriedem host mapping will only exist for compute services
23:42:54 openstackgerrit John Griffith proposed openstack/nova master: WIP/POC Implement new attach Cinder flow https://review.openstack.org/330285
23:42:59 mriedem but i think before you could technically disable nova-scheduler services
23:43:03 mriedem which doesn't mean anything, but you could do it
23:50:00 openstackgerrit Ed Leafe proposed openstack/nova master: Extract custom resource classes from flavors https://review.openstack.org/473627
23:53:16 openstack Launchpad bug 1697563 in OpenStack Compute (nova) "api-ref: GET /os-services does not describe host/binary request query filter parameters" [Medium,Triaged]
23:53:16 mriedem Kevin_Zheng: pretty easy bug to fix https://bugs.launchpad.net/nova/+bug/1697563
23:58:43 dansmith mriedem: yeah, so someone was recently asking me why disable on nova-scheduler didn't actually stop those services at the OS level
23:59:05 dansmith mriedem: we should probably return "40x: You can't do that thing" if they try
23:59:08 mriedem the api-ref docs all say "compute" service too
23:59:10 mriedem which is how i found this
23:59:13 dansmith okay
23:59:30 mriedem so that's the question i have in https://review.openstack.org/#/c/464280/9/nova/api/openstack/compute/services.py@281
23:59:35 mriedem which is, do we now make this a 400
23:59:44 mriedem before that new microversion, it's going to be a 404 now for non-compute services
#openstack-nova - 2017-06-13
00:00:01 mriedem with the microversion, we could get non-compute things again, or just close that weird gap
00:00:40 mriedem probably better to restrict to nova-compute services for things like enable/disable/forced_down
00:08:04 dansmith mriedem: you mean after we made the change it's 404 now instead of 200 right?
00:12:23 mriedem yes
00:12:41 mriedem we could get it back to a 200, but it seems better to just fix the weird behavior we used to allow
00:13:08 mriedem this would be good to remember for when someone shows up in a year asking why their weird script no longer disables nova-scheduler services...
00:13:15 dansmith agreed
00:13:19 dansmith not sure 404 is the right thing though
00:13:47 dansmith since we expose the service, they did a PUT on it.. probably needs to be 499: Dumbass or something
00:14:22 mriedem or a 400
00:14:23 mriedem but yeah
00:14:27 mriedem anyway have to run for a bit
01:22:11 Kevin_Zheng mriedem: assigned
01:46:54 openstackgerrit Ngo Quoc Cuong proposed openstack/nova master: Remove translation of log messages https://review.openstack.org/466995
03:18:50 openstackgerrit falseuser proposed openstack/nova master: Delete the instance files after revert a resize task https://review.openstack.org/469814
03:55:18 melwitt git status
03:55:27 melwitt dangit.
03:58:01 openstackgerrit melanie witt proposed openstack/nova master: Add FloatingIPList.get_count_by_project() https://review.openstack.org/446247
03:58:01 openstackgerrit melanie witt proposed openstack/nova master: Add FixedIPList.get_count_by_project() https://review.openstack.org/446246
03:58:02 openstackgerrit melanie witt proposed openstack/nova master: Add SecurityGroupList.get_counts() https://review.openstack.org/446245
03:58:02 openstackgerrit melanie witt proposed openstack/nova master: Add get_count_by_vm_state() to InstanceList object https://review.openstack.org/446244
03:58:03 openstackgerrit melanie witt proposed openstack/nova master: Remove useless quota_usage_refresh from nova-manage https://review.openstack.org/446243
03:58:03 openstackgerrit melanie witt proposed openstack/nova master: Remove 'reserved' count from used limits https://review.openstack.org/446242
03:58:04 openstackgerrit melanie witt proposed openstack/nova master: Count networks to check quota https://review.openstack.org/446241
03:58:06 openstackgerrit melanie witt proposed openstack/nova master: Count instances to check quota https://review.openstack.org/416521
03:58:11 openstackgerrit melanie witt proposed openstack/nova master: Count server group members to check quota https://review.openstack.org/457859
03:58:14 openstackgerrit melanie witt proposed openstack/nova master: Count security groups to check quota https://review.openstack.org/457860
03:58:24 openstackgerrit melanie witt proposed openstack/nova master: Make Quotas object favor the API database https://review.openstack.org/410945
03:58:25 openstackgerrit melanie witt proposed openstack/nova master: Count fixed ips to check quota https://review.openstack.org/457861
03:58:36 openstackgerrit melanie witt proposed openstack/nova master: Add online migration to move quotas to API database https://review.openstack.org/410946
03:58:39 openstackgerrit melanie witt proposed openstack/nova master: Count floating ips to check quota https://review.openstack.org/457862
04:04:38 openstackgerrit falseuser proposed openstack/nova master: Clear instance root_device_name in rebuild operation https://review.openstack.org/464991
04:14:35 openstackgerrit falseuser proposed openstack/nova master: Add img_linked_clone support in libvirt https://review.openstack.org/473362
04:25:46 openstackgerrit melanie witt proposed openstack/nova master: placement project_id, user_id in PUT /allocations https://review.openstack.org/469634
04:25:47 openstackgerrit melanie witt proposed openstack/nova master: placement: Add GET /usages to placement API https://review.openstack.org/469635
04:46:42 openstackgerrit Ngo Quoc Cuong proposed openstack/nova master: Remove translation of log messages https://review.openstack.org/467948
04:49:13 openstackgerrit falseuser proposed openstack/nova master: [BugFix] Include the video ram when comparing the difference of two flavors https://review.openstack.org/460388
04:56:40 openstackgerrit Huan Xie proposed openstack/nova master: XenAPI: Create linux bridge in dest host during live migration https://review.openstack.org/462017
05:03:50 openstackgerrit Naichuan Sun proposed openstack/nova master: xenapi: cached images should be cleaned up by time https://review.openstack.org/465954
05:10:03 openstackgerrit falseuser proposed openstack/nova master: [BugFix] Change the condition to delete a resizing instance https://review.openstack.org/463466
05:41:37 melwitt alex_xu: hey, just saw your review comment about the status code 406. I thought because of the trace you showed earlier, it would be 400? http://logs.openstack.org/34/469634/12/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/56d8abe/logs/screen-n-cpu.txt.gz#_Jun_12_22_00_17_707953
05:41:39 openstackgerrit Naichuan Sun proposed openstack/nova master: XenAPI: device tagging https://review.openstack.org/333781
05:42:48 alex_xu melwitt: that 400 is for user_id and project_id doesn't know by the old API. If we request a microversion, the api server doesn't know, the 406 will returned
05:43:00 melwitt alex_xu: ah, gotcha. thanks
05:43:07 alex_xu melwitt: np

Earlier   Later