Skip to content

libvirt: fix for Virsh edit test#710

Merged
yangdongsheng merged 2 commits intoautotest:nextfrom
yangdongsheng:virsh_edit
Aug 9, 2013
Merged

libvirt: fix for Virsh edit test#710
yangdongsheng merged 2 commits intoautotest:nextfrom
yangdongsheng:virsh_edit

Conversation

@yangdongsheng
Copy link
Copy Markdown
Member

No description provided.

@yangdongsheng
Copy link
Copy Markdown
Member Author

@cevich I need an ACK for this patchset.

Could you take a quick look at it .

Thx

yangdongsheng added 2 commits August 7, 2013 22:03
* we can not assume the guest with 1 vcpu.
Change the pattern in sed from `1<\/vcpu>` to `[0-9]*<\/vcpu>`

* add a vcpucount function in virsh module. And use it in virsh_edit
to get the vcpucount of VM.

Signed-off-by: yangdongsheng <yangds.fnst@cn.fujitsu.com>
Signed-off-by: yangdongsheng <yangds.fnst@cn.fujitsu.com>
@yangdongsheng
Copy link
Copy Markdown
Member Author

well thanx @liyangfnst for your comments.

@cevich
Copy link
Copy Markdown
Member

cevich commented Aug 8, 2013

Yep, this is fine by me.

@yangdongsheng
Copy link
Copy Markdown
Member Author

Ok, Thanx @liyangfnst and @cevich . pushing to next.

yangdongsheng pushed a commit that referenced this pull request Aug 9, 2013
libvirt: fix for Virsh edit test
@yangdongsheng yangdongsheng merged commit 8913948 into autotest:next Aug 9, 2013
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.

2 participants