| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-01-24 | |||
| 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' | |
| 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/ | |