gentoo-add_ip.sh: fix scan for IP addresses existing in /etc/conf.d/net/etc/vz/dists/scripts/gentoo-add_ip.sh of vzctl v3.0.16 has a stupid but rather nasty bug.
short description:
when a function add_ip() looking for the ip(s) to be assigned to vps it use a simple grep for each of the ip(s) in conf.d/net. btw gentoo conf.d/net is some more tricky then just an ip storage, so any presence of ip in this file will breaks a logic thus an ip will not be added to conf.d...
arpsend: proper fix for VLAN interfacesip command lists VLAN interfaces in form of ethX.Y@ethX, but arpsend
(and other utilities) expects only ethX.Y so remove the part of VLAN
interface name starting from @.
The previous fix (commit 4db38b551a44019a4fced9abf23bd6e1531cef60) was
wrong -- we do not need to exclude VLAN interfaces, we need to process those
properly.
http://bugzilla.openvz.org/show_bug.cgi?id=617
arpsend: exclude VLAN interfacesremove vlan interfaces like eth0.5@eth0 from the list of interfaces
to use arpsend on.
http://bugzilla.openvz.org/show_bug.cgi?id=617
make install: add a hook with a hint to run distro-specific installmake install now tells the user to run dist-specific install
(like make install-redhat).
vzlist.c: fix compiler warning and a minor errorAdded needed include to get rid of the following warning:
vzlist.c: In function ‘invert_ip’:
vzlist.c:1084: warning: implicit declaration of function ‘inet_ntop’
Amazingly after a fix a new warning appears:
vzlist.c: In function ‘invert_ip’:
vzlist.c:1085: warning: comparison between pointer and integer
That reveals that the check for inet_ntop error return is wrong.
Fix this as well.
configure: change default for sysconfdir to /etcOtherwise some important stuff (initscript, files for logrotate, cron, udev)
will be installed to some nonsense place like /usr/local/etc/ etc.).
We do not want to ask user to supply --sysconfdir=/etc on every ./configure
invocation thus this (somewhat controversial) patch. If you have better idea,
please tell us :)
Revert "Workaround: Export VEID from /proc/vz/veth in VZNETCFG script."This reverts commit e2db06ad9508eece3b455c0791a4753b9627adc1,
since the bug is really fixed in f7b35d8f2ac9680d6da794a72e71c51496c15a2b.
Convert ipv6 address to short form. IPv6 with aaaa:bbbb:cccc:dddd:0:0:0:xxxx is same as aaaa:bbbb:cccc:dddd::xxxx but the ip(route) command return only the short one, so all grep check fails. Also use short form in vzlist.http://bugzilla.openvz.org/show_bug.cgi?id=590
etc/init.d/vz-redhat: replace short ip cmd args with full onessome "ip"-cmd are use in full other only short form, like:
ip addr list <-> ip a l
Use long commands everywhere.
http://bugzilla.openvz.org/593
vps-functions: cosmetic cleanups* brackets are missing after vzaddrouting4/6 function names
* proper indentation in vzdelrouting()
http://bugzilla.openvz.org/show_bug.cgi?id=596
fix bashism conditionals in vps-net_addbashism conditionals, like
if [ "txt" == "txt" ]; do
should be changed to :
if [ "txt" = "txt" ]; do
http://bugzilla.openvz.org/594
Remove extra space in logging vpsnetclea/vpsreboot messages/var/log/vzctl.log :
[...]
<date> vzctl : VE <VEID> : VE start in progress...
<date> vereboot : VE <VEID> : reboot
[...]
http://bugzilla.openvz.org/597
[IPv6] add ip6tables kernel modules in VE0, if IPV6 is enabledIf IPV6 is set to "yes", load a minimal set of ip6tables modules on VE0,
so a VE can use cmd ip6tables.
http://bugzilla.openvz.org/598
etc/init.d/vz-redhat: add processing of multiple cron filesIdea and the first patch from Thorsten Schifferdecker <curx@openvz.org>:
> enable crontabs related to OpenVZ, like checking /proc/user_beancouters or
> /proc/bc/*/ or backuping VE's.
> Files /etc/vz/cron/vz* build a general vz-crontab genvz, installed to
> /etc/cron.d/vz.
>
> i.e: checks /proc/user_beancouters with monbean.pl
>
> - add a crontab file to /etc/vz/cron/vzmonbean, with:
>
> # enabl...
etc/init.d/vz-redhat: add update-cronOriginally from Thorsten Schifferdecker <curx@openvz.org>:
> Since vz-init-script handles the cron script, it's impossilbe reload
> crontabs without restart in the whole vz system, so with
> /etc/init.d/vz update-cron this can be done.
Patch reworked to do more general thing, i.e. either install or remove
of cron entries, depending on whether openvz is running or not.
Silent is_running() func...
etc/init.d/vz-redhat: remove bashisms: for (( ... )) and let var++remove bashism :
"for ((i = 0; i <= 2; i++)); do" -> "for i in `seq 0 2`; do"
"let var++" -> "var=$(($var+1))"
[kir@openvz.org: change `seq 0 2` to just 0 1 2; seq is GNU coreutils specific]
configure: check for either ppc64 or powerpc64Christian Kaiser reports that on his PPC box build_cpu is set to 'powerpc64'.
On the machine I used it was ppc64. So we should check for both.
See http://forum.openvz.org/index.php?t=tree&goto=13241&#msg_13241
vzcfgvalidate: check/fix dcachesize before kmemsizeThe sane value of kmemsize parameter depends on dcachesize,
so new value for dcachesize should be calculated before kmemsize
to avoid a need to run vzcfgvalidate twice.
http://bugzilla.openvz.org/518
vzctl: delete IPv6 route from host system after VE stopBefore this patch, the following problem occured:
VE assigned IP(v6) fc00::1, after stopping it, the VE IP(v6) route exist
fc00::1 dev venet0 metric 1024 expires 21333737sec mtu 1500 advmss 1440 hoplimit 4294967295
http://bugzilla.openvz.org/587
vzcpucheck: correct a typovzcpucheck node power has a minor inconsistency because the following line has
a typo:
if ($1 == "bogomips" || $1 = "BogoMIPS") {
of course, it should be:
if ($1 == "bogomips" || $1 == "BogoMIPS") {
http://bugzilla.openvz.org/540
[kir@openvz.org: not a bug since all modern kernels print it in lowercase]
vzctl: fix removing IPv6 address from a VEWithout this patch, IPv6 address is not removed from a VE:
# vzctl set <VEID> --ipdel fc00::1 --save
Warning: ipv6 support disabled <---*
Deleting IP address(es):
Saved parameters for VE <VEID>
Thanks to Thorsten Schifferdecker for reporting the issue.
[kir@openvz.org: changelog, porting to latest git]
vzctl: don't store garbage in store_name()/PARAM_NAMEcase PARAM_NAME: branch in store_name() adds uninitialized buffer from
stack to config. But it doesn't have to: conf_store_str() one line above
already did everything.
Closes http://bugzilla.openvz.org/show_bug.cgi?id=579
Signed-off-by: Alexey Dobriyan <adobriyan@openvz.org>
Acked-by: Kir Kolyshkin <kir@openvz.org
Grant device access even if device file is a symlink.Fix for the following bug:
dionysos ~ # vzctl start 333
The /dev/rtc is not block or character device
Invalid value for DEVNODES=rtc:r, skipped
Starting VE ...
dionysos ~ # ls -l /dev/rtc
lrwxrwxrwx 1 root root 8 May 14 13:40 /dev/rtc -> misc/rtc
dionysos ~ # ls -l /dev/misc/rtc
crw-rw-r-- 1 root root 10, 135 May 14 13:40 /dev/misc/rtc
http://bugzilla.openvz.org/575
correct ipadd handling in debian_ip_add.shthe debian_add-ip script execute the bash-function "setup_network", which
override all network/interface settings, on adding a new ip.
http://bugzilla.openvz.org/547
http://bugzilla.openvz.org/564
init.d/vz: fix check_kernel() functionAfter commit 682aab717a7758f2e316976490cd3e8f7ec3feb2
[..]
if ! [ test -d /proc/vz ]; then
[...]
-> bash error: [: -d: binary operator expected
Can be : (prefered)
if ! test -d /proc/vz ; then
or:
if [ ! -d /proc/vz ]; then
init.d/vz-redhat: fix kernel config checkOpenVZ will not start anymore, when used KERNEL is enabled with IKCONFIG and
IKCONFIG_PROC (-> /proc/config.gz).
Kir Kolyshkin
Thorsten Schifferdecker
Igor Sukhih
