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

Add rel to link for external links, IM dashboards plugin #261

Merged
merged 13 commits into from
Oct 3, 2022

Conversation

phestecAMZN
Copy link
Contributor

Description

[Describe what this change achieves]

Issues Resolved

[List any issues this PR will resolve]

Check List

  • [ x] Commits are signed per the DCO using --signoff

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.

@phestecAMZN phestecAMZN requested review from a team, AWSHurneyt and bowenlan-amzn September 15, 2022 14:58
Signed-off-by: Chris Hesterman <[email protected]>
"opensearchDashboardsVersion": "2.2.1",
"configPath": ["opensearch_index_management"],
"requiredPlugins": ["navigation"],
"version": "2.3.0.0",
Copy link
Member

Choose a reason for hiding this comment

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

Can we remove the version bump from this PR and then add a backport PR from 2.x to main for this PR. This way this PR can be backported to other branches like 2.x without issues

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yes, on it

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

Signed-off-by: Chris Hesterman <[email protected]>
Copy link
Member

@bowenlan-amzn bowenlan-amzn left a comment

Choose a reason for hiding this comment

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

I don't think we need to delete 2.3 release note here

@phestecAMZN phestecAMZN requested a review from lezzago September 29, 2022 22:27
@phestecAMZN phestecAMZN merged commit c395303 into opensearch-project:main Oct 3, 2022
@opensearch-trigger-bot
Copy link
Contributor

The backport to 2.x failed:

The process '/usr/bin/git' failed with exit code 1

To backport manually, run these commands in your terminal:

# Fetch latest updates from GitHub
git fetch
# Create a new working tree
git worktree add .worktrees/backport-2.x 2.x
# Navigate to the new working tree
cd .worktrees/backport-2.x
# Create a new branch
git switch --create backport/backport-261-to-2.x
# Cherry-pick the merged commit of this pull request and resolve the conflicts
git cherry-pick -x --mainline 1 c395303c2d9451197390579f216a7a6adb517e28
# Push it to GitHub
git push --set-upstream origin backport/backport-261-to-2.x
# Go back to the original working tree
cd ../..
# Delete the working tree
git worktree remove .worktrees/backport-2.x

Then, create a pull request where the base branch is 2.x and the compare/head branch is backport/backport-261-to-2.x.

opensearch-trigger-bot bot pushed a commit that referenced this pull request Oct 3, 2022
* Change alignment of Snapshot Management panels in pages/Main/Main.tsx

Signed-off-by: Chris Hesterman <[email protected]>

* Unify vertical button alignment across panels, ContentPanel.tsx

Signed-off-by: Chris Hesterman <[email protected]>

* Update jest snapshots, add ROUTE_STYLE variable

Signed-off-by: Chris Hesterman <[email protected]>

* Add placeholder restore button to Snapshots panel

Signed-off-by: Chris Hesterman <[email protected]>

* Comment out line 20 rollups_spec.js in cypress/integration/

Signed-off-by: Chris Hesterman <[email protected]>

* Remove unused code and comment cypress/integration/rollups_spec.js

Signed-off-by: Chris Hesterman <[email protected]>

* Set window.opener to null via anchor tag 'rel' attribute-TransformIndices

Signed-off-by: Chris Hesterman <[email protected]>

* Add rel="noopener noreferrer" to external links, update snapshots

Signed-off-by: Chris Hesterman <[email protected]>

* Include version increment to 2.3.0

Signed-off-by: Chris Hesterman <[email protected]>

* Remove 2.3 version bump

Signed-off-by: Chris Hesterman <[email protected]>

Signed-off-by: Chris Hesterman <[email protected]>
(cherry picked from commit c395303)
phestecAMZN added a commit to phestecAMZN/index-management-dashboards-plugin that referenced this pull request Oct 4, 2022
…project#261)

* Change alignment of Snapshot Management panels in pages/Main/Main.tsx

Signed-off-by: Chris Hesterman <[email protected]>

* Unify vertical button alignment across panels, ContentPanel.tsx

Signed-off-by: Chris Hesterman <[email protected]>

* Update jest snapshots, add ROUTE_STYLE variable

Signed-off-by: Chris Hesterman <[email protected]>

* Add placeholder restore button to Snapshots panel

Signed-off-by: Chris Hesterman <[email protected]>

* Comment out line 20 rollups_spec.js in cypress/integration/

Signed-off-by: Chris Hesterman <[email protected]>

* Remove unused code and comment cypress/integration/rollups_spec.js

Signed-off-by: Chris Hesterman <[email protected]>

* Set window.opener to null via anchor tag 'rel' attribute-TransformIndices

Signed-off-by: Chris Hesterman <[email protected]>

* Add rel="noopener noreferrer" to external links, update snapshots

Signed-off-by: Chris Hesterman <[email protected]>

* Include version increment to 2.3.0

Signed-off-by: Chris Hesterman <[email protected]>

* Remove 2.3 version bump

Signed-off-by: Chris Hesterman <[email protected]>

Signed-off-by: Chris Hesterman <[email protected]>
phestecAMZN added a commit to phestecAMZN/index-management-dashboards-plugin that referenced this pull request Oct 4, 2022
…project#261)

* Change alignment of Snapshot Management panels in pages/Main/Main.tsx

Signed-off-by: Chris Hesterman <[email protected]>

* Unify vertical button alignment across panels, ContentPanel.tsx

Signed-off-by: Chris Hesterman <[email protected]>

* Update jest snapshots, add ROUTE_STYLE variable

