Skip to content
This repository has been archived by the owner on Jun 7, 2021. It is now read-only.

[TRAFODION-2579] installers should allow multiple ldap hosts and ldap UID lines #1049

Merged
merged 1 commit into from Apr 8, 2017
Merged

[TRAFODION-2579] installers should allow multiple ldap hosts and ldap UID lines #1049

merged 1 commit into from Apr 8, 2017

Conversation

mkby
Copy link
Contributor

@mkby mkby commented Apr 6, 2017

changes for pyinstaller part.

@Traf-Jenkins
Copy link

Check Test Started: https://jenkins.esgyn.com/job/Check-PR-master/1709/

@Traf-Jenkins
Copy link

Test Passed. https://jenkins.esgyn.com/job/Check-PR-master/1709/

svarnau
svarnau approved these changes Apr 6, 2017
View reviewed changes
Copy link
Contributor

@svarnau svarnau left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good for the python-installer side.

DaveBirdsall
DaveBirdsall approved these changes Apr 6, 2017
View reviewed changes
Copy link
Contributor

@DaveBirdsall DaveBirdsall left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1 Looks good to me

@asfgit asfgit merged commit 4de1923 into apache:master Apr 8, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@svarnau svarnau svarnau approved these changes

@DaveBirdsall DaveBirdsall DaveBirdsall approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
5 participants
@mkby @Traf-Jenkins @svarnau @DaveBirdsall @asfgit