Skip to content

Add support to run agent as domain user on Windows installation#785

Merged
KevinFairise2 merged 3 commits intomainfrom
kfairise/support-domain-user-windows
Aug 24, 2023
Merged

Add support to run agent as domain user on Windows installation#785
KevinFairise2 merged 3 commits intomainfrom
kfairise/support-domain-user-windows

Conversation

@KevinFairise2
Copy link
Copy Markdown
Member

@KevinFairise2 KevinFairise2 commented Jul 13, 2023

What does this PR do?

Add options to run the agent as a domain user on Windows installation

Motivation

Should fix #742

Additional Notes

Describe your test plan

@KevinFairise2 KevinFairise2 marked this pull request as ready for review July 17, 2023 08:09
@KevinFairise2 KevinFairise2 requested a review from a team as a code owner July 17, 2023 08:09
file { $dst_dir:
ensure => directory,
owner => $datadog_agent::params::dd_user,
owner => $datadog_agent::dd_user,
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could we have set a different name not to confuse $datadog_agent::params::dd_user and $datadog_agent::dd_user?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it is more explicit if they have the same name because params::dd_user is the default value for dd_user variable. Also those variables should never be edited by the user and are only internal variables.

Comment thread manifests/init.pp
@KevinFairise2 KevinFairise2 merged commit d20fde9 into main Aug 24, 2023
@KevinFairise2 KevinFairise2 deleted the kfairise/support-domain-user-windows branch August 24, 2023 14:14
cegeka-jenkins pushed a commit to cegeka/puppet-datadog_agent that referenced this pull request Feb 5, 2026
…Dog#785)

* Add support to run agent as domain user on Windows installation

* Update Puppet module to launch Agent as a domain user on windows

* Update comment
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

No support for running Windows agent as domain user

3 participants