Skip to content

enhancement: add clear_license_response_on_deploy configuration option #3892

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
May 28, 2025

Conversation

Paul-Bob
Copy link
Contributor

Description

Adds clear_license_response_on_deploy, which is set to true by default and can be configured as false.

This setting is introduced to handle a very specific use case where an application could send thousands of license verification requests per day, whereas the expected amount, assuming the cache is functioning correctly, is roughly one request every 6 hours.

By default, the license response is cleared on each deploy to ensure freshness. However, in certain environments where deploys are frequent and predictable, and excessive license calls are undesirable, setting this option to false can significantly reduce the number of redundant license verifications.

An example where this might happen is when the application is configured to start a large number of pods. Each pod, when booting Avo, clears the cached response and triggers a new license request, resulting in a high volume of redundant calls.

This update introduces a new configuration attribute to control the clearing of the license response after deployment. The default value is set to true
@Paul-Bob Paul-Bob self-assigned this May 28, 2025
@Paul-Bob Paul-Bob added the Enhancement Not necessarily a feature, but something has improved label May 28, 2025
Copy link

Code Climate has analyzed commit aaf39e9 and detected 0 issues on this pull request.

View more on Code Climate.

@Paul-Bob Paul-Bob merged commit 048a4d5 into main May 28, 2025
21 checks passed
@Paul-Bob Paul-Bob deleted the enhancement/clear_license_response_on_deploy branch May 28, 2025 11:37
Copy link
Contributor

This PR has been merged into main. The functionality will be available in the next release.

Please check the release guide for more information.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement Not necessarily a feature, but something has improved
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant