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.

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

Add acceptance test with archive install_method #177

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

dhoppe
Copy link
Member

@dhoppe dhoppe commented Jun 27, 2019

This will replace #141.

@dhoppe
Copy link
Member Author

dhoppe commented Jun 27, 2019

The acceptance tests are still failing, even if the command can be executed locally. I am not sure if this is the correct way of handling the service.

In the end, I do not see a reason why we should support the installation via .tar.gz, because Grafana provides all kinds of packages.

@bastelfreak bastelfreak added needs-feedback Further information is requested tests-fail labels Jul 28, 2019
@othalla
Copy link

othalla commented Jan 29, 2020

@dhoppe i wanted to perform the installation via .tar.gz to be able to choose the release on ARM arch.

@bastelfreak
Copy link
Member

@dhoppe ping :)

@dhoppe
Copy link
Member Author

dhoppe commented Apr 6, 2020

@othalla It is not gonna work like this.

The .deb or .rpm packages create the user grafana without a login shell (/sbin/nologin):

Therefore the following command does not work:

Even if I delete the user and the directories, the remaining tests will fail:

Maybe you should use these parameters to install Grafana as package:

class { 'grafana':
  install_method = 'package',
  package_source = 'https://dl.grafana.com/oss/release/grafana_5.4.5_armhf.deb',
}

From version 6.5, Grafana even offers a mirror for ARM packages. The version can still be fixed using APT pinning:

@vox-pupuli-tasks
Copy link

Dear @dhoppe, thanks for the PR!

This is Vox Pupuli Tasks, your friendly Vox Pupuli GitHub Bot. I noticed that your pull request contains merge conflict. Can you please rebase?

You can find my sourcecode at voxpupuli/vox-pupuli-tasks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merge-conflicts needs-feedback Further information is requested tests-fail
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants