[Engine-patches] Change in ovirt-iso-uploader[master]: error handling: Provide more informative error messages

2015-01-23 Thread jamills
James W. Mills has posted comments on this change. Change subject: error handling: Provide more informative error messages .. Patch Set 3: Verified+1 Code-Review+1 -- To view, visit http://gerrit.ovirt.org/36929 To unsubscribe

[Engine-patches] Change in ovirt-log-collector[master]: Collect engine-config and domain information

2014-12-17 Thread jamills
James W. Mills has posted comments on this change. Change subject: Collect engine-config and domain information .. Patch Set 2: Verified+1 -- To view, visit http://gerrit.ovirt.org/36116 To unsubscribe, visit http://gerrit.ovi

[Engine-patches] Change in ovirt-engine[ovirt-engine-3.4]: Fix typo and use default cert

2014-03-20 Thread jamills
James W. Mills has uploaded a new change for review. Change subject: Fix typo and use default cert .. Fix typo and use default cert This script has a typo that prevents successful execution. In addition, the usage section descr

[Engine-patches] Change in ovirt-engine[ovirt-engine-3.3.3]: Fix typo and use default cert

2014-03-20 Thread jamills
James W. Mills has abandoned this change. Change subject: Fix typo and use default cert .. Abandoned Abandoning, cherry-picked to ovirt-engine-3.4.0. This change cannot be cherry-picked to master, as the file no longer exists

[Engine-patches] Change in ovirt-engine[ovirt-engine-3.4.0]: Fix typo and use default cert

2014-03-20 Thread jamills
James W. Mills has uploaded a new change for review. Change subject: Fix typo and use default cert .. Fix typo and use default cert This script has a typo that prevents successful execution. In addition, the usage section descr

[Engine-patches] Change in ovirt-engine[ovirt-engine-3.3.3]: Fix typo and use default cert

2014-03-19 Thread jamills
James W. Mills has posted comments on this change. Change subject: Fix typo and use default cert .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.ovirt.org/25906 To unsubscribe, visit http://gerrit.ovirt.org/settings

[Engine-patches] Change in ovirt-engine[ovirt-engine-3.3.3]: Fix typo and use default cert

2014-03-19 Thread jamills
James W. Mills has uploaded a new change for review. Change subject: Fix typo and use default cert .. Fix typo and use default cert This script has a typo that prevents successful execution. In addition, the usage section descr

[Engine-patches] Change in ovirt-engine[ovirt-engine-3.3]: packaging: setup: database: support ':' within password

2014-02-13 Thread jamills
James W. Mills has posted comments on this change. Change subject: packaging: setup: database: support ':' within password .. Patch Set 2: Verified+1 I've verified that on a clean RHEV 3.2.5 to 3.3 upgrade with a ":" in the DB

[Engine-patches] Change in ovirt-engine[ovirt-engine-3.3]: packaging: setup: database: support ':' within password

2014-02-13 Thread jamills
James W. Mills has posted comments on this change. Change subject: packaging: setup: database: support ':' within password .. Patch Set 2: Alon, I would be happy to verify this works in RHEV (psql 8). I'll ost my results he

[Engine-patches] Change in ovirt-engine[ovirt-engine-3.3]: packaging: setup: database: support ':' within password

2014-02-13 Thread jamills
James W. Mills has posted comments on this change. Change subject: packaging: setup: database: support ':' within password .. Patch Set 2: Code-Review+1 -- To view, visit http://gerrit.ovirt.org/24460 To unsubscribe, visit htt

[Engine-patches] Change in ovirt-engine[ovirt-engine-3.3]: packaging: setup: legacy: db: parse pgpass file with escapes

2014-02-12 Thread jamills
James W. Mills has posted comments on this change. Change subject: packaging: setup: legacy: db: parse pgpass file with escapes .. Patch Set 4: Verified+1 Code-Review+1 -- To view, visit http://gerrit.ovirt.org/24407 To unsubs

[Engine-patches] Change in ovirt-engine[ovirt-engine-3.3]: packaging: setup: legacy: db: parse pgpass file with escapes

2014-02-12 Thread jamills
James W. Mills has posted comments on this change. Change subject: packaging: setup: legacy: db: parse pgpass file with escapes .. Patch Set 3: (2 comments) http://gerrit.ovirt.org/#/c/24407/3/packaging/setup/plugins/ovirt-eng

[Engine-patches] Change in ovirt-engine[ovirt-engine-3.3]: packaging: setup: legacy: db: parse pgpass file with escapes

2014-02-12 Thread jamills
James W. Mills has posted comments on this change. Change subject: packaging: setup: legacy: db: parse pgpass file with escapes .. Patch Set 3: Verified+1 Code-Review+1 The fact that the colon was used as a seperator *and* coul

[Engine-patches] Change in ovirt-log-collector[master]: Add a controlling TTY for ausearch

2013-09-25 Thread jamills
James W. Mills has posted comments on this change. Change subject: Add a controlling TTY for ausearch .. Patch Set 2: Verified+1 I've verified that the stderr "" is properly captured in the sosreport, as well as cases where au

[Engine-patches] Change in ovirt-log-collector[master]: Add a controlling TTY for ausearch

2013-09-23 Thread jamills
James W. Mills has posted comments on this change. Change subject: Add a controlling TTY for ausearch .. Patch Set 2: I have not seen any side effects from this change so far. I need to verify what the sosreport looks like wh

[Engine-patches] Change in ovirt-log-collector[master]: Add a controlling TTY for ausearch

2013-09-23 Thread jamills
James W. Mills has uploaded a new change for review. Change subject: Add a controlling TTY for ausearch .. Add a controlling TTY for ausearch The ausearch command required a controlling TTY in order to finish execution propely.

[Engine-patches] Change in ovirt-log-collector[master]: Add a controlling TTY for ausearch

2013-09-23 Thread jamills
James W. Mills has posted comments on this change. Change subject: Add a controlling TTY for ausearch .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.ovirt.org/19471 To unsubscribe, visit http://gerrit.ovirt.org/set