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

cmdmod : run_chroot - broken in 2014.7.5 - missing kwargs #23153

Closed
cr1st1p opened this issue Apr 28, 2015 · 1 comment
Closed

cmdmod : run_chroot - broken in 2014.7.5 - missing kwargs #23153

cr1st1p opened this issue Apr 28, 2015 · 1 comment
Labels
Bug broken, incorrect, or confusing behavior fixed-pls-verify fix is linked, bug author to confirm fix P1 Priority 1 RIoT Relates to integration with cloud providers, hypervisors, API-based services, etc. Salt-Cloud severity-medium 3rd level, incorrect or bad functionality, confusing and lacks a work around
Milestone

Comments

@cr1st1p
Copy link

cr1st1p commented Apr 28, 2015

Version checked: 2014.7 (git branch) and 2014.7.5. On openSuse.

Try running something like:

salt-run virt.init ubuntu_vivid2 1 512 salt://images/vivid-server-cloudimg-amd64-disk1.img
Searching for Hypervisors
Minion will be preseeded
Creating VM ubuntu_vivid2 on hypervisor home-bigbro
VM ubuntu_vivid2 initialization failed. Returned error: TypeError encountered executing virt.init: run_chroot() got an unexpected keyword argument 'python_shell'. See debug log for more info.

Err... the error might be displayed due to my fix for #23148.

What seems very strange:

  • how could this pass the testing. Anything using lxc and virt, with seeding, would have failed instantaneous. Just check prototype of the function and the way it is called
  • check in 'develop' branch the comments about the additional parameters.
    They're like: "added in 2014.7.1" Err.. what?

Please check also #21189.
Unless I'm missing something, this looks critical to me - no lxc or virt.init works, under the default conditions where seed=True :-(

@jfindlay jfindlay added Bug broken, incorrect, or confusing behavior severity-medium 3rd level, incorrect or bad functionality, confusing and lacks a work around Salt-Cloud P1 Priority 1 labels Apr 29, 2015
@jfindlay jfindlay added this to the Approved milestone Apr 29, 2015
@jfindlay
Copy link
Contributor

@cr1st1p, thanks for the report. The problem seems to be that cmd.run_chroot didn't have the python_shell argument.

@jfindlay jfindlay added the fixed-pls-verify fix is linked, bug author to confirm fix label Apr 29, 2015
@jfindlay jfindlay added RIoT Relates to integration with cloud providers, hypervisors, API-based services, etc. and removed RIoT Relates to integration with cloud providers, hypervisors, API-based services, etc. labels Jun 9, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug broken, incorrect, or confusing behavior fixed-pls-verify fix is linked, bug author to confirm fix P1 Priority 1 RIoT Relates to integration with cloud providers, hypervisors, API-based services, etc. Salt-Cloud severity-medium 3rd level, incorrect or bad functionality, confusing and lacks a work around
Projects
None yet
Development

No branches or pull requests

2 participants