Signed-off-by: Chris Hesterman <[email protected]>

* Add placeholder restore button to Snapshots panel

Signed-off-by: Chris Hesterman <[email protected]>

* Comment out line 20 rollups_spec.js in cypress/integration/

Signed-off-by: Chris Hesterman <[email protected]>

* Remove unused code and comment cypress/integration/rollups_spec.js

Signed-off-by: Chris Hesterman <[email protected]>

* Set window.opener to null via anchor tag 'rel' attribute-TransformIndices

Signed-off-by: Chris Hesterman <[email protected]>

* Add rel="noopener noreferrer" to external links, update snapshots

Signed-off-by: Chris Hesterman <[email protected]>

* Include version increment to 2.3.0

Signed-off-by: Chris Hesterman <[email protected]>

* Remove 2.3 version bump

Signed-off-by: Chris Hesterman <[email protected]>

Signed-off-by: Chris Hesterman <[email protected]>

(cherry picked from commit c395303)

Signed-off-by: Chris Hesterman <[email protected]>
khushbr pushed a commit that referenced this pull request Oct 4, 2022
* Added release notes for 2.3 (#250)

* Added release notes for 2.3

Signed-off-by: Chris Hesterman <[email protected]>

* Remove commit 221, move 229 to Maintenance, add 250 to Documentation

Signed-off-by: Chris Hesterman <[email protected]>

* Move 230 to Maintenance, 229 back to Bug fixes

Signed-off-by: Chris Hesterman <[email protected]>

* Move 229 back to Maintenance

Signed-off-by: Chris Hesterman <[email protected]>

Signed-off-by: Chris Hesterman <[email protected]>

* Add release notes 2.3, add 's' to 'Enhancement' header -> 'Enhancements' (#257)

* Added release notes for 2.3

Signed-off-by: Chris Hesterman <[email protected]>

* Remove commit 221, move 229 to Maintenance, add 250 to Documentation

Signed-off-by: Chris Hesterman <[email protected]>

* Move 230 to Maintenance, 229 back to Bug fixes

Signed-off-by: Chris Hesterman <[email protected]>

* Move 229 back to Maintenance

Signed-off-by: Chris Hesterman <[email protected]>

* Add 's' to 'Enhancement' header.

Signed-off-by: Chris Hesterman <[email protected]>

Signed-off-by: Chris Hesterman <[email protected]>

* Version increment 2.3.0 (#247)

Signed-off-by: Chris Hesterman <[email protected]>

(cherry picked from commit 2824645)

Signed-off-by: Chris Hesterman <[email protected]>

* Add rel to link for external links, IM dashboards plugin (#261)

* Change alignment of Snapshot Management panels in pages/Main/Main.tsx

Signed-off-by: Chris Hesterman <[email protected]>

* Unify vertical button alignment across panels, ContentPanel.tsx

Signed-off-by: Chris Hesterman <[email protected]>

* Update jest snapshots, add ROUTE_STYLE variable

Signed-off-by: Chris Hesterman <[email protected]>

* Add placeholder restore button to Snapshots panel

Signed-off-by: Chris Hesterman <[email protected]>

* Comment out line 20 rollups_spec.js in cypress/integration/

Signed-off-by: Chris Hesterman <[email protected]>

* Remove unused code and comment cypress/integration/rollups_spec.js

Signed-off-by: Chris Hesterman <[email protected]>

* Set window.opener to null via anchor tag 'rel' attribute-TransformIndices

Signed-off-by: Chris Hesterman <[email protected]>

* Add rel="noopener noreferrer" to external links, update snapshots

Signed-off-by: Chris Hesterman <[email protected]>

* Include version increment to 2.3.0

Signed-off-by: Chris Hesterman <[email protected]>

* Remove 2.3 version bump

Signed-off-by: Chris Hesterman <[email protected]>

Signed-off-by: Chris Hesterman <[email protected]>

* Update cypress-workflow.yml

Signed-off-by: Chris Hesterman <[email protected]>

* Update beforeEach hook, rollups_spec.js

Signed-off-by: Chris Hesterman <[email protected]>

Signed-off-by: Chris Hesterman <[email protected]>
Angie-Zhang pushed a commit that referenced this pull request Oct 4, 2022
* Add rel to link for external links, IM dashboards plugin (#261)

* Change alignment of Snapshot Management panels in pages/Main/Main.tsx

Signed-off-by: Chris Hesterman <[email protected]>

* Unify vertical button alignment across panels, ContentPanel.tsx

Signed-off-by: Chris Hesterman <[email protected]>

* Update jest snapshots, add ROUTE_STYLE variable

Signed-off-by: Chris Hesterman <[email protected]>

* Add placeholder restore button to Snapshots panel

Signed-off-by: Chris Hesterman <[email protected]>

* Comment out line 20 rollups_spec.js in cypress/integration/

Signed-off-by: Chris Hesterman <[email protected]>

* Remove unused code and comment cypress/integration/rollups_spec.js

Signed-off-by: Chris Hesterman <[email protected]>

* Set window.opener to null via anchor tag 'rel' attribute-TransformIndices

Signed-off-by: Chris Hesterman <[email protected]>

* Add rel="noopener noreferrer" to external links, update snapshots

Signed-off-by: Chris Hesterman <[email protected]>

* Include version increment to 2.3.0

Signed-off-by: Chris Hesterman <[email protected]>

* Remove 2.3 version bump

Signed-off-by: Chris Hesterman <[email protected]>

Signed-off-by: Chris Hesterman <[email protected]>

(cherry picked from commit c395303)

Signed-off-by: Chris Hesterman <[email protected]>

* Change version 2.2 -> 2.2.1 opensearch_dashboards.json

Signed-off-by: Chris Hesterman <[email protected]>

Make small change in beforeEach hook, rollups_spec.js

Signed-off-by: Chris Hesterman <[email protected]>

Signed-off-by: Chris Hesterman <[email protected]>
phestecAMZN added a commit that referenced this pull request Oct 19, 2022
* Updated rollup help text. (#220)

Signed-off-by: AWSHurneyt <[email protected]>

* Bumping main to 2.2, address dependabot alert, enable test workflows on all PRs, fix snapshot tests. (#227)

* Bumping 2.x branch from version 2.1 to 2.2. (#218)

* Bumping 2.x branch from version 2.1 to 2.2. Bumped terser version to 4.8.1 to address CVE.

Signed-off-by: AWSHurneyt <[email protected]>

* Drafted `2.2` release notes.

Signed-off-by: AWSHurneyt <[email protected]>

* Drafted `2.2` release notes.

Signed-off-by: AWSHurneyt <[email protected]>

* Amended 2.2 release notes. (#222)

Signed-off-by: AWSHurneyt <[email protected]>

* Adjusted version of OSD used by unit and cypress tests to align with OSD branching strategy. Enabled unit and cypress test workflows on all branches.

Signed-off-by: AWSHurneyt <[email protected]>

* Updated link that was broken when ODFE was deprecated.

Signed-off-by: AWSHurneyt <[email protected]>

* Adjusted path for dependency used by unit tests to align with migration from EUI to OUI.

Signed-off-by: AWSHurneyt <[email protected]>

Signed-off-by: AWSHurneyt <[email protected]>

* Bumped moment version to resolve dependabot alert. (#230)

Signed-off-by: AWSHurneyt <[email protected]>

Signed-off-by: AWSHurneyt <[email protected]>

* Change alignment of Snapshot Management panels in pages/Main/Main.tsx

Signed-off-by: Chris Hesterman <[email protected]>

* Unify vertical button alignment across panels, ContentPanel.tsx

Signed-off-by: Chris Hesterman <[email protected]>

* Update jest snapshots, add ROUTE_STYLE variable

Signed-off-by: Chris Hesterman <[email protected]>

* Add placeholder restore button to Snapshots panel

Signed-off-by: Chris Hesterman <[email protected]>

* Version increment 2.2.1 (#235) (#239) (#240)

Signed-off-by: prudhvigodithi <[email protected]>

Signed-off-by: prudhvigodithi <[email protected]>
Signed-off-by: AWSHurneyt <[email protected]>

Signed-off-by: prudhvigodithi <[email protected]>
Signed-off-by: AWSHurneyt <[email protected]>
Co-authored-by: Prudhvi Godithi <[email protected]>
(cherry picked from commit 2f96369)

Co-authored-by: AWSHurneyt <[email protected]>

* Comment out line 20 rollups_spec.js in cypress/integration/

Signed-off-by: Chris Hesterman <[email protected]>

* Remove unused code and comment cypress/integration/rollups_spec.js

Signed-off-by: Chris Hesterman <[email protected]>

* Change alignment of Snapshot Management panels in pages/Main/Main.tsx (#236)

* Change alignment of Snapshot Management panels in pages/Main/Main.tsx

Signed-off-by: Chris Hesterman <[email protected]>

* Unify vertical button alignment across panels, ContentPanel.tsx

Signed-off-by: Chris Hesterman <[email protected]>

* Update jest snapshots, add ROUTE_STYLE variable

Signed-off-by: Chris Hesterman <[email protected]>

* Add placeholder restore button to Snapshots panel

Signed-off-by: Chris Hesterman <[email protected]>

* Comment out line 20 rollups_spec.js in cypress/integration/

Signed-off-by: Chris Hesterman <[email protected]>

* Remove unused code and comment cypress/integration/rollups_spec.js

Signed-off-by: Chris Hesterman <[email protected]>

Signed-off-by: Chris Hesterman <[email protected]>

* Removed the extra forward slash leading to a non-existing documentation (#231)

page for snapshot management

Signed-off-by: Victor Nilsson <[email protected]>

Signed-off-by: Victor Nilsson <[email protected]>

* Starting adaptation/use of existing code for Restore functionality

Signed-off-by: Chris Hesterman <[email protected]>

* Adapt Snapshots.tsx to include RestoreSnapshotFlyout

Signed-off-by: Chris Hesterman <[email protected]>

* Add restoreSnapshot method to SMservice,RestoreSnapshotResponse int

Signed-off-by: Chris Hesterman <[email protected]>

* Implement RestoreSnapshotFlyout open/close

Signed-off-by: Chris Hesterman <[email protected]>

* Make Snapshot name appear on RestoreSnapshotFlyout

Signed-off-by: Chris Hesterman <[email protected]>

* Add/adapt restoreSnapshot to server/SnapshotManagementService

Signed-off-by: Chris Hesterman <[email protected]>

* Add route for restoreSnapshot in server/routes.ts

Signed-off-by: Chris Hesterman <[email protected]>

* Rudimentary restore from snapshot functionality reached, buggy

Signed-off-by: Chris Hesterman <[email protected]>

* Fix index list-snapshot restore, restrict restore to single selection

Signed-off-by: Chris Hesterman <[email protected]>

* Remove console.logs, commented out code

Signed-off-by: Chris Hesterman <[email protected]>

* Create Advanced options accordion for Snapshot restore.

Signed-off-by: Chris Hesterman <[email protected]>

Currently only visually functional. TODO - make it apply the options

Signed-off-by: Chris Hesterman <[email protected]>

* Add initial restore/rename options to ui. Only visual functionality

Signed-off-by: Chris Hesterman <[email protected]>

* Resolve radio button events not firing

Signed-off-by: Chris Hesterman <[email protected]>

Split radio buttons into SnapshotRestoreOption and SnapshotRenameOptions

Signed-off-by: Chris Hesterman <[email protected]>

Move 'restore' button between 'delete' and 'take snapshot'

Signed-off-by: Chris Hesterman <[email protected]>

Set default rename option to 'add prefix'

Signed-off-by: Chris Hesterman <[email protected]>

* Add AddPrefixInput, remove unused component RestoreSnapshotInitialOptions

Signed-off-by: Chris Hesterman <[email protected]>

* Fix issues with restore specific indices

Signed-off-by: Chris Hesterman <[email protected]>

Implement first 4 Advanced restore options functionality

Signed-off-by: Chris Hesterman <[email protected]>

* Begin cypress testing for snapshots

Signed-off-by: Chris Hesterman <[email protected]>

Passing tests for create repo and create snapshot

Signed-off-by: Chris Hesterman <[email protected]>

* Begin cypress testing Snapshots. Create repo and Create snapshot done

Signed-off-by: Chris Hesterman <[email protected]>

* Implement add_prefix functionality for snapshot restore

Signed-off-by: Chris Hesterman <[email protected]>

* Implement rename indices option for restore snapshot

Signed-off-by: Chris Hesterman <[email protected]>

* Remove console.logs, unusted variables in SnapshotFlyout, SMservice

Signed-off-by: Chris Hesterman <[email protected]>

* Add full stop to help text/errors, add RESTORE_OPTIONS enum

Signed-off-by: Chris Hesterman <[email protected]>

* Version increament 2.3.0

Signed-off-by: Chris Hesterman <[email protected]>

* Update OSD version in cypress-workflow.yml

Signed-off-by: Chris Hesterman <[email protected]>

* Added release notes for 2.3 (#250)

* Added release notes for 2.3

Signed-off-by: Chris Hesterman <[email protected]>

* Remove commit 221, move 229 to Maintenance, add 250 to Documentation

Signed-off-by: Chris Hesterman <[email protected]>

* Move 230 to Maintenance, 229 back to Bug fixes

Signed-off-by: Chris Hesterman <[email protected]>

* Move 229 back to Maintenance

Signed-off-by: Chris Hesterman <[email protected]>

Signed-off-by: Chris Hesterman <[email protected]>

* Add back line 20 in rollups_spec, required by changes in 2.3 release.

Signed-off-by: Chris Hesterman <[email protected]>

* Add release notes 2.3, add 's' to 'Enhancement' header -> 'Enhancements' (#257)

* Added release notes for 2.3

Signed-off-by: Chris Hesterman <[email protected]>

* Remove commit 221, move 229 to Maintenance, add 250 to Documentation

Signed-off-by: Chris Hesterman <[email protected]>

* Move 230 to Maintenance, 229 back to Bug fixes

Signed-off-by: Chris Hesterman <[email protected]>

* Move 229 back to Maintenance

Signed-off-by: Chris Hesterman <[email protected]>

* Add 's' to 'Enhancement' header.

Signed-off-by: Chris Hesterman <[email protected]>

Signed-off-by: Chris Hesterman <[email protected]>

* Remove unused getRepos func, add repo prop, trim state in RestoreFlyout

Signed-off-by: Chris Hesterman <[email protected]>

* Add increment to 2.3.0

Signed-off-by: Chris Hesterman <[email protected]>

* Add rel to link for external links, IM dashboards plugin (#261)

* Change alignment of Snapshot Management panels in pages/Main/Main.tsx

Signed-off-by: Chris Hesterman <[email protected]>

* Unify vertical button alignment across panels, ContentPanel.tsx

Signed-off-by: Chris Hesterman <[email protected]>

* Update jest snapshots, add ROUTE_STYLE variable

Signed-off-by: Chris Hesterman <[email protected]>

* Add placeholder restore button to Snapshots panel

Signed-off-by: Chris Hesterman <[email protected]>

* Comment out line 20 rollups_spec.js in cypress/integration/

Signed-off-by: Chris Hesterman <[email protected]>

* Remove unused code and comment cypress/integration/rollups_spec.js

Signed-off-by: Chris Hesterman <[email protected]>

* Set window.opener to null via anchor tag 'rel' attribute-TransformIndices

Signed-off-by: Chris Hesterman <[email protected]>

* Add rel="noopener noreferrer" to external links, update snapshots

Signed-off-by: Chris Hesterman <[email protected]>

* Include version increment to 2.3.0

Signed-off-by: Chris Hesterman <[email protected]>

* Remove 2.3 version bump

Signed-off-by: Chris Hesterman <[email protected]>

Signed-off-by: Chris Hesterman <[email protected]>

* Version bump 2.3.0 to 2.3 1

Signed-off-by: Chris Hesterman <[email protected]>

* Add version bump 2.3.1 for OS in cypress-workflow.yml

Signed-off-by: Chris Hesterman <[email protected]>

* Backport main 2.3.0 fix sign off (#272)

* Added release notes for 2.3 (#250)

* Added release notes for 2.3

Signed-off-by: Chris Hesterman <[email protected]>

* Remove commit 221, move 229 to Maintenance, add 250 to Documentation

Signed-off-by: Chris Hesterman <[email protected]>

* Move 230 to Maintenance, 229 back to Bug fixes

Signed-off-by: Chris Hesterman <[email protected]>

* Move 229 back to Maintenance

Signed-off-by: Chris Hesterman <[email protected]>

Signed-off-by: Chris Hesterman <[email protected]>

* Add release notes 2.3, add 's' to 'Enhancement' header -> 'Enhancements' (#257)

* Added release notes for 2.3

Signed-off-by: Chris Hesterman <[email protected]>

* Remove commit 221, move 229 to Maintenance, add 250 to Documentation

Signed-off-by: Chris Hesterman <[email protected]>

* Move 230 to Maintenance, 229 back to Bug fixes

Signed-off-by: Chris Hesterman <[email protected]>

* Move 229 back to Maintenance

Signed-off-by: Chris Hesterman <[email protected]>

* Add 's' to 'Enhancement' header.

Signed-off-by: Chris Hesterman <[email protected]>

Signed-off-by: Chris Hesterman <[email protected]>

* Version increment 2.3.0 (#247)

Signed-off-by: Chris Hesterman <[email protected]>

(cherry picked from commit 2824645)

Signed-off-by: Chris Hesterman <[email protected]>

* Add rel to link for external links, IM dashboards plugin (#261)

* Change alignment of Snapshot Management panels in pages/Main/Main.tsx

Signed-off-by: Chris Hesterman <[email protected]>

* Unify vertical button alignment across panels, ContentPanel.tsx

Signed-off-by: Chris Hesterman <[email protected]>

* Update jest snapshots, add ROUTE_STYLE variable

Signed-off-by: Chris Hesterman <[email protected]>

* Add placeholder restore button to Snapshots panel

Signed-off-by: Chris Hesterman <[email protected]>

* Comment out line 20 rollups_spec.js in cypress/integration/

Signed-off-by: Chris Hesterman <[email protected]>

* Remove unused code and comment cypress/integration/rollups_spec.js

Signed-off-by: Chris Hesterman <[email protected]>

* Set window.opener to null via anchor tag 'rel' attribute-TransformIndices

Signed-off-by: Chris Hesterman <[email protected]>

* Add rel="noopener noreferrer" to external links, update snapshots

Signed-off-by: Chris Hesterman <[email protected]>

* Include version increment to 2.3.0

Signed-off-by: Chris Hesterman <[email protected]>

* Remove 2.3 version bump

Signed-off-by: Chris Hesterman <[email protected]>

Signed-off-by: Chris Hesterman <[email protected]>

* Update cypress-workflow.yml

Signed-off-by: Chris Hesterman <[email protected]>

* Update beforeEach hook, rollups_spec.js

Signed-off-by: Chris Hesterman <[email protected]>

Signed-off-by: Chris Hesterman <[email protected]>

* Remove 2.3.1 bump

Signed-off-by: Chris Hesterman <[email protected]>

* Snapshot Restore, basic restore functionality with basic options. Initial PR (#276)

* Change alignment of Snapshot Management panels in pages/Main/Main.tsx

Signed-off-by: Chris Hesterman <[email protected]>

* Unify vertical button alignment across panels, ContentPanel.tsx

Signed-off-by: Chris Hesterman <[email protected]>

* Update jest snapshots, add ROUTE_STYLE variable

Signed-off-by: Chris Hesterman <[email protected]>

* Add placeholder restore button to Snapshots panel

Signed-off-by: Chris Hesterman <[email protected]>

* Comment out line 20 rollups_spec.js in cypress/integration/

Signed-off-by: Chris Hesterman <[email protected]>

* Remove unused code and comment cypress/integration/rollups_spec.js

Signed-off-by: Chris Hesterman <[email protected]>

* Starting adaptation/use of existing code for Restore functionality

Signed-off-by: Chris Hesterman <[email protected]>

* Adapt Snapshots.tsx to include RestoreSnapshotFlyout

Signed-off-by: Chris Hesterman <[email protected]>

* Add restoreSnapshot method to SMservice,RestoreSnapshotResponse int

Signed-off-by: Chris Hesterman <[email protected]>

* Implement RestoreSnapshotFlyout open/close

Signed-off-by: Chris Hesterman <[email protected]>

* Make Snapshot name appear on RestoreSnapshotFlyout

Signed-off-by: Chris Hesterman <[email protected]>

* Add/adapt restoreSnapshot to server/SnapshotManagementService

Signed-off-by: Chris Hesterman <[email protected]>

* Add route for restoreSnapshot in server/routes.ts

Signed-off-by: Chris Hesterman <[email protected]>

* Rudimentary restore from snapshot functionality reached, buggy

Signed-off-by: Chris Hesterman <[email protected]>

* Fix index list-snapshot restore, restrict restore to single selection

Signed-off-by: Chris Hesterman <[email protected]>

* Remove console.logs, commented out code

Signed-off-by: Chris Hesterman <[email protected]>

* Create Advanced options accordion for Snapshot restore.

Signed-off-by: Chris Hesterman <[email protected]>

Currently only visually functional. TODO - make it apply the options

Signed-off-by: Chris Hesterman <[email protected]>

* Add initial restore/rename options to ui. Only visual functionality

Signed-off-by: Chris Hesterman <[email protected]>

* Resolve radio button events not firing

Signed-off-by: Chris Hesterman <[email protected]>

Split radio buttons into SnapshotRestoreOption and SnapshotRenameOptions

Signed-off-by: Chris Hesterman <[email protected]>

Move 'restore' button between 'delete' and 'take snapshot'

Signed-off-by: Chris Hesterman <[email protected]>

Set default rename option to 'add prefix'

Signed-off-by: Chris Hesterman <[email protected]>

* Add AddPrefixInput, remove unused component RestoreSnapshotInitialOptions

Signed-off-by: Chris Hesterman <[email protected]>

* Fix issues with restore specific indices

Signed-off-by: Chris Hesterman <[email protected]>

Implement first 4 Advanced restore options functionality

Signed-off-by: Chris Hesterman <[email protected]>

* Begin cypress testing for snapshots

Signed-off-by: Chris Hesterman <[email protected]>

Passing tests for create repo and create snapshot

Signed-off-by: Chris Hesterman <[email protected]>

* Begin cypress testing Snapshots. Create repo and Create snapshot done

Signed-off-by: Chris Hesterman <[email protected]>

* Implement add_prefix functionality for snapshot restore

Signed-off-by: Chris Hesterman <[email protected]>

* Implement rename indices option for restore snapshot

Signed-off-by: Chris Hesterman <[email protected]>

* Remove console.logs, unusted variables in SnapshotFlyout, SMservice

Signed-off-by: Chris Hesterman <[email protected]>

* Add full stop to help text/errors, add RESTORE_OPTIONS enum

Signed-off-by: Chris Hesterman <[email protected]>

* Add back line 20 in rollups_spec, required by changes in 2.3 release.

Signed-off-by: Chris Hesterman <[email protected]>

* Remove unused getRepos func, add repo prop, trim state in RestoreFlyout

Signed-off-by: Chris Hesterman <[email protected]>

* Add increment to 2.3.0

Signed-off-by: Chris Hesterman <[email protected]>

* Change 2.3 to 2.3.0, cypress-workflow.yml

Signed-off-by: Chris Hesterman <[email protected]>

* Comment out line 20 rollups_spec.js causing error in PR E2E

Signed-off-by: Chris Hesterman <[email protected]>

* Change 2.2 to 2.3 unit-tests-workflow.yml

Signed-off-by: Chris Hesterman <[email protected]>

* Uncomment line 19, rollups_spec.js to attempt E2E success on github

Signed-off-by: Chris Hesterman <[email protected]>

* Fix url, change "Sample" to "sample" rollups_spec lines 17, 21

Signed-off-by: Chris Hesterman <[email protected]>

* Change "sample data" to "Add data" rollops_spec.js

Signed-off-by: Chris Hesterman <[email protected]>

All of these variations pass seamlessly when run locally.

Signed-off-by: Chris Hesterman <[email protected]>

* Add 10s wait line19, rollups_spec to help find button next step

Signed-off-by: Chris Hesterman <[email protected]>

Signed-off-by: Chris Hesterman <[email protected]>

* Version increment 2.4 (#283)

* Change alignment of Snapshot Management panels in pages/Main/Main.tsx

Signed-off-by: Chris Hesterman <[email protected]>

* Unify vertical button alignment across panels, ContentPanel.tsx

Signed-off-by: Chris Hesterman <[email protected]>

* Update jest snapshots, add ROUTE_STYLE variable

Signed-off-by: Chris Hesterman <[email protected]>

* Add placeholder restore button to Snapshots panel

Signed-off-by: Chris Hesterman <[email protected]>

* Comment out line 20 rollups_spec.js in cypress/integration/

Signed-off-by: Chris Hesterman <[email protected]>

* Remove unused code and comment cypress/integration/rollups_spec.js

Signed-off-by: Chris Hesterman <[email protected]>

* Starting adaptation/use of existing code for Restore functionality

Signed-off-by: Chris Hesterman <[email protected]>

* Adapt Snapshots.tsx to include RestoreSnapshotFlyout

Signed-off-by: Chris Hesterman <[email protected]>

* Add restoreSnapshot method to SMservice,RestoreSnapshotResponse int

Signed-off-by: Chris Hesterman <[email protected]>

* Implement RestoreSnapshotFlyout open/close

Signed-off-by: Chris Hesterman <[email protected]>

* Make Snapshot name appear on RestoreSnapshotFlyout

Signed-off-by: Chris Hesterman <[email protected]>

* Add/adapt restoreSnapshot to server/SnapshotManagementService

Signed-off-by: Chris Hesterman <[email protected]>

* Add route for restoreSnapshot in server/routes.ts

Signed-off-by: Chris Hesterman <[email protected]>

* Rudimentary restore from snapshot functionality reached, buggy

Signed-off-by: Chris Hesterman <[email protected]>

* Fix index list-snapshot restore, restrict restore to single selection

Signed-off-by: Chris Hesterman <[email protected]>

* Remove console.logs, commented out code

Signed-off-by: Chris Hesterman <[email protected]>

* Create Advanced options accordion for Snapshot restore.

Signed-off-by: Chris Hesterman <[email protected]>

Currently only visually functional. TODO - make it apply the options

Signed-off-by: Chris Hesterman <[email protected]>

* Add initial restore/rename options to ui. Only visual functionality

Signed-off-by: Chris Hesterman <[email protected]>

* Resolve radio button events not firing

Signed-off-by: Chris Hesterman <[email protected]>

Split radio buttons into SnapshotRestoreOption and SnapshotRenameOptions

Signed-off-by: Chris Hesterman <[email protected]>

Move 'restore' button between 'delete' and 'take snapshot'

Signed-off-by: Chris Hesterman <[email protected]>

Set default rename option to 'add prefix'

Signed-off-by: Chris Hesterman <[email protected]>

* Add AddPrefixInput, remove unused component RestoreSnapshotInitialOptions

Signed-off-by: Chris Hesterman <[email protected]>

* Fix issues with restore specific indices

Signed-off-by: Chris Hesterman <[email protected]>

Implement first 4 Advanced restore options functionality

Signed-off-by: Chris Hesterman <[email protected]>

* Begin cypress testing for snapshots

Signed-off-by: Chris Hesterman <[email protected]>

Passing tests for create repo and create snapshot

Signed-off-by: Chris Hesterman <[email protected]>

* Begin cypress testing Snapshots. Create repo and Create snapshot done

Signed-off-by: Chris Hesterman <[email protected]>

* Implement add_prefix functionality for snapshot restore

Signed-off-by: Chris Hesterman <[email protected]>

* Implement rename indices option for restore snapshot

Signed-off-by: Chris Hesterman <[email protected]>

* Remove console.logs, unusted variables in SnapshotFlyout, SMservice

Signed-off-by: Chris Hesterman <[email protected]>

* Add full stop to help text/errors, add RESTORE_OPTIONS enum

Signed-off-by: Chris Hesterman <[email protected]>

* Add back line 20 in rollups_spec, required by changes in 2.3 release.

Signed-off-by: Chris Hesterman <[email protected]>

* Remove unused getRepos func, add repo prop, trim state in RestoreFlyout

Signed-off-by: Chris Hesterman <[email protected]>

* Add increment to 2.3.0

Signed-off-by: Chris Hesterman <[email protected]>

* Version bump 2.3.0 to 2.3 1

Signed-off-by: Chris Hesterman <[email protected]>

* Add version bump 2.3.1 for OS in cypress-workflow.yml

Signed-off-by: Chris Hesterman <[email protected]>

* Remove 2.3.1 bump

Signed-off-by: Chris Hesterman <[email protected]>

* Version increment 2.4

Signed-off-by: Chris Hesterman <[email protected]>

* Remove commented code, rollups_spec

Signed-off-by: Chris Hesterman <[email protected]>

* Change 2.4 back to 2.3, unit-tests-workflow

Signed-off-by: Chris Hesterman <[email protected]>

* Change Opensearch Dashboards version to 2.x in 3 files

Signed-off-by: Chris Hesterman <[email protected]>

* Change OSD version from 2.x to 2.4 in response to PR error

Signed-off-by: Chris Hesterman <[email protected]>

* Update dashboards version 2.x -> 2.4, cypress, unit test workflows.

Signed-off-by: Chris Hesterman <[email protected]>

* Change 2.4 -> 2.x, cypress and unit test workflows.

Signed-off-by: Chris Hesterman <[email protected]>

Signed-off-by: Chris Hesterman <[email protected]>

* Change 2.4 to 2.4.0 opensearch_dashboards.json

Signed-off-by: Chris Hesterman <[email protected]>

* Add/update tests, update help text links and icons, add Restore activities panel (#277)

* Change alignment of Snapshot Management panels in pages/Main/Main.tsx

Signed-off-by: Chris Hesterman <[email protected]>

* Unify vertical button alignment across panels, ContentPanel.tsx

Signed-off-by: Chris Hesterman <[email protected]>

* Update jest snapshots, add ROUTE_STYLE variable

Signed-off-by: Chris Hesterman <[email protected]>

* Add placeholder restore button to Snapshots panel

Signed-off-by: Chris Hesterman <[email protected]>

* Comment out line 20 rollups_spec.js in cypress/integration/

Signed-off-by: Chris Hesterman <[email protected]>

* Remove unused code and comment cypress/integration/rollups_spec.js

Signed-off-by: Chris Hesterman <[email protected]>

* Starting adaptation/use of existing code for Restore functionality

Signed-off-by: Chris Hesterman <[email protected]>

* Adapt Snapshots.tsx to include RestoreSnapshotFlyout

Signed-off-by: Chris Hesterman <[email protected]>

* Add restoreSnapshot method to SMservice,RestoreSnapshotResponse int

Signed-off-by: Chris Hesterman <[email protected]>

* Implement RestoreSnapshotFlyout open/close

Signed-off-by: Chris Hesterman <[email protected]>

* Make Snapshot name appear on RestoreSnapshotFlyout

Signed-off-by: Chris Hesterman <[email protected]>

* Add/adapt restoreSnapshot to server/SnapshotManagementService

Signed-off-by: Chris Hesterman <[email protected]>

* Add route for restoreSnapshot in server/routes.ts

Signed-off-by: Chris Hesterman <[email protected]>

* Rudimentary restore from snapshot functionality reached, buggy

Signed-off-by: Chris Hesterman <[email protected]>

* Fix index list-snapshot restore, restrict restore to single selection

Signed-off-by: Chris Hesterman <[email protected]>

* Remove console.logs, commented out code

Signed-off-by: Chris Hesterman <[email protected]>

* Create Advanced options accordion for Snapshot restore.

Signed-off-by: Chris Hesterman <[email protected]>

Currently only visually functional. TODO - make it apply the options

Signed-off-by: Chris Hesterman <[email protected]>

* Add initial restore/rename options to ui. Only visual functionality

Signed-off-by: Chris Hesterman <[email protected]>

* Resolve radio button events not firing

Signed-off-by: Chris Hesterman <[email protected]>

Split radio buttons into SnapshotRestoreOption and SnapshotRenameOptions

Signed-off-by: Chris Hesterman <[email protected]>

Move 'restore' button between 'delete' and 'take snapshot'

Signed-off-by: Chris Hesterman <[email protected]>

Set default rename option to 'add prefix'

Signed-off-by: Chris Hesterman <[email protected]>

* Add AddPrefixInput, remove unused component RestoreSnapshotInitialOptions

Signed-off-by: Chris Hesterman <[email protected]>

* Fix issues with restore specific indices

Signed-off-by: Chris Hesterman <[email protected]>

Implement first 4 Advanced restore options functionality

Signed-off-by: Chris Hesterman <[email protected]>

* Begin cypress testing for snapshots

Signed-off-by: Chris Hesterman <[email protected]>

Passing tests for create repo and create snapshot

Signed-off-by: Chris Hesterman <[email protected]>

* Begin cypress testing Snapshots. Create repo and Create snapshot done

Signed-off-by: Chris Hesterman <[email protected]>

* Implement add_prefix functionality for snapshot restore

Signed-off-by: Chris Hesterman <[email protected]>

* Implement rename indices option for restore snapshot

Signed-off-by: Chris Hesterman <[email protected]>

* Remove console.logs, unusted variables in SnapshotFlyout, SMservice

Signed-off-by: Chris Hesterman <[email protected]>

* Add full stop to help text/errors, add RESTORE_OPTIONS enum

Signed-off-by: Chris Hesterman <[email protected]>

* Progress on custom index settings, ignore index settings

Signed-off-by: Chris Hesterman <[email protected]>

* Implement custom index settings functionality

Signed-off-by: Chris Hesterman <[email protected]>

* Successfully implement/debug custom index settings/ignore index settings.

Signed-off-by: Chris Hesterman <[email protected]>

* Add back line 20 in rollups_spec, required by changes in 2.3 release.

Signed-off-by: Chris Hesterman <[email protected]>

* Fix bug when both custom settings and ignore settings chosen

Signed-off-by: Chris Hesterman <[email protected]>

* Remove unused getRepos func, add repo prop, trim state in RestoreFlyout

Signed-off-by: Chris Hesterman <[email protected]>

* Add increment to 2.3.0

Signed-off-by: Chris Hesterman <[email protected]>

* Remove index settings from options if none entered, RestoreFlyout

Signed-off-by: Chris Hesterman <[email protected]>

* Add rel="noopener noreferrer" to links in IndexList, add CatSnapshotIndex interface

Signed-off-by: Chris Hesterman <[email protected]>

* Current progress, cypress testing

Signed-off-by: Chris Hesterman <[email protected]>

* Add unit testing for AddPrefixInput, RenameInput components

Signed-off-by: Chris Hesterman <[email protected]>

* Add unit tests for SnapshotRenameOptions component

Signed-off-by: Chris Hesterman <[email protected]>

* Add unit tests for SnapshotRestoreOption component

Signed-off-by: Chris Hesterman <[email protected]>

* Add unit tests for SnapshotIndicesInput component

Signed-off-by: Chris Hesterman <[email protected]>

* Change anchor tags to EuiLinks, IndexSettings

Signed-off-by: Chris Hesterman <[email protected]>

* Add rel attribute to EuiLink, IndexSettingsInput

Signed-off-by: Chris Hesterman <[email protected]>

* Version bump 2.3.0 to 2.3 1

Signed-off-by: Chris Hesterman <[email protected]>

* Add version bump 2.3.1 for OS in cypress-workflow.yml

Signed-off-by: Chris Hesterman <[email protected]>

* Remove 2.3.1 bump

Signed-off-by: Chris Hesterman <[email protected]>

* Small mods to rollups_spec and snapshots_spec

Signed-off-by: Chris Hesterman <[email protected]>

* Change snapshots_spec in effort to pass E2E. Passes locally

Signed-off-by: Chris Hesterman <[email protected]>

* Fix spelling of 'index' line67, snapshots_spec

Signed-off-by: Chris Hesterman <[email protected]>

* Version bump 2.4

Signed-off-by: Chris Hesterman <[email protected]>

* Make sure 2.4 applied, add TODO in rollups_spec

Signed-off-by: Chris Hesterman <[email protected]>

* Change 2.4 back to 2.x again cypress, unit_test workflows

Signed-off-by: Chris Hesterman <[email protected]>

Remove one more 'wait' from snapshots_spec

Signed-off-by: Chris Hesterman <[email protected]>

* Add "cy.wait" back line 56 snapshots_spec

Signed-off-by: Chris Hesterman <[email protected]>

* Add '0' to OSD version,  opensearch_dashboards.json.

Signed-off-by: Chris Hesterman <[email protected]>

Remove redundant 'cleanup()' from AddPrefixInput.jest.js

Signed-off-by: Chris Hesterman <[email protected]>

Signed-off-by: Chris Hesterman <[email protected]>

Signed-off-by: AWSHurneyt <[email protected]>
Signed-off-by: Chris Hesterman <[email protected]>
Signed-off-by: Victor Nilsson <[email protected]>
Co-authored-by: AWSHurneyt <[email protected]>
Co-authored-by: opensearch-trigger-bot[bot] <98922864+opensearch-trigger-bot[bot]@users.noreply.github.com>
Co-authored-by: Victor Nilsson <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants