Skip to content
This repository has been archived by the owner on Jun 12, 2018. It is now read-only.

Add metrics resource to Go sandbox #121

Closed
wants to merge 3 commits into from

Conversation

hodgestar
Copy link
Contributor

So that JS apps can publish and update metrics.

Its not quite clear what this should look like yet. One option is to expose the metrics stuff relatively directly but that's probably not that useful so there might need to be an intervening layer that does some aggregation.

@ghost ghost assigned hodgestar Dec 5, 2012
@hodgestar
Copy link
Contributor Author

We could do this via Holodeck. We'd need to add Twisted client support to photon (https://github.com/shaunsephton/photon/blob/master/photon/client.py) and dashboard API keys to Holodeck (shaunsephton/holodeck#19).

def _process_event_command(self, etype, user_account_key, command):
try:
event = self._event_for_command(etype, command)
except Exception, e:
Copy link
Member

Choose a reason for hiding this comment

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

Why not just a KeyError here?

@hodgestar
Copy link
Contributor Author

This pull-request is being deprecated in favour of #134

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants