|
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 |
|
|
|
fa4776 |
The general workflow should be to create a branch (either directly in this repository or in another one), do your changes, recommit 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 |
|
|
|
fa4776 |
Always try to use merge requests. 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 |
Changes, that require a submit request and review:
|
|
|
fa4776 |
* changes, that might affect a bigger amount of machines - especially, if this affects machines maintained by others
|
|
|
fa4776 |
* potentially dangerous stuff, that might break existing setups
|
|
|
fa4776 |
|
|
|
fa4776 |
Changes, that could be self-merged:
|
|
|
fa4776 |
* emergency updates fixing something that is already broken (think about a new Gateway IP as example)
|
|
|
fa4776 |
* stuff that is interesting only for machines that the requester maintains
|
|
|
fa4776 |
* stuff that nobody was able to review for more than 2 months
|