)]}'
{"id":"x%2Fpackstack~66169","triplet_id":"x%2Fpackstack~master~I89a7de13724b5280789ffd4f4b09628001405de4","project":"x/packstack","branch":"master","topic":"qpid-ssl","hashtags":[],"change_id":"I89a7de13724b5280789ffd4f4b09628001405de4","subject":"Fixes qpid SSL installation errors","status":"MERGED","created":"2014-01-12 05:38:20.000000000","updated":"2014-01-13 16:26:14.000000000","submitted":"2014-01-13 16:26:14.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":1,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"6a86d1a7302817801be2ba48e3fbb97debd8781f","_number":66169,"virtual_id_number":66169,"owner":{"_account_id":7462,"name":"Ivan Chavero","email":"ichavero@redhat.com","username":"ichavero"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":2,"date":"2014-01-13 16:26:14.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":8083,"name":"Francesco Vollero","email":"fvollero@redhat.com","username":"fvollero"},{"value":0,"_account_id":7462,"name":"Ivan Chavero","email":"ichavero@redhat.com","username":"ichavero"},{"value":0,"_account_id":5241,"name":"Martin Magr","email":"mmagr@redhat.com","username":"mmagr"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":5241,"name":"Martin Magr","email":"mmagr@redhat.com","username":"mmagr"},"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":8083,"name":"Francesco Vollero","email":"fvollero@redhat.com","username":"fvollero"},{"value":0,"_account_id":7462,"name":"Ivan Chavero","email":"ichavero@redhat.com","username":"ichavero"},{"value":2,"date":"2014-01-13 16:25:28.000000000","_account_id":5241,"name":"Martin Magr","email":"mmagr@redhat.com","username":"mmagr"}],"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":{"approved":{"_account_id":5241,"name":"Martin Magr","email":"mmagr@redhat.com","username":"mmagr"},"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":8083,"name":"Francesco Vollero","email":"fvollero@redhat.com","username":"fvollero"},{"value":0,"_account_id":7462,"name":"Ivan Chavero","email":"ichavero@redhat.com","username":"ichavero"},{"value":1,"date":"2014-01-13 16:25:28.000000000","_account_id":5241,"name":"Martin Magr","email":"mmagr@redhat.com","username":"mmagr"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":5241,"name":"Martin Magr","email":"mmagr@redhat.com","username":"mmagr"},{"_account_id":7462,"name":"Ivan Chavero","email":"ichavero@redhat.com","username":"ichavero"},{"_account_id":8083,"name":"Francesco Vollero","email":"fvollero@redhat.com","username":"fvollero"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2014-01-12 05:41:48.000000000","updated_by":{"_account_id":8083,"name":"Francesco Vollero","email":"fvollero@redhat.com","username":"fvollero"},"reviewer":{"_account_id":8083,"name":"Francesco Vollero","email":"fvollero@redhat.com","username":"fvollero"},"state":"REVIEWER"},{"updated":"2014-01-13 16:25:28.000000000","updated_by":{"_account_id":5241,"name":"Martin Magr","email":"mmagr@redhat.com","username":"mmagr"},"reviewer":{"_account_id":5241,"name":"Martin Magr","email":"mmagr@redhat.com","username":"mmagr"},"state":"REVIEWER"},{"updated":"2014-01-13 16:26:14.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"c0eb06c3b5a273e063a5f6f3c9d28c32a5c8630b","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-01-12 05:39:34.000000000","message":"Patch Set 1: Works for me\n\nBuild succeeded.\n\n- gate-packstack-pep8 http://logs.openstack.org/69/66169/1/check/gate-packstack-pep8/f4dbf4c : SUCCESS in 1m 09s\n- gate-packstack-python26 http://logs.openstack.org/69/66169/1/check/gate-packstack-python26/3a6a927 : SUCCESS in 41s\n- gate-packstack-python27 http://logs.openstack.org/69/66169/1/check/gate-packstack-python27/14415b9 : SUCCESS in 57s\n","accounts_in_message":[],"_revision_number":1},{"id":"575c644294ae09db7f946101a2c0204dab2f2c7b","author":{"_account_id":5241,"name":"Martin Magr","email":"mmagr@redhat.com","username":"mmagr"},"date":"2014-01-12 17:35:56.000000000","message":"Patch Set 1: I would prefer that you didn\u0027t merge this\n\n(1 inline comment)\n\nPlease add absolute path","accounts_in_message":[],"_revision_number":1},{"id":"ba1963902ee7fafabbcb408c7a1593ee23f11150","author":{"_account_id":5241,"name":"Martin Magr","email":"mmagr@redhat.com","username":"mmagr"},"date":"2014-01-12 18:00:52.000000000","message":"Patch Set 1:\n\nBTW onlyif will be always true, since ps will catch also the grep command:\n\n    [para@virtual-rhel ~]$ /bin/ps fax | grep qpidd\n    31801 pts/0    S+     0:00              \\_ grep qpidd\n     1593 ?        Ssl    0:48 /usr/sbin/qpidd --data-dir /var/lib/qpidd --daemon\n    [para@virtual-rhel ~]$ echo $?\n    0\n    [para@virtual-rhel ~]$ sudo /sbin/service qpidd stop\n    Stopping Qpid AMQP daemon:                                 [  OK  ]\n    [para@virtual-rhel ~]$ /bin/ps fax | grep qpidd\n    31941 pts/0    S+     0:00              \\_ grep qpidd\n    [para@virtual-rhel ~]$ echo $?\n    0\n    [para@virtual-rhel ~]$ \n\nSo you will have to create some other check, like service status. Works both on RHEL and Fedora:\n\nFedora:\n    [para@elysium packstack-junk]$ sudo /sbin/service  qpidd status\n    Redirecting to /bin/systemctl status  qpidd.service\n    qpidd.service - An AMQP message broker daemon.\n       Loaded: loaded (/usr/lib/systemd/system/qpidd.service; enabled)\n       Active: active (running) since Thu 2014-01-09 10:06:46 CET; 3 days ago\n         Docs: man:qpidd(1)\n               http://qpid.apache.org/\n     Main PID: 528 (qpidd)\n       CGroup: /system.slice/qpidd.service\n               ??528 /usr/sbin/qpidd --config /etc/qpid/qpidd.conf\n\n    Jan 09 10:06:46 elysium.brq.redhat.com systemd[1]: Starting An AMQP message broker daemon....\n    Jan 09 10:06:46 elysium.brq.redhat.com systemd[1]: Started An AMQP message broker daemon..\n    Jan 09 10:06:54 elysium.brq.redhat.com qpidd[528]: 2014-01-09 10:06:54 [Security] notice SSL plugin not enabled, you must set ...le it.\n    Jan 09 10:06:57 elysium.brq.redhat.com qpidd[528]: 2014-01-09 10:06:57 [Store] notice Journal \"TplStore\": Created\n    Jan 09 10:06:57 elysium.brq.redhat.com qpidd[528]: 2014-01-09 10:06:57 [Store] notice Store module initialized; store-dir\u003d/var....qpidd\n    Jan 09 10:06:57 elysium.brq.redhat.com qpidd[528]: 2014-01-09 10:06:57 [Store] warning Message store plugin: No storage provid...lable.\n    Jan 09 10:06:57 elysium.brq.redhat.com qpidd[528]: 2014-01-09 10:06:57 [Network] notice Listening on TCP/TCP6 port 5672\n    Jan 09 10:06:57 elysium.brq.redhat.com qpidd[528]: 2014-01-09 10:06:57 [Broker] notice Broker running\n    Hint: Some lines were ellipsized, use -l to show in full.\n    [para@elysium packstack-junk]$ echo $?\n    0\n    [para@elysium packstack-junk]$ sudo /sbin/service  qpidd stop\n    Redirecting to /bin/systemctl stop  qpidd.service\n    [para@elysium packstack-junk]$ sudo /sbin/service  qpidd status\n    Redirecting to /bin/systemctl status  qpidd.service\n    qpidd.service - An AMQP message broker daemon.\n       Loaded: loaded (/usr/lib/systemd/system/qpidd.service; enabled)\n       Active: inactive (dead) since Sun 2014-01-12 18:58:59 CET; 6s ago\n         Docs: man:qpidd(1)\n               http://qpid.apache.org/\n     Main PID: 528 (code\u003dexited, status\u003d0/SUCCESS)\n\n    Jan 09 10:06:57 elysium.brq.redhat.com qpidd[528]: 2014-01-09 10:06:57 [Store] notice Journal \"TplStore\": Created\n    Jan 09 10:06:57 elysium.brq.redhat.com qpidd[528]: 2014-01-09 10:06:57 [Store] notice Store module initialized; store-dir\u003d/var....qpidd\n    Jan 09 10:06:57 elysium.brq.redhat.com qpidd[528]: 2014-01-09 10:06:57 [Store] warning Message store plugin: No storage provid...lable.\n    Jan 09 10:06:57 elysium.brq.redhat.com qpidd[528]: 2014-01-09 10:06:57 [Network] notice Listening on TCP/TCP6 port 5672\n    Jan 09 10:06:57 elysium.brq.redhat.com qpidd[528]: 2014-01-09 10:06:57 [Broker] notice Broker running\n    Jan 12 18:58:58 elysium.brq.redhat.com systemd[1]: Stopping An AMQP message broker daemon....\n    Jan 12 18:58:58 elysium.brq.redhat.com qpidd[528]: DIGEST-MD5 common mech free\n    Jan 12 18:58:58 elysium.brq.redhat.com qpidd[528]: 2014-01-12 18:58:58 [Broker] notice Shut down\n    Jan 12 18:58:59 elysium.brq.redhat.com qpidd[528]: 2014-01-12 18:58:59 [Store] notice Journal \"TplStore\": Destroyed\n    Jan 12 18:58:59 elysium.brq.redhat.com systemd[1]: Stopped An AMQP message broker daemon..\n    Hint: Some lines were ellipsized, use -l to show in full.\n    [para@elysium packstack-junk]$ echo $?\n    3\n    [para@elysium packstack-junk]$ \n\nRHEL:\n    [para@virtual-rhel ~]$ sudo /sbin/service  qpidd status\n    qpidd is stopped\n    [para@virtual-rhel ~]$ echo $?\n    3\n    [para@virtual-rhel ~]$ sudo /sbin/service  qpidd start\n    Starting Qpid AMQP daemon:                                 [  OK  ]\n    [para@virtual-rhel ~]$ sudo /sbin/service  qpidd status\n    qpidd (pid  1078) is running...\n    [para@virtual-rhel ~]$ echo $?\n    0\n    [para@virtual-rhel ~]$","accounts_in_message":[],"_revision_number":1},{"id":"249cd6596dbd221879474e4592c56da65194b823","author":{"_account_id":7462,"name":"Ivan Chavero","email":"ichavero@redhat.com","username":"ichavero"},"date":"2014-01-13 07:13:01.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"ed42085c0a117e0eb262ad43018d66123a6172a0","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-01-13 07:13:59.000000000","message":"Patch Set 2: Works for me\n\nBuild succeeded.\n\n- gate-packstack-pep8 http://logs.openstack.org/69/66169/2/check/gate-packstack-pep8/08b2d9d : SUCCESS in 36s\n- gate-packstack-python26 http://logs.openstack.org/69/66169/2/check/gate-packstack-python26/7eaa818 : SUCCESS in 30s\n- gate-packstack-python27 http://logs.openstack.org/69/66169/2/check/gate-packstack-python27/bbd73cf : SUCCESS in 54s\n","accounts_in_message":[],"_revision_number":2},{"id":"d5f32766cd5b0e6732bd89b3fbd5523b88a87b07","author":{"_account_id":7462,"name":"Ivan Chavero","email":"ichavero@redhat.com","username":"ichavero"},"date":"2014-01-13 15:12:37.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"55ef0d44d1c8e31ce75098a50aff6e0125292c82","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-01-13 15:16:43.000000000","message":"Patch Set 3: Works for me\n\nBuild succeeded.\n\n- gate-packstack-pep8 http://logs.openstack.org/69/66169/3/check/gate-packstack-pep8/b634e84 : SUCCESS in 28s\n- gate-packstack-python26 http://logs.openstack.org/69/66169/3/check/gate-packstack-python26/571bc64 : SUCCESS in 55s\n- gate-packstack-python27 http://logs.openstack.org/69/66169/3/check/gate-packstack-python27/14af25b : SUCCESS in 37s\n","accounts_in_message":[],"_revision_number":3},{"id":"6263f2b817f373fdd0a94d5cf17ac115d81669a8","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-01-13 16:25:28.000000000","message":"Patch Set 3: No score\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":3},{"id":"6a812b78d6d57bd94a080c8cff30f29bbaf57d0d","author":{"_account_id":5241,"name":"Martin Magr","email":"mmagr@redhat.com","username":"mmagr"},"date":"2014-01-13 16:25:28.000000000","message":"Patch Set 3: Looks good to me (core reviewer); Approved\n\n","accounts_in_message":[],"_revision_number":3},{"id":"e5b0c407a8c54a09b1138f9f48084e5a9bdd8d03","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-01-13 16:26:14.000000000","message":"Patch Set 3: Verified\n\nBuild succeeded.\n\n- gate-packstack-pep8 http://logs.openstack.org/69/66169/3/gate/gate-packstack-pep8/982222c : SUCCESS in 37s\n- gate-packstack-python26 http://logs.openstack.org/69/66169/3/gate/gate-packstack-python26/2925b05 : SUCCESS in 42s\n- gate-packstack-python27 http://logs.openstack.org/69/66169/3/gate/gate-packstack-python27/09e6920 : SUCCESS in 26s\n","accounts_in_message":[],"_revision_number":3},{"id":"9c9e9760930960d3bde54777801067ca5148ab3f","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-01-13 16:26:14.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":3}],"current_revision_number":3,"current_revision":"776f027d5ca190510063923d8646263b44a078de","revisions":{"f994061187b8687824a96e1dfebe0a66627f9404":{"kind":"REWORK","_number":1,"created":"2014-01-12 05:38:20.000000000","uploader":{"_account_id":7462,"name":"Ivan Chavero","email":"ichavero@redhat.com","username":"ichavero"},"ref":"refs/changes/69/66169/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/x/packstack","ref":"refs/changes/69/66169/1","commands":{"Checkout":"git fetch https://review.opendev.org/x/packstack refs/changes/69/66169/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/x/packstack refs/changes/69/66169/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/x/packstack refs/changes/69/66169/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/x/packstack refs/changes/69/66169/1"}}},"commit":{"parents":[{"commit":"5950b5c6efb42c54b40fbe3a003f7d995a198f15","subject":"Merge \"[Keystone][Firewall] Open Keystone port for ALL\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/x/packstack/commit/5950b5c6efb42c54b40fbe3a003f7d995a198f15"}]}],"author":{"name":"Ivan Chavero","email":"ichavero@redhat.com","date":"2014-01-12 05:33:55.000000000","tz":-420},"committer":{"name":"Ivan Chavero","email":"ichavero@redhat.com","date":"2014-01-12 05:33:55.000000000","tz":-420},"subject":"Fixes qpid SSL installation errors","message":"Fixes qpid SSL installation errors\n\nWhen you\u0027re running packstack with qpid SSL enabled\nenabled you get an error if qpid is not alreadt running\n\nChange-Id: I89a7de13724b5280789ffd4f4b09628001405de4\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/x/packstack/commit/f994061187b8687824a96e1dfebe0a66627f9404"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/x/packstack/commit/f994061187b8687824a96e1dfebe0a66627f9404"}]},"branch":"refs/heads/master"},"4c520c1fc0665cd17eaf1d7ae87678cd09864119":{"kind":"REWORK","_number":2,"created":"2014-01-13 07:13:01.000000000","uploader":{"_account_id":7462,"name":"Ivan Chavero","email":"ichavero@redhat.com","username":"ichavero"},"ref":"refs/changes/69/66169/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/x/packstack","ref":"refs/changes/69/66169/2","commands":{"Checkout":"git fetch https://review.opendev.org/x/packstack refs/changes/69/66169/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/x/packstack refs/changes/69/66169/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/x/packstack refs/changes/69/66169/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/x/packstack refs/changes/69/66169/2"}}},"commit":{"parents":[{"commit":"5950b5c6efb42c54b40fbe3a003f7d995a198f15","subject":"Merge \"[Keystone][Firewall] Open Keystone port for ALL\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/x/packstack/commit/5950b5c6efb42c54b40fbe3a003f7d995a198f15"}]}],"author":{"name":"Ivan Chavero","email":"ichavero@redhat.com","date":"2014-01-12 05:33:55.000000000","tz":-420},"committer":{"name":"Ivan Chavero","email":"ichavero@redhat.com","date":"2014-01-13 07:12:41.000000000","tz":-420},"subject":"Fixes qpid SSL installation errors","message":"Fixes qpid SSL installation errors\n\nWhen you\u0027re running packstack with qpid SSL enabled\nenabled you get an error if qpid is not alreadt running\n\nChange-Id: I89a7de13724b5280789ffd4f4b09628001405de4\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/x/packstack/commit/4c520c1fc0665cd17eaf1d7ae87678cd09864119"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/x/packstack/commit/4c520c1fc0665cd17eaf1d7ae87678cd09864119"}]},"branch":"refs/heads/master"},"776f027d5ca190510063923d8646263b44a078de":{"kind":"REWORK","_number":3,"created":"2014-01-13 15:12:37.000000000","uploader":{"_account_id":7462,"name":"Ivan Chavero","email":"ichavero@redhat.com","username":"ichavero"},"ref":"refs/changes/69/66169/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/x/packstack","ref":"refs/changes/69/66169/3","commands":{"Checkout":"git fetch https://review.opendev.org/x/packstack refs/changes/69/66169/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/x/packstack refs/changes/69/66169/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/x/packstack refs/changes/69/66169/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/x/packstack refs/changes/69/66169/3"}}},"commit":{"parents":[{"commit":"5950b5c6efb42c54b40fbe3a003f7d995a198f15","subject":"Merge \"[Keystone][Firewall] Open Keystone port for ALL\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/x/packstack/commit/5950b5c6efb42c54b40fbe3a003f7d995a198f15"}]}],"author":{"name":"Ivan Chavero","email":"ichavero@redhat.com","date":"2014-01-12 05:33:55.000000000","tz":-420},"committer":{"name":"Ivan Chavero","email":"ichavero@redhat.com","date":"2014-01-13 15:11:06.000000000","tz":-420},"subject":"Fixes qpid SSL installation errors","message":"Fixes qpid SSL installation errors\n\nWhen you\u0027re running packstack with qpid SSL enabled\nenabled you get an error if qpid is not alreadt running\n\nWhen qpid authentication is enabled the file\n/var/lib/qpidd/qpidd.sasldb is created with the wrong\npermissions.\n\nFixes: rhbz#1052163\n\nChange-Id: I89a7de13724b5280789ffd4f4b09628001405de4\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/x/packstack/commit/776f027d5ca190510063923d8646263b44a078de"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/x/packstack/commit/776f027d5ca190510063923d8646263b44a078de"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}