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.

Sign up for GitHub

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

Jump to bottom

Fix missing state error sagemaker #1

Merged
merged 3 commits into from
Sep 8, 2018
Merged

Fix missing state error sagemaker #1

merged 3 commits into from
Sep 8, 2018

Conversation

ronaldosaheki
Copy link

Update the sagemaker_endpoint_configuration ResourceNotFound error to the actual message when running with terraform v0.11.5 and added the same behaviour to sagemaker_endpoint

@ronaldosaheki
Copy link
Author

ronaldosaheki commented Aug 30, 2018 •
edited
Loading

Error messages, both are ValidationException:
sagemaker_endpoint_configuration

Error: Error refreshing state: 1 error(s) occurred:

* aws_sagemaker_endpoint_configuration.sagemaker: 1 error(s) occurred:

* aws_sagemaker_endpoint_configuration.sagemaker: aws_sagemaker_endpoint_configuration.sagemaker: Error reading Sagemaker endpoint configuration priceprediction-electric: ValidationException: Could not find endpoint configuration "arn:aws:sagemaker:eu-west-
1:xxx:endpoint-config/priceprediction-electric".
        status code: 400, request id: 4b3290d6-c45c-4192-aa0b-ab7f65e5d919

sagemaker_endpoint

Error: Error refreshing state: 1 error(s) occurred:
* aws_sagemaker_endpoint.endpoint: 1 error(s) occurred:

* aws_sagemaker_endpoint.endpoint: aws_sagemaker_endpoint.endpoint: ValidationException: Could not find endpoint "arn:aws:sagemaker:eu-west-1:xxxxx:endpoint/priceprediction".
        status code: 400, request id: ce3e2057-3151-4126-86f3-a877bc4cee61

@jckuester
Copy link
Owner

Looks good 👍

@jckuester jckuester merged commit 3049033 into jckuester:feature/resource-aws-sagemaker-endpoint Sep 8, 2018
jckuester pushed a commit that referenced this pull request Jan 21, 2019
@ronaldosaheki @jckuester
Fix missing state error sagemaker (#1)
068eba1 
* Update sagemaker_endpoint_configuration not found message when reading resources

* Update sagemaker_endpoint error when resource is NotFound

* Fix ResourceNotFound to ValidationException
jckuester pushed a commit that referenced this pull request Mar 25, 2019
Merge pull request #1 from terraform-providers/master
f1be555 
merging head fork
jckuester pushed a commit that referenced this pull request Mar 25, 2019
@ronaldosaheki @jckuester
Fix missing state error sagemaker (#1)
7c38b19 
* Update sagemaker_endpoint_configuration not found message when reading resources

* Update sagemaker_endpoint error when resource is NotFound

* Fix ResourceNotFound to ValidationException
jckuester pushed a commit that referenced this pull request Jul 5, 2020
Merge pull request #1 from meekmichael/meek-rt-tags
107f95b 
Meek rt tags
jckuester pushed a commit that referenced this pull request Jul 5, 2020
@mousedownmike
Merge pull request #1 from terraform-providers/master
6b03ccf 
pulling latest
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ronaldosaheki @jckuester