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

Upgrading via composer deletes dev/tests/integration/etc/install-config-mysql.php #3249

Closed
Vinai opened this issue Feb 2, 2016 · 12 comments
Closed
Labels
Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development

Comments

@Vinai
Copy link
Contributor

Vinai commented Feb 2, 2016

In order to execute integration tests, I copied the file dev/tests/integration/etc/install-config-mysql.php.dist to dev/tests/integration/etc/install-config-mysql.php and adjusted the mysql connection credentials for the test database.
After the upgrade I discovered the file has been deleted.
I would expect local copies to persist between upgrades, just like local phunit.xml copies do, so it seems like a bug to me.

@Vinai Vinai added the PS label Feb 2, 2016
@davidalger
Copy link
Member

Seems similar to #3097

@Vinai
Copy link
Contributor Author

Vinai commented Feb 2, 2016

True that. In this case the initial install was done via composer create-project, but probably the same issue.

@davidalger
Copy link
Member

My guess is the fault is buried somewhere in magento-composer-installer, since it's what's copying in the stuff from magento-base. I just don't have time to dig in to find out atm :)

@Vinai
Copy link
Contributor Author

Vinai commented Feb 3, 2016

With the modules residing in vendor now, and projects being created via composer create-project, I wonder if the magento composer installer is still needed at all. My feeling is that it should be possible to remove it without too much effort.

@davidverholen
Copy link
Member

The only things that the installer should still do is create the base folder structure for the web app and the vendor_path.php, right?

The current installer still has all the features from the hackathon magento1 installer. That seems very overloaded.

@davidalger
Copy link
Member

The composer create-project just uses the package you reference as a basis for the new project. One of the dependencies of the magento/project-community-edition package is magento/magento2-base which is what requires the composer installer. Last I checked, everything outside of vendor comes in via this magento/magento2-base package, and the composer installer is the mechanism used to copy / move the files from that into place when create-project runs.

@davidverholen
Copy link
Member

ok then, if the magento/magento2-base only contains the base folder structure for the web app, wouldn't it be better to directly download it to the web root instead of first downloading it to vendor and then copy it there.

The approach of symlinking / copying the files is only a leftover of the magento1 installer where it was necessary to have all files at specific places in your web root

@Vinai
Copy link
Contributor Author

Vinai commented Mar 31, 2016

Just experienced this issue again while upgrading from 2.0.2 to 2.0.3.
On a side note, the custom files dev/tests/*/phpunit.xml did not get deleted during the upgrade.

@alankent
Copy link

http://alankent.me/2016/03/31/m2-project-source-code-management-recommendation/ talks about this a bit as well (but probably does not add much to the above discussion).

@Vinai
Copy link
Contributor Author

Vinai commented May 9, 2016

Still happens when upgrading from 2.0.4 to 2.0.5.

@mazhalai
Copy link
Contributor

mazhalai commented May 9, 2016

@ALL have created ticket MAGETWO-52617 to investigate and fix.

@KrystynaKabannyk
Copy link

Hello @Vinai, this issue has been fixed in the 2.1.0 Release, that's why I'm closing it. If you any questions or additional information regarding the issue feel free to reopen it or create a new one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development
Projects
None yet
Development

No branches or pull requests

6 participants