Skip to content

Remove support for deprecated plugins#83

Merged
jkschulz merged 3 commits intomasterfrom
jschulz-remove-deprecated-plugins
Dec 20, 2019
Merged

Remove support for deprecated plugins#83
jkschulz merged 3 commits intomasterfrom
jschulz-remove-deprecated-plugins

Conversation

@jkschulz
Copy link
Contributor

@jkschulz jkschulz commented Dec 9, 2019

Confirmed on CentOS 7 that adding configs for PostgreSQL, MongoDB, Varnish, and ZooKeeper cause the agent to fail to start (expected). The other deprecated applications use generic plugins (Plugin:GenericJMX = Cassandra, HBase, Kafka, Tomcat; Plugin:cURL-JSON = CouchDB, Elasticsearch, RabbitMQ, Riak) so those will still work.
Also confirmed that the non-deprecated 3P applications still work as expected (Apache, JVM, Memcached, MySQL, Nginx, Redis).

@jkschulz jkschulz requested a review from a team December 9, 2019 22:40
@ghost ghost requested review from jkohen and removed request for a team December 9, 2019 22:40
Copy link
Contributor

@igorpeshansky igorpeshansky left a comment

Choose a reason for hiding this comment

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

Haven't had a chance to look at the spec changes, but have some comments on the Debian builds.

Copy link

@jkohen jkohen left a comment

Choose a reason for hiding this comment

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

So many deleted lines, I like it! Thanks for thoroughly testing the changes on CentOS 7. I see this PR affects files for many other distributions. How did you verify that those continue to work? @jkschulz

Copy link
Contributor

@igorpeshansky igorpeshansky left a comment

Choose a reason for hiding this comment

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

Both the rpm spec and the Debian build files LGTM. Same question as Javier about testing.

@sophieyfang
Copy link
Contributor

sophieyfang commented Dec 10, 2019 via email

@jkschulz
Copy link
Contributor Author

With @sophieyfang's help, the 3rd party apps we still support (apache, jvm, memcached, mysql, nginx, redis) were tested and verified on the remaining distros (centos8, ubuntu 18.04 (bionic), ubuntu 16.04 (xenial), debian 9 (stretch), debian 10 (buster), sles 12, and sles 15).

I also ran a script on each of those distros that pulled in sample configs for the 4 non-generic deprecated apps (PostgreSQL, MongoDB, Varnish, and ZooKeeper) and confirmed that the presence of each caused the agent to fail to start as expected.

@jkschulz jkschulz merged commit 47f853f into master Dec 20, 2019
@jkschulz jkschulz deleted the jschulz-remove-deprecated-plugins branch December 20, 2019 17:02
@jkohen
Copy link

jkohen commented Dec 20, 2019 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants