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.

Sign up for GitHub

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

Jump to bottom

Fix/opensearch2 6 0 support #15980

Closed
wants to merge 33 commits into from
Closed

Fix/opensearch2 6 0 support #15980

wants to merge 33 commits into from

Conversation

todvora
Copy link
Contributor

@todvora todvora commented Jul 18, 2023 •
edited
Loading

Re-introduction of changes from #15420.
/nocl

Description

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Refactoring (non-breaking change)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.

janheise and others added 30 commits May 8, 2023 13:56
@janheise
unblocking cluster create index before every test, index unblocking o…
c115877 
…nly is not enough
@janheise
unblocking cluster create index before every test, index unblocking o…
d6ab39c 
…nly is not enough
@janheise
proper method + cleanup
acf461e
@janheise
reset possible blocks from other tests
62d1110
@janheise
reset possible blocks from other tests
d5d3ea7
@janheise
reset possible blocks from other tests
f5ce81a
@janheise
reset possible blocks from other tests
393eab4
@janheise
reset possible blocks from other tests
4bec227
@janheise
removed unnecessary resets
167f149
@janheise
different mix transient/persistent settings regarding watermarks & re…
1f9e8c2 
…set of said watermark and blocks
@janheise
removed unnecessary resets
62fa69d
@janheise
adding cluster unblock temporarily
0dedaba
@janheise
undo adding cluster unblock temporarily
0ee6700
@janheise
temporarily disabling flood stage test
c468f63
@janheise
Moving Test to separate class
454662d
@janheise
Spinning up a separate container
97e5fcb
@janheise
removed unnecessary changes in FixtureImporter
c557ed3
@janheise
removed unnecessary changes in FixtureImporter
d48fccc
@todvora
merge master
c889347
@todvora
fixed forbidden call in test
c7b58db
@todvora
Merge branch 'master' into fix/opensearch2_6_0_support
b4b350a
@todvora
avoid creating new Opensearch instance for the MessagesFloodStageTestIT
751eccf
@todvora
Reset cluster block after MessagesIT if memory usage blocks the cluster
45c388d
@todvora
better cluster block reset
e730ac9
@todvora
Merge branch 'master' into fix/opensearch2_6_0_support
81cc88e
@janheise
Merge branch 'master' into fix/opensearch2_6_0_support
1f49ad1
@janheise
remove obsolete code
f446b88
@janheise
bump to 2.7
8ffaf7a
@janheise
bumping RAM
b000991
@janheise
bumping to 5g heap
b09d4fc
@todvora todvora force-pushed the fix/opensearch2_6_0_support branch from 28f80d5 to ec9947d Compare July 18, 2023 10:40
@todvora todvora marked this pull request as ready for review July 18, 2023 13:02
janheise
janheise approved these changes Jul 19, 2023
View reviewed changes
Copy link
Contributor

@janheise janheise left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks ok, can not test locally because of macOS

@todvora
Copy link
Contributor Author

todvora commented Jul 20, 2023

Closing in favor of simplified #15996

@todvora todvora closed this Jul 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@janheise janheise janheise approved these changes

@dennisoelkers dennisoelkers Awaiting requested review from dennisoelkers

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@todvora @janheise