Change in vdsm[master]: fileUtils: Warn about callers of fileUtils.createdir()

2016-03-22 Thread nsoffer
Nir Soffer has submitted this change and it was merged.

Change subject: fileUtils: Warn about callers of fileUtils.createdir()
..


fileUtils: Warn about callers of fileUtils.createdir()

To ensure we will check the callers of fileUtils.createdir() it emits
now a warning when a directory exists.  The warning is available only
when python_warnings_enable option is enabled, so it does not effect
users.

Change-Id: I5c3e7e21367eb690d0c7a6c6f79c515b18a9fff0
Signed-off-by: Nir Soffer 
Reviewed-on: https://gerrit.ovirt.org/37003
Continuous-Integration: Jenkins CI
Reviewed-by: Adam Litke 
---
M vdsm/storage/fileUtils.py
1 file changed, 2 insertions(+), 0 deletions(-)

Approvals:
  Adam Litke: Looks good to me, approved
  Nir Soffer: Verified
  Jenkins CI: Passed CI tests



-- 
To view, visit https://gerrit.ovirt.org/37003
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I5c3e7e21367eb690d0c7a6c6f79c515b18a9fff0
Gerrit-PatchSet: 7
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer 
Gerrit-Reviewer: Adam Litke 
Gerrit-Reviewer: Ala Hino 
Gerrit-Reviewer: Allon Mureinik 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Federico Simoncelli 
Gerrit-Reviewer: Freddy Rolland 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer 
Gerrit-Reviewer: gerrit-hooks 
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: fileUtils: Warn about callers of fileUtils.createdir()

2016-03-22 Thread automation
gerrit-hooks has posted comments on this change.

Change subject: fileUtils: Warn about callers of fileUtils.createdir()
..


Patch Set 7:

* Update tracker: IGNORE, no Bug-Url found
* Set MODIFIED::IGNORE, no Bug-Url found.

-- 
To view, visit https://gerrit.ovirt.org/37003
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I5c3e7e21367eb690d0c7a6c6f79c515b18a9fff0
Gerrit-PatchSet: 7
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer 
Gerrit-Reviewer: Adam Litke 
Gerrit-Reviewer: Ala Hino 
Gerrit-Reviewer: Allon Mureinik 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Federico Simoncelli 
Gerrit-Reviewer: Freddy Rolland 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer 
Gerrit-Reviewer: gerrit-hooks 
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: fileUtils: Warn about callers of fileUtils.createdir()

2016-03-22 Thread alitke
Adam Litke has posted comments on this change.

Change subject: fileUtils: Warn about callers of fileUtils.createdir()
..


Patch Set 6: Code-Review+2

-- 
To view, visit https://gerrit.ovirt.org/37003
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I5c3e7e21367eb690d0c7a6c6f79c515b18a9fff0
Gerrit-PatchSet: 6
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer 
Gerrit-Reviewer: Adam Litke 
Gerrit-Reviewer: Ala Hino 
Gerrit-Reviewer: Allon Mureinik 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Federico Simoncelli 
Gerrit-Reviewer: Freddy Rolland 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer 
Gerrit-Reviewer: gerrit-hooks 
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: fileUtils: Warn about callers of fileUtils.createdir()

2016-03-22 Thread nsoffer
Nir Soffer has posted comments on this change.

Change subject: fileUtils: Warn about callers of fileUtils.createdir()
..


Patch Set 6: Verified+1

-- 
To view, visit https://gerrit.ovirt.org/37003
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I5c3e7e21367eb690d0c7a6c6f79c515b18a9fff0
Gerrit-PatchSet: 6
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer 
Gerrit-Reviewer: Adam Litke 
Gerrit-Reviewer: Ala Hino 
Gerrit-Reviewer: Allon Mureinik 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Federico Simoncelli 
Gerrit-Reviewer: Freddy Rolland 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer 
Gerrit-Reviewer: gerrit-hooks 
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: fileUtils: Warn about callers of fileUtils.createdir()

2016-03-22 Thread automation
gerrit-hooks has posted comments on this change.

Change subject: fileUtils: Warn about callers of fileUtils.createdir()
..


Patch Set 6:

* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' 
and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.6', 
'ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])

