*** zhurong has quit IRC | 00:06 | |
*** zhurong has joined #senlin | 01:13 | |
*** yanyanhu has joined #senlin | 01:24 | |
*** Drago has joined #senlin | 01:37 | |
*** Drago has joined #senlin | 01:38 | |
*** jdandrea has quit IRC | 02:04 | |
*** elynn has joined #senlin | 02:16 | |
*** Drago has quit IRC | 02:28 | |
*** Drago has joined #senlin | 02:31 | |
*** yuanying has quit IRC | 02:46 | |
*** yuanying has joined #senlin | 02:50 | |
Qiming | lixinhui, the latest change still contains some blank lines at the end of the template, maybe that is the reason pep still fails | 03:27 |
---|---|---|
openstackgerrit | Qiming Teng proposed openstack/senlin: Remove usage of mox/mox3 https://review.openstack.org/358383 | 03:41 |
*** yuanying has quit IRC | 03:45 | |
lixinhui | Qiming, done | 03:47 |
*** yuanying has joined #senlin | 03:48 | |
Qiming | thx! | 03:49 |
openstackgerrit | Yanyan Hu proposed openstack/senlin: Add version support to schema and spec https://review.openstack.org/348709 | 03:56 |
*** elynn has quit IRC | 04:35 | |
*** elynn has joined #senlin | 04:57 | |
Qiming | hi, just submitted a patch to oslo.db to fix the node list problem | 06:09 |
Qiming | when the created_at field is None, oslo.db is messing things up by doing a '<' or '>' comparison on it | 06:10 |
Qiming | https://review.openstack.org/358410 | 06:10 |
Qiming | elynn, there? | 06:12 |
elynn | Qiming, yes | 06:12 |
Qiming | hi, what's the plan to get profile-validate and policy-validate added to sdk? | 06:13 |
Qiming | we will need that asap before 0.9.3 is cut | 06:13 |
Qiming | I don't think we will have a chance to release a 0.9.4 in coming weeks | 06:14 |
elynn | If I submit a patch to add it today, will it get merged in 0.9.3? | 06:14 |
Qiming | that chance is big | 06:14 |
elynn | okay, then today I will submit patches to sdk | 06:14 |
Qiming | great, thanks | 06:15 |
elynn | and also add api for policy validation. | 06:15 |
elynn | Might be tonight. | 06:15 |
yanyanhu | hi, Qiming, thanks a lot for proposing this patch 358410 | 06:19 |
yanyanhu | there is bug reported submitted before: https://bugs.launchpad.net/senlin/+bug/1594352 | 06:19 |
openstack | Launchpad bug 1594352 in senlin "could not list node sorted by "created_at:desc" " [High,Triaged] | 06:19 |
Qiming | yes, was proposing this to fix that bug | 06:19 |
yanyanhu | maybe we can add the link to the commit msg | 06:19 |
yanyanhu | thanks | 06:19 |
Qiming | we don't have a bug report to oslo.db | 06:20 |
yanyanhu | oh, right | 06:20 |
yanyanhu | that bug is reported in senlin side | 06:20 |
Qiming | but I have added oslo.db as an affected project | 06:20 |
yanyanhu | cool, | 06:20 |
yanyanhu | actually, ruijie from ddtdream asked me for this problem last week | 06:20 |
yanyanhu | I think this will be helpful for him as well | 06:21 |
yanyanhu | they met this issue when testing senlin | 06:21 |
Qiming | yep | 06:25 |
Qiming | they can have a local fix before oslo.db merges it | 06:25 |
yanyanhu | yes, will make them to be aware of this fix | 06:34 |
openstackgerrit | Qiming Teng proposed openstack/senlin: Improve server metadata format https://review.openstack.org/356302 | 06:56 |
yanyanhu | hi, Qiming I noticed gate jobs all failed today | 07:03 |
Qiming | yes | 07:03 |
yanyanhu | I checked the log and found it was caused by weird request from keystoneauth to 127.0.0.1 url | 07:03 |
yanyanhu | not sure why it happened since that is the home page of dashboard | 07:04 |
Qiming | we have got quite some patches not reviewed before | 07:04 |
Qiming | now blocked by the gate ... | 07:04 |
Qiming | it is a bug in sdk | 07:04 |
yanyanhu | I see... | 07:05 |
Qiming | https://review.openstack.org/#/c/354492/ | 07:05 |
Qiming | fixed by this one | 07:05 |
Qiming | but it is not part of 0.9.2 ... | 07:05 |
yanyanhu | I see | 07:05 |
yanyanhu | hope sdk 0.9.3 comes soon | 07:06 |
Qiming | but if sdk cuts 0.9.3 today, then we won't have a chance to get profile/policy validate out | 07:06 |
yanyanhu | yes, ethan still need some time to finish it | 07:07 |
yanyanhu | so 0.9.3 will be the final release of sdk in N cycle? | 07:07 |
yanyanhu | for feature freeze is coming | 07:07 |
Qiming | I am not sure we can persuade brian to do another release | 07:09 |
*** zhurong has quit IRC | 07:09 | |
Qiming | he has got pretty short a window too | 07:09 |
*** zhurong has joined #senlin | 07:13 | |
yanyanhu | I see | 07:22 |
openstackgerrit | Qiming Teng proposed openstack/senlin: Close health manager loop https://review.openstack.org/356303 | 07:27 |
openstackgerrit | Qiming Teng proposed openstack/senlin: Enable health policy to handle CLUSTER_SCALE_IN https://review.openstack.org/358454 | 07:43 |
openstackgerrit | Qiming Teng proposed openstack/senlin: Enable health policy to handle CLUSTER_SCALE_IN https://review.openstack.org/358454 | 08:18 |
openstackgerrit | Qiming Teng proposed openstack/senlin: Improve server metadata format https://review.openstack.org/356302 | 08:18 |
openstackgerrit | Qiming Teng proposed openstack/senlin: Close health manager loop https://review.openstack.org/356303 | 08:18 |
openstackgerrit | Qiming Teng proposed openstack/senlin: Enable health policy to handle CLUSTER_SCALE_IN https://review.openstack.org/358454 | 08:22 |
openstackgerrit | Qiming Teng proposed openstack/senlin: Enable health policy to handle CLUSTER_SCALE_IN https://review.openstack.org/358454 | 08:27 |
*** yuanying has quit IRC | 08:34 | |
*** yuanying has joined #senlin | 08:35 | |
*** yuanying has quit IRC | 08:47 | |
*** yuanying has joined #senlin | 08:49 | |
openstackgerrit | Ethan Lynn proposed openstack/senlin: Add new api for profile validation https://review.openstack.org/354549 | 08:56 |
openstackgerrit | Ethan Lynn proposed openstack/senlin: Fix do_validate in heat profile https://review.openstack.org/355780 | 08:56 |
openstackgerrit | Yanyan Hu proposed openstack/senlin: Add version support to schema and spec https://review.openstack.org/348709 | 09:17 |
*** yanyanhu has quit IRC | 09:21 | |
elynn | Can't push patches to review, will go home an continue.... | 09:48 |
openstackgerrit | Qiming Teng proposed openstack/senlin: Enable health policy to handle CLUSTER_SCALE_IN https://review.openstack.org/358454 | 09:49 |
*** elynn has quit IRC | 09:49 | |
Qiming | elynn, what happened? | 09:53 |
Qiming | shit ... the whole git history is now corrupted | 09:54 |
Qiming | no one ever helped review the health manager patch ... | 09:54 |
Qiming | spent a whole damn day on fixing this ... | 09:55 |
*** zhurong has quit IRC | 09:58 | |
*** xuhaiwei has quit IRC | 10:40 | |
*** lixinhui has quit IRC | 10:40 | |
*** lixinhui has joined #senlin | 10:40 | |
*** openstackgerrit has quit IRC | 11:03 | |
*** openstackgerrit has joined #senlin | 11:04 | |
*** ChanServ sets mode: +v openstackgerrit | 11:04 | |
*** openstackstatus has quit IRC | 11:36 | |
*** openstackstatus has joined #senlin | 11:39 | |
*** ChanServ sets mode: +v openstackstatus | 11:39 | |
*** shu-mutou-OFF has quit IRC | 12:08 | |
*** rvasilets__ has left #senlin | 12:45 | |
openstackgerrit | Merged openstack/python-senlinclient: Reorder required parameters https://review.openstack.org/354100 | 12:58 |
*** elynn has joined #senlin | 13:24 | |
*** elynn has quit IRC | 13:26 | |
*** elynn has joined #senlin | 13:31 | |
elynn | Hi | 13:38 |
elynn | do you know why our jenkins were failed? | 13:39 |
*** elynn has quit IRC | 13:52 | |
*** elynn has joined #senlin | 14:02 | |
*** elynn has quit IRC | 14:06 | |
*** elynn has joined #senlin | 14:08 | |
*** elynn has quit IRC | 14:17 | |
*** elynn has joined #senlin | 14:22 | |
openstackgerrit | Ethan Lynn proposed openstack/senlin: Add policy_validate function to engine https://review.openstack.org/358686 | 14:42 |
Qiming | elynn, because sdk has a buggy patch merged I think | 15:09 |
elynn | okay, so we are waiting for new sdk version? | 15:09 |
Qiming | yes | 15:10 |
Qiming | you don't have to worry about the gate | 15:11 |
Qiming | just push the changes needed for profile/policy validate | 15:11 |
elynn | yes, I will, working on it now. | 15:11 |
*** Drago has joined #senlin | 15:11 | |
Qiming | then when sdk has a 0.9.3 release, everything will be fine | 15:12 |
Qiming | speaking of validation | 15:12 |
Qiming | I'm wondering if they are meant to be synchronous calls | 15:12 |
Qiming | is stack-preview a synchronous call? | 15:12 |
elynn | stack-preview is synchronous | 15:13 |
Qiming | okay | 15:13 |
elynn | No other way to check result | 15:13 |
Qiming | let's have validation synchronous as well | 15:13 |
Qiming | it doesn't make a lot senses to make them asynchronous | 15:13 |
elynn | okay~ | 15:14 |
Qiming | it feels very strange if users have to check the action to find out if validation passes | 15:14 |
Drago | Just wanted to note, the Heat team has discussed making stack preview asynchronous because for very large stacks, validation can take a long time (minutes) | 15:15 |
elynn | Hmm... interesting... | 15:16 |
Drago | The plan was to validate just enough synchronously to know if the Heat template was well-formed enough that Heat could store *something* in the database for the user to check back to | 15:16 |
Drago | This was discussed at the Austin summit. I don't know if has been implemented yet | 15:17 |
elynn | I would say that it's better to make validation process parallels, then it would get faster ;) | 15:18 |
elynn | Haven't seen any changes on validation yet... | 15:18 |
Drago | See https://etherpad.openstack.org/p/heat-newton-validation-improvements | 15:18 |
elynn | Drago, thanks for remind :) | 15:21 |
elynn | Seems they are talking about speed up validation process, so that user can get a quick api response. | 15:24 |
Qiming | thanks, drago, there is always a gray area when we want to draw a line between "well-formed" and "working" | 15:24 |
Drago | Yes, the rationale was "just enough that we can store something in the DB" | 15:30 |
Qiming | see you guys, going to bed now, has been a long day ... | 15:31 |
elynn | see u~ | 15:40 |
openstackgerrit | Ethan Lynn proposed openstack/senlin: Add new api for policy validation https://review.openstack.org/358759 | 16:34 |
*** Drago has quit IRC | 16:55 | |
*** elynn has quit IRC | 17:02 | |
*** Drago has joined #senlin | 18:08 | |
*** Drago has quit IRC | 18:29 | |
*** Drago has joined #senlin | 18:29 | |
*** Drago has quit IRC | 18:30 | |
*** Drago has joined #senlin | 18:31 | |
*** Drago1 has joined #senlin | 19:35 | |
*** Drago has quit IRC | 19:36 | |
*** Drago1 is now known as Drago | 20:36 | |
*** Drago has quit IRC | 21:03 | |
*** Drago has joined #senlin | 21:03 | |
*** Drago has quit IRC | 21:43 | |
*** Drago has joined #senlin | 21:43 | |
*** Drago has quit IRC | 22:10 | |
*** Drago has joined #senlin | 22:11 | |
*** elynn has joined #senlin | 22:59 | |
*** haiwei has joined #senlin | 23:31 | |
*** shu-mutou has joined #senlin | 23:42 | |
*** Drago has quit IRC | 23:58 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!