Manu Garg

Results 85 comments of Manu Garg

_From [[email protected]](https://code.google.com/u/116343118656038497241/) on November 22, 2013 01:27:46_ I would also like a way to pass the retrieved wpad file into pacparser without having to first download the file to disk....

Thanks for bringing this to my notice,Ogreman. I am not sure what can be done to fix this. I have submitted feedback to NANO-Antivirus (at http://www.nanoav.ru/index.php?option=com_content&view=article&id=15&Itemid=83&lang=en). Hopefully they will get...

Look at this code for how to generate Cue schema from .proto files: https://github.com/fourierr/cue-0.2.2-fix/blob/main/encoding/protobuf/protobuf_test.go Example input with a bunch of imports: https://github.com/fourierr/cue-0.2.2-fix/blob/main/encoding/protobuf/testdata/istio.io/api/mixer/v1/mixer.proto Output: https://github.com/fourierr/cue-0.2.2-fix/blob/main/encoding/protobuf/testdata/istio.io/api/mixer/v1/mixer_proto_gen.cue Imports are part of the same...

Experimentation so far: I tried generating Cue schema from protobufs. Steps: ``` # Set up code in a temp dir: PROJECT=github.com/cloudprober/cloudprober TMPDIR=$(mktemp -d); mkdir -p ${TMPDIR}/github.com/cloudprober; cp -a ../cloudprober $TMPDIR/github.com/cloudprober...

Based on the above findings, if we generate CUE schema from proto files, we can write config files in YAML today itself. I think this is a great news. I...

https://github.com/cloudprober/cloudprober/pull/181/files shows the generated CUE schema and changes to the `tools/gen_pb_go.sh` script to generate it. > In config package, if user provides us a YAML file, convert that file to...

> One problem I see right now is that `cue` tool takes a while to eval the yaml file. I hope it will be faster if we load CUE schema...

I've been experimenting a bit more and found another issue. By the above method, enums end up showing as strings in the generated textproto (see the type field [here](https://github.com/cloudprober/cloudprober/issues/178#issuecomment-1238983045)), i.e....

Some improvisation: #189. In this change we replace enum fields in the following manner: `ip_type: "PUBLIC"` will get replaced by `ip_type: PUBLIC` in the generated textproto. With this change, Cloudprober...

@dhananjaysathe Sorry for the delay in reviewing this PR. Unfortunately, adding more metrics affects everyone, so we'd want to enable them only through a configurable option. But, before you make...