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.

Sign up for GitHub

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

Jump to bottom

Add checks after starting bundle Davex when enabling CRXDE #42 #43

Merged
merged 1 commit into from
Sep 30, 2018
Merged

Add checks after starting bundle Davex when enabling CRXDE #42 #43

merged 1 commit into from
Sep 30, 2018

Conversation

mbloch1986
Copy link
Contributor

@mbloch1986
Add checks after starting bundle Davex when enabling CRXDE shinesolut…
51954a2 
…ions#42

* Add Check login_page_is_ready and aem_health_check_is_ok after starting bundle Davex when enabling crxde shinesolutions#42
@cliffano
Copy link
Collaborator

Odd, I'm also seeing this across release/2.2 branch with AEM 6.2 .
I'll apply this patch cross branches since the change involves a timing fix which should impact all AEM versions, not just new package post AEM 6.4 SP1 .

@cliffano cliffano merged commit b83d09f into shinesolutions:master Sep 30, 2018
cliffano added a commit that referenced this pull request Oct 1, 2018
@cliffano
Merge pull request #43 from mbloch1986/fix-42/davex_configuration_fail
f47e79c 
Add checks after starting bundle Davex when enabling CRXDE #42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@mbloch1986 @cliffano