Issue 42 #43
Issue 42 #43
Conversation
antonshevelev
requested review from
pozitronik,
s1lver,
alxlapin and
Fullchaos
|
Чини тесты |
|
Последний коммит пока только для проверки |
|
Причина почему поломались тесты: либа aws/aws-sdk-php начиная с версии 3.280.0 требует указания обязательного параметра region, которому раньше можно было присвоить значение пустой строки. (src/models/S3.php:71) На что может повлиять этот параметр нужно изучать. Самым безопасным считаю пока ограничить максимальную версию библиотеки PS в списке регионов нет ничего относящегося к России https://docs.aws.amazon.com/general/latest/gr/rande.html#regional-endpoints @pozitronik что думаешь? |
|
Судя по той же доке, регионы — это способ указания предпочтительной региональной ноды при работе с «настоящим» амазоном. Для S3 это вообще пофигу. |
| @@ -64,7 +64,7 @@ services: | |||
| MINIO_ACCESS_KEY: AKIAIOSFODNN7EXAMPLE | |||
| MINIO_SECRET_KEY: wJalrXUtnFEMI | |||
| entrypoint: sh | |||
| command: -c 'mkdir -p /data/testbucket && mkdir -p /data/first_bucket && mkdir -p /data/second_bucket && minio server --console-address ":9009" --address ":9008" /data' | |||
| command: -c 'mkdir -p /data/testbucket && mkdir -p /data/first-bucket && mkdir -p /data/second-bucket && minio server --console-address ":9009" --address ":9008" /data' | |||
There was a problem hiding this comment.
В чем смысл переименовывать директории?
There was a problem hiding this comment.
Бакеты не создавались с таким неймингом, тесты падали
Это известная проблема. Тут даже есть Issue на это #40 |
|
@pozitronik ждем аппрува от Жени @s1lver? Или вливаем? |
|
@antonshevelev новый тег к мастеру добавишь? |
|
|
|
|
@pozitronik done |
В релизах до сих пор предыдущая версия. |
|
@pozitronik "Create release" релиз я не нажимал? Жму?) |
После того, как ты добавил и запушил новый тег для master, он автоматически добавляется в релизы, и обновляется актуальная версия в composer. Возможно, ты не на ту ветку тег добавил? |
|
всё делал в мастере |
No description provided.