| *** gmoro has quit IRC | 00:00 | |
| *** k_mouza has joined #heat | 00:04 | |
| *** ysandeep has joined #heat | 01:12 | |
| *** k_mouza has quit IRC | 01:15 | |
| *** hongbin has joined #heat | 01:36 | |
| *** liyi has joined #heat | 02:05 | |
| *** Roedy has quit IRC | 02:06 | |
| *** Roedy has joined #heat | 02:06 | |
| *** liyi has quit IRC | 02:10 | |
| *** links has joined #heat | 02:11 | |
| *** yangyapeng has quit IRC | 02:30 | |
| *** yangyapeng has joined #heat | 03:44 | |
| *** kiennt26 has joined #heat | 04:06 | |
| *** liyi has joined #heat | 04:06 | |
| *** liyi has quit IRC | 04:11 | |
| *** blomquisg is now known as blomquisg_gone | 04:18 | |
| *** kiennt26 has quit IRC | 05:01 | |
| *** mdnadeem has joined #heat | 05:47 | |
| *** ramishra has joined #heat | 05:50 | |
| *** hongbin has quit IRC | 05:53 | |
| *** skramaja has joined #heat | 06:03 | |
| openstackgerrit | Rabi Mishra proposed openstack/heat master: Fix race conditions in test_create_update https://review.openstack.org/498657 | 06:05 |
|---|---|---|
| openstackgerrit | OpenStack Proposal Bot proposed openstack/heat-dashboard master: Imported Translations from Zanata https://review.openstack.org/545749 | 06:07 |
| *** liyi has joined #heat | 06:07 | |
| *** liyi has quit IRC | 06:12 | |
| *** rcernin has quit IRC | 07:11 | |
| *** armaan has joined #heat | 07:15 | |
| *** hoonetorg has quit IRC | 07:30 | |
| *** hoonetorg has joined #heat | 07:47 | |
| *** links has quit IRC | 07:55 | |
| *** AlexeyAbashkin has joined #heat | 07:56 | |
| *** links has joined #heat | 08:08 | |
| *** sshnaidm|off has quit IRC | 08:35 | |
| *** sshnaidm has joined #heat | 08:44 | |
| *** dbecker has joined #heat | 09:00 | |
| *** gfidente has joined #heat | 09:00 | |
| *** jtomasek has joined #heat | 09:03 | |
| *** sshnaidm has quit IRC | 09:07 | |
| *** sshnaidm has joined #heat | 09:08 | |
| therve | ramishra, Around? Wanted to chat about https://bugs.launchpad.net/tripleo/+bug/1750270 | 09:12 |
| openstack | Launchpad bug 1750270 in tripleo "[queens promotion] - fs001 fails overcloud deployment with 'MistralOvercloud configuration failed'" [Critical,Triaged] | 09:12 |
| *** k_mouza has joined #heat | 09:13 | |
| ramishra | therve: yep, did we not fix that on Friday? | 09:14 |
| therve | ramishra, I don't think so no | 09:14 |
| therve | ramishra, You're thinking https://bugs.launchpad.net/tripleo/+bug/1749645 ? | 09:14 |
| openstack | Launchpad bug 1749645 in python-mistralclient "Overcloud deployment failing in promotion jobs at overcloud.AllNodesDeploySteps.WorkflowTasks_Step2_Execution" [Undecided,New] | 09:14 |
| ramishra | therve: yep, thought it's duplicate;) | 09:15 |
| therve | No | 09:15 |
| therve | ramishra, I wonder if we broke get_attr on ResourceGroup | 09:15 |
| therve | ramishra, We do a get_attr: group, stuff | 09:16 |
| therve | And it returns null | 09:16 |
| therve | I would expect it to return [], though I'm not sure | 09:16 |
| ramishra | Hum, we checked experimental jobs with those changes no? possibly try with a revert if you think of a particulat patch? | 09:17 |
| therve | https://review.openstack.org/#/c/531929/ yeah | 09:18 |
| therve | Looking at logs | 09:18 |
| therve | ramishra, I don't think there are enough roles on that particular job :/ | 09:19 |
| ramishra | therve: https://logs.rdoproject.org/openstack-periodic/periodic-tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset001-queens/f8fd98f/undercloud/var/log/mistral/executor.log.txt.gz#_2018-02-18_12_04_19_640 | 09:21 |
| *** derekh has joined #heat | 09:21 | |
| ramishra | not related probably? | 09:21 |
| therve | ramishra, The zaqar one? | 09:22 |
| ramishra | yeah | 09:22 |
| therve | Yeah I don't think it matters | 09:22 |
| therve | ramishra, So I took logs from the 15th, and it returned [], not null | 09:24 |
| therve | Now to see heat version... | 09:24 |
| therve | Ah no... | 09:25 |
| therve | It failed before | 09:25 |
| *** links has quit IRC | 09:32 | |
| therve | ramishra, https://logs.rdoproject.org/81/544781/2/openstack-check/gate-tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset001-master/Z2b50d57a3952433cb8228812daaa9069/postci.txt.gz see ServerIdData | 09:34 |
| therve | It returns [] not null | 09:34 |
| *** links has joined #heat | 09:46 | |
| *** SmearedBeard has joined #heat | 09:47 | |
| *** ysandeep has quit IRC | 10:00 | |
| *** serlex has joined #heat | 10:05 | |
| *** serlex has quit IRC | 10:05 | |
| *** mdnadeem has quit IRC | 10:11 | |
| *** shardy has joined #heat | 10:15 | |
| therve | ramishra, It looks like the group is size 0 | 10:25 |
| therve | But on my testing, getting attributes out returns [] properly... | 10:25 |
| ramishra | therve: Is it related to deployed-server thing in tripleo? | 10:25 |
| therve | ramishra, No I don't think so | 10:26 |
| *** mdnadeem has joined #heat | 10:27 | |
| therve | It's for config download | 10:27 |
| *** mdnadeem_ has joined #heat | 10:30 | |
| ramishra | yeah, I get that, I think they switched it as the default, was wondering how come it would behave differently in 2 different jobs for the same set of outputs, roles are different but still | 10:32 |
| *** mdnadeem has quit IRC | 10:32 | |
| ramishra | therve: Is it only happening in Queens? | 10:35 |
| therve | ramishra, Well, I don't know the status of master promotion | 10:36 |
| therve | Looks like 26 days old, so we won't see it there either :/ | 10:36 |
| *** gkadam has joined #heat | 10:39 | |
| openstackgerrit | Merged openstack/heat master: heat_docker: Require docker instead of docker-py https://review.openstack.org/533209 | 10:40 |
| *** AlexeyAbashkin has quit IRC | 11:07 | |
| *** k_mouza has quit IRC | 11:17 | |
| *** k_mouza_ has joined #heat | 11:17 | |
| *** AlexeyAbashkin has joined #heat | 11:32 | |
| *** pbourke has quit IRC | 11:37 | |
| *** pbourke has joined #heat | 11:38 | |
| *** bnemec is now known as bnemec-pto | 11:41 | |
| *** masber has joined #heat | 11:58 | |
| *** slagle has joined #heat | 12:20 | |
| *** stevebaker has quit IRC | 12:25 | |
| *** dmellado has quit IRC | 12:26 | |
| *** stevebaker has joined #heat | 12:29 | |
| *** stevebaker has quit IRC | 12:29 | |
| *** stevebaker has joined #heat | 12:29 | |
| *** stevebaker has quit IRC | 12:36 | |
| *** links has quit IRC | 12:37 | |
| *** sshnaidm has quit IRC | 12:42 | |
| openstackgerrit | Merged openstack/heat master: Fix some typos https://review.openstack.org/542630 | 12:45 |
| *** dprince has joined #heat | 12:57 | |
| *** dmellado has joined #heat | 13:07 | |
| *** mchlumsky has joined #heat | 13:17 | |
| *** skramaja has quit IRC | 13:25 | |
| *** sshnaidm has joined #heat | 13:25 | |
| *** gkadam has quit IRC | 13:31 | |
| *** mchlumsky has quit IRC | 13:35 | |
| *** mchlumsky has joined #heat | 13:38 | |
| *** dmellado has quit IRC | 13:38 | |
| *** jcoufal has joined #heat | 13:44 | |
| *** masuberu has joined #heat | 13:54 | |
| *** masber has quit IRC | 13:58 | |
| *** armaan has quit IRC | 14:01 | |
| *** armaan has joined #heat | 14:02 | |
| *** armaan has quit IRC | 14:08 | |
| *** ramishra has quit IRC | 14:31 | |
| *** jistr is now known as jistr|mtg | 14:31 | |
| *** blomquisg_gone is now known as blomquisg | 14:41 | |
| *** links has joined #heat | 14:42 | |
| *** dmellado has joined #heat | 14:44 | |
| *** sshnaidm has quit IRC | 14:46 | |
| *** armaan has joined #heat | 14:46 | |
| *** k_mouza_ has quit IRC | 14:46 | |
| *** sshnaidm has joined #heat | 14:47 | |
| *** k_mouza has joined #heat | 14:47 | |
| *** armaan has quit IRC | 14:48 | |
| *** links has quit IRC | 14:49 | |
| *** stevebaker has joined #heat | 14:49 | |
| *** k_mouza has quit IRC | 14:50 | |
| *** hjensas has quit IRC | 14:50 | |
| *** k_mouza has joined #heat | 14:50 | |
| *** dmellado has quit IRC | 15:02 | |
| *** jistr|mtg is now known as jistr | 15:02 | |
| *** links has joined #heat | 15:02 | |
| *** stevebaker has quit IRC | 15:07 | |
| openstackgerrit | Merged openstack/heat master: Fix README.rst for in-tree functional tests https://review.openstack.org/540822 | 15:07 |
| -openstackstatus- NOTICE: Zuul has been restarted to pick up latest memory fixes. Queues were saved however patches uploaded after 14:40UTC may have been missed. Please recheck if needed. | 15:14 | |
| *** armaan has joined #heat | 15:19 | |
| *** mdnadeem_ has quit IRC | 15:20 | |
| *** k_mouza_ has joined #heat | 15:24 | |
| *** k_mouza has quit IRC | 15:24 | |
| *** gieldops has joined #heat | 15:29 | |
| therve | zaneb, When you're around, need chat about https://bugs.launchpad.net/tripleo/+bug/1750270 | 15:46 |
| openstack | Launchpad bug 1750270 in tripleo "[queens promotion] - fs001 fails overcloud deployment with 'MistralOvercloud configuration failed'" [Critical,Triaged] - Assigned to Thomas Herve (therve) | 15:46 |
| * zaneb looks | 15:47 | |
| zaneb | therve: do we know the SHA of what they're trying to promote? | 15:48 |
| *** mhayden has quit IRC | 15:48 | |
| therve | zaneb, We can know :) | 15:48 |
| therve | zaneb, 40a32f8 | 15:49 |
| *** mhayden has joined #heat | 15:49 | |
| therve | zaneb, From what I understand, getting an attribute from ResourceGroup returns null instead of [] | 15:49 |
| therve | zaneb, That ResourceGroup being size 0 | 15:50 |
| *** mhayden has quit IRC | 15:53 | |
| zaneb | well, _nested_output_defns() *always* defines a list... so it shouldn't be that | 15:53 |
| *** mhayden has joined #heat | 15:53 | |
| zaneb | I wonder if it's something in the fallback code | 15:53 |
| therve | zaneb, You mean the the debug stuff? | 15:55 |
| therve | I checked and haven't seen it in the logs | 15:55 |
| *** hjensas has joined #heat | 15:55 | |
| zaneb | hmm, the mystery deepens then | 15:55 |
| zaneb | does our experimental job really only deploy a single controller and no other roles? | 16:08 |
| *** blomquisg is now known as blomquisg_gone | 16:08 | |
| *** blomquisg_gone is now known as blomquisg | 16:10 | |
| therve | Yeah seems like it... | 16:10 |
| *** mdnadeem_ has joined #heat | 16:12 | |
| zaneb | no wonder they have promotion failures all the time :( | 16:19 |
| *** masuberu has quit IRC | 16:21 | |
| therve | Heh | 16:22 |
| therve | Well we're not responsible for that many of them | 16:22 |
| *** mdnadeem_ has quit IRC | 16:22 | |
| openstackgerrit | Giel Dops proposed openstack/heat master: Add value_specs to all Neutron resources https://review.openstack.org/541788 | 16:23 |
| therve | zaneb, Yeah that's weird, we don't seem to create an OutputDefinition if there is no value? | 16:33 |
| zaneb | therve: are you referring to http://git.openstack.org/cgit/openstack/heat/tree/heat/engine/resources/openstack/heat/resource_group.py#n571 ? | 16:35 |
| therve | zaneb, yeah | 16:35 |
| *** mhayden has quit IRC | 16:35 | |
| zaneb | therve: the only way we can hit that path is if the key is in self.ATTRIBUTES but it's not ATTR_ATTRIBUTES or REFS | 16:37 |
| zaneb | IOW it means only that we don't create an output for the 'removed_rsrc_list' attribute | 16:38 |
| zaneb | or refs_map, but that's handled unconditionally after the loop | 16:40 |
| therve | Hum | 16:40 |
| therve | zaneb, key can be an arbitrary thing right? | 16:40 |
| therve | Didn't get you "in self.ATTRIBUTES but it's not ATTR_ATTRIBUTES or REFS" | 16:40 |
| zaneb | it can, yes | 16:41 |
| zaneb | http://git.openstack.org/cgit/openstack/heat/tree/heat/engine/resources/openstack/heat/resource_group.py#n565 | 16:41 |
| therve | Right that's where we ought to be | 16:41 |
| therve | And actually what I see when testing a simple stack | 16:41 |
| zaneb | if it's not an attribute in the schema then we will generate an output for it | 16:41 |
| zaneb | fwiw ServerOsCollectConfigData also contains nulls, and that's using {get_attr: [{{role.name}}, attributes, os_collect_config]} | 16:43 |
| zaneb | so it also fails if the key is ATTR_ATTRIBUTES | 16:43 |
| zaneb | in any event, if there were no output created then we should be seeing the log that says it's falling back to grouputils, and we're not | 16:44 |
| *** armaan has quit IRC | 16:44 | |
| therve | Maybe those logs are silenced somehow? | 16:45 |
| zaneb | anything is possible, but that seems extraordinarily unlikely | 16:46 |
| *** mhayden has joined #heat | 16:46 | |
| zaneb | most likely get_output is returning None or None is getting stored in a cache somewhere | 16:47 |
| zaneb | therve: btw in your test are you using an OS::Heat::Value to store an intermediate result? | 16:49 |
| therve | zaneb, Yeah | 16:50 |
| *** AlexeyAbashkin has quit IRC | 16:50 | |
| therve | But yes I think that's a clue | 16:51 |
| therve | We're storing something at the wrong point somehow | 16:51 |
| *** k_mouza_ has quit IRC | 16:52 | |
| *** k_mouza has joined #heat | 16:52 | |
| therve | Hummmm | 16:53 |
| zaneb | that data gets stored when the Value resources is created... and it comes straight from the NodeData... it's really surprising that that could be wrong | 16:53 |
| therve | zaneb, conditionals are involved it looks like | 16:53 |
| zaneb | and yet here we are ;) | 16:53 |
| zaneb | oh, really? | 16:53 |
| * zaneb checks templates again | 16:53 | |
| therve | I mean; not sure it's relevant, because it's in the nested thing, and we have a size 0 | 16:54 |
| therve | But https://github.com/openstack/tripleo-heat-templates/blob/master/puppet/role.role.j2.yaml#L715 is there | 16:54 |
| *** k_mouza has quit IRC | 16:57 | |
| zaneb | ah | 16:58 |
| zaneb | so there's no output called "nova_server_resource" | 16:59 |
| *** hjensas has quit IRC | 16:59 | |
| zaneb | they seem to be relying on it being able to pull a resource ID out of the nested nested stacks | 17:00 |
| zaneb | tbh I'm not sure I knew that was something grouputils allowed | 17:01 |
| therve | Ah! Reproduced | 17:02 |
| zaneb | oh no, ignore me, there is an output name nova_server_resource | 17:02 |
| * zaneb must have flipped back to the other template | 17:03 | |
| *** armaan has joined #heat | 17:03 | |
| zaneb | therve: what's the secret? :) | 17:03 |
| therve | Not sure yet :) | 17:03 |
| therve | Ah | 17:05 |
| therve | zaneb, Be in progress... | 17:05 |
| therve | Looks like complete here | 17:05 |
| openstackgerrit | Vadym Markov proposed openstack/heat master: Switch designate client to API v2 by default https://review.openstack.org/545885 | 17:06 |
| *** links has quit IRC | 17:09 | |
| *** dmellado has joined #heat | 17:10 | |
| *** k_mouza has joined #heat | 17:13 | |
| openstackgerrit | Giel Dops proposed openstack/heat master: Add value_specs to all Neutron resources https://review.openstack.org/541788 | 17:18 |
| *** felipemonteiro has joined #heat | 17:18 | |
| *** shardy has quit IRC | 17:41 | |
| *** k_mouza has quit IRC | 17:49 | |
| *** k_mouza has joined #heat | 17:49 | |
| *** k_mouza has quit IRC | 17:53 | |
| *** sshnaidm is now known as sshnaidm|off | 17:54 | |
| *** felipemonteiro has quit IRC | 17:54 | |
| *** felipemonteiro has joined #heat | 17:54 | |
| *** felipemonteiro has quit IRC | 17:59 | |
| *** derekh has quit IRC | 18:00 | |
| *** armaan has quit IRC | 18:05 | |
| *** hongbin has joined #heat | 18:07 | |
| *** k_mouza has joined #heat | 18:14 | |
| *** AlexeyAbashkin has joined #heat | 18:15 | |
| *** rwsu has quit IRC | 18:17 | |
| *** openstackgerrit has quit IRC | 18:18 | |
| *** AlexeyAbashkin has quit IRC | 18:20 | |
| *** hjensas has joined #heat | 18:23 | |
| *** stevebaker has joined #heat | 18:26 | |
| *** hjensas has quit IRC | 18:27 | |
| *** jrist is now known as jrist-afk | 18:28 | |
| *** jrist-afk is now known as jrist | 18:28 | |
| *** rwsu has joined #heat | 18:30 | |
| *** hjensas has joined #heat | 18:44 | |
| *** k_mouza has quit IRC | 18:46 | |
| *** rwsu has quit IRC | 18:46 | |
| *** rwsu has joined #heat | 19:01 | |
| *** felipemonteiro has joined #heat | 19:02 | |
| *** harlowja has joined #heat | 19:03 | |
| *** gfidente is now known as gfidente|afk | 19:05 | |
| *** gmoro has joined #heat | 19:06 | |
| *** blomquisg is now known as blomquisg_gone | 19:06 | |
| *** gmoro_ has quit IRC | 19:07 | |
| *** hongbin_ has joined #heat | 20:01 | |
| *** hongbin has quit IRC | 20:03 | |
| *** openstackgerrit has joined #heat | 20:30 | |
| openstackgerrit | Dirk Mueller proposed openstack/heat-cfntools master: Remove discover from test-requirements https://review.openstack.org/545972 | 20:30 |
| *** armaan has joined #heat | 20:31 | |
| *** armaan has quit IRC | 20:31 | |
| *** felipemonteiro_ has joined #heat | 20:32 | |
| *** felipemonteiro has quit IRC | 20:35 | |
| *** AlexeyAbashkin has joined #heat | 21:26 | |
| *** clayton has quit IRC | 21:28 | |
| *** hongbin has joined #heat | 21:29 | |
| *** mhayden has quit IRC | 21:29 | |
| *** hongbin_ has quit IRC | 21:30 | |
| *** mhayden has joined #heat | 21:30 | |
| *** AlexeyAbashkin has quit IRC | 21:31 | |
| *** dprince has quit IRC | 21:33 | |
| *** rcernin has joined #heat | 21:36 | |
| *** felipemonteiro_ has quit IRC | 21:45 | |
| *** felipemonteiro_ has joined #heat | 21:45 | |
| *** armaan has joined #heat | 21:52 | |
| *** felipemonteiro__ has joined #heat | 21:53 | |
| *** jcoufal has quit IRC | 21:55 | |
| *** felipemonteiro_ has quit IRC | 21:57 | |
| *** openstackgerrit has quit IRC | 22:18 | |
| *** masber has joined #heat | 22:23 | |
| *** armaan has quit IRC | 22:44 | |
| *** armaan has joined #heat | 22:46 | |
| *** k_mouza has joined #heat | 22:52 | |
| *** k_mouza has quit IRC | 22:56 | |
| *** AlexeyAbashkin has joined #heat | 23:06 | |
| *** AlexeyAbashkin has quit IRC | 23:10 | |
| *** SmearedBeard has quit IRC | 23:12 | |
| *** felipemonteiro__ has quit IRC | 23:25 | |
| *** hongbin has quit IRC | 23:27 | |
| *** masber has quit IRC | 23:34 | |
| *** masber has joined #heat | 23:37 | |
| *** masber has quit IRC | 23:57 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!