Skip to content

Conversation

@eldada
Copy link
Contributor

@eldada eldada commented Jun 22, 2025

  1. Update to newer versions of Artifactory, Xray, Prometheus, Loki and Grafana
  2. Remove Artifactory HA. We should avoid promoting it as the Artifactory chart has the HA capabilities baked into it
  3. Fix Xray DB key name (due to a breaking change in the Bitnami PostgreSQL helm chart)
  4. Improve formatting and setup instructions

@eldada eldada changed the title [DO NOT MERGE] Update versions and improve formatting Update versions, add new dashboards and improve formatting Jul 21, 2025
@deepakkjfrog
Copy link
Collaborator

image this unncessary widget should be removed.

@eldada
Copy link
Contributor Author

eldada commented Aug 7, 2025

image this unncessary widget should be removed.

For now, I'll leave it alone. Seems this is an internal Loki validation flow.

value: {{ .Values.artifactory.persistence.mountPath }}
- name: LOKI_URL
value: http://loki.{{ .Release.Namespace }}:3100
value: http://loki.monitoring.svc.cluster.local:3100
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This monitoring seems to be hardcoded here.
People have option to change this:
export OBS_NAMESPACE=monitoring

Can we please take care of this?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed

value: ../../../..{{ .Values.xray.persistence.mountPath }}/etc/fluentd/fluentd.conf
- name: LOKI_URL
value: http://loki.{{ .Release.Namespace }}:3100
value: http://loki.monitoring.svc.cluster.local:3100
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Here also, monitoring is hardcoded.

@shraddha5718 shraddha5718 merged commit a0bf840 into jfrog:master Oct 9, 2025
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.

5 participants