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

wantlist command #719

Merged
merged 4 commits into from
Feb 27, 2015
Merged

wantlist command #719

merged 4 commits into from
Feb 27, 2015

Conversation

whyrusleeping
Copy link
Member

I wanted to be able to check my wantlist when having trouble resolving things to get a better idea of what was going on. So I made this. not sure what tests to write for it, so if you have any ideas, let me know.

@whyrusleeping whyrusleeping added the status/in-progress In progress label Feb 2, 2015
@jbenet
Copy link
Member

jbenet commented Feb 2, 2015

This is a bitswap specific thing. lets put it under

ipfs bitswap wantlist

return
}

res.SetOutput(&KeyList{nd.Exchange.GetWantlist()})
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this should still cast the exchange to bitswap, as this is a command called bitswap :) -- it probably shouldnt work with non-bitswap exchanges)

@whyrusleeping
Copy link
Member Author

@jbenet is this RFM?

@jbenet
Copy link
Member

jbenet commented Feb 25, 2015

@whyrusleeping can you rebase and push again? looks like travis is still complaining about init docs hash (that got fixed, right?)

and, looks like there's a race?

This LGTM, except tests dont seem to be passing

@whyrusleeping
Copy link
Member Author

The race doesnt appear to be related to the changes in this PR, I filed a bug for it.

@jbenet
Copy link
Member

jbenet commented Feb 25, 2015

@whyrusleeping what about https://build.protocol-dev.com/job/epic/1257/console ? it ran on cassie-- is that a timeout or oom?

@whyrusleeping
Copy link
Member Author

That appears to be a timeout, looks like the "bitswap no routing" tests can hang...

@jbenet
Copy link
Member

jbenet commented Feb 25, 2015

This is new though. They weren't hanging before. Could it be the bitswap changes last week?

Understand we want to hurry on this PR and ipns but if we let bugs in it will be really hard to squash them later.


Sent from Mailbox

On Tue, Feb 24, 2015 at 7:50 PM, Jeromy Johnson [email protected]
wrote:

That appears to be a timeout, looks like the "bitswap no routing" tests can hang...

Reply to this email directly or view it on GitHub:
#719 (comment)

@whyrusleeping
Copy link
Member Author

Yeah, ive seen it twice before. And I think it has happened since my bitswap changes. Ill look into it.

@whyrusleeping
Copy link
Member Author

This is probably caused by my latest bitswap changes, and I think I maybe understand the issue. Ill delve into it more when i get a little more time.

jbenet added a commit that referenced this pull request Feb 27, 2015
@jbenet jbenet merged commit 2a9cc10 into master Feb 27, 2015
@jbenet jbenet removed the status/in-progress In progress label Feb 27, 2015
@jbenet jbenet deleted the feat/wantlistcmd branch February 27, 2015 10:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants