> Makefile by testing for JENKINS_URL? Inputs to the program make should be understandable by human. When I want to predict locally what would the checkstyle job vote, I prefer something like: make checkstyle-all rather than: JENKINS_URL=jenkins.fd.io make checkstyle
Or did you mean a different variable? Here [3] is a typical list. Vratko. [3] https://jenkins.fd.io/view/vpp/job/vpp-checkstyle-verify-master/12455/injectedEnvVars/ From: vpp-dev@lists.fd.io <vpp-dev@lists.fd.io> On Behalf Of Paul Vinciguerra Sent: Wednesday, November 13, 2019 4:28 PM To: Dave Wallace <dwallac...@gmail.com> Cc: vpp-dev <vpp-dev@lists.fd.io> Subject: Re: [vpp-dev] Multiple patch validation failures: yamllint AWOL How do people feel about managing the CI where possible via the Makefile by testing for JENKINS_URL? On Wed, Nov 13, 2019 at 10:11 AM Dave Wallace <dwallac...@gmail.com<mailto:dwallac...@gmail.com>> wrote: Status update: Florin fixed 'make checkstyle' with [1] (Thanks Florin! :) Ed Kern has added 'make install-dep' to the checkstyle verify build executors. The revert of my ci-management patch [2] will be abandoned. Florin's patch will be reverted once the new build executors are available and verified that yamllint is available. Sorry for the bog-up. Thanks, -daw- [1] https://gerrit.fd.io/r/c/vpp/+/23391 [2] https://gerrit.fd.io/r/c/ci-management/+/23410 On 11/13/2019 8:58 AM, Dave Wallace via Lists.Fd.Io<http://Lists.Fd.Io> wrote: Dave, My bad. I had Vanessa merge a patch to ci-management [0] which I tested locally, but forgot that the build executors don't run 'make install-dep'. Unfortunately the ci-management verify jobs don't actually run any of the CI jobs to ensure that the system still works with the patch being submitted... I'm in the process of reverting the patch until the build executors can be upgraded. Thanks, -daw- [0] https://gerrit.fd.io/r/c/ci-management/+/23364 On 11/13/2019 7:35 AM, Dave Barach via Lists.Fd.Io<http://Lists.Fd.Io> wrote: I wonder where “yamllint” went? Multiple patches affected... 07:28:23 clang-format version 6.0.0-1ubuntu2 (tags/RELEASE_600/final) 07:28:24 HEAD detached at dce8b380a 07:28:24 nothing to commit, working tree clean 07:28:24 ******************************************************************* 07:28:24 * VPP CHECKSTYLE SUCCESSFULLY COMPLETED 07:28:24 ******************************************************************* 07:28:24 yamllint /w/workspace/vpp-checkstyle-verify-master/src 07:28:24 make: yamllint: Command not found 07:28:24 Makefile:561: recipe for target 'checkstyle' failed 07:28:24 make: *** [checkstyle] Error 127 07:28:24 Build step 'Execute shell' marked build as failure FWIW... Dave -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#14579): https://lists.fd.io/g/vpp-dev/message/14579 Mute This Topic: https://lists.fd.io/mt/55791974/675079 Group Owner: vpp-dev+ow...@lists.fd.io<mailto:vpp-dev+ow...@lists.fd.io> Unsubscribe: https://lists.fd.io/g/vpp-dev/unsub [dwallac...@gmail.com<mailto:dwallac...@gmail.com>] -=-=-=-=-=-=-=-=-=-=-=- -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#14583): https://lists.fd.io/g/vpp-dev/message/14583 Mute This Topic: https://lists.fd.io/mt/55791974/675079 Group Owner: vpp-dev+ow...@lists.fd.io<mailto:vpp-dev+ow...@lists.fd.io> Unsubscribe: https://lists.fd.io/g/vpp-dev/unsub [dwallac...@gmail.com<mailto:dwallac...@gmail.com>] -=-=-=-=-=-=-=-=-=-=-=- -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#14589): https://lists.fd.io/g/vpp-dev/message/14589 Mute This Topic: https://lists.fd.io/mt/55791974/1594641 Group Owner: vpp-dev+ow...@lists.fd.io<mailto:vpp-dev%2bow...@lists.fd.io> Unsubscribe: https://lists.fd.io/g/vpp-dev/unsub [pvi...@vinciconsulting.com<mailto:pvi...@vinciconsulting.com>] -=-=-=-=-=-=-=-=-=-=-=-
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#14596): https://lists.fd.io/g/vpp-dev/message/14596 Mute This Topic: https://lists.fd.io/mt/55791974/21656 Group Owner: vpp-dev+ow...@lists.fd.io Unsubscribe: https://lists.fd.io/g/vpp-dev/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-