Skip to content
This repository has been archived by the owner on Mar 26, 2024. It is now read-only.

Add optimize/orphaned for postgresql database & append many modification for debian #333

Closed
wants to merge 14 commits into from

Conversation

swoopla
Copy link
Contributor

@swoopla swoopla commented Sep 15, 2015

i installed puppet-dashboard on a debian machine with postgresql database.
So i had many modifications.

Regards

@sodabrew
Copy link
Owner

Thank you! I started with some review comments, I'll continue reviewing this soon.

gem 'shoulda-matchers', '< 2.0' # support ruby 1.8.7
gem 'rspec-html-matchers'
gem 'nokogiri', '< 1.6' # support ruby 1.8.7
gem :developpement, :test do
Copy link
Owner

Choose a reason for hiding this comment

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

development on this side of the channel / pond ;)
(fixing this to match the original Gemfile should fix the unit tests on Travis)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

french word... I'll change the line

2015-09-16 10:02 GMT+02:00 Aaron Stone [email protected]:

In Gemfile
#333 (comment)
:

@@ -28,14 +28,14 @@ group :assets do
gem 'execjs' , '~> 2.0.0' # Will use system-available JS runtime
end

-group :development, :test do

  • gem 'mocha', '~> 0.13.3', :require => false
  • gem 'sqlite3'
  • gem 'rspec-rails', '~> 2.13.0'
  • gem 'factory_girl', '< 3.0' # supports ruby 1.8.7
  • gem 'shoulda-matchers', '< 2.0' # support ruby 1.8.7
  • gem 'rspec-html-matchers'
  • gem 'nokogiri', '< 1.6' # support ruby 1.8.7
    +gem :developpement, :test do

development on this side of the channel / pond ;)


Reply to this email directly or view it on GitHub
https://github.com/sodabrew/puppet-dashboard/pull/333/files#r39602590.

gem 'shoulda-matchers', '< 2.0' # support ruby 1.8.7
gem 'rspec-html-matchers'
gem 'nokogiri', '< 1.6' # support ruby 1.8.7
gem :development, :test do
Copy link
Owner

Choose a reason for hiding this comment

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

Oh, it's group not gem. Would be best to change this block to exactly match the prior Gemfile contents.

@Skunnyk Skunnyk mentioned this pull request Oct 5, 2016
@bwitt
Copy link
Collaborator

bwitt commented Aug 22, 2018

@swoopla hi thanks for the contribution! I notice your source repo is gone; if you'd like to keep working on this to get it merged I'd be glad to help!

@ZeroPointEnergy
Copy link
Collaborator

Hi and thanks @swoopla for your contribution. Since this PR contains a lot of different changes I started to separate them in order to have a better overview of what we are changing. PR #373 contains the first bits about the postgres optimizations. There are still some debian and purge task related changes left to extract. The rest of the rebased branch is here: https://github.com/ZeroPointEnergy/puppet-dashboard/commits/debian_and_optimize_postgres

I think we can let this PR open for now until everything is extracted and merged.

@ZeroPointEnergy
Copy link
Collaborator

I extracted this PR into three separate PR because it was addressing multiple issues: #373, #376, #377

Further development and discussion should take place on those PR. I will close this PR now. @swoopla thanks for your contribution again and sorry it took such a long time.

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.

4 participants