| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-01-24 | |||
| 23:37:53 | mriedem | nvm i guess we didn't, | |
| 23:38:00 | mriedem | we made placement enabled by default in d-g on master | |
| 23:38:04 | mriedem | but grenade would start on stable/newton | |
| 23:38:50 | mriedem | andreykurilin: on https://review.openstack.org/#/c/424745/ i think there are also some things in the python API code that i need to look at, | |
| 23:38:53 | mriedem | but probably won't have time right now | |
| 23:39:36 | andreykurilin | np | |
| 23:39:51 | bauzas | mriedem: because placement is not defaulted by devstack in newton | |
| 23:40:14 | bauzas | mriedem: that's why I started to write a script for from-newton | |
| 23:40:19 | dtroyer | andreykurilin: your last comment on https://review.openstack.org/#/c/424847/, you are about to release another novaclient? | |
| 23:40:19 | mriedem | yup | |
| 23:40:39 | andreykurilin | dtroyer: yes | |
| 23:41:35 | andreykurilin | dtroyer: we need just to merge https://review.openstack.org/#/c/424256/ . It is last known(for me) blocker | |
| 23:41:38 | dtroyer | any chance https://review.openstack.org/#/c/418190/ can go in too? AIUI that'ss unblock the SDK stuff | |
| 23:41:42 | mriedem | bauzas: was thinking about this https://github.com/openstack-infra/devstack-gate/commit/c1b64743b8404dad93d4c06bcb26cf87cbc5240c | |
| 23:42:20 | mriedem | andreykurilin: so i'll update the commit message in https://review.openstack.org/#/c/418190/ quick | |
| 23:42:26 | mriedem | since it sounds like we need it regardless | |
| 23:42:30 | andreykurilin | dtroyer: ^ | |
| 23:42:48 | bauzas | mriedem: yup, me too | |
| 23:43:05 | openstackgerrit | Dan Smith proposed openstack/nova: WIP: Collected release notes for Ocata CellsV2 https://review.openstack.org/424737 | |
| 23:43:06 | openstackgerrit | Dan Smith proposed openstack/nova: Move instance creation to conductor https://review.openstack.org/319379 | |
| 23:43:18 | bauzas | mriedem: that's why when I saw grenade being sad for my scheduler change, I looked around and saw there was no placement section | |
| 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 | |