)]}'
{"id":"openstack%2Fdevstack~165282","triplet_id":"openstack%2Fdevstack~master~Iba701a8cb00ef7ce5acd675e8714ad471fb4cb37","project":"openstack/devstack","branch":"master","topic":"remove_cinder_local_dev_filters","hashtags":[],"change_id":"Iba701a8cb00ef7ce5acd675e8714ad471fb4cb37","subject":"Remove configure_cinder_backend_conf_lvm","status":"ABANDONED","created":"2015-03-18 03:25:00.000000000","updated":"2015-03-18 05:08:57.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"7eec580ca94f9bd073154a7ab6571213bb5ea94b","_number":165282,"virtual_id_number":165282,"owner":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"actions":{},"labels":{"Verified":{"disliked":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":1,"date":"2015-03-18 04:40:52.000000000","_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"value":-1,"date":"2015-03-18 05:00:15.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","value":-1,"default_value":0,"optional":true},"Code-Review":{"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":3,"name":"Jenkins","username":"jenkins"}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","default_value":0,"optional":true},"Workflow":{"all":[{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"_account_id":3,"name":"Jenkins","username":"jenkins"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":3,"name":"Jenkins","username":"jenkins"}],"values":{"-1":"Branch Freeze"," 0":"No Priority","+1":"Important Change","+2":"Gate Blocker Fix / Urgent Change"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2015-03-18 04:40:52.000000000","updated_by":{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},"reviewer":{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2015-03-18 05:00:15.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"8243e38b4400aa5acad60d5e793fa722848e93e0","author":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"date":"2015-03-18 03:25:00.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"5b5510827aa235887e6849b3d82fe4ff0c430f1a","author":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"date":"2015-03-18 03:38:29.000000000","message":"Patch Set 2: Patch Set 1 was rebased","accounts_in_message":[],"_revision_number":1},{"id":"76fe6162aa2d1621e910598ba4e282a7df24922d","author":{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},"date":"2015-03-18 04:40:52.000000000","message":"Patch Set 2: Verified+1\n\nPassed using XenAPI driver with XenServer 6.2: Logs at http://dd6b71949550285df7dc-dda4e480e005aaa13ec303551d2d8155.r49.cf1.rackcdn.com/82/165282/2/13785/results.html\n\nUse \"xenserver: recheck\" to trigger only xenserver re-check.  XenServer CI contact: openstack@citrix.com.\n\nDebugging suggestions at https://wiki.openstack.org/wiki/Debugging_XenServer_CI_failures","accounts_in_message":[],"_revision_number":2},{"id":"d35961fd984185d9261d13a49f2906af81abe664","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-03-18 05:00:15.000000000","message":"Patch Set 2: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see http://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n- check-tempest-dsvm-full http://logs.openstack.org/82/165282/2/check/check-tempest-dsvm-full/f9cf403/ : SUCCESS in 45m 12s\n- check-tempest-dsvm-postgres-full http://logs.openstack.org/82/165282/2/check/check-tempest-dsvm-postgres-full/1e14dfc/ : SUCCESS in 38m 22s\n- check-tempest-dsvm-neutron-full http://logs.openstack.org/82/165282/2/check/check-tempest-dsvm-neutron-full/c5440d3/ : SUCCESS in 52m 08s\n- check-grenade-dsvm http://logs.openstack.org/82/165282/2/check/check-grenade-dsvm/2b08601/ : SUCCESS in 39m 21s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/82/165282/2/check/gate-tempest-dsvm-large-ops/ffa3c8f/ : SUCCESS in 28m 54s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/82/165282/2/check/gate-tempest-dsvm-neutron-large-ops/588c88c/ : SUCCESS in 26m 03s\n- gate-devstack-bashate http://logs.openstack.org/82/165282/2/check/gate-devstack-bashate/01ab9f6/ : SUCCESS in 16s\n- gate-devstack-unit-tests http://logs.openstack.org/82/165282/2/check/gate-devstack-unit-tests/c242b26/ : SUCCESS in 14s\n- check-devstack-dsvm-cells http://logs.openstack.org/82/165282/2/check/check-devstack-dsvm-cells/23fe25f/ : FAILURE in 20m 52s\n- check-grenade-dsvm-partial-ncpu http://logs.openstack.org/82/165282/2/check/check-grenade-dsvm-partial-ncpu/7ad1cda/ : SUCCESS in 44m 27s\n- check-swift-dsvm-functional http://logs.openstack.org/82/165282/2/check/check-swift-dsvm-functional/ab64980/ : SUCCESS in 24m 14s\n- check-tempest-dsvm-ironic-pxe_ssh http://logs.openstack.org/82/165282/2/check/check-tempest-dsvm-ironic-pxe_ssh/08bc847/ : SUCCESS in 40m 11s\n- check-devstack-dsvm-updown http://logs.openstack.org/82/165282/2/check/check-devstack-dsvm-updown/aff83b1/ : SUCCESS in 18m 53s\n- check-tempest-dsvm-f21 http://logs.openstack.org/82/165282/2/check/check-tempest-dsvm-f21/44e620b/ : FAILURE in 40m 51s (non-voting)\n- check-tempest-dsvm-centos7 http://logs.openstack.org/82/165282/2/check/check-tempest-dsvm-centos7/e35c3e8/ : FAILURE in 54m 11s (non-voting)\n- gate-devstack-publish-docs http://docs-draft.openstack.org/82/165282/2/check/gate-devstack-publish-docs/ab06cf2//doc/build/html/ : SUCCESS in 1m 45s\n- check-grenade-dsvm-neutron http://logs.openstack.org/82/165282/2/check/check-grenade-dsvm-neutron/3ccebc4/ : SUCCESS in 1h 20m 38s","accounts_in_message":[],"_revision_number":2},{"id":"01f9a15c474b5e131e5833cd7cc200e0ef010e87","author":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"date":"2015-03-18 05:08:57.000000000","message":"Abandoned\n\nNeed to merge this with the parent, otherwise the filters get screwed up and Cinder never sees the VG","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"4e89f1013cba9c5ebffe87c49a8629ee57271618","revisions":{"949950beabbbcf2b42d1faaba9918a4daca9e73b":{"kind":"REWORK","_number":1,"created":"2015-03-18 03:25:00.000000000","uploader":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"ref":"refs/changes/82/165282/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/82/165282/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/82/165282/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/82/165282/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/82/165282/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/82/165282/1"}}},"commit":{"parents":[{"commit":"44762266c63df4f3cdf5543afcfdb3c452c217a6","subject":"e global_filter to avoid scan of missing devs","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/44762266c63df4f3cdf5543afcfdb3c452c217a6"}]}],"author":{"name":"John Griffith","email":"john.griffith@solidfire.com","date":"2015-03-18 03:22:05.000000000","tz":-360},"committer":{"name":"John Griffith","email":"john.griffith@solidfire.com","date":"2015-03-18 03:22:05.000000000","tz":-360},"subject":"Remove configure_cinder_backend_conf_lvm","message":"Remove configure_cinder_backend_conf_lvm\n\nSetting the local device filter like we were doing here\ndoesn\u0027t actually solve the problem.  We actually need to\nset a global filter.\n\nSubmitted change: I5d5c48e188cbb9b4208096736807f082bce524e8\n\nOnce that lands this code won\u0027t be necessary at all and should\nbe removed.  We\u0027ll also remove the local conf usage in Cinder,\nbut currently that\u0027s harmless as it\u0027s designed to detect if\nthe local lvm.conf file exists in the cinder directory and\nbehave appropriately based on it\u0027s existence.\n\nChange-Id: Iba701a8cb00ef7ce5acd675e8714ad471fb4cb37\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/949950beabbbcf2b42d1faaba9918a4daca9e73b"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/949950beabbbcf2b42d1faaba9918a4daca9e73b"}]},"branch":"refs/heads/master"},"4e89f1013cba9c5ebffe87c49a8629ee57271618":{"kind":"NO_CHANGE","_number":2,"created":"2015-03-18 03:38:29.000000000","uploader":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"ref":"refs/changes/82/165282/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/82/165282/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/82/165282/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/82/165282/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/82/165282/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/82/165282/2"}}},"commit":{"parents":[{"commit":"51e99fc92ee74ca7054fcf7c6cf13e27f34701a5","subject":"Create global_filter to avoid scan of missing devs","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/51e99fc92ee74ca7054fcf7c6cf13e27f34701a5"}]}],"author":{"name":"John Griffith","email":"john.griffith@solidfire.com","date":"2015-03-18 03:22:05.000000000","tz":-360},"committer":{"name":"John Griffith","email":"john.griffith8@gmail.com","date":"2015-03-18 03:38:29.000000000","tz":0},"subject":"Remove configure_cinder_backend_conf_lvm","message":"Remove configure_cinder_backend_conf_lvm\n\nSetting the local device filter like we were doing here\ndoesn\u0027t actually solve the problem.  We actually need to\nset a global filter.\n\nSubmitted change: I5d5c48e188cbb9b4208096736807f082bce524e8\n\nOnce that lands this code won\u0027t be necessary at all and should\nbe removed.  We\u0027ll also remove the local conf usage in Cinder,\nbut currently that\u0027s harmless as it\u0027s designed to detect if\nthe local lvm.conf file exists in the cinder directory and\nbehave appropriately based on it\u0027s existence.\n\nChange-Id: Iba701a8cb00ef7ce5acd675e8714ad471fb4cb37\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/4e89f1013cba9c5ebffe87c49a8629ee57271618"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/4e89f1013cba9c5ebffe87c49a8629ee57271618"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}