Skip to content

Next

  • Projects
  • Groups
  • Snippets
  • Help
  • Sign in / Register
Minds Backend - Engine
Minds Backend - Engine
  • Project overview
  • Repository
  • Issues 290
    • List
    • Boards
    • Labels
    • Service Desk
    • Milestones
  • Merge Requests 37
  • CI / CD
  • Security & Compliance
  • Packages
  • Wiki
  • Snippets
  • Members
  • Collapse sidebar
  • Graph
  • Charts
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
  • Minds
  • Minds Backend - EngineMinds Backend - Engine
  • Issues
  • #777

Closed
Open
Opened 2 months ago by Mark Harding@markeharding
Report abuse New issue

(chore): Catch RedisException when using pubsub publis - RedisException: Redis server went away

Sentry Issue: ENGINE-F

RedisException: Redis server went away
  File "/Core/Data/PubSub/Redis/Client.php", line 41, in Redis::publish
    return $this->redis->publish($channel, $data);
  File "/Core/Data/PubSub/Redis/Client.php", line 41, in Minds\Core\Data\PubSub\Redis\Client::publish
    return $this->redis->publish($channel, $data);
  File "/Core/Sockets/Events.php", line 86, in Minds\Core\Sockets\Events::emit
    $this->redis->publish($this->prefix . $packet['nsp'] . '#', $packed);
  File "/Core/Notification/UpdateMarkers/Manager.php", line 46, in Minds\Core\Notification\UpdateMarkers\Manager::pushToSocketRoom
    ->emit("marker:{$marker->getEntityGuid()}", json_encode($marker->export()));
  File "/Controllers/api/v2/notifications/markers.php", line 94, in Minds\Controllers\api\v2\notifications\markers::put
    $manager->pushToSocketRoom($marker);
...
(4 additional frame(s) were not displayed)

Related issues
0

Related merge requests
1
  • (fix) Catch RedisException when using pubsub
    !314
When this merge request is accepted, this issue will be closed automatically.
  • Discussion 6
  • Designs 0
  • Mark Harding @markeharding changed weight to 1 2 months ago

    changed weight to 1

  • Mark Harding @markeharding assigned to @gthouret 2 months ago

    assigned to @gthouret

  • Mark Harding @markeharding added Priority::1 - High Type::Bug scoped labels 2 months ago

    added scoped labels

  • Guy Thouret
    Guy Thouret @gthouret · 2 months ago
    Developer

    Hoping it's not related to Redis limitations in: https://tech.trivago.com/2017/01/25/learn-redis-the-hard-way-in-production

  • Guy Thouret
    Guy Thouret @gthouret · 2 months ago
    Developer

    I modified the Redis PubSub interface so that connection issues should be handled and false returned instead of throwing a RedisException.

    Edited by Guy Thouret 2 months ago
  • Guy Thouret @gthouret mentioned in commit 69ea2858 2 months ago

    mentioned in commit 69ea2858

  • Guy Thouret @gthouret added Status::InProgress scoped label 2 months ago

    added scoped label

  • Guy Thouret @gthouret mentioned in merge request !314 (merged) 2 months ago

    mentioned in merge request !314 (merged)

  • Guy Thouret @gthouret added Status::Review scoped label and automatically removed Status::InProgress label 2 months ago

    added scoped label and automatically removed label

  • Guy Thouret @gthouret mentioned in commit c4497723 2 months ago

    mentioned in commit c4497723

  • Guy Thouret @gthouret mentioned in commit 0628b4d2 1 month ago

    mentioned in commit 0628b4d2

  • Mark Harding @markeharding closed 1 month ago

    closed

  • Mark Harding @markeharding mentioned in commit fcb4f65b 1 month ago

    mentioned in commit fcb4f65b

  • Mark Harding
    Mark Harding @markeharding · 1 month ago
    Owner

    This is still happening

    Sentry issue: ENGINE-9G

  • Mark Harding @markeharding reopened 1 month ago

    reopened

  • Mark Harding @markeharding changed milestone to %Misc. Q3 2019 1 month ago

    changed milestone to %Misc. Q3 2019

  • Mark Harding @markeharding added Sprint::09/11 - Nuanced Numbat scoped label 1 month ago

    added scoped label

  • Mark Harding @markeharding removed Status::Review label 1 month ago

    removed label

  • Mark Harding
    Mark Harding @markeharding · 1 month ago
    Owner

    Looks like the issue was the /etc/resolv.conf in docker being set to public DNS too. Removed and looks to be stable again.

  • Mark Harding @markeharding closed 1 month ago

    closed

  • Mark Harding @markeharding reopened 1 day ago

    reopened

  • Mark Harding @markeharding added Sprint::11/06 - Rolling Rabbit scoped label and automatically removed Sprint::09/11 - Nuanced Numbat label 1 day ago

    added scoped label and automatically removed label

  • Mark Harding
    Mark Harding @markeharding · 1 day ago
    Owner

    This is still happening. It has spiked again in the last week. Screenshot_2019-11-05_at_13.06.55

  • Mark Harding @markeharding changed time estimate to 2h 6 hours ago

    changed time estimate to 2h

  • Guy Thouret
    Guy Thouret @gthouret · 2 hours ago
    Developer

    Appears to be an issue with resolving the hostname minds-redis.fpcczd.0001.use1.cache.amazonaws.com I could do with being able to diagnose this within production environment and checking production config for this.

Please register or sign in to reply
Assignee
Guy Thouret's avatar
Guy Thouret @gthouret
none
Epic
None
Misc. Q3 2019
Milestone
Misc. Q3 2019
Time tracking
Estimated: 2h
None
Due date
None
3
Labels
Priority::1 - High Sprint::11/06 - Rolling Rabbit Type::Bug
1
Weight
1
Confidentiality
Not confidential
Lock issue
Unlocked
2
2 participants
user avatar
Guy Thouret
user avatar
Mark Harding
Reference: minds/engine#777