Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Module does not detect if machine is already joined #13

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

dekimsey
Copy link

I'm trying to use this module to set-up Centrify in a previously mixed environments. On machines that already have centrify registered, the module fails.

How can I configure this such that provisioning works correctly (does a selfserve join), but then does not attempt to do it on machines that are already successfully joined.

notice /Stage[main]/Centrify::Adjoin::Selfserve/Exec[adjoin_with_selfserve]/returns Error: this computer is already joined to the domain.
notice /Stage[main]/Centrify::Adjoin::Selfserve/Exec[adjoin_with_selfserve]/returns Use adleave first.
err Puppet adjoin -w -V --selfserve 'EXAMPLE.COM' returned 11 instead of one of [0]
err /Stage[main]/Centrify::Adjoin::Selfserve/Exec[adjoin_with_selfserve]/returns change from notrun to 0 failed: adjoin -w -V --selfserve 'EXAMPLE.COM' returned 11 instead of one of [0]

Perhaps checking adinfo first?

$ adinfo
Local host name:   test-foo-01
Joined to domain:  example.com
Joined as:         test-foo-01.example.com
Pre-win2K name:    test-foo-01
Current DC:        dev-ad-01.example.com
Preferred site:    Site-Examplecom
Zone:              Auto Zone
CentrifyDC mode:   connected
Licensed Features: Disabled

Parameters:

  centrify:
    domain: EXAMPLE.COM
    join_type: selfserve
    use_express_license: true

@dekimsey
Copy link
Author

Ah. I found the issue, the adinfo test already in the module is case-sensitive. To my knowledge it shouldn't be.

@onzyone
Copy link
Contributor

onzyone commented Apr 25, 2017

Feel free to put a downcase() and create a PR

@dekimsey
Copy link
Author

I wasn't sure how you might want to update the tests, so I did not change them.

@dekimsey
Copy link
Author

I don't see any errors in the CI logs, not sure what is generating the errors.

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.

2 participants