Manage agent environment based on param

[bwitt]

Only set the agent environment if agent_manage_environment is true

Fixes #593

[bwitt]

those failures do not seem related?

[shaun-rutherford]

Pinging that having this code merged is something we need as well, was going to write it myself when I stumbled on this PR.

[shaun-rutherford]

Can we get eyes on this please? I really don't want to have to fork this project internally but we need this fixed as it's causing a bit of chaos in our environment.
The failing bits seem to be completely unrelated to this PR.
The failing tests come from:
spec/classes/puppet_server_spec.rb

of which no code changes to that test suit are made in this PR?

[bwitt]

The CI failures are due to inifile breaking support for puppet 7; sounds like we may be moving forward with dropping puppet 7 support

[bwitt]

@shaun-rutherford btw some discussion is happening on the voxpupuli slack if you'd like to join the fun

[bwitt]

#966 got merged so now all tests should pass