-
Notifications
You must be signed in to change notification settings - Fork 195
Updating the index compatibility range with 9.x variables #4321
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
Conversation
✅ Vale Linting ResultsNo issues found on modified lines! |
🔍 Preview links for changed docs |
wajihaparvez
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Just one suggestion 😄
Co-authored-by: wajihaparvez <wajiha.parvez@elastic.co>
| | 6.8 | ✅ ^1^ | ✅ ^1^ | ❌ | ✅ | ❌ | ✅ | | ||
| | 7.0–7.1 | ✅ ^1, 2^ | ✅ | ✅ | ✅ | ✅ | ❌ | | ||
| | 7.2–7.17 | ✅ ^1, 2^ | ✅ | ✅ | ✅ | ❌ | ❌ | | ||
| | 8.0–8.19 | ✅ | ✅ | ✅ | ❌ | ❌ | ❌ | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
there should be a {{version.stack.base}}–{{version.stack}} row too so we can document 9-9 compatibility
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, adding row in a new commit.
|
this was a problem before, but is the label in the corner for the column or the rows? |
It is a bit awkward alright, but this is how it reads (the label is a header for the entire 1st column):
@shainaraskas any ideas how to make it less awkward to interpret? Maybe I can open a separate issue for that. |
|
@yetanothertw I don't think there's a fix for it in a simple markdown table, unfortunately. feel free to just leave it alone. |

Fixes #3599
Summary
This PR's updating the index compatibility table to include 8.19 for the latest 8.x release.
As per the request in the issue, I've swapped the columns so that the 9.x is the first column (for visibility/prominence).
Also, I've added variables to replace the 9.0 column with a 9.0-9.latest range.

Preview is available here:
PREVIEW
The overall table looks like this now:

Generative AI disclosure