Francisco Fernández Castaño

Results 11 comments of Francisco Fernández Castaño

I fail to understand how this fixes https://github.com/elastic/elasticsearch/issues/86890. The repository cannot be deleted since there's a searchable snapshot using it, and searchable snapshots should be deleted before in: ``` //...

@arteam I've been taking another look into this and I think we might be hitting a race condition here: `ESRestTestCase#wipeSearchableSnapshotsIndices` deletes the mounted searchable snapshots and then we try to...

Thanks for looking into this @arteam, I wonder if we should look into the failing test and ensure that we don't rollover or that the rolled over index is not...

Another instance of this https://gradle-enterprise.elastic.co/s/yj5yfklm2ysvg/tests/:x-pack:plugin:yamlRestTest/org.elasticsearch.xpack.test.rest.XPackRestIT/test%20%7Bp0=transform%2Ftransforms_start_stop%2FVerify%20start%20transform%20reuses%20destination%20index%7D?top-execution=1

@henningandersen and I discussed this issue during a triage session and we think this issue should be moved to the script label as the issue seems to be related to...

> I can see some value in preventing changes to the location of a repository while it contains mounted snapshots, but I can also think of cases where it might...

> And now I wonder why the 2nd bullet point exists at all? The second bullet point could be done equivalently by deleting the repository and creating a new one...

> And if ?verify=false, we skip this check, and even the check that no searchable snapshots are there. In that case we'll be changing the current [verify](https://www.elastic.co/guide/en/elasticsearch/reference/current/verify-snapshot-repo-api.html) semantics which it's...

> A [repository backup](https://www.elastic.co/guide/en/elasticsearch/reference/current/snapshots-register-repository.html#snapshots-repository-backup) would let you do this. But there's still room for improvement on that area https://github.com/elastic/elasticsearch/issues/54944

> The ?verify flag is from the [PUT /_snapshot/my_repository](https://www.elastic.co/guide/en/elasticsearch/reference/8.4/put-snapshot-repo-api.html) API, not the [verify](https://www.elastic.co/guide/en/elasticsearch/reference/8.4/verify-snapshot-repo-api.html) API itself. So we could have the PUT API do both the verify API but also check...