| *** wanghao has joined #openstack-mogan | 01:00 | |
| *** kevinz has joined #openstack-mogan | 01:11 | |
| *** kevinz has quit IRC | 01:11 | |
| *** kevinz has joined #openstack-mogan | 01:17 | |
| zhenguo | morning mogan! | 01:33 |
|---|---|---|
| wanghao | morning! | 01:53 |
| wanghao | zhenguo: https://review.openstack.org/#/c/444770/ Abstract status setting to utils. welcome to have a look if you're free. | 01:53 |
| zhenguo | wanghao: ok, thanks, will review soon, I'm now in a internal meeting :( | 01:55 |
| wanghao | zhenguo: sure | 02:04 |
| *** wanghao has quit IRC | 02:06 | |
| *** wanghao has joined #openstack-mogan | 02:06 | |
| *** wanghao_ has joined #openstack-mogan | 02:27 | |
| *** wanghao has quit IRC | 02:30 | |
| *** wanghao_ has quit IRC | 03:22 | |
| *** wanghao has joined #openstack-mogan | 03:23 | |
| *** wxy has quit IRC | 03:38 | |
| openstackgerrit | liusheng proposed openstack/mogan master: Add function to get the console info from ironic https://review.openstack.org/442345 | 03:44 |
| openstackgerrit | liusheng proposed openstack/mogan master: Add consoleauth manager https://review.openstack.org/445283 | 03:44 |
| openstackgerrit | liusheng proposed openstack/mogan master: Add rpc api for consoleauth manager https://review.openstack.org/445285 | 03:48 |
| *** wanghao_ has joined #openstack-mogan | 03:51 | |
| openstackgerrit | liusheng proposed openstack/mogan master: Add service entry for consoleauth service https://review.openstack.org/445286 | 03:52 |
| *** wanghao has quit IRC | 03:53 | |
| *** wanghao_ has quit IRC | 03:59 | |
| *** wanghao has joined #openstack-mogan | 04:06 | |
| openstackgerrit | Merged openstack/mogan master: Updated from global requirements https://review.openstack.org/445095 | 05:45 |
| openstackgerrit | liusheng proposed openstack/mogan master: Add function to get the console info from ironic https://review.openstack.org/442345 | 06:30 |
| openstackgerrit | liusheng proposed openstack/mogan master: Add consoleauth manager https://review.openstack.org/445283 | 06:30 |
| openstackgerrit | liusheng proposed openstack/mogan master: Add service entry for consoleauth service https://review.openstack.org/445286 | 06:30 |
| openstackgerrit | liusheng proposed openstack/mogan master: Add rpc api for consoleauth manager https://review.openstack.org/445285 | 06:30 |
| openstackgerrit | liusheng proposed openstack/mogan master: Add engine api for getting console info https://review.openstack.org/445321 | 06:59 |
| openstackgerrit | liusheng proposed openstack/mogan master: Expose the API of getting console info https://review.openstack.org/445322 | 06:59 |
| openstackgerrit | liusheng proposed openstack/mogan master: Add shellinabox console handler https://review.openstack.org/445323 | 06:59 |
| openstackgerrit | liusheng proposed openstack/mogan master: Add service entry for shellinabox console proxy https://review.openstack.org/445324 | 06:59 |
| openstackgerrit | bhavani proposed openstack/mogan master: Remove link to modindex https://review.openstack.org/445333 | 07:24 |
| openstackgerrit | Zhenguo Niu proposed openstack/mogan master: Add ComputePort object https://review.openstack.org/445340 | 07:41 |
| wanghao | zhenguo: ping | 07:43 |
| zhenguo | wanghao: pong | 07:43 |
| wanghao | zhenguo: Since I also found some place in manger.py which set instance's power_state too, so I think we can handle them together, that will be conveniency for developer. You can set power_state if you want. | 07:44 |
| wanghao | zhenguo: for set status patch. | 07:44 |
| zhenguo | wanghao: maybe we should not combine all things together | 07:44 |
| zhenguo | wanghao: like the utils is just process event and set instance status with the outcome of the event | 07:45 |
| wanghao | zhenguo: But if we move instance.save() into this utils, that we need to save twice in code. | 07:45 |
| wanghao | status save once, power_status save once. | 07:45 |
| zhenguo | wanghao: yes, | 07:45 |
| wanghao | zhenguo: okay, I see, | 07:46 |
| zhenguo | wanghao: hah, thanks | 07:46 |
| wanghao | zhenguo: that could be once if we change the power status before event. | 07:47 |
| wanghao | zhenguo: thanks | 07:47 |
| zhenguo | wanghao: yes, and power_state seems not related to the utils, as it's mainly for process event and move the state machine state to instance status | 07:47 |
| wanghao | zhenguo: yep | 07:48 |
| * zhenguo brb | 07:48 | |
| wanghao | zhenguo: we should split them. | 07:48 |
| zhenguo | hah, yes | 07:48 |
| wanghao | zhenguo: BTW, could this be merged now: https://review.openstack.org/#/c/444753/. Since it will conflict with the set_state patch. | 07:50 |
| wanghao | zhenguo: I want to rebase it after merging. | 07:51 |
| zhenguo | wanghao: ok | 08:04 |
| openstackgerrit | wanghao proposed openstack/mogan master: Refactor: abstract status setting for instance to utils https://review.openstack.org/444770 | 08:04 |
| wanghao | zhenguo: thanks very much :) | 08:05 |
| zhenguo | wanghao: np | 08:05 |
| openstackgerrit | liusheng proposed openstack/mogan master: Add service entry for shellinabox console proxy https://review.openstack.org/445324 | 08:20 |
| openstackgerrit | liusheng proposed openstack/mogan master: Add consoleauth manager https://review.openstack.org/445283 | 08:20 |
| openstackgerrit | liusheng proposed openstack/mogan master: Add engine api for getting console info https://review.openstack.org/445321 | 08:20 |
| openstackgerrit | liusheng proposed openstack/mogan master: Add service entry for consoleauth service https://review.openstack.org/445286 | 08:20 |
| openstackgerrit | liusheng proposed openstack/mogan master: Add shellinabox console handler https://review.openstack.org/445323 | 08:20 |
| openstackgerrit | liusheng proposed openstack/mogan master: Expose the API of getting console info https://review.openstack.org/445322 | 08:20 |
| openstackgerrit | liusheng proposed openstack/mogan master: Add rpc api for consoleauth manager https://review.openstack.org/445285 | 08:20 |
| openstackgerrit | Merged openstack/mogan master: Refactor: move fsm initiation to utils https://review.openstack.org/444753 | 08:31 |
| *** wanghao has quit IRC | 08:33 | |
| *** wanghao has joined #openstack-mogan | 08:34 | |
| zhenguo | wanghao: seems that patch needs a rebase | 09:21 |
| wanghao | zhenguo: yes, haha | 09:25 |
| zhenguo | wanghao: hah, thanks for the update | 09:25 |
| wanghao | zhenguo: you're welcome :) | 09:26 |
| zhenguo | liusheng: hey, the console patches are ready? seems all gates are green now :D | 09:30 |
| openstackgerrit | wanghao proposed openstack/mogan master: Refactor: abstract status setting for instance to utils https://review.openstack.org/444770 | 09:30 |
| liusheng | zhenguo: hah, may you can help to review, I am testing more :) | 09:30 |
| zhenguo | liusheng: sure, thanks for the hard work | 09:31 |
| liusheng | zhenguo: np :) | 09:31 |
| *** wanghao has quit IRC | 09:33 | |
| openstackgerrit | Zhenguo Niu proposed openstack/mogan master: Add get_available_resources driver method https://review.openstack.org/445378 | 09:34 |
| zhenguo | zhangyang: hi, how's everything going with the cloudboot driver, please check the final base driver requested methods with this https://review.openstack.org/445378 , we have done a lot of refactors recently. | 09:39 |
| zhangyang | zhenguo: Still working on it 😂, always watchting new codes:D | 09:42 |
| zhenguo | zhangyang: hah, and I have added a compute_nodes object | 09:42 |
| zhenguo | zhangyang: there will be many refactors following up refactors but seems not related to drivers | 09:43 |
| zhangyang | zhenguo: ah, i saw the compute node object | 09:45 |
| zhenguo | zhangyang: and I plan to save resources there instead of cache, and split scheduler out to be a separate rpc service. | 09:47 |
| zhangyang | zhenguo: these refactors do not affect the cloudboot driver currently, still working to find all abilities of cloudboot 😂 | 09:47 |
| zhenguo | zhangyang: hah, thanks | 09:47 |
| zhangyang | zhenguo: sounds good. how do we run engine now, running serval engines at the same time? | 09:49 |
| openstackgerrit | liusheng proposed openstack/mogan master: Add service entry for shellinabox console proxy https://review.openstack.org/445324 | 09:51 |
| openstackgerrit | liusheng proposed openstack/mogan master: Add shellinabox console handler https://review.openstack.org/445323 | 09:51 |
| openstackgerrit | liusheng proposed openstack/mogan master: Expose the API of getting console info https://review.openstack.org/445322 | 09:51 |
| zhenguo | zhangyang: no, just one engine service now | 09:51 |
| zhenguo | zhangyang: but we will not work like multi compute service, maybe should consider HA | 09:52 |
| zhangyang | zhenguo: yeah, saving nodes info to db maybe better for HA | 09:52 |
| zhenguo | zhangyang: hah, so I will still focus on refactors | 09:53 |
| zhangyang | zhenguo: hah, now working on cloudboot driver, maybe will do some ha job before we really launch mogan. | 09:54 |
| zhenguo | zhangyang: yes, one engine service is not stable | 09:55 |
| zhangyang | zhenguo: yes :D | 09:56 |
| *** kevinz has quit IRC | 10:17 | |
| openstackgerrit | liusheng proposed openstack/mogan master: Add desription of enabling console in quick-start doc https://review.openstack.org/445434 | 11:08 |
| *** kevinz has joined #openstack-mogan | 12:03 | |
| *** kevinz has quit IRC | 12:07 | |
| *** kevinz has joined #openstack-mogan | 12:22 | |
| openstackgerrit | bhavani proposed openstack/mogan master: Update .gitignore to ignore .eggs https://review.openstack.org/445468 | 12:32 |
| *** bani has joined #openstack-mogan | 12:35 | |
| * zhenguo goes home | 12:41 | |
| *** kevinz has quit IRC | 13:21 | |
| -openstackstatus- NOTICE: Gerrit is going to be restarted due to performance problems | 13:39 | |
| *** ChanServ changes topic to "Gerrit is going to be restarted due to performance problems" | 13:39 | |
| *** kevinz has joined #openstack-mogan | 13:44 | |
| -openstackstatus- NOTICE: Gerrit has been successfully restarted | 13:46 | |
| *** ChanServ changes topic to "Gerrit has been successfully restarted" | 13:46 | |
| *** ChanServ changes topic to "Bugs: bugs.launchpad.net/mogan | Review: https://review.openstack.org/#/q/project:openstack/mogan,n,z" | 13:52 | |
| -openstackstatus- NOTICE: Gerrit has been successfully restarted | 13:52 | |
| *** kevinz has quit IRC | 14:58 | |
| openstackgerrit | Merged openstack/mogan master: Add get_available_resources driver method https://review.openstack.org/445378 | 15:06 |
| *** IRCFrEAK has joined #openstack-mogan | 16:27 | |
| *** IRCFrEAK has quit IRC | 16:27 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!