[DOCS] Adds new installation package details#29590
Conversation
|
Pinging @elastic/es-core-infra |
There was a problem hiding this comment.
I wonder if we can make it more clear that it's free? The term "commercial features" isn't wrong - the default distribution contains features that aren't under an OSI-approved license - but this phrasing doesn't doesn't send the message that "it's totally free to use, but isn't totally Apache 2.0 licensed." I still struggle with the best way to phrase it here though.
There was a problem hiding this comment.
How about: "This package contains both open source, free commercial, and paid commercial features"?
There was a problem hiding this comment.
I think that's definitely an improvement, but I wonder if we need to explicitly highlight the distinction between OSS and "not" AND free and not-free?
Another approach would be to communicate that it's free to use, and that it's not governed by Apache 2.0. What about: "This package is free to use under the Elastic License."?
clintongormley
left a comment
There was a problem hiding this comment.
Added a suggestion
There was a problem hiding this comment.
How about: "This package contains both open source, free commercial, and paid commercial features"?
There was a problem hiding this comment.
With the change above, how about:
Start a 30-day trial license to try out all of the paid commercial features.
|
Thanks for the feedback! I've updated the text based on suggestions here and in elastic/kibana#17781 |
There was a problem hiding this comment.
I wonder if there is value in having this whole section as a fragment, and including via include::license_stuffs? it might cut down on the number of places to update going forward, if there are links and such we need to change? If done as an include, we could use that include at least 5 times in the files in this PR seemingly?
Everything else LGTM 💯
* 6.x: (62 commits) [DOCS] Adds new installation package details (#29590) Revert "Build: Move gradle wrapper jar to a dot dir (#30146)" [DOCS] Added 6.3.0 section to changelog [DOCS] Merge 6.x release notes into changelog (#30312) [DOCS] Removes broken link [DOCS] Adds file realm configuration details (#30221) [DOCS] Adds PKI realm configuration details (#30225) [DOCS] Fix 6.4-specific link in changelog (#30314) Remove RepositoriesMetaData variadic constructor (#29569) [DOCS] Adds changelog to Elasticsearch Reference (#30310) Test: increase authentication logging for debugging [DOCS] Removes redundant SAML realm settings (#30196) SQL: Teach the CLI to ignore empty commands (#30265) [DOCS] Fixes section error [DOCS] Adds Active Directory realm configuration details (#30223) [DOCS] Removes redundant file realm settings (#30192) [DOCS] Fixes users command name (#30275) Build: Move gradle wrapper jar to a dot dir (#30146) Build: Log a warning if disabling reindex-from-old (#30304) TEST: Add debug log to FlushIT ...
This PR updates the Elasticsearch Reference with respect to changes in the installation packages.