drop the redundant "hostname" and "env" labels added to metrics

[trentm]

The APM Node.js agent adds two labels to all exported metrics, e.g.:

                "hostname": "pink.local",
                "env": "development"

These end up in the Elasticsearch documents as labels.hostname and labels.env. These are redundant to the existing host.hostname and service.environment fields that are provided for all APM documents. Adding these labels is not in the APM spec for metrics.

I think we should drop these labels. They are just wasted space in the DB.

For backward compat: It is possible there are users that are using these labels for queries and they would have to adjust. Does this statement from the coming "breaking changes doc" PR for the APM specs make this not a breaking change?

In general, we don’t consider changes in the data we collect to be breaking, unless they have security or privacy implications.

I.e. we would be able to make this change without having to wait for a major release cycle?