yandex-cloud / terraform-provider-yandex

Terraform Yandex provider
https://www.terraform.io/docs/providers/yandex/
Mozilla Public License 2.0
213 stars 116 forks source link

Refreshing state для единственного s3 bucket занимает более 1 минуты #421

Open max107 opened 8 months ago

max107 commented 8 months ago

Минимально воспроизводимый пример. Наблюдается константно, вне зависимости от сети / локации / бакита.

locals {
  bucket = "foobar-example"
}

resource "yandex_iam_service_account" "root-sa" {
  name = "proto"
}

resource "yandex_resourcemanager_folder_iam_member" "root-sa" {
  folder_id = local.folder_id
  role      = "storage.editor"
  member    = "serviceAccount:${yandex_iam_service_account.root-sa.id}"
}

resource "yandex_iam_service_account_static_access_key" "static-key" {
  service_account_id = yandex_iam_service_account.root-sa.id
}

resource "yandex_storage_bucket" "proto" {
  access_key = yandex_iam_service_account_static_access_key.static-key.access_key
  secret_key = yandex_iam_service_account_static_access_key.static-key.secret_key
  bucket     = local.bucket
somewho commented 3 months ago

вроде в свежих версиях не актуально

max107 commented 3 months ago

@somewho уточните пожалуйста начиная с какой версии?

somewho commented 3 months ago

@somewho уточните пожалуйста начиная с какой версии?

на данный момент использую 0.124.0, прописан один бакет, проблем с торможением на обновлении статуса не наблюдаю, проект на пару десятков разнообразных ресурсов: terraform plan 1.10s user 0.26s system 28% cpu 4.721 total

MikailBag commented 3 months ago

Я тоже столкнулся с этой проблемой и в #437 описал свои наблюдения.

TL&DR: если операции с s3 выполняются от имени СА, у которого нет роли storage.admin, то провайдер не может считать bucket policy бакета, и ретраит это в течение одной минуты.

Я также принес связанный ПР #444, который был смержен как раз в версии 0.124.0. Но если он поправил эту проблему, то для меня это неожиданность :)

Fluffi1235 commented 2 months ago

@MikailBag удалось ли проверить, что данная проблема решена ?

Я также принес связанный ПР #444, который был смержен как раз в версии 0.124.0. Но если он поправил эту проблему, то для меня это неожиданность :)

MikailBag commented 2 months ago

Я думаю, тут уже нужно разбираться коллегам из Yandex Cloud. Я подробных тестов не проводил.