Open Stack

There is a bug in how yaql interprets its 3rd argument (aggregator). It was
supposed to be a function to aggregate values within each group but instead
was applied to the outcome of groupBy. I submitted a fix for this:

https://review.openstack.org/363191. Though I'm not sure we can release new
yaql version that late in the Newton cycle. Meanwhile it is better to use
one of alternative solutions above so that it won't break after this patch
get merged.

Sincerely yours,
Stan Lagun
Principal Software Engineer @ Mirantis

<slagun at mirantis.com>

On Tue, Aug 30, 2016 at 12:46 PM, Thomas Herve <therve at redhat.com> wrote:

> On Tue, Aug 30, 2016 at 6:02 PM, Steven Hardy <shardy at redhat.com> wrote:
> > On Tue, Aug 30, 2016 at 04:10:47PM +0200, Jiří Stránský wrote:
> >>
> >> On 30.8.2016 10:17, Steven Hardy wrote:
> >>
> >> <snip>
> >>
> >> > Yeah, that gets us closer, but we do need to handle more than one
> value
> >> > (list entry) per key, e.g:
> >> >
> >> >          data:
> >> >            l:
> >> >              - "gnocchi_metricd_node_names": ["a0", "a1", "a2"]
> >> >                "tripleo_packages_node_names": ["a0", "a1", "a2"]
> >> >              - "nova_compute_node_names": ["b0"]
> >> >                "tripleo_packages_node_names": ["b0"]
> >> >
> >> > Output needs to be like:
> >> >
> >> >              "gnocchi_metricd_node_names": ["a0", "a1", "a2"]
> >> >              "tripleo_packages_node_names": ["a0", "a1", "a2", "b0"]
> >> >              "nova_compute_node_names": ["b0"]
> >> >
> >>
> >> Hoping this could do it:
> >>
> >> [stack at instack ~]$ cat yaq.yaml
> >> heat_template_version: 2016-10-14
> >>
> >> outputs:
> >>   debug:
> >>     value:
> >>       yaql:
> >>         expression: $.data.l.reduce($1.mergeWith($2))
> >>         data:
> >>           l:
> >>             - "gnocchi_metricd_node_names": ["a0", "a1", "a2"]
> >>               "tripleo_packages_node_names": ["a0", "a1", "a2"]
> >>             - "nova_compute_node_names": ["b0"]
> >>               "tripleo_packages_node_names": ["b0"]
> >
> > Thanks for this!
> >
> > Unfortunately I dont think it works with more than two list items:
> >
> >   debug_tripleo2:
> >     value:
> >       yaql:
> >         expression: $.data.l.reduce($1.mergeWith($2))
> >         data:
> >           l:
> >             - "gnocchi_metricd_node_names": ["overcloud-controller-0",
> >               "overcloud-controller-1", "overcloud-controller-2"]
> >               "tripleo_packages_node_names": ["overcloud-controller-0",
> "overcloud-controller-1", "overcloud-controller-2"]
> >             - "nova_compute_node_names": ["overcloud-compute-0"]
> >               "tripleo_packages_node_names": ["overcloud-compute-0"]
> >               "tripleo_packages_node_names2": ["overcloud-compute-0"]
> >             - "ceph_osd_node_names": ["overcloud-cephstorage-0"]
> >               "tripleo_packages_node_names": ["overcloud-cephstorage-0"]
> >               "tripleo_packages_node_names2":
> ["overcloud-cephstorage-0"]
> >
> > $ heat output-show foo5 debug_tripleo2
> > stack output show" instead
> > Output error: can only concatenate tuple (not "list") to tuple
> >
> > I've not dug too deeply yet, but assuming that's a yaql error vs a heat
> bug
> > it looks like it won't work.
>
> I'd say it's a yaql bug (commented on the bug opened in Heat), that
> can work around using the list_merger argument:
>
> $.data.l.reduce($1.mergeWith($2, listMerger => $1.toList() + $2.toList()))
>
> Still slightly more elegant than the one I came up with.
>
> --
> Thomas
>
> __________________________________________________________________________
> OpenStack Development Mailing List (not for usage questions)
> Unsubscribe: OpenStack-dev-request at lists.openstack.org?subject:unsubscribe
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstack.org/pipermail/openstack-dev/attachments/20160830/4b619585/attachment.html>