{"push_id":2204259911,"size":5,"distinct_size":5,"ref":"refs/heads/logging","head":"4aa330340a7ccc3af342b4b9db18a0d48144c058","before":"8447b00d64f7a8ac2d5c99039ada0a431717bc67","commits":[{"sha":"b1ab517f2977be1cf3bb77962e0758d2db495052","author":{"name":"Martin Taillefer","email":"20670bc50677903193e3d5648174e6fca0294afe@users.noreply.github.com"},"message":"Finish removing glog from Mixer. (#2208)\n\nAutomatic merge from submit-queue.\n\nFinish removing glog from Mixer.\n\nNote the voodoo in pkg/runtime/logger.go\r\n\r\nOnce the dust settles, I think we'll want to review the adapter.Logger type to look like or perhaps even just be a zap logger. But that can come later.","distinct":true,"url":"https://api.github.com/repos/nmittler/istio/commits/b1ab517f2977be1cf3bb77962e0758d2db495052"},{"sha":"1f8adb3ca038f477c819f6942624a485e7353820","author":{"name":"Nathan Mittler","email":"aaeccd3650540d3bc7d551ab86ef91ff64a0f0b1@google.com"},"message":"[pilot] Workaround for glog Parsing Error (#2223)\n\nAutomatic merge from submit-queue.\n\n[pilot] Workaround for glog Parsing Error\n\nThis is copied from mixer code (introduced by https://github.com/istio/istio/commit/5ae17d07ead24e31fba01c91673122bba2b6e804)\r\n\r\nFixes #2127\r\n\r\n**Release note**:\r\n```release-note\r\nNone\r\n```","distinct":true,"url":"https://api.github.com/repos/nmittler/istio/commits/1f8adb3ca038f477c819f6942624a485e7353820"},{"sha":"dc43d71fce77a787748f8aaa8194b1dfd4271d6d","author":{"name":"Martin Taillefer","email":"20670bc50677903193e3d5648174e6fca0294afe@users.noreply.github.com"},"message":"Update docs to mention new logging package and the standard Go testing framework. (#2211)","distinct":true,"url":"https://api.github.com/repos/nmittler/istio/commits/dc43d71fce77a787748f8aaa8194b1dfd4271d6d"},{"sha":"efc7baefebc425cf5fc4d482087d70d30836b21a","author":{"name":"Zack Butcher","email":"905ac2161d6e215a8ea142f657ddf7fc6c1538a6@google.com"},"message":"Log a warning when we ingest an Ingress resource containing a rule without a body. (#2234)\n\nAutomatic merge from submit-queue.\n\nLog a warning when we ingest an Ingress resource containing a rule without a body\n\n**What this PR does / why we need it**: Fix our handling of Ingress resources, which may specify rules without HTTP bodies.\r\n\r\n**Which issue this PR fixes**: fixes #2226\r\n\r\n**Special notes for your reviewer**:\r\n\r\n**Release note**:\r\n\r\n```release-note\r\nNONE\r\n```","distinct":true,"url":"https://api.github.com/repos/nmittler/istio/commits/efc7baefebc425cf5fc4d482087d70d30836b21a"},{"sha":"4aa330340a7ccc3af342b4b9db18a0d48144c058","author":{"name":"Nathan Mittler","email":"30109e6ce5915d8b0df22a7ce0a534c3eb9d7474@gmail.com"},"message":"Switch to Istio logging\n\nThis completes the transition from the entire Istio code base over\nto the new Istio logging library (replacing glog).\n\nThe mapping is roughly as follows:\n\nglog.Debug -> log.Debug\nglog.Info -> log.Info\nglog.Warning -> log.Warn\nglog.Error -> log.Error\nglog.Fatal -> log.Error + os.Exit(-1)\nglog.V(1) -> log.Warn\nglog.V(2) -> log.Info\nglog.V(3+) -> log.Debug\nglog.Flush -> log.Sync\n\nPartially addresses #2174","distinct":true,"url":"https://api.github.com/repos/nmittler/istio/commits/4aa330340a7ccc3af342b4b9db18a0d48144c058"}]}
{"actor":{"display_login":"nmittler"}}