minor refactors to instrumentation modules due to API changes
Removal of the struts2 convention plugin from the struts2 test. This is removed because this plugin depends on a very low version of ASM which causes conflicts when the agent and struts2 are both in the same class loader. In production, the agent's version of ASM is not in the application class loader, so these two versions can exist without conflict. In the instrumentation unit test suite, we currently don't put agent dependencies on a different class loader (though we will in the future when we migrate to the otel test harness
Testing
Please describe the tests that you ran to verify your changes. Please summarize what did you test and what needs to be tested e.g. deployed and tested helm chart locally.
Checklist:
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] Any dependent changes have been merged and published in downstream modules
Documentation
Make sure that you have documented corresponding changes in this repository or hypertrace docs repo if required.
Description
Testing
Please describe the tests that you ran to verify your changes. Please summarize what did you test and what needs to be tested e.g. deployed and tested helm chart locally.
Checklist:
Documentation
Make sure that you have documented corresponding changes in this repository or hypertrace docs repo if required.