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

icinga-stack: Use icingaadmin as Icinga Web admin_user #43

Merged
merged 2 commits into from
Feb 27, 2024
Merged

icinga-stack: Use icingaadmin as Icinga Web admin_user #43

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
377c105
`icinga-stack`: Use `icingaadmin` as `Icinga Web` `admin_user`
lippserd Feb 27, 2024
b8fb94a
Merge branch 'main' into icinga-web-admin_user
mocdaniel Feb 27, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion charts/icinga-stack/charts/icingaweb2/values.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ ingress:
auth:
type: db
#resource: # Add the name of the db resource used by Icinga Web 2 here
admin_user: icingaweb
admin_user: icingaadmin
admin_password:
value: # Add a password here
credSecret: # Or use existing secret
Expand Down
2 changes: 1 addition & 1 deletion charts/icinga-stack/docs/configuration.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -147,7 +147,7 @@ These values are used by the Icinga Web sub-chart. For configuration of Icinga W
| `icingaweb2.ingress.tls[].hosts` | Hosts of the Icinga Web ingress | `[]string` | **not set** |
| `icingaweb2.ingress.tls[].secretName` | Secret name of the Icinga Web ingress | `string` | **not set** |
| `icingaweb2.auth.type` | Type of the Icinga Web authentication | `string` | `db` |
| `icingaweb2.auth.admin_user` | Admin user of the Icinga Web authentication | `string` | `icingaweb` |
| `icingaweb2.auth.admin_user` | Admin user of the Icinga Web authentication | `string` | `icingaadmin` |
| `icingaweb2.auth.admin_password.value` | Admin password of the Icinga Web authentication. Can be set from secret specified in `icingaweb2.auth.admin_password.credSecret` and `icingaweb2.auth.admin_password.secretKey` | `string` | **not set** |
| `icingaweb2.modules.<module>.enabled` | Whether or not to enable the Icinga Web module | `boolean` | **varies** |
| `icingaweb2.resources` | Resources of the Icinga Web deployment | `map[string]string` | `{}` |
Expand Down
4 changes: 2 additions & 2 deletions charts/icinga-stack/tests/icingaweb2_deployment_test.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ tests:
- contains:
path: spec.template.spec.containers[0].env
content:
name: icingaweb.passwords.icingaweb2.icingaweb
name: icingaweb.passwords.icingaweb2.icingaadmin
value: insecurepassword

- it: deploys an Icinga Web deployment with additional modules
Expand Down Expand Up @@ -74,7 +74,7 @@ tests:
- contains:
path: spec.template.spec.containers[0].env
content:
name: icingaweb.passwords.icingaweb2.icingaweb
name: icingaweb.passwords.icingaweb2.icingaadmin
valueFrom:
secretKeyRef:
name: icingaweb2-secret
Expand Down
2 changes: 1 addition & 1 deletion charts/icinga-stack/values.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -395,7 +395,7 @@ icingaweb2:
auth:
type: db
#resource: # Add the name of the db resource used by Icinga Web 2 here
admin_user: icingaweb
admin_user: icingaadmin
admin_password:
value: # Add a password here
credSecret:
Expand Down