From 84c10e0c5a3a6f64086852b05cd433a0638e193e Mon Sep 17 00:00:00 2001 From: Olena Logvinova Date: Mon, 3 Apr 2017 20:34:11 +0300 Subject: [PATCH] [contirb-guide] Jenkins checks and recheck info Change-Id: Ic1c8a5815daa54ae35ea6f3b9d884bcb21e08bcf --- doc/contributor-guide/source/docs-builds.rst | 2 ++ .../source/quickstart/first-timers.rst | 19 +++++++++++++++++++ 2 files changed, 21 insertions(+) diff --git a/doc/contributor-guide/source/docs-builds.rst b/doc/contributor-guide/source/docs-builds.rst index efa5dc816f..7f1d69aac6 100644 --- a/doc/contributor-guide/source/docs-builds.rst +++ b/doc/contributor-guide/source/docs-builds.rst @@ -213,6 +213,8 @@ To build a patch locally: #. Find the build result in ``publish-docs/index.html``. +.. _build_jobs: + Build jobs ~~~~~~~~~~ diff --git a/doc/contributor-guide/source/quickstart/first-timers.rst b/doc/contributor-guide/source/quickstart/first-timers.rst index f28920ca68..cae2df8b9a 100644 --- a/doc/contributor-guide/source/quickstart/first-timers.rst +++ b/doc/contributor-guide/source/quickstart/first-timers.rst @@ -156,8 +156,12 @@ Committing a change https://review.openstack.org/ +#. In Gerrit, wait for the automatic Jenkins checks to succeed. + Celebrate and wait for reviews! +.. seealso:: * :ref:`troubleshoot_setup` + * :ref:`docs_builds` Responding to requests ~~~~~~~~~~~~~~~~~~~~~~ @@ -291,6 +295,21 @@ git and git review `Resolving merge conflicts after a git rebase `_. +* ``FAILURE`` in the ``Jenkins check`` section of your commit in Gerrit + + #. Click the link next to the ``FAILURE`` test. + #. Verify the output of the :file:`console.html`: + + * If it contains errors or warnings related to your commit with + a problematic file name and line in question, fix the issues and + push the new changes on review. + * If the issue is not related to your commit, re-run the gate jobs by + leaving the word *recheck* in the comments section of your + Gerrit review. + + .. seealso:: * `Automated Testing `_ + * :ref:`build_jobs` + Network -------