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

failing with "fatal: could not read Username for 'https://github.com'" after 8.0.0 release #649

Closed
david-wolgemuth opened this issue Oct 31, 2022 · 3 comments · Fixed by #657
Labels
bug Something isn't working GHA / CI Issues specific to GitHub Actions or other CI environments

Comments

@david-wolgemuth
Copy link

david-wolgemuth commented Oct 31, 2022

hello,

our CI started failing after the 8.0.0 release today while running:

npx --yes rdme docs $PATH --version=$VERSION --key=$KEY

giving the following output:

git: 'credential-gcloud.sh' is not a git command. See 'git --help'.
fatal: could not read Username for 'https://github.com': No such device or address

(running within Google Cloud Build, which is why I assume it is looking for credential-gcloud.sh)

we pinned it back to 7.x for the time being, not sure the context around rdme using git

@erunion erunion added the bug Something isn't working label Oct 31, 2022
@kanadgupta kanadgupta added the GHA / CI Issues specific to GitHub Actions or other CI environments label Nov 1, 2022
@kanadgupta
Copy link
Member

Hi @david-wolgemuth, apologies for the trouble. As of v8.0, we use git commands to determine whether or not to run a GitHub Actions onboarding workflow. I have a theory for what could be going on here and I'll open up a PR shortly.

Out of curiosity, could you confirm if your CI setup uses one of the environmental variables mentioned in this list (e.g., BUILDER_OUTPUT or CI)? We use this for detecting CI environments. I see Google Cloud Builder mentioned in that list, but I'm not super familiar with the Google Cloud ecosystem.

@david-wolgemuth
Copy link
Author

Thanks for the quick turnaround @kanadgupta , looking at that lib, it seems GCP sets almost nothing.. We can manually export CI=true 👍

https://github.com/npm/ci-detect/blob/995830e84f7aff0ff1d5e3135773f7561b390367/lib/index.js#L50-L51

@kanadgupta
Copy link
Member

Sounds good @david-wolgemuth! v8.0.1 is now released with a potential fix. Let me know if that addresses the issue — ideally we shouldn't require folks using GCP to manually export a variable like that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working GHA / CI Issues specific to GitHub Actions or other CI environments
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants