Jira (PDB-4511) PQL fact path projection whitespace sensitive

16 views
Skip to first unread message

Austin Blatt (JIRA)

unread,
Sep 23, 2019, 4:22:03 PM9/23/19
to puppe...@googlegroups.com
Austin Blatt created an issue
 
PuppetDB / Bug PDB-4511
PQL fact path projection whitespace sensitive
Issue Type: Bug Bug
Assignee: Unassigned
Created: 2019/09/23 1:21 PM
Priority: Normal Normal
Reporter: Austin Blatt
Add Comment Add Comment
 
This message was sent by Atlassian JIRA (v7.7.1#77002-sha1:e75ca93)
Atlassian logo

Austin Blatt (JIRA)

unread,
Sep 23, 2019, 4:24:03 PM9/23/19
to puppe...@googlegroups.com
Austin Blatt updated an issue
Change By: Austin Blatt
The currently working PQL query for fact path projection looks like. The extra spaces are problematic and error-prone


{code}
curl -sSXGET 'http://localhost:8080/pdb/query/v4' \
    -d 'query=inventory[fact.os.family , facts.os ]{}'
> []
{code}

Austin Blatt (JIRA)

unread,
Sep 23, 2019, 4:24:04 PM9/23/19
to puppe...@googlegroups.com
Austin Blatt updated an issue
Change By: Austin Blatt
Acceptance Criteria:
{code}
curl -sSXGET 'http://localhost:8080/pdb/query/v4' \
    -d 'query=inventory[fact.os.family, facts.os]{}'
> []
{code}

Austin Blatt (JIRA)

unread,
Sep 23, 2019, 4:25:04 PM9/23/19
to puppe...@googlegroups.com
Austin Blatt updated an issue
Change By: Austin Blatt
Fix Version/s: PDB 6.7.0

Austin Blatt (JIRA)

unread,
Sep 23, 2019, 4:26:03 PM9/23/19
to puppe...@googlegroups.com
Austin Blatt updated an issue
The currently working PQL query for fact path projection looks like. The extra spaces are problematic and error-prone


{code}
curl -sSXGET 'http://localhost:8080/pdb/query/v4' \
    -d 'query=inventory[fact.os.family , facts.os ]{}'
> []
{code}

For the sake of backwards compatibility we will not fix the query fields until PDB 7.0.0

Austin Blatt (JIRA)

unread,
Sep 23, 2019, 4:26:04 PM9/23/19
to puppe...@googlegroups.com
Austin Blatt updated an issue
Change By: Austin Blatt
Acceptance Criteria:
{code}
curl -sSXGET 'http://localhost:8080/pdb/query/v4' \
    -d 'query=inventory[fact.os.family, facts.os]{}'
> []
{code}

{code}
❯ curl -sSXGET 'http://localhost:8080/pdb/query/v4' \
    -d 'query=resources { parameters.content= "a"}'
PQL parse error at line 1, column 33:

resources { parameters.content= "a"}
                     ^

Expected one of:

is not null
is null
in
=
>
<
<=
>=
~>
~
{code}

Austin Blatt (JIRA)

unread,
Sep 23, 2019, 4:27:02 PM9/23/19
to puppe...@googlegroups.com
Austin Blatt updated an issue
Change By: Austin Blatt
Acceptance Criteria: This query should now work, where it previously failed.
{code}
❯ curl -sSXGET 'http://localhost:8080/pdb/query/v4' \
    -d 'query=inventory[fact.os.family, facts.os]{}'
[]
{code}

This query should continue to fail.
{code}
❯ curl -sSXGET 'http://localhost:8080/pdb/query/v4' \
    -d 'query=resources { parameters.content= "a"}'
PQL parse error at line 1, column 33:

resources { parameters.content= "a"}
                     ^

Expected one of:

is not null
is null
in
=
>
<
<=
>=
~>
~
{code}

Austin Blatt (JIRA)

unread,
Sep 25, 2019, 7:48:03 PM9/25/19
to puppe...@googlegroups.com
Austin Blatt assigned an issue to Austin Blatt
Change By: Austin Blatt
Assignee: Austin Blatt

Austin Blatt (JIRA)

unread,
Sep 25, 2019, 7:49:03 PM9/25/19
to puppe...@googlegroups.com
Austin Blatt updated an issue

We decided to do the whole fix, so this is a duplicate.

Austin Blatt (JIRA)

unread,
Sep 25, 2019, 7:50:05 PM9/25/19
to puppe...@googlegroups.com
Austin Blatt updated an issue
Change By: Austin Blatt
Fix Version/s: PDB 6.7.0
Reply all
Reply to author
Forward
0 new messages