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 elasticsearch interface #59

Merged
merged 1 commit into from
Mar 25, 2019
Merged

Conversation

PrasadG193
Copy link
Collaborator

Fixes: #53

ELS configuration structure:

  communications:
    # Settings for ELS
    elasticsearch:
      enable: false
      server: 'ELASTICSEARCH_ADDRESS'              # e.g https://example.com:9243
      username: 'ELASTICSEARCH_USERNAME'
      password: 'ELASTICSEARCH_PASSWORD'
      # ELS index settings
      index:
        name: botkube
        type: botkube-event
        shards: 1
        replicas: 0

Helm command to deploy botkube with ELS support:

helm install --name botkube --namespace botkube \
--set config.settings.clustername=dev-gke \
--set config.settings.allowkubectl=true \
--set config.communications.elasticsearch.enable=true \
--set config.communications.elasticsearch.server=ELS_ADDRESS \
--set config.communications.elasticsearch.username=ELS_USERNAME \
--set config.communications.elasticsearch.password=ELS_PASSWORD \
helm/botkube/

@PrasadG193
Copy link
Collaborator Author

image

@sanketsudake
Copy link
Contributor

@PrasadG193 Please make sure we also update documentation before this merge to master

@sanketsudake sanketsudake merged commit 943dd63 into develop Mar 25, 2019
@PrasadG193 PrasadG193 deleted the add-els-support branch April 4, 2019 14:50
@write2abhi
Copy link

write2abhi commented Jun 28, 2019

How can we create indexes on a daily basis. Because It will generate a lot of data and we need to clear the data on daily basis?

@killcity
Copy link

Anyone else notice the cluster field not being populated in ES, but is correctly populated when calling Slack?

@codenio
Copy link
Contributor

codenio commented Aug 24, 2019

cluster field not being populated in ES

@killcity thanks for reporting this issue.
#164 fixes this.

@killcity
Copy link

@aananthraj Thanks!

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.

Add ElasticSearch support
5 participants