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.

Sign up for GitHub

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

Jump to bottom

Unused admin section in server helm chart #902

Closed
jmorganca opened this issue Feb 6, 2022 · 1 comment · Fixed by #895
Closed

Unused admin section in server helm chart #902

jmorganca opened this issue Feb 6, 2022 · 1 comment · Fixed by #895
Assignees
@mxyng
Labels
area/deploy Issue or PR related to deploying Infra. kind/bug A report or a fix for a problem with functional correctness.

Comments

@jmorganca
Copy link
Contributor

https://github.com/infrahq/infra/blob/main/helm/charts/infra/charts/server/values.yaml#L326

Users should use the adminAccessKey value for now
@jmorganca jmorganca added kind/bug A report or a fix for a problem with functional correctness. area/deploy Issue or PR related to deploying Infra. labels Feb 6, 2022
@mxyng mxyng self-assigned this Feb 7, 2022
@mxyng
Copy link
Collaborator

mxyng commented Feb 7, 2022

this was left over from the original revision and already removed from the infra chart values

mxyng added a commit that referenced this issue Feb 7, 2022
@mxyng
#902: remove admin values
f9caf0a
@mxyng mxyng mentioned this issue Feb 7, 2022
Merged
6 tasks
mxyng added a commit that referenced this issue Feb 7, 2022
@mxyng
#902: remove admin values
abe7e94
@mxyng mxyng closed this as completed in #895 Feb 8, 2022
mxyng added a commit that referenced this issue Feb 8, 2022
@mxyng
#902: remove admin values
a2b9c84
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mxyng mxyng

Labels
area/deploy Issue or PR related to deploying Infra. kind/bug A report or a fix for a problem with functional correctness.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

update docs infrahq/infra
2 participants
@jmorganca @mxyng