For compliance with the Project Testing Interface [1] as described in [2] [1] https://governance.openstack.org/tc/reference/project-testing-interface.html [2] http://lists.openstack.org/pipermail/openstack-dev/2017-December/125710.html doc8 command is dropped from docs tox envs. So this affect nothing and run in PEP8. Related-Bug: #1765348 Depends-On: Icc7fe3a8f9716281de88825e9d5b2fd84de3d00a Change-Id: Idf9a16111479ccc64004eac9508da575822a3df5
4.7 KiB
How To Contribute
Basics
- Our source code is hosted on OpenStack Kolla-Ansible Git. Bugs should be filed on launchpad.
- Please follow OpenStack Gerrit Workflow to contribute to Kolla-ansible.
- Note the branch you're proposing changes to.
master
is the current focus of development. Kolla project has a strict policy of only allowing backports instable/branch
, unless when not applicable. A bug in astable/branch
will first have to be fixed inmaster
. - Please file a blueprint of
kolla-ansible for any significant code change and a bug for any
significant bug fix, or add a
TrivialFix
tag to commit message for simple changes. See how to reference a bug or a blueprint in the commit message. - TrivialFix tags or bugs are not required for documentation changes.
Development Environment
Please follow our /user/quickstart
to deploy your environment and test
your changes.
Please use the existing sandbox repository, available at sandbox, for learning, understanding and testing the Gerrit Workflow.
Adding a new service
Kolla aims to both containerise and deploy all services within the OpenStack "big tent". This is a constantly moving target as the ecosystem grows, so these guidelines aim to help make adding a new service to Kolla a smooth experience.
When adding a role for a new service in Ansible, there are couple of patterns that Kolla uses throughout that should be followed.
The sample inventories
Entries should be added for the service in each of
ansible/inventory/multinode
andansible/inventory/all-in-one
.The playbook
The main playbook that ties all roles together is in
ansible/site.yml
, this should be updated with appropriate roles, tags, and conditions. Ensure also that supporting hosts such as haproxy are updated when necessary.The common role
A
common
role exists which sets up logging,kolla-toolbox
and other supporting components. This should be included in all services withinmeta/main.yml
of your role.Common tasks
All services should include the following tasks:
reconfigure.yml
: Used to push new configuration files to the host and restart the service.pull.yml
: Used to pre fetch the image into the Docker image cache on hosts, to speed up initial deploys.upgrade.yml
: Used for upgrading the service in a rolling fashion. May include service specific setup and steps as not all services can be upgraded in the same way.
Logrotation
For OpenStack services there should be a
cron-logrotate-PROJECT.conf.j2
template file inansible/roles/common/templates
with the following content:"/var/log/kolla/PROJECT/*.log" { }
For OpenStack services there should be an entry in the
services
list in thecron.json.j2
template file inansible/roles/common/templates
.
Log delivery
- For OpenStack services the service should add a new
rewriterule
in thematch
element in the01-rewrite.conf.j2
template file inansible/roles/common/templates/conf/filter
to deliver log messages to Elasticsearch.
- For OpenStack services the service should add a new
Documentation
- For OpenStack services there should be an entry in the list
OpenStack services
in theREADME.rst
file. - For infrastructure services there should be an entry in the list
Infrastructure components
in theREADME.rst
file.
- For OpenStack services there should be an entry in the list
Syntax
- All YAML data files should start with three dashes
(
---
).
- All YAML data files should start with three dashes
(
Other than the above, most service roles abide by the following pattern:
Register
: Involves registering the service with Keystone, creating endpoints, roles, users, etc.Config
: Distributes the config files to the nodes to be pulled into the container on startup.Bootstrap
: Creating the database (but not tables), database user for the service, permissions, etc.Bootstrap Service
: Starts a one shot container on the host to create the database tables, and other initial run time config.Start
: Start the service(s).