-
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
Updating the chef-server to the latest version & CBV configs #6726
Conversation
👷 Deploy Preview for chef-automate processing.
|
0a788e6
to
56044f1
Compare
5896190
to
9d5a563
Compare
ffd96c3
to
37b0708
Compare
f9c968b
to
527bb45
Compare
Signed-off-by: Vinay Satish <vinay.satish@progress.com>
527bb45
to
5c0c155
Compare
e628aa9
to
d1c48c6
Compare
Kudos, SonarCloud Quality Gate passed! |
@@ -46,7 +46,7 @@ do_test_deploy() { | |||
chef_server_migration_smoke_tests | |||
## skipping status test because of the missing file in automate - /etc/opscode/chef-server-running.json |
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.
It would be useful to add the status to pedant config so we are not depending on the /etc/opscode/chef-server-running.json (Not a blocker for this PR.
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.
tested the configs and ctl commands. LGTM. Thanks
Signed-off-by: Vinay Satish vinay.satish@progress.com
🔩 Description: What code changed, and why?
Added cookbook version cache configs to sys.config for chef server. Fixing the
chef-server-ctl test
by passing the version config in pedent config'sThis PR updates the automate with the latest chef-server hab package built from the
stable
channel, Cookbook version cache config's andchef-server-ctl test
issue fix⛓️ Related Resources
chef/chef-server#3193
chef/chef-server#3197
👍 Definition of Done
👟 How to Build and Test the Change
✅ 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