Skip to content
This repository has been archived by the owner on May 31, 2023. It is now read-only.

Avoid being clever about unsupported architectures #211

Merged
merged 1 commit into from
Mar 26, 2020
Merged

Avoid being clever about unsupported architectures #211

merged 1 commit into from
Mar 26, 2020

Conversation

bwolf
Copy link
Contributor

@bwolf bwolf commented Mar 25, 2020

Previously, when different packages sources where used to install Grafana,
it made sense to fail fast, if there where no supported package. But now
Grafana supports many architecture directly and this pre-flight check
hinders installation on aarch64. Removing this check, delegating the
failure to the package installer, simplifies the installation.

Previously, when different packages sources where used to install Grafana,
it made sense to fail fast, if there where no supported package. But now
Grafana supports many architecture directly and this pre-flight check
hinders installation on aarch64. Removing this check, delegating the
failure to the package installer, simplifies the installation.
@github-actions github-actions bot added the area/tasks Logic behind ansible role label Mar 25, 2020
@paulfantom paulfantom merged commit 2321a62 into cloudalchemy:master Mar 26, 2020
@paulfantom
Copy link
Member

Thanks 👍

cosandr pushed a commit to cosandr/ansible-grafana that referenced this pull request Aug 12, 2020
Previously, when different packages sources where used to install Grafana,
it made sense to fail fast, if there where no supported package. But now
Grafana supports many architecture directly and this pre-flight check
hinders installation on aarch64. Removing this check, delegating the
failure to the package installer, simplifies the installation.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area/tasks Logic behind ansible role
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants