| *** xuhaiwei_ has quit IRC | 00:02 | |
| *** pratikma_ has quit IRC | 00:02 | |
| *** pratikmallya has joined #senlin | 00:07 | |
| *** Qiming has quit IRC | 00:16 | |
| *** xuhaiwei_ has joined #senlin | 00:57 | |
| *** xuhaiwei_ has quit IRC | 01:10 | |
| *** gongysh has joined #senlin | 01:28 | |
| *** Qiming has joined #senlin | 01:36 | |
| openstackgerrit | lvdongbing proposed openstack/senlin: Remove senlin.conf.sample https://review.openstack.org/258215 | 01:36 |
|---|---|---|
| *** pratikmallya has quit IRC | 01:38 | |
| *** xuhaiwei_ has joined #senlin | 01:46 | |
| *** zhenguo has joined #senlin | 01:47 | |
| *** Yanyanhu has joined #senlin | 01:49 | |
| xuhaiwei_ | when is the spring festival vocation in China this year? | 01:51 |
| Qiming | 2016.02.08 | 01:51 |
| Qiming | vacation 2016.02.07-2016.02.13 | 01:52 |
| *** elynn has joined #senlin | 01:52 | |
| Yanyanhu | spring vacation? | 01:54 |
| xuhaiwei_ | yes | 01:55 |
| xuhaiwei_ | want to hear your available time | 01:55 |
| *** elynn_ has joined #senlin | 01:57 | |
| *** elynn has quit IRC | 01:58 | |
| openstackgerrit | lvdongbing proposed openstack/senlin: Make health_mgr_opts auto generate to config sample https://review.openstack.org/258220 | 02:02 |
| Qiming | xuhaiwei_, I think the Jan 11 week would be okay | 02:16 |
| xuhaiwei_ | it is ok for me Qiming | 02:17 |
| Qiming | before that, it will be the New Year's day, some may go vacation | 02:18 |
| Qiming | after that week, some may go vacation for the Spring Festival | 02:18 |
| *** Liuqing has joined #senlin | 02:22 | |
| Qiming | xuhaiwei_, https://bugs.launchpad.net/senlin/+bug/1517353 | 02:34 |
| openstack | Launchpad bug 1517353 in senlin "Node can't be created for KeyError: 'server'" [Undecided,New] - Assigned to Haiwei Xu (xu-haiwei) | 02:34 |
| Qiming | this is still valid bug? | 02:34 |
| xuhaiwei_ | no valid | 02:37 |
| Qiming | ok | 02:38 |
| xuhaiwei_ | fixed by SDK side | 02:38 |
| openstackgerrit | Merged openstack/senlin: Make health_mgr_opts auto generate to config sample https://review.openstack.org/258220 | 02:43 |
| *** lixinhui has joined #senlin | 03:07 | |
| Qiming | lixinhui, I'm marking bug #1525897 as an invalid report | 03:15 |
| openstack | bug 1525897 in senlin "sqlalchemy api crash for Conext has no attributes " [Undecided,Invalid] https://launchpad.net/bugs/1525897 - Assigned to Cindia-blue (miaoxinhuili) | 03:15 |
| *** yuanying has quit IRC | 03:22 | |
| lixinhui | Okay, Qiming | 03:25 |
| openstackgerrit | Qiming Teng proposed openstack/senlin: Add blue-green deployment as a feature request https://review.openstack.org/258240 | 03:25 |
| *** yuanying has joined #senlin | 04:07 | |
| *** elynn_ has quit IRC | 04:12 | |
| *** elynn_ has joined #senlin | 05:00 | |
| *** Liuqing_ has joined #senlin | 05:35 | |
| *** Liuqing has quit IRC | 05:38 | |
| *** pratikmallya has joined #senlin | 05:55 | |
| *** pratikmallya has quit IRC | 07:11 | |
| Qiming | Yanyanhu, seems the transaction isolation problem still persists | 07:24 |
| Yanyanhu | ... | 07:25 |
| Qiming | I have revised all MutableList data type back to basic strings | 07:25 |
| Yanyanhu | ok, will do more investigation about it based on this reverting | 07:25 |
| Qiming | if it is not solving the problem, switching back to basic strings won't help | 07:26 |
| Qiming | the most visible impact is about performance | 07:26 |
| Yanyanhu | yes, I guess so | 07:26 |
| Qiming | it is ..... very slow now | 07:26 |
| Yanyanhu | ... | 07:26 |
| Yanyanhu | more overhead from converting string to list and back | 07:27 |
| Qiming | yes | 07:29 |
| Qiming | I'm abandoning this change | 07:31 |
| Qiming | it is not helping, sigh ... | 07:32 |
| Yanyanhu | really a issue takes headache... | 07:32 |
| Qiming | another attempt: I'll try explicitly add action objects to sessions | 07:33 |
| Yanyanhu | you mean? | 07:34 |
| Qiming | when doing mark_succeed | 07:35 |
| Qiming | we are changing the fields of several actions together | 07:35 |
| Qiming | some of them may get missed if not explicitly added to the current session | 07:35 |
| Qiming | this could be improved by just adding a line: session.add(action) | 07:35 |
| Qiming | will try that approach | 07:36 |
| Yanyanhu | ok | 07:36 |
| Yanyanhu | actually I'm not very clear about this part. I thought eachtime an action is queried in a session, it will be added to this session by default. | 07:36 |
| Qiming | yes, that is the depended action | 07:37 |
| Yanyanhu | not sure this is a correct understanding | 07:37 |
| Qiming | but for the dependent action, they are not automatically added sometimes | 07:37 |
| Yanyanhu | oh, I see what you mean | 07:38 |
| Yanyanhu | yes, we should make a try | 07:38 |
| Qiming | that is why NODE_DELETE has been successfully marked as SUCCEEDED but cluster action is not updated correctly | 07:39 |
| Yanyanhu | yes, we met that issue before | 07:40 |
| Qiming | it is funny now | 07:41 |
| Qiming | I have killed the tox functional test | 07:41 |
| Qiming | restarted senlin-engine | 07:41 |
| Qiming | now I'm seeing that senlin-engine automatically starts running things | 07:42 |
| Yanyanhu | functional test is still running at backend? | 07:42 |
| Yanyanhu | seems crtl+z can't stop it | 07:43 |
| Yanyanhu | sometimes, have to ps and kill the thread | 07:43 |
| Qiming | yes | 07:43 |
| Qiming | em, short ids are really convenient | 07:45 |
| Qiming | $ senlin profile-delete fc4 713 341 1d8 02a 657 | 07:45 |
| Qiming | Profile deleted: ['fc4', '713', '341', '1d8', '02a', '657'] | 07:45 |
| Yanyanhu | :) | 07:46 |
| Yanyanhu | I never tried so short id | 07:46 |
| Qiming | you should | 07:46 |
| Yanyanhu | usually use id consists of 8 letters | 07:46 |
| Qiming | it is really convenient | 07:46 |
| Yanyanhu | yep | 07:46 |
| Qiming | if 'fc4' is too short, you will get warnings, not errors | 07:46 |
| Yanyanhu | administrators will definitely like it :) | 07:47 |
| Qiming | yes, moving mouse to do copy is not fun | 07:47 |
| Yanyanhu | yes, especially when you have to scroll you screen | 07:48 |
| Qiming | I think I'm now close to a better, cleaner solution | 07:48 |
| Qiming | functional tests are passing one by one | 07:48 |
| Qiming | there is still an error, but not related to this damn thing | 07:49 |
| Yanyanhu | :) | 07:49 |
| openstackgerrit | Qiming Teng proposed openstack/senlin: Fix action dependency problem https://review.openstack.org/258308 | 07:59 |
| Qiming | Yanyanhu, patch #258308 is the fix | 08:03 |
| Yanyanhu | great! | 08:04 |
| Qiming | still missing two places | 08:04 |
| Yanyanhu | will check it | 08:04 |
| openstackgerrit | Qiming Teng proposed openstack/senlin: Fix action dependency problem https://review.openstack.org/258308 | 08:04 |
| Qiming | now hopefully complete | 08:04 |
| *** R_lixh has quit IRC | 08:06 | |
| *** tianc_ has quit IRC | 08:06 | |
| xuhaiwei_ | Qiming, about using lambda to replace the inner method, it's Ok for the source, but it is still difficult to test | 08:07 |
| Qiming | you will need to mock some objects | 08:07 |
| Qiming | or else the lambda will fail | 08:08 |
| Yanyanhu | hi, Qiming, about 258308, do we need to refresh action in _mark_failed_action and _mark_cancelled_action? | 08:08 |
| Qiming | Yanyanhu, looking | 08:08 |
| Yanyanhu | like what we are doing in _add_action_dependency | 08:08 |
| Yanyanhu | ok | 08:08 |
| *** yuanying has quit IRC | 08:09 | |
| Qiming | Yanyanhu, I'm inclined to avoid doing refresh() | 08:09 |
| Qiming | it is kind of breaking the atomicity of transactions | 08:10 |
| Qiming | but not 100% sure | 08:10 |
| Qiming | Yanyanhu, check this: http://git.openstack.org/cgit/openstack/oslo.db/tree/oslo_db/sqlalchemy/session.py#n34 | 08:12 |
| Yanyanhu | understand, it looks like an operation that around the default consistent level | 08:12 |
| Qiming | yes, so I hate the 'refresh()' hack | 08:12 |
| Yanyanhu | thus could make the behavior unpredictable | 08:12 |
| Qiming | eventually we may need to understand oslo.db better | 08:13 |
| Yanyanhu | yes | 08:13 |
| Qiming | and use the enginefacade.writter kind of annotation | 08:13 |
| Qiming | or we need to consider distributed locks, like the one summarized here: https://review.openstack.org/#/c/209661/38/specs/chronicles-of-a-dlm.rst | 08:13 |
| Yanyanhu | hmm, distributed lock is really useful and also really complicated... | 08:14 |
| Qiming | yes | 08:14 |
| Qiming | anyway, we have TOOZ item in FEATURES.rst | 08:15 |
| Qiming | we won't drop this ball | 08:15 |
| Yanyanhu | yes | 08:15 |
| Qiming | I'm hoping that currently we have a not-perfect-but-working solution | 08:17 |
| Yanyanhu | yea | 08:18 |
| Qiming | based on a simple glimpse on this: http://git.openstack.org/cgit/openstack/oslo.db/tree/oslo_db/sqlalchemy/session.py line 122-132 | 08:18 |
| Qiming | we really need to do a session.add(someobject) | 08:18 |
| Qiming | the enginefacade thing could be useful for better control of using contexts | 08:19 |
| Yanyanhu | hmm, will look through this model to better understand the way oslo.db control the session | 08:19 |
| Qiming | ok | 08:19 |
| *** lixinhui has quit IRC | 08:37 | |
| openstackgerrit | xu-haiwei proposed openstack/python-senlinclient: Don't show show_deleted column in node-list when not needed https://review.openstack.org/257742 | 08:49 |
| openstackgerrit | Qiming Teng proposed openstack/senlin: Implement purge_deleted command https://review.openstack.org/257776 | 09:13 |
| openstackgerrit | Merged openstack/python-senlinclient: Don't show show_deleted column in node-list when not needed https://review.openstack.org/257742 | 09:17 |
| openstackgerrit | xu-haiwei proposed openstack/python-senlinclient: Replace inner methods with lambda function https://review.openstack.org/258368 | 09:24 |
| Qiming | sigh .... | 09:26 |
| Qiming | problem still not fixed | 09:26 |
| Yanyanhu | seems so... | 09:27 |
| Qiming | I'm suspecting that this patch is the root cause: http://git.openstack.org/cgit/openstack/oslo.db/commit/?id=936ffc187d445ed7644d755bcdfa3be682b786fa | 09:33 |
| Yanyanhu | allow_async could be true by default for read transaction after applying this patch? | 09:35 |
| Qiming | don't know | 09:35 |
| Qiming | we don't have problems before | 09:36 |
| Qiming | I don't think we have updated sqlalchemy | 09:36 |
| *** zhenguo has quit IRC | 10:00 | |
| *** Liuqing_ has quit IRC | 10:01 | |
| *** Liuqing has joined #senlin | 10:02 | |
| *** Liuqing has quit IRC | 10:02 | |
| *** Qiming has quit IRC | 10:05 | |
| *** Yanyanhu has quit IRC | 10:27 | |
| *** Liuqing has joined #senlin | 10:40 | |
| openstackgerrit | xu-haiwei proposed openstack/python-senlinclient: Unit test for /v1/shell.py part3 https://review.openstack.org/258416 | 10:46 |
| *** elynn_ has quit IRC | 10:58 | |
| *** Qiming has joined #senlin | 11:15 | |
| *** pratikmallya has joined #senlin | 11:24 | |
| openstackgerrit | Merged openstack/senlin: Implement purge_deleted command https://review.openstack.org/257776 | 11:28 |
| *** pratikmallya has quit IRC | 11:44 | |
| *** pratikmallya has joined #senlin | 13:09 | |
| *** zhenguo has joined #senlin | 13:16 | |
| *** cschulz has joined #senlin | 13:18 | |
| cschulz | Hi, Has anyone created Heat resources that represent Senlin objects such as OS::Senlin::Cluster? | 13:19 |
| *** pratikmallya has quit IRC | 13:24 | |
| Qiming | cschulz, yes | 13:38 |
| Qiming | cschulz, https://review.openstack.org/247429 | 13:39 |
| Qiming | cschulz, https://review.openstack.org/247430 | 13:39 |
| Qiming | cschulz, https://review.openstack.org/258344 | 13:39 |
| cschulz | Is there a repository I can clone to get those? They're not in github.com/openstack/heat.git. | 13:58 |
| Qiming | they are still under development | 14:22 |
| Qiming | only when the code is reviewed and merged can you get it from github | 14:22 |
| *** Liuqing has quit IRC | 14:24 | |
| Qiming | cschulz, even when cluster resource type is merged, it is not a recommended way to do autoscaling in heat | 14:28 |
| cschulz | But we are evaluating Senlin and it doesn't make much sense unless you can use it to create clusters which are part of a Heat stack. | 14:30 |
| Qiming | why is that? | 14:30 |
| *** Liuqing has joined #senlin | 14:31 | |
| Qiming | senlin creates clusters of heat stacks, and does autoscaling with more knobs (i.e. verbs) | 14:31 |
| cschulz | If, as in our case, you need to build a multi-tiered solution, you need Heat orchestration. | 14:31 |
| Qiming | with Heat, the only way to make change to your autoscaling solution is to do stack-update | 14:31 |
| cschulz | And if some part of that stack needs to be autoscaled, you need a cluster. | 14:31 |
| cschulz | Ah, but you need to think beyond that. | 14:32 |
| cschulz | With heat I should be able to create a stack that includes some alarms which get connected to a Senlin cluster. | 14:32 |
| cschulz | To do the autoscaling. | 14:33 |
| Qiming | yes, in the long run, we have reasons to support senlin clusters in Heat | 14:33 |
| Qiming | right, with heat, you will create ceilometer alarms to trigger auto-scaling | 14:34 |
| cschulz | Yes, and from our auto-scaling standpoint, Senlin isn't very compelling unless we can build Heat stacks which include Senlin clusters for auto-scaling. | 14:34 |
| Qiming | alright | 14:35 |
| *** Liuqing has quit IRC | 14:36 | |
| cschulz | I just really got my head around this stuff yesterday. And immediately came to the conclusion that what I needed was OS::Senlin::Cluster resources so I could build my stacks. | 14:36 |
| *** Liuqing has joined #senlin | 14:37 | |
| Qiming | okay, won't try convince you then, :) | 14:38 |
| cschulz | Looking at the Blueprint for Senlin::Cluster, I think an important aspect is missing. A cluster should have as one property a list of policies to be attached. | 14:38 |
| Qiming | if you need helps on this, we are always here, :) | 14:38 |
| Qiming | cschulz, makes sense | 14:39 |
| cschulz | We were ready to go off and create our own Senlin::Cluster resource till I asked the question this morning. | 14:40 |
| cschulz | Is there somewhere we can get the code that has been built? | 14:40 |
| Qiming | haha, elynn is from IBM too | 14:40 |
| cschulz | OK thanks | 14:40 |
| Qiming | you can do a 'git review -d 247429' to get a copy of the code under review | 14:41 |
| cschulz | 'review' is not a valid command for 'git' | 14:50 |
| Qiming | well... you don't have git-review installed | 14:53 |
| Qiming | try this? git fetch https://review.openstack.org/openstack/heat refs/changes/29/247429/8 && git checkout FETCH_HEAD | 14:53 |
| Qiming | or: git pull https://review.openstack.org/openstack/heat refs/changes/29/247429/8 | 14:53 |
| cschulz | got it. I'm new to Gerrit stuff. | 14:58 |
| *** Qiming has quit IRC | 15:16 | |
| *** zhenguo has quit IRC | 15:49 | |
| *** Liuqing has quit IRC | 15:50 | |
| -openstackstatus- NOTICE: Gerrit will be offline for a software upgrade from 17:00 to 21:00 UTC. See: http://lists.openstack.org/pipermail/openstack-dev/2015-December/081037.html | 16:21 | |
| *** pratikmallya has joined #senlin | 16:33 | |
| -openstackstatus- NOTICE: Gerrit is offline for a software upgrade from 17:00 to 21:00 UTC. See: http://lists.openstack.org/pipermail/openstack-dev/2015-December/081037.html | 17:06 | |
| *** ChanServ changes topic to "Gerrit is offline for a software upgrade from 17:00 to 21:00 UTC. See: http://lists.openstack.org/pipermail/openstack-dev/2015-December/081037.html" | 17:06 | |
| *** pratikmallya has quit IRC | 17:27 | |
| *** pratikmallya has joined #senlin | 19:33 | |
| *** ChanServ changes topic to "IRCLog: http://eavesdrop.openstack.org/irclogs/%23senlin/ | Bugs: bugs.launchpad.net/senlin | Review: https://review.openstack.org/#/q/project:openstack/senlin,n,z" | 21:02 | |
| -openstackstatus- NOTICE: Gerrit has been upgraded to 2.11. Please report any issues in #openstack-infra as soon as possible. | 21:02 | |
| *** Qiming has joined #senlin | 23:43 | |
| *** pratikmallya has quit IRC | 23:54 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!