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

EVM Server Main Configuration are not being passed to refreshers. #5816

Closed
jvlcek opened this issue Dec 11, 2015 · 1 comment · Fixed by #5868
Closed

EVM Server Main Configuration are not being passed to refreshers. #5816

jvlcek opened this issue Dec 11, 2015 · 1 comment · Fixed by #5868
Assignees

Comments

@jvlcek
Copy link
Member

jvlcek commented Dec 11, 2015

When defaults are changed in the EVM Server Main Configuration file via
Configure/Advanced the changes are not being passed to the Refreshers.

This can be observed using the rails console on an appliance with:

  e = ManageIQ::Providers::Amazon::CloudManager.first
  r = ManageIQ::Providers::Amazon::CloudManager::Refresher.new([e])
  r.refresher_options
  # => nil

That should not be nil.

Something is preventing the options from being passed to the refreshers.

This is related to the issue reported [1] where gathering EC2 public images fails even
after the default is changed from:
get_public_images: false to:
get_public_images: true

http://talk.manageiq.org/t/ec2-public-images-capablanca/1093

@jvlcek
Copy link
Member Author

jvlcek commented Dec 11, 2015

@miq-bot assign @jvlcek

jvlcek added a commit to jvlcek/manageiq that referenced this issue Dec 16, 2015
This fix will allow provider refreshers to correctly retieve vmdb configuration options.

Fixes ManageIQ#5816
lgalis pushed a commit to lgalis/manageiq that referenced this issue Mar 17, 2016
This fix will allow provider refreshers to correctly retieve vmdb configuration options.

Fixes ManageIQ#5816
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 a pull request may close this issue.

1 participant