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

New version of opensearch in automate #8670

Merged
merged 9 commits into from
Dec 17, 2024
Merged

Conversation

YashviJain01
Copy link
Collaborator

@YashviJain01 YashviJain01 commented Dec 3, 2024

🔩 Description: What code changed, and why?

Added anew version of opensearch which is 2.18

⛓️ Related Resources

https://progresssoftware.atlassian.net/browse/CHEF-17471
https://progresssoftware.atlassian.net/browse/CHEF-12446

👍 Definition of Done

Automate all services all working fine after the upgrade

👟 How to Build and Test the Change

rebuild component/automate-opensearch

✅ Checklist

All PRs must tick these:

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

https://progresssoftware.sharepoint.com/:v:/s/ChefCoreC/EQbO7SSHsLpHoNA7xL4elRIBJtWYC1wTJh8PnV4-n9M8_g?e=8hTqOZ

Copy link

netlify bot commented Dec 3, 2024

👷 Deploy Preview for chef-automate processing.

Name Link
🔨 Latest commit d1f59e7
🔍 Latest deploy log https://app.netlify.com/sites/chef-automate/deploys/6751591f2103d70008f1901f

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

@YashviJain01 YashviJain01 self-assigned this Dec 5, 2024
@YashviJain01 YashviJain01 requested a review from Dmaddu December 5, 2024 08:30
@YashviJain01 YashviJain01 marked this pull request as ready for review December 5, 2024 08:30
@YashviJain01 YashviJain01 changed the title Testing changes New version of opensearch in automate Dec 5, 2024
@YashviJain01 YashviJain01 changed the base branch from main to feature/major_version_5 December 9, 2024 10:47
Yashvi Jain added 9 commits December 16, 2024 12:21
Signed-off-by: Yashvi Jain <[email protected]>
Signed-off-by: Yashvi Jain <[email protected]>
Signed-off-by: Yashvi Jain <[email protected]>
Signed-off-by: Yashvi Jain <[email protected]>
Signed-off-by: Yashvi Jain <[email protected]>
Signed-off-by: Yashvi Jain <[email protected]>
Signed-off-by: Yashvi Jain <[email protected]>
@YashviJain01 YashviJain01 requested a review from a team as a code owner December 16, 2024 06:52
@github-actions github-actions bot added the documentation Anything related to the Automate docs. label Dec 16, 2024
@YashviJain01 YashviJain01 changed the base branch from feature/major_version_5 to main December 16, 2024 06:54
@YashviJain01 YashviJain01 changed the base branch from main to feature/major_version_5 December 16, 2024 06:54
@YashviJain01 YashviJain01 removed the request for review from a team December 16, 2024 06:55
@Dmaddu Dmaddu merged commit b066026 into feature/major_version_5 Dec 17, 2024
3 of 4 checks passed
@Dmaddu Dmaddu deleted the yashvi/poc_os branch December 17, 2024 01:30
YashviJain01 added a commit that referenced this pull request Dec 19, 2024
* Testing changes

Signed-off-by: Yashvi Jain <[email protected]>

* Testing changes

Signed-off-by: Yashvi Jain <[email protected]>

* Adding env in user bin

Signed-off-by: Yashvi Jain <[email protected]>

* Cleaning the docs correction in integration tests, exclusing system level index

Signed-off-by: Yashvi Jain <[email protected]>

* fixing pipelines

Signed-off-by: Yashvi Jain <[email protected]>

* Adding

Signed-off-by: Yashvi Jain <[email protected]>

* Adding

Signed-off-by: Yashvi Jain <[email protected]>

* Adding index names in cleanup

Signed-off-by: Yashvi Jain <[email protected]>

* Adding correct number of indicies

Signed-off-by: Yashvi Jain <[email protected]>

---------

Signed-off-by: Yashvi Jain <[email protected]>
Co-authored-by: Yashvi Jain <[email protected]>
YashviJain01 added a commit that referenced this pull request Dec 19, 2024
* Testing changes

Signed-off-by: Yashvi Jain <[email protected]>

* Testing changes

Signed-off-by: Yashvi Jain <[email protected]>

* Adding env in user bin

Signed-off-by: Yashvi Jain <[email protected]>

* Cleaning the docs correction in integration tests, exclusing system level index

Signed-off-by: Yashvi Jain <[email protected]>

* fixing pipelines

Signed-off-by: Yashvi Jain <[email protected]>

* Adding

Signed-off-by: Yashvi Jain <[email protected]>

* Adding

Signed-off-by: Yashvi Jain <[email protected]>

* Adding index names in cleanup

Signed-off-by: Yashvi Jain <[email protected]>

* Adding correct number of indicies

Signed-off-by: Yashvi Jain <[email protected]>

---------

Signed-off-by: Yashvi Jain <[email protected]>
Co-authored-by: Yashvi Jain <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Anything related to the Automate docs.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants