fix: only pass NRIA_PASSTHROUGH_ENVIRONMENT variables to v4 integrations

[rubenruizdegauna]

Only environment variables that are present in NRIA_PASSTHROUGH_ENVIRONMENT should be passed to integration. These should have precedence over the integration ones.

[cristianciutea]

LGTM

[paologallinaharbur]

LGTM

[roobre]

I see manager_test.go uses a fn called NewManager, which has a new argument. However I'm not seeing that change the in the diff. Am I missing something?

[paologallinaharbur]

If I understood what you meant, it is not a new parameter, but it is populating a field of the configuration struct:

[rubenruizdegauna]

I see manager_test.go uses a fn called NewManager, which has a new argument. However I'm not seeing that change the in the diff. Am I missing something?

It is the object populated from config. I'm setting the env vars to be passthrough as if they would be set from config.

[roobre]

Thanks for clarifying @paologallinaharbur @rubenruizdegauna, it's clear I should not leave review comments before coffee kicks in :P