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 REQUEST] Grain to assess whether minion is a fiber channel host #65750

Open
tacerus opened this issue Dec 27, 2023 · 0 comments · May be fixed by #65751
Open

[FEATURE REQUEST] Grain to assess whether minion is a fiber channel host #65750

tacerus opened this issue Dec 27, 2023 · 0 comments · May be fixed by #65751
Labels
Feature new functionality including changes to functionality and code refactors, etc. needs-triage

Comments

@tacerus
Copy link

tacerus commented Dec 27, 2023

Is your feature request related to a problem? Please describe.

For pillar logic it can be useful to know whether a minion is capable of hosting fiber channel storage.

Describe the solution you'd like

A grains function returning a boolean value if /sys/class/fc_host exists.

Describe alternatives you've considered

I found #38184 which would return port names, but it seems it was never implemented. If it was, one could alternatively to the boolean idea assess whether the system is a fc_host based on whether ports exist.

Additional context

I will submit a patch for this.

Please Note
If this feature request would be considered a substantial change or addition, this should go through a SEP process here https://github.com/saltstack/salt-enhancement-proposals, instead of a feature request.

@tacerus tacerus added Feature new functionality including changes to functionality and code refactors, etc. needs-triage labels Dec 27, 2023
@tacerus tacerus linked a pull request Dec 27, 2023 that will close this issue
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature new functionality including changes to functionality and code refactors, etc. needs-triage
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant