mirror of
https://github.com/CartoDB/crankshaft.git
synced 2024-11-01 10:20:48 +08:00
Reorganize the documentation into separate files
Keep a "Quickstart Guide" in the README, add separate detailed sections for development (CONTRIBUTING) and release/deployment (RELEASE).
This commit is contained in:
parent
b5281d0681
commit
9f03a9b075
90
CONTRIBUTING.md
Normal file
90
CONTRIBUTING.md
Normal file
@ -0,0 +1,90 @@
|
|||||||
|
# Development process
|
||||||
|
|
||||||
|
Please read the Working Process/Quickstart Guide in README.md first.
|
||||||
|
|
||||||
|
For any modification of crankshaft, such as adding new features,
|
||||||
|
refactoring or bug-fixing, topic branch must be created out of the `develop`
|
||||||
|
branch and be used for the development process.
|
||||||
|
|
||||||
|
Modifications are done inside `src/pg/sql` and `src/py/crankshaft`.
|
||||||
|
|
||||||
|
Take into account:
|
||||||
|
|
||||||
|
* Tests must be added for any new functionality
|
||||||
|
(inside `src/pg/test`, `src/py/crankshaft/test`) as well as to
|
||||||
|
detect any bugs that are being fixed.
|
||||||
|
* Add or modify the corresponding documentation files in the `doc` folder.
|
||||||
|
Since we expect to have highly technical functions here, an extense
|
||||||
|
background explanation would be of great help to users of this extension.
|
||||||
|
* Convention: snake case(i.e. `snake_case` and not `CamelCase`)
|
||||||
|
shall be used for all function names.
|
||||||
|
Prefix function names intended for public use with `cdb_`
|
||||||
|
and private functions (to be used only internally inside
|
||||||
|
the extension) with `_cdb_`.
|
||||||
|
|
||||||
|
Once the code is ready to be tested, update the local development installation
|
||||||
|
with `sudo make install`.
|
||||||
|
This will update the 'dev' version of the extension in `src/pg/` and
|
||||||
|
make it available to PostgreSQL.
|
||||||
|
It will also install the python package (crankshaft) in a virtual
|
||||||
|
environment `env/dev`.
|
||||||
|
|
||||||
|
The version number of the Python package, defined in
|
||||||
|
`src/pg/crankshaft/setup.py` will be overridden when
|
||||||
|
the package is released and always match the extension version number,
|
||||||
|
but for development it shall be kept as '0.0.0'.
|
||||||
|
|
||||||
|
Run the tests with `make test`.
|
||||||
|
|
||||||
|
To use the python extension for custom tests, activate the virtual
|
||||||
|
environment with:
|
||||||
|
|
||||||
|
```
|
||||||
|
source envs/dev/bin/activate
|
||||||
|
```
|
||||||
|
|
||||||
|
Update extension in a working database with:
|
||||||
|
|
||||||
|
* `ALTER EXTENSION crankshaft VERSION TO 'current';`
|
||||||
|
`ALTER EXTENSION crankshaft VERSION TO 'dev';`
|
||||||
|
|
||||||
|
Note: we keep the current development version install as 'dev' always;
|
||||||
|
we update through the 'current' alias to allow changing the extension
|
||||||
|
contents but not the version identifier. This will fail if the
|
||||||
|
changes involve incompatible function changes such as a different
|
||||||
|
return type; in that case the offending function (or the whole extension)
|
||||||
|
should be dropped manually before the update.
|
||||||
|
|
||||||
|
If the extension has not previously been installed in a database,
|
||||||
|
it can be installed directly with:
|
||||||
|
|
||||||
|
* `CREATE EXTENSION crankshaft WITH VERSION 'dev';`
|
||||||
|
|
||||||
|
Note: the development extension uses the development python virtual
|
||||||
|
environment automatically.
|
||||||
|
|
||||||
|
Before proceeding to the release process peer code reviewing of the code is
|
||||||
|
a must.
|
||||||
|
|
||||||
|
Once the feature or bugfix is completed, all the tests are passing
|
||||||
|
and the code has been accepted by peer reviewing,
|
||||||
|
the topic branch can be merged back into the `develop` branch and a
|
||||||
|
new Pull-Request can be created on it.
|
||||||
|
CI-tests must be checked to be successful.
|
||||||
|
|
||||||
|
The release manage will take hold of the PR at this moment to proceed
|
||||||
|
to the release process for a new revision of the extension.
|
||||||
|
|
||||||
|
## Relevant development tasks available in the Makefile
|
||||||
|
|
||||||
|
```
|
||||||
|
* `make help` show a short description of the available targets
|
||||||
|
|
||||||
|
* `sudo make install` will generate the extension scripts for the development
|
||||||
|
version ('dev'/'current') and install the python package into the
|
||||||
|
development virtual environment `envs/dev`.
|
||||||
|
Intended for use by developers.
|
||||||
|
|
||||||
|
* `make test` will run the tests for the installed development extension.
|
||||||
|
Intended for use by developers.
|
||||||
|
```
|
43
DEPLOYING.md
43
DEPLOYING.md
@ -1,43 +0,0 @@
|
|||||||
# Workflow
|
|
||||||
|
|
||||||
... (branching/merging flow)
|
|
||||||
|
|
||||||
# Deployment
|
|
||||||
|
|
||||||
...
|
|
||||||
|
|
||||||
Deployment to db servers: the next command will install both the Python
|
|
||||||
package and the extension.
|
|
||||||
|
|
||||||
```
|
|
||||||
sudo make install
|
|
||||||
```
|
|
||||||
|
|
||||||
Installing only the Python package:
|
|
||||||
|
|
||||||
```
|
|
||||||
sudo pip install python/crankshaft --upgrade
|
|
||||||
```
|
|
||||||
|
|
||||||
Caveat: note that `pip install ./crankshaft` will install
|
|
||||||
from local files, but `pip install crankshaft` will not.
|
|
||||||
|
|
||||||
CI: Install and run the tests on the installed extension and package:
|
|
||||||
|
|
||||||
```
|
|
||||||
(sudo make install && PGUSER=postgres make testinstalled)
|
|
||||||
```
|
|
||||||
|
|
||||||
Installing the extension in user databases:
|
|
||||||
Once installed in a server, the extension can be added
|
|
||||||
to a database with the next SQL command:
|
|
||||||
|
|
||||||
```
|
|
||||||
CREATE EXTENSION crankshaft;
|
|
||||||
```
|
|
||||||
|
|
||||||
To upgrade the extension to an specific version X.Y.Z:
|
|
||||||
|
|
||||||
```
|
|
||||||
ALTER EXTENSION crankshaft UPGRADE TO 'X.Y.Z';
|
|
||||||
```
|
|
186
README.md
186
README.md
@ -16,7 +16,7 @@ CartoDB Spatial Analysis extension for PostgreSQL.
|
|||||||
* pip, virtualenv, PostgreSQL
|
* pip, virtualenv, PostgreSQL
|
||||||
* python-scipy system package (see src/py/README.md)
|
* python-scipy system package (see src/py/README.md)
|
||||||
|
|
||||||
# Working Process
|
# Working Process -- Quickstart Guide
|
||||||
|
|
||||||
We distinguish two roles regarding the development cycle of crankshaft:
|
We distinguish two roles regarding the development cycle of crankshaft:
|
||||||
|
|
||||||
@ -41,180 +41,32 @@ who will merge into master where new releases are prepared and tagged.
|
|||||||
The `master` branch is the sole responsibility of the release masters
|
The `master` branch is the sole responsibility of the release masters
|
||||||
and developers must not commit or merge into it.
|
and developers must not commit or merge into it.
|
||||||
|
|
||||||
## Development
|
## Development Guidelines
|
||||||
|
|
||||||
For any modification of crankshaft, such as adding new features,
|
For a detailed description of the development process please see
|
||||||
refactoring or bug-fixing, topic branch must be created out of the `develop`
|
the CONTRIBUTING.md guide.
|
||||||
branch and be used for the development process.
|
|
||||||
|
|
||||||
Modifications are done inside `src/pg/sql` and `src/py/crankshaft`.
|
Any modification to the source code (`src/pg/sql` for the SQL extension,
|
||||||
|
`src/py/crankshaft` for the Python package) shall always be done
|
||||||
|
in a topic branch created from the `develop` branch.
|
||||||
|
|
||||||
Take into account:
|
Tests, documentation and peer code reviewing are required for all
|
||||||
|
modifications.
|
||||||
|
|
||||||
* Tests must be added for any new functionality
|
The tests (both for SQL and Pyhton) are executed by running,
|
||||||
(inside `src/pg/test`, `src/py/crankshaft/test`) as well as to
|
from the top directory:
|
||||||
detect any bugs that are being fixed.
|
|
||||||
* Add or modify the corresponding documentation files in the `doc` folder.
|
|
||||||
Since we expect to have highly technical functions here, an extense
|
|
||||||
background explanation would be of great help to users of this extension.
|
|
||||||
* Convention: snake case(i.e. `snake_case` and not `CamelCase`)
|
|
||||||
shall be used for all function names.
|
|
||||||
Prefix function names intended for public use with `cdb_`
|
|
||||||
and private functions (to be used only internally inside
|
|
||||||
the extension) with `_cdb_`.
|
|
||||||
|
|
||||||
Once the code is ready to be tested, update the local development installation
|
|
||||||
with `sudo make install`.
|
|
||||||
This will update the 'dev' version of the extension in `src/pg/` and
|
|
||||||
make it available to PostgreSQL.
|
|
||||||
It will also install the python package (crankshaft) in a virtual
|
|
||||||
environment `env/dev`.
|
|
||||||
|
|
||||||
The version number of the Python package, defined in
|
|
||||||
`src/pg/crankshaft/setup.py` will be overridden when
|
|
||||||
the package is released and always match the extension version number,
|
|
||||||
but for development it shall be kept as '0.0.0'.
|
|
||||||
|
|
||||||
Run the tests with `make test`.
|
|
||||||
|
|
||||||
To use the python extension for custom tests, activate the virtual
|
|
||||||
environment with:
|
|
||||||
|
|
||||||
```
|
```
|
||||||
source envs/dev/bin/activate
|
sudo make install
|
||||||
|
make test
|
||||||
```
|
```
|
||||||
|
|
||||||
Update extension in a working database with:
|
To request a new release, which will be handled by them
|
||||||
|
release manager, a Pull Request must be created in the `develop`
|
||||||
* `ALTER EXTENSION crankshaft VERSION TO 'current';`
|
branch.
|
||||||
`ALTER EXTENSION crankshaft VERSION TO 'dev';`
|
|
||||||
|
|
||||||
Note: we keep the current development version install as 'dev' always;
|
|
||||||
we update through the 'current' alias to allow changing the extension
|
|
||||||
contents but not the version identifier. This will fail if the
|
|
||||||
changes involve incompatible function changes such as a different
|
|
||||||
return type; in that case the offending function (or the whole extension)
|
|
||||||
should be dropped manually before the update.
|
|
||||||
|
|
||||||
If the extension has not previously been installed in a database,
|
|
||||||
it can be installed directly with:
|
|
||||||
|
|
||||||
* `CREATE EXTENSION crankshaft WITH VERSION 'dev';`
|
|
||||||
|
|
||||||
Note: the development extension uses the development python virtual
|
|
||||||
environment automatically.
|
|
||||||
|
|
||||||
Before proceeding to the release process peer code reviewing of the code is
|
|
||||||
a must.
|
|
||||||
|
|
||||||
Once the feature or bugfix is completed, all the tests are passing
|
|
||||||
and the code has been accepted by peer reviewing,
|
|
||||||
the topic branch can be merged back into the `develop` branch and a
|
|
||||||
new Pull-Request can be created on it.
|
|
||||||
CI-tests must be checked to be successful.
|
|
||||||
|
|
||||||
The release manage will take hold of the PR at this moment to proceed
|
|
||||||
to the release process for a new revision of the extension.
|
|
||||||
|
|
||||||
## Release
|
## Release
|
||||||
|
|
||||||
The release process of a new version of the extension
|
The release and deployment process is described in the
|
||||||
shall be performed by the designated *Release Manager*.
|
RELEASE.md guide and it is the responsibility of the designated
|
||||||
|
release manager.
|
||||||
Note that we expect to gradually automate more of this process.
|
|
||||||
|
|
||||||
Having checked PR to be released it shall be
|
|
||||||
merged back into the `master` branch to prepare the new release.
|
|
||||||
|
|
||||||
The version number in `pg/cranckshaft.control` must first be updated.
|
|
||||||
To do so [Semantic Versioning 2.0](http://semver.org/) is in order.
|
|
||||||
|
|
||||||
Thew `NEWS.md` will be updated.
|
|
||||||
|
|
||||||
We now will explain the process for the case of backwards-compatible
|
|
||||||
releases (updating the minor or patch version numbers).
|
|
||||||
|
|
||||||
TODO: document the complex case of major releases.
|
|
||||||
|
|
||||||
The next command must be executed to produce the main installation
|
|
||||||
script for the new release, `release/cranckshaft--X.Y.Z.sql` and
|
|
||||||
also to copy the python package to `release/python/X.Y.Z/crankshaft`.
|
|
||||||
|
|
||||||
```
|
|
||||||
make release
|
|
||||||
```
|
|
||||||
|
|
||||||
Then, the release manager shall produce upgrade and downgrade scripts
|
|
||||||
to migrate to/from the previous release. In the case of minor/patch
|
|
||||||
releases this simply consist in extracting the functions that have changed
|
|
||||||
and placing them in the proper `release/cranckshaft--X.Y.Z--A.B.C.sql`
|
|
||||||
file.
|
|
||||||
|
|
||||||
The new release can be deployed for staging/smoke tests with this command:
|
|
||||||
|
|
||||||
```
|
|
||||||
sudo make deploy
|
|
||||||
```
|
|
||||||
|
|
||||||
This will copy the current 'X.Y.Z' released version of the extension to
|
|
||||||
PostgreSQL. The corresponding Python extension will be installed in a
|
|
||||||
virtual environment in `envs/X.Y.Z`.
|
|
||||||
|
|
||||||
It can be activated with:
|
|
||||||
|
|
||||||
```
|
|
||||||
source envs/X.Y.Z/bin/activate
|
|
||||||
```
|
|
||||||
|
|
||||||
But note that this is needed only for using the package directly;
|
|
||||||
the 'X.Y.Z' version of the extension will automatically use the
|
|
||||||
python package from this virtual environment.
|
|
||||||
|
|
||||||
The `sudo make deploy` operation can be also used for installing
|
|
||||||
the new version after it has been released.
|
|
||||||
|
|
||||||
To install a specific version 'X.Y.Z' different from the current one
|
|
||||||
(which must be present in `releases/`) you can:
|
|
||||||
|
|
||||||
```
|
|
||||||
sudo make deploy RELEASE_VERSION=X.Y.Z
|
|
||||||
```
|
|
||||||
|
|
||||||
TODO: testing procedure for the new release.
|
|
||||||
|
|
||||||
TODO: procedure for staging deployment.
|
|
||||||
|
|
||||||
TODO: procedure for merging to master, tagging and deploying
|
|
||||||
in production.
|
|
||||||
|
|
||||||
## Relevant tasks available in the Makefile
|
|
||||||
|
|
||||||
```
|
|
||||||
* `make help` show a short description of the available targets
|
|
||||||
|
|
||||||
# Development tasks
|
|
||||||
|
|
||||||
* `sudo make install` will generate the extension scripts for the development
|
|
||||||
version ('dev'/'current') and install the python package into the
|
|
||||||
development virtual environment `envs/dev`.
|
|
||||||
Intended for use by developers.
|
|
||||||
|
|
||||||
* `make test` will run the tests for the installed development extension.
|
|
||||||
Intended for use by developers.
|
|
||||||
|
|
||||||
# Release tasks
|
|
||||||
|
|
||||||
* `make release` will generate a new release (version number defined in
|
|
||||||
`src/pg/crankshaft.control`) into `release/`.
|
|
||||||
Intended for use by the release manager.
|
|
||||||
|
|
||||||
* `sudo make deploy` will install the current release X.Y.Z from the
|
|
||||||
`release/` files into PostgreSQL and a Python virtual environment
|
|
||||||
`envs/X.Y.Z`.
|
|
||||||
Intended for use by the release manager and deployment jobs.
|
|
||||||
|
|
||||||
* `sudo make deploy RELEASE_VERSION=X.Y.Z` will install specified version
|
|
||||||
previously generated in `release/`
|
|
||||||
into PostgreSQL and a Python virtual environment `envs/X.Y.Z`.
|
|
||||||
Intended for use by the release manager and deployment jobs.
|
|
||||||
```
|
|
||||||
|
93
RELEASE.md
Normal file
93
RELEASE.md
Normal file
@ -0,0 +1,93 @@
|
|||||||
|
# Release & Deployment Process
|
||||||
|
|
||||||
|
Please read the Working Process/Quickstart Guide in README.md
|
||||||
|
and the Development guidelines in CONTRIBUTING.md.
|
||||||
|
|
||||||
|
The release process of a new version of the extension
|
||||||
|
shall be performed by the designated *Release Manager*.
|
||||||
|
|
||||||
|
Note that we expect to gradually automate more of this process.
|
||||||
|
|
||||||
|
Having checked PR to be released it shall be
|
||||||
|
merged back into the `master` branch to prepare the new release.
|
||||||
|
|
||||||
|
The version number in `pg/cranckshaft.control` must first be updated.
|
||||||
|
To do so [Semantic Versioning 2.0](http://semver.org/) is in order.
|
||||||
|
|
||||||
|
Thew `NEWS.md` will be updated.
|
||||||
|
|
||||||
|
We now will explain the process for the case of backwards-compatible
|
||||||
|
releases (updating the minor or patch version numbers).
|
||||||
|
|
||||||
|
TODO: document the complex case of major releases.
|
||||||
|
|
||||||
|
The next command must be executed to produce the main installation
|
||||||
|
script for the new release, `release/cranckshaft--X.Y.Z.sql` and
|
||||||
|
also to copy the python package to `release/python/X.Y.Z/crankshaft`.
|
||||||
|
|
||||||
|
```
|
||||||
|
make release
|
||||||
|
```
|
||||||
|
|
||||||
|
Then, the release manager shall produce upgrade and downgrade scripts
|
||||||
|
to migrate to/from the previous release. In the case of minor/patch
|
||||||
|
releases this simply consist in extracting the functions that have changed
|
||||||
|
and placing them in the proper `release/cranckshaft--X.Y.Z--A.B.C.sql`
|
||||||
|
file.
|
||||||
|
|
||||||
|
The new release can be deployed for staging/smoke tests with this command:
|
||||||
|
|
||||||
|
```
|
||||||
|
sudo make deploy
|
||||||
|
```
|
||||||
|
|
||||||
|
This will copy the current 'X.Y.Z' released version of the extension to
|
||||||
|
PostgreSQL. The corresponding Python extension will be installed in a
|
||||||
|
virtual environment in `envs/X.Y.Z`.
|
||||||
|
|
||||||
|
It can be activated with:
|
||||||
|
|
||||||
|
```
|
||||||
|
source envs/X.Y.Z/bin/activate
|
||||||
|
```
|
||||||
|
|
||||||
|
But note that this is needed only for using the package directly;
|
||||||
|
the 'X.Y.Z' version of the extension will automatically use the
|
||||||
|
python package from this virtual environment.
|
||||||
|
|
||||||
|
The `sudo make deploy` operation can be also used for installing
|
||||||
|
the new version after it has been released.
|
||||||
|
|
||||||
|
To install a specific version 'X.Y.Z' different from the current one
|
||||||
|
(which must be present in `releases/`) you can:
|
||||||
|
|
||||||
|
```
|
||||||
|
sudo make deploy RELEASE_VERSION=X.Y.Z
|
||||||
|
```
|
||||||
|
|
||||||
|
TODO: testing procedure for the new release.
|
||||||
|
|
||||||
|
TODO: procedure for staging deployment.
|
||||||
|
|
||||||
|
TODO: procedure for merging to master, tagging and deploying
|
||||||
|
in production.
|
||||||
|
|
||||||
|
## Relevant release & deployment tasks available in the Makefile
|
||||||
|
|
||||||
|
```
|
||||||
|
* `make help` show a short description of the available targets
|
||||||
|
|
||||||
|
* `make release` will generate a new release (version number defined in
|
||||||
|
`src/pg/crankshaft.control`) into `release/`.
|
||||||
|
Intended for use by the release manager.
|
||||||
|
|
||||||
|
* `sudo make deploy` will install the current release X.Y.Z from the
|
||||||
|
`release/` files into PostgreSQL and a Python virtual environment
|
||||||
|
`envs/X.Y.Z`.
|
||||||
|
Intended for use by the release manager and deployment jobs.
|
||||||
|
|
||||||
|
* `sudo make deploy RELEASE_VERSION=X.Y.Z` will install specified version
|
||||||
|
previously generated in `release/`
|
||||||
|
into PostgreSQL and a Python virtual environment `envs/X.Y.Z`.
|
||||||
|
Intended for use by the release manager and deployment jobs.
|
||||||
|
```
|
Loading…
Reference in New Issue
Block a user