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
ipatests: add check for output contents of ipa-client-samba #4164
Conversation
2b1f380
to
e8b8a86
Compare
domain_regexp_tpl = r''' | ||
Domain\ name:\s*{domain}\n | ||
\s*NetBIOS\ name:\s*{netbios}\n | ||
\s*SID:\s+S-1-5-\d+-\d+-\d+-\d+\n |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please check for S-1-5-21-<domain>
here. (21 is important). See https://support.microsoft.com/en-us/help/243330/well-known-security-identifiers-in-windows-operating-systemsfor details.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
8c6a30f
to
84c91f8
Compare
Check that ipa-client-samba tool reports specific properties of domains: name, netbios name, sid and id range Related to https://pagure.io/freeipa/issue/8149
84c91f8
to
54b7f08
Compare
@abbra Please review |
LGTM. ACK. |
54b7f08
to
d4156ad
Compare
Removed temp commit, waiting for tests |
master:
|
Check that ipa-client-samba tool reports specific properties of domains:
name, netbios name, sid and id range
Related to https://pagure.io/freeipa/issue/8149