Table of contents
- How to file a bug report
- How to contribute code
- How to write tests
- How to debug your test
How to file a bug report¶
If you have any suggestions for improvements please do not hesitate to open an issue.
How to contribute code¶
We encourage you to contribute new plugins. We use gerrithub for reviewing proposed changes. The submission process looking something like this:
Clone the citellus repository:
git clone https://github.com/citellusorg/citellus.git
Check out a branch in which to make your changes:
git checkout -b "your-new-branch"
Edit your files and validate with tox:
tox # this will check the changes for some errors
- NOTE: tox will run python 2.7, pep8 and python 3.5 tests, if your environment lacks for example python 3.5, do execute tox -l to see the available tests and skip that one, for example:
sh tox -e pep8 tox -e py27 # We're skipping tox -e py35 which is also invoked by default when tox is executed without arguments.
- In this way, errors that might be relevant to the test environment can be skipped. For example this one reported at issue tracker as 104)
Update your local repository:
1 2 3 4 5 6
git add $modified_files git commit For the message, please use a short line with the fix and the subject like `[plugins][openstack][nova] Check nova configuration XXX` If the commit fixes a github open issue, also use `Closes #$ISSUEID` so github automatically closes it once merged referencing the commit.
Submit your changes for review:
Then wait for your changes to be reviewed. It is common for reviewers to request changes; when this happens:
Edit your files and revalidate with tox:
tox # this will check the new changes for some errors
Update your existing commit. Do not create a new commit!
git add $modified_files git commit --amend
Resubmit the change:
You can see pending and already merged actual changes at: https://review.gerrithub.io/#/q/project:citellusorg//citellus
Once the new plugin has been submitted you’ll see some comments from ‘Citellus Jenkins’ which is running Unit tests against it (same ones that you run with
If Jenkins gives ‘Verified +1’, next step is wait for one reviewer to give final ACK and merge the change.
How to write tests¶
Please refer to the templates folder for examples.
If you want to contribute also Unittests for your plugins, check TESTING.md
For contributing translations check i18n.md
How to debug your test¶
We’ve included a file named
env-for.debug.sh in the root folder of citellus repo that allows to be sourced and later execute your script.
This environment file will define the standard variables Citellus does use like:
And will preload the common-functions
The way to use it is:
Then you can debug your script with:
Or test individual functions output like:
Please, do note that as some functions do ‘exit’ or ‘return’ for example executing
is_rpm_over will exit the active shell, so beware!!