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

CML <-> DVC push #656

Closed
Tracked by #196
DavidGOrtega opened this issue Jul 12, 2021 · 6 comments
Closed
Tracked by #196

CML <-> DVC push #656

DavidGOrtega opened this issue Jul 12, 2021 · 6 comments
Labels
documentation Markdown files dvc related to DVC icebox research Waiting for team investigation

Comments

@DavidGOrtega
Copy link
Contributor

Try to push dvc ephemeral commits

@DavidGOrtega DavidGOrtega added the research Waiting for team investigation label Jul 12, 2021
@DavidGOrtega DavidGOrtega self-assigned this Jul 12, 2021
@DavidGOrtega DavidGOrtega added the dvc related to DVC label Jul 12, 2021
@0x2b3bfa0
Copy link
Member

Won't work out of the box for GitHub, see #560 (comment) and jelmer/dulwich#882

@DavidGOrtega
Copy link
Contributor Author

Does not have to be in DVC, its just only find out a mechanism in CML

@0x2b3bfa0
Copy link
Member

0x2b3bfa0 commented Jul 12, 2021

Here you're a quick patch from my test repository, so you can use dvc to push experiments:

# Dulwich headless authentication patch for GitHub Actions
# See https://github.com/iterative/cml/issues/560#issuecomment-852250748

credentials="$(
  git config --local http.https://github.com/.extraheader |
  awk '{print $3}' |
  base64 --decode
)"

address="$(
  git config --local remote.origin.url |
  sed 's/^https:\/\///'
)"

git config --local remote.origin.url "https://$credentials@$address"
git config --local user.email "[email protected]"
git config --local user.name "GitHub Action"

@0x2b3bfa0
Copy link
Member

0x2b3bfa0 commented Jul 12, 2021

It would be easier to have it fixed in DVC/dulwich, though.

@casperdcl casperdcl added the documentation Markdown files label Sep 5, 2022
@casperdcl
Copy link
Contributor

also related to iterative/dvc#6823 (comment)?

@dacbd
Copy link
Contributor

dacbd commented Feb 17, 2023

out of our control for the time being, jelmer/dulwich#882

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Markdown files dvc related to DVC icebox research Waiting for team investigation
Projects
None yet
Development

No branches or pull requests

4 participants