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

Update latest environment to 6.5.0 #9146

Merged
merged 2 commits into from
Nov 22, 2018
Merged

Update latest environment to 6.5.0 #9146

merged 2 commits into from
Nov 22, 2018

Conversation

ruflin
Copy link
Member

@ruflin ruflin commented Nov 19, 2018

No description provided.

Copy link
Contributor

@ph ph left a comment

Choose a reason for hiding this comment

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

@ruflin 6.5.1 instead of 6.5.0?

@ycombinator
Copy link
Contributor

Would it make sense to update

FROM docker.elastic.co/kibana/kibana:6.4.3
and
FROM docker.elastic.co/elasticsearch/elasticsearch:6.4.3
as part of this PR as well?

@ruflin
Copy link
Member Author

ruflin commented Nov 21, 2018

@ph That is what happens if a PR is open for too long :-)

@ycombinator I would treat the 2 separately. These are the ones used for module testing and should be adjusted by the module "owner". The ones I adjusted are for overall testing / manual testing.

@ycombinator
Copy link
Contributor

Made separate PR for upgrading Docker images in module tests per @ruflin's wise guidance above: #9198.

@ruflin
Copy link
Member Author

ruflin commented Nov 21, 2018

@ph Updated to 6.5.1

@ph
Copy link
Contributor

ph commented Nov 21, 2018

WFG

@ruflin ruflin merged commit 19fb6e5 into elastic:master Nov 22, 2018
@ruflin ruflin deleted the latest-65 branch November 22, 2018 09:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants