|
|
367b4f |
[![openSUSE Heroes](https://img.shields.io/badge/openSUSE-Heroes-brightgreen.svg?logo=data%3Aimage%2Fpng%3Bbase64%2CiVBORw0KGgoAAAANSUhEUgAAABAAAAAQCAMAAAAoLQ9TAAAAk1BMVEUAAABmmQCR0AdmmQB7tAOR0Ad0qwKR0AdmmQB7tAOGwgWR0AeR0AdmmQB0qwKR0AdmmQBmmQBwpgF2rQJ7tAOR0AdvpQF5sQOR0AdyqQKIxQWR0Ad1rQKR0Ad0qwJ3rwKJxgWR0Ad5sgN9tgN%2BuASR0Ad%2FugSLyAaR0Ad7tAN%2BtwOAuwSGwgWIxQWLyQaOzAaR0Ac0QkqCAAAAKXRSTlMADw8fHx8vLz8%2FPz9PX19fb39%2Ff39%2Fj4%2BPn5%2Bfr6%2B%2Fv7%2B%2Fz8%2Ff3%2B%2Fv7%2FtiEuIAAACOSURBVBgZbcHbFkJAAAXQQ8kUJRXRGKHblKHz%2F19XLq3Vg70xSUaejX%2F8uknfRs%2BW%2FJEuAEdzYLL4Sh%2BQHLSLSK0uGuDo5PGp5gzAUbIkC4cK7LwbajfNhSTB3qZqdRKVpMGDHeWsz7UxdS5w4ED5lhCzoIBVcaTLUps9YMX3hj3zSgU6lgh32THcCkz5AIT0Glg8M2spAAAAAElFTkSuQmCC)](https://en.opensuse.org/openSUSE:Heroes) [![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**:
|
|
|
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.
|