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

:filtered_api = true is failing due to funny rbovirt behaviour #57

Open
aalba6675 opened this issue May 10, 2016 · 3 comments
Open

:filtered_api = true is failing due to funny rbovirt behaviour #57

aalba6675 opened this issue May 10, 2016 · 3 comments
Labels

Comments

@aalba6675
Copy link

Heads up:
1, Admin user (should work with no filtered_api, filtered_api = nil true or false
2. But with :filtered_api = true vagrant up is failing at
abenari/rbovirt#110
3. In rbovirt if I comment out the test for unfiltered_api (template_api.rb) vagrant up works, and the VM gets permissions assigned to the user (this is the "correct" behaviour for filtered_api and emulates ovirt-shell)

def templates(opts={})
      path = "/templates"
      ##path += search_url(opts) unless filtered_api
      path += search_url(opts)
      http_get(path).xpath('/templates/template').collect do |t|
        OVIRT::Template::new(self, t)
      end.compact
    end
@aalba6675
Copy link
Author

See abenari/rbovirt#110. Identified the cause of the nil exception: with the filtered_api search, a Blank template of id "00000000-0000-0000-0000-000000000000" is returned with no <cluster> element causing XML parsing to fail.

@myoung34
Copy link
Owner

Nice catch. Remind me here what the upstream changes affect and I'll try to make sure we get that version as a dependency

@aalba6675
Copy link
Author

aalba6675 commented May 11, 2016

I proposed to upstream to test for nil'ness of (xml/cluster).first

abenari/rbovirt#111

@myoung34 myoung34 added wontfix and removed question labels May 23, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants