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

User account mapping #140

Merged
merged 5 commits into from
Dec 8, 2016
Merged

User account mapping #140

merged 5 commits into from
Dec 8, 2016

Conversation

smalik86
Copy link

@smalik86 smalik86 commented Nov 22, 2016

Hi @olivierdagenais , these are my testing results.

I ran the build through Maven and made sure that the jar and hpi files were successfully created.
Once the hpi file was created, I uploaded the file and installed it on our company's Test Jenkins.
The plugin worked as expected and the UI interface made it easy for us to chose whether to keep the User ID with the domain name or User ID without the domain name.

Example: DomainName/User ID to Only User ID
tfs ui

@olivierdagenais
Copy link
Member

Those are some mice commits, @smalik86! 😃

Given I haven't personally done any testing, I'm assuming you found these changes to be satisfactory at your organization? If so, can I trouble you to add a comment to this pull request with the validation you performed? A great example of what I'm looking for can be seen in the Manual Testing section of #136.

Thanks!
- Oli

@olivierdagenais olivierdagenais self-assigned this Nov 23, 2016
@smoyen
Copy link

smoyen commented Dec 7, 2016

Manual testing

Test 1: In Manage Jenkins > Configure System, set User account name mapping strategy is set to Resolve user using ‘alias’ only, removing ‘DOMAIN\’

  1. Create a job using TFS for source control and turn polling on
  2. Check in a change
  3. After the build kicks off, examine the user listed in the changes. Notice that it does not include the domain
  4. If your Jenkins user ids do not include the domain, you will have the added benefit of the build history for a user including build kicked off Manually and by SCM Trigger

Test 2: In Manage Jenkins > Configure System, set User account name mapping strategy is set to Resolve user using ‘DOMAIN\alias’

  1. Create a job using TFS for source control and turn polling on
  2. Check in a change
  3. After the build kicks off, examine the user listed in the changes. Notice that it does include the domain
  4. If your Jenkins user ids do not include the domain, the build histories for builds kicked off Manually and by SCM Trigger will be listed under separate users (i.e. Manually builds will be listed under user named userId and SCM Trigger builds will be listed under user named domainName_userId)

*Jobs that use anything besides TFS for source control are not affected. Manual Build user name is not affected for TFS and non-TFS sourced jobs.

@olivierdagenais
Copy link
Member

@smalik86 and @smoyen: Thank you for validating these changes! Your contribution is greatly appreciated.

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.

3 participants