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

Unattended changes the login screen in kibana [unified] #1033

Merged
merged 4 commits into from
Dec 14, 2021

Conversation

verdx
Copy link
Contributor

@verdx verdx commented Nov 29, 2021

Related issue
closes #943

Description

Before starting the kibana service we use the same code used previously in the ova to change the kibana interface parameters and files. When kibana is started, the wazuh logo is shown, instead of the opendistro one.

Logs example

Tests

Login screen after all-in-one installation in centos 7:
custom_login_centos7

The all-in-one installation runs correctly in Centos 7 and debian 9

@verdx verdx self-assigned this Nov 29, 2021
@verdx verdx force-pushed the 982-custom-login-unified branch from 5b29025 to 5bdb75c Compare December 1, 2021 12:44
@verdx verdx force-pushed the 982-custom-login-unified branch from 4bb1bc6 to a643a76 Compare December 7, 2021 11:05
@verdx verdx requested review from rauldpm, DFolchA, okynos and myu1d157h0u54nd and removed request for rauldpm and DFolchA December 7, 2021 12:50
Copy link
Contributor

@myu1d157h0u54nd myu1d157h0u54nd left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@rauldpm rauldpm left a comment

Choose a reason for hiding this comment

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

GJ, just a minor change

unattended_scripts/install_functions/opendistro/kibana.sh Outdated Show resolved Hide resolved
Copy link
Contributor

@alberpilot alberpilot left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@rauldpm rauldpm left a comment

Choose a reason for hiding this comment

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

LGTM

@alberpilot alberpilot merged commit 1804d0d into unify-unattended Dec 14, 2021
@alberpilot alberpilot deleted the 982-custom-login-unified branch December 14, 2021 15:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants