Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update symfony packages #324

Merged
merged 4 commits into from
Dec 3, 2023
Merged

Update symfony packages #324

merged 4 commits into from
Dec 3, 2023

Conversation

melroy89
Copy link
Member

@melroy89 melroy89 commented Dec 3, 2023

@melroy89 melroy89 added dependencies Pull requests that update a dependency file backend Backend related issues and pull requests labels Dec 3, 2023
Copy link
Member

@nobodyatroot nobodyatroot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Verified update goes smoothly, no issues on my end.

@melroy89 melroy89 merged commit e104781 into main Dec 3, 2023
7 checks passed
@melroy89 melroy89 deleted the update_symfony_packages branch December 3, 2023 13:14
@e-five256
Copy link
Member

e-five256 commented Dec 6, 2023

A few instance owners mentioned receiving this error spamming their logs and growing messenger async/async_ap queues after applying this PR:

{"message":"Error thrown while running command \"messenger:consume async_ap -vv --time-limit=1800\". Message: \"Typed property Symfony\\Component\\PropertyInfo\\Type::$class must not be accessed before initialization\"","context":{"exception":{"class":"Error","message":"Typed property Symfony\\Component\\PropertyInfo\\Type::$class must not be accessed before initialization","code":0,"file":"/var/www/kbin/vendor/symfony/property-info/Type.php:134"},"command":"messenger:consume async_ap -vv --time-limit=1800","message":"Typed property Symfony\\Component\\PropertyInfo\\Type::$class must not be accessed before initialization"},"level":500,"level_name":"CRITICAL","channel":"console","datetime":"2023-12-06T16:34:10.250453+00:00","extra":{}}

It seems so far running bin/post-upgrade or clearing caches manually (specifically php bin/console cache:clear or docker compose exec php bin/console cache:clear if containerized) has helped stop the errors and fix queues

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend Backend related issues and pull requests dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants