Skip to content

Commit

Permalink
feat(lmotel): add proxy support using env (#137)
Browse files Browse the repository at this point in the history
* feat(lmotel): add proxy support using env

* feat(lmotel): add README.md

---------

Co-authored-by: Khyati Gandhi <[email protected]>
  • Loading branch information
khyatigandhi6 and khyatigandhi0612 authored Mar 5, 2024
1 parent 36fe259 commit 7bc570b
Show file tree
Hide file tree
Showing 6 changed files with 23 additions and 2 deletions.
2 changes: 1 addition & 1 deletion charts/lmotel/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ maintainers:
- name: LogicMonitor
email: [email protected]
name: lmotel
version: 1.8.0
version: 1.9.0
home: https://logicmonitor.github.io/helm-charts
appVersion: v2.0.10
dependencies:
Expand Down
8 changes: 7 additions & 1 deletion charts/lmotel/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -65,6 +65,11 @@ To enable logs add the following option
--set logs.enable=true \
```

To enable proxy support add the following option
``` console
--set envVars.HTTPS_PROXY=<proxy_server_ip:port> \
```

---
Required Values:
- **account (default: `""`):** The LogicMonitor account name.
Expand Down Expand Up @@ -114,4 +119,5 @@ Optional Values:
- **ingress.tls.secretName (default: `""`):** Name of the TLS secret containing the TLS certificates for the hostname.
- **ingress.annotations (default: `{}`):** Annotations common to all the ingress resource definitions.
- **ingress.http.annotations (default: `{}`):** Annotations specific to the ingress-http resource
- **ingress.grpc.annotations (default: `{}`):** Annotations specific to the ingress-grpc resource
- **ingress.grpc.annotations (default: `{}`):** Annotations specific to the ingress-grpc resource
- **envVars (default: `{}`):** Environment variables in form of key value pair
4 changes: 4 additions & 0 deletions charts/lmotel/templates/daemonset.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -73,6 +73,10 @@ spec:
optional: true
- name: LOGICMONITOR_OTEL_NAMESPACE
value: {{ include "lmutil.release.namespace" . }}
{{- range $key, $val := .Values.envVars }}
- name: {{ $key }}
value: {{ $val | quote }}
{{- end }}
{{- if .Values.external_config.lmconfig }}
args:
- --config
Expand Down
4 changes: 4 additions & 0 deletions charts/lmotel/templates/statefulset.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -77,6 +77,10 @@ spec:
optional: true
- name: LOGICMONITOR_OTEL_NAMESPACE
value: {{ include "lmutil.release.namespace" . }}
{{- range $key, $val := .Values.envVars }}
- name: {{ $key }}
value: {{ $val | quote }}
{{- end }}
{{- if .Values.external_config.lmconfig}}
args:
- --config
Expand Down
5 changes: 5 additions & 0 deletions charts/lmotel/values.schema.json
Original file line number Diff line number Diff line change
Expand Up @@ -252,6 +252,11 @@
"examples": [
]
},
"envVars": {
"$id": "#properties/envVars",
"title": "Environment Variables",
"type": "object"
},
"ingress": {
"$id": "#properties/ingress",
"title": "LMOTEL Collector Ingress schema",
Expand Down
2 changes: 2 additions & 0 deletions charts/lmotel/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,8 @@ external_config:
lmconfig: ""
arguments: []

envVars: {}

ingress:
enabled: false
annotations: {}
Expand Down

0 comments on commit 7bc570b

Please sign in to comment.