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 automatic labeling section with template examples, and a github workflow to add 'untriaged' label on new issues #111

Merged
merged 3 commits into from
Jan 11, 2023

Conversation

peternied
Copy link
Member

@peternied peternied commented Dec 1, 2022

Description

There has been discussion around how to be sure that issues are properly being marked and triaged. To remove the human element from reviewing issues and adding the 'untriaged' label it it was missed I've added a github workflow to add 'untriaged' label during issue lifecycle event

Testing

On my fork of this repo I enabled issues and created a couple and they were automatically tagged on open [1] and reopen [2] by the github action bot for various scenarios [2].

[1] peternied#4
[2] peternied#6
[3] https://github.com/peternied/.github/actions

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.

@peternied
Copy link
Member Author

@dblock what do you think about about having a campaign or bulk-cut-prs for this workflow in all the repositories in opensearch-project so we have a consistent 'untriaged' label usage?

@dblock
Copy link
Member

dblock commented Dec 1, 2022

I think I'd want something org-wide like we do for DCO, maybe an app?

joshuarrrr
joshuarrrr previously approved these changes Dec 1, 2022
Copy link
Member

@joshuarrrr joshuarrrr left a comment

Choose a reason for hiding this comment

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

Thanks!

@joshuarrrr
Copy link
Member

Do workflows defined here not get applied org-wide?

@dblock
Copy link
Member

dblock commented Dec 1, 2022

Do workflows defined here not get applied org-wide?

They do not, but I believe apps can be enabled for the org. cc: @VachaShah who did DCO I think

@peternied
Copy link
Member Author

@joshuarrrr @dblock Really good thoughts, this workflow is easy to author and not too difficult to deploy via meta to all the projects in the org; however, it is a bad maintenance experience compared with a GitHub App.

It would be worth building off of the backporting app we made to instead create a workflow management system for the org that could also ensure the backporting, untriaged labeling or other practices are available and up to date. Created an issue to track this notion.

@saratvemulapalli
Copy link
Member

Thinking about this, feels like Github app makes a lot more sense longer term.

  • Its enabled by default on all repos (no more cutting issues to new repos)
  • Maintenance is in one place, instead of the same workflow being replicated across repos.

peternied added a commit to peternied/security that referenced this pull request Dec 5, 2022
Sometimes when new issues are opened, they don't use the template or are
transfered in from another repo, or even are reopened when we ran triage
slightly differently.  This automation adds the `untriaged` tag if any
of these states are applied.

Coming from
opensearch-project/.github#111

Signed-off-by: Peter Nied <[email protected]>
@peternied peternied marked this pull request as draft December 5, 2022 18:58
@peternied
Copy link
Member Author

peternied commented Dec 5, 2022

Switching to draft - feels like there is missing documentation calling this workflow out as an option to adopt, rather than a requirement of the project. We can then determine if this pull request is worth merging on those merits separately from the 'uber' solution that would be compulsory for the project.

peternied added a commit to opensearch-project/security that referenced this pull request Dec 6, 2022
Sometimes when new issues are opened, they don't use the template or are
transfered in from another repo, or even are reopened when we ran triage
slightly differently.  This automation adds the `untriaged` tag if any
of these states are applied.

Coming from
opensearch-project/.github#111

Signed-off-by: Peter Nied <[email protected]>
joshuarrrr added a commit to joshuarrrr/OpenSearch-Dashboards that referenced this pull request Dec 13, 2022
@ashwin-pc
Copy link
Member

@peternied so is the goal now to make this a github app as opposed to a workflow? I like the idea but wanted clarity since we have an open PR in OSD to implement this workflow. opensearch-project/OpenSearch-Dashboards#3073

@dblock
Copy link
Member

dblock commented Dec 15, 2022

@ashwin-pc Yes, I propose closing that PR. @peternied any objections?

ananzh pushed a commit to opensearch-project/OpenSearch-Dashboards that referenced this pull request Dec 17, 2022
@peternied
Copy link
Member Author

peternied commented Jan 3, 2023

Revising this PR - I think we should have a workflow in this repository to add 'untriaged' onto new issues that are created. But I don't think its the right tool for all repositories. I've updated the responsibilities section where using this GitHub action is referenced but not required, what do y'all think?

@peternied peternied changed the title Add github workflow to add 'untriaged' label during issue lifecycle event Add automatic labeling section with template examples, and a github workflow to add 'untriaged' label on new issues Jan 3, 2023
@ashwin-pc
Copy link
Member

We have implemented this workflow for now in the OSD repo but I agree with the suggestions asking for this to be an App so that it is easier to maintain and keep in sync. @peternied have you explored what it would take to convert this workflow that you have into an app?

@peternied
Copy link
Member Author

have you explored what it would take to convert this workflow

While converting the workflow is not difficult, it could be done with a relatively small CDK project either build from the ground up or to run GitHub Organization Manager. The investment is in getting our organizations release processes.

I'd be happy to cede this pull request if someone would like to assign issue [1] to themselves. I would rather have this PR merged and a tool that might replace it in the future.

Copy link
Member

@dblock dblock left a comment

Choose a reason for hiding this comment

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

I'm good with merging this as is since nobody is taking on looking for an org-wide tool. Fix the slashes below and I'll hit merge.

cc: @bbarani, maybe your team can add this to the backlog

RESPONSIBILITIES.md Outdated Show resolved Hide resolved
Signed-off-by: Peter Nied <[email protected]>
Signed-off-by: Peter Nied <[email protected]>
@peternied
Copy link
Member Author

@saratvemulapalli @mch2 @CEHENKLE What do you think about merging this change?

@peternied peternied merged commit 51fb6fa into opensearch-project:main Jan 11, 2023
Arpit-Bandejiya pushed a commit to Arpit-Bandejiya/OpenSearch-Dashboards that referenced this pull request Jan 13, 2023
…-project#3073)

Credit to Peter Nied <[email protected]> via opensearch-project/.github#111

Signed-off-by: Josh Romero <[email protected]>

Signed-off-by: Josh Romero <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>
Arpit-Bandejiya pushed a commit to Arpit-Bandejiya/OpenSearch-Dashboards that referenced this pull request Mar 8, 2023
…-project#3073)

Credit to Peter Nied <[email protected]> via opensearch-project/.github#111

Signed-off-by: Josh Romero <[email protected]>

