-
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
Sync / Refresh is causing grains to be refreshed twice #55174
Comments
Thanks @jtraub91 for reporting this performance problem! |
Furthermore, for proxy minions, upon boot, the grains are refreshed 8 times ! The following proxy pillar was setup:
With a test grain:
Start the proxy:
Proof:
Observe the setup timestamp ^
Full debug logs:
|
@cmcmarrow I see there you have an open PR for the erroneous dual pillar refresh, but can you confirm that my above comment regarding proxy minions has been noted? Seems to be a separate issue. Shall I raise another github issue for it? |
#46713 |
A fix is waiting to be merged. |
@jtraub91 |
Raise another GitHub issue would be great @jtraub91. So for the slow reply. |
@jtraub91 have you opened another ticket? Wanting to be sure that isn't lost. Thank you! |
@sagetherage Just did so here #55658 |
Description of Issue
When issuing a sync / refresh via
saltutil.sync_grains
orsaltutil.refresh_grains
, the grains functions are being evaluated at least twice.Setup
Normal salt installation. Minion config
log_level
set to 'trace'. Create custom test grain to add clarity to the problem (not needed)Steps to Reproduce Issue
Logs from
/var/log/salt/minion
:Versions Report
The text was updated successfully, but these errors were encountered: