Merge lp://staging/~codersquid/capomastro/autotracking-docs into lp://staging/capomastro

Proposed by Sheila Miguez
Status: Merged
Approved by: Caio Begotti
Approved revision: 180
Merged at revision: 170
Proposed branch: lp://staging/~codersquid/capomastro/autotracking-docs
Merge into: lp://staging/capomastro
Diff against target: 1186 lines (+869/-84)
16 files modified
debian/control (+1/-1)
docs/Makefile (+9/-2)
docs/doc-requirements.txt (+2/-0)
docs/source/apps.dot (+285/-0)
docs/source/archives.dot (+109/-0)
docs/source/archives.rst (+6/-1)
docs/source/conf.py (+2/-0)
docs/source/dependency.rst (+53/-18)
docs/source/development.rst (+11/-0)
docs/source/index.rst (+18/-14)
docs/source/jenkins.dot (+112/-0)
docs/source/jenkins.rst (+6/-0)
docs/source/project.dot (+142/-0)
docs/source/project.rst (+27/-5)
docs/source/setup.rst (+85/-42)
projects/models.py (+1/-1)
To merge this branch: bzr merge lp://staging/~codersquid/capomastro/autotracking-docs
Reviewer Review Type Date Requested Status
Caio Begotti (community) Approve
Review via email: mp+252795@code.staging.launchpad.net

Description of the change

This change covers some cleanup and some documentation clarifications on how dependency auto-tracking works.

Cleanup

* the 'default' theme of sphinx is deprecated. changed theme to alabaster
* nested development content like setup under a new development page
* corrected grammar and minor style issues

Content changes

* reorganized index
  - intro text is at the top, toc is at the bottom
  - toc is in a suggested order rather than alpha by files.
    I tried to pick an order that seemed reasonable, starting with
    the concept of projects, next to dependencies, and on through the
    rest of the concepts.
* made changes to project, dependency, setup
* added graphviz diagrams for the app models to corresponding app pages

I put this branches compiled files here http://10.55.32.85/capomastro/new/
and trunk here http://10.55.32.85/capomastro/old/

To post a comment you must log in.
Revision history for this message
Caio Begotti (caio1982) wrote :

Comments in the diff! :-D

review: Needs Fixing
Revision history for this message
Daniel Manrique (roadmr) wrote :

Some diff comments, see below.

Revision history for this message
Caio Begotti (caio1982) wrote :
Download full text (47.6 KiB)

We still need the 100G disk for Jenkins because we can have many
different projects and builds and each one requires a LXC instance,
which takes space, so in theory and with my bad calculation we could
have about 75 containers. I think there's an user story about me
adding storage support for /var/lib/lxc somewhere, this would solve
the problem and we could reduce the disk size to, say, 10G.

On Fri, Mar 13, 2015 at 1:26 PM, Daniel Manrique
<email address hidden> wrote:
> Some diff comments, see below.
>
> Diff comments:
>
>> === added file 'docs/doc-requirements.txt'
>> --- docs/doc-requirements.txt 1970-01-01 00:00:00 +0000
>> +++ docs/doc-requirements.txt 2015-03-12 17:53:21 +0000
>> @@ -0,0 +1,2 @@
>> +sphinx
>> +sphinxcontrib-blockdiag
>>
>> === added file 'docs/source/apps.dot'
>> --- docs/source/apps.dot 1970-01-01 00:00:00 +0000
>> +++ docs/source/apps.dot 2015-03-12 17:53:21 +0000
>> @@ -0,0 +1,285 @@
>> +digraph model_graph {
>> + // Dotfile by Django-Extensions graph_models
>> + // Created: 2015-03-11 19:21
>> + // Cli Options: --pygraphviz -d -g credentials jenkins archives projects
>> +
>> + fontname = "Helvetica"
>> + fontsize = 8
>> + splines = true
>> +
>> + node [
>> + fontname = "Helvetica"
>> + fontsize = 8
>> + shape = "plaintext"
>> + ]
>> +
>> + edge [
>> + fontname = "Helvetica"
>> + fontsize = 8
>> + ]
>> +
>> + // Labels
>> + subgraph cluster_credentials_models {
>> + label=<
>> + <TABLE BORDER="0" CELLBORDER="0" CELLSPACING="0">
>> + <TR><TD COLSPAN="2" CELLPADDING="4" ALIGN="CENTER">
>> + <FONT FACE="Helvetica Bold" COLOR="Black" POINT-SIZE="12">
>> + credentials
>> + </FONT>
>> + </TD></TR>
>> + </TABLE>
>> + >
>> + color=olivedrab4
>> + style="rounded"
>> +
>> + credentials_models_SshKeyPair [label=<
>> + <TABLE BGCOLOR="palegoldenrod" BORDER="0" CELLBORDER="0" CELLSPACING="0">
>> + <TR><TD COLSPAN="2" CELLPADDING="4" ALIGN="CENTER" BGCOLOR="olivedrab4">
>> + <FONT FACE="Helvetica Bold" COLOR="white">
>> + SshKeyPair
>> + </FONT></TD></TR>
>> +
>> + </TABLE>
>> + >]
>> +
>> + }
>> + subgraph cluster_jenkins_models {
>> + label=<
>> + <TABLE BORDER="0" CELLBORDER="0" CELLSPACING="0">
>> + <TR><TD COLSPAN="2" CELLPADDING="4" ALIGN="CENTER">
>> + <FONT FACE="Helvetica Bold" COLOR="Black" POINT-SIZE="12">
>> + jenkins
>> + </FONT>
>> + </TD></TR>
>> + </TABLE>
>> + >
>> + color=olivedrab4
>> + style="rounded"
>> +
>> + jenkins_models_JenkinsServer [label=<
>> + <TABLE BGCOLOR="palegoldenrod" BORDER="0" CELLBORDER="0" CELLSPACING="0">
>> + <TR><TD COLSPAN="2" CELLPADDING="4" ALIGN="CENTER" BGCOLOR="olivedrab4">
>> + <FONT FACE="Helvetica Bold" COLOR="white">
>> + JenkinsServer
>> + </FONT></TD></TR>
>> +
>> + </TABLE>
>> + >]
>> +
>> + jenkins_models_JobType [label=<
>> + <TABLE BGCOLOR="palegoldenrod" BORDER="0" CELLBORDER="0" CELLSPACING="0">
>> + <TR><TD COLSPAN="2" CELLPADDING="4" ALIGN="CENTER" BGCOLOR="olivedrab4">
>> ...

Revision history for this message
Sheila Miguez (codersquid) wrote :

I've checked in changes based on review comments, but I haven't checked in the changes yet to add build dependencies to the package. I think those will be: python-pygraphviz, python-blockdiag, sphinxcontrib.blockdiag, python-pygraphviz.

Revision history for this message
Sheila Miguez (codersquid) wrote :

I updated http://10.55.32.85/capomastro/new/ to show the docs without the alabaster theme.

Revision history for this message
Caio Begotti (caio1982) :
178. By Sheila Miguez

Updating per review, also added graphviz build opts

179. By Sheila Miguez

add build deps for building diagrams, make build fail without them

180. By Sheila Miguez

fixed packages

Revision history for this message
Sheila Miguez (codersquid) wrote :

Fixed!

Revision history for this message
Caio Begotti (caio1982) wrote :

Thanks Sheila!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches