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

Installer f17 #408

Merged
merged 13 commits into from
Aug 2, 2012
Merged

Installer f17 #408

merged 13 commits into from
Aug 2, 2012

Conversation

lzap
Copy link
Contributor

@lzap lzap commented Jul 31, 2012

All necessary fixes to the installer for Fedora 17 support. Katello now installs fine, but the katello db migration fails because of Ruby error:

err: /Stage[main]/Katello::Config/Exec[katello_migrate_db]/returns: change from notrun to 0 failed: /usr/bin/env rake db:migrate --trace --verbose > /var/log/katello/katello-configure/db_migrate.log 2>&1 && touch /var/lib/katello/db_migrate_done returned 1 instead of one of [0] at /usr/share/katello/install/puppet/modules/katello/manifests/config.pp:167
err: /Stage[main]/Katello::Config/Exec[katello_migrate_db]: Failed to call refresh: /usr/bin/env rake db:migrate --trace --verbose > /var/log/katello/katello-configure/db_migrate.log 2>&1 && touch /var/lib/katello/db_migrate_done returned 1 instead of one of [0] at /usr/share/katello/install/puppet/modules/katello/manifests/config.pp:167

Caused by http://pastie.org/4365581

Another set of patches will follow fixing our Ruby code base for v 1.9.

@lzap
Copy link
Contributor Author

lzap commented Jul 31, 2012

@knowncitizen @mccun934 Guy installer works, after we fix Ruby-related errors it will finish :-) Getting server running is another story I'd like to submit into another pull request.

@knowncitizen
Copy link
Contributor

ACK

@lzap
Copy link
Contributor Author

lzap commented Aug 1, 2012

Well I can add require, but we already require Katello which has all these. Is this necessary? I will push LC_ALL only. But I have tested this on EL6 and it works.

@lzap
Copy link
Contributor Author

lzap commented Aug 1, 2012

@xsuchy what exactly you think will not work on f16 and el6? I am missing the bit.

@lzap
Copy link
Contributor Author

lzap commented Aug 1, 2012

Ah got it - you mean "puppet apply". Well, I tested this on EL6 (therefore it should also work on F16). Puppet apply command is really there, doublechecked.

@lzap
Copy link
Contributor Author

lzap commented Aug 1, 2012

  1. SPEC - I dont think its needed (there are many more which would be needed to be added)
  2. Fixed now
  3. It is working I tested it. Please send error if its not working for you, otherwise please MERGE :)

@ghost ghost assigned xsuchy Aug 1, 2012
@xsuchy
Copy link
Contributor

xsuchy commented Aug 1, 2012

  1. going to test it.

@xsuchy
Copy link
Contributor

xsuchy commented Aug 2, 2012

ACK

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