Murano should automatically convert script line-endings for the target guest OS

Bug #1496376 reported by Pradip Rawat
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Murano
Fix Released
Medium
Valerii Kovalchuk

Bug Description

Murano packages are often developed in envoronment other than used for deployment. For example it is often that Linux shell scripts get edited using Windows editor. As a result scripts have Windows line-endings and such scripts fail to work on Linux with confusing error messages (like file not found when it is).

This can be avoided if Murano use universal line-endings for text-mode files (those that are not enclosed in triangle brackets in
execution plan templates).

Changed in murano:
status: New → Opinion
status: Opinion → New
Stan Lagun (slagun)
Changed in murano:
importance: Undecided → Wishlist
description: updated
Stan Lagun (slagun)
Changed in murano:
milestone: none → next
tags: added: low-hanging-fruit
Changed in murano:
status: New → Confirmed
importance: Wishlist → Medium
Changed in murano:
milestone: next → mitaka-1
Changed in murano:
assignee: nobody → Ruchi Parmar (ruchiparmar143)
Changed in murano:
status: Confirmed → In Progress
Changed in murano:
milestone: mitaka-1 → mitaka-2
Revision history for this message
Kirill Zaitsev (kzaitsev) wrote :

2 months, no progress. Removing assignee and in-progress mention. ANyway you shouldn't set in-progress by hand.

Changed in murano:
assignee: Ruchi Parmar (ruchiparmar143) → nobody
status: In Progress → New
status: New → Confirmed
Changed in murano:
assignee: nobody → Ravi Shekhar Jethani (ravishekar-jethani)
Revision history for this message
Ravi Shekhar Jethani (ravishekar-jethani) wrote :

Application scripts that are run by murano-agent are an outside entity to murano so IMO murano should not be responsible for fixing problems in appliction scripts.

Also there can be thousands of such scripts executed directly or indirectly by murano-agent. Checking each and every file doesn't sound practical.

Changed in murano:
milestone: mitaka-2 → mitaka-3
Revision history for this message
Kirill Zaitsev (kzaitsev) wrote :

Ravi Shekhar Jethani, are you still working on this one?

Revision history for this message
Kirill Zaitsev (kzaitsev) wrote :

Looks like there is still no progress here, removing assignee

Changed in murano:
assignee: Ravi Shekhar Jethani (ravishekar-jethani) → nobody
Changed in murano:
milestone: mitaka-3 → mitaka-rc1
Changed in murano:
milestone: mitaka-rc1 → newton-1
Changed in murano:
assignee: nobody → Valerii Kovalchuk (vakovalchuk)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to murano (master)

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

Changed in murano:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to murano-agent (master)

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on murano (master)

Change abandoned by Valerii Kovalchuk (<email address hidden>) on branch: master
Review: https://review.openstack.org/311104
Reason: Made a solution on the agent side https://review.openstack.org/#/c/316771/

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on murano-agent (master)

Change abandoned by Valerii Kovalchuk (<email address hidden>) on branch: master
Review: https://review.openstack.org/316771

Changed in murano:
milestone: newton-1 → newton-2
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to murano (master)

Reviewed: https://review.openstack.org/311104
Committed: https://git.openstack.org/cgit/openstack/murano/commit/?id=9d0c88a47bf871c5f5698aa77c9433a4bf1baa31
Submitter: Jenkins
Branch: master

commit 9d0c88a47bf871c5f5698aa77c9433a4bf1baa31
Author: Valerii Kovalchuk <email address hidden>
Date: Fri Apr 29 15:52:31 2016 +0300

    Convert script line endings for the target OS

    This patch adds 'binary' parameter to ResourceManager 'string'
    method. It allows to open text files in 'rU' mode which recognizes
    all types of newlines, and binary files in 'rb' mode to prevent
    their corruption.

    Change-Id: I505070df187dfd910e68c555319e3ef92ce2c826
    Closes-bug: #1496376

Changed in murano:
status: In Progress → Fix Released
Revision history for this message
Davanum Srinivas (DIMS) (dims-v) wrote : Fix included in openstack/murano 3.0.0.0b2

This issue was fixed in the openstack/murano 3.0.0.0b2 development milestone.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.