-- 
To view, visit https://gerrit.ovirt.org/37003
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I5c3e7e21367eb690d0c7a6c6f79c515b18a9fff0
Gerrit-PatchSet: 6
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer 
Gerrit-Reviewer: Adam Litke 
Gerrit-Reviewer: Ala Hino 
Gerrit-Reviewer: Allon Mureinik 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Federico Simoncelli 
Gerrit-Reviewer: Freddy Rolland 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer 
Gerrit-Reviewer: gerrit-hooks 
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: fileUtils: Warn about callers of fileUtils.createdir()

2016-03-22 Thread nsoffer
Nir Soffer has posted comments on this change.

Change subject: fileUtils: Warn about callers of fileUtils.createdir()
..


Patch Set 5:

Here is a log when warnings are enabled:

Note that we get a warning for the code calling this function - this code
should be inspected and probably fixed not to create unneeded directories.

# xzgrep 'UserWarning: Using existing direcotory:' /var/log/vdsm/vdsm.log.1.xz 
MainThread::WARNING::2016-03-22 
20:19:04,667::fileUtils::157::py.warnings::(createdir) 
/usr/share/vdsm/storage/hsm.py:344: UserWarning: Using existing direcotory: 
/rhev/data-center/mnt
jsonrpc.Executor/4::WARNING::2016-03-22 
20:19:42,160::fileUtils::157::py.warnings::(createdir) 
/usr/share/vdsm/storage/sp.py:1267: UserWarning: Using existing direcotory: 
/rhev/data-center/mnt/blockSD/6c77adb1-74fc-4fa9-a0ac-3b5a4b789318/dom_md
jsonrpc.Executor/4::WARNING::2016-03-22 
20:19:42,160::fileUtils::157::py.warnings::(createdir) 
/usr/share/vdsm/storage/sp.py:1268: UserWarning: Using existing direcotory: 
/rhev/data-center/mnt/blockSD/6c77adb1-74fc-4fa9-a0ac-3b5a4b789318/images
jsonrpc.Executor/4::WARNING::2016-03-22 
20:19:42,172::fileUtils::157::py.warnings::(createdir) 
/usr/share/vdsm/storage/blockSD.py:691: UserWarning: Using existing direcotory: 
/rhev/data-center/mnt/blockSD/6acaac02-7e7d-4321-8ba2-70e07d96581d/images
jsonrpc.Executor/4::WARNING::2016-03-22 
20:19:42,172::fileUtils::157::py.warnings::(createdir) 
/usr/share/vdsm/storage/blockSD.py:695: UserWarning: Using existing direcotory: 
/rhev/data-center/mnt/blockSD/6acaac02-7e7d-4321-8ba2-70e07d96581d/dom_md
Thread-45::WARNING::2016-03-22 
20:19:42,208::fileUtils::157::py.warnings::(createdir) 
/usr/share/vdsm/storage/blockSD.py:691: UserWarning: Using existing direcotory: 
/rhev/data-center/mnt/blockSD/6acaac02-7e7d-4321-8ba2-70e07d96581d/images
Thread-45::WARNING::2016-03-22 
20:19:42,208::fileUtils::157::py.warnings::(createdir) 
/usr/share/vdsm/storage/blockSD.py:695: UserWarning: Using existing direcotory: 
/rhev/data-center/mnt/blockSD/6acaac02-7e7d-4321-8ba2-70e07d96581d/dom_md
Thread-61::WARNING::2016-03-22 
20:19:42,756::fileUtils::157::py.warnings::(createdir) 
/usr/share/vdsm/storage/blockSD.py:691: UserWarning: Using existing direcotory: 
/rhev/data-center/mnt/blockSD/6c77adb1-74fc-4fa9-a0ac-3b5a4b789318/images
Thread-61::WARNING::2016-03-22 
20:19:42,757::fileUtils::157::py.warnings::(createdir) 
/usr/share/vdsm/storage/blockSD.py:695: UserWarning: Using existing direcotory: 
/rhev/data-center/mnt/blockSD/6c77adb1-74fc-4fa9-a0ac-3b5a4b789318/dom_md
jsonrpc.Executor/1::WARNING::2016-03-22 
20:19:49,638::fileUtils::157::py.warnings::(createdir) 
/usr/share/vdsm/storage/sp.py:1237: UserWarning: Using existing direcotory: 
/rhev/data-center/591475db-6fa9-455d-9c05-7f6e30fb06d5
jsonrpc.Executor/1::WARNING::2016-03-22 
20:19:50,316::fileUtils::157::py.warnings::(createdir) 
/usr/share/vdsm/storage/sp.py:1267: UserWarning: Using existing direcotory: 
/rhev/data-center/mnt/blockSD/6c77adb1-74fc-4fa9-a0ac-3b5a4b789318/dom_md
jsonrpc.Executor/1::WARNING::2016-03-22 
20:19:50,316::fileUtils::157::py.warnings::(createdir) 
/usr/share/vdsm/storage/sp.py:1268: UserWarning: Using existing direcotory: 
/rhev/data-center/mnt/blockSD/6c77adb1-74fc-4fa9-a0ac-3b5a4b789318/images
jsonrpc.Executor/1::WARNING::2016-03-22 
20:19:50,321::fileUtils::157::py.warnings::(createdir) 
/usr/share/vdsm/storage/blockSD.py:691: UserWarning: Using existing direcotory: 
/rhev/data-center/mnt/blockSD/6acaac02-7e7d-4321-8ba2-70e07d96581d/images
jsonrpc.Executor/1::WARNING::2016-03-22 
20:19:50,322::fileUtils::157::py.warnings::(createdir) 
/usr/share/vdsm/storage/blockSD.py:695: UserWarning: Using existing direcotory: 
/rhev/data-center/mnt/blockSD/6acaac02-7e7d-4321-8ba2-70e07d96581d/dom_md
8a693e96-0e89-4a5d-a3b2-093e18988e43::WARNING::2016-03-22 
20:20:03,444::fileUtils::157::py.warnings::(createdir) 
/usr/share/vdsm/storage/blockSD.py:691: UserWarning: Using existing direcotory: 
/rhev/data-center/mnt/blockSD/6acaac02-7e7d-4321-8ba2-70e07d96581d/images
8a693e96-0e89-4a5d-a3b2-093e18988e43::WARNING::2016-03-22 
20:20:03,444::fileUtils::157::py.warnings::(createdir) 
/usr/share/vdsm/storage/blockSD.py:695: UserWarning: Using existing direcotory: 
/rhev/data-center/mnt/blockSD/6acaac02-7e7d-4321-8ba2-70e07d96581d/dom_md
8a693e96-0e89-4a5d-a3b2-093e18988e43::WARNING::2016-03-22 
20:20:04,224::fileUtils::157::py.warnings::(createdir) 
/usr/share/vdsm/storage/blockSD.py:1215: UserWarning: Using existing 
direcotory: 
/rhev/data-center/mnt/blockSD/6acaac02-7e7d-4321-8ba2-70e07d96581d/master

