permit index-pattern only setup #7285
Conversation
with setup.dashboards.enabled=true, setup.dashboards.only_index=true always throws "Error importing Kibana dashboards: fail to import the dashboards in Kibana"
|
@graphaelli Any chance you could add a system test for this? There are a few tests for dashboard loading where we could add this. As this sounds like a bug fix, worth a changelog entry? |
graphaelli
force-pushed
the
import-only-index-pattern
branch
from
9c6e1f8 to
c005619
Compare
|
|
||
| beat.check_wait(exit_code=0) | ||
|
|
||
| assert self.log_contains("Kibana dashboards successfully loaded") is True |
There was a problem hiding this comment.
Shouldn't we have a log message which reads more like index pattern successfully loaded?
There was a problem hiding this comment.
I agree it's confusing that the loading index patterns and dashboard are conflated but I think it would be best to tackle that as an enhancement rather than in this bugfix, that I'm hoping we can backport to 6.2
There was a problem hiding this comment.
Got it, agree. I added the needs_backport label. Could you open a follow up PR with the change mentioned above?
There was a problem hiding this comment.
Will do, thanks @ruflin
ruflin
added
the
needs_backport
ruflin
added
v6.3.1
and removed
needs_backport
with setup.dashboards.enabled=true, setup.dashboards.only_index=true always throws "Error importing Kibana dashboards: fail to import the dashboards in Kibana" (cherry picked from commit ee0dc3e)
with setup.dashboards.enabled=true, setup.dashboards.only_index=true always throws "Error importing Kibana dashboards: fail to import the dashboards in Kibana" (cherry picked from commit 4d678b4)
with
setup.dashboards.enabled=true,setup.dashboards.only_index=truealways throws"Error importing Kibana dashboards: fail to import the dashboards in Kibana"