Skip to content

Remove logging during temporary downtime - #181

Merged
CodeDrivenMitch merged 1 commit into
axon-4from
fix/quieter-auth-failure-logging
Sep 22, 2026
Merged

CodeDrivenMitch merged 1 commit into
axon-4from
fix/quieter-auth-failure-logging

Conversation

@CodeDrivenMitch

@CodeDrivenMitch CodeDrivenMitch commented Sep 22, 2026 •

Copy link
Copy Markdown
Collaborator

During the GKE upgrade of last night we had a short auth credential unavailibity (2.3 seconds, while a gateway pod was being replaced). This was logged by the client as WARN and ERROR, which is completely over the top.

This PR changes the logic of the failure logging. It will log on the first connection attempt of the JVM lifetime. It's very unlikely that it's booting right into unavailibility of the Platform side. If it loses connection afterwards, it will log every 10 attempts. So, this is 4 minutes after losing connection, and then every 10 minutes afterwrds.

As the Axoniq Platform client is not essential to the application, any remaining log message has been demoted to info, as to not trigger customer's monitoring during outages

During the GKE upgrade of last night we had a short auth credential unavailibity (2.3 seconds, while a gateway pod was being replaced). This was logged by the client as WARN and ERROR, which is completely over the top.

This PR changes the logic of the failure logging. It will log on the first connection attempt of the JVM lifetime. It's very unlikely that it's booting right into unavailibility of the Platform side. If it loses connection afterwards, it will log every 10 attempts. So, this is 4 minutes after losing connection, and then every 10 minutes afterwrds.

As the Axoniq Platform client is not essential to the application, any remaining log message has been demoted to info, as to not trigger customer's monitoring during outages
@CodeDrivenMitch
CodeDrivenMitch force-pushed the fix/quieter-auth-failure-logging branch from 5cd77c0 to f65b93f Compare September 22, 2026 12:29
@sonarqubecloud

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
C Reliability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

@CodeDrivenMitch
CodeDrivenMitch requested review from a team, Andrew-deVillier and stefanmirkovic and removed request for a team September 22, 2026 12:34
@CodeDrivenMitch CodeDrivenMitch self-assigned this Sep 22, 2026

@stefanmirkovic stefanmirkovic left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@CodeDrivenMitch
CodeDrivenMitch merged commit c38107a into axon-4 Sep 22, 2026
3 of 4 checks passed
@CodeDrivenMitch

Copy link
Copy Markdown
Collaborator Author

Ported to the AF5 line in #183.

This was referenced Sep 23, 2026
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