Change in vdsm[master]: use os.path.join with P_VDSM_RUN

2017-01-13 Thread Code Review
From Dan Kenigsberg : Dan Kenigsberg has uploaded a new change for review. Change subject: use os.path.join with P_VDSM_RUN .. use os.path.join with P_VDSM_RUN In three places the code assumes that P_VDSM_RUN ends with a slash.

Change in vdsm[master]: storage: move task and taskManager to lib

2017-01-13 Thread Code Review
From Dan Kenigsberg : Dan Kenigsberg has submitted this change and it was merged. Change subject: storage: move task and taskManager to lib .. storage: move task and taskManager to lib Change-Id: I97890c2600ffd8bfef7b51fdc4457

Change in vdsm[master]: automation: fixing check-merged's incorrect local repo sourc...

2017-01-13 Thread Code Review
From Dan Kenigsberg : Dan Kenigsberg has posted comments on this change. Change subject: automation: fixing check-merged's incorrect local repo source path. .. Patch Set 28: (2 comments) https://gerrit.ovirt.org/#/c/69809/28

Change in vdsm[master]: automation: clean minor OVS hook leftover

2017-01-13 Thread Code Review
From Dan Kenigsberg : Dan Kenigsberg has posted comments on this change. Change subject: automation: clean minor OVS hook leftover .. Patch Set 1: (1 comment) https://gerrit.ovirt.org/#/c/70391/1/automation/check-merged.sh Fi

Change in vdsm[master]: automation: fixing check-merged's incorrect local repo sourc...

2017-01-13 Thread Code Review
From Dan Kenigsberg : Dan Kenigsberg has posted comments on this change. Change subject: automation: fixing check-merged's incorrect local repo source path. .. Patch Set 29: Code-Review+2 I don't like the vague commit message

Change in vdsm[master]: automation: fix localsync's repository name

2017-01-13 Thread Code Review
From Dan Kenigsberg : Dan Kenigsberg has posted comments on this change. Change subject: automation: fix localsync's repository name .. Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.ovirt.org/70352 To unsubscrib

Change in vdsm[master]: automation: don't skip localsync if it is unavailable

2017-01-13 Thread Code Review
From Dan Kenigsberg : Dan Kenigsberg has posted comments on this change. Change subject: automation: don't skip localsync if it is unavailable .. Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.ovirt.org/70353 To

Change in vdsm[master]: automation: fixing check-merged's incorrect local repo sourc...

2017-01-13 Thread Code Review
From Dan Kenigsberg : Dan Kenigsberg has submitted this change and it was merged. Change subject: automation: fixing check-merged's incorrect local repo source path. .. automation: fixing check-merged's incorrect local repo so