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

Looks like some admin transactions still screw the apdex score. #19

Closed
lexmustafin opened this issue Nov 20, 2014 · 9 comments
Closed

Comments

@lexmustafin
Copy link

Hi Lucas,
Thanks a lot for the plugin. It's very helpful.

I have v0.1.12 and found the following issue:

The config is set to ignore admin requests: http://screencast.com/t/XGA26NIZ
But it looks like some of the transactions still make it to New Relic: http://screencast.com/t/ZfoTqixl.

Thanks,
Lex

@ProxiBlue
Copy link
Owner

Hi,

Have you found it is only cms_ related pages that still appear in admin?
I have been experimenting with this, and what I have found:

When a cms page is entered to edit, additional calls get fired (after page is rendered) (not 100% sure yet of they are ajaxed)
It is these additional calls that seem to get sent to newrelic, although all code traces show they are definitely set to ignore.

I am experimenting with this to try and figure out what's up with this.
I know where, just need to figure out why. (I suspect it is the setting of the app_name that negates the ignore setting)

@ProxiBlue
Copy link
Owner

Hi,

Can you test with the branch issue_19_test, and report back your findings.
I can no longer produce the issue on that branch.

@lexmustafin
Copy link
Author

Lucas,

These are other requests that I see in our New Relic account:
admin/catalog_product/validate
admin/catalog_product/edit
admin/catalog_product/save
admin/catalog_product/superConfig
admin/catalog_product/created
admin/catalog_product/index
admin/catalog_product/options

I'm going to update the plugin with the branch issue_19_test and will get back to you later with the results.

@ProxiBlue
Copy link
Owner

Hi,

any result?

@lexmustafin
Copy link
Author

Hi,

I was waiting for some work to be done in the admin area, to see if anything else pops up. It's been Thanksgiving here, so nobody actually wanted to work :)

From whatever was done though, I don't see any admin transactions. So ticket closed, I assume. Thanks for a prompt solution!

@ProxiBlue
Copy link
Owner

If anything pops in, re-open the ticket. I'll get that branch merged to
master and issue a new release.

On Wed, Dec 3, 2014 at 3:46 PM, Lex Mustafin notifications@github.com
wrote:

Hi,

I was waiting for some work to be done in the admin area, to see if
anything else pops up. It's been Thanksgiving here, so nobody actually
wanted to work :)

From whatever was done though, I don't see any admin transactions. So
ticket closed, I assume. Thanks for a prompt solution!


Reply to this email directly or view it on GitHub
#19 (comment).

[image: proxiblue_document_header.png]

http://www.proxiblue.com.au

Quality, not Quantity!

@ProxiBlue ProxiBlue mentioned this issue Dec 3, 2014
ProxiBlue pushed a commit that referenced this issue Dec 3, 2014
@ProxiBlue
Copy link
Owner

@lexmustafin

Make sure you update to latest release 0.1.13 - the testing branch was actually created of a slightly old master, thus missed a recent fix for the wishlist activation.

@lexmustafin
Copy link
Author

Thanks for the heads up!

@lexmustafin
Copy link
Author

@ProxiBlue,
One more transaction appeared. admin/cms_wysiwyg_images/thumbnail:
http://screencast.com/t/Pekh3Ij0NTd

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants