Skip to content

Conversation

@nashtapu-tibco
Copy link
Contributor

This pull request adds support for configuring an AuditSafe logs exporter and proxy in the o11y-service Helm chart. The changes introduce new configuration options and environment variables for AuditSafe, ensuring that logs can be exported to an ElasticSearch backend specifically for audit purposes. Additionally, the Jaeger chart version is incremented.

AuditSafe Exporter and Proxy Configuration:

  • Added configuration options and environment variables for the AuditSafe logs exporter, allowing it to be enabled and configured to use ElasticSearch as a backend. This includes index name, endpoint, and username variables.
  • Added configuration options and environment variables for the AuditSafe logs proxy, supporting ElasticSearch backend configuration for audit logs proxying.
  • Introduced fallback logic to set the AuditSafe exporter endpoint to the default ElasticSearch endpoint if not explicitly configured.

Pipeline Configuration:

  • Added new pipeline configuration entries for AuditSafe, including receiver, processor, and exporter variables, to support the new AuditSafe log flow.

Chart Version Update:

  • Incremented the Jaeger chart version from 0.72.39 to 0.72.40 in Chart.yaml to reflect the new changes.

@nashtapu-tibco nashtapu-tibco self-assigned this Sep 26, 2025
@nashtapu-tibco nashtapu-tibco added the bug Something isn't working label Sep 26, 2025
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This pull request promotes the o11y-charts by adding AuditSafe logs exporter and proxy support to the o11y-service Helm chart. The changes introduce comprehensive configuration options for routing audit logs to ElasticSearch backends with appropriate fallback mechanisms.

Key changes include:

  • AuditSafe logs exporter and proxy configuration with ElasticSearch backend support
  • Pipeline configuration entries for AuditSafe log flow processing
  • Jaeger chart version increment from 0.72.39 to 0.72.40

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
charts/o11y-service/templates/configmap.yaml Adds AuditSafe exporter/proxy configuration, environment variables, and pipeline entries
charts/jaeger/Chart.yaml Increments chart version to reflect the new AuditSafe functionality

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@nashtapu-tibco nashtapu-tibco merged commit e510b90 into main Sep 26, 2025
1 of 3 checks passed
@nashtapu-tibco nashtapu-tibco deleted the o11y-charts branch September 26, 2025 10:01
nashtapu-tibco added a commit that referenced this pull request Sep 30, 2025
nashtapu-tibco added a commit that referenced this pull request Sep 30, 2025
* platform-1.11-base promotion

* Platform 1.11: artifactmanager, dp-configure-namespace, dp-core-infrastructure, o11y-service, oauth2-proxy-manifests, tp-dp-monitor-agent, workshop docs & scripts (#311)

* platform-1.11: artifactmanager, dp-configure-namespace, dp-core-infrastructure, o11y-service, oauth2-proxy-manifests, platform-bootstrap, tp-dp-monitor-agent

* platform-1.11: artifactmanager, dp-configure-namespace, dp-core-infrastructure, o11y-service, oauth2-proxy-manifests, platform-bootstrap, tp-dp-monitor-agent

* platform-1.11: workshop docs

* Reverted folder platform-bootstrap to main

* platform-1.11: ha-proxy chart version update

* Publishing Tibco Developer Hub 1.11.0 Charts to public repo. (#312)

* publish bw and flogo charts for 1.11 release (#318)

* publish tp-cp-recipes and integration for bw and flogo 1.11 release (#319)

* publish tp-cp-recipes and integration for bw and flogo 1.11 release

* Msgdp pub 1.11.0.15 (#317)

* promote-cp-proxy: update chart version and values (#320)

* promote o11y-charts (#321)

* platform-1.11-base: promote 1.11.0

---------

Co-authored-by: Subrat kumar gantayat <[email protected]>
Co-authored-by: Abhijit Wakchaure <[email protected]>
Co-authored-by: Chandler Moeller <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants