Bug 969719 - setup_rewritten in managed_files.yml doesn't do anything
Summary: setup_rewritten in managed_files.yml doesn't do anything
Keywords:
Status: CLOSED NOTABUG
Alias: None
Product: OpenShift Online
Classification: Red Hat
Component: Containers
Version: 2.x
Hardware: Unspecified
OS: Unspecified
unspecified
medium
Target Milestone: ---
: ---
Assignee: Jhon Honce
QA Contact: libra bugs
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-06-02 01:47 UTC by Dan McPherson
Modified: 2015-05-14 23:20 UTC (History)
0 users

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-06-04 23:29:23 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)

Description Dan McPherson 2013-06-02 01:47:52 UTC
Description of problem:

According to:

https://github.com/openshift/origin-server/blob/master/node/README.writing_cartridges.md

It's supposed to delete those listings before rerunning setup.  It seems to do nothing.


Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1. Add blah to setup_rewritten in managed_files.yml of a cart
2. touch blah
3. Rerun a setup file that fails if blah exists and then writes blah

Actual results:
It fails

Expected results:
Should succeed because setup_rewritten specified blah

Additional info:

Comment 1 Jhon Honce 2013-06-04 23:29:23 UTC
The setup script is currently run in two scenarios:

1) on cartridge create: No files exist so there are no files to delete
2) on cartridge migration: setup_rewritten files are removed in this case

Please reopen bug if there is a scenario that has been missed.

Comment 2 Dan McPherson 2013-06-05 00:14:44 UTC
Sorry I searched for the code before and missed the code in migrate.rb (and my scenario I listed wasn't really correct.  Should have referenced the migration.).  There is one scenario missed but hasn't been implemented yet to handle upgrading downloadable carts.


Note You need to log in before you can comment on or make changes to this bug.