[processor/geoip] Detect source.address from signal's attributes

[rogercoll]

Component(s)

processor/geoip

Is your feature request related to a problem? Please describe.

The current GeoIP processor looks for the source.address field in the resource attributes of every signal. The processor should be able to detect the IP from other contexts too, for example, the source.address might be located in the attributes instead of the identifying resource attributes.

Describe the solution you'd like

Detect the IP from the signal's attributes if not present in the resource ones, and add the corresponding geographical attributes to the same context.

Describe alternatives you've considered

No response

Additional context

No response

[github-actions]

Pinging code owners:

• processor/geoip: @andrzej-stencel @michalpristas @rogercoll

See Adding Labels via Comments if you do not have permissions to add labels yourself.

[crobert-1]

Makes sense to me, and was filed by a code owner, removing needs triage.

One thing to note: The README currently specifies that source.address is taken from the resource attributes. This will need updated as a part of this change. (You're probably already aware, just wanted to point it out so we don't miss it 👍)

[rogercoll]

In order to let the user configure whether to look in the resource attributes or the signal's attributes (in the future could be any other source), I purpose to add the following configuration structure in the processor.

geoip:
  source:
    from: resource_attributes

The source configuration option would tell the processor whether to look for the source.address from the resource_attributes or attributes. The configuration naming is similar to the k8sattributes processor.

I think being a map configuration option would ease the addition of future options for narrowed IP look ups, for example, specifying a different attribute name to override the default source.address:

geoip:
  source:
    from: attributes
    name: upstream.address

[github-actions]

This issue has been inactive for 60 days. It will be closed in 60 days if there is no activity. To ping code owners by adding a component label, see Adding Labels via Comments, or if you are unsure of which component this issue relates to, please ping @open-telemetry/collector-contrib-triagers. If this issue is still relevant, please ping the code owners or leave a comment explaining why it is still relevant. Otherwise, please close it.

Pinging code owners:

• processor/geoip: @andrzej-stencel @michalpristas @rogercoll

See Adding Labels via Comments if you do not have permissions to add labels yourself.

[rogercoll]

Fixed in #34214