-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
[DOCS] Need clarification in autoaccept_grains documentation about only working with strings #63708
Comments
FYI this is related to #61436 (comment) |
@gee456 I am interested to resolve this issue. can you give more details about file structure and documentation? |
@barbaricyawps I want to give it a try, please assign this to me. |
I would assign you @Tanmoy037 , but it looks like @BabaYaga1221 would like to work on the issue as well. Who should take it? Let me know and I'll assign one of you. |
@barbaricyawps We both will appreciate working on this issue. Please, provide related information and file structure, after that whosoever gives PR first, can close others. |
I think it's a good practice to get used to the file structure yourself and see if you can map what you see on the web with where it is in the codebase. Let me give a hint:
If that hint isn't helpful enough, let me know and I'll give more. |
Thank you @barbaricyawps we will try to figure it out. |
@barbaricyawps This issue will not fix with doc change, it's a bug I'm working on it please assign it to me. |
@Tanmoy037 , I've assigned you. Thanks! |
Description
When using autoaccept_grains the documentation does not state that the match will only for for grains that are strings.
It will not work for integers, lists of strings or Booleans as well.
Suggested Fix
Note in documentation that the grain must be a string and give examples.
Here are some grains that wont work that are not custom grains.
efi-secure-boot - it's a Boolean
ipv4 - it's a list
server_id - It's an integer
Type of documentation
Current docs are here (https://docs.saltproject.io/en/latest/topics/tutorials/autoaccept_grains.html)
Location or format of documentation
https://docs.saltproject.io/en/latest/topics/tutorials/autoaccept_grains.html
Additional context
The text was updated successfully, but these errors were encountered: