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

Remove integtest.sh since it is not being used #849

Merged
merged 1 commit into from
Dec 14, 2023

Conversation

derek-ho
Copy link
Contributor

@derek-ho derek-ho commented Dec 13, 2023

Description

opensearch-project/opensearch-build#497

Issues Resolved

[List any issues this PR will resolve]

Check List

  • New functionality includes testing.
    • All tests pass
  • New functionality has been documented.
    • New functionality has javadoc added
  • 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.

@derek-ho
Copy link
Contributor Author

This should also be backported to 2.x

Copy link

codecov bot commented Dec 14, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (83a2b3e) 49.74% compared to head (e5d68ff) 49.74%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #849   +/-   ##
=======================================
  Coverage   49.74%   49.74%           
=======================================
  Files         232      232           
  Lines        6568     6568           
  Branches      940      940           
=======================================
  Hits         3267     3267           
  Misses       3298     3298           
  Partials        3        3           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@AWSHurneyt AWSHurneyt merged commit 9abddfa into opensearch-project:main Dec 14, 2023
11 of 12 checks passed
opensearch-trigger-bot bot pushed a commit that referenced this pull request Dec 14, 2023
Signed-off-by: Derek Ho <[email protected]>
(cherry picked from commit 9abddfa)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
bowenlan-amzn pushed a commit that referenced this pull request Dec 14, 2023
(cherry picked from commit 9abddfa)

