| *** openstack has joined #openstack-mogan | 00:19 | |
| *** wanghao has joined #openstack-mogan | 00:37 | |
| wanghao | morning! | 00:45 |
|---|---|---|
| liusheng | morning | 00:54 |
| openstackgerrit | wanghao proposed openstack/mogan master: Specify neutron port when creating servers https://review.openstack.org/470824 | 01:03 |
| *** harlowja has joined #openstack-mogan | 01:04 | |
| zhenguo | morning! | 01:10 |
| *** liujiong has joined #openstack-mogan | 01:13 | |
| openstackgerrit | wanghao proposed openstack/mogan master: Specify neutron port when creating servers https://review.openstack.org/470824 | 01:23 |
| *** Kevin_Zheng has joined #openstack-mogan | 01:30 | |
| openstackgerrit | Zhenguo Niu proposed openstack/mogan-specs master: New flavor for baremetal servers https://review.openstack.org/454113 | 01:50 |
| *** litao__ has joined #openstack-mogan | 01:55 | |
| Xinran | hi, have you guys met the problem when you run tempest test? I got security group in use. | 01:56 |
| zhenguo | Xinran: you can ask liusheng, but seems the problem has been addressed | 01:58 |
| liusheng | Xinran: do you have update your code to latest ? | 01:58 |
| Xinran | the problem occurs during jeckins check | 02:02 |
| zhenguo | Xinran: can you paste the patch link here? | 02:03 |
| Xinran | https://review.openstack.org/#/c/473323/ | 02:03 |
| zhenguo | Xinran: I see, you can press the Rebase button on the top right corner | 02:05 |
| zhenguo | Xiran: as the patch is based on the old codes not including the patch which address the tempest problems | 02:06 |
| openstackgerrit | Xinran WANG proposed openstack/mogan master: Add filters to list API https://review.openstack.org/473323 | 02:06 |
| Xinran | zhenguo, I think the patch is rebased on the lastest code when I do git review and git rebase | 02:10 |
| zhenguo | Xinran: if you swith to master branch and update it to latest and then go to you branch and run git rebase master, it will be rebased | 02:14 |
| * zhenguo brb | 02:14 | |
| Xinran | zhenguo, ok I see, thanks | 02:16 |
| openstackgerrit | liusheng proposed openstack/mogan master: Make floatingip can be saved to db and can be display https://review.openstack.org/475622 | 02:43 |
| openstackgerrit | liusheng proposed openstack/mogan master: Make floatingip can be saved to db and can be displayed https://review.openstack.org/475622 | 02:44 |
| openstackgerrit | liusheng proposed openstack/python-moganclient master: Make the floating ip can be show in netinfo command https://review.openstack.org/475625 | 03:00 |
| openstackgerrit | Zhenguo Niu proposed openstack/mogan master: Adds `resources` field for flavors. https://review.openstack.org/475626 | 03:02 |
| openstackgerrit | Vu Cong Tuan proposed openstack/mogan master: Stop using deprecated 'message' attribute in Exception https://review.openstack.org/475627 | 03:04 |
| *** wxy has joined #openstack-mogan | 03:11 | |
| zhenguo | hi all, please help to look at the new flavor specs https://review.openstack.org/#/c/454113/12/specs/pike/approved/new-flavor.rst . | 03:14 |
| openstackgerrit | Xinran WANG proposed openstack/mogan master: Add filters to list API https://review.openstack.org/473323 | 03:22 |
| openstackgerrit | Xinran WANG proposed openstack/mogan master: Update ironic.driver.plug_vif implementation https://review.openstack.org/469766 | 03:23 |
| zhenguo | wanghao: I tested the specifying port patch but find api-ref error with 'Parameters out of order `metadata` should be after `max_count_body`' | 03:28 |
| zhenguo | wanghao: but it seems not related to the patch, do you want to fix it with the current patch or should it be a sparated one | 03:28 |
| wanghao | zhenguo: I'd like to fix it in other patch. quickly | 03:29 |
| zhenguo | wanghao: ok thanks | 03:29 |
| wanghao | zhenguo: leave it to me. | 03:29 |
| wanghao | zhenguo: np | 03:29 |
| wanghao | zhenguo: lanuch time, will commit at afternoon. | 03:31 |
| zhenguo | wanghao: ok, no rush | 03:31 |
| openstackgerrit | liusheng proposed openstack/mogan master: Make floatingip can be saved to db and can be displayed https://review.openstack.org/475622 | 03:44 |
| openstackgerrit | wanghao proposed openstack/mogan master: Change the parameter's order in api-ref https://review.openstack.org/475642 | 04:26 |
| zhenguo | https://docs.google.com/drawings/d/1U4clumHXPZn029f8p76XE4aMzMei_Z6BCEUA0MF7114/edit?usp=sharing | 05:12 |
| zhenguo | lol | 05:12 |
| openstackgerrit | Merged openstack/mogan master: Change the parameter's order in api-ref https://review.openstack.org/475642 | 06:38 |
| zhenguo | liusheng: do you have a env with placement service for testing? | 06:47 |
| liusheng | zhenguo: yes, my env installed Nova | 06:58 |
| zhenguo | liusheng: cool | 06:58 |
| liusheng | zhenguo: lol | 06:58 |
| zhenguo | liusheng: can I check the db schema with your env | 06:59 |
| liusheng | zhenguo: yes, sure | 07:00 |
| zhenguo | liusheng: thanks | 07:00 |
| *** wanghao_ has joined #openstack-mogan | 07:36 | |
| *** wanghao has quit IRC | 07:39 | |
| *** wanghao_ has quit IRC | 07:51 | |
| *** wanghao has joined #openstack-mogan | 07:51 | |
| *** wanghao has quit IRC | 07:56 | |
| *** wanghao has joined #openstack-mogan | 08:05 | |
| openstackgerrit | Vu Cong Tuan proposed openstack/mogan master: Stop using deprecated 'message' attribute in Exception https://review.openstack.org/475627 | 08:10 |
| openstackgerrit | Zhenguo Niu proposed openstack/mogan-specs master: Track resources using Placement https://review.openstack.org/475700 | 08:34 |
| openstackgerrit | Zhenguo Niu proposed openstack/mogan-specs master: New flavor for baremetal servers https://review.openstack.org/454113 | 08:34 |
| openstackgerrit | Merged openstack/mogan master: Updated from global requirements https://review.openstack.org/475552 | 08:46 |
| *** openstackgerrit has quit IRC | 08:48 | |
| zhenguo | liusheng: hi, I just drafted a spec for placement integration, you can update it based on your POC. | 08:51 |
| *** openstackgerrit has joined #openstack-mogan | 08:51 | |
| openstackgerrit | Merged openstack/mogan master: Tempest: add tempest test cases for server interface attaching https://review.openstack.org/474835 | 08:51 |
| openstackgerrit | Merged openstack/mogan master: Specify neutron port when creating servers https://review.openstack.org/470824 | 08:51 |
| liusheng | zhenguo: hah, thanks a lot | 08:51 |
| zhenguo | liusheng: hah | 08:52 |
| openstackgerrit | Zhenguo Niu proposed openstack/mogan master: Detach interface for server https://review.openstack.org/461767 | 08:56 |
| wanghao | zhenguo: ping | 08:57 |
| zhenguo | wanghao: pong | 08:57 |
| wanghao | zhenguo: I found an issue(maybe), in some place, we use http_client.code to raise client exception when we catch some exception. | 08:59 |
| wanghao | zhenguo: but we have already set the http code in exception class. | 08:59 |
| zhenguo | wanghao: yes, I also noticed that | 08:59 |
| wanghao | zhenguo: do you think we should use the exception code directly instead of import one more. | 09:00 |
| zhenguo | wanghao: we can change to raise exceptions directly if we have set http code | 09:00 |
| wanghao | zhenguo: totally agree. | 09:00 |
| zhenguo | wanghao: just for some error no exceptions defined we should raise http exceptions | 09:00 |
| wanghao | zhenguo: yes, we may need to introduce a reraise method like other project. | 09:01 |
| zhenguo | wanghao: seems already have a reraise method | 09:02 |
| wanghao | zhenguo: okay, I'll check it. | 09:02 |
| zhenguo | wanghao: thanks | 09:02 |
| wanghao | zhenguo: I see now we raise a new wsme.exc.ClientSideError, I'm not sure is it same if we raise mogan exception directly. | 09:04 |
| zhenguo | wanghao: yes, should change to raise mogan exceptions if we already got one | 09:04 |
| wanghao | yes, I mean is there some api impact? maybe | 09:05 |
| wanghao | emm, better to test it, never mind. I'll do it. | 09:06 |
| zhenguo | wanghao: you can check to see, if our exceptions already set the correct http code, it's ok | 09:06 |
| wanghao | zhenguo: I think it too. | 09:07 |
| zhenguo | wanghao: yes, no need to catch mogan exception and wrap with wsme.exc.ClientSideError | 09:07 |
| *** wanghao has quit IRC | 09:08 | |
| *** wanghao has joined #openstack-mogan | 09:08 | |
| wanghao | guys do you have devstack env that install swfit? | 09:19 |
| wanghao | could you paste the information about 'df -h' to see the attached volume for swift. | 09:19 |
| wanghao | I lost the attached volume after rebooting the devstack env.... | 09:20 |
| zhenguo | all my env are broken..., liusheng should be able to help out | 09:21 |
| liusheng | lol | 09:21 |
| liusheng | let me read what you said | 09:21 |
| openstackgerrit | Zhenguo Niu proposed openstack/mogan-specs master: Track resources using Placement https://review.openstack.org/475700 | 09:28 |
| *** wanghao has quit IRC | 09:33 | |
| *** openstackgerrit has quit IRC | 09:33 | |
| zhenguo | liusheng: please have a look at the detack interface patch https://review.openstack.org/#/c/461767/ , it's stuck for a long time | 09:35 |
| liusheng | zhenguo: ok | 09:35 |
| liusheng | zhenguo: how about adding a tempest test for detaching interfaces, since we have already a test for attaching interface | 09:39 |
| zhenguo | liusheng:ｓｕｒｅ | 09:39 |
| *** liujiong has quit IRC | 10:02 | |
| *** dims has quit IRC | 10:16 | |
| *** dims has joined #openstack-mogan | 10:21 | |
| *** openstackgerrit has joined #openstack-mogan | 12:03 | |
| openstackgerrit | Zhenguo Niu proposed openstack/mogan master: Update api docs to add missing parameters https://review.openstack.org/475755 | 12:03 |
| * zhenguo away | 12:04 | |
| -openstackstatus- NOTICE: restarting gerrit to address slowdown issues | 12:56 | |
| *** openstack has joined #openstack-mogan | 22:18 | |
| *** harlowja has quit IRC | 23:19 | |
| *** wanghao has joined #openstack-mogan | 23:43 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!