Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Try to support nested all queries #4360

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Try to support nested all queries #4360

wants to merge 1 commit into from

Conversation

jaym
Copy link
Contributor

@jaym jaym commented Jul 17, 2024

As an example query, this should work now:

k8s.deployments.all(containers.all(name == 'kube-rbac-proxy'))

The containers should only show containers not matching kube-rbac-proxy in the results

However, list resources do not work. There is some other bug that makes it so those don't display anything. There are errors in the result because somewhere a type does not get set. I have not been able to track down this issue so for now those get filtered out

As an example query, this should work now:
```
k8s.deployments.all(containers.all(name == 'kube-rbac-proxy'))
```

The containers should only show containers not matching
`kube-rbac-proxy` in the results

However, list resources do not work. There is some other bug that makes
it so those don't display anything. There are errors in the result
because somewhere a type does not get set. I have not been able to track
down this issue so for now those get filtered out
Copy link
Contributor

Test Results

837 tests   - 2 236   835 ✅  - 2 237   26s ⏱️ -57s
 97 suites  -   266     0 💤  -     1 
  1 files    -    26     2 ❌ +    2 

For more details on these failures, see this check.

Results for commit 6e2d7b1. ± Comparison against base commit bb73b8d.

This pull request removes 2237 and adds 1 tests. Note that renamed tests count towards both.
go.mondoo.com/cnquery/v11/providers/ansible/play ‑ TestPlaybookDecoding
go.mondoo.com/cnquery/v11/providers/ansible/play ‑ TestPlaybookDecoding/load_default_playbook
go.mondoo.com/cnquery/v11/providers/ansible/play ‑ TestPlaybookDecoding/load_playbook_with_blocks_and_errors
go.mondoo.com/cnquery/v11/providers/ansible/play ‑ TestPlaybookDecoding/load_playbook_with_mulitple_plays
go.mondoo.com/cnquery/v11/providers/ansible/play ‑ TestPlaybookDecoding/load_playbook_with_roles
go.mondoo.com/cnquery/v11/providers/ansible/play ‑ TestPlaybookDecoding/load_playbook_with_serial
go.mondoo.com/cnquery/v11/providers/ansible/play ‑ TestPlaybookDecoding/load_playbook_with_vars
go.mondoo.com/cnquery/v11/providers/ansible/play ‑ TestTaskDecoding
go.mondoo.com/cnquery/v11/providers/ansible/play ‑ TestTaskDecoding/load_task_with_blocks
go.mondoo.com/cnquery/v11/providers/ansible/play ‑ TestTaskDecoding/load_task_with_vars
…
go.mondoo.com/cnquery/v11/mqlc ‑ TestCompiler_All_Issue919

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant