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

[backport 1.x] Bump moment-timezone from 0.5.34 to 0.5.37 #2361 #2472

Conversation

joshuarrrr
Copy link
Member

backport 77b6068 from #2361

kavilla and others added 30 commits May 21, 2021 01:06
This does not fix everything in group 2, there are still failures which I believe
are being caused by some data being imported but this fixes an error in the plugins
of a incomplete rename of applyOpenSearchResp.

node scripts/functional_tests.js --config test/functional/config.js --include ciGroup2

Signed-off-by: Kawika Avilla <[email protected]>
This commit rebases the versioning to OpenSearch-Dashboards 1.0.0

Signed-off-by: Mihir Soni <[email protected]>
Update OpenSearch Github repository link.

Signed-off-by: Dhiraj Kumar Jain <[email protected]>
…about merging PRs in the contributing guide(opensearch-project#305)

Signed-off-by: Dawn M. Foster <[email protected]>
This provides a bit more visibility into who could be expected to be merging those changes.
* [Tests] ciGroup2 fully passing

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

* [Tests] updated ndjson files

Removing references to kibana

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

* [Tests] ignore map-related functional tests

Ignore map related functional tests temporarily until we have a
replacement for EMS.

See: opensearch-project#221

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

* [Tests] Update dashboards data to correct config

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

* [Tests] update ndjson to big to correct type

Signed-off-by: Kawika Avilla <[email protected]>
This doesn't fix the maps out of the box but fixes the
issues that prevented users from settings the maps service
in the config.

Which can be set by adding:

map.tilemap.url: "https://tiles.maps.search-services.aws.a2z.com/tiles/{z}/{x}/{y}.png"
map.includeOpenSearchMapsService: false

Signed-off-by: Kawika Avilla <[email protected]>
…ensearch-project#302)

The Debian packages build ok with fpm, but installing it fails due
to an invalid username and group name (see useradd(8): CAVEATS).
Once this was fixed, the service failed to start due to parts of the startup
configuration wanting files that did not exist on disk.

Files renamed to where systemd expects and runtime username
and group name changed to a supported form.

Tested on Debian 10.

Signed-off-by: Cole White <[email protected]>
* [Tests] update sample data for flights

Incorrect type of esType causing failures in functional tests.

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

* [Tests] Update huge fields test

Updated because it never navigated to site it depended on previous
tests to place it in the correct location. Also, needed to update
the max payload size because we went to changing everything to
`opensearch` or `opensearch dashboards` so payload sizes got larger.

Signed-off-by: Kawika Avilla <[email protected]>
This commit adds the installation steps of yarn and npm
in the README.md file before the setup.

Signed-off-by: Venu Vardhan Reddy Tekula <[email protected]>
The 'Try our sample data' link in the opensearch dashboards
overview page is broken. It has to be pointed to the path
`/app/home#/tutorial_directory/sampleData`. This commit fixes
this bug. The snapshots are updated accordingly.

Fixes opensearch-project#193

Signed-off-by: Venu Vardhan Reddy Tekula <[email protected]>
* [Build] Update default maps client

Update to not use EMS so it works out of the box.
Related to: opensearch-project#221

Fails linter and should problem write more unit tests but working with smoke tests.

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

* [Build] fix linter for map messages

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

* [Build] enable and update map func tests

Passing on the CI. Needed to update the casing of the expected response
but that was all that was needed.

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

* [Rename] rename to osd_version

Signed-off-by: Kawika Avilla <[email protected]>
Update the notice generator so that builds come with the
expected header

Signed-off-by: Kawika Avilla <[email protected]>
* [Rename] remove remaining references

Some values were added but should not impact current cuts for the
public.

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

* [Rename] update url for configs

Signed-off-by: Kawika Avilla <[email protected]>
…oject#355)

Added Auto-focus on Add Filter Input

Done by setting the initial focus of the popover with the relevant selector

* correct autofocus

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

* use class

Signed-off-by: galangel <[email protected]>
* [Build] restore kibana index

Restoring the index from .opensearch_dashboards to .kibana and then
updated the tests. This is allowable because this is for functional
purposes and for clusters/plugins that will migrate to Dashboards.
Their index will not require re-indexing for 1.0.0 and won't require
migration after further updates.
Signed-off-by: Kawika Avilla <[email protected]>

* [Tests] updates tests for consistency

This doesn't have impact on the results of tests but to keep
consistency in the index name these updates were missed.

Signed-off-by: Kawika Avilla <[email protected]>
Restoring the index from opensearchDashboardsSavedObjectMeta to
kibanaSavedObjectMeta and then updated the tests.

This is allowable because this is for functional purposes and for
clusters/plugins that will migrate to Dashboards. Their index will
not require re-indexing for 1.0.0 and should not require a full
migration.

Signed-off-by: Kawika Avilla <[email protected]>
Restoring the opensearchTypes to esTypes and then updated the tests
(unit, integ, and func test all passing). This is allowable because
this is for functional purposes and for clusters/plugins that will
migrate to Dashboards. Their index will not require re-indexing for 1.0.0
and shouldn't require a full migration after further updates.

Signed-off-by: Kawika Avilla <[email protected]>
* Fix dashboards documentation and DQL links, expand help menu width so label isn't truncated

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

* Remove kibana references from doc links, fix aggregation links

Signed-off-by: Tommy Markley <[email protected]>
Tommy Markley and others added 25 commits April 29, 2022 17:47
…ct#1539)

* This workflow is unnecessary now that the org uses the DCO App.

Signed-off-by: Tommy Markley <[email protected]>
…ensearch-project#1562)

add back icon directive, which injects SVG

Signed-off-by: Josh Romero <[email protected]>
(cherry picked from commit bd10227)
…arch-project#1559) (opensearch-project#1588)

* Utilizes Codecov yaml configuration: https://docs.codecov.com/docs/codecov-yaml
* The purpose of enforcing increasing code coverage over time is to attempt to improve the quality of
  changes via testing, reduce the need to request tests in PRs, and stay away from enforcing an
  ambiguous coverage percentage.

Signed-off-by: Tommy Markley <[email protected]>
(cherry picked from commit 21d7b6f)
…project#1669)

Add container width to enable correct truncation
Normalize line height to prevent clipped descenders

Fixes opensearch-project#1643

Signed-off-by: Josh Romero <[email protected]>
(cherry picked from commit b888cca)
GitHub env runs on Chrome 102. This is done to get the CI
working again on 1.x however there is a follow-up issue here:
opensearch-project#1746

However, there's an issue with Node existing on an a user that
the GitHub CI doesn't have permissions too.

Issue:
n/a

Signed-off-by: Kawika Avilla <[email protected]>
…roject#1823) (opensearch-project#1824)

Maps zoom level were updated to be 14 as noted here:
opensearch-project/maps#4 (comment)

Issue:
n/a

Signed-off-by: Kawika Avilla <[email protected]>
(cherry picked from commit e7362f9)

Co-authored-by: Kawika Avilla <[email protected]>
…ct#1550) (opensearch-project#1702)

* Fix WMS can't load when unable access maps services

Signed-off-by: Junqiu Lei <[email protected]>
(cherry picked from commit b2cfb6e)

Co-authored-by: Junqiu Lei <[email protected]>
Co-authored-by: Kawika Avilla <[email protected]>
Bump chrome driver version to 104 for github actions

Signed-off-by: Su <[email protected]>
…opensearch-project#1735)

Resolves opensearch-project#592

Signed-off-by: Tommy Markley <[email protected]>
(cherry picked from commit 89d3872)

Co-authored-by: Tommy Markley <[email protected]>
Co-authored-by: Kawika Avilla <[email protected]>
…#1931) (opensearch-project#2194)

Bump moment from 2.29.2 to 2.29.4

Bumps [moment](https://github.com/moment/moment) from 2.29.2 to 2.29.4.
- [Release notes](https://github.com/moment/moment/releases)
- [Changelog](https://github.com/moment/moment/blob/develop/CHANGELOG.md)
- [Commits](moment/moment@2.29.2...2.29.4)

---
updated-dependencies:
- dependency-name: moment
  dependency-type: direct:production
...

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

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
(cherry picked from commit 30255ab)

Co-authored-by: Manasvini B Suryanarayana <[email protected]>
…ect#2259)

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

Signed-off-by: Anan Zhuang <[email protected]>
(cherry picked from commit cb5b1e9)

Co-authored-by: Anan Zhuang <[email protected]>
…lers (opensearch-project#1932) (opensearch-project#2001)

* [CVE] Handle invalid query, index and date in vega charts filter handlers

Potential way to prevent XSS vulnerability discovered in the Vega charts OSD integration.

CVE link:
https://nvd.nist.gov/vuln/detail/CVE-2022-23713

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

* new license header for new files

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

Co-authored-by: Kawika Avilla <[email protected]>
(cherry picked from commit 9496da3)

Co-authored-by: Bandini <[email protected]>
…-project#2247)

Enable filtering with custom health checks based on node attributes:
```
opensearch.optimizedHealthcheck.filters: {
  attribute_key: "attribute_value",
}
```

Also, fixes issue that expects the response to array when it was a
dictionary.

Issue:
opensearch-project#2214
opensearch-project#2203

Signed-off-by: Kawika Avilla <[email protected]>
…ch-project#2277) (opensearch-project#2300)

Original implementation incorrectly assumed the return list of
nodes was an object array.
This PR:
opensearch-project#2232

Addressed the return but didn't catch the nodes.find in the return
which is a function for an array.

Also, refactors to return a list of node_ids because the original
implementation indicated that it should but it can return node ids
but it never did. It only returned `null` or `_local`, the problem
with this approach is that it doesn't expect valid node version
with different DIs or filter out nodes when we pass `null` as the node
ID for the node info call because it was fan out the request to all
nodes.

Now this function will return `_local` if all the nodes share the
same cluster_id using a greedy approach since we can assume it is
all the same version.

Will return node ids, if the cluster_id are different so it will
pass a CSV to the node info call and return the info for those nodes.
And null if no cluster_id is present, ie, fan out the response.

Original issue:
opensearch-project#2203

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

Signed-off-by: Kawika Avilla <[email protected]>
@joshuarrrr joshuarrrr requested a review from a team as a code owner September 30, 2022 22:05
@joshuarrrr joshuarrrr closed this Sep 30, 2022
@joshuarrrr joshuarrrr deleted the backport/backport-2361-to-1.x branch September 30, 2022 22:10
@joshuarrrr joshuarrrr restored the backport/backport-2361-to-1.x branch September 30, 2022 22:20
@joshuarrrr joshuarrrr deleted the backport/backport-2361-to-1.x branch October 3, 2022 18:38
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.