Skip to content

Commit c38c086

Browse files
litian1992richm
authored andcommitted
refactor(trustee_quadlet): move quadlet repo and install dir variables to private
We don't want users to easily move away from designated Trustee quadlets. So move the repo and install path to private to reduce variants. Signed-off-by: Li Tian <litian@redhat.com>
1 parent 0ad9ec7 commit c38c086

6 files changed

Lines changed: 16 additions & 26 deletions

File tree

README.md

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -29,10 +29,6 @@ ansible-galaxy collection install -r meta/collection-requirements.yml
2929
hosts: all
3030
vars:
3131
trustee_attestation_server_trustee: true
32-
trustee_attestation_server_quadlet_repo_url: "https://github.com/litian1992/trustee-quadlet-rhel.git"
33-
trustee_attestation_server_quadlet_repo_path: "quadlet"
34-
trustee_attestation_server_quadlet_repo_branch: "main"
35-
trustee_attestation_server_quadlet_install_dir: "/etc/containers/systemd"
3632
trustee_attestation_server_secret_registration_enabled: true
3733
trustee_attestation_server_secret_registration_listen_port: 8081
3834
roles:

defaults/main.yml

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -3,13 +3,7 @@
33
# Here is the right place to put the role's input variables.
44
# This file also serves as a documentation for such a variables.
55

6-
# Trustee Server Components Quadlet repository configuration
76
trustee_attestation_server_trustee: true
8-
trustee_attestation_server_quadlet_repo_url: "https://github.com/litian1992/trustee-quadlet-rhel.git"
9-
trustee_attestation_server_quadlet_repo_path: "quadlet"
10-
trustee_attestation_server_quadlet_repo_branch: "main"
11-
trustee_attestation_server_quadlet_install_dir: "/etc/containers/systemd"
12-
trustee_attestation_server_config_dir: "/etc/trustee"
137
# Secret registration server service configuration
148
trustee_attestation_server_secret_registration_enabled: false
159
trustee_attestation_server_secret_registration_listen_port: 8081

examples/simple.yml

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -4,10 +4,6 @@
44
hosts: all
55
vars:
66
trustee_attestation_server_trustee: true
7-
trustee_attestation_server_quadlet_repo_url: "https://github.com/litian1992/trustee-quadlet-rhel.git"
8-
trustee_attestation_server_quadlet_repo_path: "quadlet"
9-
trustee_attestation_server_quadlet_repo_branch: "main"
10-
trustee_attestation_server_quadlet_install_dir: "/etc/containers/systemd"
117
trustee_attestation_server_secret_registration_enabled: false
128
roles:
139
- linux-system-roles.trustee_attestation_server

tasks/trustee_quadlet.yml

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@
99

1010
- name: Ensure quadlet install directory exists
1111
ansible.builtin.file:
12-
path: "{{ trustee_attestation_server_quadlet_install_dir }}"
12+
path: "{{ __trustee_attestation_server_quadlet_install_dir }}"
1313
state: directory
1414
mode: "0755"
1515

@@ -20,16 +20,16 @@
2020

2121
- name: Download Trustee Server quadlet files from GitHub repository
2222
ansible.builtin.git:
23-
repo: "{{ trustee_attestation_server_quadlet_repo_url }}"
23+
repo: "{{ __trustee_attestation_server_quadlet_repo_url }}"
2424
dest: "{{ __trustee_attestation_server_quadlet_repo_dir.path }}"
25-
version: "{{ trustee_attestation_server_quadlet_repo_branch }}"
25+
version: "{{ __trustee_attestation_server_quadlet_repo_branch }}"
2626
depth: 1
2727
force: true
2828
register: quadlet_repo_download
2929

3030
- name: Find Trustee Server quadlet files in repository
3131
ansible.builtin.find:
32-
paths: "{{ __trustee_attestation_server_quadlet_repo_dir.path }}/{{ trustee_attestation_server_quadlet_repo_path }}"
32+
paths: "{{ __trustee_attestation_server_quadlet_repo_dir.path }}/{{ __trustee_attestation_server_quadlet_repo_path }}"
3333
patterns:
3434
- "*.container"
3535
- "*.volume"
@@ -40,13 +40,13 @@
4040

