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

Add support for ES/Shield 2.3.3 #1

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

timgluz
Copy link

@timgluz timgluz commented Dec 25, 2016

Hi,

here's solutions for the the issue #226, which adds support for Elastic Shield.

This PR includes drop-in clients for Rest and Native transport, tests and functional configuration for TravisCI.

There are still some shortcomings:

  • it uses timgluz/elastisch:3.0.0-beta2 as dev dependency, which was built from the latest master branch to make all the test to work without changing tests and falling back to older rest-client;

  • only doc & usage example is the README;

  • i didn't have time to test support for SSL;

I added Makefile to make it easier to set up Elasticsearch instance with the shield support:

cd resources
make build
make run
docker ps
curl -u es_admin:toor123 127.0.0.1:9200
lein with-profile dev,master test

Have a nice holidays!

use shield rest client in the test for rest aggregation;

use shield client in rest query tests;

filter out failing rest specs for shield client;

fix the rest-api.delete-test;

fix rest-api.get-test when indexes were missing;

use shield.fixtures instead fixture namespace from Elastisch;

fix rest-api index tests as dynamic index isnot default anymore;

fix rest-api/bulk-test by initializing index before tests;

make test for native client to run with shield client;

set up Travis CI with dockerized ES;

fix tab issues in travis config;

add missing service to allow docker in TravisCI;

cleanups; update readme and changelogs;
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant