Skip to content

Commit

Permalink
Merge pull request #394 from jpopelka/zuul
Browse files Browse the repository at this point in the history
Zuul related refactoring changes

Reviewed-by: https://github.com/apps/softwarefactory-project-zuul
  • Loading branch information
softwarefactory-project-zuul[bot] authored Apr 24, 2020
2 parents a920fb2 + 6dc7dcb commit 8d3a95f
Show file tree
Hide file tree
Showing 8 changed files with 15 additions and 24 deletions.
4 changes: 2 additions & 2 deletions .pre-commit-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ repos:
- id: black
language_version: python3.6
- repo: https://github.com/prettier/prettier
rev: 1.19.1
rev: 2.0.5
hooks:
- id: prettier
exclude: tests/integration/test_data/*
Expand All @@ -29,7 +29,7 @@ repos:
- id: flake8
args: [--max-line-length=100]
- repo: https://github.com/pre-commit/mirrors-mypy
rev: v0.761
rev: v0.770
hooks:
- id: mypy
args: [--no-strict-optional, --ignore-missing-imports]
Expand Down
4 changes: 2 additions & 2 deletions .zuul.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@
nodeset:
nodes:
- name: test-node
label: cloud-fedora-30
label: cloud-fedora-31

- job:
name: ogr-tests-rpm
Expand All @@ -40,7 +40,7 @@
name: ogr-pre-commit
parent: ogr-parent
description: Run pre-commit
run: pre-commit-pb.yaml
run: files/zuul-pre-commit.yaml

- job:
name: ogr-reverse-dep-packit-tests
Expand Down
7 changes: 0 additions & 7 deletions files/tasks/install-ansible.yaml

This file was deleted.

9 changes: 8 additions & 1 deletion files/tasks/packit-requirements.yaml
Original file line number Diff line number Diff line change
@@ -1,5 +1,12 @@
---
- name: install packit dependencies via ansible playbooks
- name: Install ansible
dnf:
name:
- ansible
state: present
become: true

- name: Install packit dependencies via ansible playbooks
command: ansible-playbook -e "ansible_python_interpreter=/usr/bin/python3" -v -c local -i localhost, files/zuul-install-requirements-pip.yaml
args:
chdir: "{{ reverse_dir }}"
2 changes: 1 addition & 1 deletion files/zuul-install-requirements-pip.yaml
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
---
- name: Install dependencies for PIP ogr.
- name: Install dependencies for PIP ogr
hosts: all
tasks:
- include_tasks: tasks/generic-dnf-requirements.yaml
Expand Down
2 changes: 1 addition & 1 deletion files/zuul-install-requirements-rpms.yaml
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
---
- name: Install RPM dependencies for ogr.
- name: Install RPM dependencies for ogr
hosts: all
tasks:
- include_tasks: tasks/zuul-project-setup.yaml
Expand Down
10 changes: 1 addition & 9 deletions pre-commit-pb.yaml → files/zuul-pre-commit.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -12,18 +12,10 @@
name:
- git-core
- python36
become: true
- name: Install pre-commit PyPI package
pip:
name: pre-commit
- pre-commit
become: true
tags:
- stop-layering
- name: Install pre-commit for the repo
command: pre-commit install
args:
chdir: "{{ project_dir }}"
become: true
- name: Run it
command: pre-commit run --all-files
args:
Expand Down
1 change: 0 additions & 1 deletion files/zuul-reverse-dep-packit.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,6 @@
- include_tasks: tasks/zuul-project-setup.yaml
- include_tasks: tasks/generic-dnf-requirements.yaml
- include_tasks: tasks/rpm-test-deps.yaml
- include_tasks: tasks/install-ansible.yaml
- include_tasks: tasks/packit-requirements.yaml
- include_tasks: tasks/install-ogr.yaml
- include_tasks: tasks/packit-tests.yaml

0 comments on commit 8d3a95f

Please sign in to comment.