-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
New Relic agent 6.0.0.155 with PHP 7 #153
Comments
Yes, of course, will release soon! |
@dzuelke Awesome! Damn that has to be a record response time... Thanks! :) |
Will it take some time or should the extension already be available?
|
Extension is no longer available for PHP 7 due to a regression that crashes at least Laravel apps: https://devcenter.heroku.com/changelog-items/843 |
Ok, thanks. Hadn't seen that update. |
It went out just a few minutes ago, @jonahgeorge, so it's not published yet :) |
@dzuelke version 6.1.0.157 was released a few days ago. They've fixed the issue with Laravel 5.2 apps and a potential segfault. Based on feedback from the forums, no one has reported any issues with this version. Could we give it a shot? :) https://docs.newrelic.com/docs/release-notes/agent-release-notes/php-release-notes |
It's already prepared and built, @djtarazona; we just don't roll out changes on weekends :) |
@dzuelke I saw on Twitter that the updated PHP extensions, including New Relic 6.1.0.157 with PHP 7 support, was released. However I'm still receiving the following when pushing a new commit / deploy to Heroku.
Is there a delay or cache or something? Thanks! :) |
There should not be a delay @djtarazona. Can you please file a ticket at help.heroku.com for your app and grant full access? |
Or if you |
@dzuelke No I get the following when I run
|
Interesting... when I put
But when I remove it and rely on auto-detection using the For now, I'll force / fix it by requiring |
Yes please remove that explicit requirement again and open a ticket, @djtarazona, so I can reproduce and get to the bottom of this! Will do it right away, so you won't be without |
Thanks @dzuelke! Ticket #347937. |
@dzuelke does this #153 (comment) mean this issue needs/should be reopened? |
No @holtkamp, New Relic is available again for PHP 7 (https://devcenter.heroku.com/changelog-items/853) |
ok, great. Just add the add-on and indeed, it now works out of the box 😄 |
Is this still a known issue? In my build log I'm getting:
I tried adding |
You're running PHP 7.2, @tdmalone, but there is no New Relic extension yet for that version, so it skips installation. |
(and if you add an explicit requirement to |
New Relic has finally released PHP 7 support!
https://docs.newrelic.com/docs/release-notes/agent-release-notes/php-release-notes/php-agent-600155
@dzuelke can we get this added as an available 3rd-party extension? Please and thank you!
The text was updated successfully, but these errors were encountered: