Skip to content

fix(kpi,PIN-9159): add kafka timeouts to analytic audit writers (DEV,QA,PROD)#164

Merged
marco-vit-pagopa merged 2 commits intomainfrom
fix/PIN-9159_set_sessiontimeout_and_requesttimeout_parameters_DEV_QA_PROD
Feb 11, 2026
Merged

fix(kpi,PIN-9159): add kafka timeouts to analytic audit writers (DEV,QA,PROD)#164
marco-vit-pagopa merged 2 commits intomainfrom
fix/PIN-9159_set_sessiontimeout_and_requesttimeout_parameters_DEV_QA_PROD

Conversation

@marco-vit-pagopa
Copy link
Contributor

No description provided.

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 adds custom Kafka timeout configurations to application audit analytics services across DEV, QA, and PROD environments to address timeout-related issues (PIN-9159). The changes configure session, request, and batch wait timeouts for both the application-audit-archiver and application-audit-analytics-writer microservices.

Changes:

  • Added three new Kafka timeout configurations (KAFKA_SESSION_TIMEOUT_MILLIS, KAFKA_REQUEST_TIMEOUT_MILLIS, KAFKA_MAX_WAIT_BATCH_MILLIS) to both application-audit-archiver and application-audit-analytics-writer services
  • Commented out the common-kafka KAFKA_MAX_WAIT_BATCH_MILLIS reference to use custom values
  • Added ACCUMULATOR_MAX_MESSAGES and ACCUMULATOR_FLUSH_TIMEOUT_MS to QA analytics writer configuration

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 6 comments.

Show a summary per file
File Description
microservices/application-audit-archiver/qa-analytics/values.yaml Adds custom Kafka timeout values and comments out common-kafka reference for QA environment
microservices/application-audit-archiver/prod-analytics/values.yaml Adds custom Kafka timeout values and comments out common-kafka reference for PROD environment
microservices/application-audit-archiver/dev-analytics/values.yaml Adds custom Kafka timeout values and comments out common-kafka reference for DEV environment
microservices/application-audit-analytics-writer/qa-analytics/values.yaml Adds custom Kafka timeout values, accumulator settings, and comments out common-kafka reference for QA environment
microservices/application-audit-analytics-writer/prod-analytics/values.yaml Adds two new Kafka timeout values to existing custom configuration for PROD environment
microservices/application-audit-analytics-writer/dev-analytics/values.yaml Adds two new Kafka timeout values to existing custom configuration for DEV environment

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@marco-vit-pagopa marco-vit-pagopa marked this pull request as ready for review February 11, 2026 10:05
@marco-vit-pagopa marco-vit-pagopa merged commit da43acb into main Feb 11, 2026
7 checks passed
@marco-vit-pagopa marco-vit-pagopa deleted the fix/PIN-9159_set_sessiontimeout_and_requesttimeout_parameters_DEV_QA_PROD branch February 11, 2026 12:58
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.

2 participants