)]}'
{"id":"openstack%2Fdevstack~611248","triplet_id":"openstack%2Fdevstack~master~I35dd53c3b3dd0254cb1fb3b2af74fb4346dd1251","project":"openstack/devstack","branch":"master","hashtags":[],"change_id":"I35dd53c3b3dd0254cb1fb3b2af74fb4346dd1251","subject":"Do not install test-requirements.txt dependencies","status":"ABANDONED","created":"2018-10-17 06:57:32.000000000","updated":"2022-05-25 07:40:42.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"6da61740dbf8909de642d5ac28a56d8c13a533cb","_number":611248,"virtual_id_number":611248,"owner":{"_account_id":21883,"name":"Manuel Buil","email":"mbuil@suse.com","username":"mbuil"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"date":"2018-10-17 09:22:20.000000000","_account_id":14595,"name":"z Systems KVM","email":"zkvm-ci@linux.vnet.ibm.com","username":"ibm-zkvm-ci","tags":["SERVICE_USER"]},{"value":0,"date":"2018-10-17 11:44:28.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},{"date":"2018-10-17 10:18:00.000000000","_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"},{"date":"2018-10-17 11:34:46.000000000","_account_id":16376,"name":"Intel NFV CI","email":"openstack-nfv-ci@intel.com","username":"intel-nfv-ci","tags":["SERVICE_USER"]},{"date":"2018-10-25 07:39:23.000000000","_account_id":21883,"name":"Manuel Buil","email":"mbuil@suse.com","username":"mbuil"},{"_account_id":26507,"name":"Michel Peterson","email":"michel@redhat.com","username":"mpeterson"},{"value":1,"date":"2018-10-17 12:49:39.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"}],"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":{"recommended":{"_account_id":26507,"name":"Michel Peterson","email":"michel@redhat.com","username":"mpeterson"},"disliked":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":14595,"name":"z Systems KVM","email":"zkvm-ci@linux.vnet.ibm.com","username":"ibm-zkvm-ci","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":16376,"name":"Intel NFV CI","email":"openstack-nfv-ci@intel.com","username":"intel-nfv-ci","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":21883,"name":"Manuel Buil","email":"mbuil@suse.com","username":"mbuil"},{"value":1,"date":"2018-10-17 10:33:33.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":26507,"name":"Michel Peterson","email":"michel@redhat.com","username":"mpeterson"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":-1,"date":"2018-10-17 23:13:20.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"}],"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":"","value":-1,"default_value":0,"optional":true},"Workflow":{"all":[{"_account_id":14595,"name":"z Systems KVM","email":"zkvm-ci@linux.vnet.ibm.com","username":"ibm-zkvm-ci","tags":["SERVICE_USER"]},{"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},{"_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"},{"_account_id":16376,"name":"Intel NFV CI","email":"openstack-nfv-ci@intel.com","username":"intel-nfv-ci","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":-1,"max":0},"_account_id":21883,"name":"Manuel Buil","email":"mbuil@suse.com","username":"mbuil"},{"_account_id":26507,"name":"Michel Peterson","email":"michel@redhat.com","username":"mpeterson"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"}],"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":14595,"name":"z Systems KVM","email":"zkvm-ci@linux.vnet.ibm.com","username":"ibm-zkvm-ci","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":16376,"name":"Intel NFV CI","email":"openstack-nfv-ci@intel.com","username":"intel-nfv-ci","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":21883,"name":"Manuel Buil","email":"mbuil@suse.com","username":"mbuil"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":26507,"name":"Michel Peterson","email":"michel@redhat.com","username":"mpeterson"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":-1,"max":2},"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"}],"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":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},{"_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"},{"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},{"_account_id":14595,"name":"z Systems KVM","email":"zkvm-ci@linux.vnet.ibm.com","username":"ibm-zkvm-ci","tags":["SERVICE_USER"]},{"_account_id":16376,"name":"Intel NFV CI","email":"openstack-nfv-ci@intel.com","username":"intel-nfv-ci","tags":["SERVICE_USER"]},{"_account_id":21883,"name":"Manuel Buil","email":"mbuil@suse.com","username":"mbuil"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":26507,"name":"Michel Peterson","email":"michel@redhat.com","username":"mpeterson"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2018-10-17 09:22:20.000000000","updated_by":{"_account_id":14595,"name":"z Systems KVM","email":"zkvm-ci@linux.vnet.ibm.com","username":"ibm-zkvm-ci","tags":["SERVICE_USER"]},"reviewer":{"_account_id":14595,"name":"z Systems KVM","email":"zkvm-ci@linux.vnet.ibm.com","username":"ibm-zkvm-ci","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2018-10-17 10:18:00.000000000","updated_by":{"_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"},"reviewer":{"_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"},"state":"REVIEWER"},{"updated":"2018-10-17 10:33:33.000000000","updated_by":{"_account_id":26507,"name":"Michel Peterson","email":"michel@redhat.com","username":"mpeterson"},"reviewer":{"_account_id":26507,"name":"Michel Peterson","email":"michel@redhat.com","username":"mpeterson"},"state":"REVIEWER"},{"updated":"2018-10-17 11:34:46.000000000","updated_by":{"_account_id":16376,"name":"Intel NFV CI","email":"openstack-nfv-ci@intel.com","username":"intel-nfv-ci","tags":["SERVICE_USER"]},"reviewer":{"_account_id":16376,"name":"Intel NFV CI","email":"openstack-nfv-ci@intel.com","username":"intel-nfv-ci","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2018-10-17 11:44:28.000000000","updated_by":{"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},"reviewer":{"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2018-10-17 12:49:39.000000000","updated_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"reviewer":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2018-10-17 23:13:20.000000000","updated_by":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"reviewer":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"state":"REVIEWER"}],"messages":[{"id":"06f4221aaecd13827d126ed6ff23ea5b34bd5b26","author":{"_account_id":21883,"name":"Manuel Buil","email":"mbuil@suse.com","username":"mbuil"},"date":"2018-10-17 06:57:32.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"1f85662059ab1e72273a7117867428f4a99f7f3c","author":{"_account_id":26507,"name":"Michel Peterson","email":"michel@redhat.com","username":"mpeterson"},"date":"2018-10-17 07:37:32.000000000","message":"Patch Set 1: Code-Review-1\n\nI think it\u0027s sound to expect that only requirements.txt is installed and not test-requirements.txt. Mostly it LGTM, but I do want to note that this could break some code that was depending on that. \n\nAs such, \n\n(1) [reason of the -1] it should include a release note explaining the rationale and the potential impact of the change\n\n(2) [non-voting] we should weigh if we want to make it not breaking in a first iteration by adding a parameter that defaults to install test-requirements.txt but that can be overridden. Then again, I don\u0027t think we should expect test-requirements.txt to be installed as part of pip_install and forcing it sounds like incorrect to me.","accounts_in_message":[],"_revision_number":1},{"id":"3a9fcb09b957374af544d02efa95d4cffbd64d11","author":{"_account_id":21883,"name":"Manuel Buil","email":"mbuil@suse.com","username":"mbuil"},"date":"2018-10-17 07:53:25.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"cdab425530a93e3f2c526896187e8afad9619c6d","author":{"_account_id":16376,"name":"Intel NFV CI","email":"openstack-nfv-ci@intel.com","username":"intel-nfv-ci","tags":["SERVICE_USER"]},"date":"2018-10-17 08:57:54.000000000","message":"Patch Set 2:\n\nBuild succeeded (check pipeline).\n\n- tempest-dsvm-intel-nfv-xenial http://52.27.155.124/portland/2018-10-17/611248/2/check/tempest-dsvm-intel-nfv-xenial/513284f : SUCCESS in 27m 41s (non-voting)\n- tempest-dsvm-ovsdpdk-nfv-networking-xenial http://52.27.155.124/portland/2018-10-17/611248/2/check/tempest-dsvm-ovsdpdk-nfv-networking-xenial/cdd9dc5 : SUCCESS in 57m 03s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"7afb779c99306b71a1f5696a98ce11199dec93dc","author":{"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},"date":"2018-10-17 09:10:18.000000000","message":"Patch Set 2:\n\nBuild succeeded. Test completed on IBM PowerKVM platform. For rechecking only on the IBM PowerKVM CI, add a review comment with pkvm: recheck. For contact and more information, see https://wiki.openstack.org/wiki/PowerKVM\n\n- tempest-dsvm-full-xenial https://dal05.objectstorage.softlayer.net/v1/AUTH_3d8e6ecb-f597-448c-8ec2-164e9f710dd6/pkvmci/devstack/48/611248/2/check/tempest-dsvm-full-xenial/b1be331/ : SUCCESS in 1h 16m 38s","accounts_in_message":[],"_revision_number":2},{"id":"5700bea036641aad37e1fa74b133b66b35659128","author":{"_account_id":14595,"name":"z Systems KVM","email":"zkvm-ci@linux.vnet.ibm.com","username":"ibm-zkvm-ci","tags":["SERVICE_USER"]},"date":"2018-10-17 09:22:20.000000000","message":"Patch Set 2:\n\nTesting succeeded on ubuntu-xenial-s390x. For rechecking only on the ubuntu-xenail-s390x CI, add a review comment with recheck-zkvm. Contact info: zkvm-ci@linux.vnet.ibm.com. For more information, see https://wiki.openstack.org/wiki/ThirdPartySystems/IBM_zKVM_CI\n\n- check-tempest-dsvm-neutron-full-ubuntu-xenial-s390x http://sng01.objectstorage.softlayer.net/v1/AUTH_1940ea10-6e82-4501-b2f9-eb236510e575/ibmzkvmci/production/611248/2/check-tempest-dsvm-neutron-full-ubuntu-xenial-s390x/819644a : SUCCESS in 1h 28m 24s","accounts_in_message":[],"_revision_number":2},{"id":"3e2ae3f7caeefeb2b707fb4954a8284208e7c03c","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-10-17 10:17:46.000000000","message":"Patch Set 2: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttp://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n\n- legacy-devstack-dsvm-updown http://logs.openstack.org/48/611248/2/check/legacy-devstack-dsvm-updown/b2d84d3/ : SUCCESS in 28m 23s\n- tempest-full http://logs.openstack.org/48/611248/2/check/tempest-full/bc1fc27/ : SUCCESS in 1h 30m 31s\n- neutron-grenade http://logs.openstack.org/48/611248/2/check/neutron-grenade/7bb3880/ : SUCCESS in 49m 03s\n- tempest-full-py3 http://logs.openstack.org/48/611248/2/check/tempest-full-py3/0d76b52/ : FAILURE in 1h 20m 33s\n- openstack-tox-docs http://logs.openstack.org/48/611248/2/check/openstack-tox-docs/d96856b/html/ : SUCCESS in 6m 25s\n- devstack http://logs.openstack.org/48/611248/2/check/devstack/0c6fef4/ : SUCCESS in 23m 22s\n- devstack-platform-centos-7 http://logs.openstack.org/48/611248/2/check/devstack-platform-centos-7/d467623/ : SUCCESS in 1h 43m 36s (non-voting)\n- devstack-platform-opensuse-150 http://logs.openstack.org/48/611248/2/check/devstack-platform-opensuse-150/e3e9d57/ : RETRY_LIMIT in 2m 28s (non-voting)\n- devstack-platform-opensuse-tumbleweed http://logs.openstack.org/48/611248/2/check/devstack-platform-opensuse-tumbleweed/2b9c5a7/ : FAILURE in 1h 38m 37s (non-voting)\n- devstack-platform-fedora-latest http://logs.openstack.org/48/611248/2/check/devstack-platform-fedora-latest/0eaef8e/ : FAILURE in 1h 20m 07s (non-voting)\n- devstack-multinode http://logs.openstack.org/48/611248/2/check/devstack-multinode/5337cb6/ : SUCCESS in 25m 59s\n- devstack-unit-tests http://logs.openstack.org/48/611248/2/check/devstack-unit-tests/c82ae80/ : SUCCESS in 5m 38s\n- openstack-tox-bashate http://logs.openstack.org/48/611248/2/check/openstack-tox-bashate/bc79b2b/ : SUCCESS in 4m 19s\n- ironic-tempest-dsvm-ipa-wholedisk-bios-agent_ipmitool-tinyipa http://logs.openstack.org/48/611248/2/check/ironic-tempest-dsvm-ipa-wholedisk-bios-agent_ipmitool-tinyipa/c89c5f7/ : SUCCESS in 49m 33s (non-voting)\n- swift-dsvm-functional http://logs.openstack.org/48/611248/2/check/swift-dsvm-functional/edecb4e/ : SUCCESS in 27m 50s (non-voting)\n- neutron-grenade-multinode http://logs.openstack.org/48/611248/2/check/neutron-grenade-multinode/85bbb40/ : SUCCESS in 1h 11m 04s\n- neutron-tempest-linuxbridge http://logs.openstack.org/48/611248/2/check/neutron-tempest-linuxbridge/98cbe26/ : SUCCESS in 2h 01m 21s\n- tempest-multinode-full http://logs.openstack.org/48/611248/2/check/tempest-multinode-full/08fd137/ : SUCCESS in 1h 42m 18s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"c768085cf4b0d744def76de2f765e0b4cd33a91a","author":{"_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"},"date":"2018-10-17 10:18:00.000000000","message":"Patch Set 2:\n\nI noticed Zuul failed, I think you hit bug(s):\n\n- tempest-full-py3: https://bugs.launchpad.net/bugs/1763070\n\nIf you believe we\u0027ve correctly identified the failure, feel free to leave a \u0027recheck\u0027 comment to run the tests again.\nFor more details on this and other bugs, please see http://status.openstack.org/elastic-recheck/","accounts_in_message":[],"_revision_number":2},{"id":"610ee15190cdc02ab14909aaa307ff0aefde150a","author":{"_account_id":26507,"name":"Michel Peterson","email":"michel@redhat.com","username":"mpeterson"},"date":"2018-10-17 10:33:27.000000000","message":"Patch Set 2:\n\nrecheck","accounts_in_message":[],"_revision_number":2},{"id":"3b72176aed77c961b344be34785167e1eb4e9c6c","author":{"_account_id":26507,"name":"Michel Peterson","email":"michel@redhat.com","username":"mpeterson"},"date":"2018-10-17 10:33:33.000000000","message":"Patch Set 2: Code-Review+1\n\nLGTM","accounts_in_message":[],"_revision_number":2},{"id":"9b5481e6bb1dcff790ac54a8588da8a6b46228dc","author":{"_account_id":16376,"name":"Intel NFV CI","email":"openstack-nfv-ci@intel.com","username":"intel-nfv-ci","tags":["SERVICE_USER"]},"date":"2018-10-17 11:34:46.000000000","message":"Patch Set 2:\n\nBuild succeeded (check pipeline).\n\n- tempest-dsvm-intel-nfv-xenial http://52.27.155.124/portland/2018-10-17/611248/2/check/tempest-dsvm-intel-nfv-xenial/96238ec : SUCCESS in 27m 39s (non-voting)\n- tempest-dsvm-ovsdpdk-nfv-networking-xenial http://52.27.155.124/portland/2018-10-17/611248/2/check/tempest-dsvm-ovsdpdk-nfv-networking-xenial/aede780 : SUCCESS in 56m 23s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"f2183057d367d7fdcf2e0695fed15fc56b4e5241","author":{"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},"date":"2018-10-17 11:44:28.000000000","message":"Patch Set 2:\n\nBuild succeeded. Test completed on IBM PowerKVM platform. For rechecking only on the IBM PowerKVM CI, add a review comment with pkvm: recheck. For contact and more information, see https://wiki.openstack.org/wiki/PowerKVM\n\n- tempest-dsvm-full-xenial https://dal05.objectstorage.softlayer.net/v1/AUTH_3d8e6ecb-f597-448c-8ec2-164e9f710dd6/pkvmci/devstack/48/611248/2/check/tempest-dsvm-full-xenial/4d48fbb/ : SUCCESS in 1h 10m 47s","accounts_in_message":[],"_revision_number":2},{"id":"d4912383fcfc365e6c9f330e5c12fd1d8b77f466","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-10-17 12:49:39.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- legacy-devstack-dsvm-updown http://logs.openstack.org/48/611248/2/check/legacy-devstack-dsvm-updown/b3aef18/ : SUCCESS in 31m 16s\n- tempest-full http://logs.openstack.org/48/611248/2/check/tempest-full/bbf01b6/ : SUCCESS in 1h 24m 19s\n- neutron-grenade http://logs.openstack.org/48/611248/2/check/neutron-grenade/48d28ec/ : SUCCESS in 53m 22s\n- tempest-full-py3 http://logs.openstack.org/48/611248/2/check/tempest-full-py3/9301475/ : SUCCESS in 1h 30m 24s\n- openstack-tox-docs http://logs.openstack.org/48/611248/2/check/openstack-tox-docs/8a0e8e3/html/ : SUCCESS in 4m 56s\n- devstack http://logs.openstack.org/48/611248/2/check/devstack/0a84142/ : SUCCESS in 22m 21s\n- devstack-platform-centos-7 http://logs.openstack.org/48/611248/2/check/devstack-platform-centos-7/fb131c8/ : SUCCESS in 1h 33m 47s (non-voting)\n- devstack-platform-opensuse-150 http://logs.openstack.org/48/611248/2/check/devstack-platform-opensuse-150/ca51ca4/ : RETRY_LIMIT in 2m 18s (non-voting)\n- devstack-platform-opensuse-tumbleweed http://logs.openstack.org/48/611248/2/check/devstack-platform-opensuse-tumbleweed/bdc5b57/ : FAILURE in 1h 26m 14s (non-voting)\n- devstack-platform-fedora-latest http://logs.openstack.org/48/611248/2/check/devstack-platform-fedora-latest/76b6124/ : FAILURE in 1h 18m 06s (non-voting)\n- devstack-multinode http://logs.openstack.org/48/611248/2/check/devstack-multinode/294d4bb/ : SUCCESS in 31m 17s\n- devstack-unit-tests http://logs.openstack.org/48/611248/2/check/devstack-unit-tests/ee22034/ : SUCCESS in 3m 30s\n- openstack-tox-bashate http://logs.openstack.org/48/611248/2/check/openstack-tox-bashate/1a2b23c/ : SUCCESS in 4m 22s\n- ironic-tempest-dsvm-ipa-wholedisk-bios-agent_ipmitool-tinyipa http://logs.openstack.org/48/611248/2/check/ironic-tempest-dsvm-ipa-wholedisk-bios-agent_ipmitool-tinyipa/d3df944/ : SUCCESS in 47m 56s (non-voting)\n- swift-dsvm-functional http://logs.openstack.org/48/611248/2/check/swift-dsvm-functional/f7fd7ad/ : SUCCESS in 28m 29s (non-voting)\n- neutron-grenade-multinode http://logs.openstack.org/48/611248/2/check/neutron-grenade-multinode/156a910/ : SUCCESS in 1h 00m 06s\n- neutron-tempest-linuxbridge http://logs.openstack.org/48/611248/2/check/neutron-tempest-linuxbridge/1c184f2/ : SUCCESS in 1h 32m 29s\n- tempest-multinode-full http://logs.openstack.org/48/611248/2/check/tempest-multinode-full/57b9c69/ : SUCCESS in 1h 44m 15s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"04f1c853307a2e65766168b1a2b634e84d439fa9","author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"date":"2018-10-17 23:13:20.000000000","message":"Patch Set 2: Code-Review-1\n\nWe\u0027ve been back and forth with this before, at one point we had an environment variable then it was all removed with  I7c95927b9daad15766aac9d1276b10ca62efb24c.  I\u0027d be against changing the status quo of this now as I\u0027m sure plenty of people depend on these packages being installed.\n\nIf I\u0027m understanding, in the linked bug you\u0027re using extras to install ceilometer only in certain circumstances.  ISTM doing something like\n\n ceilometer[ceilometer] \u003e\u003d x.y.z\n\nin your test-requirements file would work?","accounts_in_message":[],"_revision_number":2},{"id":"0a9c1fd3e5ea74531f203caef45899e05df91da7","author":{"_account_id":21883,"name":"Manuel Buil","email":"mbuil@suse.com","username":"mbuil"},"date":"2018-10-24 13:56:25.000000000","message":"Patch Set 2:\n\n\u003e We\u0027ve been back and forth with this before, at one point we had an\n \u003e environment variable then it was all removed with \n \u003e I7c95927b9daad15766aac9d1276b10ca62efb24c.  I\u0027d be against changing\n \u003e the status quo of this now as I\u0027m sure plenty of people depend on\n \u003e these packages being installed.\n \u003e \n \u003e If I\u0027m understanding, in the linked bug you\u0027re using extras to\n \u003e install ceilometer only in certain circumstances.  ISTM doing\n \u003e something like\n \u003e \n \u003e ceilometer[ceilometer] \u003e\u003d x.y.z\n \u003e \n \u003e in your test-requirements file would work?\n\nI am not sure what you mean. Our problem is with the ceilometer dependency and not with a specific version of ceilometer. Ceilometer is just a package which we would need in very specific deployments.\n\nThe problem we have now is that we want some n-odl zuul jobs to test with ceilometer and some others without ceilometer. To do so, we added ceilometer into test-requirements.txt and then, in our tox.ini, specified whether we were testing with requirements.txt (no ceilometer) or test-requirements.txt (with ceilometer). However, that logic is wrong as devstack is always using test-requirements.txt","accounts_in_message":[],"_revision_number":2},{"id":"736af1cc563c8f190b1eeb4aa2d9837ddb114bee","author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"date":"2018-10-24 20:44:32.000000000","message":"Patch Set 2:\n\n\u003e The problem we have now is that we want some n-odl zuul jobs to test with ceilometer and some others without ceilometer. \n\nI\u0027d suggest you could use \"extras\" [1] in the requirement file to achieve this; pip_install for example deals with this package[extras] format; there\u0027s a few examples if you use codesearch.openstack.org.\n\nfrom devstack pov, we decided with those prior changes that the only point of devstack is to install a testing environment, so we stuck with installing test-requirements.txt.  since people rely on this now, we can\u0027t just stop uninstalling their requirements.\n\n[1] https://pip.readthedocs.io/en/1.1/requirements.html#the-requirements-file-format","accounts_in_message":[],"_revision_number":2},{"id":"2a0f858948004926ac40ea435cf2239e7797d17e","author":{"_account_id":21883,"name":"Manuel Buil","email":"mbuil@suse.com","username":"mbuil"},"date":"2018-10-25 07:39:23.000000000","message":"Patch Set 2:\n\n\u003e \u003e The problem we have now is that we want some n-odl zuul jobs to\n \u003e test with ceilometer and some others without ceilometer.\n \u003e \n \u003e I\u0027d suggest you could use \"extras\" [1] in the requirement file to\n \u003e achieve this; pip_install for example deals with this\n \u003e package[extras] format; there\u0027s a few examples if you use\n \u003e codesearch.openstack.org.\n \u003e \n \u003e from devstack pov, we decided with those prior changes that the\n \u003e only point of devstack is to install a testing environment, so we\n \u003e stuck with installing test-requirements.txt.  since people rely on\n \u003e this now, we can\u0027t just stop uninstalling their requirements.\n \u003e \n \u003e [1] https://pip.readthedocs.io/en/1.1/requirements.html#the-requirements-file-format\n\nI don\u0027t see how that would work because the extras in the requirements would work if we were installing ceilometer as part of another component or?\n\ncomponentX[ceilometer]\n\nThen we would just remove ceilometer from the squared brackets and that would install componentX without ceilometer.","accounts_in_message":[],"_revision_number":2},{"id":"6da61740dbf8909de642d5ac28a56d8c13a533cb","tag":"autogenerated:gerrit:abandon","author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"date":"2022-05-25 07:40:42.000000000","message":"Abandoned\n\nThis is 4 years old, so likely no longer relevant. Feel free to reopen if still needed.","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"6327a6fe40888429d6ccf79f741ee2cd17de6728","revisions":{"feb1aa1480a6ee73be382785767546606a42e4f6":{"kind":"REWORK","_number":1,"created":"2018-10-17 06:57:32.000000000","uploader":{"_account_id":21883,"name":"Manuel Buil","email":"mbuil@suse.com","username":"mbuil"},"ref":"refs/changes/48/611248/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/48/611248/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/48/611248/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/48/611248/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/48/611248/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/48/611248/1"}}},"commit":{"parents":[{"commit":"86db593653f6252331e42244aa11b59b06bde63f","subject":"Merge \"Don\u0027t use ipv6 for DNS SAN fields with python3\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/86db593653f6252331e42244aa11b59b06bde63f"}]}],"author":{"name":"Manuel Buil","email":"mbuil@suse.com","date":"2018-10-17 06:55:28.000000000","tz":120},"committer":{"name":"Manuel Buil","email":"mbuil@suse.com","date":"2018-10-17 06:59:32.000000000","tz":120},"subject":"Do not install test-requirements.txt dependencies","message":"Do not install test-requirements.txt dependencies\n\nEach project should decide whether test-requirements.txt dependencies\nshould be used or not. Right now, they are always used and that creates\nproblems for example in this patch:\n\nhttps://review.openstack.org/#/c/602622/\n\nwhere we don\u0027t want to install dependencies of test-requirements.txt for\nthe functional tests but we are forced to\n\nChange-Id: I35dd53c3b3dd0254cb1fb3b2af74fb4346dd1251\nSigned-off-by: Manuel Buil \u003cmbuil@suse.com\u003e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/feb1aa1480a6ee73be382785767546606a42e4f6"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/feb1aa1480a6ee73be382785767546606a42e4f6"}]},"branch":"refs/heads/master"},"6327a6fe40888429d6ccf79f741ee2cd17de6728":{"kind":"REWORK","_number":2,"created":"2018-10-17 07:53:25.000000000","uploader":{"_account_id":21883,"name":"Manuel Buil","email":"mbuil@suse.com","username":"mbuil"},"ref":"refs/changes/48/611248/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/48/611248/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/48/611248/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/48/611248/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/48/611248/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/48/611248/2"}}},"commit":{"parents":[{"commit":"86db593653f6252331e42244aa11b59b06bde63f","subject":"Merge \"Don\u0027t use ipv6 for DNS SAN fields with python3\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/86db593653f6252331e42244aa11b59b06bde63f"}]}],"author":{"name":"Manuel Buil","email":"mbuil@suse.com","date":"2018-10-17 06:55:28.000000000","tz":120},"committer":{"name":"Manuel Buil","email":"mbuil@suse.com","date":"2018-10-17 07:54:55.000000000","tz":120},"subject":"Do not install test-requirements.txt dependencies","message":"Do not install test-requirements.txt dependencies\n\nEach project should decide whether test-requirements.txt dependencies\nshould be used or not. Right now, they are always used and that creates\nproblems for example in this patch:\n\nhttps://review.openstack.org/#/c/602622/\n\nwhere we don\u0027t want to install dependencies of test-requirements.txt for\nthe functional tests but we are forced to\n\nChange-Id: I35dd53c3b3dd0254cb1fb3b2af74fb4346dd1251\nSigned-off-by: Manuel Buil \u003cmbuil@suse.com\u003e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/6327a6fe40888429d6ccf79f741ee2cd17de6728"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/6327a6fe40888429d6ccf79f741ee2cd17de6728"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}