Comment 23 for bug 1837877

Revision history for this message
Matt Riedemann (mriedem) wrote : Re: Error message reveals ceph information

@mnaser: Dan and I talked through this before he posted his comment and as he noted admins will still see the details in the fault which is the traceback and that's only for non-nova exceptions, but you'd still have the info you need as admin. So I don't think what you're proposing in comment 21 is worthwhile, and it would be complicated because like the examples Dan gives, where the fault is record (compute) and where it's exposed (api) are totally different and knowing the context / policy stuff at the compute isn't really possible with the current fields on the fault object, i.e. if we had message like today, details like today, and then some new field like "type" or something and in the API only showed type for the message to non-admins or whatever. That's just a bigger change.

I plan on working a patch with functional test for Dan's suggestion in comment 19. When I wrote my ideas before that I failed to remember NoValidHost which is important to not break for UX.