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 #3969: (DEVELOPMENT) Move failing inventories to the right directo... #230

Conversation

Kegeruneku
Copy link

...ry

Ticket: http://www.rudder-project.org/redmine/issues/3969

To be merged in 2.8

@@ -117,14 +117,14 @@ bundle agent sendInventoryToCmdb
policy_server.!root_server::

"${g.rudder_inventories}/incoming"
transformer => "/usr/bin/curl -f -s --proxy '' --user rudder:rudder -T ${this.promiser} http://${server_info.cfserved}/inventories/",
transformer => "/usr/bin/curl -f -s --proxy '' --user rudder:rudder -T ${this.promiser} http://${server_info.cfserved}/inventories/ ${g.rudder_inventories}/failed/",
Copy link
Member

Choose a reason for hiding this comment

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

Er, I don't think this change makes sense! curl is not the same as send-clean.sh

@jooooooon
Copy link
Member

Aside from the curl change, this looks good to me. Please fix and merge.

@Kegeruneku
Copy link
Author

Corrected, merging as required.

Kegeruneku pushed a commit that referenced this pull request Oct 21, 2013
…_inventories_to_graveyard

Fixes #3969: (DEVELOPMENT) Move failing inventories to the right directo...
@Kegeruneku Kegeruneku merged commit 803ad72 into Normation:branches/rudder/2.8 Oct 21, 2013
amousset pushed a commit to amousset/rudder-techniques that referenced this pull request Dec 8, 2015
…lank_lines_in_template

Fixes #4487: prevent the removal of blank lines in template (backport of...
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