{"push_id":819577705,"size":10,"distinct_size":10,"ref":"refs/heads/master","head":"0bc5fcffb13e3ea9ab35aeb760d31660f4aea2c8","before":"62ff210e5d6542bda3f25ed5e96c5088a840ce1b","commits":[{"sha":"b866991f0c47c95a97ae4440eef99592b27bb8a8","author":{"email":"2cf5c04c61aa466e4a47bfedc747d17279c72ffc@redhat.com","name":"Peter Krempa"},"message":"qemu: migration: Split source and destination migration checks\n\nExtract the hostdev check from qemuMigrationIsAllowed into a separate\nfunction since that is the only part that needs to be done in the v2\nmigration protocol prepare phase on the destination. All other checks\nwere added when the v3 protocol existed so they don't need to be\nextracted.\n\nThis change will allow to drop the @def argument for\nqemuMigrationIsAllowed and further simplify the function.","distinct":true,"url":"https://api.github.com/repos/crobinso/libvirt/commits/b866991f0c47c95a97ae4440eef99592b27bb8a8"},{"sha":"f558c66f17e633ae912e339bd196fcd4861a61a8","author":{"email":"2cf5c04c61aa466e4a47bfedc747d17279c72ffc@redhat.com","name":"Peter Krempa"},"message":"qemu: migration: Drop @def from qemuMigrationIsAllowed\n\nNow that qemuMigrationIsAllowed is always called with @vm, we can drop\nthe @def argument and simplify the control flow.\n\nAdditionally the comment is invalid so drop it.","distinct":true,"url":"https://api.github.com/repos/crobinso/libvirt/commits/f558c66f17e633ae912e339bd196fcd4861a61a8"},{"sha":"b6c44af0f7edb9eec3d3ba2dd1a423f17bf0e633","author":{"email":"2cf5c04c61aa466e4a47bfedc747d17279c72ffc@redhat.com","name":"Peter Krempa"},"message":"qemu: migration: Use migration flags in qemuMigrationIsAllowed\n\nUse the migration @flags for checking various migration aspects rather\nthan picking them out as booleans. Document the new semantics in the\nfunction header.","distinct":true,"url":"https://api.github.com/repos/crobinso/libvirt/commits/b6c44af0f7edb9eec3d3ba2dd1a423f17bf0e633"},{"sha":"01b4baba59db2478ad260ebc61693a2325229f5b","author":{"email":"2cf5c04c61aa466e4a47bfedc747d17279c72ffc@redhat.com","name":"Peter Krempa"},"message":"qemu: migration: Skip few checks while doing offline migration\n\nqemuMigrationIsAllowed would disallow offline migration if the VM\ncontained host devices or memory modules. Since during offline migration\nwe don't transfer any state we can safely migrate VMs with such\nconfiguration.\n\nResolves: https://bugzilla.redhat.com/show_bug.cgi?id=1265049","distinct":true,"url":"https://api.github.com/repos/crobinso/libvirt/commits/01b4baba59db2478ad260ebc61693a2325229f5b"},{"sha":"b527aa090db303caf1a16017b2810a3377f0da27","author":{"email":"3ada0bee826c753786fdbba72243dfba997094cf@redhat.com","name":"Andrea Bolognani"},"message":"qemu: Remove explicit values from virQEMUCapsFlags\n\nThis gets rid of the partially enforced alignment and makes it less\nlikely for a bogus value to be introduced in the enumeration.\nCapabilities are divided in five-element groups for better readability.\n\nUse #define for QEMU_CAPS_NET_NAME and QEMU_CAPS_HOST_NET_ADD, both\nof which are aliases for QEMU_CAPS_0_10.","distinct":true,"url":"https://api.github.com/repos/crobinso/libvirt/commits/b527aa090db303caf1a16017b2810a3377f0da27"},{"sha":"7cdf944b0af529919f9dc3f26e81f93ba67b3459","author":{"email":"3ada0bee826c753786fdbba72243dfba997094cf@redhat.com","name":"Andrea Bolognani"},"message":"qemu: Fix indentation issue\n\nTwo #define lines introduced by b527aa0 did not respect the\nindentation rules, thus breaking syntax-check.","distinct":true,"url":"https://api.github.com/repos/crobinso/libvirt/commits/7cdf944b0af529919f9dc3f26e81f93ba67b3459"},{"sha":"a946c50ded37bccab831c75d2969ee82cc8c4ad5","author":{"email":"7cd45ff6ad1fb7cf556a02797d774247851499f8@redhat.com","name":"Cole Robinson"},"message":"tests: eventtest: Fix coverity warning\n\nWe can ignore the result of virtTestResult here, because failure is\nunconditionally reported by the callers","distinct":true,"url":"https://api.github.com/repos/crobinso/libvirt/commits/a946c50ded37bccab831c75d2969ee82cc8c4ad5"},{"sha":"79bd55b3024a5d8695880b2a992b8a365bbb5b39","author":{"email":"83d82aaba2eed257f4814b0c239c260c4caaadf0@redhat.com","name":"Michal Privoznik"},"message":"virSecurityManagerNew: Turn array of booleans into flags\n\nSo imagine you want to crate new security manager:\n\n  if (!(mgr = virSecurityManagerNew(\"selinux\", \"QEMU\", false, true, false, true)));\n\nHard to parse, right? What about this:\n\n  if (!(mgr = virSecurityManagerNew(\"selinux\", \"QEMU\",\n                                    VIR_SECURITY_MANAGER_DEFAULT_CONFINED |\n                                    VIR_SECURITY_MANAGER_PRIVILEGED)));\n\nNow that's better! This is what the commit does.\n\nSigned-off-by: Michal Privoznik <mprivozn@redhat.com>","distinct":true,"url":"https://api.github.com/repos/crobinso/libvirt/commits/79bd55b3024a5d8695880b2a992b8a365bbb5b39"},{"sha":"f84c6dd26def63304e085c40bdd7a42373139cbb","author":{"email":"87558058f6f829e5ec976c8ef960720af4ff9c7d@redhat.com","name":"John Ferlan"},"message":"libxl: Resolve Coverity FORWARD_NULL\n\nSince the strtok_r call in libxlCapsInitGuests expects a non NULL first\nparameter when the third parameter is NULL, we need to check that\nthe returned 'capabilities' from a libxl_get_version_info call is\nnot NULL and error out if so since the code expects it.\n\nSigned-off-by: John Ferlan <jferlan@redhat.com>","distinct":true,"url":"https://api.github.com/repos/crobinso/libvirt/commits/f84c6dd26def63304e085c40bdd7a42373139cbb"},{"sha":"0bc5fcffb13e3ea9ab35aeb760d31660f4aea2c8","author":{"email":"87558058f6f829e5ec976c8ef960720af4ff9c7d@redhat.com","name":"John Ferlan"},"message":"qemu: Resolve Coverity FORWARD_NULL\n\nCoverity notices that net->ifname is potentially referenced after a\nVIR_FREE().  Since the net->ifname will eventually be free'd during\nvirDomainDefFree when calling virDomainNetDefFree, let's just that\nprocessing take care the free.\n\nSigned-off-by: John Ferlan <jferlan@redhat.com>","distinct":true,"url":"https://api.github.com/repos/crobinso/libvirt/commits/0bc5fcffb13e3ea9ab35aeb760d31660f4aea2c8"}]}