Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-04-12
02:54:39 openstackgerrit Matt Riedemann proposed openstack/nova-specs master: Nova support for attached volume extend https://review.openstack.org/453272
02:55:00 openstackgerrit Matt Riedemann proposed openstack/nova-specs master: Amend spec for placement project/user association https://review.openstack.org/454352
02:58:27 Kevin_Zheng ping mriedm
02:58:53 Kevin_Zheng ping mriedem
02:59:11 mriedem Kevin_Zheng: hi
03:00:04 Kevin_Zheng mriedem: Hi, about https://review.openstack.org/#/c/455183/2 , I don't quite get your suggestion
03:02:32 mriedem Kevin_Zheng: i left a reply, see if that makes sense
03:02:42 Kevin_Zheng mriedem thanks
03:02:49 mriedem if the for loop does not break, then we raise exception.ConsoleNotAvailable()
03:03:00 mriedem if the for loop does break, pty is set
03:10:47 Kevin_Zheng mriedem: thanks, I just searched it, that makes sense :)
03:11:24 mriedem cool
03:13:10 mriedem alex_xu: gmann: have you ever noticed that the host power actions on the os-hosts API are all GETs?
03:13:14 mriedem GET /os-hosts/{host_name}/reboot
03:13:20 mriedem that's really odd
03:15:44 gmann mriedem: yea all start etc are GET
03:26:05 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Fix HTTP 500 raised for getConsoleLog for stopped instance https://review.openstack.org/455183
03:31:54 mriedem gmann: i also see there are Tempest negative tests that hit the os-hosts enable/disable/reboot/shutdown/start APIs, those should probably all be moved out of tempest and into nova unit tests, if we don't already have that coverage, but I bet we do
03:32:06 mriedem i'll mention that in the spec to deprecate os-hosts
03:37:29 gmann mriedem: yes we can do and i think defcore might not be using those as admin APIs and we have positive cases covered in api sample tests and if any negative cases missing in unit tests we can add easily.
03:39:11 openstackgerrit Matt Riedemann proposed openstack/nova-specs master: Deprecate the os-hosts API (spec) https://review.openstack.org/455896
03:39:20 mriedem gmann: alex_xu: ^
04:17:53 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/455960
04:19:44 openstackgerrit OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/451049
04:21:46 openstackgerrit OpenStack Proposal Bot proposed openstack/python-novaclient master: Updated from global requirements https://review.openstack.org/456002
04:29:36 EmilienM dansmith: when you have time, any chance to help with this trace? http://logs.openstack.org/15/359215/85/check-tripleo/gate-tripleo-ci-centos-7-ovb-ha/0040e01/logs/undercloud/var/log/nova/nova-api.txt.gz#_2017-04-12_04_03_47_785
04:29:53 EmilienM I'm probably wrong but it looks like related to https://review.openstack.org/#/c/442162/
04:33:21 openstack Launchpad bug 1682001 in tripleo "[promotion] nova fails to start a server - host is not mapped to any cell" [Critical,Triaged]
04:33:21 EmilienM https://bugs.launchpad.net/tripleo/+bug/1682001
05:13:13 openstackgerrit 赵明俊 proposed openstack/nova master: [BugFix] Check the instance task status to release the memory quota for vram https://review.openstack.org/456050
05:23:45 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Fix examples for add/removeFixedIp action https://review.openstack.org/456053
05:28:53 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Fix examples for add/removeFixedIp action https://review.openstack.org/456053
06:49:01 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Fix HTTP 500 raised for getConsoleLog for stopped instance https://review.openstack.org/455183
07:06:43 openstackgerrit YuYang proposed openstack/nova master: api-ref: Nova Update Compute services Link https://review.openstack.org/456085
07:14:49 openstackgerrit 赵明俊 proposed openstack/nova master: [BugFix] Check the instance task status to release the memory quota for vram https://review.openstack.org/456050
07:36:01 openstackgerrit Gábor Antal proposed openstack/nova master: Transform instance.volume_attach.error notification https://review.openstack.org/455801
07:41:09 openstackgerrit Gergely Csatari proposed openstack/nova master: api-ref: adminPass is optional in Create Server https://review.openstack.org/451497
08:33:05 openstackgerrit Gergely Csatari proposed openstack/nova master: api-ref: adminPass is optional in Create Server https://review.openstack.org/451497
08:38:43 openstackgerrit Takashi NATSUME proposed openstack/nova-specs master: List/show all server migration types https://review.openstack.org/407237
08:38:52 openstackgerrit Takashi NATSUME proposed openstack/nova-specs master: Abort Cold Migration https://review.openstack.org/334732
08:58:35 Dinesh_Bhor alex_xu: ping
09:03:02 Dinesh_Bhor alaski: Hi, Does this mean that If there are some issues then we are not going to fix them? https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/schemas/scheduler_hints.py#L67-L70
09:03:34 Dinesh_Bhor alaski: I can see your ML thread: http://lists.openstack.org/pipermail/openstack-dev/2015-June/067996.html
09:09:46 johnthetubaguy Dinesh_Bhor: can I help with context?
09:10:06 gibi mgagne: I left some comment in the spec https://review.openstack.org/#/c/453272/
09:10:24 johnthetubaguy Dinesh_Bhor: we allow custom scheduler filters and weighers, they can have their own hints, if we blocked those in the API, it would break things. Does that help?
09:11:12 openstackgerrit 赵明俊 proposed openstack/nova master: [BugFix] Check the instance task status to release the memory quota for vram https://review.openstack.org/456050
09:11:32 openstackgerrit Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for DELETE RP inventory https://review.openstack.org/455215
09:11:33 openstackgerrit Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for GET RP inventory https://review.openstack.org/455213
09:11:33 openstackgerrit Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for PUT RP inventory https://review.openstack.org/455214
09:11:34 openstackgerrit Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for DELETE RP inventories https://review.openstack.org/452708
09:13:43 openstackgerrit Béla Vancsics proposed openstack/nova master: Transform instance.resize_confirm notification https://review.openstack.org/390654
09:16:34 openstackgerrit Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for DELETE RP inventory https://review.openstack.org/455215
09:16:35 openstackgerrit Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for GET RP inventory https://review.openstack.org/455213
09:16:35 openstackgerrit Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for PUT RP inventory https://review.openstack.org/455214
09:17:33 alex_xu Dinesh_Bhor: johnthetubaguy yea, we relax that strict validation, so it already fixed
09:18:16 johnthetubaguy alex_xu: +1 thats fixed a while back, not sure what the question was really
09:19:06 Dinesh_Bhor alex_xu, johnthetubaguy: ok, understood. One more question if any user tries to pass invalid custom scheduler hint then is it the job of nova to handle it correctly?
09:19:25 openstackgerrit Stephen Finucane proposed openstack/nova-specs master: doc: Remove crud from Sphinx conf.py https://review.openstack.org/456129
09:19:25 openstackgerrit Stephen Finucane proposed openstack/nova-specs master: Mitigate 'no "man_pages" config value' error https://review.openstack.org/456128
09:20:04 Dinesh_Bhor invalid means in wrong format. I mean actually I passed '{"os:scheduler_hints": {"key": null}'
09:20:34 Dinesh_Bhor alex_xu, johnthetubaguy: ^^ which caused ValueError at a-api
09:20:36 alex_xu Dinesh_Bhor: it is open for custom scheduler, your custom filter will handle the invalid hint
09:24:47 alex_xu Dinesh_Bhor: that is kind of thing should be fixed
09:25:37 alex_xu Dinesh_Bhor: we should limit the input for hints is string
09:27:17 Dinesh_Bhor alex_xu: ok, understood, thanks
09:28:04 alex_xu johnthetubaguy Dinesh_Bhor : the requestspec break the API a little, it limits the schedule hints to string
09:28:43 johnthetubaguy limiting to a string seems reasonable
09:29:00 johnthetubaguy longer term, I we need the registration system we spoke about before
09:30:34 alex_xu ?
09:30:34 alex_xu johnthetubaguy: longer term, the scheduler hints will be removed? I guess placement won't support that
09:30:35 johnthetubaguy alex_xu: not sure myself, I can see the scheduler bit always living in Nova and having hints
09:30:50 johnthetubaguy alex_xu: I am unclear there, what I mean it, nova still picks the "best" of the possible hosts placement says
09:30:59 alex_xu johnthetubaguy: I guess scheduler bits always living Nova for some filters can be implemented by placement?
09:31:17 alex_xu s/can be/can't be/
09:31:18 johnthetubaguy alex_xu: maybe, yeah
09:31:36 johnthetubaguy I guess it needs a better name
09:31:37 johnthetubaguy "preference engine" maybe?
09:31:49 alex_xu :)
09:37:35 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add description to policies in security_groups.py https://review.openstack.org/452309
09:38:45 openstackgerrit Jens Rosenboom proposed openstack/nova master: Fix network metadata information - part 2/3 https://review.openstack.org/450212
09:53:47 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add description to policies in hypervisors.py https://review.openstack.org/451294
10:13:36 openstackgerrit Lee Yarwood proposed openstack/nova-specs master: Libvirt: Native LUKS file and host device decryption by QEMU https://review.openstack.org/437070
10:14:47 lyarwood kashyap: ^ updated, just noticed that I didn't call out the lack of support for network volumes so feel free to -1 if you think that's required
10:16:12 kashyap lyarwood: Will look in a few
10:55:08 openstackgerrit Takashi NATSUME proposed openstack/nova-specs master: List/show all server migration types https://review.openstack.org/407237
11:00:51 openstackgerrit Ildiko Vancsa proposed openstack/nova master: Remove check_detach https://review.openstack.org/446671
11:26:31 openstackgerrit Ildiko Vancsa proposed openstack/nova master: Add Cinder V3 Detach calls https://review.openstack.org/438750
11:28:20 openstackgerrit Andrey Volkov proposed openstack/nova master: [WIP] [placement] Add api-ref resource provider https://review.openstack.org/456176
11:49:14 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/455960
12:13:44 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/nova master: Read Neutron port 'binding_profile' during boot https://review.openstack.org/449257
12:23:26 openstackgerrit Andrey Volkov proposed openstack/nova master: [placement] Fix placement-api-ref check tool https://review.openstack.org/456198
12:25:07 openstackgerrit Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for PUT RP inventory https://review.openstack.org/455214
12:25:07 openstackgerrit Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for DELETE RP inventory https://review.openstack.org/455215
12:25:08 openstackgerrit Andrey Volkov proposed openstack/nova master: [WIP] [placement] Add api-ref resource provider https://review.openstack.org/456176
12:34:45 sfinucan sdague: When we deprecate an API in a microversion, does there come a point where we can delete all the code behind it?
12:35:27 sfinucan for example, is the deprecated nova-network API code still in the code base but not exercised?
12:35:43 sfinucan alex_xu: You might be able to answer that for me too ^ :)
12:36:49 mriedem sfinucan: nova-network is still used for cells v1
12:36:58 sdague sfinucan: nova-net is kind of a special case

Earlier   Later