-- 
To view, visit https://gerrit.ovirt.org/37003
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I5c3e7e21367eb690d0c7a6c6f79c515b18a9fff0
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer 
Gerrit-Reviewer: Adam Litke 
Gerrit-

Change in vdsm[master]: fileUtils: Warn about callers of fileUtils.createdir()

2016-03-22 Thread nsoffer
Nir Soffer has posted comments on this change.

Change subject: fileUtils: Warn about callers of fileUtils.createdir()
..


Patch Set 5: Verified+1

-- 
To view, visit https://gerrit.ovirt.org/37003
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I5c3e7e21367eb690d0c7a6c6f79c515b18a9fff0
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer 
Gerrit-Reviewer: Adam Litke 
Gerrit-Reviewer: Ala Hino 
Gerrit-Reviewer: Allon Mureinik 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Federico Simoncelli 
Gerrit-Reviewer: Freddy Rolland 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer 
Gerrit-Reviewer: gerrit-hooks 
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: fileUtils: Warn about callers of fileUtils.createdir()

2015-12-13 Thread automation
gerrit-hooks has posted comments on this change.

Change subject: fileUtils: Warn about callers of fileUtils.createdir()
..


Patch Set 5:

* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' 
and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.6', 
'ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])

-- 
To view, visit https://gerrit.ovirt.org/37003
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I5c3e7e21367eb690d0c7a6c6f79c515b18a9fff0
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer 
Gerrit-Reviewer: Adam Litke 
Gerrit-Reviewer: Ala Hino 
Gerrit-Reviewer: Allon Mureinik 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Federico Simoncelli 
Gerrit-Reviewer: Freddy Rolland 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer 
Gerrit-Reviewer: gerrit-hooks 
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: fileUtils: Warn about callers of fileUtils.createdir()

