{"push_id":1299791041,"size":16,"distinct_size":1,"ref":"refs/heads/vhdx","head":"018ec0726da0c7b0c2263f0eac3bc30c9bdaa1c6","before":"25c85ce0b238a30241d580ee8cac557a91ab1eac","commits":[{"sha":"419bc8cf65ff38d9074224b3539696ee7d17d3d4","author":{"name":"Erik Skultety","email":"98031ce1a99dbd0d2aa0a4d790e6d42553acae27@redhat.com"},"message":"tests: fix incorrect status handling by virsh-self-test\n\nThe virsh-self-test script compared the test's return code with 1 and only if\nthe return code matched this value then the test was marked as failed. Problem\nis that SIGSEGV returns 139 (or 11 to be precise, since shell reserves the MSB\nfor abnormal exit signaling) which passes the check just fine and test then\nappears as successful which it most certainly wasn't.\nTherefore, flip the logic to compare against 0 instead and every other result\nwill be treated as a failed test case.\n\nSigned-off-by: Erik Skultety <eskultet@redhat.com>","distinct":false,"url":"https://api.github.com/repos/nertpinx/libvirt/commits/419bc8cf65ff38d9074224b3539696ee7d17d3d4"},{"sha":"d53fa838e18d8c192296487ff40c3a37420b6d06","author":{"name":"Michal Privoznik","email":"ff52d8b477e4489d0b3b856d550bc61214c6ad4b@redhat.com"},"message":"configure: Check for major() more strictly\n\nThing is, in f3f15cc24 I'm trying to adapt libvirt to the newest\nglibc where major()/minor()/makedev() are moved from sys/types.h\nto sys/sysmacros.h. However, my commit back then expect autoconf\nto be fixed too as we already use AC_HEADER_MAJOR to determine\nwhich header file the functions are in, but because the header\nfiles just trigger a warning and not a compile error, the\nautoconf macro detects the bad header file.\n\nThis is just a workaround until autoconf macro is fixed.\n\nSigned-off-by: Michal Privoznik <mprivozn@redhat.com>","distinct":false,"url":"https://api.github.com/repos/nertpinx/libvirt/commits/d53fa838e18d8c192296487ff40c3a37420b6d06"},{"sha":"5555dc0d7fe0267e2ff6e5a9625164f2896f9cc5","author":{"name":"Peter Krempa","email":"ad550f3a498f24c1804da389934cf0cf16278373@redhat.com"},"message":"util: numa: Remove impossible error handling\n\nThe code guarantees that virBitmapSetBit won't be called with out of\nrange values. Just ignore the return value and remove dead error\nhandling.","distinct":false,"url":"https://api.github.com/repos/nertpinx/libvirt/commits/5555dc0d7fe0267e2ff6e5a9625164f2896f9cc5"},{"sha":"77cb01bc0fec4d0da02e1d4df75d28870b0ef926","author":{"name":"Peter Krempa","email":"ad550f3a498f24c1804da389934cf0cf16278373@redhat.com"},"message":"numa: Rename virNumaGetHostNodeset and make it return only nodes with memory\n\nName it virNumaGetHostMemoryNodeset and return only NUMA nodes which\nhave memory installed. This is necessary as the kernel is not very happy\nto set the memory cgroup setting for nodes which do not have any memory.\n\nThis would break vcpu hotplug with following message on such\nconfigruation:\n\n  Invalid value '0,8' for 'cpuset.mems': Invalid argument\n\nResolves: https://bugzilla.redhat.com/show_bug.cgi?id=1375268","distinct":false,"url":"https://api.github.com/repos/nertpinx/libvirt/commits/77cb01bc0fec4d0da02e1d4df75d28870b0ef926"},{"sha":"e043ecc82d161028c04061da71597b2127651c77","author":{"name":"Daniel P. Berrange","email":"5d1320ff3c85681bb6d7ea4b0b2f371698325ec8@redhat.com"},"message":"tests: use a fixed chardev TLS path\n\nThe test qemuxml2argv-serial-tcp-tlsx509-chardev.args\nwill fail if libvirt is built with a --sysconfdir\narg that is not /etc.  Fix this by setting a hardcoded\npath in the test code.\n\nSigned-off-by: Daniel P. Berrange <berrange@redhat.com>","distinct":false,"url":"https://api.github.com/repos/nertpinx/libvirt/commits/e043ecc82d161028c04061da71597b2127651c77"},{"sha":"5d213b34de442490c66ee54b17d15e68cfeb2174","author":{"name":"Michal Privoznik","email":"ff52d8b477e4489d0b3b856d550bc61214c6ad4b@redhat.com"},"message":"qemuDomainGetBlockJobInfo: Move info translation into separate func\n\nEven though we merely just pass to users whatever qemu provided\non the monitor, we still do some translation. For instance we\nturn bytes into mebibytes, or fix job type if needed. However, in\nthe future there is more fixing to be done so this code deserves\nits own function.\n\nSigned-off-by: Michal Privoznik <mprivozn@redhat.com>","distinct":false,"url":"https://api.github.com/repos/nertpinx/libvirt/commits/5d213b34de442490c66ee54b17d15e68cfeb2174"},{"sha":"988218ca3f0de571d97d02365db9ffa0845c18da","author":{"name":"Michal Privoznik","email":"ff52d8b477e4489d0b3b856d550bc61214c6ad4b@redhat.com"},"message":"virDomainGetBlockJobInfo: Fix corner case when qemu reports no info\n\nhttps://bugzilla.redhat.com/show_bug.cgi?id=1372613\n\nApparently, some management applications use the following code\npattern when waiting for a block job to finish:\n\n  while (1) {\n    virDomainGetBlockJobInfo(dom, disk, info, flags);\n\n    if (info.cur == info.end)\n        break;\n\n    sleep(1);\n  }\n\nProblem with this approach is in its corner cases. In case of\nQEMU, libvirt merely pass what has been reported on the monitor.\nHowever, if the block job hasn't started yet, qemu reports cur ==\nend == 0 which tricks mgmt apps into thinking job is complete.\n\nThe solution is to mangle cur/end values as described here [1].\n\n1: https://www.redhat.com/archives/libvir-list/2016-September/msg00017.html\n\nSigned-off-by: Michal Privoznik <mprivozn@redhat.com>","distinct":false,"url":"https://api.github.com/repos/nertpinx/libvirt/commits/988218ca3f0de571d97d02365db9ffa0845c18da"},{"sha":"03376b6da0c1e1774513048b6c992d9836600aac","author":{"name":"Peter Krempa","email":"ad550f3a498f24c1804da389934cf0cf16278373@redhat.com"},"message":"qemu: monitor: Use a more obvious iterator name\n\nThe algorithm that matches data from query-cpus and\nquery-hotpluggable-cpus is quite complex. Start using descriptive\niterator names to avoid confusion.","distinct":false,"url":"https://api.github.com/repos/nertpinx/libvirt/commits/03376b6da0c1e1774513048b6c992d9836600aac"},{"sha":"66da0356cd62398b1e06de317458e8883cb32db6","author":{"name":"Peter Krempa","email":"ad550f3a498f24c1804da389934cf0cf16278373@redhat.com"},"message":"qemu: monitor: qemuMonitorGetCPUInfoHotplug: Add iterator 'anycpu'\n\nAdd separate iterator for iterating all the entries","distinct":false,"url":"https://api.github.com/repos/nertpinx/libvirt/commits/66da0356cd62398b1e06de317458e8883cb32db6"},{"sha":"2a0e68be9185eecb9207ebdfdc4f8f0933f38bb3","author":{"name":"Peter Krempa","email":"ad550f3a498f24c1804da389934cf0cf16278373@redhat.com"},"message":"qemu: monitor: Add vcpu state information to monitor data\n\nReturn whether a vcpu entry is hotpluggable or online so that upper\nlayers don't have to infer the information from other data.\n\nAdvantage is that this code can be tested by unit tests.","distinct":false,"url":"https://api.github.com/repos/nertpinx/libvirt/commits/2a0e68be9185eecb9207ebdfdc4f8f0933f38bb3"},{"sha":"64bc75f75606d0cc48432729b4618e2eae96accc","author":{"name":"Peter Krempa","email":"ad550f3a498f24c1804da389934cf0cf16278373@redhat.com"},"message":"qemu: domain: Don't infer vcpu state\n\nUse the state information (online, hotpluggable) provided by the monitor\ncode rather than trying to infer it. This fixes an issue where on\narchitectures that require hotplug of multiple threads at once the\nsub-cores would get updated as offline on daemon restart thus creating\nan invalid configuration.\n\nResolves: https://bugzilla.redhat.com/show_bug.cgi?id=1375783","distinct":false,"url":"https://api.github.com/repos/nertpinx/libvirt/commits/64bc75f75606d0cc48432729b4618e2eae96accc"},{"sha":"d920090c72261a7b6face9e0ebd08ef72abb2913","author":{"name":"Michal Privoznik","email":"ff52d8b477e4489d0b3b856d550bc61214c6ad4b@redhat.com"},"message":"virsh: Move cmdSelfTest to vsh\n\nThis command should be exposed to other shells of ours.\nThey are gonna need it as soon as we want to test them too.\n\nSigned-off-by: Michal Privoznik <mprivozn@redhat.com>","distinct":false,"url":"https://api.github.com/repos/nertpinx/libvirt/commits/d920090c72261a7b6face9e0ebd08ef72abb2913"},{"sha":"4a457adda649447a7873f48bf71c5139bc6404d2","author":{"name":"Michal Privoznik","email":"ff52d8b477e4489d0b3b856d550bc61214c6ad4b@redhat.com"},"message":"tests: Self test virt-admin\n\nJust like we are running 'virsh self-test' from within our test\nsuite, we should run 'virt-admin self-test' too.\n\nSigned-off-by: Michal Privoznik <mprivozn@redhat.com>","distinct":false,"url":"https://api.github.com/repos/nertpinx/libvirt/commits/4a457adda649447a7873f48bf71c5139bc6404d2"},{"sha":"ead3cfc68db1f092bdda4374085a9e2addb3c2d8","author":{"name":"Jason Miesionczek","email":"d2b78f0c00cae42a6b0d197de5311542c1d63039@datto.com"},"message":"hyperv: additional server 2008 wmi classes","distinct":false,"url":"https://api.github.com/repos/nertpinx/libvirt/commits/ead3cfc68db1f092bdda4374085a9e2addb3c2d8"},{"sha":"8563560026d192c2cf047b550ffd468692245ed6","author":{"name":"Martin Kletzander","email":"b95c1c5986c2c30eff5096e3d1c8d4510b550570@redhat.com"},"message":"tests: Use fixedcontent variable\n\nCommit ca32929908bbc94116493ad8915e7cd7ae0f57d5 added function\nvirTestCompareToFile(), but forgot to use a fixedcontent value for the\nactual comparison.  That lead to VIR_TEST_DEBUG=1 showing (for some\ntests) all the actual output from the first error to the end of the\nstring due to the difference being an endline in the end.\n\nSigned-off-by: Martin Kletzander <mkletzan@redhat.com>","distinct":false,"url":"https://api.github.com/repos/nertpinx/libvirt/commits/8563560026d192c2cf047b550ffd468692245ed6"},{"sha":"018ec0726da0c7b0c2263f0eac3bc30c9bdaa1c6","author":{"name":"Martin Kletzander","email":"b95c1c5986c2c30eff5096e3d1c8d4510b550570@redhat.com"},"message":"asdf\n\nSigned-off-by: Martin Kletzander <mkletzan@redhat.com>","distinct":true,"url":"https://api.github.com/repos/nertpinx/libvirt/commits/018ec0726da0c7b0c2263f0eac3bc30c9bdaa1c6"}]}
{"actor":{"display_login":"nertpinx"}}