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

Add GitLab token secret #276

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

HebaruSan
Copy link
Member

@HebaruSan HebaruSan commented Sep 11, 2022

Motivation

In KSP-CKAN/CKAN#3661 we added the ability to index mods hosted on GitLab. This included the ability to provide a GitLab authentication token from the command line, but the Inflator isn't using this capability yet. We should add that eventually for completeness, see KSP-CKAN/CKAN#3661 (comment).

Changes

Now if the container configuration has a GL_Token secret, it will be provided to the inflator.
KSP-CKAN/CKAN#3665 will handle passing the env var to the command line.

@HebaruSan HebaruSan added Enhancement New feature or request Inflator Receives netkans and generates ckans Environment It's about the dev/prod environment labels Sep 11, 2022
Copy link
Member

@techman83 techman83 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This have a dependency on me creating the secret + populating it. I'm going to leave it unmerged for now and merge once I get that piece sorted.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement New feature or request Environment It's about the dev/prod environment Inflator Receives netkans and generates ckans
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants