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

Modify link to sample data in the section dashboards and references to opensearch dashboards #311

Conversation

chantal-kelm
Copy link
Member

@chantal-kelm chantal-kelm commented Sep 13, 2024

Description

Modify link to sample data in the section dashboards and references to Opensearch dashboards

Issues Resolved

#307

Evidence

Grabacion.de.pantalla.2024-09-13.a.la.s.3.20.54.p.m.mov

Test

In order to test this PR you must:

From the wazuh-dashboard environment:

  • Add this configuration to the wazuh.yml file: checks.template: false

  • You must also mount as volume the plugins inside your wazuh-dashboard container in the dev.yml file:

    • ${REPO_PATH}:/home/node/kbn/plugins/wazuh
    • ${REPO_PATH}/home/node/kbn/plugins/wazuh-core
    • ${REPO_PATH}/home/node/kbn/plugins/wazuh-check-updates

From wazuh-dashboard-plugins environments raise a rel.sh environment:

  • Set up a rel environment

  • Comment out the wazuh-dashboard container

  • Map the port in the container wazuh-manager:

    ports:
          - 55000:55000
    

Go to the side menu click on the Dashboards tab and verify:

  • Verify that the link with the message “Install some sample data” redirects to sample data

  • Verify that where it used to say Opensearch dashboards it now says Wazuh dashboards

Check List

  • All tests pass
    • yarn test:jest
    • yarn test:jest_integration
  • New functionality includes testing.
  • New functionality has been documented.
  • Update CHANGELOG.md
  • Commits are signed per the DCO using --signoff

@chantal-kelm chantal-kelm self-assigned this Sep 13, 2024
@JuanGarriuz JuanGarriuz self-requested a review September 17, 2024 06:48
CHANGELOG.md Outdated Show resolved Hide resolved
@JuanGarriuz
Copy link
Member

Test

image
image

JuanGarriuz
JuanGarriuz previously approved these changes Sep 17, 2024
Copy link
Member

@JuanGarriuz JuanGarriuz 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

@Desvelao Desvelao left a comment

Choose a reason for hiding this comment

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

LGTM!

@Desvelao Desvelao merged commit 30adcb1 into 4.9.1 Sep 17, 2024
22 of 25 checks passed
@Desvelao Desvelao deleted the bug/307-modify-link-to-sample-data-in-the-section-dashboards-and-references-to-opensearch-dashboards branch September 17, 2024 08:25
Desvelao pushed a commit that referenced this pull request Sep 23, 2024
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.

Modify link to sample data in Dashboards section and references to opensearch dashboards
3 participants