search

telefonicaid / fiware-cygnus

A connector in charge of persisting context data sources into other third-party databases and storage systems, creating a historical view of the context
https://fiware-cygnus.rtfd.io/
GNU Affero General Public License v3.0
64 stars 105 forks source link

Cygnus does not shows the current version #104

Closed iariasleon closed 10 years ago

iariasleon commented 10 years ago

Severity: Minor (4)

Effort: 1 man day

frbattid commented 10 years ago

I'm guessing this is something related to the installation of Cygnus as a service. I mean, there is no way to query Cygnus about the version, mainly because there is no Cygnus binary being run, but a Flume environment launching a Cygnus agent.

fgalan commented 10 years ago

However, Cygnus could print its version in the log as INFO message. I think that solve the issue in a easy way.

frbattid commented 10 years ago

Nice, I'll do that.

frbattid commented 10 years ago

Fixed in PR https://github.com/telefonicaid/fiware-connectors/pull/116

iariasleon commented 10 years ago

I propose adding a request to cygnus that returns the current version and some other useful data. example:

   http://127.0.0.1:5050/version

and the answer might be something like this:

<orion>
  <version>0.14.0_20140731150638</version>
  <uptime>25 d, 17 h, 56 m, 48 s</uptime>
  <git_hash>0bcf8e9703819de4eb6178e141a2be58950fc23f</git_hash>
  <compile_time>Thu Jul 31 15:08:54 CEST 2014</compile_time>
  <compiled_by>develenv</compiled_by>
  <compiled_in>ci-fiware-01</compiled_in>
</orion>
frbattid commented 10 years ago

As this Flume issue describes, there is no way a Flume HttpSource can send responses since the HTTPServletResponse parameter is not passed to the handler. There is a BidirectionalHttpSource proposed, but not yet released.

Workaround: to implement a REST management interface (listening in a different port than the service ones) for this and other management issues (e.g. remote configuration, statistics retrieves, etc).

frbattid commented 10 years ago

I'm going to close this issue since a basic solution was already given (the version is logged) and propose a new one regarding the "Management Interface".