-
Notifications
You must be signed in to change notification settings - Fork 24.8k
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
Update Deprecation Info API for 7.0 breaking changes #36024
Comments
Pinging @elastic/es-core-features |
…eout This commit adds support to check for negative values for index setting `index.unassigned.node_left.delayed_timeout` for the deprecation info API. relates elastic#36024 relates elastic#26828
This commit adds support to check for a value of 'fix' for the index setting `index.shard.check_on_startup` for the deprecation info API. This is no longer a valid value for this setting. relates elastic#36024 relates elastic#33194
@gwbrown Will you be updating the Migration Assistance API as part of this as well or is that separate? As it stands right now, I'm not sure we need this API going forward. Since the Deprecation Info API provides information about indexes that need to be reindexed and as far as I know, there won't be any internal indices requiring upgrade through the Migration Upgrade API in the 7.0 upgrade, it seems like maybe we should remove this two APIs completely? Whether or not we remove them, I will need to know whether or not we anticipate any useful data coming from these in 6.7 or not. If not, I'd like to drop our usage of them completely in Kibana to simplify things quite a bit. |
@joshdover I've created #36514 to track and discuss those APIs ES-side. I don't believe we should get rid of them completely, as especially the Migration Upgrade API is highly likely to be necessary for a future major version, even if it isn't for this one. We may be able to drop their usage from Kibana however, depending on what folks familiar with Watcher and Security have to say. |
All items on the checklist above are complete, but I'm going to leave this issue open for another day or two just in case, even though I believe we're past the point of adding more. I'll continue to monitor for any new breaking changes and close this in a few days if nothing comes up. |
Closing this as nothing else has come up. |
The Deprecation Info API should be updated to check for breaking changes coming in the next major version.
Here is a list created by looking through the breaking changes list in the docs and
>breaking
PRs. I have not verified each one of these, it will likely be necessary to add and/or remove items from this list.Cluster
:
is no longer allowed in the cluster name (change in Disallow : in cluster and index/alias names #26247) Deprecation check for:
in Cluster/Index name #36185Shard preferences_primary
,_primary_first
,_replica
, and_replica_first
are removed (change in Should search preference _primary take primary relocation target into account? #26335)[^^ This is a query-time parameter]
Indices
[]standard
token filter removal (does this need a check?) (change in Upgrade to a Lucene 8 snapshot #33310)[^^ this does not need a check, as it is special-cased in 7.0 for indices created in 6.x and this token filter is a no-op anyway]
delimited_payload_filter
renamed (changes in Removedelimited_payload_filter
#27705) Removedelimited_payload_filter
and add to migration assistant #27704:
is no longer allowed in index name (change in Disallow : in cluster and index/alias names #26247) Deprecation check for:
in Cluster/Index name #36185index.unassigned.node_left.delayed_timeout
may no longer be negative (change in Forbid negative values for index.unassigned.node_left.delayed_timeout #26828) (deprecation info API: negative index.unassigned.node_left.delayed_timeout #36454)fix
value forindex.shard.check_on_startup
is removed (change in completely dropindex.shard.check_on_startup: fix
for 7.0 #33194) (deprecation info API: 'fix' value for index.shard.check_on_startup #36458)[]index_options
for numeric fields has been removed (change in Reject theindex_options
parameter for numeric fields #26668)[^^ this does not need a check as indices with `index_options` on numeric fields behave fine in 7.0 if they existed prior to upgrade]
Limiting the number ofnested
json objects (?) (change in Limit the number of nested documents #27405)[^^ can't check this without running a query]
classic
similarity has been removed (change in Improve similarity integration. #29187) Deprecation check for classic similarity #36577Similarities fail when unsupported options are provided (change in Improve similarity integration. #29187)[^^ this does not need a check as indices with unsupported similarity options behave fine in 7.0 if they existed prior to upgrade]
index.percolator.map_unmapped_fields_as_string
setting has been removed (changes in remove deprecated percolator map_unmapped_fields_as_string setting #28060) Deprecation check for percolator.map_unmapped_fields_as_string #36460Nodes
The default fornode.name
is now the hostname (changes in Core: Default node.name to the hostname #33677)[^^ There doesn't appear to be a way to check this one]
thread_pool.index.size
andthread_pool.index.queue_size
have been removed (changes in Remove the index thread pool #29556) Deprecation check for index threadpool #36520The text was updated successfully, but these errors were encountered: