Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[WIP] initiall version of PR test jobs #11

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

dahorak
Copy link
Contributor

@dahorak dahorak commented Feb 9, 2018

Fixes #10

Jobs for launching test suite for every PR.

It will create one job for each repository.
The job might be triggered by run tests phrase in the respective PR.

It is possible to specify dependencies on PRs from other repos this way:

Depends on: tendrl-ui/pr809 tendrl-notifier/pr153

either in the PR description or in the comment triggering the test execution (or on both places).

It has to be on one line without any additional formatting and has to follow quite precisely this format, to be able to correctly (and easily) parse it.
Basically the line has to start with depends on: string (case insensitive) followed by space separated list of dependencies in the form of: <reponame>/pr<id>, where <reponame> is the name of the repository (including tendrl- prefix) and <id> is the PR number.

@dahorak
Copy link
Contributor Author

dahorak commented Feb 9, 2018

The Jobs are visible on tendrl-PR CentOS CI Jenkins page and are named Tendrl test (PR) - <repo-name>.

@r0h4n
Copy link
Contributor

r0h4n commented Feb 15, 2018

@TimothyAsir Please review this

@r0h4n r0h4n requested review from a team as code owners February 17, 2018 19:01
@r0h4n r0h4n removed the request for review from nthomas-redhat February 17, 2018 19:01
- github-pull-request:
admin-list:
- dahorak
white-list:

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there a way to input this rather than hardcoding

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's the same as for the build jobs. Itshould be possible, to automatically fetch the list of people from the organization specified below in the org-list section, but there seems to be some problem with this and I'm not sure why it doesn't work and where might be the problem with that.
I'll try to check that once more and find some more information about that - that will be definitely the best solution.

property-file: env.sh
block: true

publishers:

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Where the test results will be posted?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The overall status of the installation and tests is reported back to the PR where it was triggered (as a CentOS CI - tests line in the checks section) and Details link points to the particular job.
More granular results are available in the subsequent jobs (see Subproject Builds section of the Jenkins job).

Additionally when it fails, it will notify me over email, so I can check if it is real installation or test failure, or if it is problem with the tests itself.

dahorak added 2 commits March 14, 2018 15:18
- changes related to usage of tendrl-ansible changes
- it is not possible to send emails to this list outside of RH (from
  [email protected])
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Run functional testsuite for PR builds
3 participants