istio / old_issues_repo

Deprecated issue-tracking repo, please post new issues or feature requests to istio/istio instead.
36 stars 9 forks source link

Unable to get the Mixer Adapter walkthrough to work as described in the steps. #371

Closed vinayvenkat closed 6 years ago

vinayvenkat commented 6 years ago

This is possibly a bug.

This is to do with the documentation to write a new mixer adapter specified at:

https://github.com/istio/istio/wiki/Mixer-Adapter-Walkthrough

What happened:

I followed the instructions to implement a new mixer adapter but got the following errors when I ran the mixer client command against the mixer server.

I got the following error:

2018-06-05T00:27:51.140184Z warn input set condition evaluation error: id='15', error='lookup failed: 'context.protocol'' 2018-06-05T00:27:51.140254Z error panic during handler dispatch: inconsistent label cardinality {"level":"info","time":"2017-02-01T10:12:14.000000Z","instance":"accesslog.logentry.istio-system","destinationIp":"0.0.0.0","latency":"0s","method":"","protocol":"http","requestSize":0,"responseCode":0,"responseSize":0,"sourceIp":"0.0.0.0","sourceUser":"","url":""} 2018-06-05T00:27:51.140508Z error api Report failed:1 error occurred:

What you expected to happen:

I expected the "out.txt" file to have the data that is called out in the mixer adapter example. However I don't see any output file being created. I'm not sure if I am missing anything. I followed all the steps in the example but don't see it working for me.

How to reproduce it:

Well followed the example and steps in: https://github.com/istio/istio/wiki/Mixer-Adapter-Walkthrough

Feature Request: N

vinayvenkat commented 6 years ago

@gargnupur Firstly thanks for this documentation. I followed it to the 'T' but don't seem to be getting the expected outputs. Would appreciate any help with this. Thanks!

vinayvenkat commented 6 years ago

My apologies but it looks i was looking for the file in the wrong location.

I was expecting the out.txt file to be in the mysampleadapter directory. But looks like it gets put into $ISTIO/istio/out.txt.

It might be a good idea to specify the full directory path or call this out in the document. Thanks once again for this great document. Maybe i'll send a pull request for these changes.