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

salt-call schedule.list doesn't work when minion not running #54846

Closed
OrangeDog opened this issue Oct 2, 2019 · 14 comments
Closed

salt-call schedule.list doesn't work when minion not running #54846

OrangeDog opened this issue Oct 2, 2019 · 14 comments
Assignees
Labels
expected-behavior intended functionality Feature new functionality including changes to functionality and code refactors, etc.
Milestone

Comments

@OrangeDog
Copy link
Contributor

Description of Issue

This may be expected behaviour?

# systemctl stop salt-minion
# salt-call schedule.list
local:
    ----------
    schedule:
        ----------
# systemctl start salt-minion
# salt-call schedule.list
local:
    schedule:
      enabled: true
      highstate:
        enabled: true
        function: state.apply
        jid_include: true
        kwargs:
          saltenv: base
        maxrunning: 1
        minutes: 15
        name: highstate
        returner: zabbix
        run_on_start: true
        splay: 60

https://gist.github.com/OrangeDog/fd08470a682752347efd52fe7a5b830d

@garethgreenaway
Copy link
Contributor

@OrangeDog Thanks for the report. This is actually expected behavior, in order to get the schedule items and ensure they are current, we use the event bus system. If the minion isn't running then the event bus is also not running. For things like schedule.list though we could maybe fall back and pull the results out of the saved configuration.

@garethgreenaway garethgreenaway added this to the Blocked milestone Oct 2, 2019
@garethgreenaway garethgreenaway added expected-behavior intended functionality Pending-Discussion The issue or pull request needs more discussion before it can be closed or merged labels Oct 2, 2019
@stale
Copy link

stale bot commented Jan 7, 2020

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

If this issue is closed prematurely, please leave a comment and we will gladly reopen the issue.

@stale stale bot added the stale label Jan 7, 2020
@waynew
Copy link
Contributor

waynew commented Jan 7, 2020

@garethgreenaway so would that make this a feature request, then?

@stale
Copy link

stale bot commented Jan 7, 2020

Thank you for updating this issue. It is no longer marked as stale.

@stale stale bot removed the stale label Jan 7, 2020
@stale
Copy link

stale bot commented Feb 6, 2020

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

If this issue is closed prematurely, please leave a comment and we will gladly reopen the issue.

@stale stale bot added the stale label Feb 6, 2020
@OrangeDog
Copy link
Contributor Author

OrangeDog commented Feb 6, 2020 via email

@stale
Copy link

stale bot commented Feb 6, 2020

Thank you for updating this issue. It is no longer marked as stale.

@stale stale bot removed the stale label Feb 6, 2020
@stale
Copy link

stale bot commented Mar 8, 2020

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

If this issue is closed prematurely, please leave a comment and we will gladly reopen the issue.

@stale stale bot added the stale label Mar 8, 2020
@OrangeDog
Copy link
Contributor Author

unstale

@stale
Copy link

stale bot commented Mar 8, 2020

Thank you for updating this issue. It is no longer marked as stale.

@stale stale bot removed the stale label Mar 8, 2020
@stale
Copy link

stale bot commented Apr 7, 2020

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

If this issue is closed prematurely, please leave a comment and we will gladly reopen the issue.

@stale stale bot added the stale label Apr 7, 2020
@OrangeDog
Copy link
Contributor Author

unstale

@stale
Copy link

stale bot commented Apr 7, 2020

Thank you for updating this issue. It is no longer marked as stale.

@stale stale bot removed the stale label Apr 7, 2020
@garethgreenaway garethgreenaway added Feature new functionality including changes to functionality and code refactors, etc. and removed Pending-Discussion The issue or pull request needs more discussion before it can be closed or merged labels Apr 24, 2020
@garethgreenaway garethgreenaway self-assigned this Apr 24, 2020
@garethgreenaway
Copy link
Contributor

The PR that allowed offline use of the schedule module has been merged, so closing this one out.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
expected-behavior intended functionality Feature new functionality including changes to functionality and code refactors, etc.
Projects
None yet
Development

No branches or pull requests

3 participants