Есть ли проблема с Docker Manager?
Docker Manager: FAILED TO UPGRADE
#<NoMethodError: undefined method `branch' for #<DockerManager::GitRepo:0x00007f1132b2ce20 @path="/var/www/discourse/plugins/discourse-onebox-assistant", @name="discourse-onebox-assistant", @memoize={"rev-parse HEAD"=>"0bf6e034f9a3ce0d23139417d05d2937132f2eaa", "symbolic-ref -q HEAD"=>"refs/heads/master", "for-each-ref --format='%(upstream:short)' refs/heads/master"=>"origin/master", "rev-parse --is-shallow-repository"=>"false", "show-branch remotes/origin/main"=>nil, "rev-parse origin/master^{}"=>"681fe5f804f444503c64a75e620b81f4f48dd1bf"}, @upgrade_key="upgrade:/var/www/discourse/plugins/discourse-onebox-assistant", @upgrade_version=nil>>
/var/www/discourse/plugins/docker_manager/lib/docker_manager/upgrader.rb:73:in `block in upgrade'
/var/www/discourse/plugins/docker_manager/lib/docker_manager/upgrader.rb:71:in `each'
/var/www/discourse/plugins/docker_manager/lib/docker_manager/upgrader.rb:71:in `each_with_index'
/var/www/discourse/plugins/docker_manager/lib/docker_manager/upgrader.rb:71:in `upgrade'
/var/www/discourse/plugins/docker_manager/scripts/docker_manager_upgrade.rb:19:in `block in <main>'
/var/www/discourse/plugins/docker_manager/scripts/docker_manager_upgrade.rb:6:in `fork'
/var/www/discourse/plugins/docker_manager/scripts/docker_manager_upgrade.rb:6:in `<main>'
Я что-то упустил или это баг в docker_manager?
Может ли это быть связано с этим коммитом?:
committed 05:17PM - 19 Apr 23 UTC
* FEATURE: Adds support for git tags
* Detects when the tracked branch has be… en deleted from `origin` and forces a rebuild on the command line
* Allows updating when a tag is tracked and the tag was moved to a new commit
* Improves support for shallow clones
* Works with partial clones
* Adds lots of specs for git commands and replaces the existing specs. All specs use real git repositories instead of mocks
А именно с этим изменением?:
&
Но при этом он всё ещё упоминается здесь?:
Process.kill("TSTP", unicorn_master_pid)
sleep 1
# older versions do not have support, so quickly send a cont so master process is not hung
Process.kill("CONT", unicorn_master_pid)
end
# HEAD@{upstream} is just a fancy way how to say origin/main (in normal case)
# see http://stackoverflow.com/a/12699604/84283
@repos.each_with_index do |repo, index|
# We automatically handle renames from `master` -> `main`
if repo.upstream_branch == "origin/master" && repo.branch == "origin/main"
log "Branch has changed to #{repo.branch}"
# Just in case `main` exists locally but is not used. Perhaps it was fetched?
if repo.has_local_main?
run "cd #{repo.path} && git checkout main"
else
run "cd #{repo.path} && git branch -m master main"
end
run "cd #{repo.path} && git fetch origin --tags --force"
Заранее спасибо!
1 лайк
pfaffman
20.Апрель.2023 14:44:19
2
Вчера кто-то пожаловался, что обновление UX привело к сбою на их сайте. Я не видел ошибки. Я почти уверен, что это произошло после перехода на новый базовый образ, так что это может объяснить их проблему. Похоже, это должно затронуть только тех, кто находится на бета-версии, не так ли?
1 лайк
Это было обновление с tests_passed.
Я почти уверен, что это ошибка.
3 лайка
gerhard
20.Апрель.2023 15:34:36
5
Действительно, похоже, я сломал это и не заметил, потому что во всех моих репозиториях была ветка main.
2 лайка
Да, спасибо! Похоже на пограничный случай
Хочется переместить ветку прямо сейчас, но, возможно, будет полезнее протестировать решение, если я этого не сделаю
gerhard
20.Апрель.2023 16:00:38
7
Robert:
Искушаюсь переместить ветку сейчас, но, возможно, будет полезно проверить решение, если я этого не сделаю
Это не исправит ситуацию, пока не будет исправлен docker_manager. Дайте мне пару минут, чтобы проверить моё решение и создать PR…
main ← fix-branch
merged 04:24PM - 20 Apr 23 UTC
See https://meta.discourse.org/t/upgrade-failure-docker-manager-bug/262404
2 лайка
gerhard
20.Апрель.2023 16:25:36
8
Спасибо за сообщение об этой регрессии. Она уже исправлена.
6 лайков
