Skip to content

Conversation

@lukas-vlcek
Copy link
Contributor

Custom SG build found at https://bintray.com/artifact/download/lukas-vlcek/Search-Guard-Custom/search-guard-0.5.1.zip is based on https://github.com/floragunncom/search-guard/pull/85

Basic smoke test performed using @richm script.

Before the script was executed I setup a few evn variables:

export GITHUB_REPO=lukas-vlcek
export GITHUB_BRANCH=sg-update
export OS_VOL_DIR_BASE=/home/vagrant/logging

In the end the script executes several queries against ES node which includes SG logic.

When elasticsearch pod is build the following can be seen in the log:

+ /usr/share/elasticsearch/bin/plugin -i search-guard -url https://bintray.com/artifact/download/lukas-vlcek/Search-Guard-Custom/search-guard-0.5.1.zip
-> Installing search-guard...
Trying https://bintray.com/artifact/download/lukas-vlcek/Search-Guard-Custom/search-guard-0.5.1.zip...
Downloading ..........................................................DONE
Installed search-guard into /usr/share/elasticsearch/plugins/search-guard

And we can also check installed plugins using:

$ curl -s -k --cert ./cert --key ./key 'https://localhost:9200/_cat/plugins?v'
name               component                      version            type url 
Master Pandemonium cloud-kubernetes               1.3.0              j        
Master Pandemonium searchguard                    0.5.1              j        
Master Pandemonium openshift-elasticsearch-plugin ${project.version} j    

@openshift-bot openshift-bot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Mar 4, 2016
@lukas-vlcek lukas-vlcek force-pushed the sg-update branch 2 times, most recently from f4bef90 to 8668759 Compare March 7, 2016 15:04
@lukas-vlcek
Copy link
Contributor Author

@ewolinetz ewolinetz removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Mar 7, 2016
@ewolinetz
Copy link
Contributor

[test]

@richm
Copy link
Contributor

richm commented Mar 8, 2016

Please rebase this PR to the latest master branch

@openshift-bot openshift-bot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Mar 16, 2016
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you use the following plugin name to patch the previous formatting? com.floragunn/search-guard/0.5.1

@ewolinetz ewolinetz removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Mar 17, 2016
@lukas-vlcek lukas-vlcek force-pushed the sg-update branch 2 times, most recently from 7ddc846 to 63148c7 Compare March 17, 2016 13:59
@openshift-bot
Copy link

Evaluated for aggregated logging test up to cbf27c5

@ewolinetz
Copy link
Contributor

pending tests LGTM

@openshift-bot
Copy link

Aggregated Logging Test Results: SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test-origin-aggregated-logging/46/)

ewolinetz added a commit that referenced this pull request Mar 17, 2016
Upgrade to custom build of Seach-Guard 0.5.1
@ewolinetz ewolinetz merged commit aad1232 into openshift:master Mar 17, 2016
@lukas-vlcek lukas-vlcek deleted the sg-update branch March 18, 2016 10:08
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.

4 participants