[library] newer sysctl module breaks deployment

Bug #1337415 reported by Vladimir Kuklin on 2014-07-03
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Critical
Vladimir Kuklin

Bug Description

newer sysctl module has execute path incompatibility with puppet.

Debug: /Firewall[999 drop all other requests]: [validate]
Debug: /Firewall[003 remote rabbitmq ]: [validate]
Debug: /Firewall[004 remote puppet ]: [validate]
Error: Parameter unless failed on Exec[exec_sysctl_net.ipv4.tcp_keepalive_time]: '[ "$(sysctl -n net.ipv4.tcp_keepalive_time)" = 30 ]' is not qualified and no path was specified. Please qualify the command or specify a path. at /etc/puppet/modules/sysctl/manifests/value.pp:36
Wrapped exception:
'[ "$(sysctl -n net.ipv4.tcp_keepalive_time)" = 30 ]'

It looks like 0.0.4 sysctl module is not working with our puppet version.

Reviewed: https://review.openstack.org/104607
Committed: https://git.openstack.org/cgit/stackforge/fuel-library/commit/?id=945ae2b897acec0f5ead6836385365b722dc44aa
Submitter: Jenkins
Branch: master

commit 945ae2b897acec0f5ead6836385365b722dc44aa
Author: Vladimir Kuklin <email address hidden>
Date: Thu Jul 3 16:23:07 2014 +0000

    Revert "Sync sysctl module to v0.0.4 from upstream"

    Partial-bug: #1337415

    This reverts commit 6587dca0e92c03073bac57895b2caafae6d289c5.

    Change-Id: I86d233e77ac8107f034a494a725fe6947fb94427

Fix proposed to branch: master
Review: https://review.openstack.org/104613

Changed in fuel:
status: In Progress → Fix Committed

Reviewed: https://review.openstack.org/104613
Committed: https://git.openstack.org/cgit/stackforge/fuel-library/commit/?id=07e6ed7767f6bbc00d834d3bd85857e5828687d3
Submitter: Jenkins
Branch: master

commit 07e6ed7767f6bbc00d834d3bd85857e5828687d3
Author: Vladimir Kuklin <email address hidden>
Date: Thu Jul 3 16:32:01 2014 +0000

    Revert "Adapt synced sysctl module for Fuel"

    Closes-bug: #1337415
    This reverts commit 178f4f8c2ff93f6b7cf9c08c49d04c7819228400.

    Change-Id: I7c6f3cae716956b8a2b62868655af1bfbcdd28e0

Dmitry Ilyin (idv1985) on 2014-07-16
summary: - newer sysctl module breaks deployment
+ [library] newer sysctl module breaks deployment
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers