Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

heapprofiler: an error about parsing a file name #54723

Closed
yuzefovich opened this issue Sep 24, 2020 · 2 comments · Fixed by #55364
Closed

heapprofiler: an error about parsing a file name #54723

yuzefovich opened this issue Sep 24, 2020 · 2 comments · Fixed by #55364
Assignees
Labels
C-bug Code not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior.

Comments

@yuzefovich
Copy link
Member

I noticed that I have this

W200924 03:22:19.585603 403 server/heapprofiler/profilestore.go:168 ⋮ [n1] ‹memstats.2020-09-23T22_59_51.191246336.txt›: ‹strconv.ParseUint: parsing "txt": invalid syntax›

in my logs.

Here is the relevant contents of the offending directory:

-rw-r-----   1 yuzefovich  staff    9718 Sep 23 15:47 memstats.2020-09-23T22_47_04.431.4399403008.txt
-rw-r-----   1 yuzefovich  staff    9743 Sep 23 15:47 memstats.2020-09-23T22_47_14.428.4400369664.txt
-rw-r-----   1 yuzefovich  staff    9767 Sep 23 15:47 memstats.2020-09-23T22_47_24.454.4401340416.txt
-rw-r-----   1 yuzefovich  staff    9768 Sep 23 15:47 memstats.2020-09-23T22_47_35.245.4402278400.txt
-rw-r-----   1 yuzefovich  staff    9789 Sep 23 15:47 memstats.2020-09-23T22_47_44.442.4403232768.txt
-rw-r-----   1 yuzefovich  staff   10402 Sep 23 15:59 memstats.2020-09-23T22_59_51.191246336.txt
-rw-r-----   1 yuzefovich  staff   10581 Sep 23 20:23 memstats.2020-09-24T03_23_49.596.4298784768.txt
-rw-r-----   1 yuzefovich  staff   10583 Sep 23 20:23 memstats.2020-09-24T03_23_59.6.4299747328.txt
-rw-r-----   1 yuzefovich  staff   10607 Sep 23 20:24 memstats.2020-09-24T03_24_09.607.4300742656.txt
-rw-r-----   1 yuzefovich  staff   10630 Sep 23 20:24 memstats.2020-09-24T03_24_19.599.4301668352.txt
-rw-r-----   1 yuzefovich  staff   10630 Sep 23 20:24 memstats.2020-09-24T03_24_29.608.4302569472.txt

I didn't modify any of the filenames myself, and I also remember seeing the same warning a week or so ago, so we might need to make the parser more flexible. I don't know how I got a file with such name though.

@yuzefovich yuzefovich added the C-bug Code not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior. label Sep 24, 2020
@irfansharif
Copy link
Contributor

irfansharif commented Sep 30, 2020

Seeing this as well. Lazy repro (from something unrelated I was working on adding extra stores; I'm sure we don't need the last two steps here):

roachprod destroy local; roachprod wipe local;
roachprod create local --nodes 3;
roachprod put local cockroach;
roachprod start local;
roachprod stop local:1;
roachprod start local:1 --args="--store=path=${HOME}/local/1/data --store=path=${HOME}/local/1/data2"

I was running a688bb7e4e0c1d996131bd9b586eab79cb72973f.

@knz
Copy link
Contributor

knz commented Oct 8, 2020

thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-bug Code not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants