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

Fixes #6714: Compress ldap backups #756

Conversation

matya
Copy link
Contributor

@matya matya commented Oct 15, 2015

Every time slapd is restarted, it writes a backup, that can get big in time
This modification compresses every uncompressed ldif-backups with gzip just
like the webapp-logs.

amousset pushed a commit to amousset/rudder-techniques that referenced this pull request Dec 8, 2015
…rror_in_specfile_on_rudder_agent

Fixes #7258: Syntax error in specfile on rudder-agent
@matya
Copy link
Contributor Author

matya commented Feb 3, 2016

Any chance this gets merged @peckpeck ? :-)

# Select a file using a pattern #
########################################################################

body file_select simple_pattern(pattern)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This body already exists in cfengine stdlib and is called "by_name"

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed, also rebased to latest 2.11, which fixed the previously bad ${g.gzip} variable

@matya matya force-pushed the patch-6714-compress-ldap-backups branch from cc01d92 to 80c5d01 Compare February 19, 2016 22:02
"/var/rudder/ldap/backup"
file_select => by_name("@{backups_pattern}"),
depth_search => recurse("0"),
transformer => "${g.rudder_gzip} \"${this.promiser}\"";
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this should be ${g.gzip}

@peckpeck
Copy link
Member

Hello @matya could you change the gzip line ?
Thanks !

Every time slapd is restarted, it writes a backup, that can get big in time.
This modification compresses every uncompressed ldif-backups with gzip just
like the webapp-logs.
@matya matya force-pushed the patch-6714-compress-ldap-backups branch from 80c5d01 to a816a6c Compare March 31, 2016 14:50
@matya
Copy link
Contributor Author

matya commented Mar 31, 2016

@peckpeck done.

@peckpeck
Copy link
Member

Good, merging

@peckpeck peckpeck merged commit d70bec9 into Normation:branches/rudder/2.11 Mar 31, 2016
@matya matya deleted the patch-6714-compress-ldap-backups branch March 31, 2016 15:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants