Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-01-24
23:57:44 mriedem ok i've got to run for the night, will update the ML thread on the RP / enabled filters thing in the morning
23:58:08 artom Kevin_Zheng, hah, OK, happy it works out then :D
23:58:14 bauzas dansmith: I haven't looked at zuul
23:58:24 bauzas before uploading it again
23:58:31 bauzas so I dunno :(
23:58:44 dansmith okay, well, it was like 15 minutes from a run completion or something
23:58:45 dansmith anyway
23:59:43 gmann artom: Kevin_Zheng so this is one going as fix - https://review.openstack.org/#/c/424759/
#openstack-nova - 2017-01-25
00:00:18 artom gmann, yeah, I guess so, Kevin_Zheng didn't take it personally ;)
00:00:55 gmann cool
00:11:42 bauzas folks, calling it a day
00:11:44 bauzas \o
00:13:30 bauzas dansmith: shit http://logs.openstack.org/30/424730/3/check/gate-grenade-dsvm-neutron-ubuntu-xenial/905a9b0/logs/grenade.sh.txt.gz#_2017-01-24_23_54_03_668
00:14:10 bauzas I'll have forge a new rev on the scheduler change because it requires a functional test modification
00:16:27 dansmith bauzas: I just pushed a rev to the grenade patch
00:16:33 dansmith bauzas: stop threatening to sleep and sleep
00:16:51 bauzas yeah...
00:35:34 alex_xu artom: Kevin_Zheng cool, thanks artom work on it also. today is my last work day before holidays also.
00:37:00 gmann alex_xu: oh, then you should review by today :) i ll also review as high priority
00:37:15 alex_xu gmann: yea
00:37:17 gmann alex_xu: but have a nice holiday any case :)
00:37:25 alex_xu gmann: thanks :)
00:37:34 alex_xu gmann: would you like help on review this one https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/add-whitelist-for-server-list-filter-sort-parameters
00:37:39 gmann alex_xu: this is winter vacation or something else
00:37:41 alex_xu gmann: the policy just updated last night
00:37:58 gmann alex_xu: sure ll check it
00:38:30 alex_xu gmann: it is chinese new year, we probably disapear while next week
00:38:32 alex_xu gmann: thanks
00:38:53 alex_xu s/while/whole/
00:39:25 gmann alex_xu: on policy things, we are going with approach you mentioned like- detection of overridden rule and then leave that as it is and force new behavior for everything else
00:40:37 alex_xu gmann: yea
01:02:24 openstackgerrit Vladik Romanovsky proposed openstack/nova: Adding vlans filed to Device tagging metadata https://review.openstack.org/405951
01:28:01 artom vladikr, I left a comment on https://review.openstack.org/#/c/405951/9 about the feasibility of a Tempest test
01:28:30 vladikr artom, great, thanks
01:28:31 vladikr !
01:28:32 vladikr :)
01:30:07 artom vladikr, does it make sense to you? Can you think of a third party CI where we would be able to test this?
01:31:37 vladikr artom, mellanox perhaps.. but they have issues with testing macvtap and I'm not sure if they are testing PF assignment
01:33:16 artom vladikr, hrmpf. Oh well, we'll see what Matt says tomorrow
01:33:30 vladikr artom, yeah, thanks!
01:53:14 nvela PING
01:59:56 openstackgerrit Zhenguo Niu proposed openstack/nova: Fix breaking out of nested loops https://review.openstack.org/424931
02:08:07 openstackgerrit Takashi NATSUME proposed openstack/nova: Enable cold migration with target host(2/2) https://review.openstack.org/408964
02:10:55 openstackgerrit Takashi NATSUME proposed openstack/nova: Enable cold migration with target host(1/2) https://review.openstack.org/408955
02:11:12 openstackgerrit Takashi NATSUME proposed openstack/nova: Enable cold migration with target host(2/2) https://review.openstack.org/408964
02:11:36 openstackgerrit Takashi NATSUME proposed openstack/nova: Add functional tests for cold migration to same host https://review.openstack.org/414926
02:11:53 openstackgerrit Takashi NATSUME proposed openstack/nova: api-ref: Add parameters in cold migrate action https://review.openstack.org/410042
02:12:01 openstackgerrit Takashi NATSUME proposed openstack/python-novaclient: Microversion 2.42 - Enable cold migration with target host https://review.openstack.org/406707
02:12:56 openstackgerrit Takashi NATSUME proposed openstack/nova: api-ref: Fix path parameters in os-hypervisors.inc https://review.openstack.org/416911
02:13:01 openstackgerrit Takashi NATSUME proposed openstack/nova: Avoid forcing translation on logging calls https://review.openstack.org/413876
02:13:04 openstackgerrit Takashi NATSUME proposed openstack/nova: api-ref: Fix keypair API parameters https://review.openstack.org/420626
02:13:09 openstackgerrit Takashi NATSUME proposed openstack/nova: Refactor a test method including 3 test cases https://review.openstack.org/394738
02:13:14 openstackgerrit Takashi NATSUME proposed openstack/nova: Refactor a test method including 7 test cases https://review.openstack.org/395436
02:13:17 openstackgerrit Takashi NATSUME proposed openstack/nova: api-ref: Fix parameters in os-hosts.inc https://review.openstack.org/416444
02:13:21 openstackgerrit Takashi NATSUME proposed openstack/nova: api-ref: Fix deprecated proxy API parameters https://review.openstack.org/416882
02:16:21 openstackgerrit Zhenguo Niu proposed openstack/nova: Fix breaking out of nested loops https://review.openstack.org/424931
03:07:28 openstackgerrit Zhenyu Zheng proposed openstack/nova: Add sort_key white list for server list/detail https://review.openstack.org/415142
03:17:55 duonghq Hello, have we close "remove-mox" for Ocata cycle?
03:27:26 mriedem duonghq: those can still go in until thursday
03:27:43 mriedem duonghq: but i suspect those aren't getting much review attention right now as we try to close other higher priority things out
03:28:13 mriedem duonghq: anything that does not make it in ocata will just be deferred to pike
03:28:32 duonghq mriedem, thank you
03:28:59 duonghq hope that somebody can give me review on https://review.openstack.org/#/c/342648/
03:29:11 duonghq due to this is last commit of the chain for test_shelve
03:29:12 duonghq :)
03:33:17 openstackgerrit liusheng proposed openstack/nova: Trivial-fix: replace "json" with "yaml" in policy README https://review.openstack.org/424961
03:33:42 duonghq sfinucan, are you there? due to you reviewed all others my ps about remove-mox of test_shelve, can you review the last one: https://review.openstack.org/#/c/342593/
03:47:12 Kevin_Zheng mriedem: could you have a look on https://review.openstack.org/415142 and https://review.openstack.org/#/c/415330/ ?
03:47:55 mriedem Kevin_Zheng: yeah i have the first open right now
03:48:29 Kevin_Zheng Yeah, thanks, it's the last day before alex on holiday, so kind hope it can be merged today
03:49:17 Kevin_Zheng go for launch, will address any comment at that, thanks alot'
04:14:52 gmann Kevin_Zheng: alex_xu policy one looks fine to me
04:15:49 gmann Kevin_Zheng: alex_xu my concern for overriding with default value is fine now as we will honor those in same way as they are overridden. means no affect on old deployer
04:19:51 mriedem dansmith: i guess we need to remove the WIP on this https://review.openstack.org/#/c/424730/
04:20:30 mriedem i think sdague is going to ask that the commit message point to a nova change with a release note saying why we need the from-newton step, which would be bauzas' filter scheduler change
04:21:14 mriedem i'll just leave a comment on there, but jenkins is happy with it which is good
04:24:07 mriedem Kevin_Zheng: left a question in https://review.openstack.org/#/c/415142/
04:29:52 alex_xu mriedem: thanks, looks like we just miss that key.
04:30:05 mriedem do we want to be able to filter on instance_type_id?
04:30:09 mriedem s/filter/sort/
04:30:28 alex_xu there should be alias for that, it called flavor I remember
04:30:36 mriedem ok
04:30:44 mriedem yeah i know we leak the flavor.id out of the REST API
04:31:31 alex_xu in the filter, that is flavor.flavor_id
04:31:38 mriedem ok
04:31:43 mriedem i'll check back on it tomorrow
04:31:48 mriedem i've got to go to bed
04:31:59 alex_xu mriedem: thanks, have good sleep
04:32:19 mriedem other than that it's looking good so shouldn't be a problem to get it in tomorrow
04:32:25 mriedem alex_xu: just +2 when you're happy with it
04:35:51 alex_xu gmann: thanks for the review
04:38:22 alex_xu gmann: re: https://review.openstack.org/#/c/415330/21/nova/tests/unit/api/openstack/compute/test_serversV21.py, are you looking for cases for the new rule is overriden?
04:39:04 alex_xu if you are looking for the cases for old rule is overriden, it is already there.
04:46:58 gmann alex_xu: yea, old rule is there. i was thinking we can have tests only with new rule(overridden and not overridden) which are nothing but new deployer
05:11:30 openstackgerrit Zhenyu Zheng proposed openstack/nova: Add sort_key white list for server list/detail https://review.openstack.org/415142
06:18:48 openstackgerrit Zhenyu Zheng proposed openstack/nova: Add sort_key white list for server list/detail https://review.openstack.org/415142
06:25:26 openstackgerrit Sreeram Vancheeswaran proposed openstack/nova: [WIP] Driver function call to enable attach volume for nova-dpm https://review.openstack.org/424996
06:48:12 openstackgerrit Zhenyu Zheng proposed openstack/nova: Add new policy for server list/detail with all_tenants https://review.openstack.org/415330
07:17:09 openstackgerrit Zhenyu Zheng proposed openstack/nova: Add release note and docs for filter/sort whitelist https://review.openstack.org/421760
07:45:01 openstackgerrit Ghanshyam Mann proposed openstack/nova: Add "supports_device_tagging" in fake driver for tests https://review.openstack.org/425030

Earlier   Later