Skip to content
This repository has been archived by the owner on May 16, 2023. It is now read-only.

[elasticsearch] fix deprecated note #335

Merged
merged 2 commits into from
Oct 18, 2019
Merged

[elasticsearch] fix deprecated note #335

merged 2 commits into from
Oct 18, 2019

Conversation

jmlrt
Copy link
Member

@jmlrt jmlrt commented Oct 17, 2019

related to https://github.com/elastic/helm-charts/pull/333/files#r335988708

  • Chart version not bumped (the versions are all bumped and released at the same time)
  • README.md updated with any new values or changes
  • Updated template tests in ${CHART}/tests/*.py
  • Updated integration tests in ${CHART}/examples/*/test/goss.yaml

@jmlrt jmlrt requested a review from Crazybus October 17, 2019 17:13
@jmlrt jmlrt mentioned this pull request Oct 17, 2019
Merged
7 tasks
@jmlrt jmlrt self-assigned this Oct 17, 2019
@jmlrt jmlrt added the enhancement New feature or request label Oct 17, 2019
Crazybus
Crazybus suggested changes Oct 18, 2019
View reviewed changes
@jmlrt @Crazybus
Update elasticsearch/README.md
a5e9a5e 
Co-Authored-By: Michael Russell <Crazybus@users.noreply.github.com>
@jmlrt jmlrt requested a review from Crazybus October 18, 2019 06:05
Crazybus
Crazybus approved these changes Oct 18, 2019
View reviewed changes
Copy link
Contributor

@Crazybus Crazybus left a comment

Choose a reason for hiding this comment

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

LGTM!

@jmlrt jmlrt merged commit bbf53b3 into elastic:master Oct 18, 2019
@jmlrt jmlrt deleted the fix-deprecated-notes branch October 18, 2019 13:38
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Reviewers

@Crazybus Crazybus Crazybus approved these changes

Assignees

@jmlrt jmlrt

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jmlrt @Crazybus