|
| 1 | +# Contributing to OpenShift Agent Install Helper |
| 2 | + |
| 3 | +Thank you for your interest in contributing to the OpenShift Agent Install Helper project! This document provides a quick overview of the contribution process. |
| 4 | + |
| 5 | +## Repository Overview |
| 6 | + |
| 7 | +Please read our [README.md](README.md) first to understand: |
| 8 | +- Project purpose and scope |
| 9 | +- Key components and utilities |
| 10 | +- Prerequisites and dependencies |
| 11 | +- Basic usage instructions |
| 12 | + |
| 13 | +## Repository Structure |
| 14 | + |
| 15 | +``` |
| 16 | +openshift-agent-install/ |
| 17 | +├── README.md # Main project documentation |
| 18 | +├── get-rhcos-iso.sh # RHCOS ISO download utility |
| 19 | +├── download-openshift-cli.sh # OpenShift CLI download utility |
| 20 | +├── disconnected-info.md # Disconnected installation guide |
| 21 | +├── docs/ # Documentation website |
| 22 | +├── examples/ # Example configurations |
| 23 | +│ ├── baremetal-example/ |
| 24 | +│ ├── vmware-example/ |
| 25 | +│ └── sno-examples/ |
| 26 | +├── playbooks/ # Ansible automation |
| 27 | +├── scripts/ # Utility scripts |
| 28 | +├── e2e-tests/ # End-to-end tests |
| 29 | +├── hack/ # Development scripts |
| 30 | +└── site-config/ # Site configuration |
| 31 | +
|
| 32 | +``` |
| 33 | + |
| 34 | +## Quick Start for Contributors |
| 35 | + |
| 36 | +1. Read the [README.md](README.md) thoroughly |
| 37 | +2. Fork the repository |
| 38 | +3. Clone your fork: |
| 39 | + ```bash |
| 40 | + git clone https://github.com/YOUR_USERNAME/openshift-agent-install.git |
| 41 | + cd openshift-agent-install |
| 42 | + ``` |
| 43 | +4. Create a feature branch: |
| 44 | + ```bash |
| 45 | + git checkout -b feature/your-feature-name |
| 46 | + ``` |
| 47 | + |
| 48 | +## Development Prerequisites |
| 49 | + |
| 50 | +As specified in README.md: |
| 51 | +- RHEL/CentOS system |
| 52 | +- OpenShift CLI tools (`./download-openshift-cli.sh`) |
| 53 | +- NMState CLI (`dnf install nmstate`) |
| 54 | +- Ansible Core (`dnf install ansible-core`) |
| 55 | +- Red Hat OpenShift Pull Secret |
| 56 | + |
| 57 | +For documentation work: |
| 58 | +- Ruby 3.2+ and Bundler (see docs/README.md) |
| 59 | + |
| 60 | +## Making Changes |
| 61 | + |
| 62 | +### For Code Changes |
| 63 | +1. Follow the examples in the `examples/` directory |
| 64 | +2. Use the playbooks in `playbooks/` as templates |
| 65 | +3. Add tests in `e2e-tests/` for new features |
| 66 | +4. Update relevant documentation |
| 67 | + |
| 68 | +### For Documentation |
| 69 | +1. Website changes go in `docs/` |
| 70 | +2. Core project documentation updates in: |
| 71 | + - README.md |
| 72 | + - disconnected-info.md |
| 73 | + - Example READMEs |
| 74 | + |
| 75 | +## Testing Your Changes |
| 76 | + |
| 77 | +1. For code changes: |
| 78 | + ```bash |
| 79 | + # Run end-to-end tests |
| 80 | + cd e2e-tests |
| 81 | + ./run-tests.sh |
| 82 | + |
| 83 | + # Test specific playbooks |
| 84 | + cd playbooks |
| 85 | + ansible-playbook your-playbook.yml |
| 86 | + ``` |
| 87 | + |
| 88 | +2. For documentation: |
| 89 | + ```bash |
| 90 | + # Test documentation site |
| 91 | + cd docs |
| 92 | + bundle exec jekyll serve |
| 93 | + ``` |
| 94 | + |
| 95 | +## Pull Request Process |
| 96 | + |
| 97 | +1. Update README.md if you've added: |
| 98 | + - New prerequisites |
| 99 | + - New scripts or utilities |
| 100 | + - Changed core functionality |
| 101 | + - Added new examples |
| 102 | + |
| 103 | +2. Ensure your PR includes: |
| 104 | + - Reference to relevant issues |
| 105 | + - Updates to README.md (if needed) |
| 106 | + - Updates to example configurations (if needed) |
| 107 | + - New or updated tests |
| 108 | + |
| 109 | +3. PR Description Template: |
| 110 | + ```markdown |
| 111 | + ## Description |
| 112 | + Brief description of changes |
| 113 | + |
| 114 | + ## Changes to README.md |
| 115 | + - [ ] No changes needed |
| 116 | + - [ ] Updated prerequisites |
| 117 | + - [ ] Updated usage instructions |
| 118 | + - [ ] Added new feature documentation |
| 119 | + |
| 120 | + ## Testing Done |
| 121 | + Describe testing performed |
| 122 | + |
| 123 | + ## Related Issues |
| 124 | + Fixes #issue_number |
| 125 | + ``` |
| 126 | + |
| 127 | +## Getting Help |
| 128 | + |
| 129 | +1. Check the [README.md](README.md) first |
| 130 | +2. Look for similar examples in `examples/` |
| 131 | +3. Check existing issues and discussions |
| 132 | +4. Open a new issue if needed |
| 133 | + |
| 134 | +## Additional Documentation |
| 135 | + |
| 136 | +For comprehensive documentation, including: |
| 137 | +- Detailed guides |
| 138 | +- Architecture decisions |
| 139 | +- Advanced configurations |
| 140 | +- Best practices |
| 141 | + |
| 142 | +Visit our [documentation website](https://your-org.github.io/openshift-agent-install/). |
| 143 | + |
| 144 | +## Code of Conduct |
| 145 | + |
| 146 | +This project follows the OpenShift community code of conduct. By participating, you are expected to uphold this code. |
0 commit comments