Avoid making the index read-only in the Force merge action for ILM (Closes #43426)#81162

Merged

dakrone merged 46 commits intoelastic:masterfrom jcbages:jcbages/allow-disabling-read-only-forcemerge-ilm

Jun 9, 2022

+94-36

Commits on Jun 9, 2022

allow for a readOnly parameter with default true in ForceMergeAction
jcbages
committed
fix existing tests to use either a randomBoolean or true for the readOnly parameter in ForceMergeAction
jcbages
committed
fix test failing in ForceMergeActionTest
jcbages
committed
add tests for read only separating the assert into read and nonread
jcbages
committed
remove redundant test for invalid parse value
jcbages
committed
add read_only as part of the forcemerge response
jcbages
committed
read only for docs
jcbages
committed
fix format
jcbages
committed
run spotless apply task
jcbages
committed
undo order swap
jcbages
committed
fix failing forcemerge test
jcbages
committed
make readonly the third argument for parser and IO and add BWC constants
jcbages
committed
bump min version to 8
jcbages
committed
bump up version to 8_1
jcbages
committed
set valid next step in forcemerge action
jcbages
committed
fix forcemerge tests
jcbages
committed
use read only for createfullpolicy as default behavior is read only (fixing typo of prev commit)
jcbages
committed
add test for non read only version of force merge action in timeseriesdatastreamsit
jcbages
committed
fix test typo
jcbages
committed
move index settings check inside assertbusy
jcbages
committed
add extra tests in timeseriesdatastreamit and make it more similar to forcemergeactiontest from timeserieslifecyclesactionit
jcbages
committed
add non read only checks and tests for TimeSeriesLifecycleActionsIT
jcbages
committed
apply spotlessApply action
jcbages
committed
fix bug in TimeSeriesLifecycleActionsIT in which readOnly was not being passed to forcemergeaction constructor
jcbages
committed
fix test that was expecting errors when indexing document when nonreadonly forcemerge was applied
jcbages
committed
remove read_only parameter from docs
jcbages
committed
remove readOnly from forcemergeaction and instead disable the read only settings always
jcbages
committed
change forcemergeactiontests to only use the ones that check non-readonly forcemerge steps
jcbages
committed
fix tests to avoid passing the readOnly parameter and remove readOnly tests
jcbages
committed
run spotlessApply
jcbages
committed
minor style change
jcbages
committed
fix test in PhaseCacheManagementTests.java
jcbages
committed
fix bad head conflict in test file
jcbages
committed
add a new noop step which does nothing but enables backwards compatibility whenever we change the functionality of a step
jcbages
committed
rename checkNotWriteIndex to checkNotWriteIndexKey and add the noop step with the readonlykey
jcbages
committed
update forcemergeactiontest to reflect the addition of the noop step
jcbages
committed
fix PhaseCacheManagementTests to acknowledge the new noop step
jcbages
committed
add a name constant to the noop step
jcbages
committed
fix constructor name for noopstep
jcbages
committed
fix styling
jcbages
committed
fix format error
jcbages
committed
add changelog yaml
jcbages
committed
make noopstep a clusterstatewaitstep
jcbages
committed
update javadoc
jcbages
committed
fix tostep test
jcbages
committed
Update NoopStep NAME to be lowercase by convention

jcbages
and
dakrone
authored