Bug 1069508 - [origin_cartridge_57] Some warning shown in web when visit php-5.3/5.4 upstream cakephp quickstart
Summary: [origin_cartridge_57] Some warning shown in web when visit php-5.3/5.4 upstre...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: OpenShift Online
Classification: Red Hat
Component: Image
Version: 2.x
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: ---
: ---
Assignee: Vojtech Vitek
QA Contact: libra bugs
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2014-02-25 07:37 UTC by Yan Du
Modified: 2015-05-15 00:38 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2014-03-12 03:08:33 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
cakephp (429.73 KB, image/png)
2014-02-26 07:03 UTC, Yan Du
no flags Details

Description Yan Du 2014-02-25 07:37:32 UTC
Description of problem:

Visit the app in web after create php-5.3/5.4 upstream cakephp quickstart, shown below warning:

Warning (2): include_once(/var/lib/openshift/530c01f15254670b3b00005b/app-root/runtime/repo/app/Config/database.php) [function.include-once]: failed to open stream: No such file or directory [CORE/Cake/Model/ConnectionManager.php, line 67]
Warning (2): include_once() [function.include]: Failed opening '/var/lib/openshift/530c01f15254670b3b00005b/app-root/runtime/repo/app/Config/database.php' for inclusion (include_path='/var/lib/openshift/530c01f15254670b3b00005b/app-root/runtime/repo/lib:.:/usr/share/pear:/usr/share/php') [CORE/Cake/Model/ConnectionManager.php, line 67]



Version-Release number of selected component (if applicable):
devenv_4415



How reproducible:
Always



Steps to Reproduce:

1. Create cakephp app
# rhc app create cake54 php-5.4 mysql-5.1 --from-code=https://github.com/cakephp/cakephp.git
# rhc app create cake53 php-5.3 mysql-5.1 --from-code=https://github.com/cakephp/cakephp.git

2. Visit the app in web




Actual results:
shown some warning in webpage



Expected results:
should not show such warnning as decription

Comment 1 Yan Du 2014-02-25 10:47:54 UTC
And the cakephp is not working when download the file from https://github.com/cakephp/cakephp/tarball/1.3.10

steps:
1. Create a php app

2. download cakephp package from
https://github.com/cakephp/cakephp/tarball/1.3.10 , and unzip it
tar zvf some/dir/cakephp-cakephp-1.3.10-0-g8671aa3.tar.gz

3.copy all files and dirs to php/ dir of libra app git repo
#cp -rf some/dir/cakephp-cakephp-b0abad1/* php1/php

4.git push and visit the page in web

After push, we can not get 'CakePHP: the rapid development php framework'  homepage, and before the php DocumentRoot logic changed, it works well.

Comment 2 Vojtech Vitek 2014-02-25 13:15:16 UTC
https://github.com/openshift/origin-server/pull/4818

Comment 4 Yan Du 2014-02-26 07:02:52 UTC
Test on devenv_4420

1. when create cakephp by #rhc app create cake54 php-5.4 mysql-5.1 --from-code=https://github.com/cakephp/cakephp.git, we still got some notice and warning in webpage.

2. try to create the cakephp according the comment1, after app created, the webpage is a little mess, you can refer to the pic in attachment.

Comment 5 Yan Du 2014-02-26 07:03:49 UTC
Created attachment 867817 [details]
cakephp

Comment 6 Vojtech Vitek 2014-02-26 14:14:08 UTC
@Yan Du, in my opinion, there are no more Errors/Warnings that could be considered a bug.

> Notice (1024): Please change the value of 'Security.salt' in APP/Config/core.php to a salt value specific to your application. [CORE/Cake/Utility/Debugger.php, line 845]

This tells users to set custom value of Security.salt.

> Notice (1024): Please change the value of 'Security.cipherSeed' in APP/Config/core.php to a numeric (digits only) seed value specific to your application. [CORE/Cake/Utility/Debugger.php, line 849]

This tells users to set custom value of Security.cipherSeed.

> Your version of PHP is 5.2.8 or higher.
> 
> Your tmp directory is writable.
> 
> The FileEngine is being used for core caching. To change the config edit APP/Config/core.php
> 
> Your database configuration file is NOT present.
> Rename APP/Config/database.php.default to APP/Config/database.php

This tells users to create the database config file. See warnings at bottom.

> 
> DebugKit is not installed. It will help you inspect and debug different aspects of your application.
> You can install it from GitHub
> 
> Editing this Page
> To change the content of this page, edit: APP/View/Pages/home.ctp.
> To change its layout, edit: APP/View/Layouts/default.ctp.
> You can also add some CSS styles for your pages at: APP/webroot/css.
> 
> CakePHP: the rapid development php framework
> Warning (2): include_once(/var/lib/openshift/530df479488508a1f60002b0/app-root/runtime/repo/app/Config/database.php): failed to open stream: No such file or directory [CORE/Cake/Model/ConnectionManager.php, line 67]
> Warning (2): include_once(): Failed opening '/var/lib/openshift/530df479488508a1f60002b0/app-root/runtime/repo/app/Config/database.php' for inclusion (include_path='/var/lib/openshift/530df479488508a1f60002b0/app-root/runtime/repo/lib:.:/var/lib/openshift/530df479488508a1f60002b0/app-root/runtime/repo/lib:/var/lib/openshift/530df479488508a1f60002b0/app-root/runtime/repo/libs:/var/lib/openshift/530df479488508a1f60002b0/app-root/runtime/repo/libraries:/var/lib/openshift/530df479488508a1f60002b0/app-root/runtime/repo/src:/var/lib/openshift/530df479488508a1f60002b0/app-root/runtime/repo/vendor:/var/lib/openshift/530df479488508a1f60002b0/app-root/runtime/repo/vendors:/var/lib/openshift/530df479488508a1f60002b0/php/phplib/pear/pear/php:/usr/share/pear') [CORE/Cake/Model/ConnectionManager.php, line 67]

Warnings related to the missing database config file.

Comment 7 Vojtech Vitek 2014-02-26 14:18:53 UTC
Note to the above: Don't forget we're pulling the code straight from the upstream code. There are no OpenShift-specific values (as DB string etc.) set yet - this is not OpenShift QuickStart/Example. CakePHP users are used to configure these files themselves.

Comment 8 Yan Du 2014-02-27 04:49:58 UTC
Move to verified according comment6


Note You need to log in before you can comment on or make changes to this bug.