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

Files from update package not installed #81

Closed
jjczopek opened this issue Mar 13, 2019 · 1 comment
Closed

Files from update package not installed #81

jjczopek opened this issue Mar 13, 2019 · 1 comment

Comments

@jjczopek
Copy link

Sitecore 8.2 161221
Sitecore.Ship 0.4.0.150818

Currently we are using TDS .update package to deploy code to servers. TDS is producing the *.files.update. We are using in-house deployment approach with custom controller and action doing it.

We would like to move to Sitecore.Ship, however seems like package is processed (no errors in the logs), JSON with processed entries is being returned, but the files from the package are not updated on the server.

Sitecore logs:

> 1568 19:35:15 INFO  Committing files.
> 1568 19:35:16 INFO  Executing post installation actions.
> 1568 19:35:16 INFO  Executing post installation actions finished.
> 

I had a look into the Ship implementation for building the PackageInstallationInfo and it looks like this:

            var info = new PackageInstallationInfo
            {
                Mode = InstallMode.Install,
                Action = UpgradeAction.Upgrade,
                Path = packagePath
};

Our deployment action however has one difference in that matter - ProcessingMode:

var installationInfo = new PackageInstallationInfo
                {
                    Action = UpgradeAction.Upgrade,
                    Mode = InstallMode.Install,
                    Path = path,
                    ProcessingMode = ProcessingMode.Files
                };

Is this why the files are not showing up on the server after the deployment? If yes - is there any workaround or this should become a feature request?

@jjczopek
Copy link
Author

This can be closed, duplicate of #70 and PR #69 fix it for us.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant