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

[docs] Improve visibility of 6.8 and 7.x releases #690

Merged
merged 3 commits into from
Jun 29, 2020

Conversation

fatmcgav
Copy link
Contributor

As mentioned in #688, the docs for the master branch provide documentation and examples for the "latest" version.

So if users are trying to install a 6.x ES cluster, the examples may not work.

This PR adds the "latest" release versions to the Charts table, and updates the docs to better call out the preference to match the chart version to the product version.

@fatmcgav fatmcgav added the docs label Jun 24, 2020
@fatmcgav fatmcgav requested review from mgreau, Conky5 and jmlrt June 24, 2020 14:42
@fatmcgav fatmcgav self-assigned this Jun 24, 2020
Copy link
Member

@mgreau mgreau left a comment

Choose a reason for hiding this comment

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

Thanks, @fatmcgav for helping to clarify the process.

I left a couple of comments.

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
For example if you want to deploy an Elasticsearch `7.5.1` cluster, use the
corresponding `7.5.1` [tag][elasticsearch-751].

The `master` version of these charts are intended to support the latest pre-release
Copy link
Member

Choose a reason for hiding this comment

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

I think we should wait for @jmlrt's PR to be merged before merging this one and then update these instructions to clearly explain that:

  • the master branch is used for testing the latest snapshot versions of our products
  • we don't recommend using the master git branch for production deployment but instead either the git tags or the https://helm.elastic.co repository which contain the versions tested and released by Elastic

Gavin Williams and others added 2 commits June 25, 2020 10:40
Remove mention of `6.8` GA

Co-authored-by: Maxime Gréau <[email protected]>
Switch example to `7.7.1` chart version.
Copy link
Member

@jmlrt jmlrt left a comment

Choose a reason for hiding this comment

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

LGTM⛴
Should be merged after #682

@jmlrt jmlrt merged commit 9c05ddf into elastic:master Jun 29, 2020
jmlrt pushed a commit that referenced this pull request Jun 29, 2020
jmlrt pushed a commit that referenced this pull request Jun 29, 2020
jmlrt pushed a commit that referenced this pull request Jun 29, 2020
@jmlrt
Copy link
Member

jmlrt commented Jun 29, 2020

backported to 6.8, 7.8and 7.x branches

This was referenced Jul 16, 2020
This was referenced Jul 27, 2020
@jmlrt jmlrt mentioned this pull request Oct 28, 2020
This was referenced Nov 17, 2020
@jmlrt jmlrt mentioned this pull request Feb 8, 2021
This was referenced Mar 15, 2021
@jmlrt jmlrt mentioned this pull request May 25, 2021
@jmlrt jmlrt mentioned this pull request Mar 8, 2022
@jmlrt jmlrt mentioned this pull request Apr 21, 2022
This was referenced Sep 14, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants