[libvirt] [test-API 07/17] Substitute _guest with _domain
--- .../en-US/Creating_a_configuration_file.xml| 82 ++-- .../en-US/Writing_a_test_case.xml | 24 +++--- .../libvirt-test-API_Guide/en-US/extras/log.txt| 18 ++-- repos/domain/domain_list.py| 12 ++-- repos/domain/install_linux_cdrom.py| 18 ++-- repos/domain/install_linux_net.py |8 +- repos/domain/install_windows_cdrom.py |6 +- repos/domain/update_devflag.py |6 +- 8 files changed, 87 insertions(+), 87 deletions(-) diff --git a/docs/User_Guide/libvirt-test-API_Guide/en-US/Creating_a_configuration_file.xml b/docs/User_Guide/libvirt-test-API_Guide/en-US/Creating_a_configuration_file.xml index 80ebdb1..ffac187 100644 --- a/docs/User_Guide/libvirt-test-API_Guide/en-US/Creating_a_configuration_file.xml +++ b/docs/User_Guide/libvirt-test-API_Guide/en-US/Creating_a_configuration_file.xml @@ -90,7 +90,7 @@ module:test_case example titleConfiguration file/title programlisting -Domain:install_guest +Domain:install_domain guestname rhel5u4 memory @@ -105,7 +105,7 @@ Domain:install_guest titleList data structure/title programlisting [ - [{'domain:install_guest': {'guestname': 'rhel5u4', 'memory': '1024', 'vcpu': '1'}}] + [{'domain:install_domain': {'guestname': 'rhel5u4', 'memory': '1024', 'vcpu': '1'}}] ] /programlisting /example @@ -119,7 +119,7 @@ Domain:install_guest example titleConfiguration file/title programlisting -Domain:install_guest +Domain:install_domain guestname rhel5u4 rhel5u3 @@ -136,9 +136,9 @@ Domain:install_guest titleList data structure/title programlisting [ - [{'domain:install_guest': {'guestname': 'rhel5u4', 'memory': '1024', 'vcpu': '1'}}], - [{'domain:install_guest': {'guestname': 'rhel5u3', 'memory': '1024', 'vcpu': '1'}}], - [{'domain:install_guest': {'guestname': 'rhel5u2', 'memory': '1024', 'vcpu': '1'}}] + [{'domain:install_domain': {'guestname': 'rhel5u4', 'memory': '1024', 'vcpu': '1'}}], + [{'domain:install_domain': {'guestname': 'rhel5u3', 'memory': '1024', 'vcpu': '1'}}], + [{'domain:install_domain': {'guestname': 'rhel5u2', 'memory': '1024', 'vcpu': '1'}}] ] /programlisting /example @@ -152,7 +152,7 @@ Domain:install_guest example titleConfiguration file/title programlisting -domain:install_guest +domain:install_domain guestname rhel5u4 memory @@ -171,7 +171,7 @@ storage:initialize_storage titleList data structure/title programlisting [ - [{'domain:install_guest': {'guestname': 'rhel5u4', 'memory': '1024', 'vcpu': '1'}}, {'storage:initialize_storage': {'storagename': 'rhel5u4'}}] + [{'domain:install_domain': {'guestname': 'rhel5u4', 'memory': '1024', 'vcpu': '1'}}, {'storage:initialize_storage': {'storagename': 'rhel5u4'}}] ] /programlisting /example @@ -185,7 +185,7 @@ storage:initialize_storage example titleConfiguration file/title programlisting -domain:install_guest +domain:install_domain guestname rhel5u4 rhel5u3 @@ -206,9 +206,9 @@ storage:initialize_storage titleList data structure/title programlisting [ - [{'domain:install_guest': {'guestname': 'rhel5u4', 'memory': '1024', 'vcpu': '1'}}, {'storage:initialize_storage': {'storagename': 'rhel5u4'}}], - [{'domain:install_guest': {'guestname': 'rhel5u3', 'memory': '1024', 'vcpu': '1'}}, {'storage:initialize_storage': {'storagename': 'rhel5u4'}}], - [{'domain:install_guest': {'guestname': 'rhel5u2', 'memory': '1024', 'vcpu': '1'}}, {'storage:initialize_storage': {'storagename': 'rhel5u4'}}] + [{'domain:install_domain': {'guestname': 'rhel5u4', 'memory': '1024', 'vcpu': '1'}}, {'storage:initialize_storage': {'storagename': 'rhel5u4'}}], + [{'domain:install_domain': {'guestname': 'rhel5u3', 'memory': '1024', 'vcpu': '1'}}, {'storage:initialize_storage': {'storagename': 'rhel5u4'}}], + [{'domain:install_domain': {'guestname': 'rhel5u2', 'memory': '1024', 'vcpu': '1'}}, {'storage:initialize_storage': {'storagename': 'rhel5u4'}}] ] /programlisting /example @@ -222,7 +222,7 @@ storage:initialize_storage example titleConfiguration file/title programlisting -domain:install_guest +domain:install_domain guestname rhel5u4 rhel5u3 @@ -232,7 +232,7 @@ domain:install_guest vcpu 1 -domain:shutdown_guest +domain:shutdown_domain guestname rhel5u4 only rhel5u4 rhel5u3 only rhel5u3 @@ -245,9 +245,9 @@ domain:shutdown_guest titleList data structure/title programlisting [ - [{'domain:install_guest': {'guestname': 'rhel5u4', 'memory': '1024', 'vcpu': '1'}}, {'domain:shutdown_guest': {'guestname': 'rhel5u4'}}], - [{'domain:install_guest': {'guestname':
Re: [libvirt] [test-API 07/17] Substitute _guest with _domain
On 04/20/2012 08:46 AM, Osier Yang wrote: --- .../en-US/Creating_a_configuration_file.xml| 82 ++-- .../en-US/Writing_a_test_case.xml | 24 +++--- .../libvirt-test-API_Guide/en-US/extras/log.txt| 18 ++-- repos/domain/domain_list.py| 12 ++-- repos/domain/install_linux_cdrom.py| 18 ++-- repos/domain/install_linux_net.py |8 +- repos/domain/install_windows_cdrom.py |6 +- repos/domain/update_devflag.py |6 +- 8 files changed, 87 insertions(+), 87 deletions(-) diff --git a/docs/User_Guide/libvirt-test-API_Guide/en-US/Creating_a_configuration_file.xml b/docs/User_Guide/libvirt-test-API_Guide/en-US/Creating_a_configuration_file.xml index 80ebdb1..ffac187 100644 --- a/docs/User_Guide/libvirt-test-API_Guide/en-US/Creating_a_configuration_file.xml +++ b/docs/User_Guide/libvirt-test-API_Guide/en-US/Creating_a_configuration_file.xml @@ -90,7 +90,7 @@ module:test_case example titleConfiguration file/title programlisting -Domain:install_guest +Domain:install_domain guestname rhel5u4 memory @@ -105,7 +105,7 @@ Domain:install_guest titleList data structure/title programlisting [ - [{'domain:install_guest': {'guestname': 'rhel5u4', 'memory': '1024', 'vcpu': '1'}}] + [{'domain:install_domain': {'guestname': 'rhel5u4', 'memory': '1024', 'vcpu': '1'}}] I'm not sure about this. The thing we're creating is a domain, but we're installing a guest, am I right? (thus the guestname and later guestip and so on. But I could be wrong. Also even if I'm not, the question is how much we want to separate this in order to keep it readable. Martin -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [test-API 07/17] Substitute _guest with _domain
On 2012年04月20日 16:17, Martin Kletzander wrote: On 04/20/2012 08:46 AM, Osier Yang wrote: --- .../en-US/Creating_a_configuration_file.xml| 82 ++-- .../en-US/Writing_a_test_case.xml | 24 +++--- .../libvirt-test-API_Guide/en-US/extras/log.txt| 18 ++-- repos/domain/domain_list.py| 12 ++-- repos/domain/install_linux_cdrom.py| 18 ++-- repos/domain/install_linux_net.py |8 +- repos/domain/install_windows_cdrom.py |6 +- repos/domain/update_devflag.py |6 +- 8 files changed, 87 insertions(+), 87 deletions(-) diff --git a/docs/User_Guide/libvirt-test-API_Guide/en-US/Creating_a_configuration_file.xml b/docs/User_Guide/libvirt-test-API_Guide/en-US/Creating_a_configuration_file.xml index 80ebdb1..ffac187 100644 --- a/docs/User_Guide/libvirt-test-API_Guide/en-US/Creating_a_configuration_file.xml +++ b/docs/User_Guide/libvirt-test-API_Guide/en-US/Creating_a_configuration_file.xml @@ -90,7 +90,7 @@ module:test_case example titleConfiguration file/title programlisting -Domain:install_guest +Domain:install_domain guestname rhel5u4 memory @@ -105,7 +105,7 @@ Domain:install_guest titleList data structure/title programlisting [ - [{'domain:install_guest': {'guestname': 'rhel5u4', 'memory': '1024', 'vcpu': '1'}}] + [{'domain:install_domain': {'guestname': 'rhel5u4', 'memory': '1024', 'vcpu': '1'}}] I'm not sure about this. The thing we're creating is a domain, but we're installing a guest, am I right? (thus the guestname and later guestip and so on. But I could be wrong. Also even if I'm not, the question is how much we want to separate this in order to keep it readable. I'm not quite sure about it either, but I would like to post the patch so there could be discussion, IIRC, there was discussion about the TERM domain and guest before too. Anyway, there should be guidelines for it, mixing use of both of them at liberty can just produce messy codes. Osier -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [test-API 07/17] Substitute _guest with _domain
On 04/20/2012 10:29 AM, Osier Yang wrote: On 2012年04月20日 16:17, Martin Kletzander wrote: On 04/20/2012 08:46 AM, Osier Yang wrote: --- .../en-US/Creating_a_configuration_file.xml| 82 ++-- .../en-US/Writing_a_test_case.xml | 24 +++--- .../libvirt-test-API_Guide/en-US/extras/log.txt| 18 ++-- repos/domain/domain_list.py| 12 ++-- repos/domain/install_linux_cdrom.py| 18 ++-- repos/domain/install_linux_net.py |8 +- repos/domain/install_windows_cdrom.py |6 +- repos/domain/update_devflag.py |6 +- 8 files changed, 87 insertions(+), 87 deletions(-) diff --git a/docs/User_Guide/libvirt-test-API_Guide/en-US/Creating_a_configuration_file.xml b/docs/User_Guide/libvirt-test-API_Guide/en-US/Creating_a_configuration_file.xml index 80ebdb1..ffac187 100644 --- a/docs/User_Guide/libvirt-test-API_Guide/en-US/Creating_a_configuration_file.xml +++ b/docs/User_Guide/libvirt-test-API_Guide/en-US/Creating_a_configuration_file.xml @@ -90,7 +90,7 @@ module:test_case example titleConfiguration file/title programlisting -Domain:install_guest +Domain:install_domain guestname rhel5u4 memory @@ -105,7 +105,7 @@ Domain:install_guest titleList data structure/title programlisting [ - [{'domain:install_guest': {'guestname': 'rhel5u4', 'memory': '1024', 'vcpu': '1'}}] + [{'domain:install_domain': {'guestname': 'rhel5u4', 'memory': '1024', 'vcpu': '1'}}] I'm not sure about this. The thing we're creating is a domain, but we're installing a guest, am I right? (thus the guestname and later guestip and so on. But I could be wrong. Also even if I'm not, the question is how much we want to separate this in order to keep it readable. I'm not quite sure about it either, but I would like to post the patch so there could be discussion, IIRC, there was discussion about the TERM domain and guest before too. Anyway, there should be guidelines for it, mixing use of both of them at liberty can just produce messy codes. Osier It's written in the guide: http://libvirt.org/devguide.html but I'm not sure even after reading it. It's not _defined_ there, so we should probably go with our gut on this one. After looking at the code with all the patches applied, I think this could be ok, we have only few words with guest in it and they make sense, so for the sake of stabilizing the code asap, ACK. Martin -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [test-API 07/17] Substitute _guest with _domain
On 04/20/2012 04:29 PM, Osier Yang wrote: On 2012年04月20日 16:17, Martin Kletzander wrote: On 04/20/2012 08:46 AM, Osier Yang wrote: --- .../en-US/Creating_a_configuration_file.xml| 82 ++-- .../en-US/Writing_a_test_case.xml | 24 +++--- .../libvirt-test-API_Guide/en-US/extras/log.txt| 18 ++-- repos/domain/domain_list.py| 12 ++-- repos/domain/install_linux_cdrom.py| 18 ++-- repos/domain/install_linux_net.py |8 +- repos/domain/install_windows_cdrom.py |6 +- repos/domain/update_devflag.py |6 +- 8 files changed, 87 insertions(+), 87 deletions(-) diff --git a/docs/User_Guide/libvirt-test-API_Guide/en-US/Creating_a_configuration_file.xml b/docs/User_Guide/libvirt-test-API_Guide/en-US/Creating_a_configuration_file.xml index 80ebdb1..ffac187 100644 --- a/docs/User_Guide/libvirt-test-API_Guide/en-US/Creating_a_configuration_file.xml +++ b/docs/User_Guide/libvirt-test-API_Guide/en-US/Creating_a_configuration_file.xml @@ -90,7 +90,7 @@ module:test_case example titleConfiguration file/title programlisting -Domain:install_guest +Domain:install_domain guestname rhel5u4 memory @@ -105,7 +105,7 @@ Domain:install_guest titleList data structure/title programlisting [ - [{'domain:install_guest': {'guestname': 'rhel5u4', 'memory': '1024', 'vcpu': '1'}}] + [{'domain:install_domain': {'guestname': 'rhel5u4', 'memory': '1024', 'vcpu': '1'}}] I'm not sure about this. The thing we're creating is a domain, but we're installing a guest, am I right? (thus the guestname and later guestip and so on. But I could be wrong. Also even if I'm not, the question is how much we want to separate this in order to keep it readable. I'm not quite sure about it either, but I would like to post the patch so there could be discussion, IIRC, there was discussion about the TERM domain and guest before too. Anyway, there should be guidelines for it, mixing use of both of them at liberty can just produce messy codes. I feel 'domain' should be a xen terminology such as domain 0(dom0) and domain U(domU), and domU is a user domains, as usual, called a guest in kvm/qemu, however, it's not very clear such as some guys say xen guest domain :-( Osier -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [test-API 07/17] Substitute _guest with _domain
On 04/20/2012 11:16 AM, Alex Jia wrote: On 04/20/2012 04:29 PM, Osier Yang wrote: On 2012年04月20日 16:17, Martin Kletzander wrote: On 04/20/2012 08:46 AM, Osier Yang wrote: --- .../en-US/Creating_a_configuration_file.xml| 82 ++-- .../en-US/Writing_a_test_case.xml | 24 +++--- .../libvirt-test-API_Guide/en-US/extras/log.txt| 18 ++-- repos/domain/domain_list.py| 12 ++-- repos/domain/install_linux_cdrom.py| 18 ++-- repos/domain/install_linux_net.py |8 +- repos/domain/install_windows_cdrom.py |6 +- repos/domain/update_devflag.py |6 +- 8 files changed, 87 insertions(+), 87 deletions(-) diff --git a/docs/User_Guide/libvirt-test-API_Guide/en-US/Creating_a_configuration_file.xml b/docs/User_Guide/libvirt-test-API_Guide/en-US/Creating_a_configuration_file.xml index 80ebdb1..ffac187 100644 --- a/docs/User_Guide/libvirt-test-API_Guide/en-US/Creating_a_configuration_file.xml +++ b/docs/User_Guide/libvirt-test-API_Guide/en-US/Creating_a_configuration_file.xml @@ -90,7 +90,7 @@ module:test_case example titleConfiguration file/title programlisting -Domain:install_guest +Domain:install_domain guestname rhel5u4 memory @@ -105,7 +105,7 @@ Domain:install_guest titleList data structure/title programlisting [ - [{'domain:install_guest': {'guestname': 'rhel5u4', 'memory': '1024', 'vcpu': '1'}}] + [{'domain:install_domain': {'guestname': 'rhel5u4', 'memory': '1024', 'vcpu': '1'}}] I'm not sure about this. The thing we're creating is a domain, but we're installing a guest, am I right? (thus the guestname and later guestip and so on. But I could be wrong. Also even if I'm not, the question is how much we want to separate this in order to keep it readable. I'm not quite sure about it either, but I would like to post the patch so there could be discussion, IIRC, there was discussion about the TERM domain and guest before too. Anyway, there should be guidelines for it, mixing use of both of them at liberty can just produce messy codes. I feel 'domain' should be a xen terminology such as domain 0(dom0) and domain U(domU), and domU is a user domains, as usual, called a guest in kvm/qemu, however, it's not very clear such as some guys say xen guest domain :-( I feel it a little bit different. I know I'm in no position to have a firm stance on this subject, but let me explain what I mean. From my point of view, domain is a (virtual) machine. The system running inside the VM is guest. However, there are things that might have connection with both. This is clearly the where do we draw the line kind of case. I think HW-related terms could be called 'domain' (e.g. MAC address), which would make the SW-related called 'guest'. When it sounds good, there is no problem with calling something 'guest domain', but just when it feels right (and that's probably just when it means domain, as it is the subjective noun in this case, whether guest is just an adjective). I got caught in a moment and went a little deep, I know, sorry for that. Just my $0.02 Martin -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list