Merge "Handle unbound arrays correctly"

Handle unbound arrays correctly

The change cfe54b794b3814f288009b796e82668ba8846ca1 missed
handling unbound arrays with `set -u`.

Issue: RELENG-2013
Change-Id: Ic37ce0404f13bff04aa4122c9b1c91add78e2145
Signed-off-by: Anil Belur <abelur@linuxfoundation.org>

Merge "Fix builder steps in lf-maven-docker-stage"

Change project comapre type to ANT

REG_EXP was the wrong compare type

Saw this in the logs:

Exception caught for project info-vote and pattern *, message: Dangling
meta character '*' near index 0

Signed-off-by: Aric Gardner <agardner@linuxfoundation.org>
Change-Id: I69bdda475ad85ee7aefafc559f25bdaf22a84ec3

Fix builder steps in lf-maven-docker-stage

lf-maven-versions-plugin builder step needs to run before
maven-patch-release.sh as this second script contains a
condition to confirm if the maven vesions plugin was selected
as a way to remove the 'SNAPSHOT' pattern from the pom.xml
files.
lf-maven-docker-stage was based on lf-maven-stage and it
seems that these particular builder steps were switched in
place accidentally.

Change-Id: I30ac4af2a7566cb2c1072412a738f38d4f3318fb
Signed-off-by: Jessica Wagantall <jwagantall@linuxfoundation.org>

Merge "Handle multiple search extension passed by JJB"

Merge "Update status-context to append java-version"

Update status-context to append java-version

Multiple jobs created using same job-template updating same github check status
due to hard coded status-context to Maven Verify.
Now appending status-context with maven-version and java-version to make it unique.

Issue: RELENG-2027
Signed-off-by: Suresh Channamallu <schannamallu@linuxfoundation.org>
Change-Id: I2ae048a24c9c68e73ab0086cf05d327eb375856d

Fix LOGS_SERVER to not be required

Change 0251b7fda7266760bd82a6e56a696160822a57a0 added in a regression
that forces LOGS_SERVER to be defined when it is not supposed to be
required for jobs.

Change-Id: I3800371e831616c9e9332d0a43e3721d8580a834
Signed-off-by: Andrew Grimberg <agrimberg@linuxfoundation.org>

gerrit-tox-verify: Allow skip-vote configuration

Add a new parameter controlling whether the Jenkins Gerrit Trigger
plugin should vote depending on the build outcome.

Change-Id: I0197b2df19e303a813df91e4c4e6120cc79456fc
Signed-off-by: Alexandru Avadanii <Alexandru.Avadanii@enea.com>

Handle multiple search extension passed by JJB

`lftools deploy archives` implementation supports '-p|--pattern'
as a multi-value option and does not support processing
multi-values as a single string.

Handle multiple search extension / patterns passed by upstream
${ARCHIVE_ARTIFACTS} param as a single string by spliting these values
before being passed to `lftools deploy archive`.

Issue: RELENG-2013
Change-Id: Id524a291a7fe03ea6f518def94b0f4ebf0974164
Signed-off-by: Anil Belur <abelur@linuxfoundation.org>

Pin pygments to 2.3.x

Pygments release 2.4.0 which added a python requires that excludes all
versions of Python < 3.5. The LFCI's default 3 version is 3.4 so causes
build failure.

Change-Id: Ide6d0da7357326c64153f0f74f328cee98429344
Signed-off-by: Andrew Grimberg <agrimberg@linuxfoundation.org>

Add missing file path triggers to maven stage jobs

Add missing config for triggering on file paths to maven stage macros and
templates, namely gerrit_trigger_file_paths and github_included_regions,
to match the behavior of the maven verify/merge macros and templates.

Change-Id: I5ecbdb99e15d8f80250e4081f98a17c630ee5e72
Signed-off-by: Lott, Christopher (cl778h) <cl778h@att.com>

Add submodule-disable for submodule operations

Some Akraino projects using git scm have a large number of submodules
which are not required during most CI/CD operations (e.g. we don't
want to fetch GBs of 3rd party repositories when linting the code).

Add submodule-disable parameter for both lf-infra-gerrit-scm
and lf-infra-github-scm, allowing us to disable submodule fetching
where needed.

Jenkins git plugin support for this was added in [1].

This change is very similar and mostly based on the addition of the
submodule-timeout parameter in commit 9fdb611f1.

[1] https://issues.jenkins-ci.org/browse/JENKINS-6658

Change-Id: Ia99ba7f353a39e55d7bec57778f447f94402b5e6
Signed-off-by: Alexandru Avadanii <Alexandru.Avadanii@enea.com>

Invoke mvn goal 'install' not deploy in new verify job

Change-Id: I2ac52d56c5ede921d39e6a12ae06d888916be14b
Signed-off-by: Lott, Christopher (cl778h) <cl778h@att.com>

Remove maven-versions-plugin-set-version var

Clean up newly added maven-docker-stage macro that missed a recent change.
This is a variable that does not need to be defined by the users of the jobs.
The version needed in this builder step is inherited from versions.properties
as "release_version". This also helps teams not having to define this version
in 2 places and just rely on version.properties.

Change-Id: I41083ed784982b904e12edf52d1f8e73a249b13d
Signed-off-by: Lott, Christopher (cl778h) <cl778h@att.com>

Merge "Define maven + docker verify, merge and stage jobs"

Define maven + docker verify, merge and stage jobs

This generalizes work first done for Acumos Java projects
that build a Spring-Boot application into a Docker image,
and do not need to deploy any jar libraries or POM files.

Change-Id: Id8040fe2d32ffe4f682f80ec5c4ed60327ef56bd
Signed-off-by: Lott, Christopher (cl778h) <cl778h@att.com>
Signed-off-by: Jessica Wagantall <jwagantall@linuxfoundation.org>
Signed-off-by: Anil Belur <abelur@linuxfoundation.org>

Remove maven-versions-plugin-set-version var

This is a variable that does not need to be defined by the users
of the jobs. The version needed in this builder step is inherited
from versions.properties as "release_version" and it is fixed as
that.
This also helps teams not having to define this version in 2
places and just rely on version.properties.
This upgrade only affects projects using maven-versions-plugin
to update their versions (ONAP).

Change-Id: I6393291eb555ed0b8f5220264688193c1af9fd5f
Signed-off-by: Jessica Wagantall <jwagantall@linuxfoundation.org>

Update jenkins-init script example

Update example jenking-init script to send stdout/stderr to a file.

Change-Id: I5a91858d1724da41139255f2314c2c6f691070de
Signed-off-by: Tim Johnson <tijohnson@linuxfoundation.org>

Merge "Make stripping of "SNAPSHOT" in poms optional"

Make stripping of "SNAPSHOT" in poms optional

When maven-versions-plugin is set to "true", the job will
use the versions plugin to update the versions in the pom.xml
files and will not need this step in maven-patch-release.sh to
happen.
This change DOES NOT affect current jobs which do not use the
maven versions plugin as this variable is set to false by default.
Update maven-versions-plugin variable in the parameters to be able
to parse the value in the scripts.

Change-Id: Ic25197012fd0f23a859159bb86369feb8a02b9c2
Signed-off-by: Jessica Wagantall <jwagantall@linuxfoundation.org>

Fix warning generated by deprecated syntax

Change-Id: Ib8ad97621527ec9869fdc743490706d9c9957df2
Signed-off-by: Tim Johnson <tijohnson@linuxfoundation.org>

Move Maven Central staging docs

The Maven CLM job docs had the staging variables for Maven Central
defined in it instead of in the Maven Stage job docs. This is both
confusing, and incorrect!

Change-Id: Icb05a8d717f1120ec30fa0c523295fe88f3fdcb7
Signed-off-by: Andrew Grimberg <agrimberg@linuxfoundation.org>

Reduce output in log files

Previously ran 'pip freeze' (> log) twice to get list of pip packages
installed during build.  Run 'pip list' (> file) twice and archive the
files and diffs. Configured 'logs-deploy.sh' to run as separate Jekins
script.

Issue: RELENG-1902
Change-Id: Ib0f99bd1eb125015467c7eaf91d26c76e92bffa0
Signed-off-by: Tim Johnson <tijohnson@linuxfoundation.org>

Fix execute permissions

Change-Id: I253ed35ec78f51d4b2352539d2d24260edde5459
Signed-off-by: Tim Johnson <tijohnson@linuxfoundation.org>

Merge "Upgrade packer version to 1.4.0"

Fix gerrit-packer-merge choosing strategy

Our definition of a merge job is that it always takes the HEAD of the
branch. Gerrit Trigger choosing strategy does not do this and breaks
expected outcomes when retriggering an old merge.

Change-Id: I2e3c12f1c9b59e624e742943a5dbef5123129e6c
Signed-off-by: Andrew Grimberg <agrimberg@linuxfoundation.org>

Upgrade packer version to 1.4.0

The next release of common-packer requires packer 1.3.2 or greater.
Update to 1.4.0 as it is the current release and has been verified to
work properly.

Change-Id: Iec56190d26ff98dfaf5df1346dc82e7995ab640c
Signed-off-by: Andrew Grimberg <agrimberg@linuxfoundation.org>

Add missing script name markers

Add missing script name markers useful in console logs.

Change-Id: I3c0768d3d5def6258bb8a5e556d78b4836fa957e
Signed-off-by: Anil Belur <abelur@linuxfoundation.org>

Update lftools to v0.23.1

Change-Id: I4289796af1e455a6ab1f6e59b29da60d46b6b1d1
Signed-off-by: Anil Belur <abelur@linuxfoundation.org>

Merge "Add lf-infra-wss-mvn-clean-install step"

Update jenkins-cfg-verify to use $GERRIT_REFSPEC

Update jenkins-cfg to verify images names updated from the
$GERRIT_REFSPEC instead of the master branch.

Current implementation uses the 'master' branch instead of
the $GERRIT_REFSPEC, therefore failing the verify job
on false positives.

Issue: RELENG-1950
Change-Id: I6f212b28e4189ff5c7958035c7992b55d7153d02
Signed-off-by: Anil Belur <abelur@linuxfoundation.org>

Add lf-infra-wss-mvn-clean-install step

Add Maven clean install conditional step
that runs before WhiteSource Unified Agent scan.
This allows to fetch additional dependencies.

Issue: RELENG-1951
Change-Id: Id3e070220e17ba995d976b3d033a16983b38fd8d
Signed-off-by: Jessica Wagantall <jwagantall@linuxfoundation.org>

Hardcode WSS project version

Similarly to how is done in CLM, the WhiteSource
versions for projects should reflect the branch they are
running on.

Also fix script variables to be enclosed by "{}".

Change-Id: Ica0a52adcc9223e861f213bc53c67ed2102efda5
Signed-off-by: Jessica Wagantall <jwagantall@linuxfoundation.org>

Use WS Agent config file internally

After checking with legal, it was agreed to not
allow tech teams to provide their own
wss-unified-agent.config file for WhiteSource scans.
Instead, this file should be managed in Jenkins Settings
Files

Change-Id: I20e330794ba28cb4c0f5206c290a734c1794b00e
Signed-off-by: Jessica Wagantall <jwagantall@linuxfoundation.org>

Merge "Fixes for info votes"

Merge "Add missing release notes."

Fixes for info votes

gerrit trigger is not working in opnfv.
we had this happen to some of the read the docs jobs...
An updgrade of gerrit is planned to hopefully resolve this.

Not sure were to test to approval category stuff.
That said I think the mistake is that the value is an int.

            approval-category: 'WHATEVER'
            approval-value: 2

I dont actually want the script to exit on first error, need to output
the vote verdict.

not sure why I thought that JENKINS_SSH_CREDENTIAL
was the correct var for the ssh user.
switched to $USER

Change-Id: I6714361423948eb1764e9c18da699dafd4cd24a9
Signed-off-by: Aric Gardner <agardner@linuxfoundation.org>

Add missing release notes.

Commit b2e2b5b7d3bb61ee9886debb4e010769c35be398
was missing release notes.

Change-Id: If6adc956f92113e80177930b82ba20872b4e009f
Signed-off-by: Aric Gardner <agardner@linuxfoundation.org>

Update rtd-merge to also trigger on tag changes

When we release a tag, it should also trigger a rtd-merge build
so that the doc release note link is updated appropriately.

Change-Id: I49f6555b707018cca4042917c4193138456dafeb
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

ARM64 support for update-java-alternatives

Java paths at Ubuntu contain host CPU architecture signature.
Script was updated to correctly detect architecture by dpdk.

Change-Id: I77c4aba1d600373edff283b6b85837fccda3af34
Signed-off-by: Martin Klozik <martin.klozik@tieto.com>

Fix bug in package-listing start script

Change af89ef784234e7b89984d5b2995fc6d29b6d8c34 introduced a bug where
we don't make sure that the package-listing.sh script is executable
before calling it during system startup. This means that we are no
longer currently producing the needed listing start file for instances
to be able to diff against to give us a view of package changes.

Change-Id: Iea2f0b07cdf10f7819072bcb00596bdeda4665df
Signed-off-by: Andrew Grimberg <agrimberg@linuxfoundation.org>

Merge "Fix global-jjb docs for installation"

Fix global-jjb docs for installation

The docs copy / paste script use the GitHub submodule but don't properly
name the resultant module to match the expectations of the rest of the
script.

Fix up some minor inconsistencies / errors of other parts of script

Change-Id: I66402d9ef1be80c1dd53c51d55802a864c813421
Signed-off-by: Andrew Grimberg <agrimberg@linuxfoundation.org>

Merge "Add WhiteSource Scan jobs"

Add WhiteSource Scan jobs

WhiteSource is an open source security and license compliance
management platform.
It is used to scan a great variety of languages includding Java,
Python, C, Go, among others.

{project-name}-whitesource-scan-{stream} uses WhiteSource's
Unified Agent to perform the code scans.

Issue: RELENG-1876
Change-Id: If511da2f51e08799352ee6841338f76dd63af6b6
Signed-off-by: Jessica Wagantall <jwagantall@linuxfoundation.org>

Update lftools to version v0.22.2

Change-Id: Ic1e44a73813dd42c9b376547cc4814f1d63b6ab4
Signed-off-by: Aric Gardner <agardner@linuxfoundation.org>

Merge "INFO.yaml check vote for global-jjb"

Add comment-to-gerrit macro

This macro will post a comment to the gerrit patchset if the build
creates a file named gerrit_comment.txt
To use this macro add it to the list of builders

Change-Id: I30e2b10fc4e09f534b161dd99775d66bdc4c3e1a
Signed-off-by: Aric Gardner <agardner@linuxfoundation.org>

INFO.yaml check vote for global-jjb

Global job per gerrit, that triggers on changes
to INFO.yaml file

And auto merged if a majority of committers
have voted.

This will create a review in the internal gerrit

If a tsc vote is required, RE must manually confirm
that this has happened.

We will need to give
Non-Interactive Users merge
or perhaps specifically only the jenkins
user.

Job will only trigger when a user votes +2
cludgy but it works.

Example of Job configuration:

- project:
    name: info-vote
    build-node: centos7-builder-2c-1g
    jobs:
      - 'info-vote-verify'

Change-Id: Ib386bd2ea51be113ca59610bd3281b9ad6923bce
Signed-off-by: Aric Gardner <agardner@linuxfoundation.org>

Revert Add tox-verify job to the default ci group

This reverts commit 059f6382bcbe1549201773eb4d881d43724ca8a9.

Merged it to quickly, need to ensure all projects has tox.ini
and .coafile linting in place first.

Change-Id: I083fcb526d1858d15decb58d4138f9dcc9cc600a
Signed-off-by: Bengt Thuree <bthuree@linuxfoundation.org>

Merge "Add tox-verify job to the default ci group"

Allow jjb-workers to be configurable

Default it to the number of CPU cores available on the system.

Issue: RELENG-917
Change-Id: I5f65ec7d7142f1a778b6e31cc6700ca2bca1ef88
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Skip creating source jars in verify job

Issue: RELENG-1655
Change-Id: Ibd2aad63afda3feb97619cef67c10c8b665cd924
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Add tox-verify job to the default ci group

This ensures our projects are running linters against the prjoect.

Change-Id: I5ce38eeecef8f1241a49dfe7da3eb04fec7ac385
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Update lftools to version v0.22.1

Change-Id: I728e3581f956cd56cc0446f2a3360bd9e71df6af
Signed-off-by: Anil Belur <abelur@linuxfoundation.org>

Update lftools to version 0.22.0

Change-Id: I545662ee1cf31e1609971c08bb75555acf384139
Signed-off-by: Anil Belur <abelur@linuxfoundation.org>

Merge "Fix JAVA_HOME for CentOS7"

Fix JAVA_HOME for CentOS7

JAVA_HOME for JDK-11 on CentOS7 is set to /opt/jdk-11 but JDK-11 is
installing in /usr/lib/jvm/java-11-openjdk/.

This change is to modify JAVA_HOME to use the correct path and install
JDK-11 properly.

Change-Id: I598a063988a5afea5d6827d0940fdcbd2b83e99a
Signed-off-by: Vanessa Rene Valderrama <vvalderrama@linuxfoundation.org>

Add Throttle Property to JJB jobs

This adds "Throttle Builds" configuration to JJB jobs (verify, merge,
etc) to allow global-jjb users with static builders to limit the number
of jjb jobs ran on a single node.

Issue: RELENG-1510
Change-Id: I00d2950451eae8037ded407937131723b0001179
Signed-off-by: Trevor Bramwell <tbramwell@linuxfoundation.org>

Adapt maven path search for files and dirs

The "-f" maven param can specify both a directory, in which case it
will look for "pom.xml" in the directory, or a specific file. The
original version of this search was only compatible with directories
that contain a pom.xml file.

Change-Id: I81ab161f3156fc2159a4b7ca9ca084a0f82da795
Signed-off-by: Eric Ball <eball@linuxfoundation.org>

Merge "Extract version number passed set '{java-version}'"

Add set version conditional step

Add conditional step for gerrit-maven-stage to optionally
run the versions maven plugin using the versions.properties
file.

Issue: RELENG-1860
Change-Id: I4bea0ae911a79db7ef238f5ece619be6270e2256
Signed-off-by: Jessica Wagantall <jwagantall@linuxfoundation.org>

Cleanup Jenkins init scripts

RELENG-1856 Cleanup Jenkins init scripts Make scripts portable so the
ci-management can be installed anywhere.  Update README comments. All
changes should be backward compatible.

Change-Id: I5ce6f76cfa0483b7a1b91b72ae1264c21e769dba
Signed-off-by: Tim Johnson <tijohnson@linuxfoundation.org>

Extract version number passed set '{java-version}'

When the version number string is multi-digits ex: openjdk11
"${SET_JDK_VERSION: -1}" returns '1' which is incorrect.

The change fixes version number extraction for the string
passed to env variable $SET_JDK_VERSION.

Also update the regex to handle values with special chars.

Issue: RELENG-1853
Change-Id: I52c49fec007bad70d4a42d4c20a45b35fec3997c
Signed-off-by: Anil Belur <abelur@linuxfoundation.org>

Merge "Document required params in lf-maven-stage macro"

Merge "Optimize the jjb-deploy trigger regex"

Update lf-maven-central macro docs and templates

Update lf-maven-central macro docs and templates
with the missing required params.

The change 6a3a8fd3c9430a90f3b8c0c85e7e20b8a318eb8f
missed updating the docs and sample code.

Change-Id: I09b55bbf51cc3ef7c29d50b42e52982dc40e8f72
Signed-off-by: Anil Belur <abelur@linuxfoundation.org>

Fix formatting in release notes files

The change I04af514bbb237f405eb1d31ec10e3f019c85a43e breaks the rtd
verify job[1] because of the incorrect formating of the prelude section.

The prelude section can not contain a list as all prelude section are
concatenated together when the release notes are built. Merging the
preludes into their respective feature sections ensures the contex of
the comments are preserved as otherwise they will appear at the top of
the documentation apart from any other information.

Ref: https://docs.openstack.org/reno/latest/user/examples.html

[1] https://jenkins.opendaylight.org/releng/job/lf-infra-global-jjb-rtd-verify-any/475/

Change-Id: I3d23679eb4740b5fdc22fc566811d06ac6128820
Signed-off-by: Anil Belur <abelur@linuxfoundation.org>
Signed-off-by: Trevor Bramwell <tbramwell@linuxfoundation.org>

Merge "Update sigul macro to support sign mode"

Merge "Update lftools to version 0.21.0"

Update Jenkins Init Scripts

Support ability to specify swap size and to pre-allocate the '/w' volume
for the Hyperledger Project

Change-Id: I04af514bbb237f405eb1d31ec10e3f019c85a43e
Signed-off-by: Tim Johnson <tijohnson@linuxfoundation.org>

Update sigul macro to support sign mode

Sign mode is a parameter to allow the user to choose whether or not
to sign artifacts in parallel using gnu-parallel or in serial mode
(bash for loop) methods.

Cleanup obsolete lftools_activate method.

Change-Id: Ib202f93a37e2830e5048c5a766c17d75233c6423
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Update lftools to version 0.21.0

Change-Id: Ibbcd6b25247dcad926fc67342f9eac47347902e0
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Optimize the jjb-deploy trigger regex

Optimize the jjb-deploy trigger regex to trigger the job _only_
when the Gerrit CR comment is intended as a command.

The `jjb-deploy` job presently gets triggered and is waste of infra
resources when the keyword is intended as comment in Gerrit.

Ref: https://git.opendaylight.org/gerrit/#/c/80358/

Issue: RELENG-1847
Change-Id: I3b35adb1284d3d9029ba025bf660b66fbb5d98e0
Signed-off-by: Anil Belur <abelur@linuxfoundation.org>

Document required params in lf-maven-stage macro

Change-Id: I1f281b22f7b9454ef58e3fcc5fa49c5d333263a7
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Merge "Add cmake-sonar job"

Minor cleanup of openstack-protect script

Change-Id: I45070e0b364136b28e4130aed3972603e8c2e3c1
Signed-off-by: Tim Johnson <tijohnson@linuxfoundation.org>

Rework image code to handle no results from YAML

When grep has 0 matches it returns a non-zero exit code. In this case
a 0 match grep is a possibility for YAML files so ignore grep's exit
code in this instance.

Change-Id: Ia7eb55364feeb42bf35681c4ae0798fe25bbfad1
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Search for CFG & YAML files in specific dirs

These file locations are standard locations in our CI setup so be
specific of where to search for them. This prevents the scripts
from picking up data from test files in global-jjb.

Change-Id: Ia486ae8704b77f9303b5a2e54c4ffaf5de41aef8
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Bump lftools to v0.20.0

Change-Id: I6fa72595c1843a1955f4b492c035ebc7863163b5
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Add niet needed for artifcat release job

Change-Id: I4223d00667b5701377d057e76e82e8657960d262
Signed-off-by: Aric Gardner <agardner@linuxfoundation.org>

Add Error Handling

Packer Cleanup Script was ignoring all errors

Change-Id: I881abf36420a1976c4f3af552aeeca4a4512c555
Signed-off-by: Tim Johnson <tijohnson@linuxfoundation.org>

Merge "Update lf-maven-stage macro with missing parts"

Merge "Add job to automatically announce lf releases"

Add job to automatically announce lf releases

This job triggers when a Gerrit tag is pushed and sends an email to
lf-releng@lists.linuxfoundation.org to notifiy the release.

Change-Id: I5255ac6f3afbcadfc11e9640483613bd2e84250a
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Update lf-maven-stage macro with missing parts

Added the following missing parts to the lf-maven-stage macro.

* lf-provide-maven-settings
* lf-infra-create-netrc
* lf-provide-maven-settings-cleanup

This improves the macro to be a complete function that does not require
preparatory steps to complete it.

New Usage:

      - lf-maven-stage:
          mvn-global-settings: 'global-settings'
          mvn-settings: 'settings'
          mvn-staging-id: 'staging-profile-id'

This patch also updates the lf-maven-central macro as necessary to
work along side the lf-maven-stage changes.

Issue: RELENG-1554
Change-Id: Ie4fbcb52229c74c4216df4d07f077cd691ab6aa6
Signed-off-by: Bengt Thuree <bthuree@linuxfoundation.org>

Add cmake-sonar job

This job only supports SonarCloud configuration.

Change-Id: I4f2b5100e3316f21c866155199aeed0533d13b8b
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Add support for SonarCloud

If SonarCloud is used the old method is automatically disabled so that
Sonar is not run twice.

Example Usage:

- project:
    name: yangtools-sonar
    jobs:
      - gerrit-maven-sonar

    project: 'yangtools'
    project-name: 'yangtools'
    branch: 'master'
    mvn-settings: 'yangtools-settings'
    mvn-opts: '-Xmx1024m'
    sonarcloud: true
    sonarcloud-project-key: KEY
    sonarcloud-project-organization: ORGANIZATION
    sonarcloud-api-token: TOKEN

Issue: RELENG-1721
Change-Id: Ica1cd29bfc9246f2452370dd6ceb8b7e5b8e5329
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Modify cron entry

Modify cron entry to pick from custom config

Issue-Id: RELENG-1728
Change-Id: Icd4d97a37672fd0dece6960fa6de7d9fa2fe8a1e
Signed-off-by: Suresh Channamallu <schannamallu@linuxfoundation.org>

Verify build node HARDWARE_ID

This script already checks each file's IMAGE_NAME for validity; this
adds essentially the same validation to check the HARDWARD_ID.
Also add the job to the ci-jobs groups, and set submodule-recursive
value for github-jenkins-cfg-verify.

Issue: RELENG-1418
Change-Id: I0266a0fcc276b2960c9c1749a89bc70e8115d152
Signed-off-by: Eric Ball <eball@linuxfoundation.org>

Add Release Notes for Packer Build Description

This was missed in the previous commit to add packer images to the
Jenkins build description.

Issue: RELENG-1073
Change-Id: Iced92ada13d0f576a7b560df5bc1242d83ee9689
Signed-off-by: Trevor Bramwell <tbramwell@linuxfoundation.org>

Fix disable-job param typo in docs

Change-Id: I28011767f45b0d29743c74cbec15f55bc48f603b
Signed-off-by: Anil Belur <abelur@linuxfoundation.org>

Set Packer Image name as Build Description

With the Build Description Setter plugin installed in Jenkins, the
packer image name can be extracted after a successful build.

This allows users to no longer look through the build logs when trying
to find the image name for builds.

Issue: RELENG-1073
Change-Id: Iffebd2b7e6b282d3e17a7bd6de1452e063f64603
Signed-off-by: Trevor Bramwell <tbramwell@linuxfoundation.org>

Extend ${JOB_NAME} to include {java-version} param

To support javadocs jobs to build with multiple versions of
openjdk{8,11}, extend with '{java-version}' var to the ${JOB-NAME}.
Extending the job name with (java-version) var, JJB ensures
that its not passing the multiple java-versions as a list.

The job failure can be seen on the lispflowmapping jobs.
https://logs.opendaylight.org/releng/vex-yul-odl-jenkins-1/lispflowmapping-maven-javadoc-verify-neon/71/
https://logs.opendaylight.org/releng/vex-yul-odl-jenkins-1/lispflowmapping-maven-javadoc-publish-neon/19

Change-Id: If2e189f3e76f36d73863d98f1217c0b42ada52f1
Signed-off-by: Anil Belur <abelur@linuxfoundation.org>

Fix coala warning

The warning was introduced with the change c90444b3
which fails the tox verify jobs.

Change-Id: I4b6241d53820f588b80f54b28bc92b635fda1b4f
Signed-off-by: Anil Belur <abelur@linuxfoundation.org>

Add release notes for checking openjdk $VERSION

The release notes is for I19d77c509c90bca5fea1af53dacc35772c7ac939.

Change-Id: I81d0b2082376517ec26c978d6baa3fbd4e2e3d48
Signed-off-by: Anil Belur <abelur@linuxfoundation.org>

Fix version number extraction for all cases

When the jobs upgrade to a later version of Java passing
"openjdk11" or "openjdk10" the string substitution would only
return the last char which would potentially fail to pick up
the correct version of Java.

This change fixes this issue to handle all cases.

Change-Id: If81100bbdd116ee071c3ecdeeadc0a3016fa2a15
Signed-off-by: Anil Belur <abelur@linuxfoundation.org>

Check MAVEN_PARAMS for -f flag in metadata fetch

maven-fetch-metadata.sh was not respecting the "-f" (for file path)
flag in MAVEN_PARAMS, causing lf-maven-merge jobs that utilize this
flag to fail.

Issue: RELENG-1630
Change-Id: I23803fd2e4a46022d85dc2befab1438853c7c3ca
Signed-off-by: Eric Ball <eball@linuxfoundation.org>