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

[testing] update datastore to use contexts with sharness failure #8564

Closed
wants to merge 40 commits into from

Conversation

aschmahmann
Copy link
Contributor

#8563 has a lot going on already, and there's a sharness failure we're only really seeing in CI. Using this PR to debug

marten-seemann and others added 30 commits November 12, 2021 21:27
making deprecation notice match the go-ipfs-config key
Clarified docs based on
#8522 (comment)

License: MIT
Signed-off-by: Marcin Rataj <lidel@lidel.org>
this moves relay "user" flag before the "service" to align better with
new docs in effort to limit the number of people that mistake the two.

License: MIT
Signed-off-by: Marcin Rataj <lidel@lidel.org>
License: MIT
Signed-off-by: Marcin Rataj <lidel@lidel.org>
This switches to config that supports setting and reading
Internal.Libp2pForceReachability OptionalString flag

License: MIT
Signed-off-by: Marcin Rataj <lidel@lidel.org>
does not cause issue locally, let's see how behaves on CI
This is for testing the libp2p release, since the context changes are
not plumbed all the way through.
- applies fix from go-ipfs-config which removes null when default
- escapes JSON string when passed in sharness script
* test: add tests for automatic sharding and unsharding
* refactor: changed some naming in the sharding sharness tests to make more sense

Co-authored-by: Adin Schmahmann <adin.schmahmann@gmail.com>
This is for testing the libp2p release, since the context changes are
not plumbed all the way through.
@aschmahmann aschmahmann changed the title Exp/update ds ci [testing] update datastore to use contexts with sharness failure Nov 23, 2021
@aschmahmann aschmahmann changed the base branch from master to feat/update-datastore November 23, 2021 15:47
Base automatically changed from feat/update-datastore to master November 29, 2021 18:58
@aschmahmann
Copy link
Contributor Author

Not needed anymore. Closed #8563

@hacdias hacdias deleted the exp/update-ds-ci branch May 9, 2023 11:03
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.

5 participants