Skip to content

Conversation

pquentin
Copy link
Member

@pquentin pquentin commented Oct 10, 2025

Working on generating the rest-api-spec, I'm seeing various things to fix.

Each commit has an explanation for the change.

It got removed in 2019 in Elasticsearch 7.5.
They are the default values used in the Operator enum.
Comment on lines -117 to -120
/**
* This option is deprecated because it can cause primary and replica shards to diverge.
*/
force
Copy link
Member Author

Choose a reason for hiding this comment

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

* @server_default 30s
*/
master_timeout?: TimeUnit
master_timeout?: Duration
Copy link
Member Author

Choose a reason for hiding this comment

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

Every single master_timeout uses Duration, except this one.

Copy link
Contributor

github-actions bot commented Oct 10, 2025

Following you can find the validation changes against the target branch for the APIs.

API Status Request Response
bulk 🟢 537/537 → 538/538 555/555 → 556/556
cat.nodes 🟢 → 🔴 11/11 → 12/12 11/11 → 11/12
cat.segments 🟢 → 🔴 9/9 → 9/10 9/9 → 10/10
esql.query 🟢 358/358 → 359/359 0/0
index 🟢 1437/1437 → 1440/1440 1439/1439 → 1442/1442
indices.close 🟢 50/50 → 51/51 50/50 → 51/51
indices.create 🔴 1357/1381 → 1373/1397 1381/1381 → 1397/1397
indices.refresh 🟢 328/328 → 329/329 328/328 → 329/329
xpack.usage 🔴 47/47 2/43 → 1/43

You can validate these APIs yourself by using the make validate target.

@pquentin pquentin requested review from flobernd and l-trotta October 10, 2025 13:23
@pquentin pquentin merged commit c595703 into main Oct 13, 2025
8 checks passed
@pquentin pquentin deleted the specification-fixes branch October 13, 2025 13:00
Copy link
Contributor

The backport to 8.19 failed:

The process '/usr/bin/git' failed with exit code 1

To backport manually, run these commands in your terminal:

# Fetch latest updates from GitHub
git fetch
# Create a new working tree
git worktree add .worktrees/backport-8.19 8.19
# Navigate to the new working tree
cd .worktrees/backport-8.19
# Create a new branch
git switch --create backport-5472-to-8.19
# Cherry-pick the merged commit of this pull request and resolve the conflicts
git cherry-pick -x --mainline 1 c59570341f6984f98e056f0a6df367729d57d85d
# Push it to GitHub
git push --set-upstream origin backport-5472-to-8.19
# Go back to the original working tree
cd ../..
# Delete the working tree
git worktree remove .worktrees/backport-8.19

Then, create a pull request where the base branch is 8.19 and the compare/head branch is backport-5472-to-8.19.

github-actions bot pushed a commit that referenced this pull request Oct 13, 2025
* Remove VersionType.force

It got removed in 2019 in Elasticsearch 7.5.

* Switch to lowercase default_operator

They are the default values used in the Operator enum.

* Fix streams.status master_timeout

* Run make contrib

(cherry picked from commit c595703)
pquentin added a commit that referenced this pull request Oct 15, 2025
* Remove VersionType.force

It got removed in 2019 in Elasticsearch 7.5.

* Switch to lowercase default_operator

They are the default values used in the Operator enum.

* Fix streams.status master_timeout

* Run make contrib

(cherry picked from commit c595703)

# Conflicts:
#	output/schema/schema.json
@pquentin
Copy link
Member Author

💚 All backports created successfully

Status Branch Result
8.19

Questions ?

Please refer to the Backport tool documentation

pquentin added a commit that referenced this pull request Oct 15, 2025
* Remove VersionType.force

It got removed in 2019 in Elasticsearch 7.5.

* Switch to lowercase default_operator

They are the default values used in the Operator enum.

* Fix streams.status master_timeout

* Run make contrib

(cherry picked from commit c595703)

Co-authored-by: Quentin Pradet <quentin.pradet@elastic.co>
pquentin added a commit that referenced this pull request Oct 15, 2025
* Remove VersionType.force

It got removed in 2019 in Elasticsearch 7.5.

* Switch to lowercase default_operator

They are the default values used in the Operator enum.

* Fix streams.status master_timeout

* Run make contrib

(cherry picked from commit c595703)

# Conflicts:
#	output/schema/schema.json
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants