Dave/stalwart 262

[daveaugustus]

🔩 Description: What code changed, and why?

configure errchef service through Automate config

⛓️ Related Resources

👍 Definition of Done

• The configuration needs to be patchable via chef-automate config patch
• The default configuration needs to be set as per the Standalone Chef Server values
• In case of Automate HA, if there is a different default, that needs to be set in the config.toml.erb file
• The Standalone Automate needs to be deployed and tested to see if all the service configurations are in place
• The Automate deployed Chef Server should be running fine
• The Automate HA needs to be deployed and tested to see if all the service configurations for Chef Server are in place
• The Automate deployed Chef Server in HA should be running fine
• All the changed values need to be documented

👟 How to Build and Test the Change

rebuild components/automate-deployment/
rebuild components/automate-cli/
rebuild components/automate-cs-oc-erchef/

• Adde the following conf to a .toml file

[erchef.v1.sys.iBrowse]
    ibrowse_max_pipeline_size = 15
    ibrowse_max_sessions = 17

and patch using the following command
chef-automate config patch new_config.toml

The result should look like this if you run the following command:

cat /hab/svc/automate-cs-oc-erchef/config/sys.config

✅ Checklist

All PRs must tick these:

• I have read the CONTRIBUTING document.
• All commits signed-off for the Developer Certification of Origin.

With occasional exceptions, all PRs from Progress employees must tick these:

• Is the code clear? (complicated code or lots of comments--subdivide and use well-named methods, meaningful variable names, etc.)
• Consistency checked? (user notifications, user prompts, visual patterns, code patterns, variable names)
• Repeated code blocks eliminated? (adapt and reuse existing components, blocks, functions, etc.)
• Spelling, grammar, typos checked? (at a minimum use make spell in any component directory)
• Code well-formatted? (indents, line breaks, etc. improve rather than hinder readability)

All PRs from Progress employees should tick these if appropriate:

• Tests added/updated? (all new code needs new tests)
• Docs added/updated? (all customer-facing changes)

Please add a note next to any checkbox above if you are NOT ticking it.

📷 Screenshots, if applicable

[netlify]

👷 Deploy Preview for chef-automate processing.

Name	Link
🔨 Latest commit	5d0d7f3
🔍 Latest deploy log	https://app.netlify.com/sites/chef-automate/deploys/6384b2e90d14ed00095dedc1

[kalroy]

The changes in sys.config are not correct. Needs to be fixed.

[daveaugustus]

Test on automate standalone the result looks fine

{cleanup_batch_size, 9}

{s3_url_ttl, 500},

 {ibrowse, [
        {default_max_pipeline_size, 30},
        {default_max_sessions, 300}
    ]},

[Dmaddu]

@dave-augustus Please upload the demo video as well.

[daveaugustus]

Automate HA

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
No Duplication information

[daveaugustus]

Screen.Recording.2022-11-29.at.1.44.13.PM.mov