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.

Sign up for GitHub

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

Jump to bottom

dagstore: failed to parse mount URL #2031

Closed
distractedm1nd opened this issue Apr 5, 2023 · 1 comment · Fixed by #2071
Closed

dagstore: failed to parse mount URL #2031

distractedm1nd opened this issue Apr 5, 2023 · 1 comment · Fixed by #2071
Assignees
@distractedm1nd
Labels
area:shares Shares and samples bug Something isn't working

Comments

@distractedm1nd
Copy link
Member

Noticed on a bridge on robusta-4 after startup:

2023-04-05T14:31:12.574Z	WARN	dagstore	dagstore@v0.0.0-20230404123415-177451f83136/dagstore.go:572	failed to recover state of shard /002DBB63C08B79A845A67C66BB693B6B2A573C28F9CA0ED019C7982272A6E638: failed to parse mount URL: parse "fs://%2Fhome%2Fcelestia%2Fblocks%2F002DBB63C08B79A845A67C66BB693B6B2A573C28F9CA0ED019C7982272A6E638": invalid URL escape "%2F"; skipping
2023-04-05T14:31:12.574Z	WARN	dagstore	dagstore@v0.0.0-20230404123415-177451f83136/dagstore.go:572	failed to recover state of shard /003D800650269039B956B7AA073F1447587FD0EABAD126D33BF61627E83C217B: failed to parse mount URL: parse "fs://%2Fhome%2Fcelestia%2Fblocks%2F003D800650269039B956B7AA073F1447587FD0EABAD126D33BF61627E83C217B": invalid URL escape "%2F"; skipping
2023-04-05T14:31:12.574Z	WARN	dagstore	dagstore@v0.0.0-20230404123415-177451f83136/dagstore.go:572	failed to recover state of shard /00843E5F89C6690277CEA62A0B4B4419171B3EA1BC252142140036EF72837B7F: failed to parse mount URL: parse "fs://%2Fhome%2Fcelestia%2Fblocks%2F00843E5F89C6690277CEA62A0B4B4419171B3EA1BC252142140036EF72837B7F": invalid URL escape "%2F"; skipping
2023-04-05T14:31:12.575Z	WARN	dagstore	dagstore@v0.0.0-20230404123415-177451f83136/dagstore.go:572	failed to recover state of shard /00C4BEB752A2CD377D19A3BED2B898CECFF148B67E0EA6DEBFB5950743C3FFEC: failed to parse mount URL: parse "fs://%2Fhome%2Fcelestia%2Fblocks%2F00C4BEB752A2CD377D19A3BED2B898CECFF148B67E0EA6DEBFB5950743C3FFEC": invalid URL escape "%2F"; skipping
2023-04-05T14:31:12.575Z	WARN	dagstore	dagstore@v0.0.0-20230404123415-177451f83136/dagstore.go:572	failed to recover state of shard /0136F28AE4E3CA9F61E8303FDAE0B716D4715DDB25CBAF3035F0168CC544E57C: failed to parse mount URL: parse "fs://%2Fhome%2Fcelestia%2Fblocks%2F0136F28AE4E3CA9F61E8303FDAE0B716D4715DDB25CBAF3035F0168CC544E57C": invalid URL escape "%2F"; skipping
2023-04-05T14:31:12.575Z	WARN	dagstore	dagstore@v0.0.0-20230404123415-177451f83136/dagstore.go:572	failed to recover state of shard /016D0C17464E5042121B6F77EED5A00025A3F04BD5D05705B558731A9FA54228: failed to parse mount URL: parse "fs://%2Fhome%2Fcelestia%2Fblocks%2F016D0C17464E5042121B6F77EED5A00025A3F04BD5D05705B558731A9FA54228": invalid URL escape "%2F"; skipping
@distractedm1nd distractedm1nd added area:shares Shares and samples kind:fix Attached to bug-fixing PRs bug Something isn't working and removed kind:fix Attached to bug-fixing PRs labels Apr 5, 2023
@distractedm1nd distractedm1nd self-assigned this Apr 5, 2023
@renaynay
Copy link
Member

renaynay commented Apr 6, 2023

+1 i've seen this too

This was referenced Apr 13, 2023
Open
Closed
Merged
distractedm1nd added a commit that referenced this issue Apr 13, 2023
@distractedm1nd
fix(eds/store): use Path instead of Host in FileMount serializa…
9087e0e 
…tion/deserialization (#2071)

Temporary fix until our fixes are merged upstream (and we can remove
this replace). REQUIRES a full store reset.
Didnt know whether to make the PR chore(deps) or fix(eds/store)
Closes #2031
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@distractedm1nd distractedm1nd

Labels
area:shares Shares and samples bug Something isn't working
Projects
None yet
Milestone
No milestone
2 participants
@distractedm1nd @renaynay