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

adding option to specify custom credential name #271

Closed
seldon007 opened this issue Nov 8, 2018 · 6 comments
Closed

adding option to specify custom credential name #271

seldon007 opened this issue Nov 8, 2018 · 6 comments
Labels
kind/feature Categorizes issue or PR as related to a new feature.

Comments

@seldon007
Copy link

seldon007 commented Nov 8, 2018

HI

We got a lot of existing pipelines with Jenkins Credentials ID's baked into them and they do not match the predefined name: namespace-secretname
Maybe add the option for custom names e.g. by annotation or label on the secret?

@gabemontero gabemontero added the kind/feature Categorizes issue or PR as related to a new feature. label Nov 12, 2018
@gabemontero
Copy link

Hey @seldon007

An annotation to customize the name sounds reasonable. The annotation name should start with jenkins.openshift.io/. And an update to the README to announce the feature. It would be an update to https://github.com/openshift/jenkins-sync-plugin/blob/master/src/main/java/io/fabric8/jenkins/openshiftsync/CredentialsUtils.java#L198-L200 to factor in the annotation into the name generation.

My team doesn't have the bandwidth to work on such a feature at this time, but if you are willing to submit a PR with the changes I noted above, I'll review and merge an acceptable version of it.

thanks

@seldon007
Copy link
Author

I not really a developer but I will have a look at it and see if it is above my paygrade :-)

@waveywaves
Copy link

@gabemontero @seldon007 I would like to work on this issue if that is alright. Anything I need to keep in mind before I start ?

@gabemontero
Copy link

hey @waveywaves - PRs are certainly welcome

I can not think of any other input besides what I provided in #271 (comment)

Start there and then we can iterator once you have the PR up.

@waveywaves
Copy link

@gabemontero I am working on this. Please do assign this to me.

@gabemontero
Copy link

good news @waveywaves - unfortunately, the github permissions are set such that I cannot set assignees to anyone outside the openshift org

but don't worry, we'll let you have at it, and your name of course will be on the PR and commit

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature Categorizes issue or PR as related to a new feature.
Projects
None yet
Development

No branches or pull requests

3 participants