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

Updated README #27

Merged
merged 3 commits into from
Jan 6, 2017
Merged

Updated README #27

merged 3 commits into from
Jan 6, 2017

Conversation

JuliaFernee
Copy link
Contributor

No description provided.

* Filters and transforms it to UP standard json representation
* Puts the result onto the kafka destination topic _ConceptSuggestions_

v1-suggestor service communicates with kafka via http-rest-proxy. It polls kafka-rest-proxy for messages and POSTs transfirmed messages to kafka-rest-proxy.
Copy link
Contributor

Choose a reason for hiding this comment

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

transformed not transfirmed

This service is deployed in the Delivery clusters.
## Installation

* `go get -u github.com/Financial-Times/v1-sugestor`
Copy link
Contributor

Choose a reason for hiding this comment

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

github.com/Financial-Times/v1-suggestor not github.com/Financial-Times/v1-sugestor

| **SRC_ADDR** |_http://localhost:8080_ | Url of the _http-rest-proxy_ host to connect to in order to **receive** messages from kafka. |
| **SRC_GROUP** | _v1Suggestor_ | The consumer group for receiving messages from kafka. |
| **SRC_TOPIC** | _NativeCmsMetadataPublicationEvents_ | kafka topic to consume messages from. |
| **SRC_QUEUE** | _kafka_ | Used by _Vulkan_ to route http requests based on _Host_ header. In docker cluster all hosts are at _http://localhost:8080_. This http header is supplied to distinguish one service from another. Host header _kafka_ points to _http-rest-proxy_. |
Copy link
Contributor

Choose a reason for hiding this comment

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

Vulcan not Vulkan?



## Prerequisites
1. In order to set LDFLAGS to provide correct versionning information to */build-info* endpoint we need to enable access to remote GIT repo via https (not ssh) because Docker does not have your ssh keys.
Copy link
Contributor

Choose a reason for hiding this comment

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

This our standard approach to build-info and I don't think it should be documented in every service as it will be the same for every Go service

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I agree, but I've been tripping over it every so often because it is very easy to forget and it is not sadly documented anywhere else. Perhaps we can consider a git "README" repo that has a "parent" README for all our go services. We can then reference it in the READMEs like this one. It can also contain other staff like service related specs. Why git? because it's easy to maintain the history of the docs, do reviews and all the jazz that SCM gives us.

Copy link
Contributor

Choose a reason for hiding this comment

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

I don't think it belongs here though as for it's alternative location why not bring it up the tech forum


|Endpoint | Explained |
|---|---|
| /__health | checks that v1-sugestor can communicate to kafka via http-rest-proxy|
Copy link
Contributor

Choose a reason for hiding this comment

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

v1-suggestor

|/build-info | the same as above for compatibility with Dropwizard java apps |


Note: Brigthcove video metadata is the same pipeline as metadata for Methode articles, so brands are added in the same way.
Copy link
Contributor

Choose a reason for hiding this comment

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

It is brightcove, wordpress and methode

_TODO: We should consider documenting the format of the in/out messages to kafka._
Copy link
Contributor

Choose a reason for hiding this comment

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

An example message would be nice

Copy link
Contributor

Choose a reason for hiding this comment

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

Also the message that is on Kafka is base64 encoded

@JuliaFernee JuliaFernee merged commit f044b09 into master Jan 6, 2017
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.

None yet

2 participants