Theo Chatzimichos ee78de
 [![pipeline status](https://gitlab.infra.opensuse.org/infra/salt/badges/production/pipeline.svg)](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**:
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**:
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.