[CHEF-29745] local license check, default off #4152
Open
marcparadise wants to merge 3 commits intomainfrom
Open
[CHEF-29745] local license check, default off #4152marcparadise wants to merge 3 commits intomainfrom
marcparadise wants to merge 3 commits intomainfrom
Conversation
Signed-off-by: Marc A. Paradise <marc.paradise@progress.com>
👷 Deploy Preview for chef-server processing.
|
marcparadise
force-pushed
the
CHEF-29745
branch
from
7a15d0d to
4632cf6
Compare
Signed-off-by: Marc A. Paradise <marc.paradise@progress.com>
This is controlled by a build-time compilation macro, OC_LICENSE_PATH By default, this will have a value of 'cli', which will preserve the existing license check behavior of using the automate cli when present. If the macro is set to a file path (this can be done by setting the environment variable OC_LICENSE_PATH to the target location, prior to build) then at run-time, erchef will expect to find a license file in this location. If the specified the license file is missing or invalid, it is treated as a 90 day trial license from time of upgrade to the version that implements this change. When the file is present, the expiration is pulled from the file, based on the entitlement end time furthest in the future. This was chosen because the license content does not directly contain expiration date.
|
marcparadise
commented
Feb 24, 2026
Member
Author
There was a problem hiding this comment.
Something weird happened with the unit tests in one of my internal rebases, so I have excluded them. I willl get them pushed up tomorrow.
marcparadise
commented
Feb 24, 2026
| %%% ===================== | ||
| %%% Internal functions | ||
| %%% ===================== | ||
| check_license(State) -> |
Member
Author
There was a problem hiding this comment.
This is just formatting changes.. but check_license is otherwise unchanged.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This changes adds a license check using a local license file.
This behavior is controlled by a build-time compilation macro, OC_LICENSE_PATH
By default, this will have a value of 'cli', which will preserve the
existing license check behavior of using the automate cli when present.
Existing builds (including downstream) will therefore not be impacted.
If the macro is set to a file path (this can be done by setting the
environment variable OC_LICENSE_PATH to the target location, prior to
build) then at run-time, erchef will expect to find a license file in
this location.
If the specified the license file is missing or invalid, it is treated as
a 90 day trial license from time of upgrade to the version that
implements this change.
When the file is present, the expiration is pulled from the file, based
on the entitlement end time furthest in the future. This was chosen
because the license content does not directly contain expiration date.