new grains not refreshed for matches

[knightsamar]

When a new grain is distributed to the minions by using saltutil.sync_grains, it's data is listed when doing a grains.items but the data cannot be used while matching based on the grains using -G option.

For example, writing a grain like the below

def something():
     return {'spell':'alohomora'}

And running
salt '*' saltutil.sync_grains
will distribute the grain.

And it will even show in the output of
sudo salt '*' grains.items

But matching minions, based on the newly written grain data does not work.
salt -G 'spell:alohomora' test.ping

Unless, the minion is restarted OR saltutils.sync_modules is run from the master, matches based on this new grain do not seem to work.

Shouldn't there be some command to get the minion to send/store the newly gained grain ? And shouldn't it be triggered automatically when we do sync_grains ?

[basepi]

What version are you running? We fixed a bug with regard to this issue recently, and I'm thinking you might not be running the latest.

[knightsamar]

@basepi I am using 0.15.3 on both minion and master.
What version is this issue fixed in ?

[basepi]

I'm not sure off the top of my head. Any chance you could test with the 0.16.0 RC (0.15.90) on PyPI, or from the develop branch in git?

[knightsamar]

@basepi, I just tested with 0.15.90 from PyPi on both master and minion and I still have to run saltutils.sync_modules so that the grain can be matched against.

I will test on the develop branch soon and get back.

[basepi]

I appreciate it. If it wasn't fixed in 0.15.90, then it probably is not fixed in develop, but I haven't been paying super close attention so I can't be sure. =)

[knightsamar]

@basepi, I am trying to setup master and minion using develop branch, using the instructions at https://salt.readthedocs.org/en/latest/topics/hacking.html#posting-patches-to-the-mailing-list but I still get 0.15.90 as the version when I run salt --version. Is this the right behavior and would I be testing on the right version?

[basepi]

I thought we added a 0.16 tag to make it properly parse as a 0.16 version, but I can't remember. Did you do a git pull before you installed?

In any case, it sounds like the issue is still there in develop, so we'll look into it.

[ghost]

ref: #7334

[thatch45]

@cachedout also make it so that grains get auto refreshed every n minutes, say 5 by default and make it configurabel

[cachedout]

The above PR should address the original issue. Before closing this, I still need to add functionality so that the minion will auto-update its grains every 5 minutes (configurable) and only send a pillar refresh if the grains have changed. (Per @thatch45)

[cachedout]

Pull req pending for auto-refresh.