Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ci: Add ALP-Dolomite var file #141

Merged
merged 4 commits into from
Jan 7, 2024
Merged

ci: Add ALP-Dolomite var file #141

Show file tree
Hide file tree
Changes from 3 commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
a6bb726
ci: Add ALP-Dolomite var file
HVSharma12 Jan 5, 2024
8912c35
remove redundant task
HVSharma12 Jan 6, 2024
b6e0e78
Merge remote-tracking branch 'origin/main' into add-suse-alp-vars
HVSharma12 Jan 6, 2024
858f713
Update collection requirements
HVSharma12 Jan 7, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
18 changes: 18 additions & 0 deletions tasks/setup-zypper.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,18 @@
---
- name: If choosing custom package set, ensure minimal cockpit is included
set_fact:
cockpit_packages: "{{ cockpit_packages + __cockpit_packages_minimal
if cockpit_packages not in __cockpit_package_types
else cockpit_packages }}"
when: cockpit_packages is defined

- name: Ensure Cockpit Web Console packages are installed.
zypper:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
zypper:
community.general.zypper:

Also - change meta/collection-requirements.yml to look like this:

# SPDX-License-Identifier: MIT
---
collections:
  - ansible.posix
  - community.general
  - fedora.linux_system_roles

That should make linters and ansible-plugin-scan happy

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for the suggestion, @richm . I have updated the meta/collection-requirements.yml as advised and made the necessary changes to the zypper reference

name: "{{ __cockpit_packages[cockpit_packages]
if cockpit_packages in __cockpit_package_types
else cockpit_packages }}"
state: present

- name: Reboot system
reboot:
when: ansible_distribution == 'ALP-Dolomite'
22 changes: 22 additions & 0 deletions vars/ALP-Dolomite.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,22 @@
---
__cockpit_packages_minimal:
- cockpit-system
- cockpit-ws
__cockpit_packages_default:
- cockpit-bridge
- cockpit-networkmanager
- cockpit-selinux
- cockpit-storaged
__cockpit_packages_full:
- cockpit-bridge
- cockpit-machines
- cockpit-networkmanager
- cockpit-podman
- cockpit-selinux
- cockpit-storaged
- cockpit-tukit
__cockpit_packages:
minimal: "{{ __cockpit_packages_minimal }}"
default: "{{ __cockpit_packages_minimal + __cockpit_packages_default }}"
full: "{{ __cockpit_packages_minimal + __cockpit_packages_default +
__cockpit_packages_full }}"
Loading