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

Small improvement in Zend stack #2910

Merged
merged 1 commit into from
Oct 28, 2016
Merged

Conversation

kaloyan-raev
Copy link
Contributor

What does this PR do?

This is a small improvement in the Zend stack.

When running inside a Docker container, Zend Server needs an additional configuration before starting in order for Z-Ray to run properly. The zray.zendserver_ui_url needs to be configured with the current exposed port number of port 10081 (Zend Server Admin UI port). This PR improve this configuration step.

What issues does this PR fix or reference?

Signed-off-by: Kaloyan Raev kaloyan.r@zend.com

Signed-off-by: Kaloyan Raev <kaloyan.r@zend.com>
@codenvy-ci
Copy link

Can one of the admins verify this patch?

@benoitf
Copy link
Contributor

benoitf commented Oct 27, 2016

ci-build

@codenvy-ci
Copy link

Build # 810 - FAILED

Please check console output at https://ci.codenvycorp.com/job/che-pullrequests-build/810/ to view the results.

@slemeur slemeur added the kind/enhancement A feature request - must adhere to the feature request template. label Oct 28, 2016
@slemeur
Copy link
Contributor

slemeur commented Oct 28, 2016

Thanks @kaloyan-raev OK for me.

@slemeur slemeur merged commit 28fc3f8 into eclipse-che:master Oct 28, 2016
@kaloyan-raev kaloyan-raev deleted the zend-stack branch October 28, 2016 12:37
JPinkney pushed a commit to JPinkney/che that referenced this pull request Aug 17, 2017
… Zend Stack (eclipse-che#2910)

Signed-off-by: Kaloyan Raev <kaloyan.r@zend.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/enhancement A feature request - must adhere to the feature request template.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants