elastic / cloud-sdk-go

Apache License 2.0
21 stars 46 forks source link

fix: support allocator down for vacating cluster resources #443

Closed Z3r0Sum closed 1 year ago

Z3r0Sum commented 1 year ago

Description

Related Issues

Motivation and Context

Unable to pass --allocator-down true and have it be respected from ecctl's perspective.

How Has This Been Tested?

UTs and will validate with a local version of ecctl (TODO)

Types of Changes

Readiness Checklist

Z3r0Sum commented 1 year ago

I was able to validate this locally with a run of ecctl pointing at this change:

POST /api/v1/regions/aws-eu-west-1/platform/infrastructure/allocators/i-0c1d25c3a0cb8659e/clusters/elasticsearch/_move?allocator_down=true HTTP/1.1

{"apm_clusters":null,"appsearch_clusters":null,"elasticsearch_clusters":[{"cluster_ids":["558c087999324a3d99656ebfcfed34fe"],"plan_override":{"plan_configuration":{"calm_wait_time":5,"extended_maintenance":false,"max_snapshot_age":300,"max_snapshot_attempts":3,"move_allocators":[{"allocator_down":true,"from":"i-0c1d25c3a0cb8659e","to":null}],"move_instances":null,"move_only":true,"override_failsafe":false,"preferred_allocators":[],"reallocate_instances":false,"skip_data_migration":true,"skip_post_upgrade_steps":false,"skip_snapshot":true,"skip_snapshot_post_major_upgrade":false,"skip_upgrade_checker":false,"timeout":8192},"strategy":{"grow_and_shrink":{}}}}],"enterprise_search_clusters":null,"kibana_clusters":null}