-
Notifications
You must be signed in to change notification settings - Fork 111
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
Adding s3 configurations for cookbooks in infra server #7796
Conversation
✅ Deploy Preview for chef-automate ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
Signed-off-by: Yashvi Jain <yashvi.jain@progress.com>
Signed-off-by: Yashvi Jain <yashvi.jain@progress.com>
Signed-off-by: Yashvi Jain <yashvi.jain@progress.com>
Signed-off-by: Yashvi Jain <yashvi.jain@progress.com>
Signed-off-by: Kallol Roy <kallol.roy@progress.com>
Signed-off-by: Yashvi Jain <yashvi.jain@progress.com>
Signed-off-by: Yashvi Jain <yashvi.jain@progress.com>
Signed-off-by: dishanktiwari2501 <dtiwari@progress.com>
Signed-off-by: dishanktiwari2501 <dtiwari@progress.com>
Signed-off-by: Yashvi Jain <yashvi.jain@progress.com>
d6b5ce6
to
a914a47
Compare
Signed-off-by: dishanktiwari2501 <dtiwari@progress.com>
components/docs-chef-io/content/automate/chef_infra_external_cookbooks_in_chef_automate.md
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All good except a doc change. Approved when done
Signed-off-by: Kallol Roy <kallol.roy@progress.com>
Signed-off-by: dishanktiwari2501 <dtiwari@progress.com>
Signed-off-by: dishanktiwari2501 <dtiwari@progress.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Review Done!
Signed-off-by: Yashvi Jain <yashvi.jain@progress.com>
Kudos, SonarCloud Quality Gate passed! |
🔩 Description: What code changed, and why?
Added configurations so that cookbooks can be uploaded and fetched on the automate server.
⛓️ Related Resources
https://chefio.atlassian.net/browse/CHEF-1287
https://chefio.atlassian.net/browse/CHEF-3760
👍 Definition of Done
Now if we patch the configuration s3 gets connected to chef server as a bookshelf.
👟 How to Build and Test the Change
Patch the below config:
Upload cookbooks via knife commands
✅ Checklist
All PRs must tick these:
With occasional exceptions, all PRs from Progress employees must tick these:
make spell
in any component directory)All PRs from Progress employees should tick these if appropriate:
Please add a note next to any checkbox above if you are NOT ticking it.
📷 Screenshots, if applicable
Before patching the config:-
After patching the config:-
![image](https://user-images.githubusercontent.com/91940132/229043057-9fb90587-8b09-4202-bc94-36579efceb25.png)
Operations performed after patching the config:-
![image](https://user-images.githubusercontent.com/91940132/229043517-ed059511-b2c4-4d84-a233-e93b365bdabb.png)
Checksums files in s3:-Making s3 disbaled by the s3_enabled flag using:-
Operations performed after patching the config:-
As no cookbooks are uploaded now in s3.