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

Dock sources like Git or storage systems need additional processing #57

Closed
saheljalal opened this issue Feb 6, 2022 · 3 comments
Closed
Labels
bug Something isn't working good first issue Good for newcomers help wanted Extra attention is needed

Comments

@saheljalal
Copy link
Collaborator

go-getter seems to only be able to download entire folders or repositories/buckets likely and not a single file. So if you try to reference a single file to a private git repo like this:

nostromo dock git::https://github.com/org/private-repo/blob/main/manifest.yaml?sshkey=<key>

it'll fail. We'll need to detect these folders first and keep track of the root to download the source first into the /tmp folder, and then extract the actual file from there into the downloads/ folder for processing the merge.

Also, have not successfully been able to clone private repo either yet, so needs investigation and fix.

@saheljalal saheljalal added bug Something isn't working help wanted Extra attention is needed good first issue Good for newcomers labels Feb 6, 2022
@gedw99
Copy link

gedw99 commented Jun 18, 2022

I am using go-getter to grab single files on public repos and it works.

Under the hood it downloads the whole folder and caches it

Maybe it’s a big on private repos ?

@saheljalal
Copy link
Collaborator Author

Ah maybe, will need to try out on public repo. I assumed it should work but couldn't get it since I was trying to get a file from a private repo. Thanks! Will update ticket to reflect after trying.

@saheljalal
Copy link
Collaborator Author

Finally got this working in latest release. A couple of issues I was running into:

  • The download location was being created first and I think it was causing issues for me so made unique folder references now
  • The URLs for git needed either explicit git:: prefixes or should match the exact HTTPS clone link from GitHub for me

Now dock locations should be able to read multiple manifests from repos or single files and sync.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants