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

Support private repos #224

Closed
mm318 opened this issue Jun 4, 2020 · 0 comments · Fixed by #252
Closed

Support private repos #224

mm318 opened this issue Jun 4, 2020 · 0 comments · Fixed by #252
Labels
enhancement New feature or request

Comments

@mm318
Copy link
Member

mm318 commented Jun 4, 2020

Description

As stated in the README, action-ros-ci checks out code using vcs import, which doesn't seem to work for private GitHub repos.

Related Issues

While actions/checkout@v2 supports checking out private GitHub repos, the example in the README only uses that checked out repo for generating a *.repos file; action-ros-ci doesn't actually build that checked out code.

Completion Criteria

Be able to use action-ros-ci on a private GitHub repo.

Implementation Notes / Suggestions

Probably make use of the code that actions/checkout@v2 checks out. I think this behavior is also more intuitive for a GitHub actions user, that the code from actions/checkout@v2 is actually the code subsequently being built by colcon, not a copy of the code that came from vcs import.

@mm318 mm318 added the enhancement New feature or request label Jun 4, 2020
emersonknapp added a commit that referenced this issue Jul 21, 2020
This adds a new 'import-token' input to be used when importing a private
repo. Otherwise the step fails with the following error on 'vcs import',
since it tries to read a username and a password from stdin:
fatal: could not read Username for 'https://github.com': No such device or address

The token is used in the HTTPS URL itself, e.g.:
https://[token@]github.com/user/repo.git

Note that this doesn't use the approach suggested in #224. This is more
of a simple workaround given the current logic.

Signed-off-by: Christophe Bedard <[email protected]>

Co-authored-by: Anas Abou Allaban <[email protected]>
Co-authored-by: Emerson Knapp <[email protected]>
jikawa-az pushed a commit that referenced this issue Jul 22, 2020
This adds a new 'import-token' input to be used when importing a private
repo. Otherwise the step fails with the following error on 'vcs import',
since it tries to read a username and a password from stdin:
fatal: could not read Username for 'https://github.com': No such device or address

The token is used in the HTTPS URL itself, e.g.:
https://[token@]github.com/user/repo.git

Note that this doesn't use the approach suggested in #224. This is more
of a simple workaround given the current logic.

Signed-off-by: Christophe Bedard <[email protected]>

Co-authored-by: Anas Abou Allaban <[email protected]>
Co-authored-by: Emerson Knapp <[email protected]>
Signed-off-by: Jesse Ikawa <[email protected]>
jikawa-az pushed a commit that referenced this issue Jul 22, 2020
This adds a new 'import-token' input to be used when importing a private
repo. Otherwise the step fails with the following error on 'vcs import',
since it tries to read a username and a password from stdin:
fatal: could not read Username for 'https://github.com': No such device or address

The token is used in the HTTPS URL itself, e.g.:
https://[token@]github.com/user/repo.git

Note that this doesn't use the approach suggested in #224. This is more
of a simple workaround given the current logic.

Signed-off-by: Christophe Bedard <[email protected]>

Co-authored-by: Anas Abou Allaban <[email protected]>
Co-authored-by: Emerson Knapp <[email protected]>
Signed-off-by: Jesse Ikawa <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant