Feature request: refactor CEL `kw.net.lookupHost()` to accept and return IPs from the CEL kubernetes library - Githubissues

kubewarden / cel-policy

A policy that can run CEL expressions

Apache License 2.0

3 stars 5 forks source link

Feature request: refactor CEL `kw.net.lookupHost()` to accept and return IPs from the CEL kubernetes library #19

Closed viccuad closed 4 months ago

viccuad commented 8 months ago

Right now, kw.net.lookupHost() is defined as kw.net.lookupHost(<string>) -> <list<string>>.

With Kubernetes 1.30, the CEL library will include a new library IP, which will have a new IPType and functions for it. See: https://github.com/kubernetes/kubernetes/blob/91ee30074bee617d52fc24dc85132fe948aa5153/staging/src/k8s.io/apiserver/pkg/cel/library/ip.go

Acceptance criteria

[ ] Refactor cel/library/net.go to accept and return the new IPType, and validating the IP upon return.
[ ] Refactor the cel/library/net_test.go.
[ ] Update documentation for kw.net.* functions.

fabriziosestito commented 4 months ago

won't do, after further consideration it is fine returning a string and let the user convert it by using the ip