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.file is broken #34094

Closed
avandendorpe opened this issue Jun 17, 2016 · 1 comment
Closed

cron.file is broken #34094

avandendorpe opened this issue Jun 17, 2016 · 1 comment
Labels
Bug broken, incorrect, or confusing behavior fixed-pls-verify fix is linked, bug author to confirm fix P2 Priority 2 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 State-Module
Milestone

Comments

@avandendorpe
Copy link
Contributor

The cron.file state is broken on 2016.3.0 and 2016.3.1, a regression introduced by an extra argument on get_managed().

Error:
Unable to manage file: get_managed() takes exactly 11 arguments (10 given)

It was fixed by #32396. Can the fix be included in 2016.3.2?

@rallytime
Copy link
Contributor

@avandendorpe I have back-ported the fix from develop with #34095. It will be included in the 2016.3.2 release. Thanks for filing this!

@rallytime rallytime added State-Module Bug broken, incorrect, or confusing behavior severity-medium 3rd level, incorrect or bad functionality, confusing and lacks a work around fixed-pls-verify fix is linked, bug author to confirm fix P2 Priority 2 Platform Relates to OS, containers, platform-based utilities like FS, system based apps labels Jun 17, 2016
@rallytime rallytime added this to the Approved milestone Jun 17, 2016
uqs pushed a commit to freebsd/freebsd-ports that referenced this issue Jul 13, 2016
Fixes cron.file from upstream issue #34094/#34095 [1].  This will be in
2016.3.2.
- saltstack/salt#34094
- saltstack/salt#34095

Restore patch for upstream issue #33608. This was patched in 2016.3.0 but
removed in the maintainer update to 2016.3.1 in r417508. However the patch
has yet to be merged upstream. [2]
- saltstack/salt#33608

PR:		210627 [1], 210395 [2]
Reported by:	Andres Montalban <amontalban@gmail.com>
Approved by:	Christer Edwards <christer.edwards@gmail.com> (maintainer)
MFH:		2016Q3


git-svn-id: svn+ssh://svn.freebsd.org/ports/head@418455 35697150-7ecd-e111-bb59-0022644237b5
uqs pushed a commit to freebsd/freebsd-ports that referenced this issue Jul 13, 2016
Fixes cron.file from upstream issue #34094/#34095 [1].  This will be in
2016.3.2.
- saltstack/salt#34094
- saltstack/salt#34095

Restore patch for upstream issue #33608. This was patched in 2016.3.0 but
removed in the maintainer update to 2016.3.1 in r417508. However the patch
has yet to be merged upstream. [2]
- saltstack/salt#33608

PR:		210627 [1], 210395 [2]
Reported by:	Andres Montalban <amontalban@gmail.com>
Approved by:	Christer Edwards <christer.edwards@gmail.com> (maintainer)
MFH:		2016Q3
uqs pushed a commit to freebsd/freebsd-ports that referenced this issue Jul 13, 2016
sysutils/py-salt: add patches to handle runtime regressions in 2016.3.x

Fixes cron.file from upstream issue #34094/#34095 [1].  This will be in
2016.3.2.
- saltstack/salt#34094
- saltstack/salt#34095

Restore patch for upstream issue #33608. This was patched in 2016.3.0 but
removed in the maintainer update to 2016.3.1 in r417508. However the patch
has yet to be merged upstream. [2]
- saltstack/salt#33608

PR:		210627 [1], 210395 [2]
Reported by:	Andres Montalban <amontalban@gmail.com>
Approved by:	Christer Edwards <christer.edwards@gmail.com> (maintainer)
Approved by:	ports-secteam (with hat)
uqs pushed a commit to freebsd/freebsd-ports that referenced this issue Dec 2, 2016
sysutils/py-salt: add patches to handle runtime regressions in 2016.3.x

Fixes cron.file from upstream issue #34094/#34095 [1].  This will be in
2016.3.2.
- saltstack/salt#34094
- saltstack/salt#34095

Restore patch for upstream issue #33608. This was patched in 2016.3.0 but
removed in the maintainer update to 2016.3.1 in r417508. However the patch
has yet to be merged upstream. [2]
- saltstack/salt#33608

PR:		210627 [1], 210395 [2]
Reported by:	Andres Montalban <amontalban@gmail.com>
Approved by:	Christer Edwards <christer.edwards@gmail.com> (maintainer)
Approved by:	ports-secteam (with hat)
uqs pushed a commit to freebsd/freebsd-ports that referenced this issue Apr 1, 2021
sysutils/py-salt: add patches to handle runtime regressions in 2016.3.x

Fixes cron.file from upstream issue #34094/#34095 [1].  This will be in
2016.3.2.
- saltstack/salt#34094
- saltstack/salt#34095

Restore patch for upstream issue #33608. This was patched in 2016.3.0 but
removed in the maintainer update to 2016.3.1 in r417508. However the patch
has yet to be merged upstream. [2]
- saltstack/salt#33608

PR:		210627 [1], 210395 [2]
Reported by:	Andres Montalban <amontalban@gmail.com>
Approved by:	Christer Edwards <christer.edwards@gmail.com> (maintainer)
Approved by:	ports-secteam (with hat)
svmhdvn pushed a commit to svmhdvn/freebsd-ports that referenced this issue Jan 10, 2024
Fixes cron.file from upstream issue #34094/#34095 [1].  This will be in
2016.3.2.
- saltstack/salt#34094
- saltstack/salt#34095

Restore patch for upstream issue #33608. This was patched in 2016.3.0 but
removed in the maintainer update to 2016.3.1 in r417508. However the patch
has yet to be merged upstream. [2]
- saltstack/salt#33608

PR:		210627 [1], 210395 [2]
Reported by:	Andres Montalban <amontalban@gmail.com>
Approved by:	Christer Edwards <christer.edwards@gmail.com> (maintainer)
MFH:		2016Q3
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 fixed-pls-verify fix is linked, bug author to confirm fix P2 Priority 2 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 State-Module
Projects
None yet
Development

No branches or pull requests

2 participants