2016-11-27 15:32:05 +00:00
<!DOCTYPE html>
<!-- [if IE 8]><html class="no - js lt - ie9" lang="en" > <![endif] -->
<!-- [if gt IE 8]><! --> < html class = "no-js" lang = "en" > <!-- <![endif] -->
< head >
< meta charset = "utf-8" >
< meta name = "viewport" content = "width=device-width, initial-scale=1.0" >
2017-07-20 19:22:04 +00:00
< title > 24. Hacking — cdist 4.5.0 documentation< / title >
2016-11-27 15:32:05 +00:00
2017-06-13 20:40:23 +00:00
2016-11-27 15:32:05 +00:00
< link rel = "stylesheet" href = "_static/css/theme.css" type = "text/css" / >
< link rel = "index" title = "Index"
href="genindex.html"/>
< link rel = "search" title = "Search" href = "search.html" / >
2017-07-20 19:22:04 +00:00
< link rel = "top" title = "cdist 4.5.0 documentation" href = "index.html" / >
2016-11-27 17:43:52 +00:00
< link rel = "next" title = "25. Troubleshooting" href = "cdist-troubleshooting.html" / >
< link rel = "prev" title = "23. Remote exec and copy commands" href = "cdist-remote-exec-copy.html" / >
2016-11-27 15:32:05 +00:00
< script src = "_static/js/modernizr.min.js" > < / script >
< / head >
< body class = "wy-body-for-nav" role = "document" >
2017-06-13 20:40:23 +00:00
2016-11-27 15:32:05 +00:00
< div class = "wy-grid-for-nav" >
< nav data-toggle = "wy-nav-shift" class = "wy-nav-side" >
< div class = "wy-side-scroll" >
< div class = "wy-side-nav-search" >
< a href = "index.html" class = "icon icon-home" > cdist
< / a >
< div class = "version" >
2017-07-20 19:22:04 +00:00
4.5.0
2016-11-27 15:32:05 +00:00
< / div >
< div role = "search" >
< form id = "rtd-search-form" class = "wy-form" action = "search.html" method = "get" >
< input type = "text" name = "q" placeholder = "Search docs" / >
< input type = "hidden" name = "check_keywords" value = "yes" / >
< input type = "hidden" name = "area" value = "default" / >
< / form >
< / div >
< / div >
< div class = "wy-menu wy-menu-vertical" data-spy = "affix" role = "navigation" aria-label = "main navigation" >
2017-06-13 20:40:23 +00:00
< ul class = "current" >
2016-11-27 15:32:05 +00:00
< li class = "toctree-l1" > < a class = "reference internal" href = "cdist-intro.html" > 1. cdist - usable configuration management< / a > < / li >
< li class = "toctree-l1" > < a class = "reference internal" href = "cdist-why.html" > 2. Why should I use cdist?< / a > < / li >
< li class = "toctree-l1" > < a class = "reference internal" href = "cdist-os.html" > 3. Supported Operating Systems< / a > < / li >
< li class = "toctree-l1" > < a class = "reference internal" href = "cdist-install.html" > 4. How to install cdist< / a > < / li >
< li class = "toctree-l1" > < a class = "reference internal" href = "cdist-update.html" > 5. How to update cdist< / a > < / li >
< li class = "toctree-l1" > < a class = "reference internal" href = "cdist-support.html" > 6. Support< / a > < / li >
< li class = "toctree-l1" > < a class = "reference internal" href = "cdist-features.html" > 7. Features< / a > < / li >
< li class = "toctree-l1" > < a class = "reference internal" href = "cdist-quickstart.html" > 8. Quickstart< / a > < / li >
< li class = "toctree-l1" > < a class = "reference internal" href = "man1/cdist.html" > 9. cdist(1)< / a > < / li >
< li class = "toctree-l1" > < a class = "reference internal" href = "cdist-bootstrap.html" > 10. Bootstrap< / a > < / li >
< li class = "toctree-l1" > < a class = "reference internal" href = "cdist-manifest.html" > 11. Manifest< / a > < / li >
< li class = "toctree-l1" > < a class = "reference internal" href = "cdist-type.html" > 12. cdist type< / a > < / li >
< li class = "toctree-l1" > < a class = "reference internal" href = "cdist-types.html" > 13. cdist types< / a > < / li >
< li class = "toctree-l1" > < a class = "reference internal" href = "cdist-explorer.html" > 14. Explorer< / a > < / li >
< li class = "toctree-l1" > < a class = "reference internal" href = "cdist-messaging.html" > 15. Messaging< / a > < / li >
< li class = "toctree-l1" > < a class = "reference internal" href = "cdist-parallelization.html" > 16. Parallelization< / a > < / li >
< li class = "toctree-l1" > < a class = "reference internal" href = "cdist-inventory.html" > 17. Inventory< / a > < / li >
2016-11-27 17:43:52 +00:00
< li class = "toctree-l1" > < a class = "reference internal" href = "cdist-trigger.html" > 18. Trigger< / a > < / li >
< li class = "toctree-l1" > < a class = "reference internal" href = "cdist-preos.html" > 19. PreOS< / a > < / li >
< li class = "toctree-l1" > < a class = "reference internal" href = "cdist-reference.html" > 20. Reference< / a > < / li >
< li class = "toctree-l1" > < a class = "reference internal" href = "cdist-best-practice.html" > 21. Best practice< / a > < / li >
< li class = "toctree-l1" > < a class = "reference internal" href = "cdist-stages.html" > 22. Execution stages< / a > < / li >
< li class = "toctree-l1" > < a class = "reference internal" href = "cdist-remote-exec-copy.html" > 23. Remote exec and copy commands< / a > < / li >
< li class = "toctree-l1 current" > < a class = "current reference internal" href = "#" > 24. Hacking< / a > < ul >
< li class = "toctree-l2" > < a class = "reference internal" href = "#welcome" > 24.1. Welcome< / a > < / li >
< li class = "toctree-l2" > < a class = "reference internal" href = "#reporting-bugs" > 24.2. Reporting bugs< / a > < / li >
< li class = "toctree-l2" > < a class = "reference internal" href = "#coding-conventions-everywhere" > 24.3. Coding conventions (everywhere)< / a > < / li >
< li class = "toctree-l2" > < a class = "reference internal" href = "#how-to-submit-stuff-for-inclusion-into-upstream-cdist" > 24.4. How to submit stuff for inclusion into upstream cdist< / a > < / li >
< li class = "toctree-l2" > < a class = "reference internal" href = "#how-to-submit-a-new-type" > 24.5. How to submit a new type< / a > < / li >
< li class = "toctree-l2" > < a class = "reference internal" href = "#example-git-workflow" > 24.6. Example git workflow< / a > < / li >
2016-11-27 15:32:05 +00:00
< / ul >
< / li >
2016-11-27 17:43:52 +00:00
< li class = "toctree-l1" > < a class = "reference internal" href = "cdist-troubleshooting.html" > 25. Troubleshooting< / a > < / li >
2016-11-27 15:32:05 +00:00
< / ul >
< / div >
< / div >
< / nav >
< section data-toggle = "wy-nav-shift" class = "wy-nav-content-wrap" >
< nav class = "wy-nav-top" role = "navigation" aria-label = "top navigation" >
2017-06-13 20:40:23 +00:00
< i data-toggle = "wy-nav-top" class = "fa fa-bars" > < / i >
< a href = "index.html" > cdist< / a >
2016-11-27 15:32:05 +00:00
< / nav >
< div class = "wy-nav-content" >
< div class = "rst-content" >
2017-06-13 20:40:23 +00:00
2016-11-27 15:32:05 +00:00
< div role = "navigation" aria-label = "breadcrumbs navigation" >
2017-06-13 20:40:23 +00:00
2016-11-27 15:32:05 +00:00
< ul class = "wy-breadcrumbs" >
2017-06-13 20:40:23 +00:00
< li > < a href = "index.html" > Docs< / a > » < / li >
< li > 24. Hacking< / li >
2016-11-27 15:32:05 +00:00
< li class = "wy-breadcrumbs-aside" >
2017-06-13 20:40:23 +00:00
< a href = "_sources/cdist-hacker.rst.txt" rel = "nofollow" > View page source< / a >
2016-11-27 15:32:05 +00:00
< / li >
2017-06-13 20:40:23 +00:00
2016-11-27 15:32:05 +00:00
< / ul >
2017-06-13 20:40:23 +00:00
2016-11-27 15:32:05 +00:00
< hr / >
< / div >
< div role = "main" class = "document" itemscope = "itemscope" itemtype = "http://schema.org/Article" >
< div itemprop = "articleBody" >
< div class = "section" id = "hacking" >
2016-11-27 17:43:52 +00:00
< h1 > 24. Hacking< a class = "headerlink" href = "#hacking" title = "Permalink to this headline" > ¶< / a > < / h1 >
2016-11-27 15:32:05 +00:00
< div class = "section" id = "welcome" >
2016-11-27 17:43:52 +00:00
< h2 > 24.1. Welcome< a class = "headerlink" href = "#welcome" title = "Permalink to this headline" > ¶< / a > < / h2 >
2016-11-27 15:32:05 +00:00
< p > Welcome dear hacker! I invite you to a tour of pointers to
2017-07-20 19:22:04 +00:00
get into the usable configuration management system, cdist.< / p >
2016-11-27 15:32:05 +00:00
< p > The first thing to know is probably that cdist is brought to
you by people who care about how code looks like and who think
twice before merging or implementing a feature: Less features
with good usability are far better than the opposite.< / p >
< / div >
< div class = "section" id = "reporting-bugs" >
2016-11-27 17:43:52 +00:00
< h2 > 24.2. Reporting bugs< a class = "headerlink" href = "#reporting-bugs" title = "Permalink to this headline" > ¶< / a > < / h2 >
2016-11-27 15:32:05 +00:00
< p > If you believe you've found a bug and verified that it is
in the latest version, drop a mail to the cdist mailing list,
subject prefixed with " [BUG] " or create an issue on github.< / p >
< / div >
< div class = "section" id = "coding-conventions-everywhere" >
2016-11-27 17:43:52 +00:00
< h2 > 24.3. Coding conventions (everywhere)< a class = "headerlink" href = "#coding-conventions-everywhere" title = "Permalink to this headline" > ¶< / a > < / h2 >
2017-07-20 19:22:04 +00:00
< p > If something should be improved or needs to be fixed, add the word FIXME
2016-11-27 15:32:05 +00:00
nearby, so grepping for FIXME gives all positions that need to be fixed.< / p >
2017-07-20 19:22:04 +00:00
< p > Indentation is 4 spaces (welcome to the python world).< / p >
2016-11-27 15:32:05 +00:00
< / div >
< div class = "section" id = "how-to-submit-stuff-for-inclusion-into-upstream-cdist" >
2016-11-27 17:43:52 +00:00
< h2 > 24.4. How to submit stuff for inclusion into upstream cdist< a class = "headerlink" href = "#how-to-submit-stuff-for-inclusion-into-upstream-cdist" title = "Permalink to this headline" > ¶< / a > < / h2 >
2017-07-20 19:22:04 +00:00
< p > If you did some cool changes to cdist, which you think might be of benefit to other
cdist users, you're welcome to propose inclusion into upstream.< / p >
< p > There are some requirements to ensure your changes don't break other peoples
2016-11-27 15:32:05 +00:00
work nor kill the authors brain:< / p >
< ul >
< li > < p class = "first" > All files should contain the usual header (Author, Copying, etc.)< / p >
< / li >
< li > < p class = "first" > Code submission must be done via git< / p >
< / li >
< li > < p class = "first" > Do not add cdist/conf/manifest/init - This file should only be touched in your
private branch!< / p >
< / li >
< li > < p class = "first" > Code to be included should be branched of the upstream " master" branch< / p >
< blockquote >
< div > < ul class = "simple" >
< li > Exception: Bugfixes to a version branch< / li >
< / ul >
< / div > < / blockquote >
< / li >
< li > < p class = "first" > On a merge request, always name the branch I should pull from< / p >
< / li >
< li > < p class = "first" > Always ensure < strong > all< / strong > manpages build. Use < strong > ./build man< / strong > to test.< / p >
< / li >
< li > < p class = "first" > If you developed more than < strong > one< / strong > feature, consider submitting them in
separate branches. This way one feature can already be included, even if
the other needs to be improved.< / p >
< / li >
< / ul >
< p > As soon as your work meets these requirements, write a mail
for inclusion to the mailinglist < strong > cdist-configuration-management at googlegroups.com< / strong >
or open a pull request at < a class = "reference external" href = "http://github.com/ungleich/cdist" > http://github.com/ungleich/cdist< / a > .< / p >
< / div >
< div class = "section" id = "how-to-submit-a-new-type" >
2016-11-27 17:43:52 +00:00
< h2 > 24.5. How to submit a new type< a class = "headerlink" href = "#how-to-submit-a-new-type" title = "Permalink to this headline" > ¶< / a > < / h2 >
2016-11-27 15:32:05 +00:00
< p > For detailed information about types, see < a class = "reference external" href = "cdist-type.html" > cdist type< / a > .< / p >
< p > Submitting a type works as described above, with the additional requirement
that a corresponding manpage named man.text in asciidoc format with
the manpage-name " cdist-type__NAME" is included in the type directory
AND asciidoc is able to compile it (i.e. do NOT have to many " =" in the second
line).< / p >
< p > Warning: Submitting " exec" or " run" types that simply echo their parameter in
< strong > gencode< / strong > will not be accepted, because they are of no use. Every type can output
code and thus such a type introduces redundant functionality that is given by
core cdist already.< / p >
< / div >
< div class = "section" id = "example-git-workflow" >
2016-11-27 17:43:52 +00:00
< h2 > 24.6. Example git workflow< a class = "headerlink" href = "#example-git-workflow" title = "Permalink to this headline" > ¶< / a > < / h2 >
2016-11-27 15:32:05 +00:00
< p > The following workflow works fine for most developers< / p >
< div class = "highlight-sh" > < div class = "highlight" > < pre > < span > < / span > < span class = "c1" > # get latest upstream master branch< / span >
git clone https://github.com/ungleich/cdist.git
< span class = "c1" > # update if already existing< / span >
< span class = "nb" > cd< / span > cdist< span class = "p" > ;< / span > git fetch -v< span class = "p" > ;< / span > git merge origin/master
< span class = "c1" > # create a new branch for your feature/bugfix< / span >
< span class = "nb" > cd< / span > cdist < span class = "c1" > # if you haven' t done before< / span >
git checkout -b documentation_cleanup
< span class = "c1" > # *hack*< / span >
*hack*
< span class = "c1" > # clone the cdist repository on github if you haven' t done so< / span >
< span class = "c1" > # configure your repo to know about your clone (only once)< / span >
git remote add github git@github.com:YOURUSERNAME/cdist.git
< span class = "c1" > # push the new branch to github< / span >
git push github documentation_cleanup
< span class = "c1" > # (or everything)< / span >
git push --mirror github
< span class = "c1" > # create a pull request at github (use a browser)< / span >
< span class = "c1" > # *fixthingsbecausequalityassurancefoundissuesinourpatch*< / span >
*hack*
< span class = "c1" > # push code to github again< / span >
git push ... < span class = "c1" > # like above< / span >
< span class = "c1" > # add comment that everything should be green now (use a browser)< / span >
< span class = "c1" > # go back to master branch< / span >
git checkout master
< span class = "c1" > # update master branch that includes your changes now< / span >
git fetch -v origin
git diff master..origin/master
git merge origin/master
< / pre > < / div >
< / div >
< p > If at any point you want to go back to the original master branch, you can
use < strong > git stash< / strong > to stash your changes away:< / p >
< div class = "highlight-default" > < div class = "highlight" > < pre > < span > < / span > < span class = "o" > ..< / span > < span class = "n" > code< / span > < span class = "o" > -< / span > < span class = "n" > block< / span > < span class = "p" > ::< / span > < span class = "n" > sh< / span >
< / pre > < / div >
< / div >
< blockquote >
< div > < p > # assume you are on documentation_cleanup
git stash< / p >
< p > # change to master and update to most recent upstream version
git checkout master
git fetch -v origin
git merge origin/master< / p >
< / div > < / blockquote >
2017-07-20 19:22:04 +00:00
< p > Similarly when you want to develop another new feature, you go back
2016-11-27 15:32:05 +00:00
to the master branch and create another branch based on it:< / p >
< div class = "highlight-default" > < div class = "highlight" > < pre > < span > < / span > < span class = "o" > ..< / span > < span class = "n" > code< / span > < span class = "o" > -< / span > < span class = "n" > block< / span > < span class = "p" > ::< / span > < span class = "n" > sh< / span >
< / pre > < / div >
< / div >
< blockquote >
< div > < p > # change to master and update to most recent upstream version
git checkout master
git fetch -v origin
git merge origin/master< / p >
< p > git checkout -b another_feature< / p >
< / div > < / blockquote >
< p > (you can repeat the code above for as many features as you want to develop
in parallel)< / p >
< / div >
< / div >
2017-06-13 20:40:23 +00:00
< / div >
< div class = "articleComments" >
2016-11-27 15:32:05 +00:00
< / div >
< / div >
< footer >
< div class = "rst-footer-buttons" role = "navigation" aria-label = "footer navigation" >
2017-06-13 20:40:23 +00:00
< a href = "cdist-troubleshooting.html" class = "btn btn-neutral float-right" title = "25. Troubleshooting" accesskey = "n" rel = "next" > Next < span class = "fa fa-arrow-circle-right" > < / span > < / a >
2016-11-27 15:32:05 +00:00
2017-06-13 20:40:23 +00:00
< a href = "cdist-remote-exec-copy.html" class = "btn btn-neutral" title = "23. Remote exec and copy commands" accesskey = "p" rel = "prev" > < span class = "fa fa-arrow-circle-left" > < / span > Previous< / a >
2016-11-27 15:32:05 +00:00
< / div >
< hr / >
< div role = "contentinfo" >
< p >
© Copyright .
< / p >
< / div >
Built with < a href = "http://sphinx-doc.org/" > Sphinx< / a > using a < a href = "https://github.com/snide/sphinx_rtd_theme" > theme< / a > provided by < a href = "https://readthedocs.org" > Read the Docs< / a > .
< / footer >
< / div >
< / div >
< / section >
< / div >
< script type = "text/javascript" >
var DOCUMENTATION_OPTIONS = {
URL_ROOT:'./',
2017-07-20 19:22:04 +00:00
VERSION:'4.5.0',
2016-11-27 15:32:05 +00:00
COLLAPSE_INDEX:false,
FILE_SUFFIX:'.html',
2017-06-13 20:40:23 +00:00
HAS_SOURCE: true,
SOURCELINK_SUFFIX: '.txt'
2016-11-27 15:32:05 +00:00
};
< / script >
< script type = "text/javascript" src = "_static/jquery.js" > < / script >
< script type = "text/javascript" src = "_static/underscore.js" > < / script >
< script type = "text/javascript" src = "_static/doctools.js" > < / script >
< script type = "text/javascript" src = "_static/js/theme.js" > < / script >
< script type = "text/javascript" >
jQuery(function () {
SphinxRtdTheme.StickyNav.enable();
});
< / script >
< / body >
< / html >
