Provide (or release?) a way to set apdex_t for an existing new relic application

[ryangardner]

I was about to pull down this provider and start fumbling my way through trying to implement this myself, when I came across:

https://github.com/terraform-providers/terraform-provider-newrelic/blob/master/newrelic/resource_newrelic_application.go

Which looks like it has the ability to do what I was needing. Is there a rough idea when the next release might go out that will include this feature? (If you accept bribes, what's your preferred way to be bribed?)

[jthurman42]

There is a bit of work to get this into the provider, but there is an API endpoint to do so. We're currently prepping to start a 2.x release cycle with the change required to enable this ability.

The specific endpoint in question is: https://api.newrelic.com/graphiql

with the following graphql query:

mutation {
  apmApplicationEntityChangeSettings(guid: "<MY APPLICATION ENTITY GUID>", settings: {apdexTarget: 0.5}) {
    entity {
      guid
      name
      settings {
        apdexTarget
      }
    }
  }
}

[ryangardner]

Great, thanks for the reply!

We did see that graphql endpoint - (that currently only lets you set apdexTarget - but maybe someday will let you set other settings)

Is the current master branch getting ready for the 2.x release? We did build the current master version and what's inside there that lets you set the appdex and a couple other settings does seem to work (at least to set apdex): https://github.com/terraform-providers/terraform-provider-newrelic/blob/master/newrelic/resource_newrelic_application.go#L22-L36

(and one piece of feedback on that specific resource, since it hasn't been released yet... you might want to consider naming it applicationSettings or something since the version there almost makes it seem like it will create a new application in new relic rather than just modify an existing one )

[jthurman42]

Looks like this was merged into master just after the split, but appears to be backwards compatible. @zlesnr any reason not to port the application resource to v1.x?

Good point on the naming, as this won't create an application. We overrode the 'create' function to act as an import. Read/Update/Delete should all function (note the API requires apps to stop reporting for 12 hours before a delete can occur, so this is also slightly unexpected behavior)

[zlesnr]

Not that I see @jthurman42. Looks like its all REST there.

[ctrombley]

@ryangardner Support for this has been added in v1.18.0 of the provider via the newrelic_application_settings resource. Give it a try and let us know if you have any issues.