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

Warnings on Readiness and Best Practice Analyzer #230

Closed
pcastelog opened this issue Apr 23, 2024 · 2 comments
Closed

Warnings on Readiness and Best Practice Analyzer #230

pcastelog opened this issue Apr 23, 2024 · 2 comments

Comments

@pcastelog
Copy link

pcastelog commented Apr 23, 2024

Evaluating the move to cloud, some warnings appeared in the report (https://experienceleague.adobe.com/en/docs/experience-manager-learn/cloud-service/migration/cloud-acceleration-manager/readiness-and-best-practice-analyzer)

  • /oak:index/aecuHistory-custom-1,The /oak:index/aecuHistory-custom-1 Oak index has a rule violation: reindex property is not allowed
  • /oak:index/aecuHistory-custom-1,The /oak:index/aecuHistory-custom-1 Oak index has a rule violation: seed property is not allowed

The second one I believe is an bug on the tool since can't find the property in https://github.com/valtech/aem-easy-content-upgrade/blob/develop/oak.index/src/main/content/jcr_root/_oak_index/aecuHistory-custom-1/.content.xml

Also, I see that after v.5.0.0 the oak:index was moved from /var/aecu but never removed programmatically, since the tool also found

  • /var/aecu/oak:index/aecuHistory,The /var/aecu/oak:index/aecuHistory Oak index has a rule violation: Custom definition name must follow the pattern -custom-
  • /var/aecu/oak:index/aecuHistory,The /var/aecu/oak:index/aecuHistory Oak index has a rule violation: Path must begin with /oak:/index
  • /var/aecu/oak:index/aecuHistory,The /var/aecu/oak:index/aecuHistory Oak index has a rule violation: reindex property is not allowed
  • /var/aecu/oak:index/aecuHistory,The /var/aecu/oak:index/aecuHistory Oak index has a rule violation: seed property is not allowed
@pcastelog
Copy link
Author

#231

@nhirrle
Copy link
Collaborator

nhirrle commented May 30, 2024

hi @pcastelog
Just released 6.5.1 - can we close the case?

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

2 participants