[MB-6517] deleting development ddocs can delete production index files Created: 04/Sep/12  Updated: 20/Oct/12  Resolved: 05/Sep/12

Status: Closed
Project: Couchbase Server
Component/s: view-engine
Affects Version/s: 2.0
Fix Version/s: 2.0
Security Level: Public

Type: Bug Priority: Critical
Reporter: Tommie McAfee Assignee: Farshid Ghods
Resolution: Fixed Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified
Environment: build-1672 and cluster_run
2 Node cluster

Attachments: GZip Archive ns-diag-20120904183541.txt.gz     GZip Archive ns-diag-20120904183607.txt.gz    

 Description   
Load 3k documents with any tool. I used cbsystest:
python cbsystest.py run workload --template "default" --ops 1000 --create 100


1) create 2 default design docs, each with same view name, and same default map functions.

2) Publish 1 of the design docs from dev to production

3) Delete the other development ddoc
   * deletes index file from disk for both ddocs

4) Querying published ddoc now reports view_merge errors:
Subset of nodes failed with the following error:
[ {
"from": "local",
"reason": "Undefined set view `default` for `_design/ddoc1` design document."
}, {
"from": "http://172.20.10.3:9501/_view_merge/?limit=10",
"reason": "{<<\"unknown_error\">>,<<\"view_undefined\">>}"
} ]


ns

 Comments   
Comment by Farshid Ghods [ 04/Sep/12 ]
Tommie,


please provide the map-reduce function definition for all views as well. assign this back to Filipe after
Comment by Filipe Manana [ 04/Sep/12 ]
Happens because of the change from MB-6415.
Reverting that change, will cause issues if there's ddoc updates/deletes during rebalance (view-engine can't tell when rebalance is ongoing or not).
Comment by Filipe Manana [ 04/Sep/12 ]
MB-6415 is actually a bit extreme, one needs to update a ddoc, and then after, before rebalance finishes, update it again to the previous version or create a new design document that matches the initial version of the updated one.

I'll see if I can go around this without reverting MB-6415 completely.
Comment by Peter Wansch [ 05/Sep/12 ]
Please let Iryna know to add a test case for this case found by Tommie.
Comment by Filipe Manana [ 05/Sep/12 ]
Fix merged to master: http://review.couchbase.org/#/c/20481/

Farshid, sounds like a useful test to have in testrunner.
Do you think your team should write one? Let me know, and I'll close this issue and open a QE one for that.

thanks
Comment by Farshid Ghods [ 05/Sep/12 ]
http://www.couchbase.com/issues/browse/CBQE-582
 is opened already for adding this test to testrunner.

we can close this bug once the fix is merged
Comment by Thuan Nguyen [ 05/Sep/12 ]
Integrated in github-couchdb-preview #501 (See [http://qa.hq.northscale.net/job/github-couchdb-preview/501/])
    MB-6517 Delete index files only if there are no aliases (Revision 884ba59c4e3f83db5f38cd056529d979f3e9d7e8)

     Result = SUCCESS
pwansch :
Files :
* src/couch_set_view/test/02-old-index-cleanup.t
* src/couch_set_view/src/couch_set_view.erl
* src/couch_set_view/src/couch_set_view_group.erl
Comment by Thuan Nguyen [ 21/Sep/12 ]
Integrated in centos-64-2.0-failover-regression #23 (See [http://qa.hq.northscale.net/job/centos-64-2.0-failover-regression/23/])
    CBQE-582: add a regression test for issue mb-6517 (Revision 8a813000497a7ac7683a9ec0b72c22e6972d3fcb)

     Result = UNSTABLE
Comment by Thuan Nguyen [ 21/Sep/12 ]
Integrated in single-node-2.0.x-windows7-64-view #14 (See [http://qa.hq.northscale.net/job/single-node-2.0.x-windows7-64-view/14/])
    
     Result = UNSTABLE
Comment by Thuan Nguyen [ 26/Sep/12 ]
Integrated in multi-nodes-windows-64-viewtest #16 (See [http://qa.hq.northscale.net/job/multi-nodes-windows-64-viewtest/16/])
    CBQE-582: add a regression test for issue mb-6517 (Revision 8a813000497a7ac7683a9ec0b72c22e6972d3fcb)

     Result = UNSTABLE
Comment by Thuan Nguyen [ 26/Sep/12 ]
Integrated in single-node-windows-64-install #367 (See [http://qa.hq.northscale.net/job/single-node-windows-64-install/367/])
    CBQE-582: add a regression test for issue mb-6517 (Revision 8a813000497a7ac7683a9ec0b72c22e6972d3fcb)

     Result = SUCCESS
Comment by Thuan Nguyen [ 26/Sep/12 ]
Integrated in single-node-2.0.x-windows7-32-smoke #16 (See [http://qa.hq.northscale.net/job/single-node-2.0.x-windows7-32-smoke/16/])
    CBQE-582: add a regression test for issue mb-6517 (Revision 8a813000497a7ac7683a9ec0b72c22e6972d3fcb)

     Result = UNSTABLE
Comment by Thuan Nguyen [ 26/Sep/12 ]
Integrated in single-node 2.0.x-windows-32-smoke #13 (See [http://qa.hq.northscale.net/job/single-node%202.0.x-windows-32-smoke/13/])
    CBQE-582: add a regression test for issue mb-6517 (Revision 8a813000497a7ac7683a9ec0b72c22e6972d3fcb)

     Result = UNSTABLE
Comment by Thuan Nguyen [ 10/Oct/12 ]
Integrated in multi-nodes-2.0.x-windows-64-install #15 (See [http://qa.hq.northscale.net/job/multi-nodes-2.0.x-windows-64-install/15/])
    
     Result = SUCCESS
Comment by Thuan Nguyen [ 20/Oct/12 ]
Integrated in multi-nodes-windows-64-viewquery #13 (See [http://qa.hq.northscale.net/job/multi-nodes-windows-64-viewquery/13/])
    
     Result = UNSTABLE
Generated at Sat May 18 20:17:04 CDT 2013 using JIRA 5.2.4#845-sha1:c9f4cc41abe72fb236945343a1f485c2c844dac9.