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

/i/index/refresh : Refresh additional index on same ElasticSearch #4414

Merged
merged 1 commit into from
Jan 21, 2025

Conversation

supersven
Copy link
Contributor

This case was missing: The additional index can reside on the same ElasticSearch instance as the normal/default one.

Ticket: https://wearezeta.atlassian.net/browse/WPB-15437

Checklist

@zebot zebot added the ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist label Jan 20, 2025
This case was missing: The additional index can reside on the same
ElasticSearch instance as the normal/default one.
@supersven supersven force-pushed the sventennie/deflake-index-migration-tests-2 branch from 792bff1 to 0396d26 Compare January 20, 2025 12:57
@supersven
Copy link
Contributor Author

These tests are green on CI. Unfortunately, some other flakes are failing. Merging this PR to improve the overall situation.

@supersven supersven merged commit ca0688b into develop Jan 21, 2025
9 of 11 checks passed
@supersven supersven deleted the sventennie/deflake-index-migration-tests-2 branch January 21, 2025 08:30
@echoes-hq echoes-hq bot added echoes: technical-roadmap/security More specific category, to highlight task that tackle security requirements. echoes: technical-roadmap/technical-debt More specific category, to highlight Technical Debt being tackled. labels Jan 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
echoes: technical-roadmap/security More specific category, to highlight task that tackle security requirements. echoes: technical-roadmap/technical-debt More specific category, to highlight Technical Debt being tackled. ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants