summaryrefslogtreecommitdiffstats
path: root/docs
Commit message (Collapse)AuthorAgeFilesLines
* Migrate to import_role for static role inclusionScott Dodson2018-01-052-11/+11
| | | | | | | | | | | | | | | | | | | | | | | In Ansible 2.2, the include_role directive came into existence as a Tech Preview. It is still a Tech Preview through Ansible 2.4 (and in current devel branch), but with a noteable change. The default behavior switched from static: true to static: false because that functionality moved to the newly introduced import_role directive (in order to stay consistent with include* being dynamic in nature and `import* being static in nature). The dynamic include is considerably more memory intensive as it will dynamically create a role import for every host in the inventory list to be used. (Also worth noting, there is at the time of this writing an object allocation inefficiency in the dynamic include that can in certain situations amplify this effect considerably) This change is meant to mitigate the pressure on memory for the Ansible control host. We need to evaluate where it makes sense to dynamically include roles and revert back to dynamic inclusion if and where it makes sense to do so.
* Remove all uses of openshift.common.admin_binaryScott Dodson2017-12-071-1/+1
| | | | Replace with `oc adm`
* Proposal: container_runtime roleMichael Gugino2017-11-071-0/+113
| | | | | | This is a proposal meant to facilitate discussion. Input is welcome.
* Use "requests" for CPU resources instead of limitsPeter Portante2017-10-191-3/+3
| | | | | | | | | | | | | | | We now use a CPU request to ensure logging infrastructure pods are not capped by default for CPU usage. It is still important to ensure we have a minimum amount of CPU. We keep the use of the variables *_cpu_limit so that the existing behavior is maintained. Note that we don't want to cap an infra pod's CPU usage by default, since we want to be able to use the necessary resources to complete it's tasks. Bug 1501960 (https://bugzilla.redhat.com/show_bug.cgi?id=1501960)
* Merge pull request #5322 from mtnbikenc/proposal-playbook-consolidationScott Dodson2017-09-201-0/+178
|\ | | | | [Proposal] OpenShift-Ansible Playbook Consolidation
| * Rework openshift-cluster into deploy_cluster.ymlRussell Teague2017-09-131-20/+15
| |
| * [Proposal] OpenShift-Ansible Playbook ConsolidationRussell Teague2017-09-061-0/+183
| |
* | [Proposal] OpenShift-Ansible Proposal ProcessRussell Teague2017-09-112-0/+57
|/
* Remove unsupported playbooks and utilitiesRussell Teague2017-08-251-6/+0
|
* Merge pull request #3813 from ewolinetz/role_decomposition_proposalScott Dodson2017-06-301-0/+353
|\ | | | | Creating initial proposal doc for review
| * Adding some more sections to additional considerations, being less rigid on ↵ewolinetz2017-04-271-2/+40
| | | | | | | | large roles for composing -- can also be a playbook
| * Updating snippet contents, formatting and providing urlsewolinetz2017-04-211-35/+28
| |
| * Update snippets and add bullet point on role dependencyewolinetz2017-04-051-0/+81
| |
| * Creating initial proposal doc for reviewewolinetz2017-03-301-0/+241
| |
* | docs: better documentation for PAPRJonathan Lebon2017-06-142-4/+17
| | | | | | | | | | | | | | Add a section in `repo_structure.md`, and rename from `redhat-ci` to `PAPR` and point to the new upstream repo in `pull_requests.md`. Closes: #4078
* | Merge pull request #4129 from rhcarvalho/document-redhat-ci-jobScott Dodson2017-05-101-0/+9
|\ \ | | | | | | Note existence of Fedora tests and how to rerun
| * | Note existence of Fedora tests and how to rerunRodolfo Carvalho2017-05-091-0/+9
| | |
* | | Remove vim configuration from Python filesRodolfo Carvalho2017-05-091-19/+0
|/ / | | | | | | | | | | | | In a project where contributors are free to use whatever editor they want and we have linting tools that verify the proper formatting of Python files, it should not be required to have a vim-specific line in Python files.
* | Don't double quote when conditionsScott Dodson2017-05-011-2/+2
| |
* | Document the Pull Request processRodolfo Carvalho2017-04-271-0/+86
| |
* | Move repo structure to a separate documentRodolfo Carvalho2017-04-271-0/+54
| | | | | | | | Reduces the clutter in CONTRIBUTING.md.
* | Remove outdated information about PRsRodolfo Carvalho2017-04-271-16/+0
|/ | | | | | | The contents are outdated and irrelevant since the "best practice" is automatically enforced. Documenting the PR flow is out of the Best Practices guide.
* Refactor to use Ansible package moduleRussell Teague2016-11-171-23/+15
| | | | | The Ansible package module will call the correct package manager for the underlying OS.
* Corrected syntax and typosRussell Teague2016-11-161-6/+6
|
* Fix typosRodolfo Carvalho2016-10-192-2/+2
|
* Added anchors for rules in style_guide.adoc in order to make it easier to ↵Thomas Wiest2016-01-151-6/+12
| | | | reference specific rules in PRs.
* added anchorsThomas Wiest2016-01-141-22/+45
|
* Remove yum / dnf duplicationScott Dodson2015-12-091-0/+47
|
* Changed the string Master to master to make it more readable.Avesh Agarwal2015-08-131-1/+1
|
* Changes to make documentation less specific to OSE or AE and alsoAvesh Agarwal2015-08-131-1/+1
| | | | adds README_AEP.md.
* added decisions made at the last ansible arch meeting.Thomas Wiest2015-07-222-6/+39
|
* documented ansible arch team decisionsThomas Wiest2015-07-082-4/+159
|
* Added using Yaml syntax to best practices guide.Thomas Wiest2015-06-241-0/+52
|
* Merge pull request #298 from twiest/aa2Thomas Wiest2015-06-241-0/+26
|\ | | | | added python new method params should use a default value to best practices guide.
| * added python new method params should use a default value to best practices ↵Thomas Wiest2015-06-241-0/+26
| | | | | | | | guide.
* | added .yml extension to style guide.Thomas Wiest2015-06-241-0/+14
|/
* changed Openshift to OpenShiftThomas Wiest2015-06-193-3/+3
|
* Added YAML over JSON to best practices guide.Thomas Wiest2015-06-041-2/+24
|
* Merge pull request #258 from twiest/prThomas Wiest2015-06-041-0/+1
|\ | | | | Added 'stylistic exception' to the best practices guide.
| * Added 'stylistic exception' to the best practices guide.Thomas Wiest2015-06-031-0/+1
| |
* | Added fail pattern rules to best practices docThomas Wiest2015-06-031-2/+44
|/
* added section for role naming to the best practices guide.Thomas Wiest2015-05-271-1/+15
|
* Added 80 character SHOULD to style guide, added a better example for ↵Thomas Wiest2015-05-273-21/+70
|\ | | | | | | environment in core concepts, added ansible roles flat hierarchy to best practices
| * removed references to 'we'Thomas Wiest2015-05-263-10/+10
|/
* Added concepts guide.Thomas Wiest2015-05-263-1/+43
|
* Added style and best practices guidesThomas Wiest2015-05-222-0/+229