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

Add etcd test #74

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from
Open

Add etcd test #74

wants to merge 1 commit into from

Conversation

ssyssy
Copy link
Contributor

@ssyssy ssyssy commented Mar 8, 2024

No description provided.

@ssyssy ssyssy changed the base branch from main to dev March 8, 2024 00:02
@ssyssy ssyssy requested review from LuQQiu and jja725 March 8, 2024 00:03
@jja725
Copy link
Contributor

jja725 commented Mar 8, 2024 •
edited
Loading

hmmm @ssyssy @LuQQiu should we wait for two repo to be merged and then port the change to the new code?

@LuQQiu
Copy link
Contributor

LuQQiu commented Mar 8, 2024

@jja725 I tried not to block the alluxio-py changes
@jja725 are you comfortable with merging the fsspec/alluxiofs#16 before the github action permission granted?

@ssyssy
Copy link
Contributor Author

ssyssy commented Mar 8, 2024

@LuQQiu Addressed.

@jja725
Copy link
Contributor

jja725 commented Mar 8, 2024

@LuQQiu martin replied, we can merge that PR first and see how we can enable test

@LuQQiu
Copy link
Contributor

LuQQiu commented Mar 8, 2024

Please port to https://github.com/fsspec/alluxiofs dev channel

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@LuQQiu LuQQiu LuQQiu left review comments

@jja725 jja725 Awaiting requested review from jja725

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ssyssy @jja725 @LuQQiu