Signed-off-by: Josh Romero <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>
ananzh added a commit to opensearch-project/OpenSearch-Dashboards that referenced this pull request Mar 8, 2023
* Fixes folder timestamps being updated before copying files into them (#2964)

* When updating the timestamps were requested, scanCopy first set the times and then copied content into the destination folder. On certain platforms, copying files into a folder updates its "Last Access Time" and that overwrites the just set timestamps. This PR, makes sure the timestamps are set only after copying the content.

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

Signed-off-by: Miki <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* Fixes incorrect validation of time values in JUnit Reporter (#2965)

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

Signed-off-by: Miki <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* [Windows] Make `yarn pkg-version` work across platforms (#2963)

* Makes `yarn pkg-version` work across platforms

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

* Makes `yarn pkg-version` work across platforms

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

Signed-off-by: Miki <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* Add Windows CI to the workflow (#2966)

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

Signed-off-by: Miki <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* Makes tests covering plugin installation on cluster snapshots work across platforms (#2994)

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

Signed-off-by: Miki <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* Bump loader-utils from 2.0.3 to 2.0.4 (#2892)

Bumps [loader-utils](https://github.com/webpack/loader-utils) from 2.0.3 to 2.0.4.
- [Release notes](https://github.com/webpack/loader-utils/releases)
- [Changelog](https://github.com/webpack/loader-utils/blob/v2.0.4/CHANGELOG.md)
- [Commits](https://github.com/webpack/loader-utils/compare/v2.0.3...v2.0.4)

---
updated-dependencies:
- dependency-name: loader-utils
  dependency-type: direct:development
...

Signed-off-by: dependabot[bot] <[email protected]>

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Signed-off-by: Arpit Bandejiya <[email protected]>

* Corrects the usage of leaflet-vega parameters (#3005)

Fixes #2996, #2353

- Bump `leaflet-vega` to `^0.9.0` - they merged in our upstream PR and made a release
- Fix the parameter name for passing `vega.parse` options.

Signed-off-by: Miki <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* Add @manasvinibs as maintainer (#3006)

Basic data points
[OSD] 16 submitted PRs (https://github.com/opensearch-project/OpenSearch-Dashboards/pulls/manasvinibs)
[OSD] 74 reviewed PRs (https://github.com/opensearch-project/OpenSearch-Dashboards/issues?q=reviewed-by%3Amanasvinibs)
[OSD] 28 issues involved (https://github.com/opensearch-project/OpenSearch-Dashboards/issues?page=1&q=involves%3Amanasvinibs+is%3Aissue)

Highlight
Mana is assisting with extensions project which will be the next evolution of extending core functionality from OpenSearch Dashboards
Mana implemented https://github.com/opensearch-project/OpenSearch-Dashboards/pull/2734 which allows for a huge quality of life for local development for external plugin developers to utilize snapshots with a single CLI command compared to before when they would had to pull down OpenSearch build, install their plugin on OpenSearch, and ensure the proper configurations. This has caused historically problems when plugin teams do development and miss some steps per their onboard documentation/PR suggestion and get different results than expected.
Mana has assisted reviewing PRs providing great insight on BWC tests, BWC in general, and the release process.
Mana has added documentation from insight she has gained within the informal dev doc repo https://cptnb.github.io/opensearch-dashboards-dev-docs/ ensuring the spread of knowledge.

Signed-off-by: Kawika Avilla <[email protected]>

Signed-off-by: Kawika Avilla <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* [Vis Builder] Add global data persistence for vis builder (#2896)

* Add global data persistence for vis builder

Persist filters, time range, time refresh interval for vis builder when we
refresh or navigate to other apps such as dashboard, discover, timeline and visualize

Signed-off-by: abbyhu2000 <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* Automates chromedriver version selection for tests (#2990)

Signed-off-by: Miki <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* Fixes `no-restricted-path` false-positives when `allowSameFolder` is true (#3020)

`no-restricted-paths` compares source files and import statements, and their membership in restricted zones. However, when `allowSameFolder` is true, it failed to remove a trailing slash before validation which results in a false-positive.

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

Signed-off-by: Miki <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* [Doc] Add readme for global query persistence (#3001)

Signed-off-by: Arpit Bandejiya <[email protected]>

* [MD] Feature test connection (#2973)

* test connection intial code

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

* error handling

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

* refactor

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

* removing get cluster info dependency

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

* refactor test connection

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

* adding test cases and test connection on edit datasource

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

* adding changelog comment

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

* fixing bug on edit datasource page

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

* refactor based on PR comments

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

Signed-off-by: mpabba3003 <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* [Chore] Add release notes for 1.3.7 (#3066)

Signed-off-by: Josh Romero <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* [Chore] Add 2.4.1 release notes (#3070)

* [Chore] Add 2.4.1 release notes

Signed-off-by: Josh Romero <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* [Table Visualization][BUG] Fix Url content display (#2918)

Currently, the new table can not format Url. If we
set to use URL format in index pattern field, table
will display it as string.

In this PR, we switch the format from string to html.
To make html understandable by react as a DOM element,
we use dangerouslySetInnerHTML to transform it. For the
security, since the content is not from random input but
fetched from stored data, we should be safe as long as
data is not messed.

To provide more security protection, we also add dompurify
package to sanitize the html content.

Issue Resolved:
https://github.com/opensearch-project/OpenSearch-Dashboards/issues/2905

Signed-off-by: Anan Zhuang <[email protected]>

Signed-off-by: Anan Zhuang <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* [Tests] Bumps `chromedriver` to v107 (#3017)

* Removes manual resolution of `axios`.

Signed-off-by: Tommy Markley <[email protected]>

Signed-off-by: Tommy Markley <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* Removes minimatch manual resolution (#3019)

* The minimatch resolution was no longer necessary after the upstream
library that depended on v3.0.4 was removed in #2711.

Signed-off-by: Tommy Markley <[email protected]>

Signed-off-by: Tommy Markley <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* Bumps simple-git from 3.4.0 to 3.15.0 (#3036)

Signed-off-by: Kristen Tian <[email protected]>

Signed-off-by: Kristen Tian <[email protected]>
Co-authored-by: Miki <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* Fix boktorbb alias. (#3085)

Signed-off-by: dblock <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* Bump decode-uri-component from 0.2.0 to 0.2.2 (#3009)

Bumps [decode-uri-component](https://github.com/SamVerschueren/decode-uri-component) from 0.2.0 to 0.2.2.
- [Release notes](https://github.com/SamVerschueren/decode-uri-component/releases)
- [Commits](https://github.com/SamVerschueren/decode-uri-component/compare/v0.2.0...v0.2.2)

---
updated-dependencies:
- dependency-name: decode-uri-component
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Ashwin P Chandran <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* [MD] Update url in tests to follow lychee url allowlist (#3099)

Signed-off-by: Su <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* [Infrastructure] Add `untriaged` label to issues workflow (#3073)

Credit to Peter Nied <[email protected]> via https://github.com/opensearch-project/.github/pull/111

Signed-off-by: Josh Romero <[email protected]>

Signed-off-by: Josh Romero <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* Update console to use core.http instead of jQuery.ajax (#3080)

* Update console to use core.http instead of jQuery.ajax

Signed-off-by: Yan Zeng <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* [Maintenance] Exempt feature branches from changelog verification workflow (#3123)

Signed-off-by: Josh Romero <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* Gets rid of `github-checks-reporter` (#3126)

Signed-off-by: Miki <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* Updates NOTICE file, adds validation to GitHub CI (#3051)

* Aligns NOTICE with rules outlined in
https://github.com/opensearch-project/.github/issues/21.
* Adds NOTICE validation to the build and test workflow.
* Fixes product name discrepancies between repository NOTICE file and
the generated NOTICE file for the build.
* Skips template-izing the build and test workflow since that's being
worked on in #2991.

Resolves #765

Signed-off-by: Tommy Markley <[email protected]>
Co-authored-by: Anan Zhuang <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* [Doc] Add current plugin persistence implementation readme (#3081)

* readme

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

* Add readme for current plugin data persistence implementation

Added readme in root level doc folder

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

* Changelog and minor fixes

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

* some modifications

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

Signed-off-by: abbyhu2000 <[email protected]>
Co-authored-by: Anan Zhuang <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* [Table Visualization] Replace table visualization with React and DataGrid (#2863)

* [Table Visualization] Replace table visualization with React and DataGrid

In this PR, we add back functions to make new table usage to
be consistent with the replaced one.
* total function
* percentage column
* filter in/out

Meanwhile, we also add back server. Functional tests are removed.
We will add new functional test in opensearch-dashboards-functional-test
repo. We also clean out some legacy codes.

Issue Resolved:
https://github.com/opensearch-project/OpenSearch-Dashboards/issues/2855

Signed-off-by: Anan Zhuang <[email protected]>

* add some data-test-subj and fix PR comments

Signed-off-by: Anan Zhuang <[email protected]>

* Fix PR comments and add unit tests

Signed-off-by: Anan Zhuang <[email protected]>

* remove listenOnChange

Signed-off-by: Anan Zhuang <[email protected]>

Signed-off-by: Anan Zhuang <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* [CVE] Update package info and changelog for loader-utils bump to 2.0.4 fixing CVE-2022-37599 and CVE-2022-37603 (#3031)

Signed-off-by: Zilong Xia <[email protected]>
Co-authored-by: Josh Romero <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* Fixes misleading embeddable plugin error message (#3043)

Resolves #1479
Resolves #3083

Signed-off-by: Tommy Markley <[email protected]>
Co-authored-by: Josh Romero <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* [Vis Builder] Add redux store persistence (#3088)

* add redux store persistence

implement persistence without using state container or state sync utils, and it
works with both the URL and session storage.

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

* changelog and rebase

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

* Console log the error

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

* rebase and changelog

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

* add unit tests

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

Signed-off-by: abbyhu2000 <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* Adds config override to fix obsolete theme:version config value of v8 (beta) rendering issue (#3045)

Signed-off-by: Manasvini B Suryanarayana <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* Adds retrying flaky jest tests (#2967)

Signed-off-by: Miki <[email protected]>
Co-authored-by: Anan Zhuang <[email protected]>
Co-authored-by: Josh Romero <[email protected]>
Co-authored-by: Qingyang(Abby) Hu <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* [MD] Improve test connection (#3110)

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

Signed-off-by: Su <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* Bump node version from 14.20.0 to 14.20.1 to resolve CVE-2022-35256 (#3166)

* Bump node version from 14.20.0 to 14.20.1 to resolve CVE-2022-35256

Signed-off-by: Zilong Xia <[email protected]>

* Update the PR number in CHANGELOG

Signed-off-by: Zilong Xia <[email protected]>

Signed-off-by: Zilong Xia <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* [CI] update yarn timeout for GitHub workflow on Windows (#3118)

Yarn 1.x seems to have an issue with timing on windows and
mac when running.

Source:
https://github.com/yarnpkg/yarn/issues/8242#issuecomment-776561223

Increase timeout for Windows only.

Issue:
n/a

Signed-off-by: Kawika Avilla <[email protected]>

Signed-off-by: Kawika Avilla <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* Updated MAINTAINERS.md to match recommended opensearch-project format. (#3180)

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

Signed-off-by: dblock <[email protected]>
Co-authored-by: Qingyang(Abby) Hu <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* [CI] Record functional test artifacts in case of failure (#3190)

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

Signed-off-by: Miki <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* Increase timeout waiting for the exiting of an optimizer worker (#3193)

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

Signed-off-by: Miki <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* [CI] Improve yarn's performance in workflows by caching yarn's cache folder (#3194)

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

Signed-off-by: Miki <[email protected]>
Co-authored-by: Qingyang(Abby) Hu <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* [Vis Builder] Add app filter and query persistence without using state container (#3100)

* connect without container

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

* Query and filter persistence working

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

* Rebase and changelog

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

* Simplify filter management

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

* change function name

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

* add unit test for function connect storage to query

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

* changelog change

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

* add comments and functional documentation in readme

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

* add more documentation in data persistence readme about vis builder persistence

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

* error handling

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

* add function definition in data plugin api doc

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

* document and comments

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

Signed-off-by: abbyhu2000 <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* test (#3197)

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

Signed-off-by: abbyhu2000 <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* Update json5 to 1.0.2 and 2.2.3 (#3201)

* Update json5 to 1.0.2 and 2.2.3

Signed-off-by: Matt Provost <[email protected]>

* Update changelog

Signed-off-by: Matt Provost <[email protected]>

Signed-off-by: Matt Provost <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* [Chore] Add `vega-lite` v5 dependency and bundle from source (#3076)

* [Chore] Add vega-lite v5 dep and bundle from source

- Add vega-lite@^5.6.0 dependency
- Update webpack config to build `vega-lite` from typescript source to target es5
- Add babel plugin for logical assignment operators
- Selectively import only the vega-lite modules used
- Update vega_parser tests

Co-authored-by: Miki <[email protected]>
Signed-off-by: Josh Romero <[email protected]>

* [Chore] Update vega-lite defaults and tests to v5

- Add test case for older vega-light spec parsing
- Update default schema version in default spec, test specs, mocked saved objects
- Fix indentation of comments in default spec
- Update jest mock imports to use `vega-lite`

Signed-off-by: Josh Romero <[email protected]>

Signed-off-by: Josh Romero <[email protected]>
Co-authored-by: Miki <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* [Feature] Allow order control in `createAggConfig` (#3160)

* [Feature] Allow order control in `createAggConfig`

Add new optional boolean param that adds the new `AggConfig` to the beginning of the array rather than the end.
Makes it easier to work with Pie or other visualizations with `Schemas` that set `mustBeFirst`

Signed-off-by: Josh Romero <[email protected]>

* [Chore] Add changelog

Signed-off-by: Josh Romero <[email protected]>

Signed-off-by: Josh Romero <[email protected]>
Co-authored-by: Qingyang(Abby) Hu <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* [VisBuilder] Adds field unit tests (#3211)

* Adds field tests

Signed-off-by: Ashwin P Chandran <[email protected]>

* Adds changelog

Signed-off-by: Ashwin P Chandran <[email protected]>

Signed-off-by: Ashwin P Chandran <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* Fix bug that prevents selected join field to be used (#3213)

Remove setting join field even if it is already set.

Signed-off-by: Vijayan Balasubramanian <[email protected]>

Signed-off-by: Vijayan Balasubramanian <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* [VisBuilder] fixes filters for table visualisation (#3210)

* fixes table vis for filter aggregation types

Signed-off-by: Ashwin P Chandran <[email protected]>

* Fixes filter and add error boundry

Signed-off-by: Ashwin P Chandran <[email protected]>

* Adds changelog

Signed-off-by: Ashwin P Chandran <[email protected]>

Signed-off-by: Ashwin P Chandran <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* [VisBuilder] Fixes pipeline aggs (#3137)

* fixes pipeline aggs in visbuilder

Signed-off-by: Ashwin P Chandran <[email protected]>

* adds changelog

Signed-off-by: Ashwin P Chandran <[email protected]>

* Adds unit tests

Signed-off-by: Ashwin P Chandran <[email protected]>

* fixes pipeline aggs in visbuilder

Signed-off-by: Ashwin P Chandran <[email protected]>

* adds changelog

Signed-off-by: Ashwin P Chandran <[email protected]>

* Adds unit tests

Signed-off-by: Ashwin P Chandran <[email protected]>

* fixes unit tests

Signed-off-by: Ashwin P Chandran <[email protected]>

Signed-off-by: Ashwin P Chandran <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* Add v2.5.0 release notes (#3234)

* Add v2.5.0 release notes

Signed-off-by: Manasvini B Suryanarayana <[email protected]>

* Adds aliased vega-lite v5 dependency to the notes #3151

Signed-off-by: Manasvini B Suryanarayana <[email protected]>

* Address feedback comments

Signed-off-by: Manasvini B Suryanarayana <[email protected]>

* Add #3197 to notes

Signed-off-by: Manasvini B Suryanarayana <[email protected]>

* Address feedback comments

Signed-off-by: Manasvini B Suryanarayana <[email protected]>

* Address feedback comments

Signed-off-by: Manasvini B Suryanarayana <[email protected]>

* Address feedback comments

Signed-off-by: Manasvini B Suryanarayana <[email protected]>

Signed-off-by: Manasvini B Suryanarayana <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* [MD]Update test connection button text (#3247)

Signed-off-by: Kristen Tian <[email protected]>

Signed-off-by: Kristen Tian <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* Update text and include new changes in the release notes 2.5.0 (#3251)

Signed-off-by: Manasvini B Suryanarayana <[email protected]>

Signed-off-by: Manasvini B Suryanarayana <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* [Chore] fix comment typo (#3239)

Signed-off-by: Josh Romero <[email protected]>

Signed-off-by: Josh Romero <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* [Docs] Fix documentation link for date math (#3207)

- Add new documentation link
- move from `noDocumentation` to `opensearch`

fixes #2849

Signed-off-by: Josh Romero <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* Fix detection of Chrome's version on Darwin (#3296)

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

Signed-off-by: Miki <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* [Maintenance] Bumps `re2` and `supertest` (#3018)

* Removes manual resolution on `qs`. The latest version of `supertest` depends on v6.11.0.
* Adds steps for upgrading `re2`.
* Addresses CVE-2022-24999 (no issue opened)

Signed-off-by: Tommy Markley <[email protected]>

Signed-off-by: Tommy Markley <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* Add disablePrototypePoisoningProtection configuration (#2992)

Enables the configuration of `disablePrototypePoisoningProtection` by setting
`opensearch.disablePrototypePoisoningProtection`. Enables users to store
protected logs that include reserve words from JS without the
OpenSearch JS client throwing errors.

We should still consider transforming unsafe data values if a bad actor
attempts to prototype pollute the cluster.

More information:
https://web.archive.org/web/20200319091159/https://hueniverse.com/square-brackets-are-the-enemy-ff5b9fd8a3e8?gi=184a27ee2a08

Related issue:
https://github.com/opensearch-project/OpenSearch-Dashboards/issues/1777

Signed-off-by: Kawika Avilla <[email protected]>

Signed-off-by: Kawika Avilla <[email protected]>
Co-authored-by: Anan Zhuang <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* [CVE-2022-25860] Bumps simple-git from 3.15.1 to 3.16.0 (#3345)

Issue Resolved
https://github.com/opensearch-project/OpenSearch-Dashboards/issues/3329

Signed-off-by: Anan Zhuang <[email protected]>

Signed-off-by: Anan Zhuang <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* Add 1.3.8 release notes (#3367)

* Create 1.3.8 release notes

Signed-off-by: Aozixuan Priscilla Guan <[email protected]>

* Remove unused tags

Signed-off-by: Aozixuan Priscilla Guan <[email protected]>

* Remove old changelog

Signed-off-by: Aozixuan Priscilla Guan <[email protected]>

* Fix typo

Signed-off-by: Aozixuan Priscilla Guan <[email protected]>

* Address comments

Signed-off-by: Aozixuan Priscilla Guan <[email protected]>

* Add PRs

Signed-off-by: Aozixuan Priscilla Guan <[email protected]>

* Remove unreleased PR

Signed-off-by: Aozixuan Priscilla Guan <[email protected]>

* Remove unreleased PR

Signed-off-by: Aozixuan Priscilla Guan <[email protected]>

---------

Signed-off-by: Aozixuan Priscilla Guan <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* Add BWC tests for 2.6.0 (#3356)

Signed-off-by: Manasvini B Suryanarayana <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* Prevent primitive linting limitations from being applied to unit tests found under `src/setup_node_env` (#3403)

Signed-off-by: Miki <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* [Security] Bumps hapi/statehood to 7.0.4 (#3411)

Bump hapi/statehood to 7.0.4 to solve security concerns.

Issue Resolved:
https://github.com/opensearch-project/OpenSearch-Dashboards/issues/3406

Signed-off-by: Anan Zhuang <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* [CVE-2022-25881] Resolve http-cache-semantics to 4.1.1 (#3409)

Signed-off-by: Su <[email protected]>
Co-authored-by: Anan Zhuang <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* [CVE-2023-25166] Bump formula to 3.0.1 (#3416)

* [CVE-2023-25166] Bumps formula to 3.0.1

Signed-off-by: Su <[email protected]>
Co-authored-by: Miki <[email protected]>
Co-authored-by: Anan Zhuang <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* Add new ui setting for size (#3399)

Introduce new ui setting for custom vector map's size
parameter. The default value is 1000.
Users can increase this limit by updating this value
in Advanced Settings.

Signed-off-by: Vijayan Balasubramanian <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* [search telemetry] Fixes search telemetry's observable object that won't be GC-ed (#3390)

The search telemetry was disabled by default, there is a issue when search telemetry read configuration and creates an Observable object that won't be GC-ed.

Signed-off-by: Tao Liu <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* Corrects NOTICE file copyright date range (#3308)

* Corrects NOTICE file copyright date range

* Follow-up on #3051

Resolves #765

Signed-off-by: Tommy Markley <[email protected]>

* Update CHANGELOG.md

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

Signed-off-by: Tommy Markley <[email protected]>
Co-authored-by: Zhongnan Su <[email protected]>
Co-authored-by: Miki <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* Upgrade vega-tooltip to 0.30.0 to support custom tooltips #3358 (#3359)

* Upgrade vega-tooltip to 0.30.0 to support custom tooltips #3358

Signed-off-by: Jovan Cvetkovic <[email protected]>

* Upgrade vega-tooltip to 0.30.0 to support custom tooltips #3358

Signed-off-by: Jovan Cvetkovic <[email protected]>

* Upgrade vega-tooltip to 0.30.0 to support custom tooltips #3358

Signed-off-by: Jovan Cvetkovic <[email protected]>

---------

Signed-off-by: Jovan Cvetkovic <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* Added untriaged issue workflow. (#3427)

* Created untriaged issue workflow.

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

* Remove existing untriaged workflow with wrong name

Signed-off-by: Josh Romero <[email protected]>

---------

Signed-off-by: dblock <[email protected]>
Signed-off-by: Josh Romero <[email protected]>
Co-authored-by: Josh Romero <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* [MD] Support SigV4 as a new auth type of datasource (#3058)

* [Multiple DataSource] Add support for SigV4 authentication

Signed-off-by: Su <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* Allow relaxing the Node.js runtime version requirement (#3402)

With this change, the `engines.node` of `package.json` need not be a fixed version; it can be a range using a subset of formats defined by `semver`.

A very limited subset of what [semver](https://github.com/npm/node-semver/blob/cb1ca1d5480a6c07c12ac31ba5f2071ed530c4ed/README.md#ranges) supports has been added; only a single comparator, composed of an operator and a version, is supported.

The supported operators are
 *   `>`    Greater than
 *   `>=`   Greater than or equal to
 *   `=`    Equal
 *   `~`    Tilde ranges: Allows patch changes if a minor version is specified but if only a major version is specified, it allows minor changes.
 *   `^`    Caret ranges: Allows patch and minor updates when major is non-zero (and we will never have that).

If no operator is specified, equality is assumed.

Signed-off-by: Miki <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* [CVE-2023-25653] Bump node-jose to 2.2.0 (#3445)

Signed-off-by: Arpit Bandejiya <[email protected]>

* Upgrade yarn version to be compatible with @openearch-project/opensearch (#3443)

* upgrade yarn version to 1.22.10 to be compatible with @openearch-project/opensearch

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

* Make build scripts find and use the latest version of Node.js that satisfies `engines.node` (#3467)

* While building distributables, Node.js runtime is downloaded to be placed in the archivea. This logicwas modified to honor a range for `engines.node` by fetching the latest release of Node.js that satisfied the range.
* Some tests covering the build, read a version from `.node-version` to compare with the results of actual function runs; these were changed to either use mocked values or honor the range and use the latest Node.js version.
* Some variable and functions referred to `engines.node` as a version; they were corrected to call it a range.

Signed-off-by: Miki <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* Relax Node.js version to `^14.20.1` and bump `.nvmrc` to `v14.21.3` (#3463)

* `engines.node` is relaxed from `14.20.1` to `^14.20.1`: yarn and OSD will allow versions 14.20.1 or greater, but less than 15, to be used but do not impose upgrading to avoid being a breaking change. Users will be able to install any version of Node.js that satisfies `^14.20.1`, moving ahead without waiting for a change in OSD when new versions are released.
* `.nvmrc` is bumped to the latest security patch: `14.21.3`

Signed-off-by: Miki <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* [MD] Integrate test connection to support SigV4 auth type (#3456)

Signed-off-by: Su <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* Add 2.6 release note (#3484)

* a

Signed-off-by: Anan Zhuang <[email protected]>

* Add 2.6 release note

Signed-off-by: Anan Zhuang <[email protected]>

* Update release-notes/opensearch-dashboards.release-notes-2.6.0.md

Co-authored-by: Josh Romero <[email protected]>

---------

Signed-off-by: Anan Zhuang <[email protected]>
Co-authored-by: Josh Romero <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* docs(DEVELOPER_GUIDE): Make getting started comprehensive and reorganize (#3421)

* docs(DEVELOPER_GUIDE): Make getting started comprehensive and reorganize

Enhance getting started guide for new contributors
- Add key technology breakdown with links to code guidelines
- Add robust prerequisite guide
- Clarify forking workflow
- Explain bootstrapping
- Distinguish between recommended and optional/alternative steps
- Add OS-specific guidance
- Add tarball usage instructions

Make other developer guide improvements:
- Update TOC
- Remove callout linking to EUI docs
- Reorder code guidelines
- Standardize header capitalization and make usage guide headers imperative

fixes #3396

Signed-off-by: Josh Romero <[email protected]>

Signed-off-by: Josh Romero <[email protected]>
Co-authored-by: Miki <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* [BUG][TSVB] Fix serial diff agg link (#3503)

Doc link was invalid and causing an undefined error. This
prevented the UI from mounting properly.

Issue(s) resolved:
* https://github.com/opensearch-project/OpenSearch-Dashboards/issues/3498
* https://github.com/opensearch-project/OpenSearch-Dashboards/issues/3327

Signed-off-by: Kawika Avilla <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* Remove the unused `renovate.json5` file (#3489)

Signed-off-by: Miki <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* [Docs] fix typo in TSVB README (#3518)

Fixes a small typo in TSVB README.md file.

Issue:
n/a

Signed-off-by: Kawika Avilla <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* Update caniuse to fix failed integration tests (#3538)

Ran npx browserslist@latest --update-db to update caniuse package so
the integration tests will pass.

Issue:
n/a

Will be fixed in:
https://github.com/opensearch-project/OpenSearch-Dashboards/issues/2329

Signed-off-by: Kawika Avilla <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* [CVE-2023-26486][CVE-2023-26487] Bump vega from 5.22.1 to 5.23.0 (#3533)

Bump vega from 5.22.1 to 5.23.0. This will also bump vega-function
from 5.13.0 to 5.13.1.

Issue Resolved:
https://github.com/opensearch-project/OpenSearch-Dashboards/issues/3526
https://github.com/opensearch-project/OpenSearch-Dashboards/issues/3525

Signed-off-by: Anan Zhuang <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* [Dev] enable `opensearch snapshot` for Darwin (#3537)

Enable the downloading of Darwin for running the command
`yarn opensearch snapshot`.

Darwin is not officially supported but snapshots are being
built here:
https://build.ci.opensearch.org/job/distribution-build-opensearch/

Issue resolved:
https://github.com/opensearch-project/OpenSearch-Dashboards/issues/2944

Signed-off-by: Kawika Avilla <[email protected]>
Co-authored-by: Josh Romero <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* Simplifies `re2` upgrade instructions (#3328)

* Follow-up from #3018

Signed-off-by: Tommy Markley <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

* Point in time management plugin and empty state (#2813)

* point in time management plugin and empty state

Signed-off-by: Ajay Gupta <[email protected]>

* updated to new license header

Signed-off-by: Ajay Gupta <[email protected]>

* [Vis Builder] Rename wizard on save modal and visualization table (#2645)

* Rename wizard on save modal and visualization table

Change the wizard reference in save modal title, toggle and visualization table

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

* Save visualization on save modal, VisBuilder for chart type

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

* No experimental flag

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

Signed-off-by: abbyhu2000 <[email protected]>
Signed-off-by: Ajay Gupta <[email protected]>

* [@osd/cross-platform] Adds cross-platform helpers (#2681)

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

Signed-off-by: Miki <[email protected]>
Signed-off-by: Ajay Gupta <[email protected]>

* Change save object type, wizard id and name to visBuilder (#2673)

* Change save object type, wizard ID and Name to visBuilder

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

* Rename wizard in functional tests

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

* Finalize plugin-id and plugin-name and saved-object-type

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

* Change official name to VisBuilder

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

* Delete migration

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

* Fix functional test

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

* Update snapshot

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

Signed-off-by: abbyhu2000 <[email protected]>
Signed-off-by: Ajay Gupta <[email protected]>

* [CVE] Bump loader-utils to 2.0.3 to fix CVE-2022-37601 (#2689)

* [CVE] Bump loader-utils to 2.0.3 to fix CVE-2022-37601

Signed-off-by: Zilong Xia <[email protected]>

* Update CHANGELOG.md

Signed-off-by: Zilong Xia <[email protected]>
Signed-off-by: Ajay Gupta <[email protected]>

* Add extension point in saved object management to register namespaces and show filter (#2656)

* Create filter registry for saved object management to make filters extensible

Signed-off-by: Craig Perkins <[email protected]>

* WIP on making fetchCounts generic

Signed-off-by: Craig Perkins <[email protected]>

* First step at making scroll_counts generic

Signed-off-by: Craig Perkins <[email protected]>

* Work on getting other filter counts with same object count endpoint

Signed-off-by: Craig Perkins <[email protected]>

* Get tenant count options to display

Signed-off-by: Craig Perkins <[email protected]>

* Extend find to work with namespaces for saved objects

Signed-off-by: Craig Perkins <[email protected]>

* Add missing filterFields

Signed-off-by: Craig Perkins <[email protected]>

* Update jest tests

Signed-off-by: Craig Perkins <[email protected]>

* Update saved_objects_table snapshot

Signed-off-by: Craig Perkins <[email protected]>

* Append index to id to make unique

Signed-off-by: Craig Perkins <[email protected]>

* Add semi-colon

Signed-off-by: Craig Perkins <[email protected]>

* Fix saved objects table tests with new id scheme

Signed-off-by: Craig Perkins <[email protected]>

* Only append idx on config type to ensure Advanced Settings have a unique id across tenants

Signed-off-by: Craig Perkins <[email protected]>

* Remove itemsClone in favor of showing only Advanced Settings of current tenant

Signed-off-by: Craig Perkins <[email protected]>

* Revert snapshots in table.test.tsx

Signed-off-by: Craig Perkins <[email protected]>

* Add additional parse_query test

Signed-off-by: Craig Perkins <[email protected]>

* Add comma

Signed-off-by: Craig Perkins <[email protected]>

* Create namespaceRegistry to decouple security dashboards plugin and osd core

Signed-off-by: Craig Perkins <[email protected]>

* Add ability to register an alias

Signed-off-by: Craig Perkins <[email protected]>

* Update parse query and add to CHANGELOG

Signed-off-by: Craig Perkins <[email protected]>

* Remove commented out code

Signed-off-by: Craig Perkins <[email protected]>

* Address code review comments

Signed-off-by: Craig Perkins <[email protected]>

* Override i18n if alias is regitered

Signed-off-by: Craig Perkins <[email protected]>

Signed-off-by: Craig Perkins <[email protected]>
Signed-off-by: Ajay Gupta <[email protected]>

* Removed Leftover X Pack references #2517 (#2638)

x-pack references in the code are removed as per the given files in #2517

Issue Resolved:
https://github.com/opensearch-project/OpenSearch-Dashboards/issues/2517

Signed-off-by: vimal K <[email protected]>
Signed-off-by: Ajay Gupta <[email protected]>

* Revert "[@osd/cross-platform] Adds cross-platform helpers (#2681)" (#2694)

This reverts commit 887093d2d243045029b644680a3e8d0150318143.

Now that downstream plugins and projects are unblocked from builds, we want to make these changes following our standard processes and automated checks

Signed-off-by: Josh Romero <[email protected]>

Signed-off-by: Josh Romero <[email protected]>
Signed-off-by: Ajay Gupta <[email protected]>

* [MD] Add design documents of multiple data source feature (#2538)

* Add MD design documents, including high level design, user stories, client management detailed design

Signed-off-by: Su <[email protected]>
Signed-off-by: Ajay Gupta <[email protected]>

* Adds @osd/cross-platform (#2703)

* Adds helper functions, @osd/cross-platform, to work around the differences of platforms

Signed-off-by: Miki <[email protected]>
Signed-off-by: Ajay Gupta <[email protected]>

* [MD]Update MD data source documentation link (#2693)

Signed-off-by: Kristen Tian <[email protected]>

Signed-off-by: Kristen Tian <[email protected]>
Signed-off-by: Ajay Gupta <[email protected]>

* Add `Skip-Changelog` label to skip changelog verification on certain PRs (#2726)

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

Signed-off-by: Miki <[email protected]>
Signed-off-by: Ajay Gupta <[email protected]>

* Enable visbuilder by default (#2725)

* Enable visbuilder by default

Signed-off-by: Ashwin P Chandran <[email protected]>

* Adds changelog entry

Signed-off-by: Ashwin P Chandran <[email protected]>

Signed-off-by: Ashwin P Chandran <[email protected]>
Signed-off-by: Ajay Gupta <[email protected]>

* [Save Object Aggregation View] Fix for export all after scroll count response changed in PR#2656 (#2696)

* Fix for filterSavedObjectCounts for namespaceRegistry

Signed-off-by: Craig Perkins <[email protected]>

* Fix saved_objects_table.test.tsx

Signed-off-by: Craig Perkins <[email protected]>

* Add to CHANGELOG

Signed-off-by: Craig Perkins <[email protected]>

* Correct reference to namespacesToInclude

Signed-off-by: Craig Perkins <[email protected]>

* Use filteredTypeCounts

Signed-off-by: Craig Perkins <[email protected]>

* Use namespaces similar to types for fetchObjects

Signed-off-by: Craig Perkins <[email protected]>

* Use _all to represent query for all namespaces

Signed-off-by: Craig Perkins <[email protected]>

* Pass all registered namespaces

Signed-off-by: Craig Perkins <[email protected]>

* Switch back signature of scroll_count

Signed-off-by: Craig Perkins <[email protected]>

* Change countOptions to options

Signed-off-by: Craig Perkins <[email protected]>

* Use not not instead of in

Signed-off-by: Craig Perkins <[email protected]>

* Filter namespaces to only include namespace that have been registered

Signed-off-by: Craig Perkins <[email protected]>

* Add filterQuery with tests

Signed-off-by: Craig Perkins <[email protected]>

* Update license headers and address review comments

Signed-off-by: Craig Perkins <[email protected]>

Signed-off-by: Craig Perkins <[email protected]>
Signed-off-by: Ajay Gupta <[email protected]>

* Update backport workflow to ignore changelog conflicts (#2729)

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

Signed-off-by: Miki <[email protected]>
Signed-off-by: Ajay Gupta <[email protected]>

* Tweak multiple data source design doc (#2724)

Signed-off-by: Kristen Tian <[email protected]>

Signed-off-by: Kristen Tian <[email protected]>
Signed-off-by: Ajay Gupta <[email protected]>

* [Chore] Refactor and improve Discover field summaries (#2391)

* [Chore] Refactor and improve field summaries

* Convert to typescript
* Fix types
* Add tests

Signed-off-by: Josh Romero <[email protected]>

* [Test] Update functional test

Groups are now naturally sorted by key, which requires selecting a different date filter

Signed-off-by: Josh Romero <[email protected]>

* [Chore] Add changelog entry

Signed-off-by: Josh Romero <[email protected]>

* [Chore] Refactor columns passing, fix bugs

* pass columns explicitly as props
* fix branding in core mocks
* fix `toBeUndefined()` usage in tests
* remove leftover comment
* fix test subject
* condense types

Signed-off-by: Josh Romero <[email protected]>

Signed-off-by: Josh Romero <[email protected]>
Signed-off-by: Ajay Gupta <[email protected]>

* Removes Add Integration button (#2723)

* Removes Add Integration button

Issue Resolved:
https://github.com/opensearch-project/OpenSearch-Dashboards/issues/2684

Signed-off-by: Bandini Bhopi <[email protected]>

* Adds changelog entry

Signed-off-by: Bandini Bhopi <[email protected]>

Signed-off-by: Bandini Bhopi <[email protected]>
Signed-off-by: Ajay Gupta <[email protected]>

* Prevent backport workflow from running on umerged PRs (#2746)

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

Signed-off-by: Miki <[email protected]>
Signed-off-by: Ajay Gupta <[email protected]>

* [Vis Builder] Enable VisBuilder cypress tests (#2728)

* Enable VisBuilder cypress tests

Signed-off-by: Ashwin P Chandran <[email protected]>

* Adds changelog and fix env

Signed-off-by: Ashwin P Chandran <[email protected]>

* Run all tests

Signed-off-by: Ashwin P Chandran <[email protected]>

Signed-off-by: Ashwin P Chandran <[email protected]>
Signed-off-by: Ajay Gupta <[email protected]>

* [CI] update backport custom branch name (#2766)

GitHub action for backporting got updated to change custom branch
to head template.

This was causing issues in clean up and skipping steps.

Issue:
n/a

Signed-off-by: Kawika Avilla <[email protected]>
Signed-off-by: Ajay Gupta <[email protected]>

* [Legacy Maps Plugin] Prevent reverse-tabnabbing (#2540)

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

Signed-off-by: Miki <[email protected]>
Signed-off-by: Ajay Gupta <[email protected]>

* change geckodriver version to make consistency (#2772)

* change geckodriver version to make consistency

Issue Resolved:
https://github.com/opensearch-project/OpenSearch-Dashboards/issues/2771

Signed-off-by: Anan Zhuang <[email protected]>

* combine https-proxy-agent in yarn.lock

Signed-off-by: Anan Zhuang <[email protected]>

Signed-off-by: Anan Zhuang <[email protected]>
Signed-off-by: Ajay Gupta <[email protected]>

* [Vis Builder] Add an experimental table visualization in vis builder (#2705)

* [Vis Builder] Add an experimental table visualization in vis builder

In this PR, we hook up an experimental table vis in vis builder. This
table vis is a refactor of previous table. It is written in React and
DataGrid component.

In this PR, we did two main things:
* add an experimental table visualization
* enable it in vis builder

Issue Resolved (hook up table in vis builder):
https://github.com/opensearch-project/OpenSearch-Dashboards/issues/2704

The experimental table vis has all the features from current table, including
* restore table vis in react using a Datagrid component
* datagrid component does not support splitted grids. For future transfer
to OUI Datagrid, we create a tableGroup in visData for splitted grids.
* restore basic pagenation, sort and format.
* implement datagrid columns
* display column title correctly
* deangular and re-use formatted column
* convert formatted column to data grid column
* restore filter in and filter out value functions
* format table cell to show Date and percent
* restore showTotal feature: it allows table vis to show total,
avg, min, max and count statics on count
* restore export csv feature to table vis
* split table in rows and columns

Beside of restoring original features, there are some changes:

* [IMPROVE] remove repeated column from split tables
Currently, when we split table by columns, the split column is shown
both in the table title and as a separate column. This is not needed.
In this PR, we remove the repeated column in split tables in col.
* [NEW FEATURE] adjustable table column width
In the new table visualization, customer can adjust the column width
as needed.

Issue Resolved:
https://github.com/opensearch-project/OpenSearch-Dashboards/issues/2212
https://github.com/opensearch-project/OpenSearch-Dashboards/issues/2213
https://github.com/opensearch-project/OpenSearch-Dashboards/issues/2305
https://github.com/opensearch-project/OpenSearch-Dashboards/issues/2379
https://github.com/opensearch-project/OpenSearch-Dashboards/issues/2579

Since this is a hookup PR, we remove un-used table vis types and options
because they could be defined in vis builder.

We also create follow up issues for some un-resolved PR comments.

Signed-off-by: Anan Zhuang <[email protected]>

* remove unused scss tyle

Signed-off-by: Anan Zhuang <[email protected]>

* remove total func and percentage col

total func and percentage col are two features that we might
need to remove or re-invent for future table vis. For hookup
purpose, it doesn't make sense to include some features that
we would like to remove.

this PR removes total func and percentage col in both table
vis and vis builder

Signed-off-by: Anan Zhuang <[email protected]>

* comment out cellActions
currently filter in/out cell doesn't function in vis builder.
we will coumment out cell actions for now.

Signed-off-by: Anan Zhuang <[email protected]>

Signed-off-by: Anan Zhuang <[email protected]>
Signed-off-by: Ajay Gupta <[email protected]>

* Security-CVEs fixes guidelines (#2674)

* Security-CVEs fixes guidelines

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

* styling

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

* add example

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

* add documention

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

Signed-off-by: himsgupta1122 <[email protected]>
Signed-off-by: Ajay Gupta <[email protected]>

* [MD] Update default audit log path (#2793)

- Fix the /tmp path issue seen on windows platform.
- Change audit log to disable by default.

Signed-off-by: Kristen Tian <[email protected]>

Signed-off-by: Kristen Tian <[email protected]>
Signed-off-by: Ajay Gupta <[email protected]>

* [Vis Builder] Add field summary popovers (#2682)

* [Vis Builder] Add field summary popovers

Much of the functionality was ported from `Discover`, but
largely refactored.

* Add utilities to get sampled hit summaries by field
* Add popover summaries
* Slight refactor of special `Count` pseudofield
* Use observable subscription to update sampled hits

Fixes #950

Signed-off-by: Josh Romero <[email protected]>

* [Vis Builder] Add additional unit tests

Signed-off-by: Josh Romero <[email protected]>

* [VisBuilder] Update naming of summary field components

Signed-off-by: Josh Romero <[email protected]>

* [VisBuilder] Avoid prop passing by extracting custom hooks

- refactor meta field identification

Signed-off-by: Josh Romero <[email protected]>

* [VisBuilder] Add TODOs with issue links, fix test ID

Restores previous test ID for count field button

Signed-off-by: Josh Romero <[email protected]>

Signed-off-by: Josh Romero <[email protected]>
Signed-off-by: Ajay Gupta <[email protected]>

* Bump makelogs to remove dependency on got (#2801)

* Also dusted off the lock file a bit

Fixes #1764

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

Signed-off-by: Miki <[email protected]>
Signed-off-by: Ajay Gupta <[email protected]>

* Change VisBuilder flag for docker config (#2804)

Signed-off-by: Ashwin P Chandran <[email protected]>

Signed-off-by: Ashwin P Chandran <[email protected]>
Signed-off-by: Ajay Gupta <[email protected]>

* addressed review comments

Signed-off-by: Ajay Gupta <[email protected]>

* addressed review comments

Signed-off-by: Ajay Gupta <[email protected]>

* addressed review comments

Signed-off-by: Ajay Gupta <[email protected]>

* updated unit tests

Signed-off-by: Ajay Gupta <[email protected]>

* dummy commit to retry build

Signed-off-by: Ajay Gupta <[email protected]>

* address suggestions

Signed-off-by: Ajay Gupta <[email protected]>

Signed-off-by: Ajay Gupta <[email protected]>
Signed-off-by: abbyhu2000 <[email protected]>
Signed-off-by: Miki <[email protected]>
Signed-off-by: Zilong Xia <[email protected]>
Signed-off-by: Craig Perkins <[email protected]>
Signed-off-by: vimal K <[email protected]>
Signed-off-by: Josh Romero <[email protected]>
Signed-off-by: Su <[email protected]>
Signed-off-by: Kristen Tian <[email protected]>
Signed-off-by: Ashwin P Chandran <[email protected]>
Signed-off-by: Bandini Bhopi <[email protected]>
Signed-off-by: Kawika Avilla <[email protected]>
Signed-off-by: Miki <[email protected]>
Signed-off-by: Anan Zhuang <[email protected]>
Signed-off-by: himsgupta1122 <[email protected]>
Co-authored-by: Ajay Gupta <[email protected]>
Co-authored-by: Qingyang(Abby) Hu <[email protected]>
Co-authored-by: Miki <[email protected]>
Co-authored-by: ZilongX <[email protected]>
Co-authored-by: Craig Perkins <[email protected]>
Co-authored-by: Vimal K <[email protected]>
Co-authored-by: Josh Romero <[email protected]>
Co-authored-by: Zhongnan Su <[email protected]>
Co-authored-by: Kristen Tian <[email protected]>
Co-authored-by: Ashwin P Chandran <[email protected]>
Co-authored-by: Craig Perkins <[email protected]>
Co-authored-by: Bandini <[email protected]>
Co-authored-by: Kawika Avilla <[email protected]>
Co-authored-by: Anan Zhuang <[email protected]>
Co-authored-by: himsgupta1122 <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>

---------

Signed-off-by: Miki <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>
Signed-off-by: dependabot[bot] <[email protected]>
Signed-off-by: Miki <[email protected]>
Signed-off-by: Kawika Avilla <[email protected]>
Signed-off-by: abbyhu2000 <[email protected]>
Signed-off-by: mpabba3003 <[email protected]>
Signed-off-by: Josh Romero <[email protected]>
Signed-off-by: Anan Zhuang <[email protected]>
Signed-off-by: Tommy Markley <[email protected]>
Signed-off-by: Kristen Tian <[email protected]>
Signed-off-by: dblock <[email protected]>
Signed-off-by: Su <[email protected]>
Signed-off-by: Yan Zeng <[email protected]>
Signed-off-by: Zilong Xia <[email protected]>
Signed-off-by: Manasvini B Suryanarayana <[email protected]>
Signed-off-by: Matt Provost <[email protected]>
Signed-off-by: Ashwin P Chandran <[email protected]>
Signed-off-by: Vijayan Balasubramanian <[email protected]>
Signed-off-by: Aozixuan Priscilla Guan <[email protected]>
Signed-off-by: Tao Liu <[email protected]>
Signed-off-by: Jovan Cvetkovic <[email protected]>
Signed-off-by: Derek Ho <[email protected]>
Signed-off-by: Ajay Gupta <[email protected]>
Signed-off-by: Craig Perkins <[email protected]>
Signed-off-by: vimal K <[email protected]>
Signed-off-by: Bandini Bhopi <[email protected]>
Signed-off-by: himsgupta1122 <[email protected]>
Co-authored-by: Miki <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Kawika Avilla <[email protected]>
Co-authored-by: Qingyang(Abby) Hu <[email protected]>
Co-authored-by: Manideep Pabba <[email protected]>
Co-authored-by: Josh Romero <[email protected]>
Co-authored-by: Anan Zhuang <[email protected]>
Co-authored-by: Tommy Markley <[email protected]>
Co-authored-by: Kristen Tian <[email protected]>
Co-authored-by: Daniel (dB.) Doubrovkine <[email protected]>
Co-authored-by: Ashwin P Chandran <[email protected]>
Co-authored-by: Zhongnan Su <[email protected]>
Co-authored-by: Yan Zeng <[email protected]>
Co-authored-by: ZilongX <[email protected]>
Co-authored-by: Manasvini B Suryanarayana <[email protected]>
Co-authored-by: Daniel (dB.) Doubrovkine <[email protected]>
Co-authored-by: Matt Provost <[email protected]>
Co-authored-by: Miki <[email protected]>
Co-authored-by: Vijayan Balasubramanian <[email protected]>
Co-authored-by: Aozixuan Priscilla Guan <[email protected]>
Co-authored-by: Vijayan Balasubramanian <[email protected]>
Co-authored-by: Tao Liu <[email protected]>
Co-authored-by: Jovan Cvetkovic <[email protected]>
Co-authored-by: Derek Ho <[email protected]>
Co-authored-by: Ajay Gupta <[email protected]>
Co-authored-by: Ajay Gupta <[email protected]>
Co-authored-by: Craig Perkins <[email protected]>
Co-authored-by: Vimal K <[email protected]>
Co-authored-by: Craig Perkins <[email protected]>
Co-authored-by: Bandini <[email protected]>
Co-authored-by: himsgupta1122 <[email protected]>
sikhote pushed a commit to sikhote/OpenSearch-Dashboards that referenced this pull request Apr 24, 2023
…-project#3073)

Credit to Peter Nied <[email protected]> via opensearch-project/.github#111

Signed-off-by: Josh Romero <[email protected]>

Signed-off-by: Josh Romero <[email protected]>
Signed-off-by: David Sinclair <[email protected]>
sikhote pushed a commit to sikhote/OpenSearch-Dashboards that referenced this pull request Apr 24, 2023
…-project#3073)

Credit to Peter Nied <[email protected]> via opensearch-project/.github#111

Signed-off-by: Josh Romero <[email protected]>

Signed-off-by: Josh Romero <[email protected]>
Signed-off-by: David Sinclair <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants