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

Package install with HTML reports #177

Merged
merged 14 commits into from
Aug 24, 2023
Merged

Conversation

krystian-panek-vmltech
Copy link
Contributor

@krystian-panek-vmltech krystian-panek-vmltech commented Aug 23, 2023

fixes #160

tested command: sh aemw pkg deploy -A --file '/Users/krystian.panek/Desktop/test-pkg.zip' -f --output-value NONE

benchmark results:

  1. deploy regular - 6.1s, 6.4s, 6.5s (AEM_INSTANCE_CHECK_SKIP=true)
  2. deploy log file - 6.8s, 7.0s, 7.4s (AEM_INSTANCE_CHECK_SKIP=true AEM_INSTANCE_PACKAGE_INSTALL_LOG_ENABLED=true)
  3. deploy log console - 7.0, 7.6s, 8.4s (AEM_INSTANCE_CHECK_SKIP=true AEM_INSTANCE_PACKAGE_INSTALL_LOG_ENABLED=true AEM_INSTANCE_PACKAGE_INSTALL_LOG_CONSOLE=true)

logging to file produces a HTML report coming from CRX Package manager; each installation with file with timestamp

image

summary:

this new option of package installation might be helpful when debugging package install failures however it is slower than the actual one. so keeping the existing install method to be used by default.

@krystian-panek-vmltech krystian-panek-vmltech added the new feature Completely a new feature (not existing before) label Aug 23, 2023
@krystian-panek-vmltech
Copy link
Contributor Author

krystian-panek-vmltech commented Aug 24, 2023

added nice report path in the message:

ERRO[2023-08-24 10:10:36] local_author > cannot install package '/etc/packages/my_packages/test-pkg.zip': HTML report contains errors: aem/home/var/log/package/install/local_author/test-pkg.zip-20230824101035.html

and info that HTML output contains errors:

image

@krystian-panek-vmltech krystian-panek-vmltech changed the title Package install logged Package install with HTML reports Aug 24, 2023
@krystian-panek-vmltech krystian-panek-vmltech merged commit 0ab10db into main Aug 24, 2023
@krystian-panek-vmltech krystian-panek-vmltech deleted the pkg-install-logged branch August 24, 2023 09:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new feature Completely a new feature (not existing before)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Easier troubleshooting of failing package installation
1 participant