-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Data Node Migration: remove unnecessary steps/big texts/unavailable paths #19312
Conversation
</p> | ||
<p> | ||
Enable JWT authentication in <code>opensearch-security/config.yml</code> (section <code>jwt_auth_domain</code>, <code>http_enabled: true</code>, <code>transport_enabled: true</code>) | ||
Add the signing key by converting the <code>GRAYLOG_PASSWORD_SECRET</code> to <code>base64</code> e.g. by doing <code>echo `"`The password secret you chose`"` | base64</code> and put it into the <code>signing_key</code> line | ||
If you are running <code>ElasticSearch</code> as your search backend or want to selectively migrate data |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since we do not give the user a choice if he's running Elastic, we should maybe reword that here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Frontend looks good !
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested locally, looks good to me! Thanks!
Dismissing review because reviewer is absent. Requested by @gally47.
…aths (#19312) * remove unnecessary texts + rewording * changelog * remove Certificate provisioning overview step in in-place migration * remove Provision Data Node with certificates step from remote reindexing migration * remove unused MigrationStates * add guard for disabling in-place migration for ElasticSearch * add guard for disabling in-place migration for ElasticSearch * sort indices for remote reindexing * stop fetching CurrentMigrationStatus when error * clear interval and cleanup logs when error * RetryMigrateExistingData ConfirmDialog when no errors + reordered buttons position * add default elasticsearch_hosts and allowlist to state machine response * fix buttons order * Sort Next Steps * cleanup comments * MigrateExistingData set initial values elasticsearch_hosts and allowlist_hosts * filter migrationTypeOptions show only possible paths * skip migration selection page when not compatible * added JwtAuthenticationInfo * remove prefilled host/allowlist values * reword case ElasticSearch as search backend * remove unreachable steps * fix tests --------- Co-authored-by: Matthias Oesterheld <[email protected]> Co-authored-by: Matthias Oesterheld <[email protected]> Co-authored-by: Tomas Dvorak <[email protected]>
…aths (#19312) * remove unnecessary texts + rewording * changelog * remove Certificate provisioning overview step in in-place migration * remove Provision Data Node with certificates step from remote reindexing migration * remove unused MigrationStates * add guard for disabling in-place migration for ElasticSearch * add guard for disabling in-place migration for ElasticSearch * sort indices for remote reindexing * stop fetching CurrentMigrationStatus when error * clear interval and cleanup logs when error * RetryMigrateExistingData ConfirmDialog when no errors + reordered buttons position * add default elasticsearch_hosts and allowlist to state machine response * fix buttons order * Sort Next Steps * cleanup comments * MigrateExistingData set initial values elasticsearch_hosts and allowlist_hosts * filter migrationTypeOptions show only possible paths * skip migration selection page when not compatible * added JwtAuthenticationInfo * remove prefilled host/allowlist values * reword case ElasticSearch as search backend * remove unreachable steps * fix tests --------- Co-authored-by: Matthias Oesterheld <[email protected]> Co-authored-by: Matthias Oesterheld <[email protected]> Co-authored-by: Tomas Dvorak <[email protected]>
…aths (#19312) * remove unnecessary texts + rewording * changelog * remove Certificate provisioning overview step in in-place migration * remove Provision Data Node with certificates step from remote reindexing migration * remove unused MigrationStates * add guard for disabling in-place migration for ElasticSearch * add guard for disabling in-place migration for ElasticSearch * sort indices for remote reindexing * stop fetching CurrentMigrationStatus when error * clear interval and cleanup logs when error * RetryMigrateExistingData ConfirmDialog when no errors + reordered buttons position * add default elasticsearch_hosts and allowlist to state machine response * fix buttons order * Sort Next Steps * cleanup comments * MigrateExistingData set initial values elasticsearch_hosts and allowlist_hosts * filter migrationTypeOptions show only possible paths * skip migration selection page when not compatible * added JwtAuthenticationInfo * remove prefilled host/allowlist values * reword case ElasticSearch as search backend * remove unreachable steps * fix tests --------- Co-authored-by: Matthias Oesterheld <[email protected]> Co-authored-by: Matthias Oesterheld <[email protected]> Co-authored-by: Tomas Dvorak <[email protected]>
In this PR we:
Motivation and Context
fixes #19229
Types of changes
Checklist: