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

Defuse unnecessary errors for unavailable services #1083

Merged
merged 1 commit into from
Aug 23, 2023

Conversation

der-eismann
Copy link
Member

Fixes #341

These errors pop up when you are not actually using the service. Since they are just confusing the users and are no actual errors, make them Info level and pretty-print it.

Before:

ERRO[0026] Listing MachineLearningDataSource failed:
    AccessDeniedException: Thank you for your interest in Amazon Machine Learning. AmazonML is no longer available to new customers.  Please consider using Amazon SageMaker.
    	status code: 403, request id: abcdefg-1234
ERRO[0024] Listing MGNSourceServer failed:
    UninitializedAccountException: Account not initialized
    {
      RespMetadata: {
        StatusCode: 400,
        RequestID: "abcdefg-1234"
      },
      Message_: "Account not initialized"
    } 
ERRO[0024] Listing FMSPolicy failed:
    AccessDeniedException: No default admin could be found for account 123456789012 in Region us-east-1
    	status code: 400, request id: abcdefg-1234

After:

INFO[0021] MGNSourceServer: Account not initialized for Application Migration Service. Ignore if you haven't set it up. 
INFO[0021] MGNJob: Account not initialized for Application Migration Service. Ignore if you haven't set it up.
INFO[0021] FMSPolicy: No default admin could be found for account 238905726876 in Region us-east-1. Ignore if you haven't set it up. 
INFO[0024] FMSNotificationChannel: No default admin could be found for account 238905726876 in Region us-east-1. Ignore if you haven't set it up. 
INFO[0024] MachineLearningBranchPrediction: AmazonML is no longer available to new customers. Ignore if you haven't set it up.

@der-eismann der-eismann requested a review from a team as a code owner August 23, 2023 15:32
@der-eismann der-eismann merged commit 158ca36 into main Aug 23, 2023
2 checks passed
@der-eismann der-eismann deleted the defuse-unnecessary-errors branch August 23, 2023 15:48
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.

AmazonML to SageMaker - ERRO[0009] Listing MachineLearning* failed.
2 participants