magento / inventory

Magento Inventory Project (a.k.a MSI)
Open Software License 3.0
337 stars 247 forks source link

Missing release tag 1.2.0-p1 required by Magento 2.4.0-p1 #3354

Closed Vinai closed 2 years ago

Vinai commented 2 years ago

Summary (*)

The package magento/product-community-edition 2.4.0-p1 requires magento/inventory-metapackage with version 1.2.0-p1. Obviously, the package has been built, since 2.4.0-p1 is installable, but sadly it is not possible to manually resolve the dependency without the release tag.

Proposed solution

  1. Check out the commit that was used to build magento/inventory-metapackage 1.2.0-p1
  2. git tag 1.2.0-p1
  3. git push --tags
m2-assistant[bot] commented 2 years ago

Hi @Vinai. Thank you for your report. To speed up processing of this issue, make sure that you provided sufficient information.

Add a comment to assign the issue: @magento I am working on this


sdzhepa commented 2 years ago

Hello @Vinai

Thank you for reporting and collaboration!

Adobe team start working on this issue and to process it we need additional details from you. Could you please provide more details on what the user/customer tried to do and how? (preconditions, steps, etc)

Unfortunately "sadly it is not possible to manually resolve the dependency without the release tag." is not clear enough

Also, 2.4.0-p1 is also >1 year old, I do not see why a user/customer did not have an issue until now.

Thank you in advance!

Vinai commented 2 years ago

Could you please provide more details on what the user/customer tried to do and how?

When analyzing behavior of a Magento installation without access to the code base, I often use the repositories under https://github.com/magento to see what code is included in a release. This is easier than downloading a given package in a given version, or installing an instance for the given release. In this case, I am looking at a 2.4.0-p1 Magento instance.
In the product metapackage composer.json I can see the dependency for magento/inventory-metapackage with version 1.2.0-p1. My next step is to open https://github.com/magento/inventory/ and select the release tag for 1.2.0-p1 to inspect _metapackage/composer.json. However, this is not possible as there is no 1.2.0-p1 release tag.

There are two branches, 1.2.0-p1-develop and 1.2.0-p1-alpha, but no release tag. I assume 1.2.0-p1-alpha was used as the base to generate the 1.2.0-p1 magento/inventory-metapackage package, but I can't be sure. All other releases are properly tagged. The same is true for all other magento repositories - only this tag is missing.

Another scenario is when independently generating release packages based on the repositories. Without a tag, a special exception has to be created. It would be helpful if the core Magento repositories consistently used release tags.

Also, 2.4.0-p1 is also >1 year old, I do not see why a user/customer did not have an issue until now.

This is correct - the 2.4.0-p1 inventory metapackage exists. The problem is the underlying source for the package can not be determined based on a release tag.

If it where a lot of work to fix this issue, it would probably not be worth the effort. But since it is such little work, it would be helpful if the core team would be able to add the missing tag.

Assuming the branch 1.2.0-p1-alpha was indeed used, it takes only a minute to run:

git clone git@github.com:magento/inventory.git
cd inventory
git checkout 1.2.0-p1-alpha
git tag 1.2.0-p1
git push origin --tags
cd ..
rm -rf inventory
sshrewz commented 2 years ago

Thanks for providing us with details, @Vinai, and for giving us the branch information. Rest assured that we are looking into this and will advise as soon as we have a resolution.

sidolov commented 2 years ago

Hi @Vinai ! 1.2.0-p1 tag and release were created on GitHub: https://github.com/magento/inventory/releases/tag/v1.2.0-p1 I'm closing the issue as done. Thank you for your input!

Vinai commented 2 years ago

@sidolov Thanks for your help, but you tagged the wrong ref.

The tagged version is the commit 6884797: https://github.com/magento/inventory/blob/v1.2.0-p1/_metapackage/composer.json#L3

The correct version is the commit 2a6fdb4: https://github.com/magento/inventory/blob/1.2.0-p1-alpha/_metapackage/composer.json#L3

The correct ref is the head of the branch 1.2.0-p1-alpha. This branch seems to be what was used to build the official release.

On a related note, it would be extremely nice to have consistent tags, that is, without a leading v. To create a tag 1.2.0-p1 the branch with the same name first would need to be deleted or renamed.

I can't reopen the issue. Would you like me to open a new one, or would you prefer to reopen this issue yourself?