Actions
Feature #23329
closedAdd arvados_alloy role to Ansible installer
Status:
Resolved
Priority:
Normal
Assigned To:
Category:
Deployment
Target version:
Story points:
-
Release:
Release relationship:
Auto
Description
This shares some code with arvados_loki so we need to do a little reorganization for code sharing:
- Add a
distro_grafanarole. Its responsibility is to set up the shared resources for the Grafana apt repository. It should take the tasks fromarvados_lokithat set up thedeb822_repositoryand package pins.arvados_lokishould now have a meta dependency on thedistro_grafanarole. - In the pin file in
distro_grafana, add a pin for thealloypackage to version1.*. - Add an
arvados_alloyrole that follows the pattern ofarvados_loki: it depends ondistro_grafana. It's configured with a patharvados_alloy_config_filethat it installs to/etc/alloy/config.alloy, installs thealloypackage, and enables and re/starts thealloysystemd_service.
Updated by Brett Smith 4 months ago
- Target version set to Development 2025-12-10
- Category set to Deployment
Updated by Brett Smith 4 months ago
- Related to Idea #18337: Easier install using Ansible added
Updated by Stephen Smith 4 months ago
Changes at arvados|a51b2e9f5207a7f84d84c37fce14eb8bc12e4653 branch 23329-ansible-alloy
- All agreed upon points are implemented / addressed. Describe changes from pre-implementation design.
- Yes
- Anything not implemented (discovered or discussed during work) has a follow-up story.
- none
- Code is tested and passing, both automated and manual, what manual testing was done is described.
- Tested running the install playbook with an alloy config file, verified correct version of alloy installed, config copied, and service running
- The tested code incorporates recent main branch changes.
- Based off of main from 2 days ago
- New or changed UI/UX has gotten feedback from stakeholders.
- n/a
- Documentation has been updated.
- Added example to inventory
- Behaves appropriately at the intended scale (describe intended scale).
- no change in scale
- Considered backwards and forwards compatibility issues between client and server.
- The loki role used arvados-loki.pref for the pin, but now we have both pins in arvados-grafana.pref - any system that ran the old play might have redundant pref files
- Follows our coding standards and GUI style guidelines.
- n/a
Updated by Lucas Di Pentima 4 months ago
The 'apt update' handler is declared in the 'distro_apt' role, so I think the 'distro_grafana' role should depend on it instead of having that dependency on the 'arvados_loki' & 'arvados_alloy' roles.
The rest LGTM, thanks
Updated by Stephen Smith 4 months ago
- Status changed from In Progress to Resolved
Moved that dependency and merged in arvados|e92917a952cd437b248cd23c599d3bc4893bde3e
Actions