)]}'
{"id":"openstack%2Fdevstack~163741","triplet_id":"openstack%2Fdevstack~master~Ia37d4ee6460c191c283a5df0a741bdb67485e2aa","project":"openstack/devstack","branch":"master","topic":"f22-def-dnf","hashtags":[],"change_id":"Ia37d4ee6460c191c283a5df0a741bdb67485e2aa","subject":"f22 default to dnf instead of yum","status":"ABANDONED","created":"2015-03-12 09:38:02.000000000","updated":"2017-03-06 13:40:03.000000000","total_comment_count":2,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"690a29e227251930389321fd60d76772fda30900","_number":163741,"virtual_id_number":163741,"owner":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":1,"date":"2015-03-13 08:31:18.000000000","_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"_account_id":6962,"name":"Kashyap Chamarthy","email":"kchamart@redhat.com","username":"kashyapc"},{"value":1,"date":"2015-03-13 08:52:03.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"date":"2015-03-26 08:46:01.000000000","_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"_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":{"disliked":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"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":6962,"name":"Kashyap Chamarthy","email":"kchamart@redhat.com","username":"kashyapc"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"date":"2015-03-26 13:23:35.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},{"value":1,"date":"2015-03-13 13:07:09.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},{"value":-1,"date":"2015-03-20 13:26:35.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"value":-1,"date":"2015-03-16 04:19:54.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":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"_account_id":6962,"name":"Kashyap Chamarthy","email":"kchamart@redhat.com","username":"kashyapc"},{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"permitted_voting_range":{"min":-1,"max":0},"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"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":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":6962,"name":"Kashyap Chamarthy","email":"kchamart@redhat.com","username":"kashyapc"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"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":3,"name":"Jenkins","username":"jenkins"},{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},{"_account_id":6962,"name":"Kashyap Chamarthy","email":"kchamart@redhat.com","username":"kashyapc"},{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2015-03-12 09:55:25.000000000","updated_by":{"_account_id":6962,"name":"Kashyap Chamarthy","email":"kchamart@redhat.com","username":"kashyapc"},"reviewer":{"_account_id":6962,"name":"Kashyap Chamarthy","email":"kchamart@redhat.com","username":"kashyapc"},"state":"REVIEWER"},{"updated":"2015-03-13 08:31:18.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-13 08:52:03.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"},{"updated":"2015-03-13 13:07:09.000000000","updated_by":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"reviewer":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"state":"REVIEWER"},{"updated":"2015-03-16 04:19:54.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"},{"updated":"2015-03-20 13:26:35.000000000","updated_by":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"reviewer":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"state":"REVIEWER"},{"updated":"2015-03-26 13:23:35.000000000","updated_by":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"reviewer":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"state":"REVIEWER"}],"messages":[{"id":"972effe6f839eb1fd7c02396ad80460b1d936857","author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"date":"2015-03-12 09:38:02.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"cbcee3cf4c2265b99c4dd173ab89b64adf7c8fa0","author":{"_account_id":6962,"name":"Kashyap Chamarthy","email":"kchamart@redhat.com","username":"kashyapc"},"date":"2015-03-12 09:55:25.000000000","message":"Patch Set 1: Code-Review+1\n\nAlso see the speedups (10 minutes of saved time) with `dnf` here from this patch:\n\n  https://review.openstack.org/#/c/140370/","accounts_in_message":[],"_revision_number":1},{"id":"1aa64eaf2de6e18b72b79b0481ebd72e139455bc","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-03-12 10:39:12.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- check-tempest-dsvm-full http://logs.openstack.org/41/163741/1/check/check-tempest-dsvm-full/2526e8f/ : SUCCESS in 39m 42s\n- check-tempest-dsvm-postgres-full http://logs.openstack.org/41/163741/1/check/check-tempest-dsvm-postgres-full/61f9dae/ : SUCCESS in 38m 19s\n- check-tempest-dsvm-neutron-full http://logs.openstack.org/41/163741/1/check/check-tempest-dsvm-neutron-full/4c2b131/ : SUCCESS in 53m 40s\n- check-grenade-dsvm http://logs.openstack.org/41/163741/1/check/check-grenade-dsvm/979165a/ : SUCCESS in 39m 01s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/41/163741/1/check/gate-tempest-dsvm-large-ops/08705ad/ : SUCCESS in 24m 03s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/41/163741/1/check/gate-tempest-dsvm-neutron-large-ops/45f45de/ : SUCCESS in 21m 28s\n- gate-devstack-bashate http://logs.openstack.org/41/163741/1/check/gate-devstack-bashate/422185b/ : SUCCESS in 31s\n- gate-devstack-unit-tests http://logs.openstack.org/41/163741/1/check/gate-devstack-unit-tests/10c8325/ : SUCCESS in 1m 55s\n- check-devstack-dsvm-cells http://logs.openstack.org/41/163741/1/check/check-devstack-dsvm-cells/d09c337/ : SUCCESS in 20m 26s\n- check-grenade-dsvm-partial-ncpu http://logs.openstack.org/41/163741/1/check/check-grenade-dsvm-partial-ncpu/1a12659/ : SUCCESS in 47m 22s\n- check-swift-dsvm-functional http://logs.openstack.org/41/163741/1/check/check-swift-dsvm-functional/7701d07/ : SUCCESS in 25m 54s\n- check-tempest-dsvm-ironic-pxe_ssh http://logs.openstack.org/41/163741/1/check/check-tempest-dsvm-ironic-pxe_ssh/2a6d83b/ : SUCCESS in 39m 02s\n- check-devstack-dsvm-updown http://logs.openstack.org/41/163741/1/check/check-devstack-dsvm-updown/537e656/ : SUCCESS in 19m 34s\n- check-tempest-dsvm-f21 http://logs.openstack.org/41/163741/1/check/check-tempest-dsvm-f21/4a8c816/ : FAILURE in 39m 38s (non-voting)\n- check-tempest-dsvm-centos7 http://logs.openstack.org/41/163741/1/check/check-tempest-dsvm-centos7/8ca6714/ : SUCCESS in 1h 01m 05s (non-voting)\n- gate-devstack-publish-docs http://docs-draft.openstack.org/41/163741/1/check/gate-devstack-publish-docs/2df4dc2//doc/build/html/ : SUCCESS in 1m 40s\n- check-grenade-dsvm-neutron http://logs.openstack.org/41/163741/1/check/check-grenade-dsvm-neutron/5cf247d/ : SUCCESS in 48m 26s","accounts_in_message":[],"_revision_number":1},{"id":"88646d21bdc34ef6cd0eb4b1e460cf8ff4c9f75a","author":{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},"date":"2015-03-12 10:41:23.000000000","message":"Patch Set 1: Verified+1\n\nPassed using XenAPI driver with XenServer 6.2: Logs at http://dd6b71949550285df7dc-dda4e480e005aaa13ec303551d2d8155.r49.cf1.rackcdn.com/41/163741/1/12890/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":1},{"id":"24f67b350924bff6816fa0976f0b66382f3d6241","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2015-03-12 17:08:13.000000000","message":"Patch Set 1: Code-Review-1\n\n(1 comment)\n\nAn additional comment change requested\n\nAlso, dnf \u003d\u003d \u0027did not finish\u0027 to my racing background... ;)","accounts_in_message":[],"_revision_number":1},{"id":"9f86b27501374ff9fe3d085dbf2826d8fc3c19e7","author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"date":"2015-03-13 07:10:36.000000000","message":"Patch Set 1:\n\nThere are other funny acronym understanding from the f20 era.:  \nhttp://worldofgnome.org/should-we-replace-yum-with-dnf-in-fedora-20/","accounts_in_message":[],"_revision_number":1},{"id":"16df0bb47beab542804a941a681f930bb7dcdda1","author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"date":"2015-03-13 07:29:13.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"6850414f515734828fd5974a72400efed4bc9642","author":{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},"date":"2015-03-13 08:31:18.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/41/163741/2/13098/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":"b96c42e2538913834c83e19629508a6b0e35ce04","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-03-13 08:52:03.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- check-tempest-dsvm-full http://logs.openstack.org/41/163741/2/check/check-tempest-dsvm-full/f44697c/ : SUCCESS in 43m 31s\n- check-tempest-dsvm-postgres-full http://logs.openstack.org/41/163741/2/check/check-tempest-dsvm-postgres-full/273f490/ : SUCCESS in 45m 22s\n- check-tempest-dsvm-neutron-full http://logs.openstack.org/41/163741/2/check/check-tempest-dsvm-neutron-full/af56b10/ : SUCCESS in 1h 00m 41s\n- check-grenade-dsvm http://logs.openstack.org/41/163741/2/check/check-grenade-dsvm/a780cfe/ : SUCCESS in 41m 41s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/41/163741/2/check/gate-tempest-dsvm-large-ops/fda08f7/ : SUCCESS in 27m 30s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/41/163741/2/check/gate-tempest-dsvm-neutron-large-ops/ed663dd/ : SUCCESS in 25m 24s\n- gate-devstack-bashate http://logs.openstack.org/41/163741/2/check/gate-devstack-bashate/145b5c5/ : SUCCESS in 12s\n- gate-devstack-unit-tests http://logs.openstack.org/41/163741/2/check/gate-devstack-unit-tests/466dc6f/ : SUCCESS in 12s\n- check-devstack-dsvm-cells http://logs.openstack.org/41/163741/2/check/check-devstack-dsvm-cells/160fb3d/ : SUCCESS in 19m 15s\n- check-grenade-dsvm-partial-ncpu http://logs.openstack.org/41/163741/2/check/check-grenade-dsvm-partial-ncpu/ab00aed/ : SUCCESS in 44m 18s\n- check-swift-dsvm-functional http://logs.openstack.org/41/163741/2/check/check-swift-dsvm-functional/084258d/ : SUCCESS in 21m 44s\n- check-tempest-dsvm-ironic-pxe_ssh http://logs.openstack.org/41/163741/2/check/check-tempest-dsvm-ironic-pxe_ssh/a8f7f2c/ : SUCCESS in 37m 18s\n- check-devstack-dsvm-updown http://logs.openstack.org/41/163741/2/check/check-devstack-dsvm-updown/d25ea9e/ : SUCCESS in 18m 10s\n- check-tempest-dsvm-f21 http://logs.openstack.org/41/163741/2/check/check-tempest-dsvm-f21/02a1b69/ : SUCCESS in 38m 44s (non-voting)\n- check-tempest-dsvm-centos7 http://logs.openstack.org/41/163741/2/check/check-tempest-dsvm-centos7/cb6fcef/ : SUCCESS in 55m 06s (non-voting)\n- gate-devstack-publish-docs http://docs-draft.openstack.org/41/163741/2/check/gate-devstack-publish-docs/ce54310//doc/build/html/ : SUCCESS in 1m 41s\n- check-grenade-dsvm-neutron http://logs.openstack.org/41/163741/2/check/check-grenade-dsvm-neutron/13e90cd/ : SUCCESS in 58m 39s","accounts_in_message":[],"_revision_number":2},{"id":"596d47b01d55f396a15b8313160877f3e8aa9279","author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"date":"2015-03-13 13:07:09.000000000","message":"Patch Set 2: Code-Review+1","accounts_in_message":[],"_revision_number":2},{"id":"04832d7d27668f8228e53377e781939ad60d5a83","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2015-03-13 15:51:07.000000000","message":"Patch Set 2: Code-Review+2\n\nThanks","accounts_in_message":[],"_revision_number":2},{"id":"cc1fcc0529850c6304587276542117353ded6ee7","author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"date":"2015-03-16 04:19:54.000000000","message":"Patch Set 2: Code-Review-1\n\nurgh; functions-common gets included stand-alone and still defaults itself to \"yum\" if YUM isn\u0027t set.  \n\nI think this should definitely be retained in stackrc for discoverability, but maybe the logic should be replicated at the top of functions-common to set YUM correctly in the stand-alone case too","accounts_in_message":[],"_revision_number":2},{"id":"5192c81fee6ff737cc96daad7468a370e3293106","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2015-03-20 13:26:35.000000000","message":"Patch Set 2: Code-Review-1\n\nnot everything needs to be in stackrc, so I think this should be in functions-common, otherwise things like grenade are going to revert. Put it in docs if we need it to be discoverable.","accounts_in_message":[],"_revision_number":2},{"id":"4075f5f6d03e229efe11d88fa1f71f1996066b70","author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"date":"2015-03-26 08:46:01.000000000","message":"Patch Set 2:\n\nI am think about removing the YUM variable and have the function to always use dnf when it is available.","accounts_in_message":[],"_revision_number":2},{"id":"99445a270970b9a822daa9a055366e44bc6dcf9d","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2015-03-26 13:23:35.000000000","message":"Patch Set 2: -Code-Review\n\nAttila, if you think not being able to override that is not a problem I like the idea to remove the YUM var and force the selection based on release.","accounts_in_message":[],"_revision_number":2},{"id":"238d8b096ac6c32e857baad9623162f0f3da5393","author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"date":"2015-05-28 13:47:30.000000000","message":"Patch Set 2:\n\nThe f22 final release just print warning when you try to use yum and actually uses dnf.\n\nSo this change is not important anymore.","accounts_in_message":[],"_revision_number":2},{"id":"e6ee42548e658249faf8cf3a6e6c851376431d33","author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"date":"2015-05-28 23:04:56.000000000","message":"Patch Set 2:\n\nyeah, i think we can default it on for f22 but first i\u0027d rather get f22 working in CI (working on that, got the diskimage-builder part ready for nodepool).  we can do this after that to ensure we only break one thing at once :)","accounts_in_message":[],"_revision_number":2},{"id":"073265f14020b18866fb62517b64c940faa7e67a","author":{"_account_id":17377,"name":"Federico Ressi","email":"federico.ressi@intel.com","username":"fressi"},"date":"2015-09-11 22:43:31.000000000","message":"Patch Set 2: Code-Review-1\n\n(1 comment)\n\nI suggest a more generic approach. Please look at my comment.\n\nRegards","accounts_in_message":[],"_revision_number":2},{"id":"33755e19fdf6ae663575fd264da4754577c37ab7","author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"date":"2016-01-20 11:53:46.000000000","message":"Patch Set 2:\n\nGetOSVersion is executed only when it\u0027s needed,\ndevstack should not issue any install command before GetOSVersion  run at least once.\n\nWhat about adding an another variable maintained by GetOSVersion ?\nos_PKGMGR , possible values yast,apt-get,yum,dnf ...","accounts_in_message":[],"_revision_number":2},{"id":"d9cdf982fb1ea5f5fdde9eae26be554f25ca99e5","author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"date":"2016-01-20 12:42:35.000000000","message":"Patch Set 2:\n\ns/yast/zypper/","accounts_in_message":[],"_revision_number":2},{"id":"64e2078cbb7e4e92e9ef305af54d3678d4120cd3","author":{"_account_id":17377,"name":"Federico Ressi","email":"federico.ressi@intel.com","username":"fressi"},"date":"2016-03-10 12:05:11.000000000","message":"Patch Set 2:\n\nvarious package managers (yum, apt-get) have different parameters and interfaces and they don\u0027t exactely overlaps. Therefore we should focus on having functions to abstract them (as now) instead of using variables as you proposed.\nBy the way for the specific case of Fedora / Red Hat it is fine to use a variable as you proposed because DNF is expected to be backward compatible with YUM.\nYou could hava a variable called YUM that in the case of Fedora 22+ it is set to \u0027dnf\u0027 instead of \u0027yum\u0027. As I said before I would avoid looking at the specific version (22, 23 ,etc.) because are informations binded to a specific vendor release. You should instead look at if dnf is installed and working. You would have a code easier to reuse and independently from Red Hat release strategies.\nPlease also consider that dnf could be backported to older distributions and it should add reasonable speed improvements over yum.","accounts_in_message":[],"_revision_number":2},{"id":"64aac010dcbb71f22eec64fdc5155e6da1dd3df1","author":{"_account_id":17377,"name":"Federico Ressi","email":"federico.ressi@intel.com","username":"fressi"},"date":"2016-11-02 16:44:42.000000000","message":"Removed the following votes:\n\n* Code-Review-1 by Federico Ressi \u003cfederico.ressi@intel.com\u003e\n","accounts_in_message":[],"_revision_number":2},{"id":"96d6497a32de4aa804acd136a23cff3e9cfaf7a2","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2017-03-06 13:40:03.000000000","message":"Abandoned","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"3dd4f12b1051dbbb40e0019b7f5510d3e47e4472","revisions":{"6e489b1f318c994293978d90e949698d8cde3bf8":{"kind":"REWORK","_number":1,"created":"2015-03-12 09:38:02.000000000","uploader":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"ref":"refs/changes/41/163741/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/41/163741/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/41/163741/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/41/163741/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/41/163741/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/41/163741/1"}}},"commit":{"parents":[{"commit":"df16842d046fc5dc7bd3543158e40acdbffbfd41","subject":"Merge \"Check for new versions of get-pip.py\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/df16842d046fc5dc7bd3543158e40acdbffbfd41"}]}],"author":{"name":"Attila Fazekas","email":"afazekas@redhat.com","date":"2015-03-12 09:36:50.000000000","tz":60},"committer":{"name":"Attila Fazekas","email":"afazekas@redhat.com","date":"2015-03-12 09:36:50.000000000","tz":60},"subject":"f22 default to dnf instead of yum","message":"f22 default to dnf instead of yum\n\nOn Fedora-22 the dnf is the default pakcage manager.\n\n[1] http://fedoraproject.org/wiki/Changes/ReplaceYumWithDNF\n\nChange-Id: Ia37d4ee6460c191c283a5df0a741bdb67485e2aa\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/6e489b1f318c994293978d90e949698d8cde3bf8"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/6e489b1f318c994293978d90e949698d8cde3bf8"}]},"branch":"refs/heads/master"},"3dd4f12b1051dbbb40e0019b7f5510d3e47e4472":{"kind":"REWORK","_number":2,"created":"2015-03-13 07:29:13.000000000","uploader":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"ref":"refs/changes/41/163741/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/41/163741/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/41/163741/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/41/163741/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/41/163741/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/41/163741/2"}}},"commit":{"parents":[{"commit":"68ed19b696c347d7e9d48b92b643419549bbd249","subject":"Merge \"install missing elasticsearch client\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/68ed19b696c347d7e9d48b92b643419549bbd249"}]}],"author":{"name":"Attila Fazekas","email":"afazekas@redhat.com","date":"2015-03-12 09:36:50.000000000","tz":60},"committer":{"name":"Attila Fazekas","email":"afazekas@redhat.com","date":"2015-03-13 07:28:59.000000000","tz":60},"subject":"f22 default to dnf instead of yum","message":"f22 default to dnf instead of yum\n\nOn Fedora-22 the dnf is the default pakcage manager.\n\n[1] http://fedoraproject.org/wiki/Changes/ReplaceYumWithDNF\n\nChange-Id: Ia37d4ee6460c191c283a5df0a741bdb67485e2aa\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/3dd4f12b1051dbbb40e0019b7f5510d3e47e4472"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/3dd4f12b1051dbbb40e0019b7f5510d3e47e4472"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}