On Mon, Jul 30, 2012 at 3:18 PM, Ben Pfaff <b...@nicira.com> wrote: > Here's a series that takes the "bug 681880" series from last week > a step further, both fixing a bug that Bastian Blank pointed out > (regarding compaction) and making the implementation safer. > > Ansis already reviewed the first three patches. They are already > uploaded to Debian but I haven't committed them to any branch yet, > so here they are again for completeness: > lockfile: Fix hang locking through a dangling symlink. > ovsdb: Make "ovsdb-tool create" work through a dangling symlink. > debian: Move database from /etc/openvswitch to /var/lib/openvswitch. > > The remaining patches need review: > Make the location of the database separately configurable. > debian: Configure Debian packages to use /var/lib/openvswitch for > conf.db. > tests: Slightly generalize utility function tests. > util: New function follow_symlinks(). > lockfile: Be more forgiving about lockfiles for symlinks. > ovsdb: Do not replace symlinks by regular files during compaction. > > Makefile.am | 1 + > REPORTING-BUGS | 2 +- > configure.ac | 1 + > debian/automake.mk | 1 + > debian/openvswitch-switch.dirs | 1 + > debian/openvswitch-switch.postinst | 15 +++ > debian/openvswitch-switch.postrm | 4 +- > debian/openvswitch-switch.prerm | 50 ++++++++++ > debian/rules | 4 +- > lib/automake.mk | 2 + > lib/dirs.c.in | 9 ++- > lib/dirs.h | 3 +- > lib/lockfile.c | 79 +++++++-------- > lib/util.c | 85 > +++++++++++++++++ > lib/util.h | 3 + > m4/openvswitch.m4 | 10 ++ > ovsdb/file.c | 7 +- > ovsdb/log.c | 13 ++- > ovsdb/ovsdb-server.1.in | 2 +- > ovsdb/ovsdb-server.c | 2 +- > ovsdb/ovsdb-tool.1.in | 2 +- > ovsdb/ovsdb-tool.c | 24 +++-- > python/automake.mk | 1 + > python/ovs/dirs.py | 1 + > tests/file_name.at | 97 > +++++++++++++++++++ > tests/library.at | 15 ++- > tests/lockfile.at | 2 + > tests/ofproto-macros.at | 3 +- > tests/ovs-monitor-ipsec.at | 1 + > tests/ovs-xapi-sync.at | 1 + > tests/ovsdb-server.at | 15 +++- > tests/ovsdb-tool.at | 17 +++- > tests/test-lockfile.c | 68 +++++++++++++- > tests/test-util.c | 99 > +++++++++++++------ > tests/unixctl-py.at | 2 + > tests/vlog.at | 4 + > utilities/bugtool/ovs-bugtool.in | 4 +- > .../bugtool/plugins/system-logs/openvswitch.xml | 3 +- > utilities/ovs-ctl.8 | 3 +- > utilities/ovs-ctl.in | 7 +- > utilities/ovs-lib.in | 2 + > 41 files changed, 554 insertions(+), 111 deletions(-) > create mode 100755 debian/openvswitch-switch.prerm > > -- > 1.7.2.5 > > _______________________________________________ > dev mailing list > dev@openvswitch.org > http://openvswitch.org/mailman/listinfo/dev > I will review these series.
Thanks Ansis
_______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
