automate icon indicating copy to clipboard operation
automate copied to clipboard

[fix][opensearch] Remove curl duplicated arguments in HC script

Open mauroseb opened this issue 1 year ago • 1 comments

:nut_and_bolt: Description: What code changed, and why?

The current health check script for opensearch has duplicated arguments. The functionality does not change as only the last occurrence of the argument will take precedence for curl.

:athletic_shoe: How to Build and Test the Change

:white_check_mark: Checklist

All PRs must tick these:

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

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

All PRs from Progress employees should tick these if appropriate:

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

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

:camera: Screenshots, if applicable

mauroseb avatar Dec 15 '23 11:12 mauroseb

Deploy Preview for chef-automate processing.

Name Link
Latest commit d68533951712d1d1793ade7468c9c71c6616b891
Latest deploy log https://app.netlify.com/sites/chef-automate/deploys/657c32cc7ca1530008a87e8f

netlify[bot] avatar Dec 15 '23 11:12 netlify[bot]