Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-01-24
23:43:29 openstackgerrit OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/424828
23:46:51 andreykurilin dtroyer: I think we will have a change that unblocks us soon, so we can start a process of merging updates of g-r. these two changes should not conflict.
23:47:07 openstackgerrit Matt Riedemann proposed openstack/python-novaclient: Fix functional tests to deal with multiple networks https://review.openstack.org/418190
23:47:08 mriedem andreykurilin: ^ updated the commit message
23:47:16 andreykurilin mriedem: cool!
23:48:01 mriedem i have now done exactly 2 useful things today i think
23:48:10 andreykurilin ))
23:48:11 andreykurilin thanks!
23:48:17 andreykurilin you are hero
23:48:25 mriedem that calls for a chateaubriand
23:50:37 andreykurilin mriedem bauzas: https://review.openstack.org/#/c/424256/10 posted results ;) it changes just cli, so result of neutron-tempest job should not be affected. it is the last nice-to-have feature before release
23:52:49 andreykurilin ok, I need to go sleep. bye folks!
23:53:56 dtroyer andreykurilin, mriedem: thanks,
23:54:01 gmann mriedem: alex_xu artom I am +1 to fix in this cycle only
23:55:12 openstackgerrit Sylvain Bauza proposed openstack/nova: Scheduler calling the Placement API https://review.openstack.org/417961
23:55:18 artom Kevin_Zheng, hey, yeah, I kinda stepped on your feet with my patch, apologies
23:55:38 dansmith bauzas: did you get a run?
23:55:45 artom Umm, I guess mine and yours are different enough that it's defensible to have two approaches proposed?
23:55:55 bauzas dansmith: not yet
23:56:02 bauzas dansmith: I just provided a new PS
23:56:03 dansmith bauzas: but you just rev'd it?
23:56:11 bauzas yup
23:56:18 dansmith mmkay, was waiting for that run
23:56:32 bauzas oh sorry :(
23:56:38 Kevin_Zheng artom: ah, NP, I didn't see the ML yesterday, maybe your way is the way that we all like
23:56:40 bauzas I haven't seen you rechecked it
23:56:44 bauzas sorry :(
23:56:54 dansmith it was still running, right?
23:57:27 Kevin_Zheng artom: also our holiday is coming so maybe you can fix it more faster than me :)
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'

Earlier   Later