Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 534448

Summary: [pmi] Committers with no Drupal user are skipped when provisioning a project
Product: Community Reporter: Wayne Beaton <wayne.beaton>
Component: Project Management & PortalAssignee: Eclipse Web <webdev>
Status: RESOLVED FIXED QA Contact:
Severity: critical    
Priority: P3 CC: eric.poirier, wayne.beaton, webmaster
Version: unspecified   
Target Milestone: ---   
Hardware: PC   
OS: Linux   
Whiteboard:
Attachments:
Description Flags
Something went wrong none

Description Wayne Beaton CLA 2018-05-07 15:33:14 EDT 
The project provisioning process initiates the committer paperwork process for each listed committer.

There's a test at the top of the function (projects_provisioning__provision) that initiates this entire process that checks to see if the user has a Drupal identity; all users that cannot be loaded as a Drupal user are skipped.

This basically means that all users who do not already have some sort of role, or who have not logged into projects.eclipse.org are skipped when provisioning.

I believe that the real test should be that the user exists in LDAP.

I'm marking this critical because it prevents me from provisioning any project.
Comment 1 Eric Poirier CLA 2018-05-08 13:42:08 EDT 
Hi Wayne,

I merged my patch to production. You should now be able to create paperworks for users that are in LDAP rather than in the projects websites only.

Please reopen this bug if there's an issue with the change.
Comment 2 Wayne Beaton CLA 2018-05-09 23:14:17 EDT 
Something went wrong.
Comment 3 Wayne Beaton CLA 2018-05-09 23:14:44 EDT 
Created attachment 273985 [details]
Something went wrong
Comment 4 Eric Poirier CLA 2018-05-10 08:35:02 EDT 
(In reply to Wayne Beaton from comment #3)
> Created attachment 273985 [details]
> Something went wrong

The provisioning function was still trying to create a paperwork based on the user_load() result... I created the following patch to create the paperwork using the ldap user id if it exists.

https://foundation.eclipse.org/r/#/c/2563/1

Also as a side note, if the user doesn't have an account on accounts.eclipse.org, one will be created if a record in LDAP is found.
Comment 5 Eric Poirier CLA 2018-05-10 09:49:31 EDT 
My patch is now on production. Let me know if all goes well.
Comment 6 Wayne Beaton CLA 2018-05-10 10:35:57 EDT 
(In reply to Eric Poirier from comment #5)
> My patch is now on production. Let me know if all goes well.

It works! Thanks!