Skip to content

Cherry-pick to 7.10: Add add_observer_metadata geo.name to Quickstart (#21501)#21640

Merged
EamonnTP merged 1 commit intoelastic:7.10from
EamonnTP:backport/7.10/pr-21501
Oct 14, 2020
Merged

Cherry-pick to 7.10: Add add_observer_metadata geo.name to Quickstart (#21501)#21640
EamonnTP merged 1 commit intoelastic:7.10from
EamonnTP:backport/7.10/pr-21501

Conversation

@EamonnTP
Copy link
Copy Markdown

@EamonnTP EamonnTP commented Oct 7, 2020

Backports the following commits to 7.10:

* Add `add_observer_metadata` `geo.name` to Quickstart

The observer location is very important in the Uptime app and the out-of-the-box machine learning job.

* Update getting-started.asciidoc

The instructions about editing heartbeat.yml were too wordy

* tighten it up!

Co-authored-by: DeDe Morton <dede.morton@elastic.co>

Co-authored-by: DeDe Morton <dede.morton@elastic.co>
@EamonnTP EamonnTP requested a review from a team as a code owner October 7, 2020 14:37
@botelastic botelastic bot added the needs_team Indicates that the issue/PR needs a Team:* label label Oct 7, 2020
@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Pipeline View Test View Changes Artifacts preview

Expand to view the summary

Build stats

  • Build Cause: [Pull request #21640 opened]

  • Start Time: 2020-10-07T14:38:23.472+0000

  • Duration: 25 min 36 sec

Copy link
Copy Markdown
Contributor

@dedemorton dedemorton left a comment

Choose a reason for hiding this comment

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

LGTM

@EamonnTP
Copy link
Copy Markdown
Author

@elastic/uptime, I can't merge until one from the team approves this. When you have time, can someone take a look?

For some reason, I could merge the other backports, but not this one.

@bmorelli25
Copy link
Copy Markdown
Member

For some reason, I could merge the other backports, but not this one.

Hmmm. I ran into the same problem on another PR with 7.10 and the Uptime team.

@bmorelli25 bmorelli25 added the Team:Docs Label for the Observability docs team label Oct 13, 2020
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/obs-docs (Team:Docs)

@botelastic botelastic bot removed the needs_team Indicates that the issue/PR needs a Team:* label label Oct 13, 2020
@EamonnTP
Copy link
Copy Markdown
Author

Hi @andrewvc. If you have time today, could you take a look at this PR? Thanks!

@andresrc
Copy link
Copy Markdown
Contributor

@EamonnTP this build went ok, so I think we are good to go

@EamonnTP EamonnTP merged commit 5c85676 into elastic:7.10 Oct 14, 2020
@EamonnTP EamonnTP deleted the backport/7.10/pr-21501 branch October 14, 2020 13:14
@EamonnTP
Copy link
Copy Markdown
Author

Thanks @andresrc

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport Team:Docs Label for the Observability docs team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants