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

bluecat: support getting credentials from environment variables #2072

Merged

Conversation

seanmalloy
Copy link
Member

Description

Adds support for getting BlueCat API credentials from environment
variables BLUECAT_USERNAME and BLUECAT_PASSWORD. Providing credentails
using the BlueCat JSON configuration file is still supported.
Credentials from environment variables take precedence over credentials
provided in the JSON configuration file.

Using environment variables avoids having to store credentials as k8s
secrets.

Checklist

  • Unit tests updated
  • End user documentation updated

Adds support for getting BlueCat API credentials from environment
variables BLUECAT_USERNAME and BLUECAT_PASSWORD. Providing credentails
using the BlueCat JSON configuration file is still supported.
Credentials from environment variables take precedence over credentials
provided in the JSON configuration file.

Using environment variables avoids having to store credentials as k8s
secrets.
@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label May 6, 2021
@k8s-ci-robot k8s-ci-robot added approved Indicates a PR has been approved by an approver from all required OWNERS files. size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels May 6, 2021
@seanmalloy
Copy link
Member Author

@vinny-sabatini I'll do separate PR to update the end user documentation.

/assign @vinny-sabatini
/kind feature
/uncc @Raffo

@k8s-ci-robot k8s-ci-robot added the kind/feature Categorizes issue or PR as related to a new feature. label May 6, 2021
@k8s-ci-robot k8s-ci-robot removed the request for review from Raffo May 6, 2021 04:37
@seanmalloy
Copy link
Member Author

FYI ... @sanbornick and @seegeekrow

@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: seanmalloy, vinny-sabatini

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@vinny-sabatini
Copy link
Contributor

/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label May 6, 2021
@k8s-ci-robot k8s-ci-robot merged commit caa5069 into kubernetes-sigs:master May 6, 2021
@vinny-sabatini vinny-sabatini deleted the bluecat-cred-from-env branch May 6, 2021 14:11
@seanmalloy seanmalloy mentioned this pull request May 7, 2021
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. kind/feature Categorizes issue or PR as related to a new feature. lgtm "Looks good to me", indicates that a PR is ready to be merged. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants