New issue
Advanced search Search tips

Issue 631232 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jul 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

logdog stream is using file path as name

Project Member Reported by nicholaslin@google.com, Jul 25 2016

Issue description

Logdog stream always uses the path of the file to be streamed as the name even when the name field is specified. For example, using 

logdog_butler -project ... -output ... stream -source /tmp/test.txt -stream -name=test_steam 

will return: [prefix]/+/file:_tmp_test.txt instead of [prefix]/+/test_stream
 

Comment 1 by d...@chromium.org, Jul 25 2016

Owner: d...@chromium.org
Status: Started (was: Untriaged)
Project Member

Comment 3 by bugdroid1@chromium.org, Jul 25 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/infra/infra.git/+/97e099e2a216b2fda33112e5073bd3a2462491ed

commit 97e099e2a216b2fda33112e5073bd3a2462491ed
Author: dnj <dnj@chromium.org>
Date: Mon Jul 25 23:34:53 2016

Roll "luci-go".

infra/go/src/github.com/luci/luci-go:
9e16b79 Move timing information back into quest description.
4572641 auth: Merge server/auth/signing and server/auth/info packages
together.
f9f7f55 auth: Expose primary auth service public keys in auth.DB
interface.

BUG= chromium:631232 
TEST=None

Review-Url: https://codereview.chromium.org/2179203002

[modify] https://crrev.com/97e099e2a216b2fda33112e5073bd3a2462491ed/DEPS

Project Member

Comment 4 by bugdroid1@chromium.org, Jul 25 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/external/github.com/luci/luci-go.git/+/cbf0e97aac311593a4ccf1cc20c12497097e2fbf

commit cbf0e97aac311593a4ccf1cc20c12497097e2fbf
Author: dnj <dnj@chromium.org>
Date: Mon Jul 25 23:12:05 2016

LogDog/Butler: Prefer overridden stream name.

BUG= chromium:631232 
TEST=local
  - `./logdog/client/cmd/logdog_butler -project asdf -output log stream
     -source test.cfg -stream name=test_stream` had stream name
     "test_stream".

Review-Url: https://codereview.chromium.org/2179963003

[modify] https://crrev.com/cbf0e97aac311593a4ccf1cc20c12497097e2fbf/logdog/client/cmd/logdog_butler/subcommand_stream.go

Sign in to add a comment