-
Notifications
You must be signed in to change notification settings - Fork 9.3k
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
New Setup System #673
Comments
@tzyganu, thanks for this bunch of comments and questions! We will have them reviewed and provide you with the feedback. |
@peterjaap If you are patient enough you will see it go up to 166% 😄 . |
Weird. It just fell back to 4%;
|
Thanks everyone for the detailed reports. We addressed some of the issues you mentioned and fixed them already, but they will be rolled out later with one of the next releases.
We are aware of this issue. Also nothing prevents running several installation processes at the same time, which will lead to unexpected outcome.
Once the new Setup application has all the features as the former Magento_Install module, we'll delete the latter and all its related code from Magento application.
We have fixed it already
Data "fixture" updates require a full-blown Magento application. For now the simplest solution to isolate it from Magento setup application is to run exec. We'll consider other implementation after refactoring the rest of installation logic (see note about Magento_Install above)
As I said above, we'll revisit it during implementation of the "transaction"
For now, we removed the language selector and dictionaries. It is not implemented yet. But thanks for pointing out.
Noted for the Install Wizard UX work
We are aware of the issue and working on the fix.
Fixed already (will be published soon) |
Last night I let the installer run. Last thing I see in the console is: Then it crashes with this error: |
So I ran this command in the console again: When I hit try again, the installer is insta ready and the installation is a succes :) |
@antonmakarenko my innodb settings were; innodb_buffer_pool_size = 2G So the workaround to increase this doesn't work. |
These MySQL settings are relative. Advising them to a fixed value has no upside. The |
We have reproduced this error and working on a fix. Thanks for reporting. |
On my dev machine the full install takes ~5 minutes. I'm not sure what other settings can help, so here is the full list of settings in my.ini:
After we fix the performance issue with adapters, we expect installation to perform similar to Magento 1 (which was around 30 seconds). |
@antonmakarenko The 166% hasn't bee fixed in alpha98 either. |
Hi @tzyganu |
I think it would be better to get site url on Web Configuration step instead using 'http://example.com/' |
* Released Performance Toolkit * GitHub requests: * [#665](#665) -- Main menu event in wrong area * [#666](#666) -- Update di.xml * [#602](#602) -- Magento\Sales\Model\Order::getFullTaxInfo() incorrectly combines percentages * [#673](#673) -- Fixed fatal error during installation * Functional tests: * Updated API-functional test for Customer and Address metadata service * Add cross sell * Add a product to wishlist * Add up sell * Checkout with gift messages * Create an order from a customer * Create a shipment for offline payment methods * Delete a product from mini shopping cart * Reorder * Sales order report * Updating URL rewrites from a category page * Layout updates: * Moved layout files to the page_layout directory * Moved layout validation files to framework * Theme updates: * Blank Theme layouts & templates were unified * Search Library: * Added ability to aggregate queries for MySQL adapter * Implemented automatic range aggregation for MySQL adapter * Search module: * Introduced the Search module * Moved autocomplete to the Search module * Added base UI to the Search module * Documentation: * Added basic description of modules in the README.md files * Modularity: * Created API and script to get module and dependency information * Framework Improvements: * Decomposed heavy objects basing on profiling results * Refactored the getCustomAttributesCodes method in ProductService * Refactored Customer Model to use Group Model instead of Group Service * Updated Travis configuration to run "composer install" * Performance improvements: * Removed unnecessary "save order" call during order submission step * Fixed missing installation features of the new setup: * Added missing installation parameters: admin_no_form_key, order_increment_prefix, cleanup_database * Fixed the link to the license agreement in web installer * Fixed the web installation wizard which was stuck at 96% * Fixed bugs: * Fixed an issue where newly created attribute was always added to the Product Details tab * Fixed an issue where it was impossible to change the Stock Availability status of a product from the Advanced Inventory tab * Fixed an issue where the Stock Status value changed from In Stock to Out of Stock if quantity was not specified * Fixed an issue where performance toolkit failed in case of unknown argument * Fixed an issue where 404 error page was displayed instead of the URL Rewrite Information page * Fixed an issue where the Click for price link was not working if a product name contained quote mark * Fixed an issue where the Compare products link disappeared after switching to other page * Fixed an issue where the custom logo was not displayed on the category page * Fixed an XSS vulnerability in category name * Fixed an issue where a success save message was not displayed after saving a Search term * Fixed an issue with Google Analytics where it was impossible to add the code to the pages * Fixed an issue where import custom options grid was not displayed on the product creation page * Fixed an issue where it was impossible to retrieve a product collection from category in the "adminhtml" area * Fixed an issue where product attributes were absent on product creation form after switching to another product template * Fixed an issue where the 'URL key for specified store already exists.' error message was displayed when saving a configurable product with variations which have the same name * Fixed an issue where search in the Search Terms Report grid did not work * Fixed an issue where the unnecessary tab "General" was displayed on the Category page in the Admin panel * Fixed an issue where the Stock Status value changed from In Stock to Out of Stock if quantity was not specified for a configurable product when saving to a new template * Fixed an issue where product Stock Status was always set to 'In Stock' if product quantity was specified * Fixed an IE specific issue where for bundle products the Manage Stock option was reset to Yes * Fixed an issue where backorder messages were not displayed * Fixed an issue where the Price field was always required during Bundle product update using ProductService * Fixed an issue where product name was missing in the error message * Fixed an issue where configurable product did not contain a message to select options while adding product from wishlist to shopping cart * Fixed an issue where the Validate VAT Number button did not work during order creation in the Admin panel * Fixed an issue where Item qty in Wishlist got reset after update without changes * Fixed an issue where invoice amount was incorrect when items with discount were partially invoiced * Fixed product thumbnails alignment in the storefront * Fixed an issue where inactive Categories were not greyed out in the tree in the Admin panel * Fixed an issue where it was impossible to disable debug mode * Fixed the code sample in the index.php file * Removed language selector in the setup UI * Fixed an issue where setup was broken if db_prefix was used * Implemented usage of Symfony's PHPExecutableFinder for executing CLI tools * Fixed an issue with the Import/Export functionality * Fixed an issue with catalog product/category and category/product indexers invalidation after import * Fixed an issue with entering invalid date in the Product Views Report * Fixed an issue where it was impossible to view orders for customers from a deleted customer group * Fixed an issue where a duplicate customer record was created after adding an order from the Admin panel * Fixed an issue where it was impossible to log in to the Admin panel from the first attempt
Issue referenced in the changelog by mistake instead of internally fixed bug. The team is in progress on this one. |
During installation, when magento trying to install data fixtures, I'm getting this errors:
|
@van-gog Please clear your |
@mazhalai this helped. Thanks |
@tzyganu, may I ask you to provide a real use-case of having ability to run 2 install processes on the same machine at the same time? It will helps us to understand if we should block or allow this ability. |
@tanya-soroka Sure. Here is one scenario. It is not very probable but it's possible. |
As far as the 'exec' goes, what else is setup is supposed to do after installing the fixtures? I ran the install line manually, then realized the store had to be taken out of Maintenance mode. Fixtures are 88/93 steps. If Maintenance is one step, what are the last four items? The exec should definitely be addressed and the install made to be more flexible. |
@Morgon , last steps now are:
You can find list of all steps in the installation model - https://github.com/magento/magento2/blob/master/setup/module/Magento/Setup/src/Model/Installer.php#L235, they are declared first end then executed one by one (you can approximately find last steps, while real number depends on number of install scripts) |
@tzyganu we have fixed most of the issues listed in this post. I am closing this issue, as fixed. Feel free to open new issue if we missed anything. |
@muasir everything seams to work now. Thank the team from me for the great job. |
The issues you’ve mentioned about the Magento 2 setup, such as the accessible setup page post-installation and the use of hardcoded values in setup models, are crucial points that need addressing to ensure a secure and efficient Magento 2 installation. It's important to follow best practices for a secure and optimized Magento 2 setup. For step-by-step instructions and solutions to common setup challenges, you might want to explore detailed Magento 2 setup guides available online. |
I know that the setup system available since alpha 96 is just at the beginning and you already know some issues about it but I thought to add some in here:
Magento_Install
module?authorization_role
table (oldadmin_role
table), where the role_nameAdministrators
has an other id beside 1 I cannot login in the backend because the parent role is hardcoded inMagento\Setup\Model\AdminAccount
.exec
command? It seams to me like it introduces an unnecessary dependency. Some servers may not allow this. If it's necessary, maybe a check should be made and notify the user that the server does not allowexec
.install.log
. I think it's better to have a unique name. If I run 2 install processes on the same machine at the same time the file is locked by the first install and I cannot see the progress on my second install. I know this is a rare case, but still....The text was updated successfully, but these errors were encountered: