Max has posted comments on this change. ( https://gerrit.osmocom.org/12857 )
Change subject: Add timeout wrapper to regular gerrit verification jobs ...................................................................... Patch Set 2: (1 comment) https://gerrit.osmocom.org/#/c/12857/1/jobs/gerrit-verifications.yml File jobs/gerrit-verifications.yml: https://gerrit.osmocom.org/#/c/12857/1/jobs/gerrit-verifications.yml@23 PS1, Line 23: als > How about we put the 15m into a variable at the top, that can be overridden > by each job? Then we cou […] I've updated the patch but unsure about variable substitution: when shall we use '{obj:var}' instead of '{var}'? Please double-check. -- To view, visit https://gerrit.osmocom.org/12857 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-ci Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I9d0eb6b9eddf3e17120e74d7570d42291c0750ae Gerrit-Change-Number: 12857 Gerrit-PatchSet: 2 Gerrit-Owner: Max <[email protected]> Gerrit-Reviewer: Harald Welte <[email protected]> Gerrit-Reviewer: Max <[email protected]> Gerrit-Reviewer: Pau Espin Pedrol <[email protected]> Gerrit-Reviewer: daniel <[email protected]> Gerrit-Reviewer: lynxis lazus <[email protected]> Gerrit-Reviewer: osmith <[email protected]> Gerrit-Comment-Date: Mon, 11 Feb 2019 14:38:53 +0000 Gerrit-HasComments: Yes Gerrit-HasLabels: No
