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

Reporting server v2 CEL environment does not contain v2 event group descriptor and list event groups with filter option fail (v0.4.2) #1406

Closed
Tracked by #1299
BaibaSku opened this issue Jan 10, 2024 · 2 comments · Fixed by #1417
Assignees
Labels
bug Something isn't working

Comments

@BaibaSku
Copy link

Describe the bug
(v0.4.2) When calling list event groups with filter on reporting server v2 and using fields, which are relevant to event group (v2) only (cmms_data_provider field as an example), the request fails, with following error message:

INVALID_ARGUMENT: filter is not a valid CEL expression: ERROR: :1:1: undeclared reference to 'cmms_data_provider' (in container 'wfa.measurement.reporting.v1alpha.EventGroup')\n | cmms_data_provider == 'dataProviders/a0tmyormbgQ'

Steps to reproduce

  1. Call list event groups on reporting server v2 by using cel expression, which contains v2 Event group field. Example:cmms_data_provider == 'dataProviders/a0tmyormbgQ'

Component(s) affected
Reporting Server

Version
0.4.2

@BaibaSku BaibaSku added the bug Something isn't working label Jan 10, 2024
@mariolamassaavedra
Copy link

This error seems to be cause due to the CEL Env Provider not importing the Reporting Server v2 definition of the event groups object and only referencing v1
image

@SanjayVas
Copy link
Member

@tristanvuong2021 can you take a look at this? We probably need separate CelEnvProvider impls for v1 and v2, given that it depends on an API message.

@SanjayVas SanjayVas mentioned this issue Jan 11, 2024
15 tasks
SanjayVas added a commit that referenced this issue Jan 12, 2024
…#1417)

This allows CelEnvProvider to be used for multiple Reporting API versions.

Fixes #1406
SanjayVas added a commit that referenced this issue Jan 12, 2024
…#1417)

This allows CelEnvProvider to be used for multiple Reporting API versions.

Fixes #1406

(cherry picked from commit 2306a16)
ple13 pushed a commit that referenced this issue Aug 16, 2024
…#1417)

This allows CelEnvProvider to be used for multiple Reporting API versions.

Fixes #1406
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
4 participants