From 8960ec8f1c7a6b50bc435d22dbf74b7be32d9854 Mon Sep 17 00:00:00 2001 From: Vimal K <35750792+vimalMK@users.noreply.github.com> Date: Mon, 31 Oct 2022 12:15:35 -0500 Subject: [PATCH] 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 Signed-off-by: David Sinclair --- CHANGELOG.md | 1 + packages/osd-pm/src/utils/validate_dependencies.ts | 2 +- .../saved_objects/migrations/core/index_migrator.test.ts | 8 ++------ .../saved_objects/migrations/core/opensearch_index.ts | 2 +- src/dev/notice/cli.js | 2 +- .../tutorial/instructions/apm_server_instructions.ts | 2 +- src/plugins/data/README.md | 2 +- 7 files changed, 8 insertions(+), 11 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index e6d1927102b4..3e26767f93ea 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -49,6 +49,7 @@ Inspired from [Keep a Changelog](https://keepachangelog.com/en/1.0.0/) - [Multi DataSource] Address UX comments on index pattern management stack ([#2611](https://github.com/opensearch-project/OpenSearch-Dashboards/pull/2611)) - [Multi DataSource] Apply get indices error handling in step index pattern ([#2652](https://github.com/opensearch-project/OpenSearch-Dashboards/pull/2652)) - [Vis Builder] Last Updated Timestamp for visbuilder savedobject is getting Generated ([#2628](https://github.com/opensearch-project/OpenSearch-Dashboards/pull/2628)) +- Removed Leftover X Pack references ([#2638](https://github.com/opensearch-project/OpenSearch-Dashboards/pull/2638)) ### 🚞 Infrastructure diff --git a/packages/osd-pm/src/utils/validate_dependencies.ts b/packages/osd-pm/src/utils/validate_dependencies.ts index 51a1c0eada0b..2264681b986c 100644 --- a/packages/osd-pm/src/utils/validate_dependencies.ts +++ b/packages/osd-pm/src/utils/validate_dependencies.ts @@ -175,7 +175,7 @@ export async function validateDependencies(osd: OpenSearchDashboards, yarnLock: const devOnlyProjectsInProduction = getDevOnlyProductionDepsTree(osd, 'opensearch-dashboards'); if (devOnlyProjectsInProduction) { log.error(dedent` - Some of the packages in the production dependency chain for OpenSearch Dashboards and X-Pack are + Some of the packages in the production dependency chain for OpenSearch Dashboards are flagged with "opensearchDashboards.devOnly" in their package.json. Please check changes made to packages and their dependencies to ensure they don't end up in production. diff --git a/src/core/server/saved_objects/migrations/core/index_migrator.test.ts b/src/core/server/saved_objects/migrations/core/index_migrator.test.ts index 2860de529764..4bacfda3bd5a 100644 --- a/src/core/server/saved_objects/migrations/core/index_migrator.test.ts +++ b/src/core/server/saved_objects/migrations/core/index_migrator.test.ts @@ -141,9 +141,7 @@ describe('IndexMigrator', () => { }, }); - await expect(new IndexMigrator(testOpts).migrate()).rejects.toThrow( - /use the X-Pack upgrade assistant/ - ); + await expect(new IndexMigrator(testOpts).migrate()).rejects.toThrow(); }); test('fails if root doc type is not "doc"', async () => { @@ -164,9 +162,7 @@ describe('IndexMigrator', () => { }, }); - await expect(new IndexMigrator(testOpts).migrate()).rejects.toThrow( - /use the X-Pack upgrade assistant/ - ); + await expect(new IndexMigrator(testOpts).migrate()).rejects.toThrow(); }); test('retains unknown core field mappings from the previous index', async () => { diff --git a/src/core/server/saved_objects/migrations/core/opensearch_index.ts b/src/core/server/saved_objects/migrations/core/opensearch_index.ts index 0fc03ccd05c0..7df5c3881cdf 100644 --- a/src/core/server/saved_objects/migrations/core/opensearch_index.ts +++ b/src/core/server/saved_objects/migrations/core/opensearch_index.ts @@ -311,7 +311,7 @@ function assertIsSupportedIndex(indexInfo: FullIndexInfo) { if (!isV7Index) { throw new Error( `Index ${indexInfo.indexName} belongs to a version of OpenSearch Dashboards ` + - `that cannot be automatically migrated. Reset it or use the X-Pack upgrade assistant.` + `that cannot be automatically migrated. Reset it.` ); } diff --git a/src/dev/notice/cli.js b/src/dev/notice/cli.js index f176d53c8fb1..2f8b01cd6c40 100644 --- a/src/dev/notice/cli.js +++ b/src/dev/notice/cli.js @@ -80,7 +80,7 @@ if (opts.help) { (async function run() { const path = resolve(REPO_ROOT, 'NOTICE.txt'); const newContent = await generateNoticeFromSource({ - productName: 'OpenSearch Dashboards source code with OpenSearch Dashboards X-Pack source code', + productName: 'OpenSearch Dashboards source code', directory: REPO_ROOT, log, }); diff --git a/src/plugins/apm_oss/server/tutorial/instructions/apm_server_instructions.ts b/src/plugins/apm_oss/server/tutorial/instructions/apm_server_instructions.ts index 3b95625e7063..fdfaa301b341 100644 --- a/src/plugins/apm_oss/server/tutorial/instructions/apm_server_instructions.ts +++ b/src/plugins/apm_oss/server/tutorial/instructions/apm_server_instructions.ts @@ -36,7 +36,7 @@ export const createEditConfig = () => ({ }), textPre: i18n.translate('apmOss.tutorial.editConfig.textPre', { defaultMessage: - "If you're using an X-Pack secured version of Elastic Stack, you must specify \ + "If you're using a secured version of OpenSearch, you must specify \ credentials in the `apm-server.yml` config file.", }), commands: [ diff --git a/src/plugins/data/README.md b/src/plugins/data/README.md index 383fd80a11d1..39f514e81514 100644 --- a/src/plugins/data/README.md +++ b/src/plugins/data/README.md @@ -111,7 +111,7 @@ The `SearchSource` API is a convenient way to construct and run an OpenSearch se #### Default Search Strategy -One benefit of using the low-level search API, is partial response support in X-Pack, allowing for a better and more responsive user experience. +One benefit of using the low-level search API is that it allows for a better and more responsive user experience with partial responses. In OSS only the final result is returned. ```.ts