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

Refs #6126: Change RHSM API to /rhsm. #28

Merged
merged 1 commit into from Jul 16, 2014
Merged

Conversation

ehelms
Copy link
Member

@ehelms ehelms commented Jun 10, 2014

No description provided.

@ehelms
Copy link
Member Author

ehelms commented Jun 10, 2014

Requires - Katello/katello#4239

@@ -12,21 +12,19 @@
# see /usr/share/katello/config.katello_defaults.yml for available options

common:
app_mode: <%= scope.lookupvar("katello::params::deployment") %>
app_mode: 'katello'

Choose a reason for hiding this comment

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

It's outside of the scope of this PR but can you open an issue to remove app_mode in the future if it's no longer going to be configurable?

Copy link
Member Author

Choose a reason for hiding this comment

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

http://projects.theforeman.org/issues/4121 -- exists to clean all of this up. I started on it at one point but got side tracked since it wasn't blocking anything to have them around.

@waldenraines
Copy link

ACK

@jlsherrill
Copy link
Contributor

I currently get:

[ERROR 2014-07-16 18:36:33 main]  Could not apply complete catalog: Found 1 dependency cycle:
[ INFO 2014-07-16 18:36:33 main] (Cert[centos.installer-qpid-broker] => Privkey[/etc/pki/katello/private/centos.installer-qpid-broker.key] => File[/etc/pki/katello/private/centos.installer-qpid-broker.key] => File[/etc/pki/katello/nssdb] => File[/etc/pki/katello/nssdb/nss_db_password-file] => Exec[create-nss-db] => Exec[candlepin-add-client-cert-to-nss-db] => Class[Certs::Candlepin] => Class[Candlepin] => Class[Candlepin] => Class[Certs::Qpid] => File[/etc/pki/katello/nssdb/nss_db_password-file])
[ INFO 2014-07-16 18:36:33 main] Try the '--graph' option and opening the resulting '.dot' file in OmniGraffle or GraphViz
[ INFO 2014-07-16 18:36:33 main] /usr/lib/ruby/site_ruby/1.8/puppet/graph/simple_graph.rb:242:in `report_cycles_in_graph'
[ INFO 2014-07-16 18:36:33 main] /usr/lib/ruby/site_ruby/1.8/puppet/graph/relationship_graph.rb:106:in `traverse'
[ INFO 2014-07-16 18:36:33 main] /usr/lib/ruby/site_ruby/1.8/puppet/transaction.rb:108:in `evaluate_without_trigger'
[ INFO 2014-07-16 18:36:33 main] /usr/lib/ruby/gems/1.8/gems/kafo-0.6.4/modules/kafo_configure/lib/puppet/parser/functions/add_progress.rb:25:in `evaluate'
[ INFO 2014-07-16 18:36:33 main] /usr/lib/ruby/site_ruby/1.8/puppet/resource/catalog.rb:167:in `apply'
[ INFO 2014-07-16 18:36:33 main] /usr/lib/ruby/site_ruby/1.8/puppet/util/log.rb:149:in `with_destination'
[ INFO 2014-07-16 18:36:33 main] /usr/lib/ruby/site_ruby/1.8/puppet/transaction/report.rb:112:in `as_logging_destination'
[ INFO 2014-07-16 18:36:33 main] /usr/lib/ruby/gems/1.8/gems/kafo-0.6.4/modules/kafo_configure/lib/kafo/puppet/report_wrapper.rb:30:in `send'

I really have no idea why this would cause that error, but its what i see :(

@jlsherrill
Copy link
Contributor

actually, i think maybe the branch is just out of date, one sec

@jlsherrill
Copy link
Contributor

ACK

ehelms added a commit that referenced this pull request Jul 16, 2014
Refs #6126: Change RHSM API to /rhsm.
@ehelms ehelms merged commit 9c2e270 into theforeman:master Jul 16, 2014
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.

None yet

3 participants