Alter acquia_spi to send only the necessary site variables instead of all of them.

CommentFileSizeAuthor
#6 acquia_connector-specific-variables-1404474-d6.patch1.71 KBcoltrane
#6 acquia_connector-specific-variables-1404474-d7.patch1.84 KBcoltrane
#1 acquia_connector-specific-variables-1404474-d7.patch1.41 KBcoltrane
#1 acquia_connector-specific-variables-1404474-d6.patch1.36 KBcoltrane

Comments

coltrane’s picture

coltrane
he/him
commented
Status: Active » Needs review
StatusFileSize
new acquia_connector-specific-variables-1404474-d6.patch1.36 KB
new acquia_connector-specific-variables-1404474-d7.patch1.41 KB

Variables:
'googleanalytics_cache', 'error_level', 'preprocess_js', 'page_cache_maximum_age', 'block_cache', 'preprocess_css', 'page_compression', 'cache', 'cache_lifetime', 'cron_last', 'clean_url', 'redirect_global_clean', 'theme_zen_settings', 'site_offline', 'site_name', 'user_register', 'user_admin_role'

rickmanelius’s picture

rickmanelius commented

I don't know if this qualifies as RBTC, but this patch solved my other issue #1439310: Unable to send SPI data via cron on production server in this ticket for the acquia_connector-7.x-2.1.

rickmanelius’s picture

rickmanelius commented

I made the mistake of upgrading/updating again to 7.x-2.2. I'm going to see if I can reroll the patches and get this reset again.

rickmanelius’s picture

rickmanelius commented

Actually it looks like the patch in #1 for D7 it still applicable as is without modification. Testing again...

rickmanelius’s picture

rickmanelius commented
Version: 6.x-2.x-dev » 7.x-2.2
Status: Needs review » Reviewed & tested by the community

I've been using this patch for 7.x-2.2 and 7.x-2.1 and it provides me with a working connection to the Acquia Network. Without it, cron tends to hang perpetually.

Marking RBTC and hoping it gets committed prior to the 7.x-2.3 release.

coltrane’s picture

coltrane
he/him
commented
Status: Reviewed & tested by the community » Needs review
StatusFileSize
new acquia_connector-specific-variables-1404474-d7.patch1.84 KB
new acquia_connector-specific-variables-1404474-d6.patch1.71 KB

#1 didn't make it into 2.2 because I wanted to confirm that we had all the variables desired, since this patch moves from sending all to sending a select amount.

Today I audited a couple sites and update the list of variables.

rickmanelius’s picture

rickmanelius commented
Status: Needs review » Reviewed & tested by the community

I just tested the D7 patch in #6 and I was able to get proper connect from both Drupal's status report and the Acquia Network's Insight tab "Last Data Update 6 minutes ago".

I don't have a D6 site using the acquia network, so I can't vouch for that patch yet. Marking RBTC for D7 and possibly re-open for the D6 version.

coltrane’s picture

coltrane
he/him
commented
Status: Reviewed & tested by the community » Fixed

Committed to 2.x branches. Will be in dev releases till next Connector release (planned to coincide with next Drupal core release).

http://drupalcode.org/project/acquia_connector.git/commit/b45f9b9
http://drupalcode.org/project/acquia_connector.git/commit/b34e22b

rickmanelius’s picture

rickmanelius commented

Much appreciated! My clients and I thank you :)

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.