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

[Reporting] Don't allow Reporting be completely disabled in configuration. #119914

Closed
tsullivan opened this issue Nov 29, 2021 · 3 comments
Closed
Labels
Breaking Change bug Fixes for quality problems that affect the customer experience (Deprecated) Feature:Reporting Use Reporting:Screenshot, Reporting:CSV, or Reporting:Framework instead impact:medium Addressing this issue will have a medium level of impact on the quality/strength of our product. loe:medium Medium Level of Effort needs-team Issues missing a team label

Comments

@tsullivan
Copy link
Member

tsullivan commented Nov 29, 2021

In order to support #118477, we have to make sure some parts of Reporting always run in Kibana. This is not possible if the user sets xpack.reporting.enabled: false in the kibana.yml.

Requirements:

As this is a breaking change, support for xpack.reporting.enabled: false will only go away at the point when we find that doing so will have low impact to users.

@tsullivan tsullivan added bug Fixes for quality problems that affect the customer experience Team:Reporting Services labels Nov 29, 2021
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-reporting-services (Team:Reporting Services)

@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-app-services (Team:AppServicesUx)

@exalate-issue-sync exalate-issue-sync bot added impact:needs-assessment Product and/or Engineering needs to evaluate the impact of the change. loe:small Small Level of Effort impact:high Addressing this issue will have a high level of impact on the quality/strength of our product. impact:medium Addressing this issue will have a medium level of impact on the quality/strength of our product. loe:medium Medium Level of Effort and removed impact:needs-assessment Product and/or Engineering needs to evaluate the impact of the change. impact:high Addressing this issue will have a high level of impact on the quality/strength of our product. loe:small Small Level of Effort labels Dec 1, 2021
@tsullivan
Copy link
Member Author

Rethinking this, we can come up with a plan that supports Reporting being disabled everywhere or enabled everywhere. Then the changes aren't breaking, since users legitimately disable reporting on every instance, and it doesn't create problems.

The problem to focus on is the unsupported environment where Reporting is disabled on some instances but enabled on others.

We should utilize #107678

Closing for #120995

@sophiec20 sophiec20 added the (Deprecated) Feature:Reporting Use Reporting:Screenshot, Reporting:CSV, or Reporting:Framework instead label Aug 21, 2024
@botelastic botelastic bot added the needs-team Issues missing a team label label Aug 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Breaking Change bug Fixes for quality problems that affect the customer experience (Deprecated) Feature:Reporting Use Reporting:Screenshot, Reporting:CSV, or Reporting:Framework instead impact:medium Addressing this issue will have a medium level of impact on the quality/strength of our product. loe:medium Medium Level of Effort needs-team Issues missing a team label
Projects
None yet
Development

No branches or pull requests

3 participants