[BUG] FlyteExceptions are not return user defined error_codes.

[RRap0so]

Describe the bug

We use FlyteUserException to pass our own error codes. These error codes are important for us for our several other services to know what they should do in different situations.

eg:

class CustomException(FlyteUserException):
    _ERROR_CODE = "USER:DoSomething"

With this PR, the behaviour as now changed to always return an hardcoded "USER" in the ContainerError and disregard if we're overriding the _ERROR_CODE with our own.

Expected behavior

Using the example above I would expect to have a ContainerError return USER:DoSomething not just USER.

Additional context to reproduce

Run a dummy task throwing a

class CustomException(FlyteUserException):
    _ERROR_CODE = "USER:DoSomething"

Screenshots

No response

Are you sure this issue hasn't been raised already?

• Yes

Have you read the Code of Conduct?

• Yes