Signed-off-by: Derek Ho <[email protected]>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
AWSHurneyt pushed a commit to AWSHurneyt/alerting-dashboards-plugin that referenced this pull request Feb 5, 2024
AWSHurneyt pushed a commit to AWSHurneyt/alerting-dashboards-plugin that referenced this pull request Feb 5, 2024
amsiglan added a commit that referenced this pull request Feb 6, 2024
* Support any channel types from Notification (#743)

* Support any channel types from Notification

Signed-off-by: Ashish Agrawal <[email protected]>

* Remove unused CHANNEL_TYPES constant

Signed-off-by: Ashish Agrawal <[email protected]>

* Return empty array if failed to get features

Signed-off-by: Ashish Agrawal <[email protected]>

---------

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

* Drafted 2.11 release notes. (#764)

* Drafted 2.11 release notes.

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

* Drafted 2.11 release notes.

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

---------

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

* Removed "last updated by" sections from the UI. (#767)

* Removed "last updated by" section from the UI as the SearchMonitor API can no longer return that info.

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

* Updated cypress workflow.

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

---------

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

* Onboard Jenkins prod docker image to github actions (#789)

* Onboard Jenkins prod docker image to github actions

Signed-off-by: Peter Zhu <[email protected]>

* Small typos

Signed-off-by: Peter Zhu <[email protected]>

* Add back workflows

Signed-off-by: Peter Zhu <[email protected]>

* Restore macos/windows

Signed-off-by: Peter Zhu <[email protected]>

---------

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

* Bumped babel version. (#821)

* Bumped babel version. Adjusted babel config. Refactored unit test that started failing after bump.

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

* Updated yarn file.

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

---------

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

* Fix fetching of channels for composite monitors (#820)

* fixed incorrect use of this.props

Signed-off-by: Amardeepsingh Siglani <[email protected]>

* resolved dependency conflict

Signed-off-by: Amardeepsingh Siglani <[email protected]>

* updated babel config

Signed-off-by: Amardeepsingh Siglani <[email protected]>

---------

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

* Added 2.11.1 release notes. (#828)

* Amended 2.11.0 release notes.

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

* Added 2.11.1 release notes.

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

* Added 2.11.1 release notes.

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

* Added 2.11.1 release notes.

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

* Added 2.11.1 release notes.

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

---------

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

* Fixed bucket monitor groupBy/aggregation display bug. (#827)

* Fixed a bug that was causing groupBy/aggregation fields from displaying in various areas of the UI. Related issues: 816, 817, 818.

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

* Fixed trigger context object bug in issue 791.

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

* Capitalized bucket column titles, and moved bucket columns to the end of the column array.

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

* Added wait steps to reduce test flakiness.

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

* Added wait step to reduce test flakiness. Adjusted test monitor trigger condition to always triggers on a healthy clusters.

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

* Removed unused imports.

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

* fixed bucket level monitor flaky cypress test

Signed-off-by: Amardeepsingh Siglani <[email protected]>

---------

Signed-off-by: AWSHurneyt <[email protected]>
Signed-off-by: Amardeepsingh Siglani <[email protected]>
Co-authored-by: Amardeepsingh Siglani <[email protected]>
Signed-off-by: AWSHurneyt <[email protected]>

* Issue #671 fix trigger name validation (#794)


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

* Remove integtest.sh since it is not being used (#849)

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

* do not create Message component on every text change (#854)

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

* Implemented server API call to feature backend API.

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

* Implemented remote cluster support for creating/editing query, bucket, and cluster metrics monitors.

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

* Implemented warning model when monitor execution time exceeds a certain value.

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

* Updated alert details flyout to show remote cluster info. Updated monitor details page to show data sources.

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

* Updated unit tests.

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

* Added experimental banner.

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

* Updated snapshots.

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

* Edited text on the experimental banner.

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

* Moved getSettings call to hide Data source panel for cluster metrics monitors when remote monitoring is disabled.

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

* Updated snapshots.

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

* Increased cypress test timeout.

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

---------

Signed-off-by: Ashish Agrawal <[email protected]>
Signed-off-by: AWSHurneyt <[email protected]>
Signed-off-by: Peter Zhu <[email protected]>
Signed-off-by: Amardeepsingh Siglani <[email protected]>
Signed-off-by: Chenxi Wang <[email protected]>
Signed-off-by: Derek Ho <[email protected]>
Co-authored-by: Ashish Agrawal <[email protected]>
Co-authored-by: Peter Zhu <[email protected]>
Co-authored-by: Amardeepsingh Siglani <[email protected]>
Co-authored-by: Chenxi Wang <[email protected]>
Co-authored-by: Derek Ho <[email protected]>
opensearch-trigger-bot bot pushed a commit that referenced this pull request Feb 6, 2024
* Support any channel types from Notification (#743)

* Support any channel types from Notification

Signed-off-by: Ashish Agrawal <[email protected]>

* Remove unused CHANNEL_TYPES constant

Signed-off-by: Ashish Agrawal <[email protected]>

* Return empty array if failed to get features

Signed-off-by: Ashish Agrawal <[email protected]>

---------

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

* Drafted 2.11 release notes. (#764)

* Drafted 2.11 release notes.

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

* Drafted 2.11 release notes.

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

---------

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

* Removed "last updated by" sections from the UI. (#767)

* Removed "last updated by" section from the UI as the SearchMonitor API can no longer return that info.

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

* Updated cypress workflow.

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

---------

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

* Onboard Jenkins prod docker image to github actions (#789)

* Onboard Jenkins prod docker image to github actions

Signed-off-by: Peter Zhu <[email protected]>

* Small typos

Signed-off-by: Peter Zhu <[email protected]>

* Add back workflows

Signed-off-by: Peter Zhu <[email protected]>

* Restore macos/windows

Signed-off-by: Peter Zhu <[email protected]>

---------

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

* Bumped babel version. (#821)

* Bumped babel version. Adjusted babel config. Refactored unit test that started failing after bump.

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

* Updated yarn file.

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

---------

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

* Fix fetching of channels for composite monitors (#820)

* fixed incorrect use of this.props

Signed-off-by: Amardeepsingh Siglani <[email protected]>

* resolved dependency conflict

Signed-off-by: Amardeepsingh Siglani <[email protected]>

* updated babel config

Signed-off-by: Amardeepsingh Siglani <[email protected]>

---------

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

* Added 2.11.1 release notes. (#828)

* Amended 2.11.0 release notes.

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

* Added 2.11.1 release notes.

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

* Added 2.11.1 release notes.

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

* Added 2.11.1 release notes.

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

* Added 2.11.1 release notes.

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

---------

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

* Fixed bucket monitor groupBy/aggregation display bug. (#827)

* Fixed a bug that was causing groupBy/aggregation fields from displaying in various areas of the UI. Related issues: 816, 817, 818.

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

* Fixed trigger context object bug in issue 791.

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

* Capitalized bucket column titles, and moved bucket columns to the end of the column array.

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

* Added wait steps to reduce test flakiness.

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

* Added wait step to reduce test flakiness. Adjusted test monitor trigger condition to always triggers on a healthy clusters.

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

* Removed unused imports.

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

* fixed bucket level monitor flaky cypress test

Signed-off-by: Amardeepsingh Siglani <[email protected]>

---------

Signed-off-by: AWSHurneyt <[email protected]>
Signed-off-by: Amardeepsingh Siglani <[email protected]>
Co-authored-by: Amardeepsingh Siglani <[email protected]>
Signed-off-by: AWSHurneyt <[email protected]>

* Issue #671 fix trigger name validation (#794)

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

* Remove integtest.sh since it is not being used (#849)

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

* do not create Message component on every text change (#854)

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

* Implemented server API call to feature backend API.

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

* Implemented remote cluster support for creating/editing query, bucket, and cluster metrics monitors.

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

* Implemented warning model when monitor execution time exceeds a certain value.

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

* Updated alert details flyout to show remote cluster info. Updated monitor details page to show data sources.

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

* Updated unit tests.

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

* Added experimental banner.

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

* Updated snapshots.

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

* Edited text on the experimental banner.

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

* Moved getSettings call to hide Data source panel for cluster metrics monitors when remote monitoring is disabled.

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

* Updated snapshots.

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

* Increased cypress test timeout.

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

---------

Signed-off-by: Ashish Agrawal <[email protected]>
Signed-off-by: AWSHurneyt <[email protected]>
Signed-off-by: Peter Zhu <[email protected]>
Signed-off-by: Amardeepsingh Siglani <[email protected]>
Signed-off-by: Chenxi Wang <[email protected]>
Signed-off-by: Derek Ho <[email protected]>
Co-authored-by: Ashish Agrawal <[email protected]>
Co-authored-by: Peter Zhu <[email protected]>
Co-authored-by: Amardeepsingh Siglani <[email protected]>
Co-authored-by: Chenxi Wang <[email protected]>
Co-authored-by: Derek Ho <[email protected]>
(cherry picked from commit fb82368)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
AWSHurneyt pushed a commit that referenced this pull request Feb 6, 2024
* Support any channel types from Notification (#743)

* Support any channel types from Notification



* Remove unused CHANNEL_TYPES constant



* Return empty array if failed to get features



---------




* Drafted 2.11 release notes. (#764)

* Drafted 2.11 release notes.



* Drafted 2.11 release notes.



---------



* Removed "last updated by" sections from the UI. (#767)

* Removed "last updated by" section from the UI as the SearchMonitor API can no longer return that info.



* Updated cypress workflow.



---------



* Onboard Jenkins prod docker image to github actions (#789)

* Onboard Jenkins prod docker image to github actions



* Small typos



* Add back workflows



* Restore macos/windows



---------




* Bumped babel version. (#821)

* Bumped babel version. Adjusted babel config. Refactored unit test that started failing after bump.



* Updated yarn file.



---------



* Fix fetching of channels for composite monitors (#820)

* fixed incorrect use of this.props



* resolved dependency conflict



* updated babel config



---------




* Added 2.11.1 release notes. (#828)

* Amended 2.11.0 release notes.



* Added 2.11.1 release notes.



* Added 2.11.1 release notes.



* Added 2.11.1 release notes.



* Added 2.11.1 release notes.



---------



* Fixed bucket monitor groupBy/aggregation display bug. (#827)

* Fixed a bug that was causing groupBy/aggregation fields from displaying in various areas of the UI. Related issues: 816, 817, 818.



* Fixed trigger context object bug in issue 791.



* Capitalized bucket column titles, and moved bucket columns to the end of the column array.



* Added wait steps to reduce test flakiness.



* Added wait step to reduce test flakiness. Adjusted test monitor trigger condition to always triggers on a healthy clusters.



* Removed unused imports.



* fixed bucket level monitor flaky cypress test



---------






* Issue #671 fix trigger name validation (#794)




* Remove integtest.sh since it is not being used (#849)




* do not create Message component on every text change (#854)




* Implemented server API call to feature backend API.



* Implemented remote cluster support for creating/editing query, bucket, and cluster metrics monitors.



* Implemented warning model when monitor execution time exceeds a certain value.



* Updated alert details flyout to show remote cluster info. Updated monitor details page to show data sources.



* Updated unit tests.



* Added experimental banner.



* Updated snapshots.



* Edited text on the experimental banner.



* Moved getSettings call to hide Data source panel for cluster metrics monitors when remote monitoring is disabled.



* Updated snapshots.



* Increased cypress test timeout.



---------












(cherry picked from commit fb82368)

Signed-off-by: Ashish Agrawal <[email protected]>
Signed-off-by: AWSHurneyt <[email protected]>
Signed-off-by: Peter Zhu <[email protected]>
Signed-off-by: Amardeepsingh Siglani <[email protected]>
Signed-off-by: Chenxi Wang <[email protected]>
Signed-off-by: Derek Ho <[email protected]>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: Ashish Agrawal <[email protected]>
Co-authored-by: Peter Zhu <[email protected]>
Co-authored-by: Amardeepsingh Siglani <[email protected]>
Co-authored-by: Chenxi Wang <[email protected]>
Co-authored-by: Derek Ho <[email protected]>
smuthukaruppannp pushed a commit to smuthukaruppannp/alerting-dashboards-plugin that referenced this pull request Feb 7, 2024
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