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.

Sign up for GitHub

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

Jump to bottom

[Backport][ipa-4-8] ipa_client_automount.py: fix typo (idmap.conf => idmapd.conf) #3788

Closed
wants to merge 1 commit into from
Closed

[Backport][ipa-4-8] ipa_client_automount.py: fix typo (idmap.conf => idmapd.conf) #3788

wants to merge 1 commit into from

Conversation

fcami
Copy link
Contributor

@fcami fcami commented Oct 14, 2019

This PR was opened automatically because PR #3781 was pushed to master and backport to ipa-4-8 is required.

@fcami
ipa_client_automount.py: fix typo (idmap.conf => idmapd.conf)
02d6804 
660c49 introduced --idmap-domain which sets the Domain option in
idmapd.conf. However the help message for that knob mentioned
idmap.conf which is wrong. Fix that.
Reported by Marc Muehlfeld <mmuehlfe@redhat.com>.

Signed-off-by: François Cami <fcami@redhat.com>
@fcami fcami added the ack Pull Request approved, can be merged label Oct 14, 2019
@fcami
Copy link
Contributor Author

fcami commented Oct 14, 2019

PR was ACKed automatically because this is backport of PR #3781. Wait for CI to finish before pushing. In case of questions or problems contact @fcami who is author of the original PR.

@fcami fcami added the re-run Trigger a new run of PR-CI label Oct 15, 2019
@freeipa-pr-ci freeipa-pr-ci removed the re-run Trigger a new run of PR-CI label Oct 15, 2019
@rcritten rcritten added the re-run Trigger a new run of PR-CI label Oct 15, 2019
@freeipa-pr-ci freeipa-pr-ci removed the re-run Trigger a new run of PR-CI label Oct 15, 2019
@fcami
Copy link
Contributor Author

fcami commented Oct 16, 2019 •
edited

test_smb fails with:

2019-10-15 20:53:08,658    DEBUG  An error occurred while executing the action on the 'ad-root'
2019-10-15 20:53:08,659    DEBUG  machine. Please handle this error then try again:
2019-10-15 20:53:08,659    DEBUG  
2019-10-15 20:53:08,659    DEBUG  An authorization error occurred while connecting to WinRM.
2019-10-15 20:53:08,659    DEBUG  
2019-10-15 20:53:08,659    DEBUG  User: vagrant
2019-10-15 20:53:08,659    DEBUG  Endpoint: http://192.168.121.91:5985/wsman
2019-10-15 20:53:08,660    DEBUG  Message: WinRM::WinRMAuthorizationError

@f-trivino @netoarmando do you need this reported to PR-CI?

@f-trivino f-trivino added the re-run Trigger a new run of PR-CI label Oct 16, 2019
@freeipa-pr-ci freeipa-pr-ci removed the re-run Trigger a new run of PR-CI label Oct 16, 2019
@f-trivino f-trivino added the prioritized Pull Request has higher priority for PR-CI label Oct 16, 2019
@fcami fcami added needs rebase Pull Request cannot be automatically merged - needs to be rebased pushed Pull Request has already been pushed and removed needs rebase Pull Request cannot be automatically merged - needs to be rebased labels Oct 16, 2019
@fcami
Copy link
Contributor Author

fcami commented Oct 16, 2019

ipa-4-8:

  • 8f8e450 ipa_client_automount.py: fix typo (idmap.conf => idmapd.conf)

@fcami fcami closed this Oct 16, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
ack Pull Request approved, can be merged prioritized Pull Request has higher priority for PR-CI pushed Pull Request has already been pushed
Projects
None yet
Milestone
No milestone
4 participants
@fcami @rcritten @freeipa-pr-ci @f-trivino