[libvirt] [PATCH v2 2/9] test: s/{in, out}put-data-admin-nomdns/{in, out}put-data-admin-nomdns-nonames
Erik Skultety
eskultet at redhat.com
Mon Aug 24 08:10:20 UTC 2015
On 21/08/15 23:45, Martin Kletzander wrote:
> On Fri, Aug 21, 2015 at 08:04:03PM +0200, Erik Skultety wrote:
>> This minor cosmetical change allows us to later add new test case data
>> including
>> named servers.
>> ---
>> ...nput-data-admin-nomdns.json =>
>> input-data-admin-nomdns-nonames.json} | 0
>> ...put-data-admin-nomdns.json =>
>> output-data-admin-nomdns-nonames.json} | 0
>> tests/virnetdaemontest.c |
>> 2 +-
>> 3 files changed, 1 insertion(+), 1 deletion(-)
>> rename tests/virnetdaemondata/{input-data-admin-nomdns.json =>
>> input-data-admin-nomdns-nonames.json} (100%)
>> rename tests/virnetdaemondata/{output-data-admin-nomdns.json =>
>> output-data-admin-nomdns-nonames.json} (100%)
>>
>
> There is no need to rename this, all the subsequent (new) test files
> can omit 'nomdns' from now on, for example 'admin-servers' if you add
> saving of some server data. If you really want this to be named this
> particular way, please use a normal name for the subject as this is
> not very easy to find or understand from e.g. oneline log.
No, I agree with you, it's unnecessary and I dropped the patch from my
branch.
Erik
More information about the libvir-list
mailing list