2015-11-18 Thread alitke
Adam Litke has posted comments on this change.

Change subject: fileUtils: Warn about callers of fileUtils.createdir()
..


Patch Set 4: Code-Review+2

-- 
To view, visit https://gerrit.ovirt.org/37003
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I5c3e7e21367eb690d0c7a6c6f79c515b18a9fff0
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer 
Gerrit-Reviewer: Adam Litke 
Gerrit-Reviewer: Ala Hino 
Gerrit-Reviewer: Allon Mureinik 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Federico Simoncelli 
Gerrit-Reviewer: Freddy Rolland 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer 
Gerrit-Reviewer: gerrit-hooks 
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: fileUtils: Warn about callers of fileUtils.createdir()

2015-11-18 Thread frolland
Freddy Rolland has posted comments on this change.

Change subject: fileUtils: Warn about callers of fileUtils.createdir()
..


Patch Set 4: Code-Review+1

-- 
To view, visit https://gerrit.ovirt.org/37003
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I5c3e7e21367eb690d0c7a6c6f79c515b18a9fff0
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer 
Gerrit-Reviewer: Adam Litke 
Gerrit-Reviewer: Ala Hino 
Gerrit-Reviewer: Allon Mureinik 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Federico Simoncelli 
Gerrit-Reviewer: Freddy Rolland 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer 
Gerrit-Reviewer: gerrit-hooks 
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: fileUtils: Warn about callers of fileUtils.createdir()

2015-11-17 Thread automation
gerrit-hooks has posted comments on this change.

Change subject: fileUtils: Warn about callers of fileUtils.createdir()
..


Patch Set 4:

* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' 
and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.6', 
'ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])

-- 
To view, visit https://gerrit.ovirt.org/37003
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I5c3e7e21367eb690d0c7a6c6f79c515b18a9fff0
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer 
Gerrit-Reviewer: Adam Litke 
Gerrit-Reviewer: Allon Mureinik 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Federico Simoncelli 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer 
Gerrit-Reviewer: gerrit-hooks 
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: fileUtils: Warn about callers of fileUtils.createdir()

2015-05-22 Thread automation
automat...@ovirt.org has posted comments on this change.

Change subject: fileUtils: Warn about callers of fileUtils.createdir()
..


Patch Set 3:

* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' 
and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 
'ovirt-3.4', 'ovirt-3.3'])

-- 
To view, visit https://gerrit.ovirt.org/37003
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I5c3e7e21367eb690d0c7a6c6f79c515b18a9fff0
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer 
Gerrit-Reviewer: Adam Litke 
Gerrit-Reviewer: Allon Mureinik 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Federico Simoncelli 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer 
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: fileUtils: Warn about callers of fileUtils.createdir()

2015-05-10 Thread amureini
Allon Mureinik has posted comments on this change.

Change subject: fileUtils: Warn about callers of fileUtils.createdir()
..


Patch Set 2: Code-Review+1

(1 comment)

https://gerrit.ovirt.org/#/c/37003/2//COMMIT_MSG
Commit Message:

Line 10: now a warning when a directory exists.  The warning is available only
Line 11: when python_warnings_enable option is enabled, so it does not effect
Line 12: users.
Line 13: 
Line 14: Change-Id: I5c3e7e21367eb690d0c7a6c6f79c515b18a9fff0
Consider adding "Related-to: https://bugzilla.redhat.com/1129587";


-- 
To view, visit https://gerrit.ovirt.org/37003
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I5c3e7e21367eb690d0c7a6c6f79c515b18a9fff0
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer 
Gerrit-Reviewer: Adam Litke 
Gerrit-Reviewer: Allon Mureinik 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Federico Simoncelli 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer 
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-HasComments: Yes
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: fileUtils: Warn about callers of fileUtils.createdir()

2015-05-06 Thread automation
automat...@ovirt.org has posted comments on this change.

Change subject: fileUtils: Warn about callers of fileUtils.createdir()
..


Patch Set 2:

* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' 
and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 
'ovirt-3.4', 'ovirt-3.3'])

-- 
To view, visit https://gerrit.ovirt.org/37003
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I5c3e7e21367eb690d0c7a6c6f79c515b18a9fff0
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer 
Gerrit-Reviewer: Adam Litke 
Gerrit-Reviewer: Allon Mureinik 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Federico Simoncelli 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer 
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches