Skip to content

Commit 3fee09f

Browse files
committed
Add OWNERS, CODEOWNERS, and pre-commit GitHub Action
Enable Prow-based PR review workflow for shiftstack-qa: - OWNERS and OWNERS_ALIASES for /lgtm and /approve commands - CODEOWNERS for automatic reviewer assignment - GitHub Action running pre-commit (ansible-lint) on PRs, matching gate.sh behavior Ref: OSPRH-32049 Change-Id: I7c580137f660b31a3e3f06b62d07b82838e9663c Signed-off-by: Itay Matza <imatza@redhat.com>
1 parent b820b7a commit 3fee09f

4 files changed

Lines changed: 63 additions & 0 deletions

File tree

.github/CODEOWNERS

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,4 @@
1+
# Default reviewers for all files
2+
# https://docs.github.com/en/repositories/managing-your-repositorys-settings-and-features/customizing-your-repository/about-code-owners
3+
4+
* @dlaw4608 @ekuris-redhat @eshulman2 @gryf @imatza-rh @mandre @stephenfin @tusharjadhav3302 @winiciusallan

.github/workflows/ansible-lint.yml

Lines changed: 28 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,28 @@
1+
---
2+
name: Ansible Lint
3+
4+
on:
5+
pull_request:
6+
branches:
7+
- main
8+
9+
jobs:
10+
ansible-lint:
11+
runs-on: ubuntu-latest
12+
container:
13+
image: quay.io/shiftstack-qe/shiftstack-client
14+
options: --user root
15+
steps:
16+
- name: Checkout
17+
uses: actions/checkout@v4
18+
19+
- name: Mark workspace as safe for git
20+
run: git config --global --add safe.directory "$GITHUB_WORKSPACE"
21+
22+
- name: Install Ansible collections
23+
run: ansible-galaxy collection install -r requirements.yaml
24+
25+
- name: Run pre-commit (ansible-lint)
26+
run: pre-commit run --all-files
27+
env:
28+
ANSIBLE_VAULT_PASSWORD_FILE: /home/cloud-admin/.vault-pass

OWNERS

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,8 @@
1+
# See the OWNERS docs: https://git.k8s.io/community/contributors/guide/owners.md
2+
# This file just uses aliases defined in OWNERS_ALIASES.
3+
4+
approvers:
5+
- shiftstack-approvers
6+
7+
reviewers:
8+
- shiftstack-reviewers

OWNERS_ALIASES

Lines changed: 23 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,23 @@
1+
# See the OWNERS_ALIASES docs: https://git.k8s.io/community/contributors/guide/owners.md#OWNERS_ALIASES
2+
3+
aliases:
4+
shiftstack-approvers:
5+
- dlaw4608
6+
- ekuris-redhat
7+
- eshulman2
8+
- gryf
9+
- imatza-rh
10+
- mandre
11+
- stephenfin
12+
- tusharjadhav3302
13+
- winiciusallan
14+
shiftstack-reviewers:
15+
- dlaw4608
16+
- ekuris-redhat
17+
- eshulman2
18+
- gryf
19+
- imatza-rh
20+
- mandre
21+
- stephenfin
22+
- tusharjadhav3302
23+
- winiciusallan

0 commit comments

Comments
 (0)