Skip to content
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

License Enforcement Application #3929

Merged
merged 9 commits into from
Nov 18, 2024
Merged

License Enforcement Application #3929

merged 9 commits into from
Nov 18, 2024

Conversation

jashaik
Copy link
Contributor

@jashaik jashaik commented Oct 23, 2024

Description

[Please describe what this change achieves]
License Enforcement Application

Issues Resolved

[List any existing issues this PR resolves, or any Discourse or
StackOverflow discussions that are relevant]

Check List

@jashaik jashaik requested review from a team as code owners October 23, 2024 07:48
Copy link

netlify bot commented Oct 23, 2024

👷 Deploy Preview for chef-server processing.

Name Link
🔨 Latest commit 2df3a3a
🔍 Latest deploy log https://app.netlify.com/sites/chef-server/deploys/6735d915eda863000827917c

@jashaik
Copy link
Contributor Author

jashaik commented Oct 23, 2024

jashaik and others added 8 commits November 14, 2024 15:09
Signed-off-by: jan shahid shaik <[email protected]>
Signed-off-by: jan shahid shaik <[email protected]>
License Caching and Add license enforcement to all APIs.

Signed-off-by: jan shahid shaik <[email protected]>
#3936)

* Update Gemfile.lock to fix adhoc pipeline instability

Signed-off-by: Lincoln Baker <[email protected]>

* Updated chef-server-ctl to handle invalid license + Added grace period in X-Ops-License

Signed-off-by: jan shahid shaik <[email protected]>

* Added customer name in get_license method and license cache

Signed-off-by: jan shahid shaik <[email protected]>

---------

Signed-off-by: Lincoln Baker <[email protected]>
Signed-off-by: jan shahid shaik <[email protected]>
Co-authored-by: Lincoln Baker <[email protected]>
* adding company name and license-id from automate license.

Signed-off-by: talktovikas <[email protected]>

removing trailing space.

Signed-off-by: talktovikas <[email protected]>

fixing tests.

Signed-off-by: talktovikas <[email protected]>

fixing license test case.

Signed-off-by: talktovikas <[email protected]>

Fixing bug in condition statement

Signed-off-by: talktovikas <[email protected]>

debug statements

Signed-off-by: talktovikas <[email protected]>

adding the server url.

Signed-off-by: talktovikas <[email protected]>

assignment in case of standalone chef-server.

Signed-off-by: talktovikas <[email protected]>

adding licenseId in case of Automate-T data.

Signed-off-by: talktovikas <[email protected]>

fixing T-test cases.

Signed-off-by: talktovikas <[email protected]>

fixing tests.

Signed-off-by: talktovikas <[email protected]>

fixing tests.

Signed-off-by: talktovikas <[email protected]>

* remove debug logs

Signed-off-by: talktovikas <[email protected]>

* code review changes.

Signed-off-by: talktovikas <[email protected]>

---------

Signed-off-by: talktovikas <[email protected]>
* Updated chef-server-ctl to handle invalid license + Added grace period in X-Ops-License

Signed-off-by: jan shahid shaik <[email protected]>

* Added customer name in get_license method and license cache

Signed-off-by: jan shahid shaik <[email protected]>

* debug statements

Signed-off-by: talktovikas <[email protected]>

* case for expired license.

Signed-off-by: talktovikas <[email protected]>

making it 403

Signed-off-by: talktovikas <[email protected]>

running cases only for automate.

Signed-off-by: talktovikas <[email protected]>

test for chef-zero

Signed-off-by: talktovikas <[email protected]>

test for /users

Signed-off-by: talktovikas <[email protected]>

adding env in docker

Signed-off-by: talktovikas <[email protected]>

fixing verify syntax.

Signed-off-by: talktovikas <[email protected]>

fixing placement of env variables.

Signed-off-by: talktovikas <[email protected]>

fixing placement of executor.

Signed-off-by: talktovikas <[email protected]>

test

Signed-off-by: talktovikas <[email protected]>

* adding config for expired license from vault.

Signed-off-by: talktovikas <[email protected]>

* Added few puts inside success licese case.

Signed-off-by: sreepuramsudheer <[email protected]>

* Added IS_AUTOMATE into environment variables.

Signed-off-by: sreepuramsudheer <[email protected]>

* Replaced hardcoded expired license with one received from vault.

Signed-off-by: sreepuramsudheer <[email protected]>

* Updated chef-server-ctl to handle invalid license + Added grace period in X-Ops-License

Signed-off-by: jan shahid shaik <[email protected]>

* Added customer name in get_license method and license cache

Signed-off-by: jan shahid shaik <[email protected]>

* removing debug logs

Signed-off-by: talktovikas <[email protected]>

* code review changes.

Signed-off-by: talktovikas <[email protected]>

* adding comment for the test case.

Signed-off-by: talktovikas <[email protected]>

---------

Signed-off-by: jan shahid shaik <[email protected]>
Signed-off-by: talktovikas <[email protected]>
Signed-off-by: sreepuramsudheer <[email protected]>
Co-authored-by: jan shahid shaik <[email protected]>
Co-authored-by: sreepuramsudheer <[email protected]>
@talktovikas
Copy link
Collaborator

Signed-off-by: talktovikas <[email protected]>
Copy link

sonarcloud bot commented Nov 14, 2024

Quality Gate failed Quality Gate failed

Failed conditions
0.0% Coverage on New Code (required ≥ 80%)
33.6% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube

@talktovikas
Copy link
Collaborator

@jashaik
Copy link
Contributor Author

jashaik commented Nov 14, 2024

@jashaik
Copy link
Contributor Author

jashaik commented Nov 15, 2024

@jashaik jashaik merged commit c2d6e8e into main Nov 18, 2024
13 of 15 checks passed
@jashaik jashaik deleted the feature/CHEF-11396 branch November 18, 2024 14:10
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