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

ACS-6112: re-enable disabled test #2624

Conversation

krdabrowski
Copy link
Contributor

Re-enable testSecondaryAncestorWithNodeHavingComplexSecondaryRelationship

  • open debug port for live-indexer in E2Es

…ryRelationship

- open debug port for live-indexer in E2Es
…ryRelationship

- open debug port for live-indexer in E2Es
Copy link
Contributor

@mpichura mpichura left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

.withEnv("SPRING_ACTIVEMQ_BROKERURL", "nio://activemq:61616")
.withEnv("ALFRESCO_SHAREDFILESTORE_BASEURL", "http://shared-file-store:8099/alfresco/api/-default-/private/sfs/versions/1/file/")
.withEnv("ALFRESCO_ACCEPTEDCONTENTMEDIATYPESCACHE_BASEURL", "http://transform-core-aio:8090/transform/config")
.withEnv("JAVA_TOOL_OPTIONS", "-agentlib:jdwp=transport=dt_socket,address=*:5005,server=y,suspend=n")
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

5005 because ACS is opening 8000, correct?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, and also this port is often used for live-indexer debugging... or at least I think so after seeing our d-c files 😉 ps. 5006 is often used for re-indexer debugging

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks. BTW - do not merge this PR before GA release is finished.

@krdabrowski krdabrowski merged commit a3c2456 into master Nov 2, 2023
@krdabrowski krdabrowski deleted the feature/ACS-6112_re-enable_testSecondaryAncestorWithNodeHavingComplexSecondaryRelationship branch November 2, 2023 10:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants