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.

Sign up for GitHub

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

Jump to bottom

Run Tests against ES 6 #33671

Draft
wants to merge 31 commits into
base: master
Choose a base branch
from
Draft

Run Tests against ES 6 #33671

wants to merge 31 commits into from

Conversation

AmitPhulera
Copy link
Contributor

@AmitPhulera AmitPhulera commented Oct 26, 2023 •
edited
Loading

Product Description

Opening up for tests

@dimagimon dimagimon added the dependencies Pull requests that update a dependency file label Oct 26, 2023
@AmitPhulera
add es6 client support in elasticsearch.py
fe99590
@AmitPhulera
update hq compose to depend on elasticsearch6
ff98e92
@AmitPhulera
fix removed feature
d69670c 
https://www.elastic.co/guide/en/elasticsearch/reference/6.8/breaking-changes-6.0.html\#_indices_information_apis
@AmitPhulera
fix urllib warning
3a776b9
@AmitPhulera
replace string with keyword in test adapter
e16512e
@AmitPhulera
Merge remote-tracking branch 'origin/master' into ap/es6/test-run
a898556
@AmitPhulera
remove include_in_all from user_mappings as it is not supported to ha…
159acd6 
…ve this field in multi fields
@AmitPhulera
replace string -> keyword in remaining tests
b6424f4
@AmitPhulera
pass scroll_id as kwargs
cd4a5c4
@AmitPhulera
get_task supports ES 6
114ea36
@AmitPhulera
pass kwargs in put_mappings instead of args
d0315ac 
The default order of args was changed in ES 6 python client, so to ensure we don't face this issue in future we will pass kwargs
@AmitPhulera
make index_close exception test version aware
c721a2d 
https://www.elastic.co/guide/en/elasticsearch/reference/6.8/breaking-changes-6.0.html\#_indexclosedexception_to_return_400_status_code
@AmitPhulera
index_already_exists_exception was renamed to resource_already_exists…
34e3c99 
…_exception

elastic/elasticsearch#21494
@AmitPhulera
es 6 fixes the error message
ef5b7d5 
I was not able to find the place where it was changed
@AmitPhulera
if _meta is not passed in mappings its values are not updated
36c37d7
@AmitPhulera
remove string from test index mappings
4f62109
@AmitPhulera
lint
0d9d615
@AmitPhulera
add getMillis to get the milliseconds
3e77edf
@AmitPhulera
Merge remote-tracking branch 'origin/master' into ap/es6/test-run
e0ba0bb
@AmitPhulera
Merge remote-tracking branch 'origin/master' into ap/es6/test-run
fc4efee
@AmitPhulera
merge commit
8efdeb4
@AmitPhulera
add back gthe es6 package in base requirements
3f15890
@AmitPhulera
Merge branch 'master' into ap/es6/test-run
58172db
@AmitPhulera
Merge remote-tracking branch 'origin/master' into ap/es6/test-run
c32328a
@AmitPhulera AmitPhulera mentioned this pull request Sep 4, 2024
Merged
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@millerdev millerdev Awaiting requested review from millerdev millerdev will be requested when the pull request is marked ready for review millerdev is a code owner

@esoergel esoergel Awaiting requested review from esoergel esoergel will be requested when the pull request is marked ready for review esoergel is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@AmitPhulera @dimagimon