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

Improve containerized codecommit support. #106

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

liambeguin
Copy link

No description provided.

SetupHome overrides what is done in the container-entrypoint script.
Disable it so that this setup phase is kept in a single place.

Signed-off-by: Liam Beguin <[email protected]>
Add support for AWS codecommit remotes in containerized builds.
This allows users to clone repositories using codecommit:// remotes.

According to AWS documentation, `git-remote-codecommit` is the
recommended method for supporting connections made with federated
access, identity providers, and temporary credentials

Signed-off-by: Liam Beguin <[email protected]>
Allow the containerized build to leverage the host SSO tokens,
just like it's done for SSH configurations and keys.

Signed-off-by: Liam Beguin <[email protected]>
Allow the containerized build to leverage the host's git-credentials,
just like it's done for SSH configurations and keys.

Signed-off-by: Liam Beguin <[email protected]>
@jan-kiszka
Copy link
Collaborator

Thanks for your contribution. But could you be a bit more verbose in your descriptions?

Already the very first patch leaves me a bit clueless why exactly this this is now needed with this codecommit. But also what codecommit is (some link?), why one want this, that is also missing, e.g. in patch series header / PR description.

For a broader discussion, it would be good to send the series via git send-email to our mailing list.

@liambeguin
Copy link
Author

Hi @jan-kiszka,

Thanks for taking a look at this. Apologies for not adding more context to the description.
I was going to come back to it and forgot somehow.
I did assume codecommit was known by some though as it's already partially supported.

I apologize also for not sending to the mailing list, I overlooked the CONTRIBUTING document. I'll rework and send there.

Should we consider disabling github PR on the repo if the mailing list is the preferred way?

@jan-kiszka
Copy link
Collaborator

I don't remember any follow-ups - were there any?

Regarding PRs: I don't think this can be disabled on GH (in contrast to gitlab).

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

Successfully merging this pull request may close these issues.

2 participants