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

There should be some warning before deleting the default Execution Environment #15303

Open
5 of 9 tasks
AlanCoding opened this issue Jun 26, 2024 · 2 comments
Open
5 of 9 tasks

Comments

@AlanCoding
Copy link
Member

Please confirm the following

  • I agree to follow this project's code of conduct.
  • I have checked the current issues for duplicates.
  • I understand that AWX is open source software provided for free and that I might not receive a timely response.

Feature type

Enhancement to Existing Feature

Feature Summary

Give some notice before the default execution environment gets deleted.

(all steps as superuser)

Select the relevant components

  • UI
  • API
  • Docs
  • Collection
  • CLI
  • Other

Steps to reproduce

If I got to the UI page for the default EE, it looks like this:

http://localhost:8013/#/execution_environments/1/details

Then I can just click "Delete", and it deletes as if it is an ordinary Execution Environment.

Current results

Then all jobs will give this error:

Screenshot from 2024-06-26 08-43-08

...assuming the job doesn't have a custom EE set. Which would match the Demo Job Template.

Sugested feature result

Somewhere in the process of clicking "Delete" it denies the request or gives me a warning that says specifically that it is the default EE in the system.

I would argue that an error is okay, because the user should, ideally, change the setting to a different EE, then come back and delete the old one.

Additional information

No response

@marek1712
Copy link

marek1712 commented Jun 28, 2024

Interesting. Delete is greyed out for Control Plane Execution Environment.
Furthermore, I get this when I try to delete any other EE:
obraz
I'm running 24.6.0.

@AlanCoding
Copy link
Member Author

Yeah, but it's a generic message. There's no indicator making it clear that particular image is the default. I honestly don't know for sure that it is the default. You could have kept around some old versions of AWX EE.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants