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 timelion to kibana core #7994

Merged
merged 469 commits into from
Sep 2, 2016
Merged

Conversation

rashidkpc
Copy link
Contributor

This adds the, currently external, elastic/timelion plugin to the Kibana core.

Rashid Khan and others added 30 commits February 18, 2016 16:40
@rashidkpc
Copy link
Contributor Author

rashidkpc commented Aug 29, 2016

Ok, definitely not this pull causing these failures. Master has been failing with the same issue: https://kibana-ci.elastic.co/job/elastic+kibana+master+multijob-intake/133/console

@epixa
Copy link
Contributor

epixa commented Aug 30, 2016

@rashidkpc Can you rebase or merge master into this? I'd like to see if the upgrade to karma and node have improved the situation.

@rashidkpc
Copy link
Contributor Author

@epixa done. Also added the .log() function from upstream elastic/timelion

@rashidkpc
Copy link
Contributor Author

Look, I fixed it.

@karmi
Copy link

karmi commented Sep 2, 2016

Hi @rashidkpc, we have found your signature in our records, but it seems like you have signed with a different e-mail than the one used in yout Git commit. Can you please add both of these e-mails into your Github profile (they can be hidden), so we can match your e-mails to your Github profile?

@epixa
Copy link
Contributor

epixa commented Sep 2, 2016

Damn, I can't get this running locally without getting [error][status][ui settings] Status changed from green to red - Could not find user-provided settings on startup. Trying to dig around now

@rashidkpc
Copy link
Contributor Author

Strange haven't seen that one.

@epixa
Copy link
Contributor

epixa commented Sep 2, 2016

It's not happening on master, but I can reproduce it consistently with this PR whenever Elasticsearch is new (so there is no kibana index). Timelion appears to still be functional, so I'm not sure what the downside is, but we obviously don't want startup errors in the first time use case.

Other than that, the only other thing I noticed is the logo replacement: can you update it so Timelion is not replacing the Kibana logo? Since the logo/name are highlighted in the side nav, no core applications are replacing the Kibana logo anymore.

@epixa
Copy link
Contributor

epixa commented Sep 2, 2016

It actually sounds like Spencer knows what the deal is here, so I'm chatting with him about it.

@epixa
Copy link
Contributor

epixa commented Sep 2, 2016

@rashidkpc Are you retrieving and setting advanced settings during plugin initialization? The advanced settings thing really doesn't support that in its current form.

So right now, this is technically at risk of not working during any startup since it's a timing issue, but it tends to only manifest during the initial startup since the kibana index is being created. The end result of the failure is that the timelion advanced settings do not get added on initial start, and then if you restart kibana they get added.

If you really want the configuration accessible in advanced settings, then the only real option at the moment is to add them directly to the application-wide defaults: https://github.com/elastic/kibana/blob/master/src/ui/settings/defaults.js

As an alternative, I propose adding them to kibana.yml instead. If I understand correctly, they were in timelion.json before anyway, so it has traditionally been static file based configuration.

@rashidkpc
Copy link
Contributor Author

rashidkpc commented Sep 2, 2016

@epixa I've already added them to defaults.js a1f5434

Edit: I removed the old config population stuff anyway.

@epixa
Copy link
Contributor

epixa commented Sep 2, 2016

That did it!

@epixa
Copy link
Contributor

epixa commented Sep 2, 2016

LGTM

I don't want to sweat over things like style for a separate plugin of this size. We can always address anything like that later.

@rashidkpc rashidkpc merged commit 6aa8ca4 into elastic:master Sep 2, 2016
@ddavidebor
Copy link

ddavidebor commented Sep 4, 2016

@rashidkpc should we continue to use your timelion-random example plugin to develop?

@rashidkpc
Copy link
Contributor Author

@ddavidebor Do continue to use the example, though you'll need to update the require paths (eg, to alter.js) in the examples. Once there's a release with timelion in it I'll update the examples

@tbragin tbragin added the Feature:Timelion Timelion app and visualization label Oct 7, 2016
airow pushed a commit to airow/kibana that referenced this pull request Feb 16, 2017
Add timelion to kibana core

Former-commit-id: 6aa8ca4
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:Timelion Timelion app and visualization v5.0.0-beta1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet