Skip to content

Commit

Permalink
Merge pull request #67 from jvoss/bookworm
Browse files Browse the repository at this point in the history
feat: add support for Debian Bookworm
  • Loading branch information
jvoss committed Jan 9, 2024
2 parents 9cf3a4c + 8f2464f commit 9b8632a
Show file tree
Hide file tree
Showing 7 changed files with 21 additions and 22 deletions.
1 change: 1 addition & 0 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -64,6 +64,7 @@ jobs:
# - amazonlinux2
- centos8
- debian11
- debian12
- fedora37
- rockylinux8
- ubuntu2004
Expand Down
17 changes: 4 additions & 13 deletions .github/workflows/netbox.yml
Original file line number Diff line number Diff line change
Expand Up @@ -19,25 +19,16 @@ jobs:
distro:
# - amazonlinux2
- centos8
- debian10
- debian11
- debian12
- fedora37
- rockylinux8
- ubuntu2004
- ubuntu2204
netbox:
- v3.5.0
- v3.4.10
- v3.3.10
- v3.2.9
- v3.1.11
exclude:
- distro: debian10
include:
- distro: debian10
netbox: v3.1.11
- distro: debian10
netbox: v3.0.12
- v3.7.0
- v3.6.9
- v3.5.9

services:
postgres:
Expand Down
6 changes: 3 additions & 3 deletions README.md
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
# Ansible Role: Netbox

[![Netbox](
https://img.shields.io/badge/Netbox-v3.5.0-blue)](https://github.com/netbox-community/netbox)
https://img.shields.io/badge/Netbox-v3.7.0-blue)](https://github.com/netbox-community/netbox)
[![CI](https://github.com/jvoss/ansible-role-netbox/actions/workflows/ci.yml/badge.svg?branch=main)](https://github.com/jvoss/ansible-role-netbox/actions/workflows/ci.yml)
[![Netbox](https://github.com/jvoss/ansible-role-netbox/actions/workflows/netbox.yml/badge.svg)](https://github.com/jvoss/ansible-role-netbox/actions/workflows/netbox.yml)
[![Ansible Galaxy](https://img.shields.io/badge/galaxy-jvoss.netbox-blue.svg)](https://galaxy.ansible.com/jvoss/netbox)
Expand Down Expand Up @@ -44,7 +44,7 @@ compatability information.
Tested on the following platforms:
* Amazon Linux 2
* CentOS 8
* Debian Buster (up to v3.1.11)
* Debian Bookworm
* Debian Bullseye
* Fedora Linux 37
* Rocky Linux 8 / Red Hat Enterprise Linux (RHEL) 8.2+
Expand All @@ -54,7 +54,7 @@ Tested on the following platforms:
This role will require root access (via sudo) to manage system dependencies and actions
on behalf of netbox.

Supports NetBox versions 3+
Supports NetBox versions >=3.5.9

## Role variables

Expand Down
6 changes: 3 additions & 3 deletions handlers/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,8 @@
args:
chdir: "{{ netbox_current_path }}/netbox"
executable: /bin/bash
become: yes
become_method: sudo
become: true
become_method: ansible.builtin.sudo
become_user: "{{ netbox_user }}"
listen: restart_netbox
register: config_check
Expand All @@ -17,7 +17,7 @@
ansible.builtin.service:
name: "{{ item }}"
state: restarted
become: yes
become: true
listen: restart_netbox
loop:
- netbox
Expand Down
4 changes: 2 additions & 2 deletions tasks/superusers.yml
Original file line number Diff line number Diff line change
Expand Up @@ -11,8 +11,8 @@
args:
chdir: "{{ netbox_current_path }}/netbox"
executable: /bin/bash
become: yes
become_method: sudo
become: true
become_method: ansible.builtin.sudo
become_user: "{{ netbox_user }}"
register: result
loop: "{{ netbox_superusers }}"
Expand Down
2 changes: 1 addition & 1 deletion tasks/upgrade.yml
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
environment:
PYTHON: "python{{ python_version | default(__python_version) }}"
become: true
become_method: sudo
become_method: ansible.builtin.sudo
become_user: "{{ netbox_user }}"
notify: restart_netbox
# task vs handler to trigger at specific point in run
Expand Down
7 changes: 7 additions & 0 deletions vars/Debian-12.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
---
__python_version: "3.11"
__python_packages:
- "python{{ python_version | default(__python_version) }}"
- "python3-pip"
- "python{{ python_version | default(__python_version) }}-venv"
- "python{{ python_version | default(__python_version) }}-dev"

0 comments on commit 9b8632a

Please sign in to comment.