Rename service.Exit

[bufdev]

It's kind of an awkward name a little? Also is there duplication in the Reason and Error fields? If not, it would be nice to have some documentation. Note that in the Observer interface, you have OnShutdown(reason Exit) which is already confusing.

[bufdev]

Any thoughts on this? @glibsm @sectioneight @madhuravi @alsamylkin @anuptalwalkar

[glibsm]

Agreed on string/error redundancy, we should drop the string.

Are we talking about renaming Exit struct to something else, or dropping it all together and just using one error field?

[HelloGrayson]

Closing - this API doesn't exist in the modular branch, which will soon makes it way to the dev branch and beyond.