|
 |
367b4f |
[](https://en.opensuse.org/openSUSE:Heroes) [](https://gitlab.infra.opensuse.org/infra/salt/commits/production)
|
|
Theo Chatzimichos |
ee78de |
|
|
 |
93c121 |
Documentation at the [openSUSE-admin progress.o.o wiki](https://progress.opensuse.org/projects/opensuse-admin-wiki/wiki)
|
|
 |
93c121 |
|
|
Theo Chatzimichos |
ee78de |
Debugging salt on your client (the machine running the salt-minion) via:
|
|
Theo Chatzimichos |
ee78de |
|
|
Theo Chatzimichos |
ee78de |
`salt-call state.apply -l debug test=True`
|
|
 |
93c121 |
|
|
 |
93c121 |
...and remember to have a lot of fun! :-)
|
|
 |
93c121 |
|
|
Theo Chatzimichos |
d91a13 |
|
|
Theo Chatzimichos |
d91a13 |
```
|
|
Theo Chatzimichos |
d91a13 |
_____ ____ _____ __ __
|
|
Theo Chatzimichos |
d91a13 |
/ ___/____ _/ / /_/ ___// /_____ ______/ /__
|
|
Theo Chatzimichos |
d91a13 |
\__ \/ __ `/ / __/\__ \/ __/ __ `/ ___/ //_/
|
|
Theo Chatzimichos |
d91a13 |
___/ / /_/ / / /_ ___/ / /_/ /_/ / /__/ ,<
|
|
Theo Chatzimichos |
d91a13 |
/____/\__,_/_/\__//____/\__/\__,_/\___/_/|_|
|
|
Theo Chatzimichos |
d91a13 |
```
|
|
 |
fa4776 |
|
|
 |
fa4776 |
|
|
 |
fa4776 |
Rules and workflows
|
|
 |
fa4776 |
-------------------
|
|
 |
fa4776 |
|
|
Daniel Schmidt |
39ec61 |
The general workflow should be to create a branch (either directly in this repository or in a clone), do your changes, commit and create a merge request for review. This gives other team members the possibility to notice and review your changes. It even sends out Emails, so other team members get aware of changes.
|
|
 |
fa4776 |
|
|
 |
fa4776 |
On the other side, we do not want to block anyone from being productive. So here are the general rules:
|
|
 |
fa4776 |
|
|
Lars Vogdt |
e90281 |
* **Always use merge requests.**
|
|
Lars Vogdt |
e90281 |
* We allow to merge those requests on your own - but we want to make use of the benefits of merge requests (notifications, tests, visibility).
|
|
 |
fa4776 |
|
|
 |
fa4776 |
|
|
Lars Vogdt |
e90281 |
Merge requests that **require a review**:
|
|
 |
367b4f |
|
|
Lars Vogdt |
e90281 |
* changes that might affect a bigger amount of machines - especially, if this affects machines maintained by others
|
|
Lars Vogdt |
e90281 |
* potentially dangerous stuff that might break existing setups
|
|
Lars Vogdt |
e90281 |
|
|
Lars Vogdt |
d99f18 |
Merge requests that **could be self-merged**:
|
|
 |
367b4f |
|
|
 |
fa4776 |
* emergency updates fixing something that is already broken (think about a new Gateway IP as example)
|
|
Lars Vogdt |
e90281 |
* typo fixes (includes whitespace fixes)
|
|
 |
fa4776 |
* stuff that is interesting only for machines that the requester maintains
|
|
Lars Vogdt |
e90281 |
* stuff that nobody was able to review for more than one week
|
|
Lars Vogdt |
e90281 |
|
|
Lars Vogdt |
e90281 |
This README.md file should be reviewed every quarter during a Heroes Meeting.
|