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

Install owncloud-server instead of owncloud (remove apache dependency) #21

Merged
merged 1 commit into from
Nov 19, 2015
Merged

Install owncloud-server instead of owncloud (remove apache dependency) #21

merged 1 commit into from
Nov 19, 2015

Conversation

AndreaGiardini
Copy link
Contributor

The package owncloud depends on apache. Since i want to use this module with nginx i am supposed to write in my manifest:

class { '::owncloud':
  manage_apache => false,
  manage_vhost  => false,
  datadirectory => '/var/www/cloud.myprecise.box',
  db_host => 'localhost',
  db_name => 'owncloud',
  db_user => 'owncloud',
  db_pass => 'p4ssw0rd',
}

What i get is a non-working machine since nginx and apache compete to bind on the 80 port. The solution is to replace the owncloud package with owncloud-server (no apache dep)

@AndreaGiardini AndreaGiardini changed the title Install owncloud-server instead of owncloud (remove apache dependency) [WIP] Install owncloud-server instead of owncloud (remove apache dependency) Oct 30, 2015
@AndreaGiardini AndreaGiardini changed the title [WIP] Install owncloud-server instead of owncloud (remove apache dependency) Install owncloud-server instead of owncloud (remove apache dependency) Oct 30, 2015
@AndreaGiardini
Copy link
Contributor Author

Updates?

@AndreaGiardini
Copy link
Contributor Author

Can you please merge/review this? Thanks

shoekstra added a commit that referenced this pull request Nov 19, 2015
Install owncloud-server instead of owncloud (remove apache dependency)
@shoekstra shoekstra merged commit 49d8b4e into shoekstra:develop Nov 19, 2015
@shoekstra
Copy link
Owner

Looks good!

Sorry for the delay, have been crazy busy last few weeks and don't work that much with Puppet anymore.

@AndreaGiardini AndreaGiardini deleted the package_centos7 branch November 19, 2015 13:29
@AndreaGiardini
Copy link
Contributor Author

Can you please release a new version on puppetforge? The module is broken for me since i need to configure owncloud using nginx

@shoekstra
Copy link
Owner

0.4.2 has been released to the Forge.

@AndreaGiardini
Copy link
Contributor Author

Well... Have a look at #24 because CentOS support is now broken

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants