Remove gaelogger from luci-go

Remove gaelogger from luci-go

Counterpart to https://codereview.chromium.org/1498963003. prod.Use now
implements luci-go/common/logging.

R=vadimsh@chromium.org, dnj@chromium.org, estaab@chromium.org
BUG=https://github.com/luci/gae/issues/22

Committed: https://github.com/luci/luci-go/commit/ac20e5654cce81e37ca7dd88896a3aa9a88fda48

[Vadim Sh., 2015-12-04 05:02:40]

lgtm with one addition (or rather deletion)

https://codereview.chromium.org/1492273003/diff/1/appengine/gaemiddleware/con...
File appengine/gaemiddleware/context.go (left):

https://codereview.chromium.org/1492273003/diff/1/appengine/gaemiddleware/con...
appengine/gaemiddleware/context.go:48: c = gaelogger.Use(c)
remove memlogger from gaetesting/middleware.go, TestingContext too, since
memory.Use() installs it now.

[iannucci, 2015-12-04 20:09:00]

https://chromiumcodereview.appspot.com/1492273003/diff/1/appengine/gaemiddlew...
File appengine/gaemiddleware/context.go (left):

https://chromiumcodereview.appspot.com/1492273003/diff/1/appengine/gaemiddlew...
appengine/gaemiddleware/context.go:48: c = gaelogger.Use(c)
On 2015/12/04 at 05:02:40, Vadim Sh. wrote:
> remove memlogger from gaetesting/middleware.go, TestingContext too, since
memory.Use() installs it now.

good call, done, ptal

[Vadim Sh., 2015-12-04 20:09:33]

lgtm

[iannucci, 2015-12-04 20:12:46]

The CQ bit was checked by iannucci@chromium.org

[commit-bot: I haz the power, 2015-12-04 20:12:52]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1492273003/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1492273003/20001

[iannucci, 2015-12-04 20:13:26]

On 2015/12/04 at 20:12:52, commit-bot wrote:
> CQ is trying da patch. Follow status at
>  https://chromium-cq-status.appspot.com/patch-status/1492273003/20001
> View timeline at
>  https://chromium-cq-status.appspot.com/patch-timeline/1492273003/20001

(this may require a DEPS roll in infra first, but if it commits then yay)

[commit-bot: I haz the power, 2015-12-04 20:14:21]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-12-04 20:14:22]

Try jobs failed on following builders:
  Luci-go Mac Tester on master.tryserver.infra (JOB_FAILED,
https://build.chromium.org/p/tryserver.infra/builders/Luci-go%20Mac%20Tester/...)

[iannucci, 2015-12-04 20:22:40]

Description was changed from

==========
Remove gaelogger from luci-go

Counterpart to https://codereview.chromium.org/1498963003. prod.Use now
implements luci-go/common/logging.

R=dnj@chromium.org, estaab@chromium.org, vadimsh@chromium.org
BUG=https://github.com/luci/gae/issues/22
==========

to

==========
Remove gaelogger from luci-go

Counterpart to https://codereview.chromium.org/1498963003. prod.Use now
implements luci-go/common/logging.

R=vadimsh@chromium.org, dnj@chromium.org, estaab@chromium.org
BUG=https://github.com/luci/gae/issues/22

Committed:
https://github.com/luci/luci-go/commit/ac20e5654cce81e37ca7dd88896a3aa9a88fda48
==========

[iannucci, 2015-12-04 20:22:41]

Committed patchset #2 (id:20001) manually as
ac20e5654cce81e37ca7dd88896a3aa9a88fda48 (presubmit successful).