4141
- name: Fail if no Trustee Server quadlet files found
4242
ansible.builtin.fail:
43-
msg: "No quadlet files found in {{ trustee_attestation_server_quadlet_repo_url }}/{{ trustee_attestation_server_quadlet_repo_path }}"
43+
msg: "No quadlet files found in {{ __trustee_attestation_server_quadlet_repo_url }}/{{ __trustee_attestation_server_quadlet_repo_path }}"
4444
when: quadlet_files_found.files | length == 0
4545

4646
- name: Copy Trustee Server quadlet files to install directory
4747
ansible.builtin.copy:
4848
src: "{{ item.path }}"
49-
dest: "{{ trustee_attestation_server_quadlet_install_dir }}/{{ item.path | basename }}"
49+
dest: "{{ __trustee_attestation_server_quadlet_install_dir }}/{{ item.path | basename }}"
5050
mode: "0644"
5151
remote_src: true
5252
force: true
@@ -109,7 +109,7 @@
109109

110110
- name: Get the installed Trustee Server pod name
111111
ansible.builtin.find:
112-
paths: "{{ trustee_attestation_server_quadlet_install_dir }}"
112+
paths: "{{ __trustee_attestation_server_quadlet_install_dir }}"
113113
patterns: "*.pod"
114114
register: __trustee_attestation_server_pod_name
115115

tests/tests_default.yml

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -20,17 +20,17 @@
2020

2121
- name: Check trustee quadlet install directory exists
2222
ansible.builtin.stat:
23-
path: "{{ trustee_attestation_server_quadlet_install_dir }}"
23+
path: "{{ __trustee_attestation_server_quadlet_install_dir }}"
2424
register: quadlet_dir
2525

2626
- name: Assert quadlet directory exists
2727
ansible.builtin.assert:
2828
that: quadlet_dir.stat.exists
29-
fail_msg: "Quadlet install directory {{ trustee_attestation_server_quadlet_install_dir }} was not created"
29+
fail_msg: "Quadlet install directory {{ __trustee_attestation_server_quadlet_install_dir }} was not created"
3030

3131
- name: Find quadlet files in install directory
3232
ansible.builtin.find:
33-
paths: "{{ trustee_attestation_server_quadlet_install_dir }}"
33+
paths: "{{ __trustee_attestation_server_quadlet_install_dir }}"
3434
patterns:
3535
- "*.container"
3636
- "*.volume"
@@ -42,7 +42,7 @@
4242
- name: Assert quadlet files exist
4343
ansible.builtin.assert:
4444
that: quadlet_files.matched | int > 0
45-
fail_msg: "No quadlet files found in {{ trustee_attestation_server_quadlet_install_dir }}"
45+
fail_msg: "No quadlet files found in {{ __trustee_attestation_server_quadlet_install_dir }}"
4646

4747
- name: Check trustee certificates and keys were generated
4848
ansible.builtin.stat:
@@ -65,7 +65,7 @@
6565

6666
- name: Find trustee pod file
6767
ansible.builtin.find:
68-
paths: "{{ trustee_attestation_server_quadlet_install_dir }}"
68+
paths: "{{ __trustee_attestation_server_quadlet_install_dir }}"
6969
patterns: "*.pod"
7070
recurse: false
7171
register: trustee_pod_files

vars/main.yml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,10 @@ __trustee_attestation_server_trustee_packages:
1212
__trustee_attestation_server_services: []
1313
__trustee_attestation_server_secret_registration_packages:
1414
- python3
15+
__trustee_attestation_server_quadlet_repo_url: "https://github.com/litian1992/trustee-quadlet-rhel.git"
16+
__trustee_attestation_server_quadlet_repo_path: "quadlet"
17+
__trustee_attestation_server_quadlet_repo_branch: "main"
18+
__trustee_attestation_server_quadlet_install_dir: "/etc/containers/systemd"
1519
__trustee_attestation_server_config_dir: "/etc/trustee"
1620
# ansible_facts required by the role
1721
__trustee_attestation_server_required_facts:

0 commit comments

Comments
 (0)