Rohit Agrawal
Rohit Agrawal
> I finally have some spare cycles to work on this again, I will pick this up where it was left off and incorporate the suggested changes next week! Thank...
> I'm not sure how to increase the code coverage for some of the sad path, they seem unnecessary. Need some assistance if this needs to be covered! That's okay....
> I'm not sure how to increase the code coverage for some of the sad path, they seem unnecessary. Need some assistance if this needs to be covered! Sorry, I...
@Sudhamsh Could you please sign your commits so that DCO is happy?
> @agrawroh - I signed my commits. Please approve. The two checks are still failing:  Try changing the title of the PR...
Since I already touched some of the code for Argo integration with Istio VS in one of my other PRs, I could work on this one. We have a similar...
Created a PR for this: https://github.com/argoproj/argo-rollouts/pull/1420 Would appreciate any feedback/comments.
@simha5009 Looks like in this case response status code = **0** which means that the downstream closed the connection. You can get more details from `%RESPONSE_CODE_DETAILS%` [[Ref]](https://www.envoyproxy.io/docs/envoy/latest/configuration/http/http_conn_man/response_code_details). From what I...
Sorry for the delayed response. This definitely looks like a bug. I'll try to repro on my end and would open a fix PR. @chrisplim Could you please share the...