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

Update_kibana_full_to_use_new_homebrew_tap #152

Merged
merged 2 commits into from
Mar 21, 2024

Conversation

Step-henC
Copy link
Contributor

Attempting to download kibana for elasticsearch using homebrew on an M1 mac resulted in this error: Calling plist_options is disabled! Use service.require_root instead. Please report this issue to the elastic/tap tap (not Homebrew/brew or Homebrew/homebrew-core), or even better, submit a PR to fix it: /opt/homebrew/Library/Taps/elastic/homebrew-tap/Formula/kibana-full.rb:50

This is the PR to fix it, as suggested by the prompt. The fix is simply substituting plist_options with service.require_root

This PR is inspired by a similar PR for mongoDb https://github.com/mongodb/homebrew-brew/pull/176/files

Kibana seems like a really cool visualization tool for my performance testing results and I want myself and future users to be able to leverage this tool.

@Step-henC
Copy link
Contributor Author

Confirmed it work by manually pulling local changes from this branch using the following commands:

Which brew

cd /opt/homebrew/Library/Taps/elastic/homebrew-tap

git fetch origin pull/152/head:update-07212023

git checkout update-07212023

Then installed and ran kibana on port 5601 using these commands in the terminal

brew tap elastic/tap

brew install elastic/tap/kibana-full

brew services start elastic/tap/kibana-full

to stop use brew services stop elastic/tap/kibana-full

@audreyanngoing
Copy link

+1 I ran into the same issue and need this as well!

@Step-henC
Copy link
Contributor Author

If you want a quick fix @audreyanngoing you can perform the steps above to get the local fix

Copy link
Member

@mgreau mgreau left a comment

Choose a reason for hiding this comment

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

LGTM, thanks

@mgreau mgreau merged commit a157ed2 into elastic:main Mar 21, 2024
mgreau added a commit that referenced this pull request Mar 21, 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.

3 participants