Skip to content

Commit

Permalink
Merge branch 'main' into sos2403b
Browse files Browse the repository at this point in the history
  • Loading branch information
gsmith-sas authored Feb 22, 2024
2 parents fb9ae4a + f220694 commit 019bb4a
Show file tree
Hide file tree
Showing 6 changed files with 27 additions and 4 deletions.
10 changes: 10 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,16 @@
## unreleased
* **Overall**
* [CHANGE] Drop support for OpenShift 4.11; the minimum supported version of OpenShift is now 4.12.
* [FIX] Revised `samples/azure-deployment/README.md` to remove obsolete information and bring content up-to-date. (Fixes #612)

* **Metrics**
* [FIX] Set environment variable `MON_TLS_PATH_INGRESS` to ensure correct datasource connection between Grafana
and Promethues in [Azure Deployment sample](samples/azure-deployment). (Fixes #614)

* **Logging**
* [FIX] Corrected comments referencing OpenSearch connection information in `samples/generic-base/logging/user-values-es-exporter.yaml`
and `logging/user-values-es-exporter.yaml`.
* [FIX] Corrected typo in `logging/bin/deploy_fluentbit_azmonitor.sh` that prevented the script from executing properly.

## Version 1.2.22 (13FEB2024)
* **Overall**
Expand Down
2 changes: 1 addition & 1 deletion logging/bin/deploy_fluentbit_azmonitor.sh
Original file line number Diff line number Diff line change
Expand Up @@ -158,7 +158,7 @@ kubectl -n $LOG_NS label configmap fbaz-env-vars managed-by=v4m-es-script
log_debug "Fluent Bit Helm Chart: repo [$FLUENTBIT_HELM_CHART_REPO] name [$FLUENTBIT_HELM_CHART_NAME] version [$FLUENTBIT_HELM_CHART_VERSION]"
chart2install="$(get_helmchart_reference $FLUENTBIT_HELM_CHART_REPO $FLUENTBIT_HELM_CHART_NAME $FLUENTBIT_HELM_CHART_VERSION)"
versionstring="$(get_helm_versionstring $FLUENTBIT_HELM_CHART_VERSION)"
log_debug "Installing Helm chart from artifact [$chart2install]
log_debug "Installing Helm chart from artifact [$chart2install]"

# Deploy Fluent Bit via Helm chart
helm $helmDebug upgrade --install v4m-fbaz --namespace $LOG_NS \
Expand Down
2 changes: 1 addition & 1 deletion logging/user-values-es-exporter.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -8,4 +8,4 @@
# es:
## The env vars ES_USER and ES_PASSWORD are set, by default,
## based on the K8s secret internal-user-metricgetter
# uri: https://$(ES_USER):$(ES_PASSWORD)@v4m-es-client-service:9200
# uri: https://$(ES_USER):$(ES_PASSWORD)@v4m-search:9200
14 changes: 13 additions & 1 deletion samples/azure-deployment/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -39,8 +39,20 @@ path-based ingress and are available at the following URLs. Be sure to
replace the host names in the URLs with the host names for your deployment.

* `https://host.mycluster.example.com/grafana`
* `https://host.mycluster.example.com/dashboards`

This sample does not make Prometheus and Alertmanager available since these
web applications do not provide any sort of user-authentication. However,
if you want to make these applications available, you can do so by setting
the `enabled` key available in the appropriate section of the `monitoring/user-values=prom-operator.yaml`
file (e.g. set the key `alertmanager.ingress.enabled` to 'true' to make
Alertmanager accessible).

If you have chosen to make these applications available, the applications
are available at the following URLs. As mentioned above, be sure to
replace the host names in the URLs with the host names for your deployment.

* `https://host.mycluster.example.com/prometheus`
* `https://host.mycluster.example.com/alertmanager`
* `https://host.mycluster.example.com/kibana`

For an example of using host-based ingress, see the [ingress sample](../ingress).
1 change: 1 addition & 0 deletions samples/azure-deployment/monitoring/user.env
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
MON_TLS_PATH_INGRESS=true
2 changes: 1 addition & 1 deletion samples/generic-base/logging/user-values-es-exporter.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -8,4 +8,4 @@
# es:
## The env vars ES_USER and ES_PASSWORD are set, by default,
## based on the K8s secret internal-user-metricgetter
# uri: https://$(ES_USER):$(ES_PASSWORD)@v4m-es-client-service:9200
# uri: https://$(ES_USER):$(ES_PASSWORD)@v4m-search:9200

0 comments on commit 019bb4a

Please sign in to comment.