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

cron.rm cannot remove @special entries #28171

Closed
srkunze opened this issue Oct 21, 2015 · 3 comments
Closed

cron.rm cannot remove @special entries #28171

srkunze opened this issue Oct 21, 2015 · 3 comments
Labels
Bug broken, incorrect, or confusing behavior Execution-Module fixed-pls-verify fix is linked, bug author to confirm fix help-wanted Community help is needed to resolve this P3 Priority 3 Platform Relates to OS, containers, platform-based utilities like FS, system based apps severity-medium 3rd level, incorrect or bad functionality, confusing and lacks a work around
Milestone

Comments

@srkunze
Copy link
Contributor

srkunze commented Oct 21, 2015

I have tried several ways removing a crontrab entry created by

salt-ssh minion cron.set_special root @hourly 'echo foobar > /dev/null'

All these variants don't work:

salt-ssh minion cron.rm_job root 'echo foobar > /dev/null'
salt-ssh minion cron.rm_job root '@hourly echo foobar > /dev/null'

Always the same output:

salt:/srv # salt-ssh minion cron.raw_cron root
minion:
    # Lines below here are managed by Salt, do not edit
    @hourly echo foobar > /dev/null

We currently use 2015.8.1

@jfindlay jfindlay added Execution-Module Bug broken, incorrect, or confusing behavior severity-medium 3rd level, incorrect or bad functionality, confusing and lacks a work around help-wanted Community help is needed to resolve this P4 Priority 4 Platform Relates to OS, containers, platform-based utilities like FS, system based apps labels Oct 22, 2015
@jfindlay jfindlay added this to the Approved milestone Oct 22, 2015
@jfindlay
Copy link
Contributor

@srkunze, thanks for the report.

@jfindlay jfindlay added P3 Priority 3 and removed P4 Priority 4 labels Oct 23, 2015
cachedout pushed a commit to cachedout/salt that referenced this issue Dec 30, 2015
@cachedout
Copy link
Contributor

All fixed up. See #30096. Thanks for alerting us to this issue! That's a pain to have a crontab entry you can't easily remove. ;]

@cachedout cachedout added the fixed-pls-verify fix is linked, bug author to confirm fix label Dec 30, 2015
@srkunze
Copy link
Contributor Author

srkunze commented Dec 30, 2015

Neat! Thanks for your effort here. :)

@srkunze srkunze closed this as completed Dec 30, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug broken, incorrect, or confusing behavior Execution-Module fixed-pls-verify fix is linked, bug author to confirm fix help-wanted Community help is needed to resolve this P3 Priority 3 Platform Relates to OS, containers, platform-based utilities like FS, system based apps severity-medium 3rd level, incorrect or bad functionality, confusing and lacks a work around
Projects
None yet
Development

No branches or pull requests

3 participants