Skip to content

Commit 42a1401

Browse files
RanabirChakrabortyrpelisse
authored andcommitted
AMW-366 amq_streams collection GitHub action molecule pipelines are breaking because of sudo permission issue
1 parent c766590 commit 42a1401

File tree

10 files changed

+25
-10
lines changed

10 files changed

+25
-10
lines changed

.github/workflows/ci.yml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,9 +10,10 @@ on:
1010

1111
jobs:
1212
ci:
13-
uses: ansible-middleware/github-actions/.github/workflows/ci.yml@main
13+
uses: ansible-middleware/github-actions/.github/workflows/ci.yml@rootperm
1414
secrets: inherit
1515
with:
1616
fqcn: 'middleware_automation/amq_streams'
17+
root_permission_varname: 'amq_streams_install_requires_become'
1718
molecule_tests: >-
1819
[ "default" ]

README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
# Ansible Collection - middleware_automation.amq_streams
22

3-
[![Build Status](https://github.com/ansible-middleware/amq_streams/workflows/CI/badge.svg?branch=main)](https://github.com/ansible-middleware/amq_streams/actions/workflows/ci.yml)
3+
[![Build Status](https://github.com/ansible-middleware/amq_streams/workflows/CI/badge.svg)](https://github.com/ansible-middleware/amq_streams/actions/workflows/ci.yml)
44

55
Collection to install and configure [Apache Kafka](https://kafka.apache.org/) / [Red Hat AMQ Streams](https://access.redhat.com/documentation/en-us/red_hat_amq_streams).
66

molecule/default/converge.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,8 @@
1010
- name: otherTopic
1111
partitions: 4
1212
replication_factor: 1
13+
vars_files:
14+
- vars.yml
1315
roles:
1416
- role: amq_streams_zookeeper
1517
tasks:

molecule/default/vars.yml

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,12 @@
1+
---
2+
amq_streams_common_escalade_privilege_group_create: "{{ amq_streams_install_requires_become | default(true) }}"
3+
amq_streams_common_escalade_privilege_user_create: "{{ amq_streams_install_requires_become | default(true) }}"
4+
amq_streams_common_archive_extraction_requires_privilege_escalation: "{{ amq_streams_install_requires_become | default(true) }}"
5+
amq_streams_common_dependencies_require_priv: "{{ amq_streams_install_requires_become | default(true) }}"
6+
amq_streams_zookeeper_data_require_priv_escalation: "{{ amq_streams_install_requires_become | default(true) }}"
7+
amq_streams_zookeeper_restart_requires_priv_escalation: "{{ amq_streams_install_requires_become | default(true) }}"
8+
amq_streams_broker_tls_truststore_client_require_priv_escalation: "{{ amq_streams_install_requires_become | default(true) }}"
9+
amq_streams_broker_config_files_requires_privilege_escalation: "{{ amq_streams_install_requires_become | default(true) }}"
10+
amq_streams_cruise_control_path_to_capacity_file_require_priv_escalation: "{{ amq_streams_install_requires_become | default(true) }}"
11+
amq_streams_cruise_control_logfiles_requires_priv_escalation: "{{ amq_streams_install_requires_become | default(true) }}"
12+
amq_streams_connect_source_file_require_priv_escalation: "{{ amq_streams_install_requires_become | default(true) }}"

molecule/prepare.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@
2222
- sudo_pkg_name in ansible_facts.packages
2323

2424
- name: "Ensure required packages are installed."
25-
become: yes
25+
become: "{{ amq_streams_install_requires_become | default(true) }}"
2626
ansible.builtin.yum:
2727
name:
2828
- sudo

roles/amq_streams_broker/handlers/main.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,4 +3,4 @@
33
ansible.builtin.service:
44
name: "{{ amq_streams_broker.service_name }}"
55
state: restarted
6-
become: yes
6+
become: "{{ amq_streams_install_requires_become | default(true) }}"

roles/amq_streams_common/tasks/firewalld.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,15 +7,15 @@
77
quiet: true
88

99
- name: "Ensure firewalld is available."
10-
become: true
10+
become: "{{ amq_streams_install_requires_become | default(true) }}"
1111
ansible.builtin.package:
1212
name: "{{ amq_streams_common_firewalld_package_name }}"
1313
state: present
1414
when:
1515
- firewalld_state_check is not defined or firewalld_state_check
1616

1717
- name: "Ensure firewalld is running."
18-
become: true
18+
become: "{{ amq_streams_install_requires_become | default(true) }}"
1919
ansible.builtin.service:
2020
name: "{{ amq_streams_common_firewalld_service_name | default('firewalld') }}"
2121
state: started

roles/amq_streams_connect/handlers/main.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,4 +3,4 @@
33
ansible.builtin.service:
44
name: "{{ amq_streams_connect_service_name }}"
55
state: restarted
6-
become: yes
6+
become: "{{ amq_streams_install_requires_become | default(true) }}"

roles/amq_streams_cruise_control/handlers/main.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,13 +3,13 @@
33
ansible.builtin.service:
44
name: "{{ amq_streams_cruise_control_service_name }}"
55
state: restarted
6-
become: yes
6+
become: "{{ amq_streams_install_requires_become | default(true) }}"
77

88
- name: "Stop {{ amq_streams_cruise_control_service_name }}"
99
ansible.builtin.service:
1010
name: "{{ amq_streams_cruise_control_service_name }}"
1111
state: stopped
12-
become: yes
12+
become: "{{ amq_streams_install_requires_become | default(true) }}"
1313

1414
- name: "Refresh capacity.json"
1515
ansible.builtin.include_tasks: capacity.yml

roles/amq_streams_exporter/handlers/main.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,4 +3,4 @@
33
ansible.builtin.service:
44
name: "{{ amq_streams_exporter_service_name }}"
55
state: restarted
6-
become: yes
6+
become: "{{ amq_streams_install_requires_become | default(true) }}"

0 commit comments

Comments
 (0)