Rename OPENTELEMETRY_COLLECTOR_CONFIG_FILE to appropriate usage

[vasireddy99]

Currently OPENTELEMETRY_COLLECTOR_CONFIG_FILE is a environment variable used to set the config file from any of the config map providers such as http, s3, yaml. etc., as per the naming convention it is not exactly a file as the input to this variable could be URI format example here. Hence creating this issue to see if this can be renamed to OPENTELEMETRY_COLLECTOR_CONFIG instead.

[Aneurysm9]

I would suggest using OPENTELEMETRY_COLLECTOR_CONFIG_URI as the name of the variable to hold a URI pointing to a collector config.

We should ensure that we continue to support the prior variable for some period of time as a fallback if the new variable is not populated. We should emit a warning into the logs if the old variable is used.

[github-actions]

This issue was marked stale. It will be closed in 30 days without additional activity.

[github-actions]

Closed as inactive. Feel free to reopen if this issue is still relevant.

[gshpychka]

Still relevant. The name cause some confusion for me initially.

[tylerbenson]

Certainly open to PRs if someone wants to contribute this change. LMK and I can reopen the issue.

[gshpychka]

Certainly open to PRs if someone wants to contribute this change. LMK and I can reopen the issue.

Would the original name have to be kept for backwards compatibility?

[tylerbenson]

Assuming that would not be difficult, I'd say yes.

[gshpychka]

Assuming that would not be difficult, I'd say yes.

What's the process for deprecating env vars in this case?

[tylerbenson]

We don't have any documented process. I would mainly suggest remove it from the documentation so people start using the new one going forward.

[gshpychka]

@tylerbenson can you reopen this issue so that it can be properly closed as complete if/when my PR is merged?