Upgrade to Pro — share decks privately, control downloads, hide ads and more …

Systematic error management - we ported rudder ...

fanf42
October 30, 2019

Systematic error management - we ported rudder to zio

This talk was given at ScalaIO 2019.
It explains how you can manage errors in a systematic way in your applications, and show how we did it in Rudder with the functional library ZIO.

It presents 4 big principles which direct my devloper job:
- 1/ Our work as developers is to discover and assess failure modes.
- 2/ ERRORS are a SOCIAL construction to give AGENCY to the receiver of the error.
- 3/ An application has always at least 3 kinds of users: users; devs; and ops. Don’t forget any.
- 4/ It’s YOUR work to choose the SEMANTIC between nominal case and error and KEEP your PROMISES.

The talk gives 5 guidelines to help you implement these principles. It also introduces a very light glimpse on system thinking that you can explore in more details in the related article "Understand things as interacting systems": https://medium.com/@fanf42/understand-things-as-interacting-systems-b273bdba5dec

If you have any questions, please ask: there is several way to contact me at the end of the deck (slide 87)!

fanf42

October 30, 2019
Tweet

More Decks by fanf42

Other Decks in Programming

Transcript

  1. Hi! devops automation/compliance app manage ten of thousands computers 2

    François ARMAND CTO Founder Free Software Company “Stay Up”
  2. Hi! devops automation/compliance app manage ten of thousands computers 3

    François ARMAND CTO Founder Free Software Company “Stay Up” Developer
  3. Developer ? • Model the world into code ◦ Try

    to make it useful • Nominal case necessary (of course) 5
  4. Developer ? • Model the world into code ◦ Try

    to make it useful • Nominal case necessary (of course) • But not sufficient (models are false) ◦ Bugs ◦ Misunderstanding of needs ◦ open world ◦ Damn users using your app ▪ 6
  5. This talk • systematic management of errors • caveat emptor:

    ◦ I’m a scala dev, mainly ▪ ▪ ◦ application, not library ▪ 7
  6. This talk • It's an important talk for me •

    Much harder to do than expected ◦ based on lots of deeply rooted, fuzzy, experimental knowledge • Please, please, I beg you: if anything unclear, come chat with me / ask questions (whatever the medium) 8
  7. Our work as developers is to discover and assess failure

    modes 10 Not so popular opinion 1/4
  8. ERRORS are a SOCIAL construction to give AGENCY to the

    receiver of the error 11 Not so popular opinion 2/4
  9. An application has always at least 3 kinds of users:

    users ; devs ; and ops. Don’t forget any. 12 Not so popular opinion 3/4
  10. It’s YOUR work to choose the SEMANTIC between nominal case

    and error and KEEP your PROMISES Not so popular opinion 4/4 13
  11. 15 Assess failure modes. Give agency to your users and

    don’t forget any of them. You are responsible to keep promises made.
  12. 16 Pure, total functions Explicit error channel Program to strict

    interfaces and protocols Composition and tooling 1. 2. 4. 5. Failures vs Errors 3. Assess failure modes. Give agency to your users and don’t forget any of them. You are responsible to keep promises made.
  13. 17 1. 2. 4. 5. These points are also important

    and cans be translated at architecture / UX / team / ecosystem levels. But let’s keep it simple with code. 3. Assess failure modes. Give agency to your users and don’t forget any of them. You are responsible to keep promises made.
  14. Don’t lie! 21 Divide By Zero ? • non total

    functions are a lie ◦ your promises are unsound ◦ your users can’t react appropriately divide(a: Int, b: Int): Int
  15. Don’t lie! 24 No such user ? (non total) DB

    connexion error? getUserFromDB(id: UserId): User
  16. Don’t lie! 25 No such user ? (non total) DB

    connexion error? • non pure functions are a lie ◦ your promises are unsound ◦ your users can’t react appropriately getUserFromDB(id: UserId): User
  17. Sound promises 26 • use total functions ◦ or make

    them total with union return type • use pure functions ◦ or make them pure with IO monad • Don’t lie to your users, • allow them to react efficiently:
  18. • Don’t assume what’s obvious • It’s an open world

    out there • Don’t force users to revert-engineer possible cases 29 It’s a signal make it unambiguous give agency
  19. Which intent is less ambiguous? 30 blobzurg(a: Int, b: Int):

    Option[Int] blobzurg(a: Int, b: Int): PureResult[DivideByZero, Int] It’s a signal make it unambiguous give agency
  20. 31 It’s a signal make it unambiguous give agency automate

    it • Use the type system to automate classification of errors?
  21. 32 A type system is a tractable syntactic method for

    proving the absence of certain program behaviors by classifying phrases according to the kinds of values they compute. Benjamin Pierce It’s a signal make it unambiguous give agency automate it • Use the type system to automate classification of errors?
  22. 33 By definition, a type system automatically categorize results ⟹

    need for a dedicated error chanel + a common error trait A type system is a tractable syntactic method for proving the absence of certain program behaviors by classifying phrases according to the kinds of values they compute. Benjamin Pierce It’s a signal make it unambiguous give agency automate it def divide(a: Int, b: Int): PureResult[Int]
  23. 34 A type system is a tractable syntactic method for

    proving the absence of certain program behaviors by classifying phrases according to the kinds of values they compute. Benjamin Pierce trait MyAppError // common properties of errors type PureResult[A] = Either[MyAppError, A] It’s a signal make it unambiguous give agency automate it def divide(a: Int, b: Int): PureResult[Int] By definition, a type system automatically categorize results ⟹ need for a dedicated error chanel + a common error trait
  24. 35 It’s a signal make it unambiguous give agency automate

    it def getUser(id: UserId): IOResult[User] By definition, a type system automatically categorize results ⟹ need for a dedicated error chanel + a common error trait Same for effectful functions!
  25. Same for effectful functions! 36 trait MyAppError // common properties

    of errors type IOResult[A] = IO[MyAppError, A] It’s a signal make it unambiguous give agency automate it def getUser(id: UserId): IOResult[User] By definition, a type system automatically categorize results ⟹ need for a dedicated error chanel + a common error trait
  26. 37 It’s a signal make it unambiguous give agency automate

    it • Use a dedicated error channel ◦ ~ Either[E, A] for pure code, ◦ else ~ IO[E, A] monad • use a parent trait for common error properties… • and for automatic categorization of errors by compiler
  27. Model everything? 41 java.lang.SecurityException? (jvm perm to access FS) ⟹

    where do you put the limit? writeFile(path: String, value: String): IOResult[Unit]
  28. Systems? Need for a systematic approach to error management 43

    ◦ BOUNDED group of things ◦ with a NAME Interacting ◦ with others systems A school of systems
  29. Systems have horizon. Horrors lie beyond. 45 ◦ nothing exists

    beyond horizon ◦ Like with Lovecraft: if something from beyond interact with a system, the system becomes inconsistent
  30. Errors vs Failures 46 Errors • expected non nominal case

    • signal for users • social construction: you choose alternative or error • reflected in types Failures • unexpected case: by definition, application is in an unknown state • only choice is stop as cleanly as possible • not reflected in types
  31. Horizon limit is your choice - by definition 48 java.lang.SecurityException?

    execScript(js: String): IOResult[String] In Rudder, we have a JS engine (JS from users):
  32. Horizon limit is your choice - by definition 49 java.lang.SecurityException?

    execScript(js: String): IOResult[String] In Rudder, we have a JS engine (JS from users): ⟹ SecurityException is an expected error case here
  33. Horizon limit is your choice - by definition 50 java.lang.SecurityException?

    execScript(js: String): IOResult[String] In Rudder, we have a JS engine (JS from users): ⟹ SecurityException is an expected error case here … but nowhere else in Rudder. By our choice.
  34. Need for a systematic approach to error management 52 ◦

    BOUNDED group of things ◦ with a NAME Interacting ◦ with others systems A school of systems A bit more about systems
  35. A bit more about systems Need for a systematic approach

    to error management 53 ◦ BOUNDED group of things ◦ with a NAME Interacting ◦ via INTERFACES ◦ by a PROTOCOL with other systems ◦ And PROMISING to have a behavior A school of systems
  36. Example? 56 Typical web application. How to keep contradictory promises?

    Promises to third parties about REST behaviour Promises to business and developers about code manageability
  37. Make promises, Keep them 59 • systems allow to bound

    responsibilities Business Core sub-system: • own ADT / logic (mostly pure) • lifecycle bounded to developers understanding of needs (rapid changes)
  38. Make promises, Keep them 60 • systems allow to bound

    responsibilities Business Core sub-system: • own ADT / logic (mostly pure) • lifecycle bounded to developers understanding of needs (rapid changes) Pattern: “A pure heart (core) surrounded by side effects”* * works better in French: “un coeur pur encerclé par les effets de bords”
  39. Make promises, Keep them 61 • systems allow to bound

    responsibilities Users of the API want stability and to know what errors can happen Business Core sub-system: • own ADT / logic (mostly pure) • lifecycle bounded to developers understanding of needs (rapid changes)
  40. Make promises, Keep them 62 • systems allow to bound

    responsibilities Business Core sub-system: • own ADT / logic (mostly pure) • lifecycle bounded to developers understanding of needs (rapid changes) REST sub-system : • own ADT / logic (mostly effects) • lifecycle bounded to REST contract: strict versioning, changes are breaking changes Users of the API want stability and to know what errors can happen
  41. Make promises, Keep them 63 • systems allow to bound

    responsibilities Business Core sub-system: • own ADT / logic (mostly pure) • lifecycle bounded to developers understanding of needs (rapid changes) REST sub-system : • own ADT / logic (mostly effects) • lifecycle bounded to REST contract: strict versioning, changes are breaking changes Stable API : interface, strict protocol & promises (nominal cases + errors) Users of the API have agency (able to react efficiently)
  42. Make promises, Keep them 64 • systems allow to bound

    responsibilities Business Core sub-system: • own ADT / logic (mostly pure) • lifecycle bounded to developers understanding of needs (rapid changes) REST sub-system : • own ADT / logic (mostly effects) • lifecycle bounded to REST contract: strict versioning, changes are breaking changes Stable API : interface, strict protocol & promises (nominal cases + errors) Users of the API have agency (able to react efficiently) Translation between sub-systems: API: interface, protocol & promises!
  43. Make promises, Keep them 65 • systems allow to bound

    responsibilities • translate errors between sub-systems ◦ make errors relevant to their users • It’s a model, it’s false ◦ there is NO definitive answer. ◦ discuss, share, iterate • the bigger the promises, the stricter the API
  44. What’s missing for good error management in code ? •

    signal must be unambiguous ◦ exception are a pile of ambiguity ◦ Error ? ◦ Fatal error ? ◦ Checked ? Unchecked ? 69
  45. What’s missing for good error management in code ? •

    signal must be unambiguous ◦ exception are a pile of ambiguity • exceptions are A PAIN to use ◦ no tooling, no inference, nothing ▪ ▪ ◦ no composition ▪ 70
  46. Make it a joy! 71 • managing error should be

    enjoyable ! ◦ automatic (in for loop + inference) ◦ or as expressive as nominal case! • safely, easely managing error should be the default ! ◦ composition (referential transparency…) ◦ higher level resource management: bracket, etc • make the code extremely readable ◦ add all the combinators you need! ◦ it’s cheap with pure, total functions
  47. Why ZIO ? 74 • you still have to think

    in systems by yourself • then ZIO provides : ◦ effect management ◦ with an explicit error channel ◦ IO[+E, +A] val pureCode = IO.effect(effectfulCode)
  48. Why ZIO ? 75 • you still have to think

    in systems by yourself • then ZIO provides : ◦ debuggable failures Complex error composition Async code trace
  49. Why ZIO ? 76 • you still have to think

    in systems by yourself • then ZIO provides : ◦ tons of convenience to manipulate errors ▪ ▪ ▪ ◦ composable effects ▪ • safe, composable resource management
  50. Why ZIO ? 77 • you still have to think

    in system by yourself • then ZIO provides : ◦ effect management ◦ with an explicit error channel ◦ debuggable failures ◦ tons of convenience to manipulate errors ◦ composable
  51. Why ZIO ? 78 • you still have to think

    in system by yourself • then ZIO provides : ◦ effect management ◦ with an explicit error channel ◦ debuggable failures ◦ tons of convenience to manipulate errors ◦ composable • Everything work in parallel, asynchronous code too! • Inference just work!
  52. Why ZIO ? 79 • you still have to think

    in system by yourself • then ZIO provides : ◦ effect management ◦ with an explicit error channel ◦ debuggable failures ◦ tons of convenience to manipulate errors ◦ composable • Everything work in parallel, concurrent code too! • Inference just work! Lots of details: “Error Management: Future vs ZIO” https://www.slideshare.net/jdegoes/error-management-future-vs-zio
  53. Generic, useful errors 83 • java exceptions are translated into

    SystemError • Chained allows to add context for humans • Accumulated groups several errors into one
  54. Specialized error for subsystems 84 • real code from rudder

    ⇒ specialized errors for the LDAP subsystem ⇒ adapt semantic from java lib (exceptions) to pure value that can be composed and behave as others errors in Rudder (printable information)
  55. Full example - real code from Rudder 85 • inference

    just works • each sub-system add relevant information • simple combinators (in white) used as syntax sugar (None, msg) => Unexpected(msg) PureResult[A] => IOResult[A] (err: RudderError[A], msg) => Chained(msg, err) error contextualisation between systems
  56. 86 Pure, total functions don’t lie about your promises Explicit

    error channel make it unambiguous in your types Program to strict interfaces and protocols use systems to materialize promises Composition and tooling make it extremely convenient to use Assess failure modes. Give agency to your users and don’t forget any of them. You are responsible to keep promises made. 1. 2. 4. 5. Failures vs Errors models are false by construction 3.
  57. Question? Contact me / Chat with me! https://twitter.com/fanf42 https://github.com/fanf https://keybase.io/fanf42

    irc/freenode: fanf [email protected] 87 Ressources ◦ Error management: future vs ZIO A much more detailed presentation of ZIO error management capabilities https://www.slideshare.net/jdegoes/error-management-future-vs-zio ◦ Understand Things As Interacting Systems More insights on systems. https://medium.com/@fanf42/understand-things-as-interacting-systems-b273bdba5dec ◦ Stay Up! Journey of a Free Software Company. One decade in search for a sustainable model https://medium.com/@fanf42/stay-up-5b780511109d
  58. • What about making impossible state unrepresentable from the beginning?

    ◦ That’s a very good point and you should ALWAYS try to do so. The idea is to change method’s domain definition (ie, the parameter’s shape) to only work on inputs that can’t rise errors. Typically, in my trivial “divide” example, we should have use “non zero integer” for denominator input. ◦ Alexis King (@lexy_lambda) wrote a wonderful article on that, so just go read it, she explains it better than I can: “Parse, don’t validate” https://lexi-lambda.github.io/blog/2019/11/05/parse-don-t-validate/ ◦ We use that technique a lot in Rudder to drive understanding of what is possible. Each time we can restrict domain definition, we try to keep that information for latter use. ◦ Typical example: parsing plugin license (we have 4 “xxxLicenses” classes depending what we now about its state); Validating user policy (again several “SomethingPolicyDraft” with different hypothesis needed to build the “Something”). ◦ the general goal is the same than with error management: assess failure mode, give agency to users to react efficiently. ◦ There’s still plenty of cases where that technique is hard to use (fluzzy business cases…) or not what you are looking for (you just want to tell users that something is the nominal case, or not, and give them agency to react accordingly). Some questions asked after the talk 88
  59. Some questions asked after the talk 89 • Is SystemError

    used to catch / materialize failure ? ◦ no, SystemError is here to translate Error that need to be dealts with (like connection error to DB, FS related problem, etc) but are encoded in Java with an Exception. SystemError is not used to catch Java “OutOfMemoryError”. These exception kills Rudder. We use the JVM Thread.setDefaultUncaughtExceptionHandler to try to give more information to dev/ops and clean things before killing the app.
  60. Some questions asked after the talk 90 • You have

    only one parent type for errors. Don’t you lose a lot of details with all special errors in subsystems losing the specificities when they are seen as RudderError? ◦ this is a very pertinent question, and we spend a log of time pondering between the current design and one where all sub-systems would have their own error type (with no common super type). In the end, we settled on the current design because: ▪ ▪ ▪ ▪ ◦ So all in all, the wins in convenience and joy of just having evering working without boilerplate clearly outpaced the not clear gain of having different error hierarchies. ◦ The problem would have been different if Rudder was not one monolithic app with a need of separated compilation between services. I think we would have made an “error” lib in that case.
  61. Some questions asked after the talk 91 • We use

    Future[Either[E,A]] + MTL, why should we switch to ZIO? ◦ Well, the decision to switch is yours, and I don’t know the specific context of your company to give an advice on that. Nonetheless, here is my personal opinion: ▪ ▪ ▪ ▪ ▪ pertinent stack trace in concurrent code • But at the end of the day, you decide!
  62. Some questions asked after the talk 92 • How long

    did it took to port Rudder to ZIO? ◦ It’s complicated :). 1 month of part time (me), plus lots more time for teaching, refactoring, understanding new paradigm limits, etc ▪ ▪ ▪ ▪ ▪