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

feat: add deal IPNI announce command #1877

Merged
merged 2 commits into from
Feb 2, 2024
Merged

feat: add deal IPNI announce command #1877

merged 2 commits into from
Feb 2, 2024

Conversation

LexLuthr
Copy link
Collaborator

@LexLuthr LexLuthr commented Feb 1, 2024

This command will help resolve situations where we have published a removal ad for a deal but it wasn't processed by IPNI correctly. Users can republish the add index Ad and then publish the removal again.

@LexLuthr LexLuthr requested a review from masih February 1, 2024 13:01
@LexLuthr
Copy link
Collaborator Author

LexLuthr commented Feb 2, 2024

Tested on devnet:

root@9cec9155e90e:/app# boostd index announce-deal af700e4b-1706-4fb5-8c01-8a44b41a6c67
Announced the deal with Ad cid baguqeera4icgz5jemxqsvxoe43ffq3gpcgwje3gpuogglvlqoed5apyjdlea

root@9cec9155e90e:/app# boostd index announce-deal af700e4b-1706-4fb5-8c01-8a44b41a6c67
Deal already announced


root@9cec9155e90e:/app# boostd index announce-remove-deal af700e4b-1706-4fb5-8c01-8a44b41a6c67
the proposal cid is bafyreiebxj5zc3veft44cbulowkmoeuqhqant7p4oa6v73o6x6kmwizqgyAnnounced the removal Ad with cid baguqeera4w6amnbzfpuhazfddjsty5ickvsipfqxbae2ql5tl2ebqre5cwma

root@9cec9155e90e:/app# boostd index announce-remove-deal af700e4b-1706-4fb5-8c01-8a44b41a6c67
the proposal cid is bafyreiebxj5zc3veft44cbulowkmoeuqhqant7p4oa6v73o6x6kmwizqgyError: failed to send removal ad: failed to announce deal removal to index provider: context ID not found

@LexLuthr LexLuthr merged commit 5348fa5 into main Feb 2, 2024
22 checks passed
@LexLuthr LexLuthr deleted the feat/ipni-announce branch February 2, 2024 09:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants