Skip to content

Conversation

alex-spies
Copy link
Contributor

No description provided.

Copy link
Contributor

Documentation preview:

@elasticsearchmachine elasticsearchmachine added v9.0.0 Team:Docs Meta label for docs team Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) labels Sep 16, 2024
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-docs (Team:Docs)

@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-analytical-engine (Team:Analytics)

@alex-spies
Copy link
Contributor Author

alex-spies commented Sep 16, 2024

Hey @elastic/es-docs , could you have a look and tell me if I'm doing this correctly? Thanks in advance!

I'm trying to add known issues to the already released versions 8.15.0 and 8.15.1.

I couldn't find release notes for 8.15.1 on main, so I went and added them - is there another place or branch that this should go into instead?

@alex-spies
Copy link
Contributor Author

I think this needs to go in the 8.15 branch, main doesn't have the 8.15.1 release notes. Closing this in favor of #112930, which is on the 8.15 branch.

@alex-spies alex-spies closed this Sep 16, 2024
@leemthompo
Copy link
Contributor

I think this needs to go in the 8.15 branch, main doesn't have the 8.15.1 release notes. Closing this in favor of #112930, which is on the 8.15 branch.

@alex-spies euggh sorry about that. I just merged the forward porting of 8.15.1 release notes #112568, so really should target the main branch in your PR alright and backport.

@leemthompo
Copy link
Contributor

I had suggested rewording in this comment

@alex-spies
Copy link
Contributor Author

@leemthompo and @tylerperk , I took over Liam's version (much nicer), added the downgrade option back in, and the suggestion to use a different default view is also in there.

Let me know if you like it this way - if yes, I think this is good to go :)

@leemthompo
Copy link
Contributor

I agree with @nik9000 that downgrading is not easy, and similarly waiting for 8.15.2 isn't ideal. But they are options so I think it makes sense to have them. The others are more practical and quicker to implement.

Copy link

@tylerperk tylerperk left a comment

Choose a reason for hiding this comment

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

LGTM thanks

@nik9000 nik9000 added the auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) label Sep 16, 2024
@elasticsearchmachine elasticsearchmachine merged commit 70fcc54 into elastic:main Sep 16, 2024
5 checks passed
@alex-spies alex-spies deleted the known-issue-8.15-from-logs-oom branch September 16, 2024 19:31
alex-spies added a commit to alex-spies/elasticsearch that referenced this pull request Sep 16, 2024
@elasticsearchmachine
Copy link
Collaborator

💔 Backport failed

Status Branch Result
8.15
8.x Commit could not be cherrypicked due to conflicts

You can use sqren/backport to manually backport by running backport --upstream elastic/elasticsearch --pr 112926

@nik9000
Copy link
Member

nik9000 commented Sep 16, 2024

Backported to 8.x with #112959

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Analytics/ES|QL AKA ESQL auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport pending >docs General docs changes Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) Team:Docs Meta label for docs team v8.15.0 v8.15.1 v8.16.0 v9.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants