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

Use supports in more instances #21989

Open
5 of 18 tasks
kbrock opened this issue Jul 12, 2022 · 6 comments
Open
5 of 18 tasks

Use supports in more instances #21989

kbrock opened this issue Jul 12, 2022 · 6 comments
Assignees

Comments

@kbrock
Copy link
Member

kbrock commented Jul 12, 2022

This is a continuation from #21179

We have a number if implementations that are basically a reimplementation of SupportsMixin.
Convert these implementations over to using supports

review:

  • InventoryRefresh#supports_sti?
  • MiqEnvironment#supports_systemd?
  • MiqEnvironment#supports_nohup_and_backgrounding
  • Flavor#supports_instance_store (virtual column)
  • Flavor#supports_ebs
  • NetworkManager#supports_security_groups ?
  • Button#supports_button_action
  • Kubevirt::ProvisionWorkflow#supports_native_clone
  • Awx::Dialog supports_limit
@miq-bot
Copy link
Member

miq-bot commented Feb 27, 2023

This issue has been automatically marked as stale because it has not been updated for at least 3 months.

If you can still reproduce this issue on the current release or on master, please reply with all of the information you have about it in order to keep the issue open.

Thank you for all your contributions! More information about the ManageIQ triage process can be found in the triage process documentation.

@kbrock kbrock removed the stale label Mar 7, 2023
@kbrock
Copy link
Member Author

kbrock commented Mar 7, 2023

brought console_supported up to date.
adam and I keep coming back to the same solution:
convert blocks with complicated logic to a simple supports in the leaf nodes

@miq-bot
Copy link
Member

miq-bot commented Jun 12, 2023

This issue has been automatically marked as stale because it has not been updated for at least 3 months.

If you can still reproduce this issue on the current release or on master, please reply with all of the information you have about it in order to keep the issue open.

Thank you for all your contributions! More information about the ManageIQ triage process can be found in the triage process documentation.

@miq-bot miq-bot added the stale label Jun 12, 2023
@miq-bot miq-bot closed this as completed Sep 18, 2023
@miq-bot
Copy link
Member

miq-bot commented Sep 18, 2023

This issue has been automatically closed because it has not been updated for at least 3 months.

Feel free to reopen this issue if this issue is still valid.

Thank you for all your contributions! More information about the ManageIQ triage process can be found in the triage process documentation.

@kbrock kbrock reopened this Sep 18, 2023
@kbrock kbrock removed the stale label Sep 18, 2023
@miq-bot miq-bot added the stale label Dec 25, 2023
@miq-bot
Copy link
Member

miq-bot commented Dec 25, 2023

This issue has been automatically marked as stale because it has not been updated for at least 3 months.

If you can still reproduce this issue on the current release or on master, please reply with all of the information you have about it in order to keep the issue open.

@miq-bot
Copy link
Member

miq-bot commented Jul 22, 2024

This issue has been automatically marked as stale because it has not been updated for at least 3 months.

If you can still reproduce this issue on the current release or on master, please reply with all of the information you have about it in order to keep the issue open.

@jrafanie jrafanie added pinned and removed stale labels Jul 24, 2024
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

4 participants