cloud-custodian / cloud-custodian/cel-python

CompiledRunner Functions Bug

Open
#145 4 comments 0 reactions 2 assignees Claimed by @Quantum0uasar View on GitHub
bug
Dominant language
Python
Stars
174
Forks
40
PR merge metrics
No merged PRs in 30d

Description

Hello,

First of all thanks for producing this library very useful for some applications I'm looking at right now.

I'm trying to use the CompiledRunner with some additional functions defined, short example below:

```
{
'lower': celpy.celtypes.FunctionType,
'upper': celpy.celtypes.FunctionType
}

def lower(lhs: celpy.celtypes.StringType) -> celpy.celtypes.StringType:
return celpy.celtypes.StringType(lhs.lower())

def upper(lhs: celpy.celtypes.StringType) -> celpy.celtypes.StringType:
return celpy.celtypes.StringType(lhs.upper())

env = celpy.Environment(annotations=declarations, runner_class=CompiledRunner)

rule = "size(items) == 1 && meta.device.lower() == 'tablet'"
ast = env.compile(rule)

prgm = env.program(ast, functions={
'lower': lower,
'upper': upper
})
prgm.evaluate({'items': [1,2,3], 'meta': {'device': 'tablet'}})
```

This works normally when using the InterpretedRunner, however with the compiled runner I get the following:

```
TypeError: CELEvalError(*('no such overload', , (' CELEvalError(*(\'undeclared reference to "name \\\'__main__\\\' is not defined" (in container ())\', , ("name \'__main__\' is not defined",))) and CELEvalError(*(\'undeclared reference to "name \\\'__main__\\\' is not defined" (in container ())\', , ("name \'__main__\' is not defined",)))',))) or CELEvalError(*('undeclared reference to "name \'__main__\' is not defined" (in container ())', , ("name '__main__' is not defined",)))
```

I dumped out the source from the compiled runner:

```
# conditionaland:
ex_2_l = lambda activation: celpy.evaluation.bool_eq(celpy.evaluation.function_size(activation.items), celpy.celtypes.IntType(1))
ex_2_r = lambda activation: celpy.evaluation.bool_eq(__main__.lower(activation.meta.get('device')), celpy.celtypes.StringType('tablet'))
ex_2 = lambda activation: celpy.celtypes.logical_and(celpy.evaluation.result(activation, ex_2_l), celpy.evaluation.result(activation, ex_2_r))
CEL = celpy.evaluation.result(base_activation, ex_2)

Internal error: CELEvalError(*('undeclared reference to "name \'__main__\' is not defined" (in container ())', , ("name '__main__' is not defined",)))
```

This also happens when you use a lambda rather than referencing a function.

Looks like this may not be referencing the function correctly? Happy to assist if you want to point me in the right direction for resolution, or if this is an error on my part please let me know.

Thanks

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.