Skip to content

add note to use 127.0.0.1 when connecting to Kibana docker container #2091

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 3 commits into from
May 2, 2022
Merged

add note to use 127.0.0.1 when connecting to Kibana docker container #2091

merged 3 commits into from
May 2, 2022

Conversation

@DanRoscigno
Copy link
Contributor

@DanRoscigno DanRoscigno commented Mar 30, 2022

The Kibana docker image provides a URL to connect to to continue the integration with Elasticsearch. It uses 0.0.0.0 as the kibana.yml specifies to listen on all interfaces. In order to connect to the container we generally will need to use 127.0.0.1.

@mergify
Copy link
Contributor

mergify bot commented Mar 30, 2022

This pull request does not have a backport label. Could you fix it @DanRoscigno? 🙏
To fixup this pull request, you need to add the backport labels for the needed
branches, such as:

  • backport-/d./d is the label to automatically backport to the /d./d branch. /d is the digit
    NOTE: backport-skip has been added to this pull request.

@mergify mergify bot added the backport-skip Skip automated backport with mergify label Mar 30, 2022
@DanRoscigno DanRoscigno added the backport-8.1 Automated backport with mergify label Mar 30, 2022
@DanRoscigno DanRoscigno requested review from a team and lcawl March 30, 2022 16:41
@mergify mergify bot removed the backport-skip Skip automated backport with mergify label Mar 30, 2022
debadair
debadair approved these changes Mar 30, 2022
View reviewed changes
Copy link
Contributor

@debadair debadair left a comment

Choose a reason for hiding this comment

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

LGTM

@DanRoscigno DanRoscigno enabled auto-merge (squash) May 2, 2022 11:03
@DanRoscigno DanRoscigno merged commit c97f29f into elastic:main May 2, 2022
mergify bot pushed a commit that referenced this pull request May 2, 2022
@DanRoscigno @mergify
add note to use 127.0.0.1 (#2091)
ba850db 
(cherry picked from commit c97f29f)
@mergify mergify bot mentioned this pull request May 2, 2022
Merged
DanRoscigno pushed a commit that referenced this pull request May 2, 2022
@mergify
add note to use 127.0.0.1 (#2091) (#2122)
150e05a 
(cherry picked from commit c97f29f)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@lcawl lcawl Awaiting requested review from lcawl

1 more reviewer

@debadair debadair debadair approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

backport-8.1 Automated backport with mergify

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@DanRoscigno @debadair