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

gdrive stuck on runner #42

Closed
elleobrien opened this issue Apr 2, 2020 · 6 comments
Closed

gdrive stuck on runner #42

elleobrien opened this issue Apr 2, 2020 · 6 comments

Comments

@elleobrien
Copy link
Contributor

I am using GDrive for remote storage. This is great on my local machine, but when I go to the runner, it seems to be stuck forever "Pulling from DVC remote".

I don't have proof but these seems like an authentication issue? My bet is that on my local machine, the first time I try to access GDrive as a remote I am given a link to visit in my browser, and then I get a validation code that I copy and paste back into the CLI. I'm guessing we are simply not getting past this authentication stage.

I have followed the instructions on the README for cml for GDrive (to my understanding); I copied and pasted the contents of .dvc/tmp/gdrive-user-credentials.json into the value field for the Secret "GDRIVE_USER_CREDENTIALS_DATA".

The repo is here if needed: https://github.com/iterative/mnist_classifier

@DavidGOrtega what do you think?

@shcheklein
Copy link
Member

@andronovhopf it should be GDRIVE_CREDENTIALS_DATA I believe - could you give it a try?

@elleobrien
Copy link
Contributor Author

@shcheklein hmmm i made another commit with GDRIVE_CREDENTIALS_DATA in my dvc.yaml file and added it as a secret (again, copying and pasting the contents of the recommended .json file into the secret value field). Still looks stuck on the same spot though 🤔

@shcheklein
Copy link
Member

It's probably a bug here:

https://github.com/iterative/dvc-cml/blob/master/src/dvc.js#L85-L88 cc @DavidGOrtega

and in dvc.yaml env var name should be GDRIVE_CREDENTIALS_DATA.

@shcheklein
Copy link
Member

@andronovhopf I've fixed it - please wait for the new version to deploy - https://github.com/iterative/dvc-cml/actions

also updated README

@elleobrien
Copy link
Contributor Author

🙏

@pranavchat14
Copy link

@elleobrien I struggled quite a bit before I actually succeeded with this. I really wish you would have covered this part on youtube.

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

No branches or pull requests

3 participants