Relax allowed elasticsearch dependency version

[mjankowski]

Saw the comment here - #847 (comment) - which notes that newer 7.x versions of elasticsearch gem allow either Faraday 1.x or 2.x.

This change might be a good stopgap to include in a Chewy 7.x release while work on ES8 support (in Chewy 8) continues on that PR.

[mjankowski]

Any feedback on this direction?

[konalegi]

I'll take a loon on Monday

[konalegi]

yeah, let's try

[konalegi]

@mjankowski can you please rebase pr?

[mjankowski]

Rebased.

[konalegi]

I'm afraid we cannot simply bump the ES version. With elastic search 7.17 all specs are failing

[mjankowski]

Hmm, yeah. I had issues getting the full suite to run locally for master branch (ES issue, I think), so had not been able to correctly compare before. I sorted that out and now see passing master but same failures as CI with this PR branch. Looks like something changed with how you access the logger/tracer between versions 7.13.x and 7.14.x.

I just pushed another change which:

• Keeps the max version change from before
• Adds a new min version (7.14.0)
• Updates config code based on how access the logger and tracer values changed between 7.13.x and 7.14.x

This passes for me locally.

[konalegi]

Specs are passing, let me try it in our internal projects, if specs there will pass as well, I'll release it. Thanks for the change btw

[konalegi]

yep, works fine, please update changelog as here https://github.com/toptal/chewy/pull/925/files#diff-06572a96a58dc510037d5efa622f9bec8519bc1beab13c9f251e97e657a9d4edR7

[mjankowski]

Pushed changelog commit.

[konalegi]

Thank you! Merging!