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

AEMaaCS | Missing users in Publishers #3285

Closed
3 tasks done
singhal425 opened this issue Mar 5, 2024 · 4 comments
Closed
3 tasks done

AEMaaCS | Missing users in Publishers #3285

singhal425 opened this issue Mar 5, 2024 · 4 comments
Labels
Milestone

Comments

@singhal425
Copy link
Contributor

Required Information

  • AEM Version, including Service Packs, Cumulative Fix Packs, etc: AEMaaCS 2023.12.14697.20231215T125030Z
  • ACS AEM Commons Version: 6.0.8
  • Reproducible on Latest? yes/no: yes

Expected Behavior

On publishers, errors appear in logs for missing service users.
2024-01-01T16:23:34+01:00 01.01.2024 15:23:34.744 [cm-p23958-e477620-aem-publish-856487b4bb-xznlf] ERROR [pool-7-thread-1] com.adobe.granite.repository.impl.SystemPrincipalsValidation Principal 'acs-commons-twitter-updater-service' does not exist. Unable to validate.

2024-01-01T16:23:34+01:00 01.01.2024 15:23:34.730 [cm-p23958-e477620-aem-publish-856487b4bb-xznlf] ERROR [pool-7-thread-1] com.adobe.granite.repository.impl.SystemPrincipalsValidation Principal 'acs-commons-package-replication-status-event-service' does not exist. Unable to validate.

Actual Behavior

These errors shouldn't appear in logs.

Steps to Reproduce

Deploy mentioned version of ACS common on AEM as cloud service and these logs would start to appear with missing service user information.

Links

Links to related assets, e.g. content packages containing test components
Following config doesn't have mentioned service users defined for Publisher.

@davidjgonzalez davidjgonzalez added this to the 6.5.2 milestone Mar 22, 2024
@davidjgonzalez
Copy link
Contributor

It looks like the service user mappings are set in the config but the repo init scripts that create the users are only in config.author .. we should move the user mappings to config.author as well

@singhal425
Copy link
Contributor Author

@davidjgonzalez Thanks and yes, it's exactly the same. I want to raise a PR for the same but not able to create branch. Is it possible for me to do so?

@kwin
Copy link
Contributor

kwin commented Apr 2, 2024

Please create a PR from a branch in your fork.

@singhal425
Copy link
Contributor Author

@kwin @davidjgonzalez Here's the PR #3310

@kwin kwin closed this as completed May 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants