pre check for rollover action #88
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
thalurur
reviewed
Jun 30, 2021
...lin/org/opensearch/indexmanagement/indexstatemanagement/step/rollover/AttemptRolloverStep.kt
Outdated
Show resolved
Hide resolved
Codecov Report
@@ Coverage Diff @@
## main #88 +/- ##
============================================
+ Coverage 77.84% 78.02% +0.17%
- Complexity 1865 1877 +12
============================================
Files 255 255
Lines 10334 10378 +44
Branches 1525 1534 +9
============================================
+ Hits 8045 8097 +52
+ Misses 1470 1451 -19
- Partials 819 830 +11
Continue to review full report at Codecov.
|
Signed-off-by: bowenlan-amzn <bowenlan23@gmail.com>
dbbaughe
reviewed
Jul 1, 2021
...lin/org/opensearch/indexmanagement/indexstatemanagement/step/rollover/AttemptRolloverStep.kt
Outdated
Show resolved
Hide resolved
dbbaughe
reviewed
Jul 1, 2021
...lin/org/opensearch/indexmanagement/indexstatemanagement/step/rollover/AttemptRolloverStep.kt
Outdated
Show resolved
Hide resolved
dbbaughe
reviewed
Jul 1, 2021
...lin/org/opensearch/indexmanagement/indexstatemanagement/step/rollover/AttemptRolloverStep.kt
Outdated
Show resolved
Hide resolved
dbbaughe
reviewed
Jul 1, 2021
...lin/org/opensearch/indexmanagement/indexstatemanagement/step/rollover/AttemptRolloverStep.kt
Outdated
Show resolved
Hide resolved
dbbaughe
reviewed
Jul 1, 2021
...lin/org/opensearch/indexmanagement/indexstatemanagement/step/rollover/AttemptRolloverStep.kt
Outdated
Show resolved
Hide resolved
bowenlan-amzn
force-pushed
the
rollover
branch
2 times, most recently
from
d4854cc to
5a28273
Compare
dbbaughe
approved these changes
Jul 1, 2021
thalurur
pushed a commit
to thalurur/open-index-management
that referenced
this pull request
Oct 22, 2021
* Fixes small issues on new ISM UI Signed-off-by: Drew Baugher <46505179+dbbaughe@users.noreply.github.com> * Updates snapshots Signed-off-by: Drew Baugher <46505179+dbbaughe@users.noreply.github.com> * Updates release notes Signed-off-by: Drew Baugher <46505179+dbbaughe@users.noreply.github.com>
thalurur
pushed a commit
to thalurur/open-index-management
that referenced
this pull request
Oct 22, 2021
* Rename plugin helper file * Update TransformIndices.tsx * Backport commits from main (opensearch-project#90) * Release 1.1.0.0 (opensearch-project#83) * Release 1.1.0.0 Signed-off-by: bowenlan-amzn <bowenlan23@gmail.com> * Correct copyright notices * UI fixes for new ISM UI (opensearch-project#84) * Removes X icon from action/transition flyout footer next to cancel Signed-off-by: Drew Baugher <46505179+dbbaughe@users.noreply.github.com> * Adds tooltips to edit/delete icon buttons on the draggable action/transition components Signed-off-by: Drew Baugher <46505179+dbbaughe@users.noreply.github.com> * Removes overlay for flyout so clicking outside doesn't close flyout and removes X close button on top right Signed-off-by: Drew Baugher <46505179+dbbaughe@users.noreply.github.com> * Adds edit/delete tooltips for state component and removes underline from hovering states Signed-off-by: Drew Baugher <46505179+dbbaughe@users.noreply.github.com> * Changes state flyout from X Close to Cancel Signed-off-by: Drew Baugher <46505179+dbbaughe@users.noreply.github.com> * Adds JSON editor for allocation action Signed-off-by: Drew Baugher <46505179+dbbaughe@users.noreply.github.com> * Adds isValid to action interface and implements in actions Signed-off-by: Drew Baugher <46505179+dbbaughe@users.noreply.github.com> * Correctly removes unused rollover keys from rollover object, handles uncontrolled inputs and NaN minDocs Signed-off-by: Drew Baugher <46505179+dbbaughe@users.noreply.github.com> * Adds error for state name already existing in policy Signed-off-by: Drew Baugher <46505179+dbbaughe@users.noreply.github.com> * Fixes duplicated actions when editing as previously we had a new ID and matching logic didn't find an action Signed-off-by: Drew Baugher <46505179+dbbaughe@users.noreply.github.com> * Fixes other action inputs that can return NaN and creates a no conditions option that is the default Signed-off-by: Drew Baugher <46505179+dbbaughe@users.noreply.github.com> * Updates snapshot Signed-off-by: Drew Baugher <46505179+dbbaughe@users.noreply.github.com> * Correctly show danger toast on update/create failures Signed-off-by: Drew Baugher <46505179+dbbaughe@users.noreply.github.com> * Fixes rollup action nesting multiple ism_template keys Signed-off-by: Drew Baugher <46505179+dbbaughe@users.noreply.github.com> * Fixes timeout/retry settings turning into uncontrolled inputs and NaN value Signed-off-by: Drew Baugher <46505179+dbbaughe@users.noreply.github.com> * Removes text transformation on inputs and updates isValid methods and passes to form labels Signed-off-by: Drew Baugher <46505179+dbbaughe@users.noreply.github.com> * Fixes allocation and rollup adding UIAction properties to policy JSON on updates Signed-off-by: Drew Baugher <46505179+dbbaughe@users.noreply.github.com> * Updates release notes w/ new PR changes Signed-off-by: Drew Baugher <46505179+dbbaughe@users.noreply.github.com> * Wraps actions/transitions in states component and updates default rollup to include ism_rollup key Signed-off-by: Drew Baugher <46505179+dbbaughe@users.noreply.github.com> * Fixes some small UI touchups/issues for new ISM UI (opensearch-project#85) * Moves cancel secondary button next to primary on action/transition flyout Signed-off-by: Drew Baugher <46505179+dbbaughe@users.noreply.github.com> * Removes default ISM template and increases width of empty prompt Signed-off-by: Drew Baugher <46505179+dbbaughe@users.noreply.github.com> * Adds punctuation Signed-off-by: Drew Baugher <46505179+dbbaughe@users.noreply.github.com> * Adds back X icon next to cancel for state flyout Signed-off-by: Drew Baugher <46505179+dbbaughe@users.noreply.github.com> * Makes inputs in flyout full width, updates help text, and adds some spacing Signed-off-by: Drew Baugher <46505179+dbbaughe@users.noreply.github.com> * Updates transition default to not include conditions empty object, and on change transition to delete conditions when selecting none Signed-off-by: Drew Baugher <46505179+dbbaughe@users.noreply.github.com> * Moves edit button out of policy settings content panel and on to global page for view policy Signed-off-by: Drew Baugher <46505179+dbbaughe@users.noreply.github.com> * Updates release note Signed-off-by: Drew Baugher <46505179+dbbaughe@users.noreply.github.com> * Fixes broken link Signed-off-by: Drew Baugher <46505179+dbbaughe@users.noreply.github.com> * Fixes small issues on new ISM UI (opensearch-project#88) * Fixes small issues on new ISM UI Signed-off-by: Drew Baugher <46505179+dbbaughe@users.noreply.github.com> * Updates snapshots Signed-off-by: Drew Baugher <46505179+dbbaughe@users.noreply.github.com> * Updates release notes Signed-off-by: Drew Baugher <46505179+dbbaughe@users.noreply.github.com> * Updates workflows to trigger on 1.* branches Signed-off-by: Drew Baugher <46505179+dbbaughe@users.noreply.github.com> Co-authored-by: Bowen Lan <62091230+bowenlan-amzn@users.noreply.github.com> * Support data filter when viewing sample data * Update TransformOptions.tsx * Add TransformService.test.ts * Draft of date histogram panel * Add helper method and move import * Split calendar and fixed interval to 2 panels * Add rendering test * Add close popover test * Update cypress-workflow.yml * Update links.yml * Update unit-tests-workflow.yml * Update TransformOptions.tsx * Update tests * Update DateHistogramPanel.test.tsx * Make scripted metrics editor larger * Expand code editor related panels * Undo changes to workflow Signed-off-by: Annie Lee <leeyun@amazon.com> * Update TimeAggregation.tsx * Update DefineTransforms.tsx undo changes to style * Clean up code * Update snapshot Signed-off-by: Annie Lee <leeyun@amazon.com> * Refactor the order of checking if name is defined before checking for duplicate name * Update CreateTransformForm.tsx * Update DefineTransforms.tsx Signed-off-by: Annie Lee <leeyun@amazon.com> * Update DefineTransforms.tsx Signed-off-by: Annie Lee <leeyun@amazon.com> * Update DefineTransforms.tsx * Update DefineTransforms.tsx * Update DefineTransforms.tsx Co-authored-by: Drew Baugher <46505179+dbbaughe@users.noreply.github.com> Co-authored-by: Bowen Lan <62091230+bowenlan-amzn@users.noreply.github.com>
wuychn
pushed a commit
to ochprince/index-management
that referenced
this pull request
Mar 16, 2023
* pre check for rollover action Signed-off-by: bowenlan-amzn <bowenlan23@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Signed-off-by: bowenlan-amzn bowenlan23@gmail.com
Issue #, if available:
#45
Description of changes:
Add pre-condition check to rollover action.
We are only using info from cluster state, but not doing a
GET _aliasescall. Theoretically this could have a 30s delay. But even this delay bypass our pre-check (user did manually rollover just within 30s delay and write index moved to new index), this wont break the whole workflow.CheckList:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.