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

Auth job variables #18

Merged
merged 2 commits into from
Feb 27, 2020
Merged

Auth job variables #18

merged 2 commits into from
Feb 27, 2020

Conversation

jrob
Copy link
Contributor

@jrob jrob commented Feb 27, 2020

What this PR does / why we need it:

This PR adds to variables for configuration of the post install auth job. This is to help in scenarios where the installation is taking longer but will eventually finish if given more time.

Special notes for your reviewer:

Both of the variables default to either the previous value or the kubernetes default value.

@jrob
Copy link
Contributor Author

jrob commented Feb 27, 2020

@naseemkullah Please review.

naseemkullah
naseemkullah previously approved these changes Feb 27, 2020
@naseemkullah
Copy link
Collaborator

cc @rawkode @gitirabassi @Aisuko

rawkode
rawkode previously approved these changes Feb 27, 2020
@rawkode
Copy link
Contributor

rawkode commented Feb 27, 2020

@jrob could you rebase this against master please?

Thank you for your contributions 👍

@jrob jrob dismissed stale reviews from rawkode and naseemkullah via ded676d February 27, 2020 12:50
@jrob jrob force-pushed the auth-job-variables branch from 0cb84df to ded676d Compare February 27, 2020 12:50
@jrob
Copy link
Contributor Author

jrob commented Feb 27, 2020

@rawkode Glad to help out. Rebased.

@rawkode rawkode merged commit 8666787 into influxdata:master Feb 27, 2020
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.

3 participants