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

404 Not Found error if the test is deleted from the gui #112

Closed
Sraza11 opened this issue Oct 4, 2022 · 9 comments · Fixed by #149
Closed

404 Not Found error if the test is deleted from the gui #112

Sraza11 opened this issue Oct 4, 2022 · 9 comments · Fixed by #149

Comments

@Sraza11
Copy link

Sraza11 commented Oct 4, 2022

I have a test created using Terraform. if I delete that test from the Thousand eyes GUI shouldn't Terraform try to re-create that test? I am getting the following error if I delete the test from the GUI

 Error: Failed call API endpoint. HTTP response code: 404. Error: 404 Not Found
│ 
│   with thousandeyes_agent_to_server.XXXXXXX,
│   on XXXX.tf line 928, in resource "thousandeyes_agent_to_server" "XXXXXX":
│  928: resource "thousandeyes_agent_to_server" "XXXXXXXXX" {
│ 
╵
@Sraza11
Copy link
Author

Sraza11 commented Oct 4, 2022

If a test is not found and 404 error is returned tf provider should re-create the test.

@Sraza11
Copy link
Author

Sraza11 commented Oct 15, 2022

@pedro-te any update on this?

@pedro-te
Copy link
Collaborator

@pedro-te any update on this?

Hi @Sraza11 ,

Thank you for reporting this and bringing it to our attention. I will create an internal ticket for this so we can put it in our sprints and have a look at it. We will get back to you when someone has looked into it.

In the meantime, please do not hesitate to reach out to us.

Thanks,
Pedro

@Sraza11
Copy link
Author

Sraza11 commented Oct 17, 2022

Thank you for the update @pedro-te

@Sraza11
Copy link
Author

Sraza11 commented Jan 23, 2023

Any update on this?

@joaomper-TE
Copy link
Contributor

Hey @Sraza11 ! Apologies for the delayed response.

I picked up this issue now, will keep you updated

@Sraza11
Copy link
Author

Sraza11 commented Nov 29, 2023 via email

@joaomper-TE
Copy link
Contributor

@Sraza11 tested in v2.0.4 and it's fixed. Make sure to test when you can, and thanks for reporting!

@joaomper-TE
Copy link
Contributor

Also, if I may ask, from the other issues that you opened, is there a specific one that is blocking you should have priority? We are beginning to triage them and would like to start by solving the ones that are blocking users

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 a pull request may close this issue.

3 participants