For Acquia Search Index - it is set to readonly in code if conditions are not met. This is good but during fresh install it requires manual change from admin as it is never reverted back to writable when all conditions are met.

CommentFileSizeAuthor
#16 2922201-16.patch1.84 KBsanket1007
#9 2922201-9.patch1.8 KBnikunjkotecha
#7 2922201-7.patch1.87 KBnikunjkotecha
#5 2922201-4.patch1.7 KBnikunjkotecha
#2 2922201.patch1.7 KBnikunjkotecha

Comments

nikunjkotecha created an issue. See original summary.

nikunjkotecha’s picture

nikunjkotecha
he/him
Primary language Gujarati
Location India, Gujarat, Rajkot
commented
StatusFileSize
new 2922201.patch1.7 KB

Using variable in State to check if read_only status was set through code and conditions are met now, we unset read_only flag.

nikunjkotecha’s picture

nikunjkotecha
he/him
Primary language Gujarati
Location India, Gujarat, Rajkot
commented
Status: Active » Needs review

Status: Needs review » Needs work

The last submitted patch, 2: 2922201.patch, failed testing. View results

nikunjkotecha’s picture

nikunjkotecha
he/him
Primary language Gujarati
Location India, Gujarat, Rajkot
commented
Status: Needs work » Needs review
StatusFileSize
new 2922201-4.patch1.7 KB

Wrong sequence of branches while creating patch :(

Corrected now

nikunjkotecha’s picture

nikunjkotecha
he/him
Primary language Gujarati
Location India, Gujarat, Rajkot
commented
Status: Needs review » Needs work

Didn't work :(

nikunjkotecha’s picture

nikunjkotecha
he/him
Primary language Gujarati
Location India, Gujarat, Rajkot
commented
Status: Needs work » Needs review
StatusFileSize
new 2922201-7.patch1.87 KB

Trying again.

Status: Needs review » Needs work

The last submitted patch, 7: 2922201-7.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

nikunjkotecha’s picture

nikunjkotecha
he/him
Primary language Gujarati
Location India, Gujarat, Rajkot
commented
Status: Needs work » Needs review
StatusFileSize
new 2922201-9.patch1.8 KB

Verified with livedev on Cloud, should work now.

Status: Needs review » Needs work

The last submitted patch, 9: 2922201-9.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

danepowell’s picture

danepowell commented
Status: Needs work » Postponed (maintainer needs more info)

Is this still an issue? If so can you please provide steps to reproduce? I'm not sure what's meant by a "fresh install" here. Normally when you enable Acquia Connector and Acquia Search for the first time they should automatically connect to the appropriate search core.

danepowell’s picture

danepowell commented
Status: Postponed (maintainer needs more info) » Closed (outdated)

Closing after a week of inactivity, per the contribution guidelines. Feel free to reopen if you can provide the requested information. Cheers.

nikunjkotecha’s picture

nikunjkotecha
he/him
Primary language Gujarati
Location India, Gujarat, Rajkot
commented
Status: Closed (outdated) » Postponed (maintainer needs more info)

@Dane: This would need testing in specific conditions, I reported for a project which uses ACSF and custom profile.

I would need to revert the patch and test on some environment with Acquia Cloud connectivity which is not possible for me now and this also requires custom code to be present, so I don't think it is straight forward

Steps:
* Add a custom module which uses acquia_search and adds an Index (config/install) which uses acquia_search.
* Install a new site on Acquia Cloud (ACSF) with a profile that enables the module during installation (using ACSF interface)
* Observe the Index status after site is installed

nikunjkotecha’s picture

nikunjkotecha
he/him
Primary language Gujarati
Location India, Gujarat, Rajkot
commented
Status: Postponed (maintainer needs more info) » Needs review
danepowell’s picture

danepowell commented
Status: Needs review » Closed (cannot reproduce)

I still cannot reproduce this. If you are able to provide a failing test case feel free to reopen!

sanket1007’s picture

sanket1007 commented
StatusFileSize
new 2922201-16.patch1.84 KB