Skip to content
This repository has been archived by the owner on Sep 15, 2021. It is now read-only.

Publishing updated Tasks in Kafka #56

Merged
merged 1 commit into from
Apr 28, 2020

Conversation

slemrmartin
Copy link
Contributor

@slemrmartin slemrmartin commented Apr 27, 2020

Issue RedHatInsights/topological_inventory-ansible_tower#83

Tasks were updated through Topological API, which published Kafka event.
Catalog update depends on this event so it has to be published from persister too


@slemrmartin
Copy link
Contributor Author

@agrare I haven't tested it yet, but do you agree with this concept?

@lindgrenj6 on catalog side, do you need whole Insights::API::Common::Request.current_forwardable or x-rh-insights-request-id is enough?
Because this value has to be saved in database, I want it as small as possible

cc @syncrou

@slemrmartin slemrmartin force-pushed the tasks-update-kafka branch 2 times, most recently from 9d21284 to b330be0 Compare April 28, 2020 12:56
@agrare agrare merged commit 50bdeaf into RedHatInsights:master Apr 28, 2020
@slemrmartin slemrmartin deleted the tasks-update-kafka branch April 28, 2020 13:24
@syncrou
Copy link
Contributor

syncrou commented Apr 28, 2020

@slemrmartin I talked with @eclarizio in regards to Insights::API::Common::Request.current_forwardable and as of now Catalog is only looking up the task_id. So this should be fine as it was merged.

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

3 participants