Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

EY-5010 Flytte grunnlag-rivers til behandling-kafka #6976

Merged
merged 2 commits into from
Feb 6, 2025

Conversation

Watercolours
Copy link
Contributor

Har fått luket ut div smårusk fra forrige PR. Testet OK i dev.

Gjør flytting av grunnlag til behandling litt enklere.

@Watercolours Watercolours requested a review from a team as a code owner February 5, 2025 13:45
Copy link
Contributor

@henrikg henrikg left a comment

Choose a reason for hiding this comment

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

Når rivers fjernes fra grunnlag og flyttes til behandling, er det noe risiko for at vi mister meldinger? Er det behov for at behandling vet hvilke offset som er aktuell startstidspunkt for grunnlagsrivers, og at grunnlag vet hvilke offset som den skal slutte å lese?

Tror @sebassonav var borti tilsvarende problemstilling på brev.

@sebassonav
Copy link
Contributor

@henrikg ja stemmer det.
La inn kommentar i denne jiraen om valgt strategi:
https://jira.adeo.no/browse/EY-4499

Så satte jeg offsett til topicen til latest etter at produsentene var skrudd av.

@sebassonav
Copy link
Contributor

Er en del props i yaml filene til grunnlag rel til topics som kan fjernes også.

@sebassonav
Copy link
Contributor

sebassonav commented Feb 6, 2025

Er det tenkt noe på hvordan evt feil i en river skal håndteres ifbm at behandling-kafka da stopper prosessering av alle andre topics/rivers? Det er kritisk at vi agerer på feks adressebeskyttelseshendelser og de kan bli rammet av dette. Vet ikke hvor kritisk andre hendelser er.

@Watercolours
Copy link
Contributor Author

@sebassonav: Vi må helt klart se på feilhåndtering i behandling-kafka. Selv PDL-hendelser kan i dag ta ned hele appen. Må få på plass en løsning som sikrer at den ikke dør hvis den mottar data som ikke kan behandles.

@henrikg: Deploy-strategi er på plass, men bra du passer på 😄

@Watercolours Watercolours merged commit 88c581f into main Feb 6, 2025
107 checks passed
@Watercolours Watercolours deleted the EY-5010_flytte-grunnlag-rivers branch February 6, 2025 20:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants