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

[Uptime] Uptime index config using kibana.yml #115775

Merged
merged 6 commits into from
Oct 27, 2021

Conversation

shahzad31
Copy link
Contributor

@shahzad31 shahzad31 commented Oct 20, 2021

Summary

Fix #115774

Now you can config uptime index settings using kibana.yml key

xpack.uptime.index: remote_cluster:heartbeat-*

This will allow robots team to config ccs with new olbt-cli setup

Testing

set the key for above config and check that when uptime settings page loads,

it show that key

image

but when user override it, saved objects uptime settings should take precedence

@shahzad31 shahzad31 requested review from a team as code owners October 20, 2021 14:05
@botelastic botelastic bot added the Team:Uptime - DEPRECATED Synthetics & RUM sub-team of Application Observability label Oct 20, 2021
@elasticmachine
Copy link
Contributor

Pinging @elastic/uptime (Team:uptime)

Copy link
Member

@jbudz jbudz left a comment

Choose a reason for hiding this comment

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

kibana-docker LGTM

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

Copy link
Contributor

@justinkambic justinkambic left a comment

Choose a reason for hiding this comment

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

I encountered an error after setting this. Is there some additional configuration I need to do in order to get this working?

image

Saw error logs like this in Kibana server log:

[2021-10-20T16:37:30.230-04:00][ERROR][http] ResponseError: security_exception: [no_such_remote_cluster_exception] Reason: no such remote cluster: [remote_cluster]

@shahzad31
Copy link
Contributor Author

I encountered an error after setting this. Is there some additional configuration I need to do in order to get this working?

image

Saw error logs like this in Kibana server log:

[2021-10-20T16:37:30.230-04:00][ERROR][http] ResponseError: security_exception: [no_such_remote_cluster_exception] Reason: no such remote cluster: [remote_cluster]

@justinkambic no, it should just work, what key you were using?

@shahzad31
Copy link
Contributor Author

@elasticmachine merge upstream

@justinkambic
Copy link
Contributor

@shahzad31 still seeing that error, my only config entry is:

xpack.uptime.index: remote_cluster:heartbeat-*

We can pair if you want to see, not sure how to debug.

@shahzad31
Copy link
Contributor Author

@shahzad31 still seeing that error, my only config entry is:

xpack.uptime.index: remote_cluster:heartbeat-*

We can pair if you want to see, not sure how to debug.

I actually had remote_cluster: configured using olbt-cli, in your case, you just need to add your heartbeat index pattern like heartbeat-*

@shahzad31
Copy link
Contributor Author

@elasticmachine merge upstream

@justinkambic
Copy link
Contributor

Ok, this works in that case. We may want to create a follow-up to improve error handling where CCS is not configured. Lots of nasty error messages show up in the Kibana log for unrelated components of the app.

Copy link
Contributor

@justinkambic justinkambic left a comment

Choose a reason for hiding this comment

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

LGTM. One nit comment.

@@ -32,6 +33,10 @@ export class Plugin implements PluginType {
}

public setup(core: CoreSetup, plugins: UptimeCorePlugins) {
const config = this.initContext.config.get<UptimeConfig>();
Copy link
Contributor

Choose a reason for hiding this comment

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

We can probably skip this declaration, since we're only using the symbol one time right below here.

@kibanamachine
Copy link
Contributor

💛 Build succeeded, but was flaky

Metrics [docs]

✅ unchanged

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@shahzad31 shahzad31 added the auto-backport Deprecated - use backport:version if exact versions are needed label Oct 27, 2021
@shahzad31 shahzad31 merged commit 98231f4 into elastic:master Oct 27, 2021
@shahzad31 shahzad31 deleted the uptime-index-config branch October 27, 2021 08:15
@kibanamachine
Copy link
Contributor

💔 Backport failed

The backport operation could not be completed due to the following error:
There are no branches to backport to. Aborting.

The backport PRs will be merged automatically after passing CI.

To backport manually run:
node scripts/backport --pr 115775

@shahzad31 shahzad31 added v7.16.0 and removed auto-backport Deprecated - use backport:version if exact versions are needed labels Oct 27, 2021
@shahzad31 shahzad31 added auto-backport Deprecated - use backport:version if exact versions are needed and removed auto-backport Deprecated - use backport:version if exact versions are needed v7.16.0 labels Oct 27, 2021
kibanamachine added a commit to kibanamachine/kibana that referenced this pull request Oct 27, 2021
Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
@kibanamachine
Copy link
Contributor

💚 Backport successful

Status Branch Result
7.16

This backport PR will be merged automatically after passing CI.

jloleysens added a commit to jloleysens/kibana that referenced this pull request Oct 27, 2021
…-migrate-away-from-injected-css-js

* 'master' of github.com:elastic/kibana: (61 commits)
  [ML] Nodes overview for the Model Management page (elastic#116361)
  [Uptime] Uptime index config using kibana.yml (elastic#115775)
  [Controls] Dashboard Integration (elastic#115991)
  skip flaky suite (elastic#104260)
  Include Files in GitHub UI (elastic#115956)
  skip flaky suite (elastic#116060)
  [Canvas] By-Value Embeddables (elastic#113827)
  Skip failing test (elastic#115366)
  [Osquery] Fix live query search doesn't return relevant results for agents (elastic#116332)
  [Integrations] Added link in old Add Data description and fixed alignment in cards (elastic#116213)
  [Actions] Extended ActionTypeRegistry with connector validation to validate config with secrets (elastic#116079)
  skip flaky suite (elastic#109329)
  Grant access to machine learning features when base privileges are used (elastic#115444)
  Skipping failing test (elastic#84957)
  [RAC][Security Solution] Adds migration to new SecuritySolution rule types (elastic#112113)
  skip flaky suite (elastic#115366)
  [Fleet] Marking API spec as experimental (elastic#116331)
  [Docs] Cleaning up the versions in the upgrade paths. Closes elastic#116223 (elastic#116228)
  [Reporting] Suppress debug logs in the mock logger (elastic#116012)
  [Metrics UI] Clear threshold alert groups state when filterQuery changes (elastic#116205)
  ...

# Conflicts:
#	src/plugins/dashboard/public/application/embeddable/dashboard_container.tsx
#	src/plugins/dashboard/public/types.ts
@kibanamachine
Copy link
Contributor

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create backports run node scripts/backport --pr 115775 or prevent reminders by adding the backport:skip label.

@kibanamachine kibanamachine added the backport missing Added to PRs automatically when the are determined to be missing a backport. label Oct 28, 2021
@shahzad31 shahzad31 added the backport:skip This commit does not require backporting label Oct 28, 2021
@kibanamachine kibanamachine removed the backport missing Added to PRs automatically when the are determined to be missing a backport. label Oct 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:skip This commit does not require backporting release_note:enhancement Team:Uptime - DEPRECATED Synthetics & RUM sub-team of Application Observability v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Uptime] Configure index patterns on the kibana.yml file
5 participants