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

Feature: config dont_cleanup_on_failed_enable #3200

Open
orndorffgrant opened this issue Jul 15, 2024 · 2 comments
Open

Feature: config dont_cleanup_on_failed_enable #3200

orndorffgrant opened this issue Jul 15, 2024 · 2 comments
Labels
feature idea for a new feature P - Low S - Triaged Must have priority label

Comments

@orndorffgrant
Copy link
Collaborator

Please describe the scenario where the new feature would be useful

Somewhat frequently, there are issues with dependency versions or other package installability issues that cause pro enable fips to fail. In some cases, a valid workaround would be to enable access to the fips repo and then install the packages manually.

Describe the solution you'd like

pro enable fips --access-only should work for fips, fips-updates, and fips-preview.

Current behavior

root@teast:~# pro enable fips --access-only
One moment, checking your subscription first
FIPS does not support being enabled with --access-only
Could not enable FIPS.
@orndorffgrant orndorffgrant added the feature idea for a new feature label Jul 15, 2024
@orndorffgrant
Copy link
Collaborator Author

This would mostly be useful for situations where pro enable fips fails and something needs to be debugged with fips sources enabled. An alternative could be a config item of dont_cleanup_on_failed_enable, so that the system could be examined after the failure with the fips sources still in place.

@orndorffgrant
Copy link
Collaborator Author

Discussed with team. +1 to config dont_cleanup_on_failed_enable in some form.

@orndorffgrant orndorffgrant changed the title Feature: allow --access-only for fips Feature: config dont_cleanup_on_failed_enable Aug 21, 2024
@lucasmoura lucasmoura added S - Triaged Must have priority label P - Low and removed triaged labels Nov 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature idea for a new feature P - Low S - Triaged Must have priority label
Projects
None yet
Development

No branches or pull requests

2 participants