Skip to content
This repository was archived by the owner on Dec 27, 2023. It is now read-only.

Add Debian#2

Open
mukaibot wants to merge 3 commits into
maestrodev:masterfrom
mukaibot:debian
Open

Add Debian#2
mukaibot wants to merge 3 commits into
maestrodev:masterfrom
mukaibot:debian

Conversation

@mukaibot

@mukaibot mukaibot commented Aug 4, 2014

Copy link
Copy Markdown

Add support for Debian / Ubuntu (tested on Ubuntu 14.04 and Centos 6.5).

Artifactory provides an RPM which can be used on Redhat based distros like Centos, but Debian users just get a zip file. This Puppet module will still continue to use the RPM to install on Centos, however this Pull Request adds support for Debian too. In the case of Debian, the zip file for the specified version (defaulting to 3.3.0) will be downloaded and extracted, a user for Artifactory created and an init script placed into /etc/init.d. This will be configured to execute at startup. I think this gives equivalent functionality to what the RPM does.

@carlossg

Copy link
Copy Markdown
Member

Thanks. The build is failing though, could you add the necessary staging module to the Modulefile, squash the commits into one, and run puppet-lint?

@mukaibot mukaibot closed this Aug 21, 2014
@mukaibot mukaibot reopened this Aug 21, 2014
Break out functionality into smaller manifests
Minor changes after running puppet-lint
@mukaibot

Copy link
Copy Markdown
Author

Hi Carlos,

Please find the updated PR. Note that I converted the Modulefile into a metadata.json file as this is the newer format, and I was getting deprecation warnings.

Thanks!
Timothy.

@carlossg

Copy link
Copy Markdown
Member

I have pulled it into the debian branch with fixes to run the tests again, but the patch is still failing, see https://travis-ci.org/maestrodev/puppet-artifactory/builds/33145919

@mukaibot

Copy link
Copy Markdown
Author

Hmm, it's saying "Cannot assign to variables in other namespaces", but one of the warnings I got with puppet-lint was about top-level namespaced variables, so I added the "artifactory::" scope.

Do you know what might be causing this? :(

Comment thread manifests/install.pp Outdated

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

you can't assign to a variable in other scope, just do $extracted

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants