Сбой сборки: S3 rake aborted

Хорошо, мой Discourse работал нормально, и я был на последней версии, когда кто-то сказал мне, что добавлен переход JS → Ember, поэтому мне нужно пересобрать.

http://forum.duelistsunite.org/NjA5ZGZhNjY4NzkxZTI3.txt

Никаких изменений не вносилось. Я просто выполнил пересборку. В файле app.yml изменений нет. Плагины не добавлялись.

Это мой лог discourse-doctor.

2 лайка

Это недавно было слито

@cvx @martin

2 лайка

Спасибо за сообщение об этой проблеме! Я создал pull request с исправлением: FIX: Don't fail if `s3_install_cors_rule` is off by cvx · Pull Request #14865 · discourse/discourse · GitHub

4 лайка

Спасибо, пожалуйста, сообщите, когда это будет объединено, и я смогу попробовать пересобрать.

1 лайк

Просто жду, пока Мартин проверит PR.

Извините, я уже объединил PR.

3 лайка

Я запустил пересборку, но она всё равно не удалась:

2 лайка

Будем признательны за любые советы или обновления по этому вопросу.

1 лайк

Ошибка предполагает, что задачи rake не могут возвращать значения, поэтому я считаю, что упомянутая ранее объединённая правка неверна. У меня нет настроенной среды для тестирования или создания pull request, но, думаю, использование следующего кода исправит проблему. (cc @martin)

Замена строк 180–187:

  if !result
    puts "Пропуск"
  else
    puts "Статус правил для ассетов: #{result[:assets_rules_status]}."
    puts "Статус правил для резервных копий: #{result[:backup_rules_status]}."
    puts "Статус правил для прямой загрузки: #{result[:direct_upload_rules_status]}."
  end
2 лайка

Моя ошибка! Исправил это (и другие задачи Rake) в FIX: `LocalJumpError` in rake tasks by cvx · Pull Request #14874 · discourse/discourse · GitHub

3 лайка

Боюсь, пока ничего не вышло. Это после слияния, о котором говорилось выше.

1 лайк

Обратите внимание, что пересборка прямо сейчас (на момент написания) всё ещё завершится ошибкой, если вы находитесь на ветке tests-passed, так как тесты ещё не завершены. Если вы перейдёте на страницу коммита, то увидите оранжевую точку слева от заголовка, что указывает на то, что тесты всё ещё выполняются:

Когда она сменится на зелёную галочку или (что более вероятно) на красный крест, тесты будут завершены. После этого можно выполнить пересборку. (Экспериментальный тест, скорее всего, завершится ошибкой, отсюда и красный крест, но это не влияет на статус tests-passed.)

Редактирование: Как предлагает @cvx, более полезно смотреть на ветку tests-passed. Похоже, что сейчас всё готово к использованию.

3 лайка

Это ещё не попало в ветку tests-passed: GitHub - discourse/discourse at tests-passed · GitHub

4 лайка

Рада сообщить, что всё работает!

7 лайков