Content-Length: 241745 | pFad | https://github.com/newrelic/newrelic-dotnet-agent/issues/702

55 Remove instrumentation for Castle.Monorail · Issue #702 · newrelic/newrelic-dotnet-agent · GitHub
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.

Sign up for GitHub

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

Jump to bottom

Remove instrumentation for Castle.Monorail #702

Closed
1 of 2 tasks
jaffinito opened this issue Aug 28, 2021 · 1 comment · Fixed by #1177
Closed
1 of 2 tasks

Remove instrumentation for Castle.Monorail #702

jaffinito opened this issue Aug 28, 2021 · 1 comment · Fixed by #1177
Assignees
@vuqtran88
Labels
chore Something the New Relic engineering team needs to keep track of doing enhancement New feature or request version: major To tag issues that may cause breaking changes or removal of deprecated apis/config
Milestone
Major Release 10.0

Comments

@jaffinito
Copy link
Member

jaffinito commented Aug 28, 2021 •
edited
Loading

Castle.Monorail is deprecated and has been for some time. The non-deprecated project doesn't even have Monorail.

The repo has not been updated in many years and there is no way outside of sourceforge to even get Castle.Monorail v2 - our supported version.

We don't even have integration tests (or apps) for Monorail.

Given the above, removal should be pretty simple.

  • Add deprecation to change log for 9.x
  • ~6 months after 9.x release, remove instrumentation.
@jaffinito jaffinito added enhancement New feature or request chore Something the New Relic engineering team needs to keep track of doing labels Aug 28, 2021
@angelatan2 angelatan2 added this to the Major Release 9.0 milestone Aug 30, 2021
@jaffinito jaffinito self-assigned this Aug 31, 2021
@jaffinito
Copy link
Member Author

Deprecation done in PR #710

@jaffinito jaffinito removed this from the Major Release 9.0 milestone Sep 2, 2021
@angelatan2 angelatan2 added the version: major To tag issues that may cause breaking changes or removal of deprecated apis/config label Sep 20, 2021
@angelatan2 angelatan2 added this to the Major Release 10.0 milestone Mar 28, 2022
@vuqtran88 vuqtran88 assigned vuqtran88 and unassigned jaffinito Jun 29, 2022
@vuqtran88 vuqtran88 linked a pull request Jun 30, 2022 that will close this issue
Remove CastleMonoRail2 instrumentation. #1177
Merged
@vuqtran88 vuqtran88 mentioned this issue Jun 30, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@vuqtran88 vuqtran88

Labels
chore Something the New Relic engineering team needs to keep track of doing enhancement New feature or request version: major To tag issues that may cause breaking changes or removal of deprecated apis/config
Projects
None yet
Milestone
Major Release 10.0
Development

Successfully merging a pull request may close this issue.

Remove CastleMonoRail2 instrumentation. newrelic/newrelic-dotnet-agent
3 participants
@jaffinito @vuqtran88 @angelatan2








ApplySandwichStrip

pFad - (p)hone/(F)rame/(a)nonymizer/(d)eclutterfier!      Saves Data!


--- a PPN by Garber Painting Akron. With Image Size Reduction included!

Fetched URL: https://github.com/newrelic/newrelic-dotnet-agent/issues/702

Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy